@smartmemory/compose 0.1.7-beta → 0.1.9-beta

package/server/compose-mcp.js

@@ -50,6 +50,7 @@ import {
   toolLinkFeatures,
   toolGetFeatureArtifacts,
   toolGetFeatureLinks,
+  toolProposeFollowup,
   toolAddChangelogEntry,
   toolGetChangelogEntries,
   toolWriteJournalEntry,
@@ -58,7 +59,12 @@ import {
   toolGetCompletions,
   toolValidateFeature,
   toolValidateProject,
+  toolSetWorkspace,
+  toolGetWorkspace,
+  _getBinding,
 } from './compose-mcp-tools.js';
+import { switchProject, getTargetRoot } from './project-root.js';
+import { resolveWorkspace } from '../lib/resolve-workspace.js';
 
 // ---------------------------------------------------------------------------
 // Tool definitions
@@ -150,6 +156,20 @@ const TOOLS = [
       required: ['featureCode'],
     },
   },
+  {
+    name: 'set_workspace',
+    description: 'Bind this MCP session to a workspace. Required when cwd contains multiple workspaces. Lives in process memory; lost on MCP restart.',
+    inputSchema: {
+      type: 'object',
+      required: ['workspaceId'],
+      properties: { workspaceId: { type: 'string', description: 'Workspace ID (kebab-case)' } },
+    },
+  },
+  {
+    name: 'get_workspace',
+    description: 'Get the current MCP workspace binding plus all candidates discovered from cwd.',
+    inputSchema: { type: 'object', properties: {} },
+  },
   {
     name: 'get_feature_lifecycle',
     description: 'Get the lifecycle state of a feature: current phase, phase history, artifacts, warnings.',
@@ -175,11 +195,15 @@ const TOOLS = [
   },
   {
     name: 'complete_feature',
-    description: 'Mark a feature as complete. Only callable from the ship phase.',
+    description: 'Mark a feature as complete. Only callable from the ship phase. When commit_sha is provided, the lifecycle endpoint also writes a commit-bound completion record via record_completion (which atomically flips feature.status to COMPLETE and regenerates ROADMAP.md). Without commit_sha, the lifecycle transitions but no completion record is written; a `cockpit_completion_skipped` decision event explains the skip.',
     inputSchema: {
       type: 'object',
       properties: {
         id: { type: 'string', description: 'Item ID' },
+        commit_sha: { type: 'string', description: 'Full 40-char commit SHA. Required to write a completion record.' },
+        tests_pass: { type: 'boolean', description: 'Defaults to true when commit_sha is provided.' },
+        files_changed: { type: 'array', items: { type: 'string' }, description: 'Repo-relative paths committed in the SHA.' },
+        notes: { type: 'string', description: 'One-line note for the completion record.' },
       },
       required: ['id'],
     },
@@ -348,6 +372,23 @@ const TOOLS = [
       },
     },
   },
+  {
+    name: 'propose_followup',
+    description: 'File a follow-up feature against a parent. Auto-numbers the next code in the parent\'s namespace (parent_code-N), adds the ROADMAP row, links surfaced_by from new → parent, and scaffolds design.md with a "## Why" rationale block. Idempotent on (parent_code, idempotency_key); resumes across partial failures via an inflight ledger.',
+    inputSchema: {
+      type: 'object',
+      required: ['parent_code', 'description', 'rationale'],
+      properties: {
+        parent_code: { type: 'string', description: 'Parent feature code (e.g. "COMP-MCP-MIGRATION"). Must exist; must not be KILLED/SUPERSEDED.' },
+        description: { type: 'string', description: 'One-line description for the ROADMAP cell.' },
+        rationale: { type: 'string', description: 'Why this follow-up exists. Persisted as a "## Why" block in the new design.md and in the audit event.' },
+        complexity: { type: 'string', enum: ['S', 'M', 'L', 'XL'] },
+        phase: { type: 'string', description: 'Phase heading. Defaults to the parent\'s phase if omitted.' },
+        status: { type: 'string', enum: ['PLANNED', 'IN_PROGRESS', 'PARTIAL', 'COMPLETE', 'BLOCKED', 'KILLED', 'PARKED', 'SUPERSEDED'] },
+        idempotency_key: { type: 'string', description: 'Optional retry-safety key. Without it, repeated calls allocate new codes.' },
+      },
+    },
+  },
 
   // -------------------------------------------------------------------------
   // Linker — COMP-MCP-ARTIFACT-LINKER
@@ -548,7 +589,13 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
 server.setRequestHandler(CallToolRequestSchema, async (request) => {
   const { name, arguments: args = {} } = request.params;
 
+  const WORKSPACE_EXEMPT = new Set(['set_workspace', 'get_workspace']);
+
   try {
+    if (!WORKSPACE_EXEMPT.has(name)) {
+      const ws = resolveWorkspace({ getBinding: _getBinding });
+      if (ws.root !== getTargetRoot()) switchProject(ws.root);
+    }
     let result;
     switch (name) {
       case 'get_vision_items':    result = toolGetVisionItems(args); break;
@@ -557,6 +604,8 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       case 'get_blocked_items':   result = toolGetBlockedItems(); break;
       case 'get_current_session': result = await toolGetCurrentSession(args); break;
       case 'bind_session':             result = await toolBindSession(args); break;
+      case 'set_workspace':            result = toolSetWorkspace(args); break;
+      case 'get_workspace':            result = toolGetWorkspace(); break;
       case 'get_feature_lifecycle':    result = toolGetFeatureLifecycle(args); break;
       case 'kill_feature':             result = await toolKillFeature(args); break;
       case 'complete_feature':         result = await toolCompleteFeature(args); break;
@@ -582,6 +631,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
       case 'get_completions':          result = await toolGetCompletions(args); break;
       case 'validate_feature':         result = await toolValidateFeature(args); break;
       case 'validate_project':         result = await toolValidateProject(args); break;
+      case 'propose_followup':         result = await toolProposeFollowup(args); break;
       // agent_run removed — STRAT-DEDUP-AGENTRUN v1. Use mcp__stratum__stratum_agent_run.
       default:
         return {
@@ -601,6 +651,21 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     let text = err && err.code
       ? `Error [${err.code}]: ${err.message}`
       : `Error: ${err.message}`;
+    // For workspace ambiguity, include the candidate list and the call to fix it
+    // so Claude can prompt the user without a follow-up tool call.
+    if (err && err.code === 'WorkspaceAmbiguous' && Array.isArray(err.candidates)) {
+      text += '\n\nCandidates:';
+      for (const c of err.candidates) text += `\n  - ${c.id}  (${c.root})`;
+      text += '\n\nNext step: call set_workspace({"workspaceId": "<id>"}) and retry.';
+    }
+    if (err && err.code === 'WorkspaceIdCollision' && Array.isArray(err.roots)) {
+      text += `\n\nworkspaceId "${err.id}" is used by multiple roots:`;
+      for (const r of err.roots) text += `\n  - ${r}`;
+      text += '\n\nFix: set an explicit workspaceId in each .compose/compose.json.';
+    }
+    if (err && err.code === 'WorkspaceUnset') {
+      text += '\n\nNo .compose/ workspace was found. Run `compose init` to scaffold one.';
+    }
     if (err && err.cause && typeof err.cause.message === 'string') {
       text += err.cause.code
         ? `\n  Caused by [${err.cause.code}]: ${err.cause.message}`

package/server/project-root.js

@@ -48,6 +48,10 @@ export function getTargetRoot() { return _targetRoot; }
 /** Data directory for Compose state. Lives in the target project. */
 export function getDataDir() { return _dataDir; }
 
+let _currentWorkspaceId = null;
+export function getCurrentWorkspaceId() { return _currentWorkspaceId; }
+export function setCurrentWorkspaceId(id) { _currentWorkspaceId = id; }
+
 // ---------------------------------------------------------------------------
 // Switch project at runtime
 // ---------------------------------------------------------------------------

package/server/vision-routes.js

@@ -338,7 +338,7 @@ export function attachVisionRoutes(app, { store, scheduleBroadcast, broadcastMes
     }
   });
 
-  app.post('/api/vision/items/:id/lifecycle/complete', (req, res) => {
+  app.post('/api/vision/items/:id/lifecycle/complete', async (req, res) => {
     try {
       const item = store.items.get(req.params.id);
       if (!item?.lifecycle) return res.status(404).json({ error: 'No lifecycle on this item' });
@@ -360,7 +360,56 @@ export function attachVisionRoutes(app, { store, scheduleBroadcast, broadcastMes
       emitDriftAxes(broadcastMessage, store, item, projectRoot, now);
       // COMP-OBS-STATUS: emit status snapshot after lifecycle transition (complete)
       emitStatusSnapshot(broadcastMessage, store, item.lifecycle.featureCode, now);
-      res.json({ completedAt: now });
+
+      // COMP-MCP-MIGRATION: reconcile cockpit complete with record_completion
+      // (commit-bound completion writer). Best-effort: failures emit a decision
+      // event but never roll back the lifecycle transition.
+      const featureCode = item.lifecycle.featureCode;
+      const completionResult = { partial: false };
+      if (featureCode) {
+        const { commit_sha, tests_pass, files_changed, notes } = req.body || {};
+        if (commit_sha) {
+          try {
+            const { recordCompletion } = await import('../lib/completion-writer.js');
+            await recordCompletion(projectRoot, {
+              feature_code: featureCode,
+              commit_sha,
+              tests_pass: tests_pass ?? true,
+              files_changed: files_changed ?? [],
+              notes: notes ?? `cockpit lifecycle: ${featureCode} complete`,
+            });
+          } catch (err) {
+            completionResult.partial = true;
+            const eventType = err.code === 'STATUS_FLIP_AFTER_COMPLETION_RECORDED'
+              ? 'cockpit_completion_partial_status_flip'
+              : 'cockpit_completion_failed';
+            completionResult.completion_failed = err.code || 'UNKNOWN';
+            completionResult.message = err.message;
+            try {
+              emitDecisionEvent(broadcastMessage, {
+                type: eventType,
+                featureCode,
+                timestamp: now,
+                error: { code: err.code, message: err.message },
+              });
+            } catch { /* decision event emit best-effort */ }
+            // eslint-disable-next-line no-console
+            console.warn(`[lifecycle/complete] record_completion ${eventType} for ${featureCode}: ${err.message}`);
+          }
+        } else {
+          // No SHA — emit a skip event so validate_feature can flag drift later
+          try {
+            emitDecisionEvent(broadcastMessage, {
+              type: 'cockpit_completion_skipped',
+              featureCode,
+              timestamp: now,
+              reason: 'no_commit_sha',
+            });
+          } catch { /* decision event emit best-effort */ }
+        }
+      }
+
+      res.json({ completedAt: now, ...completionResult });
     } catch (err) {
       const status = err.message.includes('not found') ? 404 : 400;
       res.status(status).json({ error: err.message });

package/templates/ROADMAP.md

@@ -5,6 +5,7 @@
 
 ---
 
+<!-- preserved-section: roadmap-conventions -->
 ## Roadmap Conventions
 
 - **Status:** `PLANNED` | `IN_PROGRESS` | `PARTIAL` | `COMPLETE` | `SUPERSEDED` | `PARKED`
@@ -12,6 +13,8 @@
 - Items are numbered sequentially across all phases — never reuse a number.
 - Cross-reference stable IDs (e.g. `FEAT-1`, `Phase 2`) not section headings.
 
+<!-- /preserved-section -->
+
 ---
 
 ## Phase 1: Foundation — PLANNED
@@ -37,6 +40,7 @@ Bootstrap: establish the core structure and first working milestone.
 
 ---
 
+<!-- preserved-section: dogfooding-milestones -->
 ## Dogfooding Milestones
 
 | Milestone | Description | Status |
@@ -44,3 +48,5 @@ Bootstrap: establish the core structure and first working milestone.
 | D0: Bootstrap | Manual, out-of-band. | PLANNED |
 | D1: [First self-use milestone] | [Description] | PLANNED |
 | D2: [Second self-use milestone] | [Description] | PLANNED |
+
+<!-- /preserved-section -->

package/dist/assets/channel-LRG9kHqJ.js

@@ -1 +0,0 @@
-import{aq as o,ar as n}from"./index-DkRKLuNr.js";const t=(r,a)=>o.lang.round(n.parse(r)[a]);export{t as c};

package/dist/assets/classDiagram-6PBFFD2Q-BC9a6pDE.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-4TB4RGXK-BdlMSdEA.js";import{_ as i}from"./index-DkRKLuNr.js";import"./chunk-FMBD7UC4-Cd6Hrux2.js";import"./chunk-YZCP3GAM-DFN9X99H.js";import"./chunk-55IACEB6-vrQHZTdv.js";import"./chunk-EDXVE4YY-B8wioVlW.js";var u={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{u as diagram};

package/dist/assets/classDiagram-v2-HSJHXN6E-BC9a6pDE.js

@@ -1 +0,0 @@
-import{s as a,c as s,a as e,C as t}from"./chunk-4TB4RGXK-BdlMSdEA.js";import{_ as i}from"./index-DkRKLuNr.js";import"./chunk-FMBD7UC4-Cd6Hrux2.js";import"./chunk-YZCP3GAM-DFN9X99H.js";import"./chunk-55IACEB6-vrQHZTdv.js";import"./chunk-EDXVE4YY-B8wioVlW.js";var u={parser:e,get db(){return new t},renderer:s,styles:a,init:i(r=>{r.class||(r.class={}),r.class.arrowMarkerAbsolute=r.arrowMarkerAbsolute},"init")};export{u as diagram};

package/dist/assets/clone-dRxgFrBv.js

@@ -1 +0,0 @@
-import{b as r}from"./graph-CRnO_ifT.js";var e=4;function a(o){return r(o,e)}export{a as c};

package/dist/assets/stateDiagram-v2-QKLJ7IA2-BW0ezXb4.js

@@ -1 +0,0 @@
-import{s as t,b as r,a,S as s}from"./chunk-OYMX7WX6-CfrhdQXY.js";import{_ as i}from"./index-DkRKLuNr.js";import"./chunk-55IACEB6-vrQHZTdv.js";import"./chunk-EDXVE4YY-B8wioVlW.js";var l={parser:a,get db(){return new s(2)},renderer:r,styles:t,init:i(e=>{e.state||(e.state={}),e.state.arrowMarkerAbsolute=e.arrowMarkerAbsolute},"init")};export{l as diagram};