@tscircuit/parts-engine 0.0.10 → 0.0.12

package/lib/jlc-parts-engine.ts

@@ -1,7 +1,7 @@
 import type { PartsEngine, SupplierPartNumbers } from "@tscircuit/props"
 import { getJlcpcbPackageName } from "./footprint-translators/index"
 
-const cache = new Map<string, any>()
+export const cache = new Map<string, any>()
 
 const getJlcPartsCached = async (name: any, params: any) => {
   const paramString = new URLSearchParams({
@@ -19,6 +19,13 @@ const getJlcPartsCached = async (name: any, params: any) => {
   return responseJson
 }
 
+const withBasicPartPreference = (parts: any[] | undefined) => {
+  if (!parts) return []
+  return [...parts].sort(
+    (a, b) => Number(b.is_basic ?? false) - Number(a.is_basic ?? false),
+  )
+}
+
 export const jlcPartsEngine: PartsEngine = {
   findPart: async ({
     sourceComponent,
@@ -36,7 +43,9 @@ export const jlcPartsEngine: PartsEngine = {
       })
 
       return {
-        jlcpcb: (resistors ?? []).map((r: any) => `C${r.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(resistors)
+          .map((r: any) => `C${r.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -48,7 +57,9 @@ export const jlcPartsEngine: PartsEngine = {
       })
 
       return {
-        jlcpcb: (capacitors ?? []).map((c: any) => `C${c.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(capacitors)
+          .map((c: any) => `C${c.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -72,7 +83,9 @@ export const jlcPartsEngine: PartsEngine = {
             },
       )
       return {
-        jlcpcb: (headers ?? []).map((h: any) => `C${h.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(headers)
+          .map((h: any) => `C${h.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -83,7 +96,7 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (potentiometers ?? [])
+        jlcpcb: withBasicPartPreference(potentiometers)
           .map((p: any) => `C${p.lcsc}`)
           .slice(0, 3),
       }
@@ -95,17 +108,24 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (diodes ?? []).map((d: any) => `C${d.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(diodes)
+          .map((d: any) => `C${d.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
       sourceComponent.ftype === "simple_chip"
     ) {
+      if (!jlcpcbPackage || !footprinterString) {
+        return {}
+      }
       const { chips } = await getJlcPartsCached("chips", {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (chips ?? []).map((c: any) => `C${c.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(chips)
+          .map((c: any) => `C${c.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -116,7 +136,9 @@ export const jlcPartsEngine: PartsEngine = {
         transistor_type: sourceComponent.transistor_type,
       })
       return {
-        jlcpcb: (transistors ?? []).map((t: any) => `C${t.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(transistors)
+          .map((t: any) => `C${t.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -127,7 +149,9 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (power_sources ?? []).map((p: any) => `C${p.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(power_sources)
+          .map((p: any) => `C${p.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -138,7 +162,9 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (inductors ?? []).map((i: any) => `C${i.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(inductors)
+          .map((i: any) => `C${i.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -150,7 +176,9 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (crystals ?? []).map((c: any) => `C${c.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(crystals)
+          .map((c: any) => `C${c.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -162,7 +190,9 @@ export const jlcPartsEngine: PartsEngine = {
         channel_type: sourceComponent.channel_type,
       })
       return {
-        jlcpcb: (mosfets ?? []).map((m: any) => `C${m.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(mosfets)
+          .map((m: any) => `C${m.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -173,7 +203,9 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (resonators ?? []).map((r: any) => `C${r.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(resonators)
+          .map((r: any) => `C${r.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -184,7 +216,9 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (switches ?? []).map((s: any) => `C${s.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(switches)
+          .map((s: any) => `C${s.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -194,7 +228,9 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (leds ?? []).map((l: any) => `C${l.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(leds)
+          .map((l: any) => `C${l.lcsc}`)
+          .slice(0, 3),
       }
     } else if (
       sourceComponent.type === "source_component" &&
@@ -204,7 +240,9 @@ export const jlcPartsEngine: PartsEngine = {
         package: jlcpcbPackage,
       })
       return {
-        jlcpcb: (fuses ?? []).map((l: any) => `C${l.lcsc}`).slice(0, 3),
+        jlcpcb: withBasicPartPreference(fuses)
+          .map((l: any) => `C${l.lcsc}`)
+          .slice(0, 3),
       }
     }
     return {}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tscircuit/parts-engine",
-  "version": "0.0.10",
+  "version": "0.0.12",
   "type": "module",
   "main": "./dist/index.js",
   "devDependencies": {

package/tests/jlc-parts-engine.test.ts

@@ -1,8 +1,9 @@
 import { describe, test, expect, beforeEach } from "bun:test"
-import { jlcPartsEngine } from "../lib/jlc-parts-engine"
+import { jlcPartsEngine, cache } from "../lib/jlc-parts-engine"
 import type { AnySourceComponent } from "circuit-json"
 describe("jlcPartsEngine", () => {
   beforeEach(() => {
+    cache.clear()
     // Reset fetch fake between tests
     globalThis.fetch = (async (url: string) => {
       if (url.includes("/resistors/")) {
@@ -431,4 +432,42 @@ describe("jlcPartsEngine", () => {
 
     expect(result).toEqual({ jlcpcb: [] })
   })
+
+  test("should prefer basic parts when available", async () => {
+    // Override the global fetch mock for this specific test
+    globalThis.fetch = (async (url: string) => {
+      if (url.includes("/resistors/")) {
+        return {
+          json: async () => ({
+            resistors: [
+              { lcsc: "1111" }, // is_basic is undefined, treated as false
+              { lcsc: "2222", is_basic: true },
+              { lcsc: "3333", is_basic: false },
+              { lcsc: "4444", is_basic: true },
+              { lcsc: "5555" },
+            ],
+          }),
+        } as Response
+      }
+      return {} as Response
+    }) as unknown as typeof fetch
+
+    const resistor: AnySourceComponent = {
+      type: "source_component",
+      ftype: "simple_resistor",
+      resistance: 10000,
+      source_component_id: "source_component_0",
+      name: "R1",
+    }
+
+    const result = await jlcPartsEngine.findPart({
+      sourceComponent: resistor,
+      footprinterString: "0603",
+    })
+
+    // Expecting basic parts to be prioritized
+    expect(result).toEqual({
+      jlcpcb: ["C2222", "C4444", "C1111"],
+    })
+  })
 })