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

package/dist/commands/smart-context.command.js

@@ -8,6 +8,7 @@ import { runGitLines } from '@shrkcrft/shared';
 import { EdgeKind, GraphQueryApi, GraphStore, NodeKind } from '@shrkcrft/graph';
 import { buildProjectOverview, buildTaskPacket, contextTuningBoostFor, inspectSharkcraft, renderOverviewText, } from '@shrkcrft/inspector';
 import { flagBool, flagList, flagNumber, flagString, resolveCwd, } from "../command-registry.js";
+import { PositionalMode } from "../dispatch/positional-mode.js";
 import { DeclarationKind, PLAN_CACHE_SCHEMA, PlanCache, SemanticIndex, TaskType, buildFocusedContext, classifyTask, encodeEmbedding, getDefaultSourceRoots, listIndexableFiles, parseTaskTypeOverride, pruneDeletedHits, renderFocusedContextForPrompt, } from '@shrkcrft/embeddings';
 import { SmartContextDetailedPlanSchema, SmartContextExpansionRequestSchema, } from "../schemas/json-schemas.js";
 import { asJson, header, kv } from "../output/format-output.js";
@@ -48,8 +49,31 @@ const SMART_CONTEXT_BOOLEAN_FLAGS = new Set([
     'plus', 'rebuild', 'refresh', 'save', 'save-conversation', 'stream',
     'tiny-only',
 ]);
+/**
+ * THE complete accepted flag set of `smart-context "<task>"` — every flag its
+ * code reads (r75-flag-read-ledger locks it: a forwarding handler's declared
+ * set covers every read). Declared because the handler forwards its argv to a
+ * worker child (and runs `--dry-run` inline with the same exemption), so no
+ * process ever judged a flag after the run: `--budgt 5` ran at exit 0 and the
+ * requested wall-clock cap silently did not apply. The dispatcher guard now
+ * refuses a flag outside this set before anything runs or spawns.
+ */
+const SMART_CONTEXT_FLAGS = new Set([
+    'ai-plan', 'budget', 'cache-reference-threshold', 'cache-replay-threshold', 'debug', 'dry-run', 'enhance',
+    'enhance-passes', 'expansion-limit', 'expansion-tokens', 'focused', 'instructions', 'json', 'log-prompt',
+    'max-tokens', 'model', 'no-cache', 'no-enhance', 'no-instructions', 'no-polish', 'no-refresh-index', 'plan',
+    'plus', 'provider', 'refresh', 'save', 'save-conversation', 'seed-tokens', 'since', 'stage1-max-tokens',
+    'stream', 'task', 'task-type', 'tiny-only',
+]);
 export const smartContextCommand = {
     name: 'smart-context',
+    // The positionals are the free-form task. The brief runs in a child that
+    // re-executes process.argv (runSmartContextInChild), so this process's flag
+    // reads prove nothing — the post-run unknown-flag detector stays out, and
+    // the declared `flags` are judged by the pre-run guard instead.
+    positionals: PositionalMode.Free,
+    forwardsArgv: true,
+    flags: SMART_CONTEXT_FLAGS,
     booleanFlags: SMART_CONTEXT_BOOLEAN_FLAGS,
     description: 'Build deterministic context and ask an AI provider to synthesise an enriched brief (default), structured plan (--plan), or two-stage development plan (--ai-plan).',
     usage: 'shrk smart-context "<task>" | --task "<task>" [--plus] [--budget <seconds>] [--plan] [--ai-plan] [--save] [--provider auto|ollama|llamacpp] [--enhance|--no-enhance] [--enhance-passes N] [--instructions <path>] [--no-instructions] [--model <id>] [--max-tokens N] [--stage1-max-tokens N] [--seed-tokens N] [--expansion-tokens N] [--expansion-limit N] [--log-prompt] [--save-conversation[=<path>]] [--dry-run] [--debug] [--json]',
@@ -455,6 +479,9 @@ export const smartContextAuditTemplatesCommand = {
         const wantFixPlan = flagBool(args, 'fix-plan') || flagBool(args, 'only-plan');
         const onlyPlan = flagBool(args, 'only-plan');
         const inspection = await inspectSharkcraft({ cwd });
+        // Template `related` ids resolve against EVERY registry — warm first, or a
+        // construct / playbook id reads NOT VERIFIED instead of resolving.
+        await (await import("../surface/cli-command-resolver.js")).warmCliReferenceRegistries(inspection);
         let report = buildTemplateAudit(inspection, templateId ? { templateId } : {});
         if (report.templates.length === 0) {
             if (json) {
@@ -695,6 +722,10 @@ export const smartContextAuditKnowledgeCommand = {
         const wantFixPlan = flagBool(args, 'fix-plan') || flagBool(args, 'only-plan');
         const onlyPlan = flagBool(args, 'only-plan');
         const inspection = await inspectSharkcraft({ cwd });
+        // The audit folds in the stale report — warm before it resolves any id.
+        if (!noStaleCheck) {
+            await (await import("../surface/cli-command-resolver.js")).warmCliReferenceRegistries(inspection);
+        }
         let report = buildKnowledgeAudit(inspection, {
             ...(entryId ? { entryId } : {}),
             ...(noStaleCheck ? { skipStaleCheck: true } : {}),
@@ -1233,6 +1264,11 @@ function runEmbeddingsBuildInChild() {
 }
 /** `shrk smart-context embeddings build` — (re)build the semantic index. */
 export const smartContextEmbeddingsBuildCommand = {
+    // Re-executes process.argv in an isolated child (runEmbeddingsBuildInChild):
+    // nothing judges a flag after the run, so the complete set is declared and
+    // the pre-run guard refuses anything else.
+    forwardsArgv: true,
+    flags: new Set(['json', 'max-files', 'model', 'rebuild', 'root']),
     name: 'embeddings-build',
     description: 'Build or incrementally refresh the semantic index. Defaults to incremental updates when an index already exists; pass --rebuild for a full rebuild.',
     usage: 'shrk smart-context embeddings-build [--model <hf-id>] [--root <dir>]... [--max-files N] [--rebuild] [--json]',
@@ -1568,14 +1604,14 @@ const FOCUSED_ARCHITECTURE_PREAMBLE = [
     '     "explicitlyNotInScope": [string]                              // what we are NOT building yet',
     '  },',
     '  "firstSpike": {                                                  // small, concrete, runnable',
-    '     "proposedCommand": string | null,                             // e.g. "shrk context-feed start --interval 5s"',
+    '     "proposedCommand": string | null,                             // e.g. "shrk watch <task> --interval 5"',
     '     "proposedFiles": [{ "path": string, "purpose": string }],     // e.g. ".sharkcraft/context-stream/<timestamp>.json"',
     '     "schemaOutline": string,                                      // minimal JSON sketch of any context packet shape',
     '     "successCriteria": [string]                                   // observable pass/fail bullets',
     '  },',
     '  "integrationPoints": [{                                          // where this WOULD touch existing code',
     '     "surface": "cli-command"|"cli-watcher"|"mcp-tool-call"|"mcp-resource-read"|"file-read"|"file-write"|"stdout-stream"|"background-watcher",',
-    '     "name": string,                                                // e.g. "shrk context-feed start", "context-packet/next"',
+    '     "name": string,                                                // e.g. "shrk watch", "smart_context_feed"',
     '     "why": string',
     '  }],',
     '  "concerns": {                                                     // pick the ones that apply to the design',
@@ -1631,7 +1667,7 @@ const FOCUSED_ARCHITECTURE_POLISH_PREAMBLE = [
     '   - Drop any candidate that, after deduplication, has no unique pro or no unique con.',
     '2. Wrong vocabulary in integrationPoints.surface.',
     '   - Replace any HTTP verb / generic surface ("CLI"/"MCP server") with the canonical kebab-case vocabulary: `cli-command`, `cli-watcher`, `mcp-tool-call`, `mcp-resource-read`, `file-read`, `file-write`, `stdout-stream`, `background-watcher`.',
-    '   - Each integrationPoint must name an actual surface (e.g. `shrk context-feed`, `context-packet/next`), not just "GET".',
+    '   - Each integrationPoint must name an actual surface (e.g. `shrk watch`, `smart_context_feed`), not just "GET".',
     '3. No recommendedMvp picked, or `recommendation` field missing.',
     '   - Exactly ONE candidate gets `recommendation: "recommended"`. Others split between `possible-later` and `not-for-mvp`.',
     '   - Populate `recommendedMvp.architectureName` to match. Fill `recommendedMvp.explicitlyNotInScope` with at least 2 items.',

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

@@ -1 +1 @@
-{"version":3,"file":"spec.command.d.ts","sourceRoot":"","sources":["../../src/commands/spec.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AA0CH,OAAO,EAUL,gBAAgB,EAMjB,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAKL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAShC,eAAO,MAAM,qBAAqB,iCAAiC,CAAC;AACpE,eAAO,MAAM,wBAAwB,oCAAoC,CAAC;AAY1E,eAAO,MAAM,iBAAiB,EAAE,eA2K/B,CAAC;AAcF,eAAO,MAAM,iBAAiB,EAAE,eAiE/B,CAAC;AAqBF,eAAO,MAAM,eAAe,EAAE,eAmC7B,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,eA8B7B,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,eA2C7B,CAAC;AAEF,eAAO,MAAM,iBAAiB,EAAE,eAoE/B,CAAC;AASF,eAAO,MAAM,oBAAoB,EAAE,eAsRlC,CAAC;AAyDF,eAAO,MAAM,iBAAiB,EAAE,eAqP/B,CAAC;AAEF,eAAO,MAAM,iBAAiB,EAAE,eA4B/B,CAAC;AAkBF,OAAO,EAAE,gBAAgB,EAAE,CAAC"}
+{"version":3,"file":"spec.command.d.ts","sourceRoot":"","sources":["../../src/commands/spec.command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;GAaG;AA0CH,OAAO,EAUL,gBAAgB,EAMjB,MAAM,qBAAqB,CAAC;AAC7B,OAAO,EAKL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAUhC,eAAO,MAAM,qBAAqB,iCAAiC,CAAC;AACpE,eAAO,MAAM,wBAAwB,oCAAoC,CAAC;AAY1E,eAAO,MAAM,iBAAiB,EAAE,eA2K/B,CAAC;AAcF,eAAO,MAAM,iBAAiB,EAAE,eAiE/B,CAAC;AAqBF,eAAO,MAAM,eAAe,EAAE,eAmC7B,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,eA8B7B,CAAC;AAEF,eAAO,MAAM,eAAe,EAAE,eA2C7B,CAAC;AAEF,eAAO,MAAM,iBAAiB,EAAE,eAoE/B,CAAC;AASF,eAAO,MAAM,oBAAoB,EAAE,eAsRlC,CAAC;AAyDF,eAAO,MAAM,iBAAiB,EAAE,eAqP/B,CAAC;AAEF,eAAO,MAAM,iBAAiB,EAAE,eA8B/B,CAAC;AAkBF,OAAO,EAAE,gBAAgB,EAAE,CAAC"}

package/dist/commands/spec.command.js

@@ -19,6 +19,7 @@ import * as nodePath from 'node:path';
 import { appendSpecEvent, buildSavedPlan, buildSpecId, canonicalJson, deriveSpecJson, FileChangeType, generate, listSpecIds, loadSpec, OverwriteStrategy, persistSpecArtifacts, PLAN_SECRET_ENV, readSpecEvents, renderSpecMd, savePlanToFile, signPlan, specDir, specMdPath, specPlanPath, specVerificationPath, splitSpecMd, SPEC_SCHEMA_V1, SpecStatus, validateSpecStructural, verifyPlan, writeSpecJson, writeSpecMd, } from '@shrkcrft/generator';
 import { AssetProvenanceOperation, AssetProvenanceSource, buildSpecList, buildSpecReview, buildTaskPacket, inspectSharkcraft, mapChecklistToEvidence, recordProvenance, SPEC_LIST_SCHEMA, } from '@shrkcrft/inspector';
 import { flagBool, flagList, flagString, resolveCwd, } from "../command-registry.js";
+import { PositionalMode } from "../dispatch/positional-mode.js";
 import { collectChangedPaths } from "../diff/collect-changed-paths.js";
 import { asJson, header } from "../output/format-output.js";
 // Bonus DX item — `spec implement --apply` now dispatches into
@@ -1005,6 +1006,8 @@ export const specVerifyCommand = {
 };
 export const specParentCommand = {
     name: 'spec',
+    // Mixed-mode: every spec verb is a trie child.
+    positionals: PositionalMode.None,
     description: 'Spec-driven development. Subcommand required: create | review | implement | verify | list | show | status | lint.',
     usage: 'shrk spec <create|review|implement|verify|list|show|status|lint> [...]',
     async run(args) {

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

@@ -1 +1 @@
-{"version":3,"file":"stats.command.d.ts","sourceRoot":"","sources":["../../src/commands/stats.command.ts"],"names":[],"mappings":"AACA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAoBhC,eAAO,MAAM,YAAY,EAAE,eAqF1B,CAAC"}
+{"version":3,"file":"stats.command.d.ts","sourceRoot":"","sources":["../../src/commands/stats.command.ts"],"names":[],"mappings":"AACA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAqBhC,eAAO,MAAM,YAAY,EAAE,eAuF1B,CAAC"}

package/dist/commands/stats.command.js

@@ -1,5 +1,6 @@
 import { buildRepositoryStats } from '@shrkcrft/inspector';
 import { flagBool, flagString, resolveCwd, } from "../command-registry.js";
+import { PositionalMode } from "../dispatch/positional-mode.js";
 import { asJson, header, kv } from "../output/format-output.js";
 const BYTES_PER_KB = 1024;
 const BYTES_PER_MB = BYTES_PER_KB * 1024;
@@ -18,6 +19,8 @@ function leftPad(s, n) {
 }
 export const statsCommand = {
     name: 'stats',
+    // Flag-driven (`--language <id>` scopes it): a bare token used to be ignored.
+    positionals: PositionalMode.None,
     description: 'Repository statistics — per-language file counts, lines of code (code/comment/blank), bytes, averages, largest files.',
     usage: 'shrk [--cwd <dir>] stats [--json] [--top <n>] [--language <id>]',
     async run(args) {

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

@@ -8,9 +8,14 @@ import { type ICommandHandler } from '../command-registry.js';
  *   - disable <command>     undo a prior enable
  *   - hide    <command>     hide an extended command from --help
  *   - unhide  <command>     reverse hide
- *   - reset                 clear surface.enabled + surface.hidden
+ *   - deny    <selector>    disable a command or a group (`<group> *`)
+ *   - allow   <selector>    reverse a deny
+ *   - reset                 clear surface.enabled + hidden + disabled
  *   - explain <command>     why this command has its current tier
  *   - profiles [get <id>]   list surface profiles (or show one)
+ *
+ * Every mutation edits the project config's OWN `surface{}` block (never the
+ * profile-merged view) and preserves its `profile`.
  */
 export declare const surfaceCommand: ICommandHandler;
 //# sourceMappingURL=surface.command.d.ts.map

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

@@ -1 +1 @@
-{"version":3,"file":"surface.command.d.ts","sourceRoot":"","sources":["../../src/commands/surface.command.ts"],"names":[],"mappings":"AAAA,OAAO,EAIL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAiBhC;;;;;;;;;;;;GAYG;AACH,eAAO,MAAM,cAAc,EAAE,eAmC5B,CAAC"}
+{"version":3,"file":"surface.command.d.ts","sourceRoot":"","sources":["../../src/commands/surface.command.ts"],"names":[],"mappings":"AACA,OAAO,EAGL,KAAK,eAAe,EAErB,MAAM,wBAAwB,CAAC;AAyBhC;;;;;;;;;;;;;;;;;GAiBG;AACH,eAAO,MAAM,cAAc,EAAE,eA6D5B,CAAC"}