Merge pull request #609 from ShiboSoftwareDev/main

Add fabricationnotetext selector support in pcbSx

Author: ShiboSoftwareDev

generated/COMPONENT_TYPES.md

@@ -735,6 +735,7 @@ export interface PcbSxValue {
   fontSize?: string | number
   pcbX?: string | number
   pcbY?: string | number
+  visible?: boolean
 }
 export type PcbSx = PcbSxBase & {
   [K in PcbSxSelector]?: PcbSxValue
@@ -743,6 +744,7 @@ export const pcbSxValue = z.object({
   fontSize: length.optional(),
   pcbX: pcbCoordinate.optional(),
   pcbY: pcbCoordinate.optional(),
+  visible: z.boolean().optional(),
 })
 ```
 

generated/PROPS_OVERVIEW.md

@@ -1360,6 +1360,7 @@ export interface PcbSxValue {
   fontSize?: string | number
   pcbX?: string | number
   pcbY?: string | number
+  visible?: boolean
 }
 
 

lib/common/pcbSx.ts

@@ -6,11 +6,13 @@ import { z } from "zod"
 export type PcbSxSelector =
   | "& footprint[src^='kicad:'] silkscreentext"
   | "& silkscreentext"
+  | "& fabricationnotetext"
 
 export interface PcbSxValue {
   fontSize?: string | number
   pcbX?: string | number
   pcbY?: string | number
+  visible?: boolean
 }
 
 type PcbSxBase = Record<string, PcbSxValue>
@@ -23,6 +25,7 @@ export const pcbSxValue = z.object({
   fontSize: length.optional(),
   pcbX: pcbCoordinate.optional(),
   pcbY: pcbCoordinate.optional(),
+  visible: z.boolean().optional(),
 })
 
 export const pcbSx = z.record(

tests/fabrication-note.test.ts

@@ -7,6 +7,7 @@ import {
   fabricationNotePathProps,
   type FabricationNotePathProps,
 } from "lib/components/fabrication-note-path"
+import { fabricationNoteTextProps } from "lib/components/fabrication-note-text"
 
 test("fabrication note rect parses minimal props", () => {
   const rect: FabricationNoteRectProps = {
@@ -82,3 +83,16 @@ test("fabrication note path parses route points", () => {
   expect(parsed.strokeWidth).toBe(0.5)
   expect(parsed.color).toBe("#0000ff")
 })
+
+test("fabrication note text accepts pcbSx fabrication note text visibility", () => {
+  const parsed = fabricationNoteTextProps.parse({
+    text: "FAB NOTE",
+    pcbSx: {
+      "& fabricationnotetext": {
+        visible: true,
+      },
+    },
+  })
+
+  expect(parsed.pcbSx?.["& fabricationnotetext"]?.visible).toBe(true)
+})