@hybridlabor-api/bdb-hardware-pcb 0.1.0 → 0.1.2

package/config/mcp/antigravity.json

@@ -5,7 +5,7 @@
       "command": "__INSTALL_DIR__/scripts/run_kicad_mcp.sh",
       "args": [],
       "env": {
-        "KICAD_CLI_PATH": "/Applications/KiCad/KiCad.app/Contents/MacOS/kicad-cli"
+        "KICAD_CLI_PATH": "__KICAD_CLI_PATH__"
       }
     },
     "openscad": {

package/config/mcp/claude.json

@@ -5,7 +5,7 @@
       "command": "__INSTALL_DIR__/scripts/run_kicad_mcp.sh",
       "args": [],
       "env": {
-        "KICAD_CLI_PATH": "/Applications/KiCad/KiCad.app/Contents/MacOS/kicad-cli"
+        "KICAD_CLI_PATH": "__KICAD_CLI_PATH__"
       }
     },
     "openscad": {

package/config/mcp/codex.toml

@@ -14,7 +14,7 @@ args = []
 enabled = true
 
 [mcp_servers.kicad.env]
-KICAD_CLI_PATH = "/Applications/KiCad/KiCad.app/Contents/MacOS/kicad-cli"
+KICAD_CLI_PATH = "__KICAD_CLI_PATH__"
 
 [mcp_servers.openscad]
 command = "__INSTALL_DIR__/scripts/run_openscad_mcp.sh"

package/installer.js

@@ -25,6 +25,7 @@
 const os = require('os');
 const path = require('path');
 const fs = require('fs');
+const crypto = require('crypto');
 const { spawnSync } = require('child_process');
 
 const AUTO = process.argv.includes('--auto');
@@ -71,12 +72,71 @@ function findPython() {
     return null;
 }
 
+function findKicadCliPath() {
+    if (process.env.KICAD_CLI_PATH) return process.env.KICAD_CLI_PATH;
+    if (commandExists('kicad-cli')) return 'kicad-cli';
+    let candidates = [];
+    if (IS_WIN) {
+        const base = path.join(process.env.ProgramFiles || 'C:\\Program Files', 'KiCad');
+        if (fs.existsSync(base)) {
+            candidates = fs.readdirSync(base)
+                .map((v) => path.join(base, v, 'bin', 'kicad-cli.exe'))
+                .sort()
+                .reverse(); // prefer the highest version dir (e.g. 9.0 over 7.0)
+        }
+    } else if (process.platform === 'darwin') {
+        candidates = ['/Applications/KiCad/KiCad.app/Contents/MacOS/kicad-cli'];
+    } else {
+        candidates = ['/usr/bin/kicad-cli', '/usr/local/bin/kicad-cli'];
+    }
+    const found = candidates.find((p) => fs.existsSync(p));
+    if (found) return found;
+    log.warn('kicad-cli not found on PATH or in known install locations — set the KICAD_CLI_PATH env var manually once KiCad is installed.');
+    return 'kicad-cli';
+}
+
+// Hash of pyproject.toml, used as a cheap "did the dependency spec change"
+// marker. AOS's swap-in-place upgrade preserves .venv across a version bump
+// (that's the whole point of the PRESERVE list), so "the venv dir exists" is
+// NOT enough on its own to prove it's still current for THIS version -- the
+// hash comparison is what makes skipping safe across an upgrade that did
+// change deps, not just across an unattended re-run that didn't.
+function depsSpecHash(serverDir) {
+    const p = path.join(serverDir, 'pyproject.toml');
+    if (!fs.existsSync(p)) return null;
+    return crypto.createHash('sha256').update(fs.readFileSync(p)).digest('hex');
+}
+
 function setupServerEnv(server, uvBin, pythonBin) {
     const venvDir = path.join(server.dir, '.venv');
+    const markerPath = path.join(venvDir, '.aos-deps-hash');
+    const currentHash = depsSpecHash(server.dir);
+
+    // Every Quick Update used to re-run `uv sync` / `pip install` here
+    // unconditionally, even when downloadOrUpdateModule() (in the main AOS
+    // installer) had already determined this module's version hadn't
+    // changed at all -- real network calls and, on the pip fallback path
+    // with no uv on PATH, a genuinely slow full dependency resolve, every
+    // single run, for nothing.
+    if (currentHash && fs.existsSync(venvDir) && fs.existsSync(markerPath)) {
+        try {
+            if (fs.readFileSync(markerPath, 'utf8').trim() === currentHash) {
+                log.ok(`${server.name}-mcp-server: dependencies already up to date, skipping sync`);
+                return true;
+            }
+        } catch (e) { /* fall through to a real sync if the marker is unreadable */ }
+    }
+
+    const markDone = () => {
+        if (!currentHash) return;
+        try { fs.writeFileSync(markerPath, currentHash); } catch (e) { /* non-fatal: just means next run re-syncs once more */ }
+    };
+
     if (uvBin) {
         const res = spawnSync(uvBin, ['sync'], { cwd: server.dir, stdio: 'inherit' });
         if (res.status === 0) {
             log.ok(`${server.name}-mcp-server: dependencies synced via uv`);
+            markDone();
             return true;
         }
         log.warn(`${server.name}-mcp-server: 'uv sync' failed (exit ${res.status}); falling back to venv+pip`);
@@ -99,6 +159,7 @@ function setupServerEnv(server, uvBin, pythonBin) {
         return false;
     }
     log.ok(`${server.name}-mcp-server: dependencies installed via venv+pip`);
+    markDone();
     return true;
 }
 
@@ -166,7 +227,7 @@ function mcpServerEntries() {
         kicad: {
             command: kicad.command,
             args: kicad.args,
-            env: { KICAD_CLI_PATH: process.env.KICAD_CLI_PATH || '/Applications/KiCad/KiCad.app/Contents/MacOS/kicad-cli' },
+            env: { KICAD_CLI_PATH: findKicadCliPath() },
         },
         openscad: {
             command: openscad.command,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hybridlabor-api/bdb-hardware-pcb",
-  "version": "0.1.0",
+  "version": "0.1.2",
   "description": "BDB Hardware & PCB — KiCad + OpenSCAD MCP servers and AI-era skills for schematic capture, PCB layout/routing, DFM sign-off, and parametric 3D enclosure design",
   "main": "installer.js",
   "bin": {