@hone-ai/cli 1.11.0 → 1.12.1

package/hone-cli.js

@@ -1087,13 +1087,24 @@ program
       // Write generated skills to disk
       console.log('Writing generated skills...');
       if (result.skills) {
+        let preservedCount = 0;
         for (const [skillName, content] of Object.entries(result.skills)) {
           if (!content || content.length < 50) continue;
           const skillDir  = path.join(repoRoot, '.github', 'skills', skillName);
           const skillFile = path.join(skillDir, 'SKILL.md');
           fs.mkdirSync(skillDir, { recursive: true });
-          fs.writeFileSync(skillFile, content);
-          console.log(`  ✓ .github/skills/${skillName}/SKILL.md`);
+          const { merged, preserved } = splicePreserveRepoSpecific(skillFile, content);
+          fs.writeFileSync(skillFile, merged);
+          if (preserved) {
+            preservedCount++;
+            console.log(`  ✓ .github/skills/${skillName}/SKILL.md (preserved adopter REPO-SPECIFIC content)`);
+          } else {
+            console.log(`  ✓ .github/skills/${skillName}/SKILL.md`);
+          }
+        }
+        if (preservedCount > 0) {
+          console.log(`\n  Preserved adopter REPO-SPECIFIC content in ${preservedCount} skill(s).`);
+          console.log(`  (HC-019y-followup-1: derive no longer overwrites curated REPO-SPECIFIC sections)`);
         }
       }
 
@@ -2599,6 +2610,14 @@ function sleep(ms) {
   return new Promise(resolve => setTimeout(resolve, ms));
 }
 
+// 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).
 // Reads .github/pipeline/<STORY-ID>/metadata.yml + filesystem; uses

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.11.0",
+  "version": "1.12.1",
   "description": "Hone AI — Enterprise SDLC Pipeline CLI",
   "main": "hone-cli.js",
   "bin": {