@harness-engineering/cli 2.7.0 → 2.8.0

package/dist/agents/skills/tests/harness-strategy.test.ts

@@ -0,0 +1,145 @@
+// agents/skills/tests/harness-strategy.test.ts
+//
+// Contract tests for the harness-strategy skill. The skill prose itself is
+// executed by an agent (not by node), so behavioral correctness is
+// human-judged in practice. Phase 2 of the strategic-anchor spec owns:
+//   - the three pushback rules are *named and cited* in references/interview.md
+//   - the 2-round cap is documented
+//   - the anti-pattern fixtures exist for each rule
+//   - the writer integration is documented in SKILL.md
+//
+// Generic structure, schema, and platform-parity coverage lives in sibling files
+// (structure.test.ts, schema.test.ts, references.test.ts, platform-parity.test.ts).
+
+import { describe, it, expect } from 'vitest';
+import { readFileSync, existsSync } from 'fs';
+import { resolve, dirname } from 'path';
+import { fileURLToPath } from 'url';
+import { ALLOWED_PLATFORMS } from './schema';
+
+const __dirname = dirname(fileURLToPath(import.meta.url));
+const SKILLS_DIR = resolve(__dirname, '..');
+
+const SKILL_NAME = 'harness-strategy';
+
+function skillFile(platform: string, file: string): string {
+  return resolve(SKILLS_DIR, platform, SKILL_NAME, file);
+}
+
+describe('harness-strategy skill — Phase 2 contract', () => {
+  describe('skill files exist in every supported platform', () => {
+    for (const platform of ALLOWED_PLATFORMS) {
+      it(`${platform}/${SKILL_NAME}/SKILL.md exists`, () => {
+        expect(existsSync(skillFile(platform, 'SKILL.md'))).toBe(true);
+      });
+      it(`${platform}/${SKILL_NAME}/skill.yaml exists`, () => {
+        expect(existsSync(skillFile(platform, 'skill.yaml'))).toBe(true);
+      });
+      it(`${platform}/${SKILL_NAME}/references/interview.md exists`, () => {
+        expect(existsSync(skillFile(platform, 'references/interview.md'))).toBe(true);
+      });
+    }
+  });
+
+  describe('references/interview.md documents the three pushback rules', () => {
+    const interview = readFileSync(skillFile('claude-code', 'references/interview.md'), 'utf-8');
+
+    it('names "Fluff detection"', () => {
+      expect(interview).toMatch(/Fluff detection/);
+    });
+
+    it('names "Goal-as-strategy"', () => {
+      expect(interview).toMatch(/Goal-as-strategy/);
+    });
+
+    it('names "Feature-list-as-strategy"', () => {
+      expect(interview).toMatch(/Feature-list-as-strategy/);
+    });
+
+    it('includes a repair script keyword for each rule', () => {
+      // Each rule's repair script anchors on a specific keyword the agent
+      // quotes when pushing back. Test these by name to keep prose tweaks
+      // free while catching accidental rule deletion.
+      expect(interview).toMatch(/concrete diagnosis/i);
+      expect(interview).toMatch(/\bbet\b/);
+      expect(interview).toMatch(/coherent action/i);
+    });
+
+    it('documents the 2-round cap explicitly', () => {
+      expect(interview).toMatch(/2[ -]?[Rr]ound|two[ -]?round|AT MOST TWICE|at most twice/);
+      expect(interview).toMatch(/cap/i);
+    });
+
+    it('documents separation from docs/roadmap.md', () => {
+      expect(interview).toMatch(/docs\/roadmap\.md/);
+      expect(interview).toMatch(/[Dd]ecision 1/);
+    });
+
+    it('includes anti-pattern fixtures for each rule', () => {
+      expect(interview).toMatch(/Fluff detection fixture/);
+      expect(interview).toMatch(/Goal-as-strategy fixture/);
+      expect(interview).toMatch(/Feature-list-as-strategy fixture/);
+    });
+  });
+
+  describe('SKILL.md documents the writer integration', () => {
+    const skillMd = readFileSync(skillFile('claude-code', 'SKILL.md'), 'utf-8');
+
+    it('references writeStrategyDoc as the sanctioned write path', () => {
+      expect(skillMd).toMatch(/writeStrategyDoc/);
+    });
+
+    it('references @harness-engineering/core as the import source', () => {
+      expect(skillMd).toMatch(/@harness-engineering\/core/);
+    });
+
+    it('documents the stdin-piped Node one-liner pattern', () => {
+      // The pattern protects against shell injection from user-supplied
+      // section bodies. SKILL.md must show the stdin route, not a shell-arg
+      // route, so the agent doesn't paste user input into the command line.
+      expect(skillMd).toMatch(/readFileSync\(0/);
+      expect(skillMd).toMatch(/JSON\.parse/);
+    });
+
+    it('documents Phase 0 routing for present-but-invalid files', () => {
+      expect(skillMd).toMatch(/present-but-invalid|present and invalid/i);
+      expect(skillMd).toMatch(/STRATEGY\.md\.bak\.[<{]/);
+    });
+
+    it('documents the four downstream consumers in the handoff phase', () => {
+      expect(skillMd).toMatch(/harness-brainstorming/);
+      expect(skillMd).toMatch(/harness-ideate/);
+      expect(skillMd).toMatch(/harness-roadmap-pilot/);
+      expect(skillMd).toMatch(/BusinessKnowledgeIngestor/);
+    });
+  });
+
+  describe('skill.yaml metadata', () => {
+    const yamlPath = skillFile('claude-code', 'skill.yaml');
+    const yaml = readFileSync(yamlPath, 'utf-8');
+
+    it('declares type: rigid (configuration-interviewer)', () => {
+      expect(yaml).toMatch(/type:\s*rigid/);
+      expect(yaml).toMatch(/cognitive_mode:\s*configuration-interviewer/);
+    });
+
+    it('declares the four required platforms', () => {
+      expect(yaml).toMatch(/claude-code/);
+      expect(yaml).toMatch(/gemini-cli/);
+      expect(yaml).toMatch(/cursor/);
+      expect(yaml).toMatch(/codex/);
+    });
+
+    it('declares manual-only trigger (strategy is never auto-fired)', () => {
+      expect(yaml).toMatch(/triggers:\s*\n\s*-\s*manual\s*\n/);
+      // No other triggers allowed — strategy is human-owned.
+      expect(yaml).not.toMatch(/-\s*on_/);
+    });
+
+    it('declares persistent state with STRATEGY.md and .bak files', () => {
+      expect(yaml).toMatch(/persistent:\s*true/);
+      expect(yaml).toMatch(/STRATEGY\.md\s*$/m);
+      expect(yaml).toMatch(/STRATEGY\.md\.bak/);
+    });
+  });
+});

package/dist/agents-md-WMJO5HSM.js

@@ -0,0 +1,13 @@
+import {
+  generateAgentsMd
+} from "./chunk-7EQAZKJ5.js";
+import "./chunk-GOHWCHCS.js";
+import "./chunk-N55WOGN3.js";
+import "./chunk-BDGTZRZ6.js";
+import "./chunk-PP6ZRL5T.js";
+import "./chunk-QCLOIUMX.js";
+import "./chunk-XARGEOJV.js";
+import "./chunk-KFQGP6VL.js";
+export {
+  generateAgentsMd
+};

package/dist/analyzer-V633GUHY-MBCQWKDU.js

@@ -0,0 +1,26 @@
+import {
+  EntropyAnalyzer,
+  init_chunk_MUWJHO2S,
+  init_chunk_S4DTDJ6J
+} from "./chunk-N55WOGN3.js";
+import "./chunk-BDGTZRZ6.js";
+import "./chunk-PP6ZRL5T.js";
+import {
+  init_chunk_7P6ASYW6
+} from "./chunk-XARGEOJV.js";
+import {
+  __esm
+} from "./chunk-KFQGP6VL.js";
+
+// ../core/dist/analyzer-V633GUHY.mjs
+var init_analyzer_V633GUHY = __esm({
+  "../core/dist/analyzer-V633GUHY.mjs"() {
+    init_chunk_S4DTDJ6J();
+    init_chunk_MUWJHO2S();
+    init_chunk_7P6ASYW6();
+  }
+});
+init_analyzer_V633GUHY();
+export {
+  EntropyAnalyzer
+};

package/dist/{architecture-ZTEHGIHD.js → architecture-EVYVGPRZ.js}

@@ -1,14 +1,14 @@
 import {
   checkDependenciesDefinition,
   handleCheckDependencies
-} from "./chunk-6T5AYG5S.js";
-import "./chunk-VG6UK6G6.js";
+} from "./chunk-UJFNIWC7.js";
+import "./chunk-HRSE6IQD.js";
 import "./chunk-RC3OI5NK.js";
 import "./chunk-W6Y7ZW3Y.js";
-import "./chunk-IY2TZLY5.js";
-import "./chunk-W3PDVZ4I.js";
-import "./chunk-OH3NXXUY.js";
-import "./chunk-H2ZJOJ7A.js";
+import "./chunk-GOHWCHCS.js";
+import "./chunk-N55WOGN3.js";
+import "./chunk-BDGTZRZ6.js";
+import "./chunk-PP6ZRL5T.js";
 import "./chunk-QCLOIUMX.js";
 import "./chunk-XARGEOJV.js";
 import "./chunk-KFQGP6VL.js";

package/dist/{assess-project-OKDW3YSK.js → assess-project-KZG563SO.js}

@@ -2,7 +2,7 @@ import {
   assessProjectDefinition,
   handleAssessProject,
   parseToolResponse
-} from "./chunk-TIZ3L6CU.js";
+} from "./chunk-FZ5MBXEG.js";
 import "./chunk-RC3OI5NK.js";
 import "./chunk-W6Y7ZW3Y.js";
 import "./chunk-KFQGP6VL.js";

package/dist/bin/harness-mcp.js

@@ -1,33 +1,33 @@
 #!/usr/bin/env node
 import {
   startServer
-} from "../chunk-4EGPFB7V.js";
-import "../chunk-67RP7MLR.js";
-import "../chunk-HMMSWRWR.js";
-import "../chunk-TIZ3L6CU.js";
-import "../chunk-QUPC37UB.js";
-import "../chunk-CXAHK5PJ.js";
+} from "../chunk-CAQPSFPT.js";
+import "../chunk-KBCPELBC.js";
+import "../chunk-LADPYELQ.js";
+import "../chunk-FZ5MBXEG.js";
+import "../chunk-ACOOG3UW.js";
+import "../chunk-UUO4K7NX.js";
 import "../chunk-IOW3MW2K.js";
-import "../chunk-Z2GUITAS.js";
+import "../chunk-UXPWSV3B.js";
 import "../chunk-3ISINLYT.js";
-import "../chunk-TIH53QXY.js";
-import "../chunk-MQG7FHXX.js";
-import "../chunk-6T5AYG5S.js";
-import "../chunk-VG6UK6G6.js";
-import "../chunk-P7EO7USC.js";
+import "../chunk-CLI4K2CH.js";
+import "../chunk-KI6PHH6Q.js";
+import "../chunk-UJFNIWC7.js";
+import "../chunk-HRSE6IQD.js";
+import "../chunk-C7J55C6E.js";
 import "../chunk-RC3OI5NK.js";
 import "../chunk-W6Y7ZW3Y.js";
 import "../chunk-27AJKSQY.js";
 import "../chunk-6B6UN6SG.js";
 import "../chunk-HIK6OEUF.js";
-import "../chunk-B4OLYXZ4.js";
+import "../chunk-B7KMTHN3.js";
 import "../chunk-ZCZD7FHU.js";
 import "../chunk-EBJQ6N4M.js";
 import "../chunk-3WGJMBKH.js";
-import "../chunk-IY2TZLY5.js";
-import "../chunk-W3PDVZ4I.js";
-import "../chunk-OH3NXXUY.js";
-import "../chunk-H2ZJOJ7A.js";
+import "../chunk-GOHWCHCS.js";
+import "../chunk-N55WOGN3.js";
+import "../chunk-BDGTZRZ6.js";
+import "../chunk-PP6ZRL5T.js";
 import "../chunk-QCLOIUMX.js";
 import "../chunk-XARGEOJV.js";
 import "../chunk-KFQGP6VL.js";

package/dist/bin/harness.js

@@ -3,38 +3,38 @@ import {
   createProgram,
   installCommandTelemetry,
   printFirstRunWelcome
-} from "../chunk-HLPY3RHE.js";
-import "../chunk-S7GEONXL.js";
-import "../chunk-OQE6ME2Z.js";
-import "../chunk-B3B64OPD.js";
-import "../chunk-PARV3G2F.js";
-import "../chunk-ZKA3TGYS.js";
+} from "../chunk-4BDOSCNY.js";
+import "../chunk-5XL5QJ5B.js";
+import "../chunk-7EQAZKJ5.js";
+import "../chunk-STESM73J.js";
+import "../chunk-VBNJAKUH.js";
+import "../chunk-EEFRPEWT.js";
 import "../chunk-KET4QQZB.js";
-import "../chunk-6UHDQP2N.js";
-import "../chunk-WOXR6I3R.js";
+import "../chunk-2N4OENGE.js";
+import "../chunk-UBZQE3JN.js";
 import "../chunk-CQ553GZO.js";
 import "../chunk-ZP5E7YET.js";
-import "../chunk-XTIPZUPR.js";
+import "../chunk-X4EFYVCZ.js";
 import "../chunk-GISMXMVL.js";
 import "../chunk-WWACIIBA.js";
 import "../chunk-TEFCFC4H.js";
-import "../chunk-OLTVDPA2.js";
+import "../chunk-2SNRCAC3.js";
 import {
   dispatchSkillsFromGit
-} from "../chunk-4EGPFB7V.js";
-import "../chunk-67RP7MLR.js";
-import "../chunk-HMMSWRWR.js";
-import "../chunk-TIZ3L6CU.js";
-import "../chunk-QUPC37UB.js";
-import "../chunk-CXAHK5PJ.js";
+} from "../chunk-CAQPSFPT.js";
+import "../chunk-KBCPELBC.js";
+import "../chunk-LADPYELQ.js";
+import "../chunk-FZ5MBXEG.js";
+import "../chunk-ACOOG3UW.js";
+import "../chunk-UUO4K7NX.js";
 import "../chunk-IOW3MW2K.js";
-import "../chunk-Z2GUITAS.js";
+import "../chunk-UXPWSV3B.js";
 import "../chunk-3ISINLYT.js";
-import "../chunk-TIH53QXY.js";
-import "../chunk-MQG7FHXX.js";
-import "../chunk-6T5AYG5S.js";
-import "../chunk-VG6UK6G6.js";
-import "../chunk-P7EO7USC.js";
+import "../chunk-CLI4K2CH.js";
+import "../chunk-KI6PHH6Q.js";
+import "../chunk-UJFNIWC7.js";
+import "../chunk-HRSE6IQD.js";
+import "../chunk-C7J55C6E.js";
 import "../chunk-RC3OI5NK.js";
 import "../chunk-W6Y7ZW3Y.js";
 import "../chunk-27AJKSQY.js";
@@ -43,7 +43,7 @@ import "../chunk-HIK6OEUF.js";
 import {
   findConfigFile,
   loadConfig
-} from "../chunk-B4OLYXZ4.js";
+} from "../chunk-B7KMTHN3.js";
 import "../chunk-ZCZD7FHU.js";
 import "../chunk-EBJQ6N4M.js";
 import {
@@ -56,13 +56,13 @@ import {
   readCheckState,
   shouldRunCheck,
   spawnBackgroundCheck
-} from "../chunk-IY2TZLY5.js";
+} from "../chunk-GOHWCHCS.js";
 import {
   CLI_VERSION
 } from "../chunk-BM3PWGXQ.js";
-import "../chunk-W3PDVZ4I.js";
-import "../chunk-OH3NXXUY.js";
-import "../chunk-H2ZJOJ7A.js";
+import "../chunk-N55WOGN3.js";
+import "../chunk-BDGTZRZ6.js";
+import "../chunk-PP6ZRL5T.js";
 import "../chunk-QCLOIUMX.js";
 import "../chunk-XARGEOJV.js";
 import "../chunk-KFQGP6VL.js";

package/dist/{check-phase-gate-YJWFDW66.js → check-phase-gate-MK364VXS.js}

@@ -1,15 +1,15 @@
 import {
   createCheckPhaseGateCommand,
   runCheckPhaseGate
-} from "./chunk-OLTVDPA2.js";
-import "./chunk-B4OLYXZ4.js";
+} from "./chunk-2SNRCAC3.js";
+import "./chunk-B7KMTHN3.js";
 import "./chunk-ZCZD7FHU.js";
 import "./chunk-EBJQ6N4M.js";
 import "./chunk-3WGJMBKH.js";
-import "./chunk-IY2TZLY5.js";
-import "./chunk-W3PDVZ4I.js";
-import "./chunk-OH3NXXUY.js";
-import "./chunk-H2ZJOJ7A.js";
+import "./chunk-GOHWCHCS.js";
+import "./chunk-N55WOGN3.js";
+import "./chunk-BDGTZRZ6.js";
+import "./chunk-PP6ZRL5T.js";
 import "./chunk-QCLOIUMX.js";
 import "./chunk-XARGEOJV.js";
 import "./chunk-KFQGP6VL.js";

package/dist/{chunk-6UHDQP2N.js → chunk-2N4OENGE.js}

@@ -41,7 +41,7 @@ async function runScan(projectPath) {
     KnowledgeIngestor,
     GitIngestor,
     RequirementIngestor
-  } = await import("./dist-R3TOOPJ7.js");
+  } = await import("./dist-OWHMNL4W.js");
   const store = new GraphStore();
   const start = Date.now();
   const ingestOptions = loadIngestOptions(projectPath);

package/dist/{chunk-OLTVDPA2.js → chunk-2SNRCAC3.js}

@@ -1,6 +1,6 @@
 import {
   resolveConfig
-} from "./chunk-B4OLYXZ4.js";
+} from "./chunk-B7KMTHN3.js";
 import {
   logger
 } from "./chunk-EBJQ6N4M.js";
@@ -9,10 +9,10 @@ import {
 } from "./chunk-3WGJMBKH.js";
 import {
   init_dist
-} from "./chunk-IY2TZLY5.js";
+} from "./chunk-GOHWCHCS.js";
 import {
   Ok
-} from "./chunk-OH3NXXUY.js";
+} from "./chunk-BDGTZRZ6.js";
 
 // src/commands/check-phase-gate.ts
 init_dist();