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

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');
@@ -94,12 +95,48 @@ function findKicadCliPath() {
     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`);
@@ -122,6 +159,7 @@ function setupServerEnv(server, uvBin, pythonBin) {
         return false;
     }
     log.ok(`${server.name}-mcp-server: dependencies installed via venv+pip`);
+    markDone();
     return true;
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hybridlabor-api/bdb-hardware-pcb",
-  "version": "0.1.1",
+  "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": {