knoxis-helper 1.8.1 → 1.8.2

package/lib/knoxis-local-agent.js

@@ -595,14 +595,14 @@ function resolvePairProgramScript() {
 }
 
 const KIT_MODES = new Set(['kickoff', 'feature-kickoff', 'resume', 'session-end', 'recovery']);
-// kickoff, feature-kickoff, and resume rely on Claude actually writing files
-// to disk (docs/state/OPEN_QUESTIONS.md, docs/features/<slug>/*, and the rest
-// of the context pack). The interactive runner invokes `claude -p
-// --session-id`, which gives Claude proper tool access; pair-program.js pipes
-// to `claude` without -p and silently skips the file writes. Route those
-// kits through the interactive runner so their kit prompts actually do
-// their work.
-const KIT_MODES_VIA_INTERACTIVE = new Set(['kickoff', 'feature-kickoff', 'resume']);
+// Every kit mode rewrites files on disk — STATUS / HANDOFF / DECISIONS for
+// session-end, the docs/state/* + docs/features/<slug>/* bag for kickoff /
+// feature-kickoff / resume, and recovery's triage notes. The interactive
+// runner invokes `claude -p --session-id`, which gives Claude proper tool
+// access; the older pair-program.js pipes to `claude` without -p and silently
+// skips the file writes. Route every kit mode through the interactive runner
+// so their kit prompts actually do their work.
+const KIT_MODES_VIA_INTERACTIVE = new Set(['kickoff', 'feature-kickoff', 'resume', 'session-end', 'recovery']);
 
 function buildPairProgramCommand({ scriptPath, workspace, mode, archetype, pattern, productSlug, projectSlug, taskPrompt, userId, workspaceId, taskIds }) {
   const q = v => `"${escapeForDoubleQuotedShellArg(v)}"`;

package/lib/session-recorder.js

@@ -34,39 +34,106 @@ function extractFilesTouched(diff) {
   return Array.from(seen);
 }
 
-// Snapshot the 6 well-known state files (docs/state/*.md) so the
-// session record carries the latest STATUS / HANDOFF / DECISIONS / etc.
-// up to the portal for display on qig.ai.
+// Snapshot the canonical state files so the session record carries the
+// latest STATUS / HANDOFF / DECISIONS / etc. up to the portal for display
+// on qig.ai. Three groups are captured into a single bag keyed by relative
+// path; the frontend distinguishes them by path prefix.
+//
+//   docs/state/*.md            — project-level state (always present after scaffold)
+//   docs/architecture/*.md     — ARCHITECTURE.md + ADRs from project-kickoff
+//   docs/features/<slug>/*     — per-feature ROADMAP / PHASES / STATUS / manifest
+//                                from feature-kickoff (one entry per feature)
+//
+// All three flow through `stateFiles` on the session record. The Java
+// PairProgrammerSessionService persists the bag to the `state_files` JSONB
+// column; no schema change is needed to add new path prefixes.
 const STATE_FILES_TO_CAPTURE = [
   'docs/state/STATUS.md',
   'docs/state/HANDOFF.md',
   'docs/state/DECISIONS.md',
   'docs/state/CHANGELOG.md',
   'docs/state/OPEN_QUESTIONS.md',
-  'docs/state/RISKS.md'
+  'docs/state/RISKS.md',
+  // Project architecture pack from kickoff v7.
+  'docs/architecture/ARCHITECTURE.md'
+];
+const FEATURE_FILE_NAMES = [
+  'ROADMAP.md',
+  'PHASES.md',
+  'STATUS.md',
+  'OPEN_QUESTIONS.md',
+  'manifest.json'
 ];
 const MAX_STATE_FILE_BYTES = 256 * 1024; // 256KB per file
 
+function readSafe(absPath, relPath, out) {
+  try {
+    const stat = fs.statSync(absPath);
+    if (!stat.isFile()) return;
+    if (stat.size > MAX_STATE_FILE_BYTES) {
+      out[relPath] = `[File omitted: ${stat.size} bytes exceeds ${MAX_STATE_FILE_BYTES}-byte cap]`;
+      return;
+    }
+    out[relPath] = fs.readFileSync(absPath, 'utf8');
+  } catch (e) {
+    // Missing file is normal — skip silently.
+  }
+}
+
 function readStateFiles(workspace) {
   const out = {};
   if (!workspace) return out;
+
+  // Fixed list (state pack + ARCHITECTURE).
   for (const rel of STATE_FILES_TO_CAPTURE) {
-    const abs = path.join(workspace, rel);
-    try {
-      const stat = fs.statSync(abs);
-      if (!stat.isFile()) continue;
-      if (stat.size > MAX_STATE_FILE_BYTES) {
-        out[rel] = `[File omitted: ${stat.size} bytes exceeds ${MAX_STATE_FILE_BYTES}-byte cap]`;
-        continue;
+    readSafe(path.join(workspace, rel), rel, out);
+  }
+
+  // ADRs — docs/architecture/adr/*.md.
+  const adrDir = path.join(workspace, 'docs', 'architecture', 'adr');
+  try {
+    for (const entry of fs.readdirSync(adrDir, { withFileTypes: true })) {
+      if (!entry.isFile() || !entry.name.endsWith('.md')) continue;
+      const rel = `docs/architecture/adr/${entry.name}`;
+      readSafe(path.join(adrDir, entry.name), rel, out);
+    }
+  } catch (e) {
+    // No ADR dir is normal.
+  }
+
+  // Per-feature files — docs/features/<slug>/{ROADMAP,PHASES,STATUS,OPEN_QUESTIONS}.md
+  // and manifest.json. One slug = one folder.
+  const featuresDir = path.join(workspace, 'docs', 'features');
+  try {
+    for (const entry of fs.readdirSync(featuresDir, { withFileTypes: true })) {
+      if (!entry.isDirectory()) continue;
+      for (const filename of FEATURE_FILE_NAMES) {
+        const rel = `docs/features/${entry.name}/${filename}`;
+        readSafe(path.join(featuresDir, entry.name, filename), rel, out);
       }
-      out[rel] = fs.readFileSync(abs, 'utf8');
-    } catch (e) {
-      // Missing file is normal — skip silently.
     }
+  } catch (e) {
+    // No features dir is normal.
   }
+
   return out;
 }
 
+// Enumerate feature slugs from the workspace so the frontend can list features
+// without parsing the bag. Returns sorted array of slug strings.
+function listFeatureSlugs(workspace) {
+  if (!workspace) return [];
+  const featuresDir = path.join(workspace, 'docs', 'features');
+  try {
+    return fs.readdirSync(featuresDir, { withFileTypes: true })
+      .filter(e => e.isDirectory())
+      .map(e => e.name)
+      .sort();
+  } catch (e) {
+    return [];
+  }
+}
+
 // Snapshot product.local.json from the project's parent directory (per
 // kickoff v7 Phase -1) so the session record carries product context up to
 // the portal. Multiple projects under one product share this file; the
@@ -270,10 +337,16 @@ class SessionRecorder {
       completedSteps,
       closedCleanly,
       filesTouched: extractFilesTouched(totalDiff),
-      // Snapshot of docs/state/*.md at session-end so the QIG frontend can
-      // display the latest STATUS / HANDOFF / DECISIONS / etc. without
-      // needing live filesystem access on the dev's machine.
+      // Snapshot of state-pack + architecture + feature files at session-end
+      // so the QIG frontend can display the latest STATUS / HANDOFF /
+      // DECISIONS / ROADMAP / etc. without needing live filesystem access
+      // on the dev's machine. Single bag keyed by relative path; frontend
+      // filters by prefix (docs/state/*, docs/architecture/*,
+      // docs/features/<slug>/*).
       stateFiles: readStateFiles(this.workspace),
+      // Quick index of feature slugs found in docs/features/. Lets the
+      // frontend enumerate features without parsing the bag.
+      featureSlugs: listFeatureSlugs(this.workspace),
       // Snapshot of product.local.json from the parent directory so the
       // frontend can render product context alongside the session record.
       productContext: readProductContext(this.workspace),
@@ -299,5 +372,6 @@ module.exports = {
   slugify,
   extractFilesTouched,
   readStateFiles,
-  readProductContext
+  readProductContext,
+  listFeatureSlugs
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "knoxis-helper",
-  "version": "1.8.1",
+  "version": "1.8.2",
   "description": "Local helper for Knoxis pair programming - connects your machine to Knoxis on qig.ai",
   "bin": {
     "knoxis-helper": "./bin/knoxis-helper.js"