@shrkcrft/cli 0.1.0-alpha.29 → 0.1.0-alpha.31

package/dist/commands/drift.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"drift.command.d.ts","sourceRoot":"","sources":["../../src/commands/drift.command.ts"],"names":[],"mappings":"AAUA,OAAO,EAKL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAOhC,eAAO,MAAM,YAAY,EAAE,eA4E1B,CAAC"}
+{"version":3,"file":"drift.command.d.ts","sourceRoot":"","sources":["../../src/commands/drift.command.ts"],"names":[],"mappings":"AAWA,OAAO,EAKL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAOhC,eAAO,MAAM,YAAY,EAAE,eAuF1B,CAAC"}

package/dist/commands/drift.command.js

@@ -1,3 +1,4 @@
+import { verdictLine } from "../gates/verdict-line.js";
 import { buildDriftReport, classifyChangedScope, classifyRuleDrift, inspectSharkcraft, resolveChangedFiles, RuleEnforcementState, summariseChangedScope, } from '@shrkcrft/inspector';
 import { flagBool, flagString, flagList, resolveCwd, } from "../command-registry.js";
 import { asJson, header, kv } from "../output/format-output.js";
@@ -52,16 +53,25 @@ export const driftCommand = {
             : null;
         if (flagBool(args, 'json')) {
             process.stdout.write(asJson({ ...report, ...(classification ? { changedScope: classification } : {}) }) + '\n');
-            return report.counts.error > 0 ? 1 : 0;
+            // THE drift verdict (settled in the inspector over the boundary
+            // orchestrator's coverage): 2 when a boundary rule's scope was never
+            // fully examined — never a clean 0 over an unreadable governed file.
+            return report.exitCode;
         }
         process.stdout.write(header('Drift report'));
         process.stdout.write(kv('totals', `${report.counts.error} errors, ${report.counts.warning} warnings, ${report.counts.info} info`) + '\n\n');
         if (classification) {
             process.stdout.write(`changed-scope: ${summariseChangedScope(classification)}\n\n`);
         }
+        const settled = {
+            exit: report.exitCode,
+            verdict: report.verdict,
+            shortfalls: report.shortfalls,
+            accepted: report.accepted ?? [],
+        };
         if (report.findings.length === 0) {
-            process.stdout.write('No drift detected.\n');
-            return 0;
+            process.stdout.write(`${verdictLine(settled, 'No drift detected.')}\n`);
+            return report.exitCode;
         }
         for (const f of report.findings) {
             const tag = f.severity.toUpperCase().padEnd(8);
@@ -69,7 +79,10 @@ export const driftCommand = {
             if (f.suggestedFix)
                 process.stdout.write(`           ↳ ${f.suggestedFix}\n`);
         }
-        return report.counts.error > 0 ? 1 : 0;
+        const tail = verdictLine(settled, '');
+        if (tail)
+            process.stdout.write(`\n${tail}\n`);
+        return report.exitCode;
     },
 };
 /**

package/dist/commands/eslint.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"eslint.command.d.ts","sourceRoot":"","sources":["../../src/commands/eslint.command.ts"],"names":[],"mappings":"AAGA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AA2DhC,eAAO,MAAM,qBAAqB,EAAE,eA8DnC,CAAC;AAiCF,eAAO,MAAM,mBAAmB,EAAE,eAiFjC,CAAC;AAgFF,eAAO,MAAM,kBAAkB,EAAE,eAsDhC,CAAC;AAkCF,eAAO,MAAM,+BAA+B,EAAE,eAyC7C,CAAC;AAmBF,eAAO,MAAM,aAAa,EAAE,eAuC3B,CAAC"}
+{"version":3,"file":"eslint.command.d.ts","sourceRoot":"","sources":["../../src/commands/eslint.command.ts"],"names":[],"mappings":"AAGA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AA4DhC,eAAO,MAAM,qBAAqB,EAAE,eA8DnC,CAAC;AAiCF,eAAO,MAAM,mBAAmB,EAAE,eAiFjC,CAAC;AAgFF,eAAO,MAAM,kBAAkB,EAAE,eAsDhC,CAAC;AAkCF,eAAO,MAAM,+BAA+B,EAAE,eAyC7C,CAAC;AAmBF,eAAO,MAAM,aAAa,EAAE,eA0D3B,CAAC"}

package/dist/commands/eslint.command.js

@@ -2,6 +2,7 @@ import { existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import * as nodePath from 'node:path';
 import { inspectSharkcraft } from '@shrkcrft/inspector';
 import { flagBool, flagString, resolveCwd, } from "../command-registry.js";
+import { PositionalMode } from "../dispatch/positional-mode.js";
 import { asJson, bullet, header } from "../output/format-output.js";
 /**
  * ESLint bridge. NOT a full ESLint plugin: a thin adapter that
@@ -387,6 +388,25 @@ function renderEslintReportText(files) {
 }
 export const eslintCommand = {
     name: 'eslint',
+    // positional[0] is the bridge verb — `run` refuses anything else. Each spec
+    // reuses its verb handler's usage, so the flags the verb reads (`--force`,
+    // `--output`, `--from`, …) are documented for the post-run flag detector.
+    positionals: PositionalMode.None,
+    subverbs: [
+        {
+            name: 'scaffold',
+            aliases: ['config'],
+            description: eslintScaffoldCommand.description,
+            usage: eslintScaffoldCommand.usage,
+        },
+        { name: 'report', description: eslintReportCommand.description, usage: eslintReportCommand.usage },
+        { name: 'rules', description: eslintRulesCommand.description, usage: eslintRulesCommand.usage },
+        {
+            name: 'explain-limitations',
+            description: eslintExplainLimitationsCommand.description,
+            usage: eslintExplainLimitationsCommand.usage,
+        },
+    ],
     description: 'ESLint bridge. `scaffold` emits a flat-config snippet that ignores SharkCraft generated paths; `report` re-emits boundary violations in the ESLint result format; `rules` inventories what is bridgeable; `explain-limitations` documents what cannot be bridged. `config` is an alias for `scaffold`.',
     usage: 'shrk eslint <scaffold|config|report|rules|explain-limitations> [...flags]',
     async run(args) {

package/dist/commands/export.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"export.command.d.ts","sourceRoot":"","sources":["../../src/commands/export.command.ts"],"names":[],"mappings":"AASA,OAAO,EAKL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAgBhC,eAAO,MAAM,aAAa,EAAE,eAuF3B,CAAC"}
+{"version":3,"file":"export.command.d.ts","sourceRoot":"","sources":["../../src/commands/export.command.ts"],"names":[],"mappings":"AASA,OAAO,EAKL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAiBhC,eAAO,MAAM,aAAa,EAAE,eA4G3B,CAAC"}

package/dist/commands/export.command.js

@@ -4,6 +4,7 @@ import { inspectSharkcraft } from '@shrkcrft/inspector';
 import { ALL_EXPORT_FORMATS, isExportFormat, renderExport, } from "../export/export-formats.js";
 import { buildClaudeCommands } from "../export/claude-commands-export.js";
 import { flagBool, flagNumber, flagString, resolveCwd, } from "../command-registry.js";
+import { PositionalMode } from "../dispatch/positional-mode.js";
 import { asJson, bullet, header } from "../output/format-output.js";
 import { exportBundleCommand, exportSessionCommand, exportQualityCommand, exportReviewCommand, } from "./export-bundle.command.js";
 const ARCHIVE_SUBCOMMANDS = {
@@ -14,8 +15,29 @@ const ARCHIVE_SUBCOMMANDS = {
 };
 export const exportCommand = {
     name: 'export',
+    // positional[0] is the FORMAT (or an archive verb): every other token is
+    // refused with the list, as `run` already did — now before anything runs.
+    positionals: PositionalMode.None,
+    subverbs: [
+        ...ALL_EXPORT_FORMATS.map((format) => ({
+            name: format,
+            description: `Render the ${format} export (dry-run unless --write).`,
+            usage: `shrk export ${format} [--write [--force]] [--output <path>] [--task "<task>"] [--max-rules N] [--max-paths N] [--json]`,
+        })),
+        {
+            name: 'claude-commands',
+            description: 'Per-project slash commands under .claude/commands/ (dry-run unless --write).',
+            usage: 'shrk export claude-commands [--write [--force]] [--output <path>] [--json]',
+        },
+        ...Object.entries(ARCHIVE_SUBCOMMANDS).map(([name, handler]) => ({
+            name,
+            description: handler.description,
+            usage: handler.usage,
+            positionals: PositionalMode.Free,
+        })),
+    ],
     description: 'Inversion — pull SharkCraft rules into the agent\'s prompt instead of the agent calling back to shrk. Single-file outputs: claude-skill (.claude/skills/<name>/SKILL.md, recommended), agents-md (AGENTS.md), claude-md (CLAUDE.md), cursor-rules (.cursor/rules/*.mdc), copilot-instructions. Multi-file output: claude-commands (.claude/commands/*.md — per-project slash commands like /new-service, /check-changes, /follow-shrk). Dry-run by default; pass --write to save.',
-    usage: 'shrk [--cwd <dir>] export <format> [--write] [--output <path>] [--task "<task>"] [--max-rules N] [--max-paths N] [--json]',
+    usage: 'shrk [--cwd <dir>] export <format> [--write [--force]] [--output <path>] [--task "<task>"] [--max-rules N] [--max-paths N] [--json]',
     async run(args) {
         const format = args.positional[0];
         if (!format) {

package/dist/commands/feedback-dispatch.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feedback-dispatch.command.d.ts","sourceRoot":"","sources":["../../src/commands/feedback-dispatch.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AACH,OAAO,KAAK,EAAE,eAAe,EAAc,MAAM,wBAAwB,CAAC;AA6B1E,eAAO,MAAM,eAAe,EAAE,eAmD7B,CAAC"}
+{"version":3,"file":"feedback-dispatch.command.d.ts","sourceRoot":"","sources":["../../src/commands/feedback-dispatch.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AACH,OAAO,KAAK,EAAE,eAAe,EAAc,MAAM,wBAAwB,CAAC;AA8B1E,eAAO,MAAM,eAAe,EAAE,eA+E7B,CAAC"}

package/dist/commands/feedback-dispatch.command.js

@@ -1,3 +1,4 @@
+import { PositionalMode } from "../dispatch/positional-mode.js";
 import { feedbackActionsCommand, feedbackBacklogCommand, feedbackConvertToBacklogCommand, feedbackIngestCommand, feedbackPlanCommand, feedbackPromptCommand, feedbackRulesDoctorCommand, feedbackRulesListCommand, feedbackSummarizeCommand, } from "./feedback.command.js";
 const VERBS = {
     ingest: feedbackIngestCommand,
@@ -15,6 +16,34 @@ const RULES_VERBS = {
 };
 export const feedbackCommand = {
     name: 'feedback',
+    // positional[0] is a verb or the feedback FILE (`feedback <file>` ingests):
+    // a verb-shaped token that is neither is refused before it becomes a path.
+    positionals: PositionalMode.Path,
+    subverbs: [
+        { name: 'ingest', description: 'Parse a feedback file into structured findings.', usage: 'shrk feedback ingest <file> [--with-pack-rules] [--json]', positionals: PositionalMode.Path },
+        { name: 'summarize', description: 'Summarize a feedback file.', usage: 'shrk feedback summarize <file> [--json]', positionals: PositionalMode.Path },
+        { name: 'actions', description: 'The actions a feedback file implies.', usage: 'shrk feedback actions <file> [--json]', positionals: PositionalMode.Path },
+        {
+            name: 'convert-to-backlog',
+            description: 'Convert a feedback file into backlog items.',
+            usage: 'shrk feedback convert-to-backlog <file> [--output <path>]',
+            positionals: PositionalMode.Path,
+        },
+        { name: 'backlog', description: 'The backlog view of a feedback file.', usage: 'shrk feedback backlog <file>', positionals: PositionalMode.Path },
+        { name: 'prompt', description: 'An agent prompt from a feedback file.', usage: 'shrk feedback prompt <file>', positionals: PositionalMode.Path },
+        { name: 'plan', description: 'A plan from a feedback file.', usage: 'shrk feedback plan <file>', positionals: PositionalMode.Path },
+        {
+            name: 'rules',
+            description: 'The feedback rules (`list` is the default).',
+            usage: 'shrk feedback rules <list|doctor> [--json]',
+            // An unknown token used to fall through to `list` at exit 0.
+            positionals: PositionalMode.None,
+            subverbs: [
+                { name: 'list', description: 'List the feedback rules.', usage: 'shrk feedback rules list [--json]' },
+                { name: 'doctor', description: 'Validate the feedback rules.', usage: 'shrk feedback rules doctor [--json]' },
+            ],
+        },
+    ],
     description: 'Parse freeform feedback markdown into structured findings. Subcommands: ingest|summarize|actions|convert-to-backlog|rules. Read-only.',
     usage: 'shrk feedback <ingest|summarize|actions|convert-to-backlog> <file> [--with-pack-rules] [--json]\n  shrk feedback rules <list|doctor> [--json]',
     async run(args) {

package/dist/commands/feedback.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"feedback.command.d.ts","sourceRoot":"","sources":["../../src/commands/feedback.command.ts"],"names":[],"mappings":"AAiBA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAiBhC,eAAO,MAAM,qBAAqB,EAAE,eAiCnC,CAAC;AAEF,eAAO,MAAM,wBAAwB,EAAE,eA0CtC,CAAC;AAEF,eAAO,MAAM,sBAAsB,EAAE,eAyDpC,CAAC;AAEF,eAAO,MAAM,+BAA+B,EAAE,eAwB7C,CAAC;AAIF,eAAO,MAAM,wBAAwB,EAAE,eAqBtC,CAAC;AAEF,eAAO,MAAM,0BAA0B,EAAE,eAoDxC,CAAC;AAGF,eAAO,MAAM,sBAAsB,EAAE,eAqBpC,CAAC;AAEF,eAAO,MAAM,qBAAqB,EAAE,eAqBnC,CAAC;AAEF,eAAO,MAAM,mBAAmB,EAAE,eA6BjC,CAAC"}
+{"version":3,"file":"feedback.command.d.ts","sourceRoot":"","sources":["../../src/commands/feedback.command.ts"],"names":[],"mappings":"AAkBA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAkBhC,eAAO,MAAM,qBAAqB,EAAE,eAiCnC,CAAC;AAEF,eAAO,MAAM,wBAAwB,EAAE,eA0CtC,CAAC;AAEF,eAAO,MAAM,sBAAsB,EAAE,eAyDpC,CAAC;AAEF,eAAO,MAAM,+BAA+B,EAAE,eAwB7C,CAAC;AAIF,eAAO,MAAM,wBAAwB,EAAE,eAyBtC,CAAC;AAEF,eAAO,MAAM,0BAA0B,EAAE,eAoDxC,CAAC;AAGF,eAAO,MAAM,sBAAsB,EAAE,eAqBpC,CAAC;AAEF,eAAO,MAAM,qBAAqB,EAAE,eAqBnC,CAAC;AAEF,eAAO,MAAM,mBAAmB,EAAE,eA6BjC,CAAC"}

package/dist/commands/feedback.command.js

@@ -7,10 +7,11 @@
  */
 import { mkdirSync, writeFileSync } from 'node:fs';
 import * as nodePath from 'node:path';
-import { FeedbackBucket, ingestFeedbackFile, inspectSharkcraft, loadFeedbackRules, renderFeedbackBacklog, } from '@shrkcrft/inspector';
+import { ContributionKind, FeedbackBucket, ingestFeedbackFile, inspectSharkcraft, loadFeedbackRules, renderFeedbackBacklog, } from '@shrkcrft/inspector';
 import { flagBool, flagString, resolveCwd, } from "../command-registry.js";
 import { asJson, header } from "../output/format-output.js";
 import { feedbackRulesDoctorHints, renderFailureHints } from "../output/failure-hints.js";
+import { writeRejectedEntriesNote } from "../output/rejected-entries-note.js";
 function ingestPositional(args) {
     return args.positional[0];
 }
@@ -194,8 +195,11 @@ export const feedbackRulesListCommand = {
         const cwd = resolveCwd(args);
         const inspection = await inspectSharkcraft({ cwd });
         const rules = await loadFeedbackRules(inspection);
+        // A rule its loader refused (no id, a reused id) is named (round 12, 12.1).
+        const note = { next: 'shrk packs contributions' };
         if (flagBool(args, 'json')) {
             process.stdout.write(asJson({ count: rules.length, rules }) + '\n');
+            await writeRejectedEntriesNote(inspection, [ContributionKind.FeedbackRule], { ...note, json: true });
             return 0;
         }
         process.stdout.write(header(`Feedback rules (${rules.length})`));
@@ -207,6 +211,7 @@ export const feedbackRulesListCommand = {
             if (r.targetArea)
                 process.stdout.write(`       target:   ${r.targetArea}\n`);
         }
+        await writeRejectedEntriesNote(inspection, [ContributionKind.FeedbackRule], note);
         return 0;
     },
 };

package/dist/commands/finish.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"finish.command.d.ts","sourceRoot":"","sources":["../../src/commands/finish.command.ts"],"names":[],"mappings":"AACA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AA8DhC,eAAO,MAAM,aAAa,EAAE,eAmB3B,CAAC"}
+{"version":3,"file":"finish.command.d.ts","sourceRoot":"","sources":["../../src/commands/finish.command.ts"],"names":[],"mappings":"AACA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAqFhC,eAAO,MAAM,aAAa,EAAE,eA0B3B,CAAC"}

package/dist/commands/finish.command.js

@@ -1,6 +1,9 @@
 import { flagBool, flagString, resolveCwd, } from "../command-registry.js";
 import { asJson, bullet, header, kv } from "../output/format-output.js";
-import { runFinishGates } from "../finish/run-finish.js";
+import { runFinishGates, } from "../finish/run-finish.js";
+import { ExitCode } from "../exit-codes.js";
+import { ALLOW_EMPTY_FLAG, allowEmptyValve } from "../gates/allow-empty.js";
+import { verdictLine } from "../gates/verdict-line.js";
 function resolveScope(args, cwd) {
     const staged = flagBool(args, 'staged');
     const since = flagString(args, 'since');
@@ -20,12 +23,16 @@ const STATUS_GLYPH = {
     pass: '✓',
     fail: '✗',
     skipped: '–',
+    // A pass over part of its scope is not a pass (round 11).
+    partial: '~',
 };
 function renderText(report) {
     process.stdout.write(header('Finish — is this changeset safe to complete?'));
     process.stdout.write(kv('scope', `${report.scope.mode} (${report.scope.fileCount} file${report.scope.fileCount === 1 ? '' : 's'})`) +
         '\n');
     for (const g of report.gates) {
+        // The status is settled once in runFinishGates (`partial` for a pass over
+        // part of its scope), so this line and --json say the same word.
         process.stdout.write(`  ${STATUS_GLYPH[g.status]} ${g.name.padEnd(11)} ${g.status.padEnd(8)} ${g.detail}\n`);
     }
     if (report.impact.ran) {
@@ -35,8 +42,9 @@ function renderText(report) {
     else if (report.impact.note) {
         process.stdout.write(kv('impact', `(skipped — ${report.impact.note})`) + '\n');
     }
-    process.stdout.write(kv('verdict', `${report.verdict} (exit ${report.exit})`) + '\n\n');
-    process.stdout.write(report.summary + '\n');
+    process.stdout.write(kv('verdict', `${report.verdict} (exit ${report.exit})`) + '\n');
+    if (report.exit === ExitCode.Failure)
+        process.stdout.write(`\n${report.summary}\n`);
     const failing = report.gates.filter((g) => g.status === 'fail');
     for (const g of failing) {
         process.stdout.write(`\n${g.name} — failing items:\n`);
@@ -48,20 +56,36 @@ function renderText(report) {
             process.stdout.write(`  … and ${g.items.length - 15} more (pass --json for the full list).\n`);
         }
     }
+    // The final line comes from the SETTLED envelope: the "Safe to finish ✓"
+    // sentence exists only at exit 0; at 2 the summary leads a NOT VERIFIED line
+    // naming every sub-gate that fell short.
+    const line = verdictLine(report.gate, `${report.summary} ✓`, report.exit === ExitCode.NotVerified ? report.summary : undefined);
+    if (line)
+        process.stdout.write(`\n${line}\n`);
+    if (report.exit === ExitCode.NotVerified && report.gate.coverage.expected === 0) {
+        process.stdout.write(`  Pass --${ALLOW_EMPTY_FLAG} to accept an empty changeset explicitly.\n`);
+    }
     process.stdout.write(`\nNext: ${report.nextAction}\n`);
 }
 export const finishCommand = {
     name: 'finish',
-    description: 'Composite "is this changeset safe to finish?" gate: EXECUTES every deterministic changed-only check inline — boundaries + import-hygiene + wiring + unprovided (DI graph) + policy + deleted-orphans + arch (advisory cycles) — over tracked AND untracked changes, and returns ONE honest 0/1/2 verdict (0 pass · 1 fail · 2 not-verified — "evaluated nothing" is 2, never a green 0). The single trustworthy "done?" call after editing (superset of `diff-check`). Read-only.',
-    usage: 'shrk [--cwd <dir>] finish [files... | --files a.ts,b.ts | --staged | --since <ref>] [--json]',
-    booleanFlags: new Set(['json', 'staged']),
+    description: 'Composite "is this changeset safe to finish?" gate: EXECUTES every deterministic changed-only check inline — boundaries (with rule-edit escalation) + import-hygiene + wiring + unprovided (DI graph) + policy + deleted-orphans + arch (advisory cycles) — over tracked AND untracked changes, and returns ONE honest 0/1/2 verdict (0 pass · 1 fail · 2 not-verified — "evaluated nothing" and "passed over part of its scope" are 2, never a green 0). The single trustworthy "done?" call after editing (superset of `diff-check`). --json carries the shared gate envelope. Read-only.',
+    usage: 'shrk [--cwd <dir>] finish [files... | --files a.ts,b.ts | --staged | --since <ref>] [--allow-empty] [--json]',
+    // `--allow-empty` is the shared verdict valve — declared boolean so it can
+    // never swallow a following positional file.
+    booleanFlags: new Set(['json', 'staged', ALLOW_EMPTY_FLAG]),
     async run(args) {
         const cwd = resolveCwd(args);
         const wantJson = flagBool(args, 'json');
         const { mode, options } = resolveScope(args, cwd);
-        const report = await runFinishGates({ cwd, mode, scope: options });
+        const report = await runFinishGates({
+            cwd,
+            mode,
+            scope: options,
+            emptyValve: (expected) => allowEmptyValve(args, expected),
+        });
         if (wantJson) {
-            process.stdout.write(asJson(report) + '\n');
+            process.stdout.write(asJson({ ...report, exitCode: report.exit }) + '\n');
             return report.exit;
         }
         renderText(report);

package/dist/commands/fix.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"fix.command.d.ts","sourceRoot":"","sources":["../../src/commands/fix.command.ts"],"names":[],"mappings":"AAwBA,OAAO,EAKL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AA0PhC,eAAO,MAAM,UAAU,EAAE,eAwBxB,CAAC"}
+{"version":3,"file":"fix.command.d.ts","sourceRoot":"","sources":["../../src/commands/fix.command.ts"],"names":[],"mappings":"AAwBA,OAAO,EAKL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AA2PhC,eAAO,MAAM,UAAU,EAAE,eAmCxB,CAAC"}

package/dist/commands/fix.command.js

@@ -13,6 +13,7 @@ import { existsSync, mkdirSync, writeFileSync } from 'node:fs';
 import * as nodePath from 'node:path';
 import { AssetKind, AssetProvenanceOperation, buildFixPreview, FixKind, inspectSharkcraft, listFixKinds, recordProvenance, renderFixPreviewMarkdown, } from '@shrkcrft/inspector';
 import { flagBool, flagString, flagList, resolveCwd, } from "../command-registry.js";
+import { PositionalMode } from "../dispatch/positional-mode.js";
 import { asJson, header } from "../output/format-output.js";
 import { applyActionHintStub, } from "../asset-preview/apply-action-hint-stub.js";
 import { applyKnowledgeStaleFix } from "../asset-preview/apply-knowledge-stale-fix.js";
@@ -259,6 +260,16 @@ async function runActionHintApply(cwd, inspection, report, args) {
 }
 export const fixCommand = {
     name: 'fix',
+    positionals: PositionalMode.None,
+    subverbs: [
+        { name: 'list', description: 'List the fixable findings.', usage: 'shrk fix list [--kinds <a,b>] [--json]' },
+        { name: 'doctor', description: 'Fix-system health.', usage: 'shrk fix doctor [--json]' },
+        {
+            name: 'preview',
+            description: 'Preview the fixes (the default).',
+            usage: 'shrk fix preview [--action-hints|--knowledge-stale|--template-drift] [--kinds <a,b>] [--target <id>] [--write-preview] [--apply [--allow-divergent] [--drop-stale] [--drop-missing] [--rename-strategy strict|wide]] [--json]',
+        },
+    ],
     description: 'Fix preview system. Preview-only by default. `--write-preview` writes drafts to .sharkcraft/fixes/. `--action-hints --apply` splices stubbed actionHints. `--knowledge-stale --apply [--drop-stale] [--drop-missing]` removes the offending reference in place. `--rename-strategy=wide` surfaces multi-candidate rename suggestions that strict mode silently drops.',
     usage: 'shrk fix [list|doctor|preview] [--action-hints|--knowledge-stale|--template-drift] [--kinds <a,b>] [--target <id>] [--write-preview] [--apply [--allow-divergent] [--drop-stale] [--drop-missing] [--rename-strategy strict|wide]] [--json]',
     async run(args) {
@@ -309,6 +320,9 @@ async function runKnowledgeStaleApply(cwd, inspection, _report, args) {
     // Rebuild the stale report directly so we have full IKnowledgeReferenceCheck
     // shape (the FixKind suggestions are stringy and lose the kind/path/symbol/id).
     const { buildKnowledgeStaleReport, ReferenceCheckOutcome, RenameStrategy } = await import('@shrkcrft/inspector');
+    // Warm before resolving, or a correct playbook / policy id reads as broken
+    // and a --drop-stale would delete a TRUE reference.
+    await (await import("../surface/cli-command-resolver.js")).warmCliReferenceRegistries(inspection);
     const stale = buildKnowledgeStaleReport(inspection, {
         renameStrategy: useWide ? RenameStrategy.Wide : RenameStrategy.Strict,
     });
@@ -514,6 +528,9 @@ async function runKnowledgeStaleApply(cwd, inspection, _report, args) {
 async function runTemplateDriftApply(cwd, inspection, _report, args) {
     const wantJson = flagBool(args, 'json');
     const { buildTemplateDriftReport } = await import('@shrkcrft/inspector');
+    // Warm first: an unwarmed related id is NOT VERIFIED (never "unresolved"), so
+    // without it this fix would find nothing — and must never drop a real id.
+    await (await import("../surface/cli-command-resolver.js")).warmCliReferenceRegistries(inspection);
     const drift = buildTemplateDriftReport(inspection, {});
     // Pull related-id-unresolved findings and pair each with its source.
     const templateSources = inspection.templateSources;
@@ -750,6 +767,8 @@ function runFixList(args) {
 async function runFixDoctor(args) {
     const cwd = resolveCwd(args);
     const inspection = await inspectSharkcraft({ cwd });
+    // The preview folds in the stale report — warm before it resolves any id.
+    await (await import("../surface/cli-command-resolver.js")).warmCliReferenceRegistries(inspection);
     const report = buildFixPreview(inspection);
     const errors = report.suggestions.filter((s) => s.severity === 'error').length;
     const warnings = report.suggestions.filter((s) => s.severity === 'warning').length;

package/dist/commands/gate.command.d.ts

@@ -8,8 +8,13 @@ import { type ICommandHandler } from '../command-registry.js';
  *   - 0 if overall status is `pass` (no failures, no warnings)
  *   - 0 if overall is `warn` (default — opt-in to fail via --strict)
  *   - 1 if overall is `fail`
+ *   - 2 if nothing failed but a gate examined only PART of what it was asked
+ *     to (e.g. a wiring rule that passed over part of its scope, or a config
+ *     that did not load): its `coverage` settles the proposed 0 to NOT
+ *     VERIFIED through the CLI's one guard (`settleVerdict`) — a `warn` that
+ *     says "this is not a pass" never exits 0.
  *
- * Pass `--strict` to treat `warn` as failure.
+ * Pass `--strict` to treat `warn` as failure (1).
  */
 export declare const gateCommand: ICommandHandler;
 //# sourceMappingURL=gate.command.d.ts.map

package/dist/commands/gate.command.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"gate.command.d.ts","sourceRoot":"","sources":["../../src/commands/gate.command.ts"],"names":[],"mappings":"AAcA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAGhC;;;;;;;;;;;GAWG;AACH,eAAO,MAAM,WAAW,EAAE,eAwNzB,CAAC"}
+{"version":3,"file":"gate.command.d.ts","sourceRoot":"","sources":["../../src/commands/gate.command.ts"],"names":[],"mappings":"AAUA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAIhC;;;;;;;;;;;;;;;;GAgBG;AACH,eAAO,MAAM,WAAW,EAAE,eAiJzB,CAAC"}

package/dist/commands/gate.command.js

@@ -1,10 +1,10 @@
-import { QualityGateReportStore, renderGateReportMarkdown, runQualityGates, } from '@shrkcrft/quality-gates';
-import { ArchReportStore, runArchCheck } from '@shrkcrft/architecture-guard';
-import { inspectSharkcraft, resolveChangedFiles, resolveProjectConfig, } from '@shrkcrft/inspector';
+import { prepareQualityGateRun, QualityGateReportStore, renderGateReportMarkdown, runQualityGates, settleQualityGateReport, } from '@shrkcrft/quality-gates';
+import { ArchReportStore, archStoreMissing, runArchCheck } from '@shrkcrft/architecture-guard';
 import { chmodSync, existsSync, mkdirSync, writeFileSync } from 'node:fs';
 import * as nodePath from 'node:path';
 import { flagBool, flagString, resolveCwd, } from "../command-registry.js";
 import { asJson, header, kv } from "../output/format-output.js";
+import { verdictLine } from "../gates/verdict-line.js";
 /**
  * `shrk gate` — run all code-intelligence quality gates and emit one
  * pass/fail report. Designed as the single command CI should call
@@ -14,8 +14,13 @@ import { asJson, header, kv } from "../output/format-output.js";
  *   - 0 if overall status is `pass` (no failures, no warnings)
  *   - 0 if overall is `warn` (default — opt-in to fail via --strict)
  *   - 1 if overall is `fail`
+ *   - 2 if nothing failed but a gate examined only PART of what it was asked
+ *     to (e.g. a wiring rule that passed over part of its scope, or a config
+ *     that did not load): its `coverage` settles the proposed 0 to NOT
+ *     VERIFIED through the CLI's one guard (`settleVerdict`) — a `warn` that
+ *     says "this is not a pass" never exits 0.
  *
- * Pass `--strict` to treat `warn` as failure.
+ * Pass `--strict` to treat `warn` as failure (1).
  */
 export const gateCommand = {
     name: 'gate',
@@ -73,116 +78,30 @@ export const gateCommand = {
         const staged = flagBool(args, 'staged');
         const filesRaw = flagString(args, 'files');
         const fileList = filesRaw ? filesRaw.split(',').map((s) => s.trim()).filter(Boolean) : [];
-        const wantChangedScope = changedOnly || staged || Boolean(sinceRef) || fileList.length > 0;
-        let changedFiles;
-        if (wantChangedScope) {
-            const resolved = resolveChangedFiles({
-                projectRoot: cwd,
-                ...(fileList.length > 0 ? { files: fileList } : {}),
-                ...(staged ? { staged: true } : {}),
-                ...(sinceRef ? { since: sinceRef } : {}),
-                ...(changedOnly && !staged && !sinceRef && fileList.length === 0
-                    ? { includeWorktree: true }
-                    : {}),
-            });
-            changedFiles = resolved.files;
-        }
-        // Wiring + policy rules come from the project config; each gate is skipped
-        // (never red) when none are declared, so they're inert for projects that
-        // don't opt in. An INVALID config is surfaced (warn) rather than silently
-        // disabling the plane.
-        const loadedConfig = await resolveProjectConfig(cwd);
-        const wiringRules = loadedConfig.ok ? loadedConfig.value.config.wiringRules ?? [] : [];
-        const policyRules = loadedConfig.ok ? loadedConfig.value.config.policyRules ?? [] : [];
-        const configError = loadedConfig.ok ? undefined : loadedConfig.error.message;
+        // THE gate-run assembly (`prepareQualityGateRun`, @shrkcrft/quality-gates):
+        // the project's wiring + policy rules (an INVALID config surfaced, never a
+        // silent disable), THE plane scan scope, the knowledge inspection, the
+        // change-scoped arch gate and the advisory impact gate. MCP
+        // `get_quality_gate` calls the same one, so the two run the same gate set
+        // (round 11 review: MCP passed the impact options only and read `pass`
+        // where this verb exited 1).
+        const prepared = await prepareQualityGateRun({
+            cwd,
+            ...(sinceRef ? { sinceRef } : {}),
+            ...(changedOnly ? { changedOnly: true } : {}),
+            ...(staged ? { staged: true } : {}),
+            ...(fileList.length > 0 ? { files: fileList } : {}),
+            ...(failOn ? { failOn } : {}),
+            ...(archAll ? { archAll: true } : {}),
+            ...(disable ? { disable } : {}),
+            ...(apiBaseline ? { apiDiff: { baselinePath: apiBaseline, failOnBreaking: !noFailOnBreaking } } : {}),
+        });
         // Pack-plane merge notes (missing/invalid pack rule files, dropped
         // collisions) go to stderr so they never pollute the JSON/markdown report
         // on stdout that CI consumes.
-        if (loadedConfig.ok) {
-            for (const d of loadedConfig.value.planeDiagnostics) {
-                process.stderr.write(`plane: ${d}\n`);
-            }
-        }
-        const scopeOpts = wantChangedScope
-            ? { changedOnly: true, changedFiles: changedFiles ?? [] }
-            : {};
-        // §3.1 — the architecture gate's "NEW" is ALWAYS change-scoped: a NEW error
-        // means one introduced by the working change (diff vs HEAD), never drift
-        // against a frozen (possibly months-old) baseline in a file the change never
-        // touched. When an explicit scope flag is passed we reuse its resolution;
-        // otherwise we default to the worktree diff vs HEAD — empty in a clean tree,
-        // so pre-existing baseline drift stays informational and can't red the gate
-        // on its own. `--arch-all` (baselineRelative:false) ignores this and fails
-        // on total errors, keeping a clean-tree CI demand expressible.
-        const archChangedFiles = wantChangedScope
-            ? changedFiles ?? []
-            : resolveChangedFiles({ projectRoot: cwd, includeWorktree: true }).files;
-        // Knowledge symbol-ref integrity needs the loaded knowledge entries. The
-        // inspection is async, so we build it here and inject it; the gate stays
-        // synchronous and resolves the code graph itself. Best-effort — a failed
-        // inspection just skips the gate rather than failing `shrk gate`.
-        let inspection;
-        if (!disable?.includes('knowledge-symbol')) {
-            try {
-                inspection = await inspectSharkcraft({ cwd });
-            }
-            catch {
-                inspection = undefined;
-            }
-        }
-        const report = runQualityGates({
-            projectRoot: cwd,
-            arch: {
-                ...(archAll ? { baselineRelative: false } : {}),
-                changedFiles: archChangedFiles,
-            },
-            wiring: {
-                ...(configError
-                    ? { configError }
-                    : wiringRules.length > 0
-                        ? { rules: wiringRules }
-                        : {}),
-                ...scopeOpts,
-            },
-            policy: {
-                ...(configError
-                    ? { configError }
-                    : policyRules.length > 0
-                        ? { rules: policyRules }
-                        : {}),
-                ...scopeOpts,
-            },
-            ...(inspection
-                ? {
-                    knowledgeSymbol: {
-                        inspection,
-                        ...(wantChangedScope ? { changedFiles: changedFiles ?? [] } : {}),
-                    },
-                }
-                : {}),
-            impact: {
-                ...(sinceRef ? { sinceRef } : {}),
-                // Blast-radius risk is inherently PRE-EXISTING structure (touching a hub
-                // is risky but not a new failure this change introduced), so the composite
-                // gate treats it as ADVISORY by default — `failOn: []` warns instead of
-                // redding, keeping the verdict change-attributable. `--fail-on critical`
-                // opts into a hard fail; `--strict` escalates the advisory warn.
-                failOn: failOn ?? [],
-                // Scope the impact gate to the changeset too: with `--since` we keep the
-                // gitref diff; with `--changed-only` / `--staged` / `--files` (and no
-                // `--since`) we analyze the resolved changed-file set directly.
-                ...(wantChangedScope && !sinceRef ? { files: changedFiles ?? [] } : {}),
-            },
-            ...(apiBaseline
-                ? {
-                    apiDiff: {
-                        baselinePath: apiBaseline,
-                        failOnBreaking: !noFailOnBreaking,
-                    },
-                }
-                : {}),
-            ...(disable ? { disable } : {}),
-        });
+        for (const d of prepared.planeDiagnostics)
+            process.stderr.write(`plane: ${d}\n`);
+        const report = runQualityGates(prepared.options);
         // Persist the report so dashboards and follow-up tooling can read
         // it without re-running every gate. Opt out with `--no-persist`.
         if (!flagBool(args, 'no-persist')) {
@@ -194,8 +113,14 @@ export const gateCommand = {
                 // disk-write error.
             }
         }
+        // Settle first, render second: the proposed exit (from `overall`) is vetoed
+        // by any gate coverage with a shortfall, for text, --json and --markdown alike.
+        // THE settle (`settleQualityGateReport`) MCP `get_quality_gate` returns too.
+        const settled = settleQualityGateReport(report, strict);
+        const exit = settled.exit;
+        const line = verdictLine(settled, '');
         if (wantMarkdown) {
-            const md = renderGateReportMarkdown(report);
+            const md = renderGateReportMarkdown(report) + (line ? `\n${line}\n` : '');
             if (outputPath) {
                 const abs = nodePath.isAbsolute(outputPath)
                     ? outputPath
@@ -206,11 +131,17 @@ export const gateCommand = {
             else {
                 process.stdout.write(md);
             }
-            return exitCode(report.overall, strict);
+            return exit;
         }
         if (wantJson) {
-            process.stdout.write(asJson(report) + '\n');
-            return exitCode(report.overall, strict);
+            process.stdout.write(asJson({
+                ...report,
+                exitCode: exit,
+                verdict: settled.verdict,
+                shortfalls: settled.shortfalls,
+                accepted: settled.accepted,
+            }) + '\n');
+            return exit;
         }
         process.stdout.write(header(`Quality gates: ${report.overall.toUpperCase()}`));
         process.stdout.write(kv('total duration', `${report.totalDurationMs}ms`) + '\n');
@@ -225,16 +156,11 @@ export const gateCommand = {
                     process.stdout.write(`              → ${c}\n`);
             }
         }
-        return exitCode(report.overall, strict);
+        if (line)
+            process.stdout.write(`\n${line}\n`);
+        return exit;
     },
 };
-function exitCode(overall, strict) {
-    if (overall === 'fail')
-        return 1;
-    if (overall === 'warn' && strict)
-        return 1;
-    return 0;
-}
 const GITHUB_WORKFLOW = `name: shrk gate
 
 on:
@@ -441,7 +367,7 @@ async function runGateBaseline(args) {
         return 2;
     }
     const report = runArchCheck({ projectRoot: cwd });
-    if (report.diagnostics.some((d) => d.includes('code-graph store missing'))) {
+    if (archStoreMissing(report)) {
         process.stderr.write('Cannot refreeze — graph index missing. Run `shrk graph index` first.\n');
         return 2;
     }