@hone-ai/cli 1.12.0 → 1.12.2

package/hone-cli.js

@@ -1035,6 +1035,11 @@ program
         docUrls,
         isRefresh,
         existingSkills,
+        // HC-019z-followup-2: derive is always batch from the CLI —
+        // there's no channel to relay user answers to the LLM mid-job.
+        // The server's prompt Step 0c uses this to emit STEP_0C_SKIPPED
+        // instead of stopping to ask, when no team docs are configured.
+        interactive: false,
       });
       jobId = r.data.jobId;
       console.log(`Job queued: ${jobId}`);
@@ -1084,6 +1089,25 @@ program
       fs.writeFileSync(reportPath, result.changeReport || '');
       console.log(`\nChange report saved to: .github/skill-refresh-report.md`);
     } else {
+      // HC-019z-followup-2: surface the non-interactive skip notice if
+      // the prompt emitted STEP_0C_SKIPPED. The marker appears in the
+      // server's rawOutputTail when the LLM honored the non-interactive
+      // guard. Without this message, the user is mystified by an empty
+      // skill set.
+      const rawOutput = result.rawOutputTail || result.rawOutput || '';
+      if (rawOutput.includes('STEP_0C_SKIPPED')) {
+        console.log('');
+        console.log('⚠  Phase 0c (team documentation URLs) skipped — derive is non-interactive.');
+        console.log('   To include team docs (Confluence, wiki, runbooks, etc.) in derivation,');
+        console.log('   add them to .github/.pipeline-config.yml under documentation.external_urls:');
+        console.log('     documentation:');
+        console.log('       external_urls:');
+        console.log('         - https://yourwiki.example/architecture');
+        console.log('         - https://yourwiki.example/conventions');
+        console.log('   Then re-run `hone derive`. (HC-019z-followup-2)');
+        console.log('');
+      }
+
       // Write generated skills to disk
       console.log('Writing generated skills...');
       if (result.skills) {
@@ -2610,49 +2634,13 @@ function sleep(ms) {
   return new Promise(resolve => setTimeout(resolve, ms));
 }
 
-/**
- * HC-019y-followup-1: when `hone derive` writes a SKILL.md, splice the
- * existing adopter REPO-SPECIFIC content over whatever the LLM produced
- * for that section. The semantic guarantee: adopter-curated content
- * (everything below the `<!-- REPO-SPECIFIC -->` marker) is NEVER lost
- * across derive runs.
- *
- * Pre-fix: derive wrote the file blind (fs.writeFileSync(skillFile,
- * content)). 5 OptionsFlow regression tests broke after a single derive
- * because 378 lines of E33-A/E34-B-added content disappeared.
- *
- * Returns { merged, preserved }:
- *   - merged:    the bytes to write (always a string)
- *   - preserved: true if adopter REPO-SPECIFIC content was preserved
- *                across the splice; false for fresh writes
- */
-function splicePreserveRepoSpecific(existingPath, newContent) {
-  const MARKER = '<!-- REPO-SPECIFIC';
-  if (!fs.existsSync(existingPath)) {
-    // Fresh adopter — no existing file. Write new content as-is.
-    return { merged: newContent, preserved: false };
-  }
-  const existing = fs.readFileSync(existingPath, 'utf8');
-  const oldIdx = existing.indexOf(MARKER);
-  if (oldIdx < 0) {
-    // Existing file has no REPO-SPECIFIC marker (likely a non-skill or
-    // legacy file). Nothing to preserve.
-    return { merged: newContent, preserved: false };
-  }
-  const newIdx = newContent.indexOf(MARKER);
-  if (newIdx < 0) {
-    // New content has no REPO-SPECIFIC section. Append the existing one.
-    return { merged: newContent.trimEnd() + '\n\n' + existing.slice(oldIdx), preserved: true };
-  }
-  // Both have REPO-SPECIFIC. Take new content up to the marker, then
-  // append everything from existing's marker to EOF. This preserves
-  // adopter-curated content while updating the body before the marker.
-  return {
-    merged: newContent.slice(0, newIdx) + existing.slice(oldIdx),
-    preserved: true,
-  };
-}
-module.exports.splicePreserveRepoSpecific = splicePreserveRepoSpecific;
+// HC-019y-followup-1: splice helper extracted to cli/lib/ so tests
+// can require it without dragging in the commander entry point.
+// (Previous attempt to gate `program.parse` behind `require.main ===
+// module` broke the CLI in 1.12.0 because bin/hone.js does
+// `require('../hone-cli.js')` — making require.main the bin wrapper,
+// not hone-cli.js, so the gate never fired and parse never ran.)
+const { splicePreserveRepoSpecific } = require('./lib/splice-repo-specific');
 
 // ── STATUS command (H-009) ────────────────────────────────────────────────────
 // Show pipeline state for a story (or use --all for every in-flight story).
@@ -5295,9 +5283,4 @@ program
   .description('Hone AI — Enterprise SDLC Pipeline CLI')
   .version(pkg.version);
 
-// HC-019y-followup-1: gate commander invocation so the file can be
-// require()'d by tests for helpers like splicePreserveRepoSpecific
-// without auto-running the CLI's --help.
-if (require.main === module) {
-  program.parse(process.argv);
-}
+program.parse(process.argv);

package/lib/splice-repo-specific.js

@@ -0,0 +1,61 @@
+'use strict';
+/**
+ * HC-019y-followup-1: when `hone derive` writes a SKILL.md, splice the
+ * existing adopter REPO-SPECIFIC content over whatever the LLM produced
+ * for that section. The semantic guarantee: adopter-curated content
+ * (everything below the `<!-- REPO-SPECIFIC -->` marker) is NEVER lost
+ * across derive runs.
+ *
+ * Pre-fix: derive wrote the file blind (fs.writeFileSync(skillFile,
+ * content)). 5 OptionsFlow regression tests broke after a single derive
+ * because 378 lines of E33-A/E34-B-added content disappeared.
+ *
+ * Extracted to its own file (cli/lib/) so it can be require()'d by
+ * tests without dragging in cli/hone-cli.js's commander entry point.
+ * The earlier attempt to gate `program.parse` behind
+ * `require.main === module` broke the CLI in production because
+ * bin/hone.js calls `require('../hone-cli.js')` — making require.main
+ * the bin/hone.js wrapper, not hone-cli.js itself, so the gate never
+ * fired and `program.parse` never ran. Lesson: don't gate side-effect
+ * top-level code; extract testable helpers to their own modules.
+ */
+const fs = require('node:fs');
+
+/**
+ * Splice adopter REPO-SPECIFIC tail into derive output.
+ *
+ * @param {string} existingPath  absolute path of the file being overwritten
+ * @param {string} newContent    the content from `hone derive` server response
+ * @returns {{ merged: string, preserved: boolean }}
+ *   merged:    bytes to write
+ *   preserved: true if adopter REPO-SPECIFIC content was preserved
+ *              across the splice; false for fresh writes
+ */
+function splicePreserveRepoSpecific(existingPath, newContent) {
+  const MARKER = '<!-- REPO-SPECIFIC';
+  if (!fs.existsSync(existingPath)) {
+    // Fresh adopter — no existing file. Write new content as-is.
+    return { merged: newContent, preserved: false };
+  }
+  const existing = fs.readFileSync(existingPath, 'utf8');
+  const oldIdx = existing.indexOf(MARKER);
+  if (oldIdx < 0) {
+    // Existing file has no REPO-SPECIFIC marker (likely a non-skill or
+    // legacy file). Nothing to preserve.
+    return { merged: newContent, preserved: false };
+  }
+  const newIdx = newContent.indexOf(MARKER);
+  if (newIdx < 0) {
+    // New content has no REPO-SPECIFIC section. Append the existing one.
+    return { merged: newContent.trimEnd() + '\n\n' + existing.slice(oldIdx), preserved: true };
+  }
+  // Both have REPO-SPECIFIC. Take new content up to the marker, then
+  // append everything from existing's marker to EOF. This preserves
+  // adopter-curated content while updating the body before the marker.
+  return {
+    merged: newContent.slice(0, newIdx) + existing.slice(oldIdx),
+    preserved: true,
+  };
+}
+
+module.exports = { splicePreserveRepoSpecific };

package/package.json

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