@skrr-ai/cli 0.1.116 → 0.1.118

package/dist/generated/managed-instruction-guides.json

@@ -2,7 +2,7 @@
     "oversky-task-management": {
         "key": "oversky-task-management",
         "name": "skrr Task Management",
-        "templateVersion": 20,
+        "templateVersion": 22,
         "body": "## OverSky Task Management\n\nUse `skrr` as the durable record for work that must outlive this conversation.\n\n### One concept, one record\n\n- **Task** says what is owed and where it stands.\n- **Expectation** says what must be true before the Task can close.\n- **Update** says what meaningfully changed during execution.\n- **Resume Point** says where one Run can safely continue after interruption.\n- **Result** says what the work concluded or returned.\n- **Deliverable** identifies a concrete Resource version produced by the work.\n- **Review** says whether another actor accepted one exact Result.\n- **Comment** is replyable collaboration.\n- **Activity** is the immutable audit projection. A Run transcript is detailed telemetry, not Activity.\n\nDo not put a Result in a Comment, use an Update as a transcript, or use review words such as\n“approved” for a domain conclusion such as NO-SHIP.\n\nThe platform derives Human, Agent, Run, session, and recording provenance from the authenticated\nruntime. Never supply or invent an Agent identity. An Update has one author; a multi-Agent summary\nmay cite contributors and their source Runs or Updates.\n\n### During execution: write Updates at meaningful boundaries\n\nDo not log routine reads, tool calls, heartbeats, token counts, or repeated “still working” narration.\nRecord an Update only when a future reader or replacement Run needs to know that shared state changed:\n\nEach type has a REQUIRED `--data` shape, and the array fields are required even when empty.\nOptional fields are marked `?`. The server rejects a missing required field, so guessing from the\nprose costs a round trip:\n\n- `progress`: a material part is complete.\n  `{completedSinceLast: string[], nextActions: string[], currentFocus?}`\n- `finding`: a newly discovered fact. `impact` is optional; the evidence references are not.\n  `{finding, evidenceRefs: [], impact?, severity?}`\n- `decision`: a choice now constrains the work.\n  `{decision, rationale, alternativesConsidered?: string[]}`\n- `blocker`: work cannot continue. `nextOwner` is a principal OBJECT, not a name.\n  `{cause, requiredAction, nextOwner: {kind: \"human\", userId}}`\n- `verification`: a claim was checked. `result` is one of `passed`, `failed`, `inconclusive`.\n  `{claim, result, evidenceRefs: []}`\n- `ready_for_review`: created by Result submission; do not author it separately.\n\n`--reference <kind>:<id>` sets the Update's OWN references. It does not populate\n`data.evidenceRefs`, which belongs inside `--data`.\n\nUse one stable idempotency key only for retries of the identical write. Reusing it for different\ncontent is rejected.\n\nExample:\n\n```bash\nskrr tasks updates add <task-id> \\\n  --type progress \\\n  --summary \"Migration review complete\" \\\n  --data '{\"completedSinceLast\":[\"Checked migrations 001-014\"],\"currentFocus\":\"Authorization review\",\"nextActions\":[\"Inspect legacy adapters\"]}' \\\n  --idempotency-key \"<task-id>:migration-review\"\n```\n\nFor a decision:\n\n```bash\nskrr tasks updates add <task-id> \\\n  --type decision \\\n  --summary \"Use a first-class Resource identity\" \\\n  --data '{\"decision\":\"Use WorkResource plus immutable versions\",\"rationale\":\"A URL can move and cannot identify the version a reviewer accepted\"}' \\\n  --idempotency-key \"<task-id>:resource-model\"\n```\n\nFor a finding — `evidenceRefs` is required and may be empty:\n\n```bash\nskrr tasks updates add <task-id> \\\n  --type finding \\\n  --summary \"The closure gate reads assessments by exact specVersion\" \\\n  --data '{\"finding\":\"A cosmetic title edit bumps specVersion, so every prior assessment stops counting\",\"impact\":\"A task with expectations cannot be closed after a rename\",\"evidenceRefs\":[{\"kind\":\"code\",\"uri\":\"api/server/services/TaskClosurePolicy/index.js\"}]}' \\\n  --idempotency-key \"<task-id>:spec-bump-revocation\"\n```\n\nFor a blocker — `nextOwner` is a principal object:\n\n```bash\nskrr tasks updates add <task-id> \\\n  --type blocker \\\n  --summary \"Waiting on production database credentials\" \\\n  --data '{\"cause\":\"The migration needs write access to the prod cluster\",\"requiredAction\":\"Grant the deploy role write access, or run the migration yourself\",\"nextOwner\":{\"kind\":\"human\",\"userId\":\"<user-id>\"}}' \\\n  --idempotency-key \"<task-id>:needs-prod-credentials\"\n```\n\nFor a verification:\n\n```bash\nskrr tasks updates add <task-id> \\\n  --type verification \\\n  --summary \"Re-ran the closure flow end to end after the fix\" \\\n  --data '{\"claim\":\"A renamed task with one assessed expectation can close\",\"result\":\"passed\",\"evidenceRefs\":[{\"kind\":\"task\",\"id\":\"<task-id>\"}]}' \\\n  --idempotency-key \"<task-id>:closure-reverify\"\n```\n\n### Media: screenshots and recordings\n\n`--attach <path>` (repeatable) on `updates add`, `result submit`, `complete`, or `comments add`\nuploads a file as a Task asset and cites it; identical bytes dedupe. In-run proof goes on a\n`verification` or `progress` Update, the final before/after on the Result, never the description.\nAn upload is `self_reported` evidence and counts toward the storage quota. Images keep full\nresolution minus metadata (Runs get a model-sized copy); undecodable bytes get 422, a type the bytes\ncontradict 415, and SVG downloads as a file. An agent uploads evidence (the default) or\n`--role deliverable`; `input` is refused (403): ask a person to run `attachments set-role`. Cite a\ndeliverable as `--evidence task_attachment:<asset-id>` when you assess its Expectation.\n\n### Recovery: Update delta and Resume Point snapshot\n\nWhen a semantic boundary should also be resumable, add `--run-id` and `--resume` to the same Update\ncommand. The server commits the visible Update and the Run-bound Resume Point together. The snapshot\nmust be self-contained enough for another capable Agent to continue without replaying the transcript:\ncompleted work, current focus, next actions, open questions, and references to decisions, evidence,\nDeliverables, and durable workspace state.\n\nThe runtime may save a silent crash-safety snapshot with:\n\n```bash\nskrr tasks resume save <task-id> --run-id <run-id> --snapshot '<resume-json>'\n```\n\nNever put credentials, secrets, permission grants, bearer tokens, or leases in a Resume Point. A\nresumed Run reloads the current Task Revision, re-authorizes, validates referenced files/commits, and\nreads Updates recorded after the snapshot:\n\n```bash\nskrr tasks resume show <task-id> --run-id <run-id>\n```\n\n### Deliverables: lightweight input, first-class identity\n\nA user may provide one coordinate, but the platform records a WorkResource, its exact version, the\nTask relationship, and an evidence-backed availability observation. External reference is the\ndefault; OverSky-managed content is used only when retention, stable access, preview/search, or\npolicy requires a copy.\n\n```bash\nskrr tasks deliverable add <task-id> \\\n  file/markdown:docs/reviews/report.md@docs/reviews/report.md \\\n  --status in_review\n```\n\nA recording made by its producer is always self-reported; the caller cannot promote itself to a\nHuman confirmation, system observation, or provider receipt. A local path is only local availability. A commit, provider revision, digest, or managed object\nfreezes the exact version. Do not call “in the conversation” a Deliverable.\n\nA Human confirms the exact linked version, not a retyped URL or name:\n\n```bash\nskrr tasks deliverable confirm <task-id> <task-resource-link-id> --reason \"Opened and checked this exact version\"\n```\n\n### Result and Review are separate\n\nSubmit the finished answer once:\n\n```bash\nskrr tasks result submit <task-id> \\\n  --summary \"NO-SHIP until remediation Gates A-D pass\" \\\n  --conclusion-type release_recommendation \\\n  --conclusion-json '{\"posture\":\"no_ship\"}' \\\n  --deliverable <task-resource-link-id>\n```\n\nAn amendment creates a new Result version and explicitly supersedes the previous one. There is one\ncurrent Result; old versions remain in Activity and are not ordinary Updates.\n\nOnly another actor records acceptance, against that exact Result:\n\n```bash\nskrr tasks review <task-id> --result-id <result-id> --decision accepted --rationale \"Evidence checked\"\n```\n\n“Accepted” means the submitted work was accepted. NO-SHIP, BUY, REJECT, root cause, experiment\ndecision, and similar subject-matter conclusions belong in `Result.conclusion`, never in Review.\n\nUse `skrr tasks complete` only when submitting the Result and advancing workflow in one closeout.\nDo not call `tasks report` before or after it; `tasks complete` already creates the canonical Result.\n\nSubmitting a Result is not the same as closing the Task, and being an Agent is not by itself a\nreason to stop. Closing is always the closer's call — unmet Expectations or a Changes-requested\nReview do not prevent it; they are RECORDED on the close (`resolution.closure`), so closing over\nthem hides nothing and settles nothing. The one exception is an owner-declared required Review\n(`--review-policy required`): a non-owner cannot close without it — unless they close with\n`--force`, which records the override on the resolution — and the owner always can.\n\nSo do not predict the outcome. `result submit` reports where the Task stands after the write —\nread that, and if you need to ask again later:\n\n```bash\nskrr tasks expectations <task-id>     # what is owed, and why the Result cannot be accepted yet\n```\n\nSay “ready for review” only when something you read actually said so.\n\n### Expectations\n\nExpectations are free text first. A typed check explains how the statement can be assessed; it never\nreplaces the sentence.\n\nAn Expectation is an obligation, not a lock: it does not stop a close, it is recorded on one — and\nclosing over it settles nothing. Write each one so it can\nbe judged — say what would satisfy it and how that could be checked — because an Expectation authored\nwithout a thought about verification is one you will later be unable to settle honestly. When you\ncreate several across several Tasks, say how many you created and who they fall to; a set of gates\nnobody was told about is work assigned by accident. A Deliverable requirement is one typed Expectation, not a second\n`requiredDeliverables` list. Actual Deliverables are recorded only after something exists.\n\nRead what is owed, then settle it — never rewrite the declaration to make it settleable:\n\n```bash\nskrr tasks expectations <task-id>                       # what is owed, and how far each one got\nskrr tasks expectations assess <task-id> <expectation-id> \\\n  --decision met \\\n  --reason \"Ran the suite named in the statement; 49/49 pass\" \\\n  --evidence code:<commit>\nskrr tasks expectations assess <task-id> --all \\\n  --decision met --reason \"...\"                         # every one still awaiting a judgement\n```\n\n**You may assess any Expectation that takes a judgement.** A `deliverable` Expectation is not one —\nit settles through its linked Resource and evidence, so record the Deliverable rather than\nassessing it. The assessment is recorded as YOURS — every surface shows `met (agent)` against\n`met (person)` — and a human can override it. The control is attribution, not permission, so what\nyou owe is not restraint but evidence: cite what you actually checked, and prefer `--evidence` and\n`--resource-link-id` over prose.\n\nAssess what you VERIFIED. Not what you intended, not what you inferred from having finished the\nwork. “I could not check this” is a real and useful answer — record it `unmet` with the reason\nrather than leaving it silent, because an unassessed Expectation and one you tried and failed to\nverify send the next reader to different places.\n\nA waiver is a judgement about scope — that this is no longer owed — and it stays a person's call:\nif the honest answer is \"this should be waived,\" say so in an Update or Comment and let a person\nrecord it. Do not delete the Expectation to hide that it was owed.\n\nChanging an Expectation's wording advances the Task specification and **revokes every assessment\nalready recorded** — fix the statement before people start settling it, not after.\n\n### Labels — say what PROGRAM this belongs to\n\nA label is cross-cutting classification: which recurring program a Space or Task\nbelongs to (`dogfood`, `incident`, `experiment`, `migration`), not what state it\nis in. Status, priority and assignee are first-class fields and already answer\ntheir own questions — a label that restates one is noise that will drift out of\nsync with the field it copies.\n\nThe vocabulary is DECLARED, not invented. Read it before you use it:\n\n```bash\nskrr labels list\n```\n\nAttach from that list, never a name you made up. A second spelling of an\nexisting label (`dog-food` beside `dogfood`) is worse than no label at all,\nbecause the whole value of one is that filtering on it returns everything:\n\n```bash\nskrr spaces labels add <space-id> --label <label-id>\nskrr tasks labels attach <task-id> --label <label-id>\n```\n\nCreate a label only when the program it names is real and recurring, and say\nwhich entities it applies to — a label created without `--applicable-to space`\ncannot be attached to a Space, which is how a vocabulary comes to exist that\nnothing can use:\n\n```bash\nskrr labels create --name <name> --applicable-to space --applicable-to task\n```\n\nWhen you create a Space or Task for a recurring program, label it as you create\nit (`--label <id>` on `spaces`, `tasks` and `goals create`). The create\nresponse carries a `_labelGuidance` hint naming the applicable vocabulary WITH\nITS IDS when you attached none — read it, and attach in one follow-up call if\none of them fits. It may also carry `suggested` — a proposal, not a decision;\nattach is your call. `skrr tasks labels suggest <task-id>` asks it of a Task.\n\nIf nothing in the vocabulary fits, attach nothing. \"No label\" is a correct\nanswer and a wrong label is not.\n\n### Comments and blockers\n\nUse a Comment only for a question, answer, or review discussion where a reply is useful. For a\nblocker, first record the typed blocker Update, then move the Task to blocked. Add a Comment only if\nsomeone actually needs to respond.\n\n### Creating and placing Tasks\n\nCreate a Task without asking when work needs tracking across meaningful steps, sessions, or\nfollow-up. Avoid duplicates and do not create Tasks for trivial actions that finish immediately.\n\n```bash\nskrr tasks list --q \"<relevant title>\"\nskrr tasks create --title \"<concise requested result>\" --description \"<scope and constraints>\" --json\nskrr tasks move <task-id> --status in_progress\n```\n\nA Space is a durable ownership, collaboration, and authorization boundary. Use an identified Space\nwhen the work clearly belongs there; otherwise create the Task unscoped. Ask before creating a new\nSpace. An Agent works a Task in a Space only as a Space member, or when it is shared with no one\nelse and its owner assigns and starts it; anything else is refused. Do not widen Agent membership\nmerely to force one task assignment.\n\n### Closing a Task out\n\nThe model above says what each record IS. This is the order they go in, and what each step can\nrefuse:\n\n1. **Record what exists** — `skrr tasks deliverable add`, binding it to the Expectation it answers\n   with `--expectation <id>` whenever one applies. Evidence that arrives bound is evidence the next\n   reader does not have to reconstruct.\n2. **Submit the answer** — `skrr tasks result submit`. This does not close the Task. Read what it\n   reports back.\n3. **Read what is still owed** — `skrr tasks expectations <task-id>`.\n4. **Settle what you verified** — `skrr tasks expectations assess`, citing evidence. `--all` covers\n   every Expectation still awaiting a judgement.\n5. **Read the Task state again and report THAT** — not what you intended it to be.\n\nTwo things still matter regardless of how complete the work is: an unmet Expectation, and a\n`changes_requested` Review. Neither blocks a close — but both are stamped onto it\n(`resolution.closure`), so closing over them is a recorded choice, not a clean finish. Neither is\na failure state; both are somebody's turn.\n\nStep 5 is the one that goes wrong quietly, and not only here: before reporting a check as passed,\nconfirm you measured what you think you measured. A pipeline's exit status is the LAST command's, so\n`cmd | tail` reports on `tail`. A filter may match something other than what you meant — a path\npattern can match the directory you are working in. When a result will be reported to a person as\nfact, re-run it in the simplest form that can only mean one thing.\n\nAcross a whole Space, `skrr tasks review-queue --space <space-id>` names every outstanding\njudgement at once, with its evidence and the command that settles it.\n\n### Share readable links and close the loop\n\nShare clickable Task, Space, Goal, and Commitment URLs, never bare IDs. Use the URL returned by the\nCLI; do not construct one. If needed:\n\n```bash\nskrr tasks show <task-id> --json\nskrr spaces show <space-id> --json\nskrr goals show <goal-id> --json\nskrr commitments show <commitment-id> --json\n```\n\nBefore your final response, verify every Task you created or touched reflects reality. If work is\nready for independent review, leave it in review. If blocked, record the blocker and required action.\nIf genuinely complete and accepted under the Task's policy, close it. Include the Task URL in the\nfinal handoff. Never claim a write succeeded unless the CLI returned success.\n\n### Coming back later\n\nYou can arrange to return to something. Four shapes, and picking the right one\nmatters more than the wording:\n\n**Reminder** — the user asked to be reminded. Keep their words; a reminder you\nhave rephrased into your own voice is no longer the thing they asked for.\n\n**Watch** — something has an asynchronous tail: CI, a review, an approval, a\nlong job, a metric. If the condition is machine-checkable (an endpoint plus a\npredicate), say so — that costs nothing and is guaranteed to terminate. Only\nwrite a prose check recipe when the check genuinely needs your judgement or your\ntools, because that one burns a full turn every time it runs. Always give it an\nabort condition: a watch that cannot stop is a leak.\n\n**Check-in** — you decided the conversation is worth returning to. This is\npermission AND restraint. Come back when you will have something worth saying —\na result that will have arrived, a decision that will be ripe, a thread you left\ngenuinely open. When you see one, arrange the return yourself and say in one\nline when you will look again. Do not ask whether you should: that hands the\nperson a decision they did not come to make. Do not come back to ask how it is\ngoing. A legacy check-in holds one pending return per conversation. For a sequence\nof checks, use one scheduled lookback plan with 1–10 lifetime steps. When you\nwake, read the original goal, current state and earlier outcomes. Saying\nnothing is a legitimate outcome and often the right one. Stop or cancel\nremaining checks when the work resolves, becomes obsolete or the person cancels.\n\n**Act** — a specific action you already know how to take and the user has already\nagreed to. Not for anything you would still need to ask about.\n\nSettle what you arranged. When you wake and the thing resolved, say so and close\nit out; when it did not, either come back once more with a reason or hand it to\nthe person. Leaving one armed forever is the failure mode here.\n\nFrom a local session, arrange these with `skrr followups` —\n`remind`, `watch`, `checkin`, `act` — and settle them with\n`skrr followups resolve <id> --outcome resolved`.\nFor a bounded lookback plan, use `skrr followups plans create --from-json plan.json`\nand inspect, revise or cancel it with `skrr followups plans` (read `--help`).\nUse 1–10 concrete checks, relative offsets from the completed reply or explicit\nabsolute/local times, and a stop condition. Omit a plan when none is useful.\nA saved plan is not necessarily armed; read the returned receipt before promising a return.\n",
         "hash": "a4bcb9d4c39e33e8a1dcd12b8a669efe80c67d81c3a8ad90119be73f8352c8b6"
     }

package/dist/help.js

@@ -48,6 +48,7 @@ class SkrrHelp extends core_1.Help {
         this.log('GETTING STARTED');
         this.log('  skrr login             Sign in. Offers to install the background service if this computer has none.');
         this.log('  skrr daemon install    Install the background service by hand (fetches the signed release).');
+        this.log("  skrr guide             Read a Skrr domain's guide before you write in it.");
         this.log('');
         return super.showRootHelp();
     }
@@ -73,11 +74,11 @@ class SkrrHelp extends core_1.Help {
                     ...(flagStart === -1 ? [] : argv.slice(flagStart)),
                 ]);
             }
-            throw new Error((0, command_miss_1.describeCommandMiss)(lookup, id));
+            throw new Error((0, command_miss_1.describeCommandMiss)(lookup, id, words));
         }
-        if (id === 'tasks' && this.config.findCommand('tasks:guide')) {
+        if (id === 'tasks' && this.config.findCommand('guide')) {
             this.log('TASK OPERATIONS');
-            this.log(`  ${this.config.bin} tasks guide              Find planning, dependencies, execution, and result commands.`);
+            this.log(`  ${this.config.bin} guide tasks               Read the tasks guide: planning, dependencies, execution, and results.`);
             if (this.config.findCommand('tasks:preflight'))
                 this.log(`  ${this.config.bin} tasks preflight <task>   Inspect blockers and execution prerequisites without starting work.`);
             this.log('');

package/dist/hooks/command-not-found.js

@@ -102,7 +102,7 @@ const hook = async function (opts) {
     // — a typo would silently look like success, which is far worse than the
     // message this hook was written to improve. (I shipped exactly that for one
     // build; `skrr shortcuts lst` said nothing and exited 0.)
-    throw new Error((0, command_miss_1.describeCommandMiss)(lookup, id));
+    throw new Error((0, command_miss_1.describeCommandMiss)(lookup, id, words));
 };
 exports.default = hook;
 class VirtualUsageFlagError extends Error {

package/dist/lib/action-listings.js

@@ -4,6 +4,7 @@ exports.ACTION_LISTING_SORTS = exports.ACTION_LISTING_CATEGORIES = void 0;
 exports.renderActionListings = renderActionListings;
 exports.renderActionListing = renderActionListing;
 const format_1 = require("./format");
+const store_listing_1 = require("./store-listing");
 exports.ACTION_LISTING_CATEGORIES = [
     'code',
     'data',
@@ -47,7 +48,7 @@ function renderActionListing(listing, log) {
     log(`Kind:        ${listing.kind}`);
     log(`Category:    ${listing.category}`);
     log(`Public:      ${listing.isPublic ? 'yes' : 'no'}`);
-    log(`Price:       ${listing.price ?? 0}`);
+    log(`Price:       ${(0, store_listing_1.formatListingPrice)(listing.price)}`);
     log(`Installs:    ${listing.installCount ?? 0}`);
     log(`Rating:      ${listing.ratingCount ? `${listing.rating} (${listing.ratingCount})` : '-'}`);
     if (listing.builtinType)

package/dist/lib/agent-guide-reference-gate.d.ts

@@ -0,0 +1,28 @@
+import { type CommandManifest } from './command-manifest';
+export type AgentGuideSourceLine = {
+    text: string;
+    line: number;
+};
+export type AgentGuideSourceDocument = {
+    guide: string;
+    section: string;
+    surface: 'cli' | 'mcp';
+    file: string;
+    lines: readonly AgentGuideSourceLine[];
+};
+export type McpToolSource = {
+    server: string;
+    tools: readonly string[];
+    /** Argument names from the tools' input schemas; never mistaken for a tool. */
+    fields?: readonly string[];
+};
+/** Validates the CLI surface against the command manifest the CLI ships. */
+export declare function assertCliGuideReferences(documents: readonly AgentGuideSourceDocument[], manifest: CommandManifest, globalFlags?: readonly string[]): void;
+/**
+ * Validates explicitly named MCP tools. Unknown snake_case values are only
+ * treated as tools when their namespace exists and prose calls them a tool
+ * right beside the name; a tool argument (`agent_id`, from the input schemas)
+ * and fields such as `task_id` stay ordinary prose rather than gate false
+ * positives.
+ */
+export declare function assertMcpGuideToolReferences(documents: readonly AgentGuideSourceDocument[], sources: readonly McpToolSource[]): void;

package/dist/lib/agent-guide-reference-gate.js

@@ -0,0 +1,186 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.assertCliGuideReferences = assertCliGuideReferences;
+exports.assertMcpGuideToolReferences = assertMcpGuideToolReferences;
+const data_provider_1 = require("@skrr-ai/data-provider");
+const command_miss_1 = require("./command-miss");
+const command_manifest_1 = require("./command-manifest");
+const FENCE = /^ {0,3}(`{3,}|~{3,})(.*)$/;
+const INLINE_CODE = /`([^`\n]+)`/g;
+const FLAG = /(?<!-)--([a-z][a-z0-9-]*)\b/g;
+const BARE_TOOL = /^[a-z][a-z0-9]*(?:_[a-z0-9]+)+$/;
+const PREFIXED_TOOL = /mcp__[a-z0-9-]+__[a-z0-9_]+/g;
+function location(document, line, problem) {
+    return new Error(`[AgentGuides] guide "${document.guide}", section "${document.section}", ` +
+        `surface "${document.surface}" (${document.file}:${line}): ${problem}`);
+}
+function commandWords(text) {
+    const words = [];
+    for (const token of text.trim().split(/\s+/)) {
+        if (token.startsWith('--') || token.startsWith('<') || token.startsWith('['))
+            break;
+        if (!/^[a-z][a-z0-9-]*$/i.test(token))
+            break;
+        words.push(token.toLowerCase());
+    }
+    return words;
+}
+function codeSpans(document) {
+    const spans = [];
+    let fence = null;
+    for (const source of document.lines) {
+        const marker = FENCE.exec(source.text);
+        if (fence) {
+            if (marker &&
+                marker[1][0] === fence[0] &&
+                marker[1].length >= fence.length &&
+                !marker[2].trim()) {
+                fence = null;
+            }
+            else {
+                spans.push(source);
+            }
+            continue;
+        }
+        if (marker) {
+            fence = marker[1];
+            continue;
+        }
+        for (const match of source.text.matchAll(INLINE_CODE)) {
+            spans.push({ text: match[1], line: source.line });
+        }
+    }
+    return spans;
+}
+function commandCatalog(manifest) {
+    const ids = [];
+    const spellings = [];
+    const flags = new Set();
+    for (const entry of (0, command_manifest_1.visibleCommandEntries)(manifest)) {
+        ids.push(entry.id);
+        for (const name of [entry.command, ...entry.aliases]) {
+            const words = commandWords(name.replace(/:/g, ' '));
+            if (words.length)
+                spellings.push({ words, id: entry.id });
+        }
+        for (const [name, flag] of Object.entries(entry.flags)) {
+            flags.add(name);
+            // oclif accepts `--no-<name>` for a boolean declared with `allowNo`.
+            if (flag.allowNo === true)
+                flags.add(`no-${name}`);
+            const aliases = flag.aliases;
+            if (Array.isArray(aliases)) {
+                for (const alias of aliases) {
+                    if (typeof alias === 'string')
+                        flags.add(alias.replace(/^--/, ''));
+                }
+            }
+        }
+    }
+    return { ids, spellings, flags };
+}
+/** Validates the CLI surface against the command manifest the CLI ships. */
+function assertCliGuideReferences(documents, manifest, globalFlags = []) {
+    const catalog = commandCatalog(manifest);
+    for (const flag of globalFlags)
+        catalog.flags.add(flag.replace(/^--/, ''));
+    for (const document of documents) {
+        if (document.surface !== 'cli')
+            continue;
+        for (const span of codeSpans(document)) {
+            for (const match of span.text.matchAll(/\bskrr(?:\s+[^\s`]+)*/g)) {
+                const typed = commandWords(match[0].slice('skrr'.length));
+                if (!typed.length)
+                    continue;
+                const matching = catalog.spellings
+                    .filter((candidate) => candidate.words.every((word, index) => typed[index] === word))
+                    .sort((a, b) => b.words.length - a.words.length)[0];
+                if (matching)
+                    continue;
+                const id = typed.join(':');
+                // A bare topic in prose (for example `skrr followups`) names a
+                // command family, not an invocation of a missing no-argument command.
+                if (catalog.ids.some((known) => known.startsWith(`${id}:`)))
+                    continue;
+                const nearest = (0, command_miss_1.nearestCommands)(id, catalog.ids)[0];
+                throw location(document, span.line, `unknown \`skrr ${typed.join(' ')}\` command${nearest ? `; nearest real command: \`skrr ${nearest.replace(/:/g, ' ')}\`` : ''}`);
+            }
+        }
+        for (const source of document.lines) {
+            for (const match of source.text.matchAll(FLAG)) {
+                const name = match[1];
+                if (catalog.flags.has(name))
+                    continue;
+                const nearest = (0, command_miss_1.nearestCommands)(name, [...catalog.flags])[0];
+                throw location(document, source.line, `unknown flag \`--${name}\`${nearest ? `; nearest real flag: \`--${nearest}\`` : ''}`);
+            }
+        }
+    }
+}
+function isToolReference(line, start, end, name, known) {
+    if (known.has(name))
+        return true;
+    const before = line.slice(0, start);
+    const after = line.slice(end);
+    return (/\b(?:call|use|invoke|run)\s*$/i.test(before) ||
+        // "the `x` tool" / "tool `x`": the word must sit beside the name, since a
+        // line that mentions tools elsewhere often names an argument too.
+        /\b(?:MCP\s+)?tools?\s*$/i.test(before) ||
+        /^\s*(?:MCP\s+)?tool\b/i.test(after) ||
+        /^\s*(?:[-*]|\d+\.)\s*$/.test(before));
+}
+function nearestTool(name, names) {
+    return (0, data_provider_1.nearestAgentGuideIds)(name, names, 1)[0] ?? '';
+}
+/**
+ * Validates explicitly named MCP tools. Unknown snake_case values are only
+ * treated as tools when their namespace exists and prose calls them a tool
+ * right beside the name; a tool argument (`agent_id`, from the input schemas)
+ * and fields such as `task_id` stay ordinary prose rather than gate false
+ * positives.
+ */
+function assertMcpGuideToolReferences(documents, sources) {
+    const owners = new Map();
+    for (const source of sources) {
+        for (const name of source.tools) {
+            const servers = owners.get(name) ?? new Set();
+            servers.add(source.server);
+            owners.set(name, servers);
+        }
+    }
+    const names = [...owners.keys()];
+    const known = new Set(names);
+    const argumentNames = new Set(sources.flatMap((source) => source.fields ?? []));
+    const namespaces = new Set(names.map((name) => name.split('_', 1)[0]));
+    for (const document of documents) {
+        if (document.surface !== 'mcp')
+            continue;
+        for (const source of document.lines) {
+            for (const match of source.text.matchAll(PREFIXED_TOOL)) {
+                const name = match[0].split('__').at(-1) ?? match[0];
+                const nearest = nearestTool(name, names);
+                throw location(document, source.line, `MCP tools use bare names, not \`${match[0]}\`${nearest ? `; nearest real tool: \`${nearest}\`` : ''}`);
+            }
+            for (const match of source.text.matchAll(INLINE_CODE)) {
+                const name = match[1];
+                if (!BARE_TOOL.test(name))
+                    continue;
+                if (argumentNames.has(name) && !known.has(name))
+                    continue;
+                const namespace = name.split('_', 1)[0];
+                if (!namespaces.has(namespace) ||
+                    !isToolReference(source.text, match.index, match.index + match[0].length, name, known)) {
+                    continue;
+                }
+                const servers = owners.get(name);
+                if (!servers) {
+                    const nearest = nearestTool(name, names);
+                    throw location(document, source.line, `unknown MCP tool \`${name}\`${nearest ? `; nearest real tool: \`${nearest}\`` : ''}`);
+                }
+                if (servers.size > 1) {
+                    throw location(document, source.line, `MCP tool \`${name}\` is ambiguous across ${[...servers].sort().join(', ')}; name a unique bare tool`);
+                }
+            }
+        }
+    }
+}

package/dist/lib/agent-guides.d.ts

@@ -0,0 +1,51 @@
+import { type AgentGuideMiss, type AgentGuideResponse } from '@skrr-ai/data-provider';
+import type { Command } from '@oclif/core';
+export type AgentGuideRequest = {
+    id?: string;
+    section?: string;
+};
+export type AgentGuideFallbackReason = 'no-session' | 'server-unavailable' | 'server-hash-mismatch' | 'offline-requested';
+export type AgentGuideRead = {
+    found: true;
+    response: AgentGuideResponse;
+    source: 'server' | 'bundled';
+    /** Why the bundled copy answered; set only when `source` is 'bundled'. */
+    reason?: AgentGuideFallbackReason;
+} | {
+    found: false;
+    miss: AgentGuideMiss;
+};
+/** The flag subset a guide read consults; every caller's parsed flags satisfy it. */
+type AgentGuideReadFlags = {
+    offline?: boolean;
+    profile?: string;
+    token?: string;
+    bare?: boolean;
+};
+/**
+ * The one read path behind `skrr guide` and the legacy
+ * `instructions guide task-management`: the server when a credential answers,
+ * the bundled release copy otherwise, with a distinct `server-hash-mismatch` fallback
+ * reason (OSK-17944). Documentation stays readable with no session at all.
+ */
+export declare function readAgentGuide(request: AgentGuideRequest, flags: AgentGuideReadFlags): Promise<AgentGuideRead>;
+/**
+ * The `--json` body every guide read prints: the response itself plus where it
+ * was read from, so `skrr guide --json` and each kept alias say provenance the
+ * same way.
+ */
+export declare function agentGuideJson(read: Extract<AgentGuideRead, {
+    found: true;
+}>): string;
+/**
+ * The one fallback a text-mode reader is told about. No session, `--offline`
+ * and an unreachable server are ordinary reasons to read the bundled copy. A
+ * server whose body does not match its own hash is an integrity failure, so it
+ * is said on stderr and stdout stays the guide (OSK-17944).
+ */
+export declare function agentGuideFallbackNotice(read: Extract<AgentGuideRead, {
+    found: true;
+}>): string | null;
+/** A miss is steering, not a crash: print it with its nearest addresses and exit 1. */
+export declare function writeAgentGuideMiss(command: Command, miss: AgentGuideMiss, json: boolean): never;
+export {};

package/dist/lib/agent-guides.js

@@ -0,0 +1,170 @@
+"use strict";
+var __importDefault = (this && this.__importDefault) || function (mod) {
+    return (mod && mod.__esModule) ? mod : { "default": mod };
+};
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.readAgentGuide = readAgentGuide;
+exports.agentGuideJson = agentGuideJson;
+exports.agentGuideFallbackNotice = agentGuideFallbackNotice;
+exports.writeAgentGuideMiss = writeAgentGuideMiss;
+const data_provider_1 = require("@skrr-ai/data-provider");
+const node_crypto_1 = require("node:crypto");
+const agent_guides_json_1 = __importDefault(require("../generated/agent-guides.json"));
+const auth_core_init_1 = require("./auth-core-init");
+const credential_resolver_1 = require("./credential-resolver");
+const harness_context_1 = require("./harness-context");
+const http_error_1 = require("./http-error");
+const keychain_1 = require("./keychain");
+const config_1 = require("./config");
+const node_adapter_1 = require("./node-adapter");
+const bundle = agent_guides_json_1.default;
+function responseHashMatches(response) {
+    return ((0, node_crypto_1.createHash)('sha256').update((0, data_provider_1.renderAgentGuideText)(response), 'utf8').digest('hex') ===
+        response.hash);
+}
+function guideMissFrom(error) {
+    if ((0, http_error_1.httpStatusOf)(error) !== 404)
+        return null;
+    const body = (0, http_error_1.httpBodyOf)(error);
+    if (body?.code !== 'GUIDE_NOT_FOUND' ||
+        typeof body.message !== 'string' ||
+        !Array.isArray(body.suggestions)) {
+        return null;
+    }
+    return body;
+}
+function readBundledGuide(request) {
+    if (!request.id)
+        return { found: true, response: bundle.routedIndex };
+    const resolved = (0, data_provider_1.resolveAgentGuideRequest)(bundle.index.guides, { id: request.id, ...(request.section ? { section: request.section } : {}) }, { surface: 'cli' });
+    if (!resolved.found)
+        return resolved;
+    const response = resolved.section
+        ? bundle.sections[`${resolved.id}/${resolved.section}`]
+        : bundle.cards[resolved.id];
+    if (!response) {
+        throw new Error('The bundled agent guides are incomplete. Run node cli/scripts/generate-agent-guides.cjs.');
+    }
+    return {
+        found: true,
+        response: resolved.aliasOf ? { ...response, aliasOf: resolved.aliasOf } : response,
+    };
+}
+async function readServerGuide(request, options) {
+    if (request.section) {
+        return data_provider_1.dataService.getAgentGuideSection(request.id ?? '', request.section, { surface: 'cli' }, options);
+    }
+    if (request.id)
+        return data_provider_1.dataService.getAgentGuide(request.id, { surface: 'cli' }, options);
+    return data_provider_1.dataService.getAgentGuideIndex({ surface: 'cli' }, options);
+}
+/**
+ * The one read path behind `skrr guide` and the legacy
+ * `instructions guide task-management`: the server when a credential answers,
+ * the bundled release copy otherwise, with a distinct `server-hash-mismatch` fallback
+ * reason (OSK-17944). Documentation stays readable with no session at all.
+ */
+async function readAgentGuide(request, flags) {
+    let response;
+    let miss;
+    let source = 'bundled';
+    let reason = flags.offline ? 'offline-requested' : 'no-session';
+    if (!flags.offline) {
+        (0, keychain_1.setActiveProfile)(flags.profile ?? process.env.OVERSKY_PROFILE ?? '');
+        let credential;
+        let autonomous = false;
+        try {
+            const trusted = (0, credential_resolver_1.resolveTrustedAutonomousCredential)();
+            autonomous = Boolean(trusted);
+            if (!trusted && !flags.bare && !(0, harness_context_1.isHarnessCliContext)())
+                await (0, auth_core_init_1.prefetchCliAuth)();
+            credential =
+                trusted ??
+                    (await (0, credential_resolver_1.resolveCredential)({
+                        flagToken: flags.token,
+                        bareMode: flags.bare,
+                        allowUnreadableStoredCredential: true,
+                    }));
+        }
+        catch {
+            credential = undefined;
+        }
+        if (credential?.token) {
+            try {
+                const baseURL = autonomous ? process.env.OVERSKY_SERVER_URL?.trim() : (0, config_1.loadConfig)().baseURL;
+                if (!baseURL)
+                    throw new Error('No API base URL is configured.');
+                (0, data_provider_1.setBaseURL)(baseURL);
+                (0, node_adapter_1.setAdapterCredentialOverride)(credential);
+                (0, data_provider_1.setHttpAdapter)((0, node_adapter_1.createNodeAdapter)());
+                const options = { signal: AbortSignal.timeout(3_000) };
+                response = await readServerGuide(request, options);
+                if (!responseHashMatches(response)) {
+                    response = undefined;
+                    reason = 'server-hash-mismatch';
+                }
+                else {
+                    source = 'server';
+                }
+            }
+            catch (error) {
+                miss = guideMissFrom(error) ?? undefined;
+                if (!miss)
+                    reason = 'server-unavailable';
+            }
+        }
+    }
+    if (miss)
+        return { found: false, miss };
+    if (!response) {
+        const bundled = readBundledGuide(request);
+        if (!bundled.found)
+            return bundled;
+        response = bundled.response;
+    }
+    return {
+        found: true,
+        response,
+        source,
+        ...(source === 'bundled' ? { reason } : {}),
+    };
+}
+/**
+ * The `--json` body every guide read prints: the response itself plus where it
+ * was read from, so `skrr guide --json` and each kept alias say provenance the
+ * same way.
+ */
+function agentGuideJson(read) {
+    return `${JSON.stringify({
+        ...read.response,
+        provenance: {
+            source: read.source,
+            fallback: read.source === 'bundled',
+            ...(read.reason ? { reason: read.reason } : {}),
+        },
+    }, null, 2)}\n`;
+}
+/**
+ * The one fallback a text-mode reader is told about. No session, `--offline`
+ * and an unreachable server are ordinary reasons to read the bundled copy. A
+ * server whose body does not match its own hash is an integrity failure, so it
+ * is said on stderr and stdout stays the guide (OSK-17944).
+ */
+function agentGuideFallbackNotice(read) {
+    return read.reason === 'server-hash-mismatch'
+        ? 'The server guide’s body does not match its hash; using the guide bundled with this CLI.'
+        : null;
+}
+/** A miss is steering, not a crash: print it with its nearest addresses and exit 1. */
+function writeAgentGuideMiss(command, miss, json) {
+    if (json) {
+        process.stdout.write(`${JSON.stringify({ ok: false, error: miss }, null, 2)}\n`);
+    }
+    else {
+        process.stdout.write(`${miss.message}\n`);
+        for (const suggestion of miss.suggestions) {
+            process.stdout.write(`${(0, data_provider_1.agentGuideReadCommand)(suggestion, 'cli')}\n`);
+        }
+    }
+    return command.exit(1);
+}

package/dist/lib/api-fetch.js

@@ -46,6 +46,7 @@ const delegated_cli_1 = require("./delegated-cli");
 const daemonBroker_1 = require("./daemonBroker");
 const dpop_auth_1 = require("./dpop-auth");
 const workspace_header_1 = require("./workspace-header");
+const task_runtime_headers_1 = require("./task-runtime-headers");
 const non_json_response_1 = require("./non-json-response");
 const rate_limit_retry_1 = require("./rate-limit-retry");
 class ApiFetchError extends Error {
@@ -189,12 +190,13 @@ async function apiFetch(pathOrUrl, opts = {}) {
         // OSK-12020 — a fresh proof per attempt for a DPoP-bound hand-off token
         // (the 401 retry below re-enters here and gets its own). No-op otherwise.
         Object.assign(headers, await (0, dpop_auth_1.dpopHeadersFor)(credential.token, method, url));
-        const sent = (0, workspace_header_1.withWorkspaceHeader)(headers);
+        const sent = (0, task_runtime_headers_1.withTaskRuntimeHeaders)((0, workspace_header_1.withWorkspaceHeader)(headers), url, method);
         // E6: a 429 on a GET or an idempotency-keyed write is retried (bounded,
         // jittered, honouring Retry-After). The last 429's wait is kept for the error.
         const { response, retryAfterMs } = await (0, rate_limit_retry_1.fetchWithRateLimitRetry)(() => fetch(url, {
             method,
             headers: sent,
+            ...(0, task_runtime_headers_1.taskRuntimeRedirectOptions)(sent),
             body,
             ...(opts.signal ? { signal: opts.signal } : {}),
         }), { method, headers: sent, signal: opts.signal });

package/dist/lib/command-miss.d.ts

@@ -29,6 +29,17 @@ export type CommandLookup = {
     commandIDs: string[];
     topicNames: string[];
 };
+/**
+ * Commands behind the Task spellings agents reach for before they have read
+ * the guide. Keep these ids here with their redirects so the manifest contract
+ * test can prove every printed target ships in this CLI build.
+ */
+export declare const TASK_COMMAND_MISS_TARGETS: Readonly<{
+    topic: "tasks";
+    search: "tasks:list";
+    resultShow: "tasks:result:show";
+    resultSubmit: "tasks:result:submit";
+}>;
 /** Cheap edit distance, bounded — this runs only on a miss. */
 export declare function editDistance(a: string, b: string): number;
 /**
@@ -68,7 +79,7 @@ export type CommandMiss = {
     suggestions: string[];
 };
 /** The text form: exactly what a miss has always printed. */
-export declare function describeCommandMiss(config: CommandLookup, id: string): string;
+export declare function describeCommandMiss(config: CommandLookup, id: string, typedWords?: readonly string[]): string;
 /**
  * The miss for a colon-joined id that did not resolve.
  *
@@ -81,7 +92,7 @@ export declare function describeCommandMiss(config: CommandLookup, id: string):
  *     suggest the nearest siblings under the same topic before falling back to
  *     the whole command list.
  */
-export declare function analyzeCommandMiss(config: CommandLookup, id: string): CommandMiss;
+export declare function analyzeCommandMiss(config: CommandLookup, id: string, typedWords?: readonly string[]): CommandMiss;
 /** Adapt an oclif `Config` to the lookup shape above. */
 export declare function lookupFromConfig(config: {
     bin: string;