@dzhechkov/harness-cli 0.3.230 → 0.3.232

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dzhechkov/harness-cli",
-  "version": "0.3.230",
+  "version": "0.3.232",
   "description": "The dz CLI — install AI skills for Claude Code, Codex, OpenCode, Hermes, OpenClaude, GitHub Copilot. 35 commands, 13 presets, 6 platform targets.",
   "type": "module",
   "license": "MIT",
@@ -55,7 +55,7 @@
     "@dzhechkov/skills-reverse-engineering": "^0.1.0",
     "@dzhechkov/skills-presentation-storyteller": "^0.1.0",
     "@dzhechkov/skills-website-cloner": "^0.1.0",
-    "@dzhechkov/harness-core": "0.3.122"
+    "@dzhechkov/harness-core": "0.3.124"
   },
   "devDependencies": {
     "@types/node": "^25.6.0",

package/src/cli.ts

@@ -4282,6 +4282,21 @@ function gatherGuardFacts(op: string, root: string, text: string | undefined, st
     facts['packages'] = packages;
     try { facts['drift'] = sweepSkillDrift(root, { scope: 'packages', allowlist: readDriftAllowlist(root) }).drifted.map((d) => d.name); } catch { /* skip */ }
     facts['counts'] = gatherReadmeCounts(root);
+    // readme-first: from the WORKING-TREE diff (publishes happen pre-commit here), per package: does the
+    // change set contain its package.json (the version-bump signal) without its README.md?
+    try {
+      const status = execSync('git status --porcelain -- "packages/@dzhechkov/"', { cwd: root, encoding: 'utf-8' });
+      const perPack = new Map<string, { pkgJson: boolean; readme: boolean }>();
+      for (const line of status.split('\n')) {
+        const m = line.match(/packages\/@dzhechkov\/([^/]+)\/(.+)$/);
+        if (!m || !m[1] || !m[2]) continue;
+        const e = perPack.get(m[1]) ?? { pkgJson: false, readme: false };
+        if (m[2] === 'package.json') e.pkgJson = true;
+        if (m[2] === 'README.md') e.readme = true;
+        perPack.set(m[1], e);
+      }
+      facts['readmeFirst'] = [...perPack.entries()].map(([name, e]) => ({ name: '@dzhechkov/' + name, versionBumped: e.pkgJson, readmeChanged: e.readme }));
+    } catch { /* not a git repo — rule skips */ }
   }
   if (op === 'consolidate') {
     try { facts['drift'] = sweepSkillDrift(root, { scope: 'packages', allowlist: readDriftAllowlist(root) }).drifted.map((d) => d.name); } catch { /* skip */ }
@@ -4767,10 +4782,21 @@ async function cmdRetro(options: Map<string, string>, flags: Set<string>, cwd: s
  */
 function cmdFeatureAdrSetup(options: Map<string, string>, flags: Set<string>, cwd: string, write: Write): number {
   let repoRoot = cwd;
-  try { repoRoot = execSync('git rev-parse --show-toplevel', { cwd, encoding: 'utf-8' }).trim() || cwd; } catch { /* not git */ }
+  try { repoRoot = execSync('git rev-parse --show-toplevel', { cwd, encoding: 'utf-8', stdio: ['ignore', 'pipe', 'ignore'] }).trim() || cwd; } catch { /* not git */ }
+
+  // P3 (fa-improvements): --guards scaffolds the deterministic guard set (guards.config.json + the
+  // zero-dependency check.mjs runner) — usable standalone (`dz feature-adr-setup --guards --apply`) or
+  // together with --from-spec. --loc-cap <n> tunes the LOC cap (default 700).
+  const wantGuards = flags.has('guards');
+  const locCapRaw = options.get('loc-cap');
+  const locCap = locCapRaw !== undefined ? Number(locCapRaw) : undefined;
+  if (locCapRaw !== undefined && (!Number.isFinite(locCap) || (locCap as number) <= 0)) {
+    write('dz feature-adr-setup: --loc-cap must be a positive finite number');
+    return 1;
+  }
 
   const specPath = options.get('from-spec');
-  if (specPath === undefined) {
+  if (specPath === undefined && !wantGuards) {
     // default + --plan: the read-only "which documents, and where?" answer.
     const plan = buildSetupPlan(scanForSetup(repoRoot));
     if (flags.has('json')) { write(JSON.stringify(plan, null, 2)); return 0; }
@@ -4783,10 +4809,15 @@ function cmdFeatureAdrSetup(options: Map<string, string>, flags: Set<string>, cw
     return 0;
   }
 
-  // --from-spec: scaffold (preview, or --apply to write).
+  // --from-spec and/or --guards: scaffold (preview, or --apply to write).
   let spec: SetupSpec;
-  try { spec = JSON.parse(readFileSync(resolve(cwd, specPath), 'utf-8')) as SetupSpec; }
-  catch (e) { write(`dz feature-adr-setup: cannot read spec ${specPath}: ${e instanceof Error ? e.message : String(e)}`); return 1; }
+  if (specPath !== undefined) {
+    try { spec = JSON.parse(readFileSync(resolve(cwd, specPath), 'utf-8')) as SetupSpec; }
+    catch (e) { write(`dz feature-adr-setup: cannot read spec ${specPath}: ${e instanceof Error ? e.message : String(e)}`); return 1; }
+  } else {
+    spec = {}; // --guards standalone: scaffold just the guard set
+  }
+  if (wantGuards) spec = { ...spec, guards: locCap !== undefined ? { locCap } : true };
 
   const result = scaffoldFromSpec(spec, readExistingForScaffold(repoRoot));
   if (flags.has('json') && !flags.has('apply')) { write(JSON.stringify(result, null, 2)); return 0; }