compound-agent 1.4.0 → 1.4.2

package/CHANGELOG.md

@@ -9,6 +9,20 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [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
+
+- **Broader retrieval messaging**: `ca search` and `ca knowledge` descriptions in prime output and AGENTS.md now encourage general-purpose use beyond mandatory architectural triggers
+
 ## [1.4.0] - 2026-02-22
 
 ### Fixed
@@ -701,10 +715,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Vitest test suite
   - tsup build configuration
 
-[Unreleased]: https://github.com/Nathandela/learning_agent/compare/v1.4.0...HEAD
+[Unreleased]: https://github.com/Nathandela/learning_agent/compare/v1.4.1...HEAD
+[1.4.1]: https://github.com/Nathandela/learning_agent/compare/v1.4.0...v1.4.1
 [1.4.0]: https://github.com/Nathandela/learning_agent/compare/v1.3.9...v1.4.0
 [1.3.9]: https://github.com/Nathandela/learning_agent/compare/v1.3.8...v1.3.9
-[1.3.8]: https://github.com/Nathandela/learning_agent/compare/v1.3.3...v1.3.8
+[1.3.8]: https://github.com/Nathandela/learning_agent/compare/v1.3.7...v1.3.8
 [1.3.7]: https://github.com/Nathandela/learning_agent/compare/v1.3.3...v1.3.7
 [1.3.3]: https://github.com/Nathandela/learning_agent/compare/v1.3.2...v1.3.3
 [1.3.2]: https://github.com/Nathandela/learning_agent/compare/v1.3.1...v1.3.2

package/dist/cli.js

@@ -1957,7 +1957,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 +1990,12 @@ function playBannerAudio() {
       } catch {
       }
     };
+    proc.on("error", () => {
+      try {
+        unlinkSync(tmpPath);
+      } catch {
+      }
+    });
     proc.on("exit", () => {
       try {
         unlinkSync(tmpPath);
@@ -3266,8 +3272,8 @@ This project uses compound-agent for session memory via **CLI commands**.
 
 | Command | Purpose |
 |---------|---------|
-| \`npx ca search "query"\` | Search lessons - use BEFORE architectural decisions |
-| \`npx ca knowledge "query"\` | Search docs knowledge - use BEFORE architectural decisions |
+| \`npx ca search "query"\` | Search lessons - MUST call before architectural decisions; use anytime you need context |
+| \`npx ca knowledge "query"\` | Ask the project docs any question - MUST call before architectural decisions; use freely |
 | \`npx ca learn "insight"\` | Capture lessons - use AFTER corrections or discoveries |
 | \`npx ca list\` | List all stored lessons |
 | \`npx ca show <id>\` | Show details of a specific lesson |
@@ -3282,6 +3288,8 @@ You MUST call \`npx ca search\` and \`npx ca knowledge\` BEFORE:
 
 **NEVER skip search for complex decisions.** Past mistakes will repeat.
 
+Beyond mandatory triggers, use these commands freely \u2014 they are lightweight queries, not heavyweight operations. Uncertain about a pattern? \`ca search\`. Need a detail from the docs? \`ca knowledge\`. The cost of an unnecessary search is near-zero; the cost of a missed one can be hours.
+
 ### Capture Protocol
 
 Run \`npx ca learn\` AFTER:
@@ -3305,7 +3313,7 @@ Before capturing, verify the lesson is:
 The JSONL file requires proper ID generation, schema validation, and SQLite sync.
 Use CLI (\`npx ca learn\`) \u2014 never manual edits.
 
-See [documentation](https://github.com/Nathandela/compound_agent) for more details.
+See [documentation](https://github.com/Nathandela/learning_agent) for more details.
 ${AGENTS_SECTION_END_MARKER}
 `;
 var LEGACY_ROOT_SLASH_COMMANDS = [
@@ -6692,7 +6700,8 @@ async function installDocTemplates(repoRoot) {
   }
   return created;
 }
-async function installResearchDocs(repoRoot) {
+async function installResearchDocs(repoRoot, options) {
+  const force = options?.force ?? false;
   const pkgRoot = join(dirname(fileURLToPath(import.meta.url)), "..");
   const srcDir = join(pkgRoot, "docs", "research");
   if (!existsSync(srcDir)) {
@@ -6709,20 +6718,24 @@ async function installResearchDocs(repoRoot) {
       if (entry.isDirectory()) {
         await mkdir(destPath, { recursive: true });
         await copyDir(srcPath, destPath);
-      } else if (!existsSync(destPath) && entry.name.endsWith(".md")) {
-        let content = await readFile(srcPath, "utf-8");
-        if (entry.name === "index.md") {
-          const patched = content.replace(
-            /^# .*/m,
-            "$&\n\n> Shipped by compound-agent. Source: `docs/research/` in the compound-agent package."
-          );
-          content = patched !== content ? patched : `> Shipped by compound-agent.
+        continue;
+      }
+      if (!entry.name.endsWith(".md")) continue;
+      const exists = existsSync(destPath);
+      if (exists && !force) continue;
+      let content = await readFile(srcPath, "utf-8");
+      if (entry.name === "index.md") {
+        const patched = content.replace(
+          /^# .*/m,
+          "$&\n\n> Shipped by compound-agent. Source: `docs/research/` in the compound-agent package."
+        );
+        content = patched !== content ? patched : `> Shipped by compound-agent.
 
 ${content}`;
-        }
-        await writeFile(destPath, content, "utf-8");
-        created = true;
       }
+      if (exists && await readFile(destPath, "utf-8") === content) continue;
+      await writeFile(destPath, content, "utf-8");
+      created = true;
     }
   }
   try {
@@ -6994,6 +7007,7 @@ async function runSetup(options) {
   await installPhaseSkills(repoRoot);
   await installAgentRoleSkills(repoRoot);
   await installDocTemplates(repoRoot);
+  await installResearchDocs(repoRoot);
   let gitHooks = "skipped";
   if (!options.skipHooks) {
     gitHooks = (await installPreCommitHook(repoRoot)).status;
@@ -7068,6 +7082,9 @@ async function runUpdate(repoRoot, dryRun) {
     const content = template.replace("{{VERSION}}", VERSION).replace("{{DATE}}", (/* @__PURE__ */ new Date()).toISOString().slice(0, 10));
     await processFile(join(repoRoot, "docs", "compound", filename), content);
   }
+  if (!dryRun) {
+    await installResearchDocs(repoRoot, { force: true });
+  }
   for (const filename of LEGACY_ROOT_SLASH_COMMANDS) {
     const filePath = join(repoRoot, ".claude", "commands", filename);
     if (existsSync(filePath)) {
@@ -8073,8 +8090,8 @@ var TRUST_LANGUAGE_TEMPLATE = `# Compound Agent Active
 
 | Command | Purpose |
 |---------|---------|
-| \`npx ca search "query"\` | Search lessons - call BEFORE architectural decisions |
-| \`npx ca knowledge "query"\` | Search docs knowledge - use for architecture context |
+| \`npx ca search "query"\` | Search lessons - MUST call before architectural decisions; use anytime you need context |
+| \`npx ca knowledge "query"\` | Ask the project docs any question - MUST call before architectural decisions; use freely |
 | \`npx ca learn "insight"\` | Capture lessons - call AFTER corrections or discoveries |
 
 ## Core Constraints
@@ -8093,6 +8110,8 @@ You MUST call \`npx ca search\` and \`npx ca knowledge\` BEFORE:
 
 **NEVER skip search for complex decisions.** Past mistakes will repeat.
 
+Beyond mandatory triggers, use these commands freely \u2014 they are lightweight queries, not heavyweight operations. Uncertain about a pattern? \`ca search\`. Need a detail from the docs? \`ca knowledge\`. The cost of an unnecessary search is near-zero; the cost of a missed one can be hours.
+
 ## Capture Protocol
 
 Run \`npx ca learn\` AFTER:
@@ -8576,50 +8595,29 @@ function registerVerifyGatesCommand(program2) {
 }
 
 // src/changelog-data.ts
-var CHANGELOG_RECENT = `## [1.4.0] - 2026-02-22
+var CHANGELOG_RECENT = `## [1.4.2] - 2026-02-23
 
 ### Fixed
 
-- **Plugin manifest**: Corrected repository URL from \`compound_agent\` to \`learning_agent\`
-
-### Changed
-
-- **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
 
-## [1.3.8] - 2026-02-22
+## [1.4.0] - 2026-02-22
 
 ### Fixed
 
-- **Integration test reliability**: Dynamic assertion on workflow command count instead of hardcoded magic number; 30s test timeout for integration suite; conditional build in global-setup; 30s timeout on all bare \`execSync\` calls in init tests
-- **Data integrity**: Indexing pipeline wraps delete/upsert/hash-set in a single transaction for atomic file re-indexing
-- **FTS5 sanitization**: Extended regex to strip parentheses, colons, and braces in addition to existing special chars
-- **Safe JSON.parse**: \`rowToMemoryItem\` now uses \`safeJsonParse\` with fallbacks instead of bare \`JSON.parse\`
-- **ENOENT on schema migration**: \`unlinkSync\` in lessons DB wrapped in try/catch (matches knowledge DB pattern)
-- **Worktree hook support**: \`getGitHooksDir\` resolves \`.git\` file (\`gitdir:\` reference) in worktrees
+- **Plugin manifest**: Corrected repository URL from \`compound_agent\` to \`learning_agent\`
 
 ### Changed
 
-- **Two-phase vector search**: Knowledge vector search loads only IDs + embeddings in phase 1, hydrates full text for top-k only in phase 2 (reduces memory from O(n * text) to O(n * embedding) + O(k * text))
-- **Deduplicated FTS5 search**: \`searchKeyword\` and \`searchKeywordScored\` share a single \`executeFtsQuery\` helper
-- **Removed redundant COUNT pre-checks**: FTS5 naturally returns empty on empty tables
-- **Extracted chunk count helpers**: \`getChunkCount\` / \`getChunkCountByFilePath\` replace raw SQL in \`knowledge.ts\` and \`indexing.ts\`
-- **Immutable extension sets**: \`SUPPORTED_EXTENSIONS\` typed as \`ReadonlySet\`; new \`CODE_EXTENSIONS\` constant replaces hardcoded array in chunking
-- **\`test:all\` builds first**: Script now runs \`pnpm build\` before model download and test run
-- **Test describe label**: Fixed misleading \`'when stop_hook_active is false'\` to match actual test condition
-
-### Added
-
-- \`filesErrored\` field in \`IndexResult\` to track file read failures during indexing
-- \`tsx\` added to devDependencies (was used but not declared)`;
+- **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)`;
 
 // src/commands/about.ts
 function registerAboutCommand(program2) {