@hone-ai/cli 1.25.0 → 1.26.0

package/hone-cli.js

@@ -17,6 +17,7 @@
  */
 
 const { Command } = require('commander');
+const { isEditorParked, editorParkedGuidance, currentStep, EDITOR_PARKED_EXIT_CODE } = require('./lib/editor-park'); // HC-COMM-010-followup-1
 const axios        = require('axios');
 const fs           = require('fs');
 const path         = require('path');
@@ -2385,6 +2386,19 @@ program
       const { isEditorStubbedAgent, editorStubContent } = require('./lib/editor-stub');
       const { readExecutionMode } = require('./lib/pipeline-config');
       const syncExecutionMode = readExecutionMode(process.cwd());
+      // HC-COMM-010-followup-3: in editor mode, register the Hone MCP server for
+      // Claude Code by writing a project-scoped .mcp.json (merge, never clobber).
+      // Replaces the manual `claude mcp add hone -- hone-mcp` runbook step.
+      try {
+        const { ensureClaudeMcpConfig, MCP_FILE, FALLBACK_LINE } = require('./lib/mcp-config');
+        const mcp = ensureClaudeMcpConfig(process.cwd(), { mode: syncExecutionMode });
+        if (mcp.action === 'written' || mcp.action === 'updated') console.log(`  ✓ Registered the Hone MCP server for Claude Code in ${MCP_FILE} (commit it; Claude Code asks once to approve project servers)`);
+        else if (mcp.action === 'skipped' && syncExecutionMode === 'editor') console.log(`  ⚠ ${mcp.reason}`);
+        else if (mcp.action === 'unchanged') console.log(`  ✓ Hone MCP server already registered in ${MCP_FILE}`);
+        void FALLBACK_LINE;
+      } catch (e) {
+        console.log(`  ⚠ MCP registration skipped (non-fatal): ${e.message}`);
+      }
 
       let synced = 0;
       for (const agent of agents) {
@@ -5535,7 +5549,7 @@ program
   .option('--status', 'Show status of the latest workflow run for this story')
   .option('--kill', 'Kill the latest workflow run for this story')
   .option('--axes <axes>', 'Comma-separated classifier axes for this run (e.g. shared-chokepoint) — composition input (HC-COMM-010 C0)')
-  .option('--approve <step>', 'Approve a paused gate (e.g., --approve step_0)')
+  .option('--approve <step>', 'Approve a paused gate (e.g., --approve step_0). Exit codes: 2 = paused at a human gate, 3 = parked for your editor (execution.mode: editor)')
   .option('--format <fmt>', 'Output format: pretty or json', 'pretty')
   .option('--poll-interval <s>', 'Poll interval in seconds', '5')
   .option('--include-paths <paths>', 'Comma-separated list of source file paths to bundle into the codebase context (HC-019n-followup-12). Augments auto-detected paths from the issue body.')
@@ -5569,9 +5583,15 @@ program
         for (const s of (data.steps || [])) {
           const icon = s.status === 'completed' ? '✓' : s.status === 'running' ? '⏳' : s.status === 'failed' ? '✗' : '⬜';
           const gate = s.gateResult ? ` (${s.gateResult})` : '';
-          console.log(`  ${icon} ${s.stepKey}  (${s.agent})  ${s.status}${gate}`);
+          // HC-019n-followup-40: one line per step = its CURRENT state; a retried
+          // step says so instead of listing a stale failed row above a completed one.
+          const tries = (s.attempts || 1) > 1 ? `  [attempt ${s.attempt} of ${s.attempts}]` : '';
+          console.log(`  ${icon} ${s.stepKey}  (${s.agent})  ${s.status}${gate}${tries}`);
         }
-        if (data.awaitingApproval) {
+        if (data.awaitingApproval && isEditorParked(data)) {
+          console.log('');
+          console.log(editorParkedGuidance(data.runId, data.currentStep, currentStep(data)?.agent));
+        } else if (data.awaitingApproval) {
           console.log('');
           console.log(`Awaiting approval at ${data.currentStep}.`);
           console.log(`Approve: hone run-story ${data.runId} --approve ${data.currentStep}`);
@@ -5877,6 +5897,37 @@ program
       console.warn('    Fix: hone run-story <id> --include-paths path/one.js,path/two.js');
     }
 
+    // HC-COMM-010 axes wiring (CLI half): when --axes wasn't given, derive
+    // the DETERMINISTIC axes from the extractors (SC-002) — security labels/
+    // text, cross-repo markers, recently-modified overlap vs the bundled
+    // paths. Judgment axes (shared-chokepoint etc.) come from the groomer at
+    // step_0 and are harvested server-side. Fail-open; explicit --axes wins.
+    if (!opts.axes) {
+      try {
+        const { classifyStory } = require('./lib/story-classifier');
+        const {
+          extractType, extractSecurityImplications, extractCrossRepoDependency, extractRecentlyModifiedOverlap,
+        } = require('./lib/story-classifier-extract');
+        const autoBody = issueBodyForFiles || '';
+        const autoLabels = [];
+        const touched = Array.isArray(orchestrateConfig.codebaseFiles)
+          ? orchestrateConfig.codebaseFiles.map(f => f.path).filter(Boolean)
+          : Object.keys(orchestrateConfig.codebaseFiles || {});
+        const auto = classifyStory({
+          type: extractType(autoLabels, autoBody, String(storyIdOrRunId)),
+          has_security_implications: extractSecurityImplications(autoLabels, autoBody),
+          cross_repo_dependency: extractCrossRepoDependency(autoBody),
+          recently_modified_overlap: extractRecentlyModifiedOverlap(process.cwd(), touched, 14),
+        });
+        if (auto.architect.engage && auto.architect.axes.length > 0) {
+          orchestrateConfig.classifier_axes = auto.architect.axes;
+          console.log(`  → auto-detected classifier axes: ${auto.architect.axes.join(', ')}`);
+        }
+      } catch (e) {
+        console.warn(`  ⚠ axes auto-detection failed (non-fatal): ${(e && e.message || String(e)).split('\n')[0]}`);
+      }
+    }
+
     // HC-101-followup-2: pass the adopter's CI gate config to the
     // orchestrator so step_5c can branch (github / local / both / none).
     // Defaults to gate=github + local_command='make ci' when the config
@@ -6049,6 +6100,12 @@ program
 
         if (status.status === 'paused' && status.awaitingApproval) {
           const step = status.steps.find(s => s.stepKey === status.currentStep);
+          // HC-COMM-010-followup-1: an editor-parked run is a gate the EDITOR
+          // satisfies, not a human — never offer --approve here (B4 refuses it).
+          if (isEditorParked(status)) {
+            console.log(editorParkedGuidance(data.runId, status.currentStep, step?.agent));
+            process.exit(EDITOR_PARKED_EXIT_CODE);
+          }
           console.log(`⏳ Step ${status.currentStep} (${step?.agent || 'unknown'}) — awaiting approval`);
 
           // Interactive prompt

package/lib/editor-park.js

@@ -0,0 +1,43 @@
+'use strict';
+/**
+ * editor-park.js — HC-COMM-010-followup-1: recognize a run that is PARKED FOR
+ * THE EDITOR (execution.mode: editor, step awaiting_editor) and say what to do.
+ *
+ * Why: the run-status endpoint reports an editor-parked run as
+ * `paused` + `awaitingApproval: true` (it IS at a gate, just not one a human
+ * approves) — so run-story's poll loop used to print "--approve" guidance,
+ * which the B4 guard then refuses. Detection keys on the CURRENT step's
+ * status, reliable since HC-019n-followup-40 collapsed steps[] to the latest
+ * attempt. Pure: no I/O, no process access — the CLI decides how to exit.
+ */
+const EDITOR_PARKED_EXIT_CODE = 3; // distinct from 2 (paused at a human gate)
+
+function currentStep(status) {
+  if (!status || !Array.isArray(status.steps)) return null;
+  return status.steps.find(s => s && s.stepKey === status.currentStep) || null;
+}
+
+/** @returns {boolean} true when the run is paused because the editor owes output */
+function isEditorParked(status) {
+  if (!status || status.status !== 'paused') return false;
+  const step = currentStep(status);
+  return !!(step && step.status === 'awaiting_editor');
+}
+
+/**
+ * The parked-state copy (ux-copy pass, 2026-09-05): what happened, where it is,
+ * the two ways forward (editor MCP tools / CLI verbs), and the closing step.
+ */
+function editorParkedGuidance(runId, stepKey, agent) {
+  const a = agent ? ` (${agent})` : '';
+  return [
+    `⏸ Parked for your editor at ${stepKey}${a} — this step runs in your editor AI, not on Hone.`,
+    `  In Claude Code:  call hone_step_instruction, build the step there, then hone_submit_step.`,
+    `  From the CLI:    hone step-instruction ${runId}`,
+    `                   … build … then: hone submit-step ${runId} ${stepKey} --file out.md`,
+    `  When submitted:  hone run-story ${runId} --approve ${stepKey}`,
+    `  Guide: docs/runbooks/editor-mode.md`,
+  ].join('\n');
+}
+
+module.exports = { EDITOR_PARKED_EXIT_CODE, isEditorParked, editorParkedGuidance, currentStep };

package/lib/mcp-config.js

@@ -0,0 +1,41 @@
+'use strict';
+/**
+ * mcp-config.js — HC-COMM-010-followup-3: register the Hone MCP server for
+ * Claude Code by writing a PROJECT-scoped `.mcp.json` when the repo runs in
+ * editor mode. Replaces the manual `claude mcp add hone -- hone-mcp` step.
+ *
+ * Contract: merge, never clobber. Other servers in an existing `.mcp.json` are
+ * preserved; only the `hone` entry is added/refreshed; a malformed file is left
+ * untouched (fail-open with a reason). Pure over an injected repoRoot.
+ */
+const fs = require('node:fs');
+const path = require('node:path');
+
+const MCP_FILE = '.mcp.json';
+const HONE_SERVER = Object.freeze({ command: 'hone-mcp', args: [] });
+const FALLBACK_LINE = 'claude mcp add hone -- hone-mcp';
+
+function ensureClaudeMcpConfig(repoRoot, { mode } = {}) {
+  if (mode !== 'editor') return { action: 'skipped', reason: 'execution.mode is not editor' };
+  const file = path.join(repoRoot, MCP_FILE);
+  let doc = { mcpServers: {} };
+  if (fs.existsSync(file)) {
+    try {
+      const parsed = JSON.parse(fs.readFileSync(file, 'utf8'));
+      if (!parsed || typeof parsed !== 'object' || Array.isArray(parsed)) throw new Error('not an object');
+      doc = parsed;
+      if (!doc.mcpServers || typeof doc.mcpServers !== 'object' || Array.isArray(doc.mcpServers)) doc.mcpServers = {};
+    } catch (e) {
+      return { action: 'skipped', reason: `${MCP_FILE} is not valid JSON (${e.message}); left untouched — register manually: ${FALLBACK_LINE}` };
+    }
+  }
+  const existing = doc.mcpServers.hone;
+  if (existing && existing.command === HONE_SERVER.command && JSON.stringify(existing.args || []) === '[]') {
+    return { action: 'unchanged', file };
+  }
+  doc.mcpServers.hone = { ...HONE_SERVER, args: [] };
+  fs.writeFileSync(file, JSON.stringify(doc, null, 2) + '\n');
+  return { action: existing ? 'updated' : 'written', file };
+}
+
+module.exports = { ensureClaudeMcpConfig, MCP_FILE, HONE_SERVER, FALLBACK_LINE };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hone-ai/cli",
-  "version": "1.25.0",
+  "version": "1.26.0",
   "description": "Hone AI — Enterprise SDLC Pipeline CLI",
   "main": "hone-cli.js",
   "bin": {