compound-agent 1.4.1 → 1.4.3

package/CHANGELOG.md

@@ -9,6 +9,27 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.4.3] - 2026-02-23
+
+### Fixed
+
+- **Setup reports success when SQLite is broken**: `npx ca setup` now verifies that `better-sqlite3` actually loads after configuring `pnpm.onlyBuiltDependencies`, and auto-rebuilds if needed (escalates from `pnpm rebuild` to `pnpm install + rebuild`)
+- **Misleading error message**: `ensureSqliteAvailable()` no longer suggests "Run: npx ca setup" (which didn't fix the problem); now provides per-package-manager rebuild instructions and build tools hint
+
+### Added
+
+- **SQLite health check in `ca doctor`**: New check reports `[FAIL]` with fix hint when `better-sqlite3` cannot load
+- **SQLite status in `ca setup --status`**: Shows "OK" or "not available" alongside other status checks
+- **`resetSqliteAvailability()` export**: Allows re-probing SQLite after native module rebuild
+
+## [1.4.2] - 2026-02-23
+
+### Fixed
+
+- **Banner audio crash on headless Linux**: Async `ENOENT` error from missing `aplay` no longer crashes `ca setup --update`
+- **PowerShell path injection on Windows**: Temp paths containing apostrophes no longer break or inject commands in `banner-audio.ts`
+- **Banner audio test coverage**: Rewrote tests with proper mock isolation (`vi.spyOn` + file-scope `vi.mock`), covering async ENOENT, sync throw, stop() idempotency, and normal exit cleanup
+
 ## [1.4.1] - 2026-02-22
 
 ### Changed

package/dist/cli.js

@@ -33,7 +33,7 @@ function ensureSqliteAvailable() {
     checked = true;
   } catch (cause) {
     throw new Error(
-      'better-sqlite3 failed to load.\nRun: npx ca setup (auto-configures pnpm native builds)\nOr manually add to your package.json:\n  "pnpm": { "onlyBuiltDependencies": ["better-sqlite3", "node-llama-cpp"] }\nThen run: pnpm install && pnpm rebuild better-sqlite3\nFor npm/yarn, run: npm rebuild better-sqlite3',
+      'better-sqlite3 failed to load.\nFor pnpm projects:\n  1. Ensure package.json has: "pnpm": { "onlyBuiltDependencies": ["better-sqlite3"] }\n  2. Run: pnpm install && pnpm rebuild better-sqlite3\nFor npm/yarn projects:\n  Run: npm rebuild better-sqlite3\nIf the error persists, check that build tools (python3, make, g++) are installed.',
       { cause }
     );
   }
@@ -42,6 +42,10 @@ function getDatabaseConstructor() {
   ensureSqliteAvailable();
   return DatabaseConstructor;
 }
+function resetSqliteAvailability() {
+  checked = false;
+  DatabaseConstructor = null;
+}
 var require2, checked, DatabaseConstructor;
 var init_availability = __esm({
   "src/memory/storage/sqlite/availability.ts"() {
@@ -1086,6 +1090,9 @@ async function syncIfNeeded(repoRoot, options = {}) {
   return false;
 }
 
+// src/memory/storage/sqlite/index.ts
+init_availability();
+
 // src/memory/search/hybrid.ts
 var DEFAULT_VECTOR_WEIGHT = 0.7;
 var DEFAULT_TEXT_WEIGHT = 0.3;
@@ -1957,7 +1964,7 @@ function spawnPlayer(filePath) {
       case "win32":
         return spawn("powershell", [
           "-c",
-          `(New-Object Media.SoundPlayer '${filePath}').PlaySync()`
+          `(New-Object Media.SoundPlayer "${filePath.replace(/"/g, '`"')}").PlaySync()`
         ], { stdio: "ignore", detached: true });
       default:
         return null;
@@ -1990,6 +1997,12 @@ function playBannerAudio() {
       } catch {
       }
     };
+    proc.on("error", () => {
+      try {
+        unlinkSync(tmpPath);
+      } catch {
+      }
+    });
     proc.on("exit", () => {
       try {
         unlinkSync(tmpPath);
@@ -3611,6 +3624,19 @@ function printPnpmConfigStatus(result) {
     console.log(`  pnpm config: Added onlyBuiltDependencies [${result.added.join(", ")}]`);
   }
 }
+var SQLITE_STATUS_MSG = {
+  already_ok: "OK",
+  rebuilt: "OK (rebuilt native module)",
+  installed_and_rebuilt: "OK (installed + rebuilt native module)",
+  failed: "FAILED"
+};
+function printSqliteStatus(result) {
+  const msg = SQLITE_STATUS_MSG[result.action];
+  console.log(`  SQLite:             ${msg}`);
+  if (result.error) {
+    console.log(`                      ${result.error}`);
+  }
+}
 function printBeadsFullStatus(check) {
   console.log(`  Beads CLI:          ${check.cliAvailable ? "OK" : "not found"}`);
   if (check.cliAvailable) {
@@ -3645,6 +3671,13 @@ async function runStatus(repoRoot) {
   } catch {
   }
   console.log(`  Hooks:              ${hooksInstalled ? "installed" : "not installed"}`);
+  let sqliteOk = false;
+  try {
+    ensureSqliteAvailable();
+    sqliteOk = true;
+  } catch {
+  }
+  console.log(`  SQLite:             ${sqliteOk ? "OK" : "not available (run: pnpm rebuild better-sqlite3)"}`);
   const fullBeads = runFullBeadsCheck(repoRoot);
   printBeadsFullStatus(fullBeads);
   const scope = checkUserScope(repoRoot);
@@ -6796,6 +6829,60 @@ async function mergePnpmConfig(pkgPath, pkg) {
   await writeFile(pkgPath, JSON.stringify(pkg, null, 2) + "\n", "utf-8");
   return { isPnpm: true, alreadyConfigured: false, added };
 }
+function trySqliteLoad() {
+  try {
+    ensureSqliteAvailable();
+    return true;
+  } catch {
+    return false;
+  }
+}
+function verifySqlite(repoRoot, pnpmConfig) {
+  if (trySqliteLoad()) {
+    return { available: true, action: "already_ok" };
+  }
+  if (!pnpmConfig.isPnpm) {
+    return {
+      available: false,
+      action: "failed",
+      error: "better-sqlite3 failed to load. Run: npm rebuild better-sqlite3"
+    };
+  }
+  try {
+    execFileSync("pnpm", ["rebuild", "better-sqlite3"], {
+      cwd: repoRoot,
+      stdio: "pipe",
+      timeout: 6e4
+    });
+  } catch {
+  }
+  resetSqliteAvailability();
+  if (trySqliteLoad()) {
+    return { available: true, action: "rebuilt" };
+  }
+  try {
+    execFileSync("pnpm", ["install"], {
+      cwd: repoRoot,
+      stdio: "pipe",
+      timeout: 12e4
+    });
+    execFileSync("pnpm", ["rebuild", "better-sqlite3"], {
+      cwd: repoRoot,
+      stdio: "pipe",
+      timeout: 6e4
+    });
+  } catch {
+  }
+  resetSqliteAvailability();
+  if (trySqliteLoad()) {
+    return { available: true, action: "installed_and_rebuilt" };
+  }
+  return {
+    available: false,
+    action: "failed",
+    error: "Auto-rebuild failed. Run manually: pnpm install && pnpm rebuild better-sqlite3"
+  };
+}
 async function runUninstall(repoRoot, dryRun) {
   const actions = [];
   const dirsToRemove = [
@@ -6992,6 +7079,7 @@ async function runSetup(options) {
     upgrade = await runUpgrade(repoRoot);
   }
   const pnpmConfig = await ensurePnpmBuildConfig(repoRoot);
+  const sqlite = verifySqlite(repoRoot, pnpmConfig);
   const lessonsDir = await ensureLessonsDirectory(repoRoot);
   const agentsMdUpdated = await updateAgentsMd(repoRoot);
   await ensureClaudeMdReference(repoRoot);
@@ -7034,6 +7122,7 @@ async function runSetup(options) {
     postCommitHook,
     model: modelStatus,
     pnpmConfig,
+    sqlite,
     beads,
     scope,
     upgrade,
@@ -7135,6 +7224,7 @@ async function printSetupResult(result, quiet, repoRoot) {
   printSetupGitHooksStatus(result.gitHooks);
   printPostCommitHookStatus(result.postCommitHook);
   printPnpmConfigStatus(result.pnpmConfig);
+  printSqliteStatus(result.sqlite);
   printGitignoreStatus(result.gitignore);
   console.log(`  Model: ${MODEL_STATUS_MSG[result.model]}`);
   const fullBeads = runFullBeadsCheck(repoRoot);
@@ -7786,6 +7876,13 @@ async function runDoctor(repoRoot) {
   } catch {
   }
   checks.push(modelOk ? { name: "Embedding model", status: "pass" } : { name: "Embedding model", status: "warn", fix: "Run: npx ca download-model" });
+  let sqliteOk = false;
+  try {
+    ensureSqliteAvailable();
+    sqliteOk = true;
+  } catch {
+  }
+  checks.push(sqliteOk ? { name: "SQLite (better-sqlite3)", status: "pass" } : { name: "SQLite (better-sqlite3)", status: "fail", fix: "Run: pnpm rebuild better-sqlite3 (or npm rebuild better-sqlite3)" });
   const beadsResult = checkBeadsAvailable();
   checks.push(beadsResult.available ? { name: "Beads CLI", status: "pass" } : { name: "Beads CLI", status: "warn", fix: "Install beads: https://github.com/Nathandela/beads" });
   checks.push(checkGitignoreHealth(repoRoot) ? { name: ".gitignore health", status: "pass" } : { name: ".gitignore health", status: "warn", fix: "Run: npx ca setup --update" });
@@ -8589,30 +8686,32 @@ function registerVerifyGatesCommand(program2) {
 }
 
 // src/changelog-data.ts
-var CHANGELOG_RECENT = `## [1.4.1] - 2026-02-22
+var CHANGELOG_RECENT = `## [1.4.3] - 2026-02-23
 
-### Changed
+### Fixed
 
-- **Broader retrieval messaging**: \`ca search\` and \`ca knowledge\` descriptions in prime output and AGENTS.md now encourage general-purpose use beyond mandatory architectural triggers
+- **Setup reports success when SQLite is broken**: \`npx ca setup\` now verifies that \`better-sqlite3\` actually loads after configuring \`pnpm.onlyBuiltDependencies\`, and auto-rebuilds if needed (escalates from \`pnpm rebuild\` to \`pnpm install + rebuild\`)
+- **Misleading error message**: \`ensureSqliteAvailable()\` no longer suggests "Run: npx ca setup" (which didn't fix the problem); now provides per-package-manager rebuild instructions and build tools hint
 
-## [1.4.0] - 2026-02-22
+### Added
 
-### Fixed
+- **SQLite health check in \`ca doctor\`**: New check reports \`[FAIL]\` with fix hint when \`better-sqlite3\` cannot load
+- **SQLite status in \`ca setup --status\`**: Shows "OK" or "not available" alongside other status checks
+- **\`resetSqliteAvailability()\` export**: Allows re-probing SQLite after native module rebuild
 
-- **Plugin manifest**: Corrected repository URL from \`compound_agent\` to \`learning_agent\`
+## [1.4.2] - 2026-02-23
 
-### Changed
+### Fixed
 
-- **Version consolidation**: Roll-up release of v1.3.7\u2013v1.3.9 production readiness fixes (test pipeline hardening, data integrity, two-phase vector search, FTS5 sanitization)
+- **Banner audio crash on headless Linux**: Async \`ENOENT\` error from missing \`aplay\` no longer crashes \`ca setup --update\`
+- **PowerShell path injection on Windows**: Temp paths containing apostrophes no longer break or inject commands in \`banner-audio.ts\`
+- **Banner audio test coverage**: Rewrote tests with proper mock isolation (\`vi.spyOn\` + file-scope \`vi.mock\`), covering async ENOENT, sync throw, stop() idempotency, and normal exit cleanup
 
-## [1.3.9] - 2026-02-22
+## [1.4.1] - 2026-02-22
 
-### Fixed
+### Changed
 
-- **Integration test pipeline reliability**: Moved \`pnpm build\` from vitest globalSetup to npm script pre-step, eliminating EPERM errors from tsx/IPC conflicts inside vitest's process
-- **Fail-fast globalSetup**: Missing \`dist/cli.js\` now throws a clear error instead of cascading 68+ test failures
-- **Integration pool isolation**: Changed from \`threads\` to \`forks\` for integration tests \u2014 proper process isolation for subprocess-spawning tests
-- **Timeout safety net**: Added \`testTimeout: 30_000\` to fallback vitest.config.ts, preventing 5s default under edge conditions`;
+- **Broader retrieval messaging**: \`ca search\` and \`ca knowledge\` descriptions in prime output and AGENTS.md now encourage general-purpose use beyond mandatory architectural triggers`;
 
 // src/commands/about.ts
 function registerAboutCommand(program2) {