@ybotok/loam 0.1.0-beta.3 → 0.2.0-alpha.13

package/dist/commands/validate/report.js

@@ -0,0 +1,145 @@
+/**
+ * Everything that happens to a `TargetReport` around the checks themselves:
+ * how one is built when a target could not be read at all, how a finding's
+ * evidence is capped, and how the set is summed. How it PRINTS is
+ * `./render/text.ts`.
+ *
+ * A module rather than the tail of validate.ts because the rollup and the
+ * renderer have to agree. The `--all` footer and the `--json` summary are the
+ * same `summary()` — computed here once and HANDED to the renderer, which
+ * never counts for itself — and the cap reaches both payloads from one place;
+ * a second copy of either is how a footer starts printing a number its own
+ * findings contradict.
+ */
+import { isAbsolute } from "node:path";
+import { DocsRepoUnavailableError } from "../../core/repo/state.js";
+import { repoPath } from "../../core/envelope/json.js";
+import { countSeverity, subjectsWith, } from "../../core/vocabulary/report.js";
+export function summary(targets) {
+    return {
+        services: targets.filter((t) => t.kind === "service").length,
+        features: targets.filter((t) => t.kind === "feature").length,
+        errors: countSeverity(targets, "error"),
+        warnings: countSeverity(targets, "warn"),
+    };
+}
+/**
+ * How many services this run could not check, for the rollup line and the
+ * `--json` payload.
+ *
+ * Counted off the findings rather than alongside them, so the rollup line
+ * and the per-service findings can never disagree about how many services
+ * this run could not check. The by-SUBJECT rule (a service whose spec.md and
+ * arch.spec.md both name `sources` raises two findings and is one service) is
+ * `subjectsWith`'s, spelled once for every findings-derived rollup — the
+ * fleet scorecard counts through the same function on the same code, so its
+ * number and this one are structurally the same number. The code is a LITERAL,
+ * as it is at the emission site (checks/sources.ts), rather than a shared
+ * constant, because the stable-code collector reads counting sites too, and it
+ * refuses a slot it cannot read — the collector is also what convicts a typo
+ * here.
+ */
+export function unverifiableSubjects(targets) {
+    return subjectsWith(targets, "sources.unverifiable-from-here");
+}
+/**
+ * How many `details` lines any one finding may print before the rest are
+ * summarised away.
+ *
+ * A finding's details are evidence, not a log: LikeC4 reports one syntax error
+ * as dozens of cascading diagnostics, and a fleet-sized repo multiplies that by
+ * every target that mentions the file. The report is read by a person scrolling
+ * a CI log and by an agent with a context window, and neither of them is helped
+ * by the four-hundredth copy. The cap is applied to the JSON payload too, on
+ * purpose: `--json` is the interface an agent pipes, and an unbounded array is
+ * the same denial-of-attention there, just machine-readable.
+ */
+const DETAIL_LIMIT = 10;
+/** Truncate every finding's details, marking what was dropped so nothing looks complete when it is not. */
+export function capDetails(t) {
+    return {
+        ...t,
+        findings: t.findings.map((f) => {
+            const details = f.details ?? [];
+            if (details.length <= DETAIL_LIMIT)
+                return f;
+            return {
+                ...f,
+                details: [...details.slice(0, DETAIL_LIMIT), `… (+${details.length - DETAIL_LIMIT} more)`],
+            };
+        }),
+    };
+}
+/**
+ * One target's checks, with an IO exception turned into a finding ON that
+ * target instead of aborting the run.
+ *
+ * A fleet gate that dies on the first unreadable file reports nothing about the
+ * other ninety-nine services — one bad permission bit, one file that is a
+ * dangling symlink, and CI's answer to "how is the fleet" becomes a stack
+ * trace. The failure is real and it is an error, so the run still exits 1; what
+ * changes is that everything else is still graded, and the finding names the
+ * service and the path instead of arriving as the `internal` catch-all.
+ * `docsDir` is how it names the path: see `shownPath`.
+ */
+export async function guarded(target, docsDir, run) {
+    try {
+        return await run();
+    }
+    catch (err) {
+        // A docs repo that vanished mid-run is not one target's problem — it is the
+        // whole run's, and the action's own catch reports it.
+        if (err instanceof DocsRepoUnavailableError)
+            throw err;
+        const abs = err.path;
+        const path = abs === undefined ? undefined : shownPath(docsDir, abs);
+        // Node spells the same absolute path inside its own message
+        // (`EACCES: permission denied, open '…'`), so the reason is respelled
+        // with it — otherwise the one finding would name the file twice, two ways.
+        const said = err instanceof Error ? err.message : String(err);
+        const reason = abs === undefined || path === undefined ? said : said.replaceAll(abs, path);
+        return {
+            kind: target.kind,
+            id: target.id,
+            findings: [
+                {
+                    severity: "error",
+                    code: target.kind === "service" ? "service.unreadable" : "feature.unreadable",
+                    subject: target.id,
+                    message: `${target.id}: ${path === undefined ? "an artifact" : path} could not be read — ` +
+                        `nothing about this ${target.kind} was checked. ${reason}`,
+                },
+            ],
+        };
+    }
+}
+/**
+ * A path as a finding's message names it: repo-relative and `/`-separated when
+ * it is inside the docs repo, so the report reads the same on every machine
+ * and in every CI log — an absolute path names the checkout that ran the
+ * check, not the file in the repository. A path OUTSIDE the docs repo (a
+ * source file in the service repo loam is standing in) keeps its absolute
+ * spelling: the `../` route to it depends on where two checkouts happen to
+ * sit, which is exactly what a relative path was meant to stop mattering.
+ */
+function shownPath(docsDir, abs) {
+    const rel = repoPath(docsDir, abs);
+    return rel === "" || rel === ".." || rel.startsWith("../") || isAbsolute(rel) ? abs : rel;
+}
+/**
+ * `target.ambiguous` — the positional named a service AND a feature. The tie is
+ * still broken the way it always was (the feature wins), because changing which
+ * one is picked would silently re-target every script that relies on it; what
+ * is new is that the run says which reading it took and how to force the other.
+ */
+export function ambiguousTarget(arg, chosen) {
+    const other = chosen === "feature" ? "service" : "feature";
+    return {
+        severity: "warn",
+        code: "target.ambiguous",
+        subject: arg,
+        message: `'${arg}' names both a service and a feature — validated as the ${chosen}. ` +
+            `Pass --${other} ${arg} for the other reading.`,
+    };
+}
+//# sourceMappingURL=report.js.map

package/dist/commands/validate/report.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"report.js","sourceRoot":"","sources":["../../../src/commands/validate/report.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;GAYG;AACH,OAAO,EAAE,UAAU,EAAE,MAAM,WAAW,CAAC;AACvC,OAAO,EAAE,wBAAwB,EAAE,MAAM,0BAA0B,CAAC;AACpE,OAAO,EAAE,QAAQ,EAAE,MAAM,6BAA6B,CAAC;AACvD,OAAO,EACL,aAAa,EACb,YAAY,GAGb,MAAM,iCAAiC,CAAC;AAiBzC,MAAM,UAAU,OAAO,CAAC,OAAuB;IAC7C,OAAO;QACL,QAAQ,EAAE,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,SAAS,CAAC,CAAC,MAAM;QAC5D,QAAQ,EAAE,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,SAAS,CAAC,CAAC,MAAM;QAC5D,MAAM,EAAE,aAAa,CAAC,OAAO,EAAE,OAAO,CAAC;QACvC,QAAQ,EAAE,aAAa,CAAC,OAAO,EAAE,MAAM,CAAC;KACzC,CAAC;AACJ,CAAC;AAED;;;;;;;;;;;;;;;GAeG;AACH,MAAM,UAAU,oBAAoB,CAAC,OAAuB;IAC1D,OAAO,YAAY,CAAC,OAAO,EAAE,gCAAgC,CAAC,CAAC;AACjE,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,YAAY,GAAG,EAAE,CAAC;AAExB,2GAA2G;AAC3G,MAAM,UAAU,UAAU,CAAC,CAAe;IACxC,OAAO;QACL,GAAG,CAAC;QACJ,QAAQ,EAAE,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE;YAC7B,MAAM,OAAO,GAAG,CAAC,CAAC,OAAO,IAAI,EAAE,CAAC;YAChC,IAAI,OAAO,CAAC,MAAM,IAAI,YAAY;gBAAE,OAAO,CAAC,CAAC;YAC7C,OAAO;gBACL,GAAG,CAAC;gBACJ,OAAO,EAAE,CAAC,GAAG,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,YAAY,CAAC,EAAE,OAAO,OAAO,CAAC,MAAM,GAAG,YAAY,QAAQ,CAAC;aAC3F,CAAC;QACJ,CAAC,CAAC;KACH,CAAC;AACJ,CAAC;AAED;;;;;;;;;;;GAWG;AACH,MAAM,CAAC,KAAK,UAAU,OAAO,CAC3B,MAAmD,EACnD,OAAe,EACf,GAAgC;IAEhC,IAAI,CAAC;QACH,OAAO,MAAM,GAAG,EAAE,CAAC;IACrB,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,4EAA4E;QAC5E,sDAAsD;QACtD,IAAI,GAAG,YAAY,wBAAwB;YAAE,MAAM,GAAG,CAAC;QACvD,MAAM,GAAG,GAAI,GAA6B,CAAC,IAAI,CAAC;QAChD,MAAM,IAAI,GAAG,GAAG,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC;QACrE,4DAA4D;QAC5D,sEAAsE;QACtE,2EAA2E;QAC3E,MAAM,IAAI,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;QAC9D,MAAM,MAAM,GAAG,GAAG,KAAK,SAAS,IAAI,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,GAAG,EAAE,IAAI,CAAC,CAAC;QAC3F,OAAO;YACL,IAAI,EAAE,MAAM,CAAC,IAAI;YACjB,EAAE,EAAE,MAAM,CAAC,EAAE;YACb,QAAQ,EAAE;gBACR;oBACE,QAAQ,EAAE,OAAO;oBACjB,IAAI,EAAE,MAAM,CAAC,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,oBAAoB,CAAC,CAAC,CAAC,oBAAoB;oBAC7E,OAAO,EAAE,MAAM,CAAC,EAAE;oBAClB,OAAO,EACL,GAAG,MAAM,CAAC,EAAE,KAAK,IAAI,KAAK,SAAS,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,uBAAuB;wBACjF,sBAAsB,MAAM,CAAC,IAAI,iBAAiB,MAAM,EAAE;iBAC7D;aACF;SACF,CAAC;IACJ,CAAC;AACH,CAAC;AAED;;;;;;;;GAQG;AACH,SAAS,SAAS,CAAC,OAAe,EAAE,GAAW;IAC7C,MAAM,GAAG,GAAG,QAAQ,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC;IACnC,OAAO,GAAG,KAAK,EAAE,IAAI,GAAG,KAAK,IAAI,IAAI,GAAG,CAAC,UAAU,CAAC,KAAK,CAAC,IAAI,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,GAAG,CAAC;AAC5F,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,eAAe,CAAC,GAAW,EAAE,MAA6B;IACxE,MAAM,KAAK,GAAG,MAAM,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC;IAC3D,OAAO;QACL,QAAQ,EAAE,MAAM;QAChB,IAAI,EAAE,kBAAkB;QACxB,OAAO,EAAE,GAAG;QACZ,OAAO,EACL,IAAI,GAAG,2DAA2D,MAAM,IAAI;YAC5E,UAAU,KAAK,IAAI,GAAG,yBAAyB;KAClD,CAAC;AACJ,CAAC","sourcesContent":["/**\n * Everything that happens to a `TargetReport` around the checks themselves:\n * how one is built when a target could not be read at all, how a finding's\n * evidence is capped, and how the set is summed. How it PRINTS is\n * `./render/text.ts`.\n *\n * A module rather than the tail of validate.ts because the rollup and the\n * renderer have to agree. The `--all` footer and the `--json` summary are the\n * same `summary()` — computed here once and HANDED to the renderer, which\n * never counts for itself — and the cap reaches both payloads from one place;\n * a second copy of either is how a footer starts printing a number its own\n * findings contradict.\n */\nimport { isAbsolute } from \"node:path\";\nimport { DocsRepoUnavailableError } from \"../../core/repo/state.js\";\nimport { repoPath } from \"../../core/envelope/json.js\";\nimport {\n  countSeverity,\n  subjectsWith,\n  type Finding,\n  type TargetReport,\n} from \"../../core/vocabulary/report.js\";\n\n/**\n * The rollup `--json` carries and the `--all` footer prints.\n *\n * Four named fields rather than a `Record<string, number>`: the keys ARE the\n * contract, and under an index signature every read of them was an assertion\n * (`s.errors!`) that tsc could not check — rename one and the compiler stayed\n * quiet while the footer printed `undefined errors`.\n */\nexport interface ValidateSummary {\n  services: number;\n  features: number;\n  errors: number;\n  warnings: number;\n}\n\nexport function summary(targets: TargetReport[]): ValidateSummary {\n  return {\n    services: targets.filter((t) => t.kind === \"service\").length,\n    features: targets.filter((t) => t.kind === \"feature\").length,\n    errors: countSeverity(targets, \"error\"),\n    warnings: countSeverity(targets, \"warn\"),\n  };\n}\n\n/**\n * How many services this run could not check, for the rollup line and the\n * `--json` payload.\n *\n * Counted off the findings rather than alongside them, so the rollup line\n * and the per-service findings can never disagree about how many services\n * this run could not check. The by-SUBJECT rule (a service whose spec.md and\n * arch.spec.md both name `sources` raises two findings and is one service) is\n * `subjectsWith`'s, spelled once for every findings-derived rollup — the\n * fleet scorecard counts through the same function on the same code, so its\n * number and this one are structurally the same number. The code is a LITERAL,\n * as it is at the emission site (checks/sources.ts), rather than a shared\n * constant, because the stable-code collector reads counting sites too, and it\n * refuses a slot it cannot read — the collector is also what convicts a typo\n * here.\n */\nexport function unverifiableSubjects(targets: TargetReport[]): number {\n  return subjectsWith(targets, \"sources.unverifiable-from-here\");\n}\n\n/**\n * How many `details` lines any one finding may print before the rest are\n * summarised away.\n *\n * A finding's details are evidence, not a log: LikeC4 reports one syntax error\n * as dozens of cascading diagnostics, and a fleet-sized repo multiplies that by\n * every target that mentions the file. The report is read by a person scrolling\n * a CI log and by an agent with a context window, and neither of them is helped\n * by the four-hundredth copy. The cap is applied to the JSON payload too, on\n * purpose: `--json` is the interface an agent pipes, and an unbounded array is\n * the same denial-of-attention there, just machine-readable.\n */\nconst DETAIL_LIMIT = 10;\n\n/** Truncate every finding's details, marking what was dropped so nothing looks complete when it is not. */\nexport function capDetails(t: TargetReport): TargetReport {\n  return {\n    ...t,\n    findings: t.findings.map((f) => {\n      const details = f.details ?? [];\n      if (details.length <= DETAIL_LIMIT) return f;\n      return {\n        ...f,\n        details: [...details.slice(0, DETAIL_LIMIT), `… (+${details.length - DETAIL_LIMIT} more)`],\n      };\n    }),\n  };\n}\n\n/**\n * One target's checks, with an IO exception turned into a finding ON that\n * target instead of aborting the run.\n *\n * A fleet gate that dies on the first unreadable file reports nothing about the\n * other ninety-nine services — one bad permission bit, one file that is a\n * dangling symlink, and CI's answer to \"how is the fleet\" becomes a stack\n * trace. The failure is real and it is an error, so the run still exits 1; what\n * changes is that everything else is still graded, and the finding names the\n * service and the path instead of arriving as the `internal` catch-all.\n * `docsDir` is how it names the path: see `shownPath`.\n */\nexport async function guarded(\n  target: { kind: \"service\" | \"feature\"; id: string },\n  docsDir: string,\n  run: () => Promise<TargetReport>,\n): Promise<TargetReport> {\n  try {\n    return await run();\n  } catch (err) {\n    // A docs repo that vanished mid-run is not one target's problem — it is the\n    // whole run's, and the action's own catch reports it.\n    if (err instanceof DocsRepoUnavailableError) throw err;\n    const abs = (err as NodeJS.ErrnoException).path;\n    const path = abs === undefined ? undefined : shownPath(docsDir, abs);\n    // Node spells the same absolute path inside its own message\n    // (`EACCES: permission denied, open '…'`), so the reason is respelled\n    // with it — otherwise the one finding would name the file twice, two ways.\n    const said = err instanceof Error ? err.message : String(err);\n    const reason = abs === undefined || path === undefined ? said : said.replaceAll(abs, path);\n    return {\n      kind: target.kind,\n      id: target.id,\n      findings: [\n        {\n          severity: \"error\",\n          code: target.kind === \"service\" ? \"service.unreadable\" : \"feature.unreadable\",\n          subject: target.id,\n          message:\n            `${target.id}: ${path === undefined ? \"an artifact\" : path} could not be read — ` +\n            `nothing about this ${target.kind} was checked. ${reason}`,\n        },\n      ],\n    };\n  }\n}\n\n/**\n * A path as a finding's message names it: repo-relative and `/`-separated when\n * it is inside the docs repo, so the report reads the same on every machine\n * and in every CI log — an absolute path names the checkout that ran the\n * check, not the file in the repository. A path OUTSIDE the docs repo (a\n * source file in the service repo loam is standing in) keeps its absolute\n * spelling: the `../` route to it depends on where two checkouts happen to\n * sit, which is exactly what a relative path was meant to stop mattering.\n */\nfunction shownPath(docsDir: string, abs: string): string {\n  const rel = repoPath(docsDir, abs);\n  return rel === \"\" || rel === \"..\" || rel.startsWith(\"../\") || isAbsolute(rel) ? abs : rel;\n}\n\n/**\n * `target.ambiguous` — the positional named a service AND a feature. The tie is\n * still broken the way it always was (the feature wins), because changing which\n * one is picked would silently re-target every script that relies on it; what\n * is new is that the run says which reading it took and how to force the other.\n */\nexport function ambiguousTarget(arg: string, chosen: \"service\" | \"feature\"): Finding {\n  const other = chosen === \"feature\" ? \"service\" : \"feature\";\n  return {\n    severity: \"warn\",\n    code: \"target.ambiguous\",\n    subject: arg,\n    message:\n      `'${arg}' names both a service and a feature — validated as the ${chosen}. ` +\n      `Pass --${other} ${arg} for the other reading.`,\n  };\n}\n"]}

package/dist/commands/validate/scope.js

@@ -0,0 +1,113 @@
+/**
+ * What a `--base` run does to the landscape target, which is the one target
+ * whose subject is not a directory.
+ *
+ * Every other target is graded whole or not at all: a service or a feature the
+ * branch touched is graded by exactly the checks `--all` runs on it. The
+ * landscape cannot be, for two reasons that pull in opposite directions, and
+ * both are defects a docs-repo PR gate actually shipped with.
+ *
+ * IT GRADED TOO MUCH. The landscape target carries the fleet-wide
+ * vocabularies' warnings — a permission nobody requires, a capability nobody
+ * realizes, an obligation nobody applies, a glossary term nobody cites — and
+ * the AGENTS.md stamp. Every one of those grades a file other than the map, so
+ * any `architecture/` branch re-graded all of them, and under `--strict` a
+ * one-line map edit on the adopted example fleet went red on four warnings that
+ * were all there at the base. Those findings now ride a scoped run only when the
+ * file that carries them changed.
+ *
+ * IT GRADED TOO LITTLE. `spine.op-undefined` is graded on the PROVIDER's
+ * service target, so an edge the branch drew or re-spelled on the map was
+ * graded by no scoped run unless the provider happened to change too —
+ * README's own `authorisePayment` example merged green through both PR gates.
+ * Those edges are now graded here, against their providers' contracts, and
+ * ONLY those edges: pulling the provider into scope instead would grade every
+ * document it already has, which on a spine-first fleet means
+ * `service.no-model` reddening exactly the labour that path asks for.
+ */
+import { existsSync } from "node:fs";
+import { serviceResolver } from "../../core/c4/resolve/service.js";
+import { edgeKey } from "../../core/diff/scope/edges.js";
+import { touched } from "../../core/diff/scope/changed-paths.js";
+import { repoPath } from "../../core/envelope/json.js";
+import { inOrder } from "../../core/kernel/concurrency.js";
+import { serviceTreePath } from "../../core/kernel/ids/dirs.js";
+import { readOpenapi } from "../../core/openapi/doc.js";
+import { capabilityDocsDir, glossaryDir } from "../../core/repo/authored/paths.js";
+import { agentsPath, capabilitiesPath, obligationsPath, permissionsPath, servicePathsAt } from "../../core/repo/paths.js";
+import { contractOf } from "./service/api.js";
+import { spineFindings } from "./service/spine.js";
+/**
+ * The landscape target's findings that grade a file other than the map, with
+ * the file — or directory — that carries each.
+ *
+ * Only the families that are a VERDICT ON THAT FILE are listed: the vocabulary
+ * that does not read, and the declaration nothing cites. The obligation codes
+ * the MAP causes stay off this table — `obligation.unknown` (a tag naming no
+ * rule) and `obligation.uncovered` (an applied rule no requirement covers) are
+ * written by an edit to `architecture/`, and a scoped run over that edit owes
+ * them. A code on no row rides the landscape target exactly as before.
+ *
+ * What this costs, said once: the "nothing cites it" grades also move when a
+ * CITATION goes away — a branch deleting the last `Requires:` line naming a
+ * permission. A branch that touches only a service never graded the landscape
+ * at all, so that is not a new blind spot; the unscoped `validate --all` keeps
+ * reporting every one of these.
+ */
+const CARRIED = [
+    { carries: (code) => code.startsWith("permissions."), by: (d) => [permissionsPath(d)] },
+    { carries: (code) => code.startsWith("capability."), by: (d) => [capabilitiesPath(d), capabilityDocsDir(d)] },
+    { carries: (code) => code === "obligation.invalid" || code === "obligation.unapplied", by: (d) => [obligationsPath(d)] },
+    { carries: (code) => code.startsWith("glossary."), by: (d) => [glossaryDir(d)] },
+    // The stamp is a fact about which binary wrote the file, and a branch that
+    // did not edit the file did not move it — `loam doctor` is where a binary
+    // upgrade is noticed, not every map PR that happens to follow one.
+    { carries: (code) => code === "agents.stale" || code === "docs.binary-behind", by: (d) => [agentsPath(d)] },
+];
+/** The landscape target's `findings` as a `--base` run reports them. */
+export async function scopedLandscapeFindings(findings, scoped) {
+    const { docsDir, scope } = scoped;
+    const kept = findings.filter((f) => {
+        const carried = CARRIED.find((row) => row.carries(f.code));
+        return carried === undefined || carried.by(docsDir).some((abs) => touched(scope, repoPath(docsDir, abs)));
+    });
+    return [...kept, ...(await changedEdgeFindings(scoped))];
+}
+/**
+ * The spine of the edges this branch wrote, graded per provider by the
+ * service target's own `spineFindings` over just those edges — so a changed
+ * edge reads exactly as it would on the provider's target, and an unreadable
+ * or absent contract proves nothing here for the reason it proves nothing
+ * there.
+ *
+ * A provider the run already grades is skipped: its own target walks every
+ * inbound edge, these included, and grading them twice would report one broken
+ * join as two.
+ */
+async function changedEdgeFindings(scoped) {
+    const { docsDir, scope, land, services, fleet } = scoped;
+    // A map that does not parse has no edges to read, and `landscape.invalid`
+    // on this same target already says so.
+    if (land === null || land.errors.length > 0 || scope.edges.size === 0)
+        return [];
+    const svcOf = serviceResolver(land.elements, new Set(services.map((s) => s.id)));
+    const providerOf = (target) => svcOf(target);
+    const edges = land.relationships.filter((r) => scope.edges.has(edgeKey(r)));
+    const graded = new Set(scope.services.map((s) => s.id));
+    const providers = services.filter((s) => !graded.has(s.id) && edges.some((r) => providerOf(r.target) === s.id));
+    const perProvider = await inOrder(providers, async (svc) => {
+        const paths = servicePathsAt(svc.dir);
+        return spineFindings({
+            service: svc.id,
+            docsDir,
+            treePath: serviceTreePath(svc),
+            me: svc.id,
+            land: { ...land, relationships: edges.filter((r) => providerOf(r.target) === svc.id) },
+            landSvcOf: svcOf,
+            contract: contractOf(await readOpenapi(paths.openapi, fleet), !existsSync(paths.openapi)),
+            landscapeReported: true,
+        });
+    });
+    return perProvider.flat();
+}
+//# sourceMappingURL=scope.js.map

package/dist/commands/validate/scope.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"scope.js","sourceRoot":"","sources":["../../../src/commands/validate/scope.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;;;;GA0BG;AACH,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,eAAe,EAAE,MAAM,kCAAkC,CAAC;AAEnE,OAAO,EAAE,OAAO,EAAE,MAAM,gCAAgC,CAAC;AACzD,OAAO,EAAE,OAAO,EAAsB,MAAM,wCAAwC,CAAC;AACrF,OAAO,EAAE,QAAQ,EAAE,MAAM,6BAA6B,CAAC;AAEvD,OAAO,EAAE,OAAO,EAAE,MAAM,kCAAkC,CAAC;AAC3D,OAAO,EAAE,eAAe,EAAgB,MAAM,+BAA+B,CAAC;AAC9E,OAAO,EAAE,WAAW,EAAE,MAAM,2BAA2B,CAAC;AACxD,OAAO,EAAE,iBAAiB,EAAE,WAAW,EAAE,MAAM,mCAAmC,CAAC;AAEnF,OAAO,EAAE,UAAU,EAAE,gBAAgB,EAAE,eAAe,EAAE,eAAe,EAAE,cAAc,EAAE,MAAM,0BAA0B,CAAC;AAE1H,OAAO,EAAE,UAAU,EAAE,MAAM,kBAAkB,CAAC;AAC9C,OAAO,EAAE,aAAa,EAAE,MAAM,oBAAoB,CAAC;AAEnD;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,OAAO,GAA0F;IACrG,EAAE,OAAO,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,cAAc,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,EAAE;IACvF,EAAE,OAAO,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,aAAa,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,gBAAgB,CAAC,CAAC,CAAC,EAAE,iBAAiB,CAAC,CAAC,CAAC,CAAC,EAAE;IAC7G,EAAE,OAAO,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,KAAK,oBAAoB,IAAI,IAAI,KAAK,sBAAsB,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC,EAAE;IACxH,EAAE,OAAO,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,CAAC,UAAU,CAAC,WAAW,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC,EAAE;IAChF,2EAA2E;IAC3E,0EAA0E;IAC1E,mEAAmE;IACnE,EAAE,OAAO,EAAE,CAAC,IAAI,EAAE,EAAE,CAAC,IAAI,KAAK,cAAc,IAAI,IAAI,KAAK,oBAAoB,EAAE,EAAE,EAAE,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,EAAE;CAC5G,CAAC;AAaF,wEAAwE;AACxE,MAAM,CAAC,KAAK,UAAU,uBAAuB,CAAC,QAAmB,EAAE,MAAuB;IACxF,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,GAAG,MAAM,CAAC;IAClC,MAAM,IAAI,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE;QACjC,MAAM,OAAO,GAAG,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC;QAC3D,OAAO,OAAO,KAAK,SAAS,IAAI,OAAO,CAAC,EAAE,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,EAAE,CAAC,OAAO,CAAC,KAAK,EAAE,QAAQ,CAAC,OAAO,EAAE,GAAG,CAAC,CAAC,CAAC,CAAC;IAC5G,CAAC,CAAC,CAAC;IACH,OAAO,CAAC,GAAG,IAAI,EAAE,GAAG,CAAC,MAAM,mBAAmB,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;AAC3D,CAAC;AAED;;;;;;;;;;GAUG;AACH,KAAK,UAAU,mBAAmB,CAAC,MAAuB;IACxD,MAAM,EAAE,OAAO,EAAE,KAAK,EAAE,IAAI,EAAE,QAAQ,EAAE,KAAK,EAAE,GAAG,MAAM,CAAC;IACzD,0EAA0E;IAC1E,uCAAuC;IACvC,IAAI,IAAI,KAAK,IAAI,IAAI,IAAI,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC,IAAI,KAAK,CAAC,KAAK,CAAC,IAAI,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IACjF,MAAM,KAAK,GAAG,eAAe,CAAC,IAAI,CAAC,QAAQ,EAAE,IAAI,GAAG,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC;IACjF,MAAM,UAAU,GAAG,CAAC,MAAc,EAAU,EAAE,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;IAC7D,MAAM,KAAK,GAAG,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,KAAK,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IAC5E,MAAM,MAAM,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;IACxD,MAAM,SAAS,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,IAAI,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;IAChH,MAAM,WAAW,GAAG,MAAM,OAAO,CAAC,SAAS,EAAE,KAAK,EAAE,GAAG,EAAE,EAAE;QACzD,MAAM,KAAK,GAAG,cAAc,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QACtC,OAAO,aAAa,CAAC;YACnB,OAAO,EAAE,GAAG,CAAC,EAAE;YACf,OAAO;YACP,QAAQ,EAAE,eAAe,CAAC,GAAG,CAAC;YAC9B,EAAE,EAAE,GAAG,CAAC,EAAE;YACV,IAAI,EAAE,EAAE,GAAG,IAAI,EAAE,aAAa,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,UAAU,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,GAAG,CAAC,EAAE,CAAC,EAAE;YACtF,SAAS,EAAE,KAAK;YAChB,QAAQ,EAAE,UAAU,CAAC,MAAM,WAAW,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,EAAE,CAAC,UAAU,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;YACzF,iBAAiB,EAAE,IAAI;SACxB,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IACH,OAAO,WAAW,CAAC,IAAI,EAAE,CAAC;AAC5B,CAAC","sourcesContent":["/**\n * What a `--base` run does to the landscape target, which is the one target\n * whose subject is not a directory.\n *\n * Every other target is graded whole or not at all: a service or a feature the\n * branch touched is graded by exactly the checks `--all` runs on it. The\n * landscape cannot be, for two reasons that pull in opposite directions, and\n * both are defects a docs-repo PR gate actually shipped with.\n *\n * IT GRADED TOO MUCH. The landscape target carries the fleet-wide\n * vocabularies' warnings — a permission nobody requires, a capability nobody\n * realizes, an obligation nobody applies, a glossary term nobody cites — and\n * the AGENTS.md stamp. Every one of those grades a file other than the map, so\n * any `architecture/` branch re-graded all of them, and under `--strict` a\n * one-line map edit on the adopted example fleet went red on four warnings that\n * were all there at the base. Those findings now ride a scoped run only when the\n * file that carries them changed.\n *\n * IT GRADED TOO LITTLE. `spine.op-undefined` is graded on the PROVIDER's\n * service target, so an edge the branch drew or re-spelled on the map was\n * graded by no scoped run unless the provider happened to change too —\n * README's own `authorisePayment` example merged green through both PR gates.\n * Those edges are now graded here, against their providers' contracts, and\n * ONLY those edges: pulling the provider into scope instead would grade every\n * document it already has, which on a spine-first fleet means\n * `service.no-model` reddening exactly the labour that path asks for.\n */\nimport { existsSync } from \"node:fs\";\nimport { serviceResolver } from \"../../core/c4/resolve/service.js\";\nimport type { LoadedDoc } from \"../../core/c4/likec4.js\";\nimport { edgeKey } from \"../../core/diff/scope/edges.js\";\nimport { touched, type ValidateScope } from \"../../core/diff/scope/changed-paths.js\";\nimport { repoPath } from \"../../core/envelope/json.js\";\nimport type { FleetContext } from \"../../core/fleet-context.js\";\nimport { inOrder } from \"../../core/kernel/concurrency.js\";\nimport { serviceTreePath, type DocsDir } from \"../../core/kernel/ids/dirs.js\";\nimport { readOpenapi } from \"../../core/openapi/doc.js\";\nimport { capabilityDocsDir, glossaryDir } from \"../../core/repo/authored/paths.js\";\nimport type { ServiceEntry } from \"../../core/repo/entries.js\";\nimport { agentsPath, capabilitiesPath, obligationsPath, permissionsPath, servicePathsAt } from \"../../core/repo/paths.js\";\nimport type { Finding } from \"../../core/vocabulary/report.js\";\nimport { contractOf } from \"./service/api.js\";\nimport { spineFindings } from \"./service/spine.js\";\n\n/**\n * The landscape target's findings that grade a file other than the map, with\n * the file — or directory — that carries each.\n *\n * Only the families that are a VERDICT ON THAT FILE are listed: the vocabulary\n * that does not read, and the declaration nothing cites. The obligation codes\n * the MAP causes stay off this table — `obligation.unknown` (a tag naming no\n * rule) and `obligation.uncovered` (an applied rule no requirement covers) are\n * written by an edit to `architecture/`, and a scoped run over that edit owes\n * them. A code on no row rides the landscape target exactly as before.\n *\n * What this costs, said once: the \"nothing cites it\" grades also move when a\n * CITATION goes away — a branch deleting the last `Requires:` line naming a\n * permission. A branch that touches only a service never graded the landscape\n * at all, so that is not a new blind spot; the unscoped `validate --all` keeps\n * reporting every one of these.\n */\nconst CARRIED: readonly { carries: (code: string) => boolean; by: (docsDir: DocsDir) => string[] }[] = [\n  { carries: (code) => code.startsWith(\"permissions.\"), by: (d) => [permissionsPath(d)] },\n  { carries: (code) => code.startsWith(\"capability.\"), by: (d) => [capabilitiesPath(d), capabilityDocsDir(d)] },\n  { carries: (code) => code === \"obligation.invalid\" || code === \"obligation.unapplied\", by: (d) => [obligationsPath(d)] },\n  { carries: (code) => code.startsWith(\"glossary.\"), by: (d) => [glossaryDir(d)] },\n  // The stamp is a fact about which binary wrote the file, and a branch that\n  // did not edit the file did not move it — `loam doctor` is where a binary\n  // upgrade is noticed, not every map PR that happens to follow one.\n  { carries: (code) => code === \"agents.stale\" || code === \"docs.binary-behind\", by: (d) => [agentsPath(d)] },\n];\n\n/** What the scoped landscape grade needs from the run. */\nexport interface ScopedLandscape {\n  docsDir: DocsDir;\n  scope: ValidateScope;\n  /** The parsed `architecture/` project, or null when the repo draws no fleet. */\n  land: LoadedDoc | null;\n  /** The run's FULL service enumeration: a changed edge's provider is found here whether or not it is in scope. */\n  services: readonly ServiceEntry[];\n  fleet: FleetContext;\n}\n\n/** The landscape target's `findings` as a `--base` run reports them. */\nexport async function scopedLandscapeFindings(findings: Finding[], scoped: ScopedLandscape): Promise<Finding[]> {\n  const { docsDir, scope } = scoped;\n  const kept = findings.filter((f) => {\n    const carried = CARRIED.find((row) => row.carries(f.code));\n    return carried === undefined || carried.by(docsDir).some((abs) => touched(scope, repoPath(docsDir, abs)));\n  });\n  return [...kept, ...(await changedEdgeFindings(scoped))];\n}\n\n/**\n * The spine of the edges this branch wrote, graded per provider by the\n * service target's own `spineFindings` over just those edges — so a changed\n * edge reads exactly as it would on the provider's target, and an unreadable\n * or absent contract proves nothing here for the reason it proves nothing\n * there.\n *\n * A provider the run already grades is skipped: its own target walks every\n * inbound edge, these included, and grading them twice would report one broken\n * join as two.\n */\nasync function changedEdgeFindings(scoped: ScopedLandscape): Promise<Finding[]> {\n  const { docsDir, scope, land, services, fleet } = scoped;\n  // A map that does not parse has no edges to read, and `landscape.invalid`\n  // on this same target already says so.\n  if (land === null || land.errors.length > 0 || scope.edges.size === 0) return [];\n  const svcOf = serviceResolver(land.elements, new Set(services.map((s) => s.id)));\n  const providerOf = (target: string): string => svcOf(target);\n  const edges = land.relationships.filter((r) => scope.edges.has(edgeKey(r)));\n  const graded = new Set(scope.services.map((s) => s.id));\n  const providers = services.filter((s) => !graded.has(s.id) && edges.some((r) => providerOf(r.target) === s.id));\n  const perProvider = await inOrder(providers, async (svc) => {\n    const paths = servicePathsAt(svc.dir);\n    return spineFindings({\n      service: svc.id,\n      docsDir,\n      treePath: serviceTreePath(svc),\n      me: svc.id,\n      land: { ...land, relationships: edges.filter((r) => providerOf(r.target) === svc.id) },\n      landSvcOf: svcOf,\n      contract: contractOf(await readOpenapi(paths.openapi, fleet), !existsSync(paths.openapi)),\n      landscapeReported: true,\n    });\n  });\n  return perProvider.flat();\n}\n"]}

package/dist/commands/validate/service/api.js

@@ -0,0 +1,252 @@
+/**
+ * The HTTP contract axis: openapi.yaml graded against the living spec.
+ *
+ * What it decides for everything downstream is whether there is a contract to
+ * read at all. A missing or unparseable openapi.yaml is ONE finding about ONE
+ * file, and that fact has to travel: grading the fleet map's inbound edges
+ * against the empty operation set it leaves behind turns one broken YAML into
+ * one `spine.op-undefined` per consumer, each of them accusing the landscape of
+ * a fault that is not there. `ServiceContract` is how it travels; `./spine.js`
+ * is what reads it.
+ */
+import { existsSync } from "node:fs";
+import { closeIds } from "../../../core/c4/arch.js";
+import {} from "../../../core/c4/likec4.js";
+import {} from "../../../core/kernel/ids/service.js";
+import {} from "../../../core/repo/paths.js";
+import {} from "../../../core/vocabulary/report.js";
+import { responseGovernanceFindings } from "../checks/requirements.js";
+import {} from "../../../core/document/spec.js";
+import { readOpenapi } from "../../../core/openapi/doc.js";
+import { FleetContext } from "../../../core/fleet-context.js";
+import { repoPath } from "../../../core/envelope/json.js";
+import {} from "../../../core/kernel/ids/dirs.js";
+export async function apiAxisFindings(axis) {
+    const { docsDir, service, me, paths, reqs, livingReqs, land, landSvcOf, fleet } = axis;
+    const findings = [];
+    // API coverage: every operation in openapi.yaml is governed by a requirement.
+    const api = await readOpenapi(paths.openapi, fleet);
+    const removeMarkers = api.ops.filter((o) => o.remove);
+    const liveOps = api.ops.filter((o) => !o.remove);
+    /** No contract to read: deleted, renamed, or never written. */
+    const contractMissing = !existsSync(paths.openapi);
+    const contract = contractOf(api, contractMissing);
+    const { ops, deprecated: deprecatedOps } = contract;
+    // Inbound calls the landscape can PROVE: op-linked edges whose target
+    // resolves to this service. Null when the landscape proves nothing at all
+    // (absent, or it did not parse) — which is not the same fact as "nobody
+    // calls me", and the two must not be graded alike.
+    const inboundOps = land === null || land.errors.length > 0
+        ? null
+        : land.relationships.filter((r) => r.op !== undefined && landSvcOf(r.target) === me);
+    // The other half of the evidence a contract is owed: the living spec's own
+    // `Operations:` lines. A requirement on its way out governs nothing.
+    const governedOps = livingReqs.flatMap((r) => r.operations);
+    if (contractMissing) {
+        // A contract that is gone is not a reason to stop grading the API axis —
+        // it IS the axis's answer, and every check below used to live inside the
+        // file-exists branch, so deleting or misspelling the file turned the whole
+        // axis (including the documented `spec-api.op-undefined`) green.
+        //
+        // What decides the severity is whether anything already written down is
+        // holding a join into it: a living `Operations:` line, or an op-linked
+        // landscape edge. With one of those, the absence breaks a link somebody
+        // authored — an error. With neither, this is a service that legitimately
+        // has no HTTP surface (a worker nobody calls), and a fleet mid-rollout
+        // must not go red for it — a warning, or, when the landscape PROVES nobody
+        // calls it, the documented silence.
+        //
+        // One finding, not one per dangling link: the fix is a single file, and
+        // the links it strands ride along as details.
+        const dangling = [...new Set([...governedOps, ...(inboundOps ?? []).map((r) => r.op)])].sort();
+        if (dangling.length > 0) {
+            findings.push({
+                severity: "error",
+                code: "service.no-openapi",
+                subject: service,
+                message: `No OpenAPI contract at ${repoPath(docsDir, paths.openapi)}, and ${dangling.length} operation link(s) already point into it — ` +
+                    `every requirement and landscape edge naming one of them resolves to nothing until the file is back`,
+                details: dangling,
+            });
+        }
+        else if (inboundOps === null) {
+            findings.push({
+                severity: "warn",
+                code: "service.no-openapi",
+                message: `No OpenAPI contract at ${repoPath(docsDir, paths.openapi)} — API coverage and the landscape spine are unchecked`,
+            });
+        }
+    }
+    else if (api.unreadable) {
+        // A contract that EXISTS but does not read is a broken source of truth, not
+        // an empty one: swallowing it into zero operations used to grade every
+        // inbound landscape edge `spine.op-undefined` — a false diagnosis pointing
+        // at the landscape when the truth was this file. So the file is the error,
+        // and every check that reads the contract (api.*, the spine's op
+        // resolution) is suspended below, the landscape.invalid discipline.
+        findings.push({
+            severity: "error",
+            code: "openapi.invalid",
+            message: `${service}: openapi.yaml does not parse — API coverage and the landscape spine are unchecked`,
+            ...(api.error === undefined ? {} : { details: [api.error] }),
+        });
+    }
+    else {
+        if (removeMarkers.length > 0) {
+            findings.push({
+                severity: "error",
+                code: "openapi.remove-marker-living",
+                message: `${service}: living openapi.yaml contains ${removeMarkers.length} x-loam-remove marker(s) (${removeMarkers.map((op) => op.id).join(", ")}) — removal markers are valid only in feature deltas`,
+            });
+        }
+        // The same marker written at PATH level, beside the methods. `readOpenapi`
+        // is keyed by (path, method), which is precisely why the check above cannot
+        // see this one and precisely how one reached a living contract in the first
+        // place: it addresses no operation, so it retired nothing, and it stayed
+        // invisible afterwards. Error, like its method-level sibling — a
+        // feature-only key published to every consumer of the fleet, and one that
+        // keeps the empty-path cleanup from ever firing. Same code as the archive
+        // plan's: one breach, one name, wherever it is found.
+        for (const removed of api.pathLevelRemovals) {
+            findings.push({
+                severity: "error",
+                code: "openapi.remove-marker-path-level",
+                subject: service,
+                message: `${service}: living openapi.yaml carries x-loam-remove at PATH level on '${removed}' — ` +
+                    `a removal marker names ONE operation, so beside the methods it retires nothing and is not a contract key either, ` +
+                    `and no id-keyed check can see it. Delete it from the living contract; retire an operation through a feature delta whose marker sits inside the operation.`,
+            });
+        }
+        // Two slots claiming one operationId in a LIVING contract. `readOpenapi`
+        // has computed this since the merge needed it, and only the feature path
+        // (coherence) ever read it — so the ambiguity was reachable only when some
+        // unrelated feature happened to carry a delta for this service, and the
+        // fleet gate that actually runs in CI was blind to it. Same code and same
+        // sentence as the feature-scope check: one breach, one name.
+        for (const id of api.duplicateIds) {
+            const slots = api.ops.filter((op) => op.id === id).map((op) => `${op.method} ${op.path}`);
+            findings.push({
+                severity: "warn",
+                code: "openapi.duplicate-operationid",
+                subject: service,
+                message: `${service}: the living OpenAPI defines operationId '${id}' at ${slots.join(" and ")} — every join on the id (a requirement's Operations: line, an edge's metadata { op }, a removal marker) picks one of those slots arbitrarily`,
+            });
+        }
+        // The contract-depth probes. Form validated and depth did not: an
+        // operation whose one response is `description: OK`, and a `$ref` to a
+        // schema nobody wrote, both read as cleanly as a complete contract while
+        // a reader still cannot rebuild its payloads. Presence
+        // probes over the document's own keys; nothing reads what a schema says.
+        const undescribed = liveOps.filter((op) => op.undescribed === true).map((op) => op.id);
+        if (undescribed.length > 0) {
+            findings.push({
+                severity: "warn",
+                code: "openapi.response-undescribed",
+                subject: service,
+                message: `${service}: ${undescribed.length} operation(s) declare no response schema (${undescribed.join(", ")}) — ` +
+                    `the contract names the operation and says nothing about what it returns, so a reader still ` +
+                    `needs the code; declare content with a schema on at least one response (204/304 excepted)`,
+            });
+        }
+        if (api.danglingRefs.length > 0) {
+            findings.push({
+                severity: "warn",
+                code: "openapi.ref-unresolved",
+                subject: service,
+                message: `${service}: openapi.yaml contains ${api.danglingRefs.length} internal $ref(s) that resolve to ` +
+                    `nothing in the document — the operation validates while its schema is silently absent ` +
+                    `(the archive plan refuses the same breach in a feature delta, as an error)`,
+                details: api.danglingRefs,
+            });
+        }
+        const defined = new Set(ops);
+        // `Operations:` on a LIVING requirement, resolved against this service's own
+        // contract. Nothing did this before: the same spine is checked inside a
+        // feature delta (coherence's spec-api.op-undefined) and then never again, so
+        // a typo that shipped, or an operation later renamed out of openapi.yaml,
+        // left a living requirement governing an operation that does not exist —
+        // green forever, and every downstream join through that id silently empty.
+        // Same code and severity as the feature-scope check on purpose: one breach,
+        // one name, wherever it is found.
+        for (const r of livingReqs) {
+            for (const op of r.operations) {
+                if (defined.has(op))
+                    continue;
+                const close = closeIds(op, ops);
+                findings.push({
+                    severity: "error",
+                    code: "spec-api.op-undefined",
+                    subject: service,
+                    message: `${service}: requirement '${r.name}' governs '${op}', not defined in ${service}'s OpenAPI` +
+                        (close.length > 0 ? `. Did you mean: ${close.join(", ")}?` : ""),
+                });
+            }
+        }
+        if (ops.length > 0) {
+            const governed = new Set(livingReqs.flatMap((r) => r.operations));
+            const orphans = ops.filter((op) => !governed.has(op));
+            if (orphans.length === 0) {
+                findings.push({
+                    severity: "ok",
+                    code: "api.covered",
+                    // Both counts on purpose: the ratio is the one signal that lets a
+                    // reviewer smell a thin baseline from the summary — forty behaviours
+                    // filed under one requirement passes every check, and "covered" alone
+                    // read as done.
+                    message: `${service}: API covered (${ops.length} operation(s) governed by requirements · ${livingReqs.length} living requirement(s))`,
+                });
+            }
+            else {
+                findings.push({
+                    severity: "warn",
+                    code: "api.ungoverned",
+                    message: `${service}: ${orphans.length} operation(s) not governed by any requirement — ${orphans.join(", ")}`,
+                });
+            }
+            findings.push(...responseGovernanceFindings(service, liveOps, livingReqs));
+            // The migration-debt case: requirements exist, the API exists, and no
+            // `Operations:` line ties them — every cross-axis check above and in
+            // feature mode is vacuously green. Once per service, not per operation;
+            // with zero requirements the spec (or its absence) is the finding instead.
+            if (reqs.length > 0 && reqs.every((r) => r.operations.length === 0)) {
+                findings.push({
+                    severity: "warn",
+                    code: "api.ops-unlinked",
+                    message: `${service}: openapi.yaml defines ${ops.length} operation(s) but no requirement links any — the API axis is unchecked for this service`,
+                });
+            }
+            // Lifecycle: a requirement whose `Operations:` list resolves ONLY to
+            // deprecated operations governs behaviour the contract is retiring.
+            // Deprecation is the documented first step of removing an op; the explicit
+            // feature marker is the final step. Until that delta archives, the op stays
+            // live, so the fix is migration or a coordinated retirement. Ops the contract does not define at all
+            // prove nothing here and are left to spec-api.op-undefined above.
+            for (const r of reqs) {
+                const resolved = r.operations.filter((op) => defined.has(op));
+                if (resolved.length === 0 || !resolved.every((op) => deprecatedOps.has(op)))
+                    continue;
+                findings.push({
+                    severity: "warn",
+                    code: "api.requirement-deprecated",
+                    message: `${service}: requirement '${r.name}' governs only deprecated operation(s) (${resolved.join(", ")}) — the behaviour it describes is on its way out; migrate it to the replacement operation, or retire it`,
+                });
+            }
+        }
+    }
+    return { findings, contract };
+}
+/**
+ * What the spine may conclude from one contract read — spelled once, because a
+ * `--base` run now grades a changed edge against a provider it does not grade
+ * as a target (`../scope.ts`), and two spellings of "which operations are
+ * live" would be two chances for a removal marker to count as defined.
+ */
+export function contractOf(api, missing) {
+    const live = api.ops.filter((o) => !o.remove);
+    return {
+        ops: live.map((o) => o.id),
+        deprecated: new Set(live.filter((o) => o.deprecated).map((o) => o.id)),
+        unreadable: api.unreadable || missing,
+    };
+}
+//# sourceMappingURL=api.js.map

package/dist/commands/validate/service/api.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"api.js","sourceRoot":"","sources":["../../../../src/commands/validate/service/api.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AACH,OAAO,EAAE,UAAU,EAAE,MAAM,SAAS,CAAC;AACrC,OAAO,EAAE,QAAQ,EAAE,MAAM,0BAA0B,CAAC;AACpD,OAAO,EAAkB,MAAM,4BAA4B,CAAC;AAC5D,OAAO,EAA8C,MAAM,qCAAqC,CAAC;AACjG,OAAO,EAAqB,MAAM,6BAA6B,CAAC;AAChE,OAAO,EAAgB,MAAM,oCAAoC,CAAC;AAClE,OAAO,EAAE,0BAA0B,EAAE,MAAM,2BAA2B,CAAC;AACvE,OAAO,EAAoB,MAAM,gCAAgC,CAAC;AAClE,OAAO,EAAE,WAAW,EAAmB,MAAM,8BAA8B,CAAC;AAC5E,OAAO,EAAE,YAAY,EAAE,MAAM,gCAAgC,CAAC;AAC9D,OAAO,EAAE,QAAQ,EAAE,MAAM,gCAAgC,CAAC;AAC1D,OAAO,EAAgB,MAAM,kCAAkC,CAAC;AAoChE,MAAM,CAAC,KAAK,UAAU,eAAe,CACnC,IAAa;IAEb,MAAM,EAAE,OAAO,EAAE,OAAO,EAAE,EAAE,EAAE,KAAK,EAAE,IAAI,EAAE,UAAU,EAAE,IAAI,EAAE,SAAS,EAAE,KAAK,EAAE,GAAG,IAAI,CAAC;IACvF,MAAM,QAAQ,GAAc,EAAE,CAAC;IAE/B,8EAA8E;IAC9E,MAAM,GAAG,GAAG,MAAM,WAAW,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC;IACpD,MAAM,aAAa,GAAG,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IACtD,MAAM,OAAO,GAAG,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IACjD,+DAA+D;IAC/D,MAAM,eAAe,GAAG,CAAC,UAAU,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC;IACnD,MAAM,QAAQ,GAAG,UAAU,CAAC,GAAG,EAAE,eAAe,CAAC,CAAC;IAClD,MAAM,EAAE,GAAG,EAAE,UAAU,EAAE,aAAa,EAAE,GAAG,QAAQ,CAAC;IACpD,sEAAsE;IACtE,0EAA0E;IAC1E,wEAAwE;IACxE,mDAAmD;IACnD,MAAM,UAAU,GACd,IAAI,KAAK,IAAI,IAAI,IAAI,CAAC,MAAM,CAAC,MAAM,GAAG,CAAC;QACrC,CAAC,CAAC,IAAI;QACN,CAAC,CAAC,IAAI,CAAC,aAAa,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,KAAK,SAAS,IAAI,SAAU,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,EAAE,CAAC,CAAC;IAC1F,2EAA2E;IAC3E,qEAAqE;IACrE,MAAM,WAAW,GAAG,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC;IAC5D,IAAI,eAAe,EAAE,CAAC;QACpB,yEAAyE;QACzE,yEAAyE;QACzE,2EAA2E;QAC3E,iEAAiE;QACjE,EAAE;QACF,wEAAwE;QACxE,uEAAuE;QACvE,wEAAwE;QACxE,yEAAyE;QACzE,uEAAuE;QACvE,2EAA2E;QAC3E,oCAAoC;QACpC,EAAE;QACF,wEAAwE;QACxE,8CAA8C;QAC9C,MAAM,QAAQ,GAAG,CAAC,GAAG,IAAI,GAAG,CAAC,CAAC,GAAG,WAAW,EAAE,GAAG,CAAC,UAAU,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAG,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;QAChG,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACxB,QAAQ,CAAC,IAAI,CAAC;gBACZ,QAAQ,EAAE,OAAO;gBACjB,IAAI,EAAE,oBAAoB;gBAC1B,OAAO,EAAE,OAAO;gBAChB,OAAO,EACL,0BAA0B,QAAQ,CAAC,OAAO,EAAE,KAAK,CAAC,OAAO,CAAC,SAAS,QAAQ,CAAC,MAAM,6CAA6C;oBAC/H,oGAAoG;gBACtG,OAAO,EAAE,QAAQ;aAClB,CAAC,CAAC;QACL,CAAC;aAAM,IAAI,UAAU,KAAK,IAAI,EAAE,CAAC;YAC/B,QAAQ,CAAC,IAAI,CAAC;gBACZ,QAAQ,EAAE,MAAM;gBAChB,IAAI,EAAE,oBAAoB;gBAC1B,OAAO,EAAE,0BAA0B,QAAQ,CAAC,OAAO,EAAE,KAAK,CAAC,OAAO,CAAC,uDAAuD;aAC3H,CAAC,CAAC;QACL,CAAC;IACH,CAAC;SAAM,IAAI,GAAG,CAAC,UAAU,EAAE,CAAC;QAC1B,4EAA4E;QAC5E,uEAAuE;QACvE,2EAA2E;QAC3E,2EAA2E;QAC3E,iEAAiE;QACjE,oEAAoE;QACpE,QAAQ,CAAC,IAAI,CAAC;YACZ,QAAQ,EAAE,OAAO;YACjB,IAAI,EAAE,iBAAiB;YACvB,OAAO,EAAE,GAAG,OAAO,oFAAoF;YACvG,GAAG,CAAC,GAAG,CAAC,KAAK,KAAK,SAAS,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,EAAE,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;SAC7D,CAAC,CAAC;IACL,CAAC;SAAM,CAAC;QACN,IAAI,aAAa,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC7B,QAAQ,CAAC,IAAI,CAAC;gBACZ,QAAQ,EAAE,OAAO;gBACjB,IAAI,EAAE,8BAA8B;gBACpC,OAAO,EAAE,GAAG,OAAO,kCAAkC,aAAa,CAAC,MAAM,6BAA6B,aAAa,CAAC,GAAG,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,sDAAsD;aACxM,CAAC,CAAC;QACL,CAAC;QACD,2EAA2E;QAC3E,4EAA4E;QAC5E,4EAA4E;QAC5E,yEAAyE;QACzE,iEAAiE;QACjE,0EAA0E;QAC1E,0EAA0E;QAC1E,sDAAsD;QACtD,KAAK,MAAM,OAAO,IAAI,GAAG,CAAC,iBAAiB,EAAE,CAAC;YAC5C,QAAQ,CAAC,IAAI,CAAC;gBACZ,QAAQ,EAAE,OAAO;gBACjB,IAAI,EAAE,kCAAkC;gBACxC,OAAO,EAAE,OAAO;gBAChB,OAAO,EACL,GAAG,OAAO,iEAAiE,OAAO,MAAM;oBACxF,mHAAmH;oBACnH,2JAA2J;aAC9J,CAAC,CAAC;QACL,CAAC;QACD,yEAAyE;QACzE,yEAAyE;QACzE,2EAA2E;QAC3E,wEAAwE;QACxE,0EAA0E;QAC1E,6DAA6D;QAC7D,KAAK,MAAM,EAAE,IAAI,GAAG,CAAC,YAAY,EAAE,CAAC;YAClC,MAAM,KAAK,GAAG,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,EAAE,KAAK,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,GAAG,EAAE,CAAC,MAAM,IAAI,EAAE,CAAC,IAAI,EAAE,CAAC,CAAC;YAC1F,QAAQ,CAAC,IAAI,CAAC;gBACZ,QAAQ,EAAE,MAAM;gBAChB,IAAI,EAAE,+BAA+B;gBACrC,OAAO,EAAE,OAAO;gBAChB,OAAO,EAAE,GAAG,OAAO,6CAA6C,EAAE,QAAQ,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,8IAA8I;aAC5O,CAAC,CAAC;QACL,CAAC;QACD,kEAAkE;QAClE,uEAAuE;QACvE,yEAAyE;QACzE,uDAAuD;QACvD,yEAAyE;QACzE,MAAM,WAAW,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,WAAW,KAAK,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,EAAE,CAAC,EAAE,CAAC,CAAC;QACvF,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC3B,QAAQ,CAAC,IAAI,CAAC;gBACZ,QAAQ,EAAE,MAAM;gBAChB,IAAI,EAAE,8BAA8B;gBACpC,OAAO,EAAE,OAAO;gBAChB,OAAO,EACL,GAAG,OAAO,KAAK,WAAW,CAAC,MAAM,6CAA6C,WAAW,CAAC,IAAI,CAAC,IAAI,CAAC,MAAM;oBAC1G,6FAA6F;oBAC7F,2FAA2F;aAC9F,CAAC,CAAC;QACL,CAAC;QACD,IAAI,GAAG,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAChC,QAAQ,CAAC,IAAI,CAAC;gBACZ,QAAQ,EAAE,MAAM;gBAChB,IAAI,EAAE,wBAAwB;gBAC9B,OAAO,EAAE,OAAO;gBAChB,OAAO,EACL,GAAG,OAAO,2BAA2B,GAAG,CAAC,YAAY,CAAC,MAAM,oCAAoC;oBAChG,wFAAwF;oBACxF,4EAA4E;gBAC9E,OAAO,EAAE,GAAG,CAAC,YAAY;aAC1B,CAAC,CAAC;QACL,CAAC;QACD,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,GAAG,CAAC,CAAC;QAC7B,6EAA6E;QAC7E,wEAAwE;QACxE,6EAA6E;QAC7E,0EAA0E;QAC1E,yEAAyE;QACzE,2EAA2E;QAC3E,4EAA4E;QAC5E,kCAAkC;QAClC,KAAK,MAAM,CAAC,IAAI,UAAU,EAAE,CAAC;YAC3B,KAAK,MAAM,EAAE,IAAI,CAAC,CAAC,UAAU,EAAE,CAAC;gBAC9B,IAAI,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC;oBAAE,SAAS;gBAC9B,MAAM,KAAK,GAAG,QAAQ,CAAC,EAAE,EAAE,GAAG,CAAC,CAAC;gBAChC,QAAQ,CAAC,IAAI,CAAC;oBACZ,QAAQ,EAAE,OAAO;oBACjB,IAAI,EAAE,uBAAuB;oBAC7B,OAAO,EAAE,OAAO;oBAChB,OAAO,EACL,GAAG,OAAO,kBAAkB,CAAC,CAAC,IAAI,cAAc,EAAE,qBAAqB,OAAO,YAAY;wBAC1F,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,mBAAmB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC;iBACnE,CAAC,CAAC;YACL,CAAC;QACH,CAAC;QACD,IAAI,GAAG,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACrB,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC;YAClE,MAAM,OAAO,GAAG,GAAG,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;YACtD,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;gBACzB,QAAQ,CAAC,IAAI,CAAC;oBACZ,QAAQ,EAAE,IAAI;oBACd,IAAI,EAAE,aAAa;oBACnB,kEAAkE;oBAClE,qEAAqE;oBACrE,sEAAsE;oBACtE,gBAAgB;oBAChB,OAAO,EAAE,GAAG,OAAO,kBAAkB,GAAG,CAAC,MAAM,4CAA4C,UAAU,CAAC,MAAM,yBAAyB;iBACtI,CAAC,CAAC;YACL,CAAC;iBAAM,CAAC;gBACN,QAAQ,CAAC,IAAI,CAAC;oBACZ,QAAQ,EAAE,MAAM;oBAChB,IAAI,EAAE,gBAAgB;oBACtB,OAAO,EAAE,GAAG,OAAO,KAAK,OAAO,CAAC,MAAM,mDAAmD,OAAO,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE;iBAC9G,CAAC,CAAC;YACL,CAAC;YACD,QAAQ,CAAC,IAAI,CAAC,GAAG,0BAA0B,CAAC,OAAO,EAAE,OAAO,EAAE,UAAU,CAAC,CAAC,CAAC;YAC3E,sEAAsE;YACtE,qEAAqE;YACrE,wEAAwE;YACxE,2EAA2E;YAC3E,IAAI,IAAI,CAAC,MAAM,GAAG,CAAC,IAAI,IAAI,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,MAAM,KAAK,CAAC,CAAC,EAAE,CAAC;gBACpE,QAAQ,CAAC,IAAI,CAAC;oBACZ,QAAQ,EAAE,MAAM;oBAChB,IAAI,EAAE,kBAAkB;oBACxB,OAAO,EAAE,GAAG,OAAO,0BAA0B,GAAG,CAAC,MAAM,yFAAyF;iBACjJ,CAAC,CAAC;YACL,CAAC;YACD,qEAAqE;YACrE,oEAAoE;YACpE,2EAA2E;YAC3E,4EAA4E;YAC5E,qGAAqG;YACrG,kEAAkE;YAClE,KAAK,MAAM,CAAC,IAAI,IAAI,EAAE,CAAC;gBACrB,MAAM,QAAQ,GAAG,CAAC,CAAC,UAAU,CAAC,MAAM,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC,CAAC;gBAC9D,IAAI,QAAQ,CAAC,MAAM,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,EAAE,CAAC,aAAa,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;oBAAE,SAAS;gBACtF,QAAQ,CAAC,IAAI,CAAC;oBACZ,QAAQ,EAAE,MAAM;oBAChB,IAAI,EAAE,4BAA4B;oBAClC,OAAO,EAAE,GAAG,OAAO,kBAAkB,CAAC,CAAC,IAAI,2CAA2C,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,yGAAyG;iBACnN,CAAC,CAAC;YACL,CAAC;QACD,CAAC;IACH,CAAC;IAED,OAAO,EAAE,QAAQ,EAAE,QAAQ,EAAE,CAAC;AAChC,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,UAAU,CAAC,GAAe,EAAE,OAAgB;IAC1D,MAAM,IAAI,GAAG,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IAC9C,OAAO;QACL,GAAG,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QAC1B,UAAU,EAAE,IAAI,GAAG,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC;QACtE,UAAU,EAAE,GAAG,CAAC,UAAU,IAAI,OAAO;KACtC,CAAC;AACJ,CAAC","sourcesContent":["/**\n * The HTTP contract axis: openapi.yaml graded against the living spec.\n *\n * What it decides for everything downstream is whether there is a contract to\n * read at all. A missing or unparseable openapi.yaml is ONE finding about ONE\n * file, and that fact has to travel: grading the fleet map's inbound edges\n * against the empty operation set it leaves behind turns one broken YAML into\n * one `spine.op-undefined` per consumer, each of them accusing the landscape of\n * a fault that is not there. `ServiceContract` is how it travels; `./spine.js`\n * is what reads it.\n */\nimport { existsSync } from \"node:fs\";\nimport { closeIds } from \"../../../core/c4/arch.js\";\nimport { type LoadedDoc } from \"../../../core/c4/likec4.js\";\nimport { type DeclaredService, type PathableService } from \"../../../core/kernel/ids/service.js\";\nimport { type ServicePaths } from \"../../../core/repo/paths.js\";\nimport { type Finding } from \"../../../core/vocabulary/report.js\";\nimport { responseGovernanceFindings } from \"../checks/requirements.js\";\nimport { type Requirement } from \"../../../core/document/spec.js\";\nimport { readOpenapi, type OpenapiDoc } from \"../../../core/openapi/doc.js\";\nimport { FleetContext } from \"../../../core/fleet-context.js\";\nimport { repoPath } from \"../../../core/envelope/json.js\";\nimport { type DocsDir } from \"../../../core/kernel/ids/dirs.js\";\n\n/** What the service target hands the HTTP axis. */\nexport interface ApiAxis {\n  /** The docs root, so a message names the contract by its repo-relative path. */\n  docsDir: DocsDir;\n  service: PathableService;\n  /** `service` widened for `===` against the resolver's DOCUMENT text — see service.ts. */\n  me: string;\n  paths: ServicePaths;\n  /** Every requirement the living spec declares, REMOVED ones included. */\n  reqs: Requirement[];\n  /** The ones that still govern something. */\n  livingReqs: Requirement[];\n  /** The living landscape, or null when the repo has none. */\n  land: LoadedDoc | null;\n  /** Its element→service resolver, built once per run; null exactly when `land` is. */\n  landSvcOf: ((id: string) => DeclaredService) | null;\n  fleet: FleetContext;\n}\n\n/**\n * What the spine may conclude from this service's contract.\n *\n * `unreadable` covers both shapes of \"there is nothing to read\" — absent and\n * unparseable — because they have the same consequence for an inbound edge:\n * the contract proves neither that the operation exists nor that it does not.\n */\nexport interface ServiceContract {\n  /** operationIds the living contract defines, empty when there is nothing to read. */\n  ops: string[];\n  /** Those of them the contract marks `deprecated: true`. */\n  deprecated: ReadonlySet<string>;\n  unreadable: boolean;\n}\n\nexport async function apiAxisFindings(\n  axis: ApiAxis,\n): Promise<{ findings: Finding[]; contract: ServiceContract }> {\n  const { docsDir, service, me, paths, reqs, livingReqs, land, landSvcOf, fleet } = axis;\n  const findings: Finding[] = [];\n\n  // API coverage: every operation in openapi.yaml is governed by a requirement.\n  const api = await readOpenapi(paths.openapi, fleet);\n  const removeMarkers = api.ops.filter((o) => o.remove);\n  const liveOps = api.ops.filter((o) => !o.remove);\n  /** No contract to read: deleted, renamed, or never written. */\n  const contractMissing = !existsSync(paths.openapi);\n  const contract = contractOf(api, contractMissing);\n  const { ops, deprecated: deprecatedOps } = contract;\n  // Inbound calls the landscape can PROVE: op-linked edges whose target\n  // resolves to this service. Null when the landscape proves nothing at all\n  // (absent, or it did not parse) — which is not the same fact as \"nobody\n  // calls me\", and the two must not be graded alike.\n  const inboundOps =\n    land === null || land.errors.length > 0\n      ? null\n      : land.relationships.filter((r) => r.op !== undefined && landSvcOf!(r.target) === me);\n  // The other half of the evidence a contract is owed: the living spec's own\n  // `Operations:` lines. A requirement on its way out governs nothing.\n  const governedOps = livingReqs.flatMap((r) => r.operations);\n  if (contractMissing) {\n    // A contract that is gone is not a reason to stop grading the API axis —\n    // it IS the axis's answer, and every check below used to live inside the\n    // file-exists branch, so deleting or misspelling the file turned the whole\n    // axis (including the documented `spec-api.op-undefined`) green.\n    //\n    // What decides the severity is whether anything already written down is\n    // holding a join into it: a living `Operations:` line, or an op-linked\n    // landscape edge. With one of those, the absence breaks a link somebody\n    // authored — an error. With neither, this is a service that legitimately\n    // has no HTTP surface (a worker nobody calls), and a fleet mid-rollout\n    // must not go red for it — a warning, or, when the landscape PROVES nobody\n    // calls it, the documented silence.\n    //\n    // One finding, not one per dangling link: the fix is a single file, and\n    // the links it strands ride along as details.\n    const dangling = [...new Set([...governedOps, ...(inboundOps ?? []).map((r) => r.op!)])].sort();\n    if (dangling.length > 0) {\n      findings.push({\n        severity: \"error\",\n        code: \"service.no-openapi\",\n        subject: service,\n        message:\n          `No OpenAPI contract at ${repoPath(docsDir, paths.openapi)}, and ${dangling.length} operation link(s) already point into it — ` +\n          `every requirement and landscape edge naming one of them resolves to nothing until the file is back`,\n        details: dangling,\n      });\n    } else if (inboundOps === null) {\n      findings.push({\n        severity: \"warn\",\n        code: \"service.no-openapi\",\n        message: `No OpenAPI contract at ${repoPath(docsDir, paths.openapi)} — API coverage and the landscape spine are unchecked`,\n      });\n    }\n  } else if (api.unreadable) {\n    // A contract that EXISTS but does not read is a broken source of truth, not\n    // an empty one: swallowing it into zero operations used to grade every\n    // inbound landscape edge `spine.op-undefined` — a false diagnosis pointing\n    // at the landscape when the truth was this file. So the file is the error,\n    // and every check that reads the contract (api.*, the spine's op\n    // resolution) is suspended below, the landscape.invalid discipline.\n    findings.push({\n      severity: \"error\",\n      code: \"openapi.invalid\",\n      message: `${service}: openapi.yaml does not parse — API coverage and the landscape spine are unchecked`,\n      ...(api.error === undefined ? {} : { details: [api.error] }),\n    });\n  } else {\n    if (removeMarkers.length > 0) {\n      findings.push({\n        severity: \"error\",\n        code: \"openapi.remove-marker-living\",\n        message: `${service}: living openapi.yaml contains ${removeMarkers.length} x-loam-remove marker(s) (${removeMarkers.map((op) => op.id).join(\", \")}) — removal markers are valid only in feature deltas`,\n      });\n    }\n    // The same marker written at PATH level, beside the methods. `readOpenapi`\n    // is keyed by (path, method), which is precisely why the check above cannot\n    // see this one and precisely how one reached a living contract in the first\n    // place: it addresses no operation, so it retired nothing, and it stayed\n    // invisible afterwards. Error, like its method-level sibling — a\n    // feature-only key published to every consumer of the fleet, and one that\n    // keeps the empty-path cleanup from ever firing. Same code as the archive\n    // plan's: one breach, one name, wherever it is found.\n    for (const removed of api.pathLevelRemovals) {\n      findings.push({\n        severity: \"error\",\n        code: \"openapi.remove-marker-path-level\",\n        subject: service,\n        message:\n          `${service}: living openapi.yaml carries x-loam-remove at PATH level on '${removed}' — ` +\n          `a removal marker names ONE operation, so beside the methods it retires nothing and is not a contract key either, ` +\n          `and no id-keyed check can see it. Delete it from the living contract; retire an operation through a feature delta whose marker sits inside the operation.`,\n      });\n    }\n    // Two slots claiming one operationId in a LIVING contract. `readOpenapi`\n    // has computed this since the merge needed it, and only the feature path\n    // (coherence) ever read it — so the ambiguity was reachable only when some\n    // unrelated feature happened to carry a delta for this service, and the\n    // fleet gate that actually runs in CI was blind to it. Same code and same\n    // sentence as the feature-scope check: one breach, one name.\n    for (const id of api.duplicateIds) {\n      const slots = api.ops.filter((op) => op.id === id).map((op) => `${op.method} ${op.path}`);\n      findings.push({\n        severity: \"warn\",\n        code: \"openapi.duplicate-operationid\",\n        subject: service,\n        message: `${service}: the living OpenAPI defines operationId '${id}' at ${slots.join(\" and \")} — every join on the id (a requirement's Operations: line, an edge's metadata { op }, a removal marker) picks one of those slots arbitrarily`,\n      });\n    }\n    // The contract-depth probes. Form validated and depth did not: an\n    // operation whose one response is `description: OK`, and a `$ref` to a\n    // schema nobody wrote, both read as cleanly as a complete contract while\n    // a reader still cannot rebuild its payloads. Presence\n    // probes over the document's own keys; nothing reads what a schema says.\n    const undescribed = liveOps.filter((op) => op.undescribed === true).map((op) => op.id);\n    if (undescribed.length > 0) {\n      findings.push({\n        severity: \"warn\",\n        code: \"openapi.response-undescribed\",\n        subject: service,\n        message:\n          `${service}: ${undescribed.length} operation(s) declare no response schema (${undescribed.join(\", \")}) — ` +\n          `the contract names the operation and says nothing about what it returns, so a reader still ` +\n          `needs the code; declare content with a schema on at least one response (204/304 excepted)`,\n      });\n    }\n    if (api.danglingRefs.length > 0) {\n      findings.push({\n        severity: \"warn\",\n        code: \"openapi.ref-unresolved\",\n        subject: service,\n        message:\n          `${service}: openapi.yaml contains ${api.danglingRefs.length} internal $ref(s) that resolve to ` +\n          `nothing in the document — the operation validates while its schema is silently absent ` +\n          `(the archive plan refuses the same breach in a feature delta, as an error)`,\n        details: api.danglingRefs,\n      });\n    }\n    const defined = new Set(ops);\n    // `Operations:` on a LIVING requirement, resolved against this service's own\n    // contract. Nothing did this before: the same spine is checked inside a\n    // feature delta (coherence's spec-api.op-undefined) and then never again, so\n    // a typo that shipped, or an operation later renamed out of openapi.yaml,\n    // left a living requirement governing an operation that does not exist —\n    // green forever, and every downstream join through that id silently empty.\n    // Same code and severity as the feature-scope check on purpose: one breach,\n    // one name, wherever it is found.\n    for (const r of livingReqs) {\n      for (const op of r.operations) {\n        if (defined.has(op)) continue;\n        const close = closeIds(op, ops);\n        findings.push({\n          severity: \"error\",\n          code: \"spec-api.op-undefined\",\n          subject: service,\n          message:\n            `${service}: requirement '${r.name}' governs '${op}', not defined in ${service}'s OpenAPI` +\n            (close.length > 0 ? `. Did you mean: ${close.join(\", \")}?` : \"\"),\n        });\n      }\n    }\n    if (ops.length > 0) {\n    const governed = new Set(livingReqs.flatMap((r) => r.operations));\n    const orphans = ops.filter((op) => !governed.has(op));\n    if (orphans.length === 0) {\n      findings.push({\n        severity: \"ok\",\n        code: \"api.covered\",\n        // Both counts on purpose: the ratio is the one signal that lets a\n        // reviewer smell a thin baseline from the summary — forty behaviours\n        // filed under one requirement passes every check, and \"covered\" alone\n        // read as done.\n        message: `${service}: API covered (${ops.length} operation(s) governed by requirements · ${livingReqs.length} living requirement(s))`,\n      });\n    } else {\n      findings.push({\n        severity: \"warn\",\n        code: \"api.ungoverned\",\n        message: `${service}: ${orphans.length} operation(s) not governed by any requirement — ${orphans.join(\", \")}`,\n      });\n    }\n    findings.push(...responseGovernanceFindings(service, liveOps, livingReqs));\n    // The migration-debt case: requirements exist, the API exists, and no\n    // `Operations:` line ties them — every cross-axis check above and in\n    // feature mode is vacuously green. Once per service, not per operation;\n    // with zero requirements the spec (or its absence) is the finding instead.\n    if (reqs.length > 0 && reqs.every((r) => r.operations.length === 0)) {\n      findings.push({\n        severity: \"warn\",\n        code: \"api.ops-unlinked\",\n        message: `${service}: openapi.yaml defines ${ops.length} operation(s) but no requirement links any — the API axis is unchecked for this service`,\n      });\n    }\n    // Lifecycle: a requirement whose `Operations:` list resolves ONLY to\n    // deprecated operations governs behaviour the contract is retiring.\n    // Deprecation is the documented first step of removing an op; the explicit\n    // feature marker is the final step. Until that delta archives, the op stays\n    // live, so the fix is migration or a coordinated retirement. Ops the contract does not define at all\n    // prove nothing here and are left to spec-api.op-undefined above.\n    for (const r of reqs) {\n      const resolved = r.operations.filter((op) => defined.has(op));\n      if (resolved.length === 0 || !resolved.every((op) => deprecatedOps.has(op))) continue;\n      findings.push({\n        severity: \"warn\",\n        code: \"api.requirement-deprecated\",\n        message: `${service}: requirement '${r.name}' governs only deprecated operation(s) (${resolved.join(\", \")}) — the behaviour it describes is on its way out; migrate it to the replacement operation, or retire it`,\n      });\n    }\n    }\n  }\n\n  return { findings, contract };\n}\n\n/**\n * What the spine may conclude from one contract read — spelled once, because a\n * `--base` run now grades a changed edge against a provider it does not grade\n * as a target (`../scope.ts`), and two spellings of \"which operations are\n * live\" would be two chances for a removal marker to count as defined.\n */\nexport function contractOf(api: OpenapiDoc, missing: boolean): ServiceContract {\n  const live = api.ops.filter((o) => !o.remove);\n  return {\n    ops: live.map((o) => o.id),\n    deprecated: new Set(live.filter((o) => o.deprecated).map((o) => o.id)),\n    unreadable: api.unreadable || missing,\n  };\n}\n"]}