@drawbridge/drawbridge-agents 0.1.6 → 0.1.8

package/bin/preflight-graphify.js

@@ -1,14 +1,22 @@
 #!/usr/bin/env node
-// Ensures the graphify runtime is installed (required for the family knowledge graph) and
-// registers its MCP server with the local assistant. Called by drawbridge-agents-sync.
-// Set DRAWBRIDGE_SKIP_GRAPHIFY=1 to skip in CI / headless environments.
+// Ensures the graphify runtime is available for the family knowledge graph. NON-FATAL by design:
+// if graphify can't be found or installed, warn and continue — `npm run sync` must never be
+// blocked by it (the drift hook nudges separately, and the graph is built on demand).
+// Set DRAWBRIDGE_SKIP_GRAPHIFY=1 to skip silently.
 const { execFileSync } = require('child_process')
+const os = require('os')
+const path = require('path')
 
 if (process.env.DRAWBRIDGE_SKIP_GRAPHIFY === '1') {
   console.log('drawbridge-agents-sync: DRAWBRIDGE_SKIP_GRAPHIFY set — skipping graphify preflight')
   process.exit(0)
 }
 
+// npm runs scripts under a minimal shell (sh), so tools installed by uv/pipx into ~/.local/bin
+// are often not on PATH. Add common user-bin dirs so an installed graphify is still found.
+const extra = [ path.join(os.homedir(), '.local', 'bin'), '/opt/homebrew/bin' ]
+process.env.PATH = [ ...extra, process.env.PATH || '' ].join(path.delimiter)
+
 const has = (cmd) => {
   try {
     execFileSync(cmd, [ '--version' ], { stdio: 'ignore' })
@@ -18,37 +26,30 @@ const has = (cmd) => {
   }
 }
 
-if (!has('graphify')) {
-  console.log('drawbridge-agents-sync: graphify runtime not found, attempting install...')
-  const attempts = [
-    [ 'uv', [ 'tool', 'install', 'graphifyy' ] ],
-    [ 'pipx', [ 'install', 'graphifyy' ] ],
-    [ 'pip', [ 'install', '--user', 'graphifyy' ] ],
-    [ 'pip3', [ 'install', '--user', 'graphifyy' ] ]
-  ]
-  for (const [ cmd, args ] of attempts) {
-    if (!has(cmd)) continue
-    try {
-      execFileSync(cmd, args, { stdio: 'inherit' })
-      break
-    } catch (error) {
-      // try the next installer
-    }
-  }
-  if (!has('graphify')) {
-    console.error([
-      'drawbridge-agents-sync: could not install the graphify runtime automatically.',
-      'Install it, then re-run npm run sync:',
-      '  uv tool install graphifyy   (recommended)',
-      '  pipx install graphifyy',
-      '  pip install --user graphifyy',
-      'Or set DRAWBRIDGE_SKIP_GRAPHIFY=1 to skip (CI / headless only).'
-    ].join('\n'))
-    process.exit(1)
+if (has('graphify')) {
+  console.log('drawbridge-agents-sync: graphify runtime ready')
+  process.exit(0)
+}
+
+// Best-effort install if a Python tool installer is available; never fail if none is.
+for (const [ cmd, args ] of [
+  [ 'uv', [ 'tool', 'install', 'graphifyy' ] ],
+  [ 'pipx', [ 'install', 'graphifyy' ] ],
+  [ 'pip', [ 'install', '--user', 'graphifyy' ] ],
+  [ 'pip3', [ 'install', '--user', 'graphifyy' ] ]
+]) {
+  if (!has(cmd)) continue
+  try {
+    execFileSync(cmd, args, { stdio: 'inherit' })
+    break
+  } catch (error) {
+    // try the next installer
   }
 }
 
-// Note: we deliberately do NOT run `graphify install` — it writes a graphify section into the
-// repo's CLAUDE.md and a PreToolUse hook, which would mutate a committed file and clash with the
-// shared CLAUDE.md. Agents use the graphify CLI directly (see conventions/graphify.md).
-console.log('drawbridge-agents-sync: graphify runtime ready (run `npx drawbridge-agents-graph` to build the graph)')
+if (has('graphify')) {
+  console.log('drawbridge-agents-sync: graphify runtime installed')
+} else {
+  console.warn('drawbridge-agents-sync: graphify not found (optional) — install with `uv tool install graphifyy` to enable the family knowledge graph. Continuing.')
+}
+process.exit(0)

package/bin/sync-claude.js

@@ -90,9 +90,13 @@ const migrateOldDelivery = () => {
   }
 }
 
-// 3. Ensure the graphify runtime is present + reachable (hard-fails if it can't be).
+// 3. Ensure the graphify runtime is present (best-effort, never blocks sync).
 const graphifyPreflight = () => {
-  execFileSync('node', [ path.join(packageRoot, 'bin', 'preflight-graphify.js') ], { stdio: 'inherit' })
+  try {
+    execFileSync('node', [ path.join(packageRoot, 'bin', 'preflight-graphify.js') ], { stdio: 'inherit' })
+  } catch (error) {
+    console.warn('drawbridge-agents-sync: graphify preflight skipped (' + error.message + ')')
+  }
 }
 
 mirrorTemplates()

package/claude/CLAUDE.md

@@ -16,3 +16,4 @@
 @../conventions/superpowers-docs.md
 @../conventions/git-branching.md
 @../conventions/graphify.md
+@../conventions/testing.md

package/conventions/testing.md

@@ -0,0 +1,25 @@
+# Testing
+
+Full strategy, conventions, and per-repo coverage map live in
+`drawbridge-docs/testing/README.md`. The essentials:
+
+- **Run `npm test` before shipping.** Every service and package has a
+  `node:test` suite (`node --test test/`). Run it with the family Node
+  (`nvm use`).
+- **Unit tests live in the repo they test** — `test/*.test.js`, zero
+  dependencies, no README (knowledge is centralized in drawbridge-docs). They
+  version with the code and gate package publishes
+  (`tsup && node --test test/ && npm publish`).
+- **Black-box tests live in `drawbridge-tests`** — the Playwright smoke pack,
+  the k6 load scenarios, and the family results ledger. No app or package repo
+  carries E2E or load tooling.
+- **Two standing policies:** every repo keeps an export-smoke test (a dropped
+  export is a silent-undefined production throw — this has happened); every
+  fixed production incident gets a same-day regression test naming the
+  incident.
+- **Next.js apps get pure-logic tests only** — no jsdom/component testing; UI
+  correctness is the smoke pack's job. Extract a pure seam into `lib/` rather
+  than importing a react/next module into a test.
+- When a module reads env at require time, stub dummies BEFORE the require.
+  Node v20.9's MockTimers mishandles interleaved clears — use real timers for
+  timer-based logic.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@drawbridge/drawbridge-agents",
-  "version": "0.1.6",
+  "version": "0.1.8",
   "description": "Shared agent-instruction content (rules, code style, conventions) for the drawbridge-* monorepo.",
   "license": "UNLICENSED",
   "publishConfig": {
@@ -8,7 +8,8 @@
   },
   "scripts": {
     "sync": ". \"$HOME/.nvm/nvm.sh\" && nvm use && npm prune && npm install",
-    "build": "npm publish"
+    "test": ". \"$HOME/.nvm/nvm.sh\" && nvm use && node --test",
+    "build": ". \"$HOME/.nvm/nvm.sh\" && nvm use && node --test && npm publish"
   },
   "bin": {
     "drawbridge-agents-sync": "bin/sync-claude.js",