Fix: Improve error handling for components with no PCB connection and missing footprint (#580)

* Fix: Improve error message for components with missing footprint or PCB connection

* fix: improve error message for missing footprint on components

* Adding test

* format

* Chore: fix test msg

* format

* add new test and if condition for footprint checking

* Add .toLowerCase()

Author: Ayushjhawar8

lib/components/primitive-components/Port/Port.ts

@@ -57,10 +57,18 @@ export class Port extends PrimitiveComponent<typeof portProps> {
 
   _getGlobalPcbPositionBeforeLayout(): { x: number; y: number } {
     const matchedPcbElm = this.matchedComponents.find((c) => c.isPcbPrimitive)
+    const parentComponent = this.parent
+
+    // First check if parent component has a footprint
+    if (parentComponent && !parentComponent.props.footprint) {
+      throw new Error(
+        `${parentComponent.componentName} "${parentComponent.props.name}" does not have a footprint. Add a footprint prop, e.g. <${parentComponent.componentName.toLowerCase()} footprint="..." />`,
+      )
+    }
 
     if (!matchedPcbElm) {
       throw new Error(
-        `Port ${this} has no matched pcb component, can't get global pcb position`,
+        `Port ${this} has no matching PCB primitives. This often means the footprint's pads lack matching port hints.`,
       )
     }
 

tests/components/normal-components/led-error-messages.test.tsx

@@ -0,0 +1,23 @@
+import { test, expect } from "bun:test"
+import { getTestFixture } from "../../../tests/fixtures/get-test-fixture"
+
+test("LED without footprint should throw appropriate error", async () => {
+  const { circuit } = getTestFixture()
+
+  circuit.add(
+    <board width="10mm" height="10mm">
+      <led name="LED1" />
+    </board>,
+  )
+
+  try {
+    circuit.render()
+  } catch (err) {
+    if (!(err instanceof Error)) {
+      throw new Error("Expected err to be an Error instance")
+    }
+    expect(err.message).toBe(
+      'Led "LED1" does not have a footprint. Add a footprint prop, e.g. <led footprint="..." />',
+    )
+  }
+})