@smartmemory/compose 0.2.48-beta → 0.2.50-beta

package/.claude/skills/compose/SKILL.md

@@ -596,6 +596,8 @@ Authoritative list lives in `compose/.compose-deps.json`. Run `compose doctor` t
 
 The manifest declares 12 external skills/commands across `superpowers:*`, `interface-design:*`, `codex:review`, `refactor`, and `update-docs`. Each entry carries `id`, `required_for`, `install`, `fallback` (or null), and `optional`.
 
+The manifest also has an optional `external_binaries` array for CLI tools (not skills). Today it declares `rtk` ([Rust Token Killer](https://github.com/rtk-ai/rtk), COMP-RTK-INTEROP) — a lossy output compressor compose routes its one LLM-bound `git diff` (the Codex review diff) through when present, degrading byte-identically when absent. `compose doctor` reports binaries alongside skills and surfaces each binary's `recommend` (for rtk: `rtk init -g`, which installs RTK's Claude Code hook so the agent's *own* Bash output is compressed too — the larger token win). Binary entries carry `id`, `detect`, `install`, `recommend` (or null), and `optional`; all are optional and never block the lifecycle.
+
 ### Degrade pattern
 
 At lifecycle entry (Phase 1), run `compose doctor --json`. For each missing dep:

package/.compose-deps.json

@@ -85,5 +85,18 @@
       "fallback": "manual TODO surfaced in implementation report",
       "optional": true
     }
+  ],
+  "external_binaries": [
+    {
+      "id": "rtk",
+      "detect": "rtk --version",
+      "required_for": [
+        "LLM-bound git-diff compression (build review)",
+        "agent Bash output compression via `rtk init -g` hook"
+      ],
+      "install": "brew install rtk  (or: cargo install --git https://github.com/rtk-ai/rtk)",
+      "recommend": "rtk init -g",
+      "optional": true
+    }
   ]
 }

package/bin/compose.js

@@ -17,6 +17,7 @@ import { fileURLToPath } from 'url'
 import { findProjectRoot } from '../server/find-root.js'
 import { resolveWorkspace, getWorkspaceFlag } from '../lib/resolve-workspace.js'
 import { resolvePort } from '../lib/resolve-port.js'
+import { resolveRoadmapPath, resolveFeaturesPath, resolveContextPathFromConfig, resolveFeaturesPathFromConfig, resolveRoadmapPathFromConfig } from '../lib/project-paths.js'
 
 const __dirname = dirname(fileURLToPath(import.meta.url))
 const PACKAGE_ROOT = resolve(__dirname, '..')
@@ -81,7 +82,7 @@ function resolveCwdWithWorkspace(args) {
 // --team flag (COMP-TEAMS)
 // ---------------------------------------------------------------------------
 import { parseTeamFlag } from '../lib/team-flag.js';
-import { loadDeps, checkExternalSkills, printDepReport, buildDepReport } from '../lib/deps.js';
+import { loadDeps, checkExternalSkills, printDepReport, buildDepReport, checkExternalBinaries, printBinaryReport, buildBinaryReport } from '../lib/deps.js';
 import { checkLatestVersion } from '../lib/version-check.js';
 
 const [,, cmd, ...args] = process.argv
@@ -271,6 +272,7 @@ function syncSkills(agents) {
   if (deps) {
     const result = checkExternalSkills(deps)
     printDepReport(result)
+    printBinaryReport(checkExternalBinaries(deps))
   }
 }
 
@@ -302,7 +304,8 @@ async function runDoctor(flags = []) {
     // Single top-level JSON document — the deps report and the version block share one root
     // so consumers like `JSON.parse(stdout)` work. (Previously two concatenated objects.)
     const report = buildDepReport(result)
-    console.log(JSON.stringify({ ...report, version: versionInfo }, null, 2))
+    const binaries = buildBinaryReport(checkExternalBinaries(deps))
+    console.log(JSON.stringify({ ...report, binaries, version: versionInfo }, null, 2))
     const allRequiredPresent = result.missing.every(d => d.optional)
     if (strict && !allRequiredPresent) process.exit(1)
     return
@@ -323,6 +326,7 @@ async function runDoctor(flags = []) {
   }
 
   const allRequiredPresent = printDepReport(result, { json: false, verbose })
+  printBinaryReport(checkExternalBinaries(deps))
   if (strict && !allRequiredPresent) process.exit(1)
 }
 
@@ -409,6 +413,7 @@ async function runInit(flags, cwdOverride) {
     },
     paths: {
       docs: 'docs',
+      roadmap: 'ROADMAP.md',
       features: 'docs/features',
       journal: 'docs/journal',
       context: 'docs/context',
@@ -444,8 +449,9 @@ async function runInit(flags, cwdOverride) {
     }
   }
 
-  // 5b. Scaffold docs/context/ with ambient context templates
-  const contextDir = join(cwd, config.paths.context)
+  // 5b. Scaffold docs/context/ with ambient context templates.
+  // COMP-PATHS-EXTERNAL: resolve from the in-memory config (may be relocated).
+  const contextDir = resolveContextPathFromConfig(cwd, config)
   mkdirSync(contextDir, { recursive: true })
   const contextTemplates = {
     'tech-stack.md': '# Tech Stack\n\nDescribe your technology stack here.\n',
@@ -461,8 +467,8 @@ async function runInit(flags, cwdOverride) {
   }
 
   // 5c. Scaffold docs/product/ideabox.md if absent
-  const ideaboxRel = config.paths.ideabox || 'docs/product/ideabox.md'
-  const ideaboxDest = join(cwd, ideaboxRel)
+  const { resolveIdeaboxPath: _resolveIdeaboxPathInit } = await import('../lib/project-paths.js')
+  const ideaboxDest = _resolveIdeaboxPathInit(cwd)
   if (!existsSync(ideaboxDest)) {
     mkdirSync(dirname(ideaboxDest), { recursive: true })
     const { IDEABOX_TEMPLATE } = await import('../lib/ideabox.js')
@@ -510,8 +516,10 @@ async function runInit(flags, cwdOverride) {
     }
   }
 
-  // 7. Scaffold ROADMAP.md from template if absent
-  const roadmapDest = join(cwd, 'ROADMAP.md')
+  // 7. Scaffold ROADMAP.md from template if absent. COMP-PATHS-EXTERNAL:
+  // honor a configured (possibly relocated) paths.roadmap instead of always
+  // seeding <cwd>/ROADMAP.md. Byte-identical for the default.
+  const roadmapDest = resolveRoadmapPathFromConfig(cwd, config)
   if (!existsSync(roadmapDest)) {
     const roadmapSrc = join(PACKAGE_ROOT, 'templates', 'ROADMAP.md')
     if (existsSync(roadmapSrc)) {
@@ -939,10 +947,12 @@ if (cmd === 'feature') {
 
   let config = {}
   try { config = JSON.parse(readFileSync(configPath, 'utf-8')) } catch {}
-  const featuresDir = config.paths?.features || 'docs/features'
+  // COMP-PATHS-EXTERNAL: absolute features dir (may be relocated outside cwd).
+  // Never join(cwd, …) below — that would re-root an absolute/../-escaping override.
+  const featuresDir = resolveFeaturesPath(cwd)
 
   // 1. Create feature folder + seed design doc
-  const featureDir = join(cwd, featuresDir, featureCode)
+  const featureDir = join(featuresDir, featureCode)
   const designPath = join(featureDir, 'design.md')
 
   if (existsSync(designPath)) {
@@ -982,7 +992,7 @@ if (cmd === 'feature') {
   // Count existing features for complexity estimate
   const existingFeatureCount = (() => {
     try {
-      const fdir = join(cwd, featuresDir)
+      const fdir = featuresDir
       if (!existsSync(fdir)) return 0
       return readdirSync(fdir, { withFileTypes: true })
         .filter(e => e.isDirectory() && existsSync(join(fdir, e.name, 'feature.json')))
@@ -1039,7 +1049,7 @@ _This is a seed design doc created by \`compose feature\`. The \`compose build\`
   console.log(`Created ${designPath}`)
 
   // 2. Add entry to ROADMAP.md
-  const roadmapPath = join(cwd, 'ROADMAP.md')
+  const roadmapPath = resolveRoadmapPath(cwd)
   if (existsSync(roadmapPath)) {
     let roadmap = readFileSync(roadmapPath, 'utf-8')
 
@@ -1154,7 +1164,7 @@ if (cmd === 'roadmap') {
     const { checkRoundtrip, describeLossyDiff } = await import('../lib/roadmap-roundtrip.js')
     const { loadExternalPrefixes } = await import('../lib/project-paths.js')
     const { root: cwd } = resolveCwdWithWorkspace(args)
-    const roadmapPath = join(cwd, 'ROADMAP.md')
+    const roadmapPath = resolveRoadmapPath(cwd)
     if (!existsSync(roadmapPath)) {
       console.error('No ROADMAP.md found. Run: compose roadmap generate')
       process.exit(1)
@@ -1347,7 +1357,7 @@ if (cmd === 'roadmap') {
   }
 
   const { root: cwd } = resolveCwdWithWorkspace(args)
-  const roadmapPath = join(cwd, 'ROADMAP.md')
+  const roadmapPath = resolveRoadmapPath(cwd)
 
   if (existsSync(roadmapPath)) {
     // Show cwd roadmap
@@ -2423,7 +2433,11 @@ if (cmd === 'build') {
   } = await import('../lib/ideabox.js')
 
   const ibRelPath = getIdeaboxRelPath(ibCwd)
-  const ibFullPath = join(ibCwd, ibRelPath)
+  // ibFullPath is used for direct file ops (existsSync/mkdirSync/writeFileSync/
+  // display); resolve it absolute-safe. ibRelPath stays relative because the
+  // lib readIdeabox/writeIdeabox readers re-join it under ibCwd.
+  const { resolveIdeaboxPath: _resolveIdeaboxPathCli } = await import('../lib/project-paths.js')
+  const ibFullPath = _resolveIdeaboxPathCli(ibCwd)
 
   if (!ibSubcmd || ibSubcmd === '--help' || ibSubcmd === '-h') {
     console.log('Usage: compose ideabox <subcommand>')
@@ -2541,9 +2555,10 @@ if (cmd === 'build') {
       resolvedCode = `IDEA-${idea.num}-${slug}`.toUpperCase()
     }
 
-    // Create feature folder if missing — respect paths.features from compose.json
-    const featuresRel = ibConfig.paths?.features || 'docs/features'
-    const featuresDir = join(ibCwd, featuresRel, resolvedCode)
+    // Create feature folder if missing — respect paths.features (may be
+    // relocated outside ibCwd — COMP-PATHS-EXTERNAL; resolve absolute, no re-root)
+    const featuresBase = resolveFeaturesPathFromConfig(ibCwd, ibConfig)
+    const featuresDir = join(featuresBase, resolvedCode)
     if (!existsSync(featuresDir)) {
       // COMP-MCP-VALIDATE-1: route through the validated writer (schema-guarded)
       // instead of a raw writeFileSync.
@@ -2554,8 +2569,8 @@ if (cmd === 'build') {
         status: 'PLANNED',
         promotedFrom: ideaId,
         createdAt: new Date().toISOString(),
-      }, featuresRel)
-      console.log(`Created feature folder: ${featuresRel}/${resolvedCode}/`)
+      }, featuresBase)
+      console.log(`Created feature folder: ${join(featuresBase, resolvedCode)}/`)
     }
 
     _promoteIdea(parsed, ideaId, resolvedCode)