@nifrajs/cli 2.13.0 → 2.14.1

package/src/mcp.ts

@@ -41,10 +41,17 @@ import { basename, isAbsolute, relative, resolve, sep } from "node:path"
 import { fileURLToPath } from "node:url"
 import { type ReflectedRoute, reflectRoutes } from "@nifrajs/core/reflection"
 import { Glob } from "bun"
+import {
+  type CommandCatalogEntry,
+  type CommandCtx,
+  type CommandSpec,
+  commandCatalog,
+  commandMcpName,
+  findCommandSpec,
+} from "./command-catalog.ts"
 import { collectContractProof } from "./contract-proof.ts"
 import { loadDocsCorpus } from "./docs-search.ts"
 import { loadExamplesCorpus } from "./examples.ts"
-import { describeProject } from "./introspect.ts"
 import type { LoadAppOptions, LoadedApp } from "./load.ts"
 import { detectMonorepo, loadMonorepoApps } from "./load.ts"
 import { docsTools } from "./mcp-docs-tools.ts"
@@ -76,7 +83,6 @@ import {
   type ToolingDrift,
 } from "./mcp-root.ts"
 import { loadTypesCorpus } from "./types-search.ts"
-import { collectProjectWorkGraph } from "./work-graph.ts"
 
 /** Path to a sibling child entry (`mcp-run` / `mcp-render` / `mcp-ws`), resolved next to this module (`.ts` in
  * dev, `.js` once built). Each runs in a FRESH subprocess per call so the project's current code loads. */
@@ -746,13 +752,85 @@ function dirError(dir: string | undefined): string {
 }
 
 /** Build the project-scoped tools for `cwd`. */
+export interface CommandMcpToolOptions {
+  readonly cwd: string
+  readonly loadAppCached?: () => Promise<LoadedApp>
+}
+
+/** Adapt one executable catalog spec to its MCP descriptor and project-scoped handler. */
+export function toMcpTool(
+  source: CommandSpec<unknown, unknown> | CommandCatalogEntry,
+  options: CommandMcpToolOptions,
+): McpTool {
+  const spec = "run" in source ? source : findCommandSpec(source.name)
+  if (spec === undefined) throw new Error(`missing command spec for ${source.name}`)
+  const entry = "run" in source ? undefined : source
+  const { cwd } = options
+  const loadAppCached = options.loadAppCached ?? createCachedAppLoader(cwd)
+  return {
+    name: commandMcpName(entry?.name ?? spec.name),
+    description: entry?.summary ?? spec.summary,
+    inputSchema: entry?.inputSchema ?? spec.input.jsonSchema,
+    handler: async (args: Record<string, unknown>, context: McpToolContext) => {
+      const raw = { ...args }
+      const dir = raw.dir
+      if (dir !== undefined && typeof dir !== "string") return dirError(undefined)
+      const target = resolveProjectDir(cwd, dir as string | undefined)
+      if (target === null) return dirError(dir as string | undefined)
+      if (typeof raw.config === "string") {
+        const config = resolve(target, raw.config)
+        if (config !== target && !config.startsWith(`${target}${sep}`))
+          return JSON.stringify(
+            { ok: false, error: "config must stay inside the selected project directory" },
+            null,
+            2,
+          )
+      }
+      delete raw.dir
+      let input: unknown
+      try {
+        input = spec.input.parse(raw)
+      } catch (error) {
+        return JSON.stringify(
+          { ok: false, error: error instanceof Error ? error.message : String(error) },
+          null,
+          2,
+        )
+      }
+      const commandContext: CommandCtx = {
+        cwd: target,
+        signal: context.signal,
+        progress: () => context.reportProgress?.(0.5, 1),
+        ...(target === cwd ? { loadApp: loadAppCached } : {}),
+      }
+      const output = await spec.run(input, commandContext)
+      const value = spec.json?.(output, input) ?? output
+      return typeof value === "string" ? value : JSON.stringify(value, null, 2)
+    },
+  }
+}
+
+export function catalogProjectTools(
+  cwd: string,
+  loadAppCached: () => Promise<LoadedApp> = createCachedAppLoader(cwd),
+): McpTool[] {
+  return commandCatalog
+    .filter((entry) => entry.transports.includes("mcp"))
+    .map((entry) => {
+      const spec = findCommandSpec(entry.name)
+      if (spec === undefined) throw new Error(`missing command spec for ${entry.name}`)
+      return toMcpTool(spec, { cwd, loadAppCached })
+    })
+}
+
 export function projectTools(
   cwd: string,
   loadAppCached: (outDirName?: string) => Promise<LoadedApp> = createCachedAppLoader(cwd),
 ): McpTool[] {
   const warmRun = createWarmHandler("mcp-run", cwd, "run")
   const warmRender = createWarmHandler("mcp-render", cwd, "render")
-  return [
+  const catalogTools = catalogProjectTools(cwd, () => loadAppCached())
+  const legacyTools: McpTool[] = [
     {
       name: "nifra_contract_proof",
       description:
@@ -809,90 +887,6 @@ export function projectTools(
         }
       },
     },
-    {
-      name: "nifra_prove",
-      description:
-        "Build the static verification work graph for this project. Pass changed files to get only impacted routes and the cheapest proof plan. The result includes a serializable evidence bundle and a machine-checkable stop condition. Requires a fresh build and never probes a running application.",
-      inputSchema: {
-        type: "object",
-        properties: {
-          files: {
-            type: "array",
-            items: { type: "string" },
-            description: "Project-relative changed files. Omit for a graph overview.",
-          },
-          minLevel: {
-            type: "integer",
-            minimum: 0,
-            maximum: 4,
-            description: "Required proof level for the stop condition. Defaults to 1.",
-          },
-        },
-        additionalProperties: false,
-      },
-      handler: async (args) => {
-        const files = args.files
-        const minLevel = args.minLevel
-        if (
-          (files !== undefined &&
-            (!Array.isArray(files) || files.some((file) => typeof file !== "string"))) ||
-          (minLevel !== undefined &&
-            (typeof minLevel !== "number" ||
-              !Number.isInteger(minLevel) ||
-              minLevel < 0 ||
-              minLevel > 4))
-        ) {
-          return JSON.stringify(
-            {
-              ok: false,
-              error: "files must be strings and minLevel must be an integer from 0 to 4",
-            },
-            null,
-            2,
-          )
-        }
-        try {
-          return JSON.stringify(
-            await collectProjectWorkGraph(cwd, {
-              ...(files === undefined ? {} : { changedFiles: files }),
-              ...(minLevel === undefined ? {} : { minLevel }),
-            }),
-            null,
-            2,
-          )
-        } catch (error) {
-          return JSON.stringify(
-            { ok: false, error: error instanceof Error ? error.message : "work graph failed" },
-            null,
-            2,
-          )
-        }
-      },
-    },
-    {
-      name: "nifra_context",
-      description:
-        "Get this nifra project's surface. Call it once UNFILTERED for a tight INDEX: the route list (API routes as `METHOD path`, page routes as `pattern → file`) + framework conventions + a pointer - cheap even on a big app, no per-route schema dump. Then pass `path` (a route prefix like /api/orders) and/or `kind` (api|pages) to fetch the FULL contract for that slice (body/query/response TS shapes + the exact typed-client call form).",
-      inputSchema: {
-        type: "object",
-        properties: {
-          path: {
-            type: "string",
-            description: "Only routes whose path/pattern starts with this prefix.",
-          },
-          kind: {
-            type: "string",
-            enum: ["api", "pages"],
-            description: "Limit to API routes or page routes.",
-          },
-        },
-        additionalProperties: false,
-      },
-      handler: async (args) => {
-        const filter = args as { path?: string; kind?: "api" | "pages" }
-        return describeProject(await loadAppCached(), filter)
-      },
-    },
     {
       name: "nifra_explain",
       description:
@@ -1063,26 +1057,6 @@ export function projectTools(
         }
       },
     },
-    {
-      name: "nifra_routes",
-      description:
-        "List this project's API routes as STRUCTURED JSON - each `{ method, path, call, body?, query?, response? }`, where `call` is the exact typed-client call form and the shapes are compact TS-typed contracts. For programmatic use (list_routes / get_route_schema) instead of parsing the nifra_context Markdown. No args = every route; pass `path` (a path or prefix like /api/orders) to narrow to those routes.",
-      inputSchema: {
-        type: "object",
-        properties: {
-          path: {
-            type: "string",
-            description: "Only routes whose path starts with this prefix (omit for all routes).",
-          },
-        },
-        additionalProperties: false,
-      },
-      handler: async (args) => {
-        const { routesToJson } = await import("./introspect.ts")
-        const { path } = args as { path?: string }
-        return JSON.stringify(routesToJson(await loadAppCached(), path), null, 2)
-      },
-    },
     {
       name: "nifra_openapi",
       description:
@@ -1279,211 +1253,6 @@ export function projectTools(
         return `${status}\n\n${renderScaffold(path, framework)}`
       },
     },
-    {
-      name: "nifra_check",
-      description:
-        'Run the project\'s drift gate and return a structured result { ok, typecheck, diagnostics[], pipeline? }: typecheck (the frontend↔backend contract), plus lints for hand-rolled fetch() to your own API, untyped client("…") calls missing <typeof app>, and server-only imports in routes/. `pipeline` answers "which bundler does this app run on" (bun|vite, for dev AND build alike) without starting a server, and its `pipeline` diagnostics catch the hazards of having two: a plugin in the slot the other bundler reads (accepted, then never called), a dev toolchain imported by the file `nifra build` bundles into the production server, and `conditions` that cannot reach Bun\'s dev client bundle. Read it before adding a plugin or a compiler. Pass lintsOnly:true for a near-instant lint pass while iterating; run the full gate (default) to confirm the work is done - fix every diagnostic before finishing.',
-      inputSchema: {
-        type: "object",
-        properties: {
-          lintsOnly: {
-            type: "boolean",
-            description: "Skip tsc; run only the near-instant source lints (inner-loop mode).",
-          },
-          dir: {
-            type: "string",
-            description:
-              'Run the check in this subdirectory (relative to the project root), e.g. "app" or "packages/api". Use it when the MCP server\'s root is a monorepo but you want to check just one app. Default: the project root.',
-          },
-        },
-        additionalProperties: false,
-      },
-      handler: async (args, context) => {
-        const { collectCheckResult } = await import("./check.ts")
-        const opts = args as { lintsOnly?: boolean; dir?: string }
-        const target = resolveProjectDir(cwd, opts.dir)
-        if (target === null) return dirError(opts.dir)
-        const result = await collectCheckResult(target, {
-          lintsOnly: opts.lintsOnly ?? false,
-          signal: context.signal,
-          // Bound the result so a large project can't emit an MCP message big enough to break the stdio
-          // transport (`-32000: Connection closed`). If `truncated` comes back, fix the shown diagnostics
-          // and re-run. The scan already skips gitignored trees (walkSource), so this is the safety net.
-          maxDiagnostics: 100,
-        })
-        return JSON.stringify(
-          result.structuredDiagnostics === undefined
-            ? result
-            : { ...result, diagnostics: result.structuredDiagnostics },
-          null,
-          2,
-        )
-      },
-    },
-    {
-      name: "nifra_fix",
-      description:
-        "Automatically fix diagnostic lints (such as rewriting hand-rolled fetch() calls to the typed nifra client, adding generic types to client factory calls, and resolving dependency drift in package.json). Runs diagnostics, applies all mechanical edit suggestions, applies doctor dependency fixes, and returns the remaining unresolved diagnostics.",
-      inputSchema: {
-        type: "object",
-        properties: {
-          code: {
-            type: "string",
-            description:
-              "Apply the registered recipe for one stable diagnostic code when available.",
-          },
-        },
-        additionalProperties: false,
-      },
-      handler: async (args, context) => {
-        const { collectCheckResult } = await import("./check.ts")
-        const { applyDiagnosticRecipe } = await import("./fix-recipes.ts")
-        const { applyDoctorAutoFix } = await import("./doctor.ts")
-        const { resolveInsideProject } = await import("./project-path.ts")
-        const { writeFile, readFile } = await import("node:fs/promises")
-
-        let doctorResult: Awaited<ReturnType<typeof applyDoctorAutoFix>> | null = null
-        try {
-          doctorResult = await applyDoctorAutoFix(cwd)
-        } catch {
-          // ignore doctor errors
-        }
-
-        const checkResult = await collectCheckResult(cwd, {
-          lintsOnly: true,
-          signal: context.signal,
-          maxDiagnostics: 100,
-        })
-
-        const requestedCode = (args as { code?: unknown }).code
-        const recipeFixed: string[] = []
-        if (typeof requestedCode === "string") {
-          for (const diagnostic of checkResult.structuredDiagnostics ?? []) {
-            if (diagnostic.code !== requestedCode) continue
-            recipeFixed.push(...(await applyDiagnosticRecipe(cwd, diagnostic)))
-          }
-        }
-
-        const fixed: Array<{ file: string; line: number; title: string }> = []
-
-        for (const diag of checkResult.diagnostics) {
-          if (diag.file && diag.line && diag.suggestion?.kind === "edit" && diag.suggestion.diff) {
-            try {
-              const diffLines = diag.suggestion.diff.split("\n")
-              const beforeLine = diffLines.find((l) => l.startsWith("-"))?.slice(1)
-              const afterLine = diffLines.find((l) => l.startsWith("+"))?.slice(1)
-              if (beforeLine !== undefined && afterLine !== undefined) {
-                const filePath = await resolveInsideProject(cwd, diag.file)
-                if (filePath === undefined) continue
-                const content = await readFile(filePath, "utf-8")
-                const lines = content.split("\n")
-                const idx = diag.line - 1
-                if (lines[idx] === beforeLine) {
-                  lines[idx] = afterLine
-                  await writeFile(filePath, lines.join("\n"), "utf-8")
-                  fixed.push({
-                    file: diag.file,
-                    line: diag.line,
-                    title: diag.suggestion.title,
-                  })
-                }
-              }
-            } catch {
-              // ignore edit errors
-            }
-          }
-        }
-
-        const finalResult = await collectCheckResult(cwd, {
-          lintsOnly: false,
-          signal: context.signal,
-          maxDiagnostics: 100,
-        })
-
-        return JSON.stringify(
-          {
-            ok: finalResult.ok,
-            fixed,
-            recipeFixed,
-            doctorFixed: doctorResult?.fixed ?? [],
-            remainingDiagnostics: finalResult.diagnostics,
-          },
-          null,
-          2,
-        )
-      },
-    },
-    {
-      name: "nifra_assure",
-      description:
-        "Evaluate nifra.assurance.ts and return the complete route-assurance report: every reflected route's first matching policy rule, enforcement evidence, missing/forbidden evidence, and the fail-closed ok bit. Use after adding or changing routes/security middleware; fix every finding before finishing.",
-      inputSchema: {
-        type: "object",
-        properties: {
-          config: {
-            type: "string",
-            description:
-              "Config path relative to the selected project directory. Default: nifra.assurance.ts.",
-          },
-          bundle: {
-            type: "boolean",
-            description: "Return the single gate bundle with verdict and explicit skipped gates.",
-          },
-          strict: {
-            type: "boolean",
-            description: "Treat warning diagnostics as failures in the bundle.",
-          },
-          hydration: {
-            type: "boolean",
-            description: "Run the hydration assurance gate.",
-          },
-          interact: {
-            type: "boolean",
-            description: "Run declared hydration probes after the client mounts.",
-          },
-          dir: {
-            type: "string",
-            description:
-              'Evaluate this project subdirectory (relative to the MCP root), e.g. "apps/api". Default: the project root.',
-          },
-        },
-        additionalProperties: false,
-      },
-      handler: async (args) => {
-        const opts = args as {
-          config?: string
-          dir?: string
-          bundle?: boolean
-          strict?: boolean
-          hydration?: boolean
-          interact?: boolean
-        }
-        const target = resolveProjectDir(cwd, opts.dir)
-        if (target === null) return dirError(opts.dir)
-        const config = opts.config === undefined ? undefined : resolve(target, opts.config)
-        if (config !== undefined && config !== target && !config.startsWith(target + sep)) {
-          return JSON.stringify(
-            { ok: false, error: "config must stay inside the selected project directory" },
-            null,
-            2,
-          )
-        }
-        const { collectAssuranceReport, collectAssureBundle } = await import("./assure.ts")
-        if (opts.bundle === true || opts.strict === true || opts.hydration === true) {
-          return JSON.stringify(
-            await collectAssureBundle(target, {
-              ...(config === undefined ? {} : { config }),
-              ...(opts.strict === undefined ? {} : { strict: opts.strict }),
-              ...(opts.hydration === undefined ? {} : { hydration: opts.hydration }),
-              ...(opts.interact === undefined ? {} : { interact: opts.interact }),
-            }),
-            null,
-            2,
-          )
-        }
-        return JSON.stringify(await collectAssuranceReport(target, config), null, 2)
-      },
-    },
     {
       name: "nifra_hydrate",
       description: "Run the hydration assurance gate and return stable NF-H diagnostics.",
@@ -1507,109 +1276,8 @@ export function projectTools(
         )
       },
     },
-    {
-      name: "nifra_replay",
-      description: "Validate and dispatch a token-only replay metadata file.",
-      inputSchema: {
-        type: "object",
-        properties: {
-          file: { type: "string", description: "Replay file relative to the project root." },
-        },
-        required: ["file"],
-        additionalProperties: false,
-      },
-      handler: async (args) => {
-        const file = (args as { file?: unknown }).file
-        if (typeof file !== "string" || file.trim() === "")
-          return JSON.stringify({ ok: false, error: "file is required" })
-        const { runReplay } = await import("./replay.ts")
-        try {
-          return JSON.stringify(await runReplay(cwd, file), null, 2)
-        } catch (error) {
-          return JSON.stringify(
-            { ok: false, error: error instanceof Error ? error.message : String(error) },
-            null,
-            2,
-          )
-        }
-      },
-    },
-    {
-      name: "nifra_levels",
-      description:
-        "Compute this project's verification ladder and return { achieved, levels[] }, where each level carries { level, name, ok, reasons[] }. The ladder is cumulative: L0 typed contract (nifra check), L1 route assurance, L2 reviewed capability lockfile, L3 route trust manifest in sync, L4 contract invariants, so a level only holds when every level below it holds, and `achieved` is -1 when even L0 fails. This is the single gate that reports what a change actually proves: run it after finishing work, and treat any `reasons` on a level the project already claimed as a regression to fix.",
-      inputSchema: {
-        type: "object",
-        properties: {
-          config: {
-            type: "string",
-            description:
-              "Assurance config path relative to the selected project directory. Default: nifra.assurance.ts.",
-          },
-          seed: {
-            type: "number",
-            description: "Deterministic seed for the L4 invariant run. Default 1.",
-          },
-          dir: {
-            type: "string",
-            description:
-              'Evaluate this project subdirectory (relative to the MCP root), e.g. "apps/api". Default: the project root.',
-          },
-        },
-        additionalProperties: false,
-      },
-      handler: async (args) => {
-        const opts = args as { config?: string; seed?: number; dir?: string }
-        const target = resolveProjectDir(cwd, opts.dir)
-        if (target === null) return dirError(opts.dir)
-        if (opts.seed !== undefined && !Number.isSafeInteger(opts.seed)) {
-          return JSON.stringify({ ok: false, error: "seed must be a safe integer" }, null, 2)
-        }
-        const config = opts.config === undefined ? undefined : resolve(target, opts.config)
-        if (config !== undefined && config !== target && !config.startsWith(target + sep)) {
-          return JSON.stringify(
-            { ok: false, error: "config must stay inside the selected project directory" },
-            null,
-            2,
-          )
-        }
-        const { collectVerificationLevels } = await import("./levels-tool.ts")
-        return JSON.stringify(
-          await collectVerificationLevels(target, {
-            ...(opts.config === undefined ? {} : { config: opts.config }),
-            ...(opts.seed === undefined ? {} : { seed: opts.seed }),
-          }),
-          null,
-          2,
-        )
-      },
-    },
-    {
-      name: "nifra_doctor",
-      description:
-        "Check this project for packages imported in source but NOT declared in package.json - the Bun-workspace trap where an import resolves at runtime (hoisting/workspace) so tests pass and `bun install` says no changes, yet tsc fails and a fresh/standalone install can't resolve it. Also reports `pipeline`: which bundler (bun|vite) this app's dev AND build phases run on and why, plus config hazards that only exist because there are two - a plugin sitting in the slot the other bundler reads (accepted, then never called), a dev toolchain imported by the file `nifra build` bundles into the production server (builds clean, dies at startup), and `conditions` that cannot reach the client bundle Bun's dev server serves. Returns { ok, ran, findings[], pipeline?, fixed?, skippedFixes? }. Pass autoFix:true to update package.json only when the dependency version can be inferred locally from an ancestor package.json or installed package metadata; otherwise the tool returns the exact bun add command to run.",
-      inputSchema: {
-        type: "object",
-        properties: {
-          autoFix: {
-            type: "boolean",
-            description:
-              "When true, write safe package.json fixes using only locally inferred versions. Does not run install or use the network.",
-          },
-        },
-        additionalProperties: false,
-      },
-      handler: async (args) => {
-        const opts = args as { autoFix?: boolean }
-        const { applyDoctorAutoFix, collectDoctorResult } = await import("./doctor.ts")
-        return JSON.stringify(
-          opts.autoFix === true ? await applyDoctorAutoFix(cwd) : await collectDoctorResult(cwd),
-          null,
-          2,
-        )
-      },
-    },
   ]
+  return [...catalogTools, ...legacyTools]
 }
 
 type ToolBackend = {

package/src/project-facts.ts

@@ -0,0 +1,86 @@
+import type { CapabilityProjectReport } from "./capabilities-tool.ts"
+import type {
+  CheckAssuranceContext,
+  CheckConfig,
+  CheckDiagnostic,
+  ManifestDriftFinding,
+  SourceFinding,
+  StaticRouteFinding,
+  TransitiveServerImportFinding,
+} from "./check.ts"
+import type { DoctorResult } from "./doctor.ts"
+import type { PipelineReport } from "./pipeline-report.ts"
+import type { RulePack, SourceIndex } from "./rules/index.ts"
+
+export interface ProjectSourceFindings {
+  readonly fetches: readonly SourceFinding[]
+  readonly untypedClients: readonly SourceFinding[]
+  readonly removedImports: readonly SourceFinding[]
+  readonly responseRoutes: readonly SourceFinding[]
+  readonly interpolatedSql: readonly SourceFinding[]
+}
+
+export interface ProjectPackageFacts {
+  readonly doctor: DoctorResult
+  readonly manifestDrift: readonly ManifestDriftFinding[]
+}
+
+export interface ProjectPolicyFacts {
+  readonly assurance?: CheckAssuranceContext
+  readonly capability?: CapabilityProjectReport
+  readonly checkConfig: CheckConfig
+  readonly rulePacks: readonly RulePack[]
+}
+
+export interface ProjectFacts {
+  readonly source: SourceIndex
+  readonly routes: readonly StaticRouteFinding[]
+  readonly importGraph: readonly TransitiveServerImportFinding[]
+  readonly packages: ProjectPackageFacts
+  readonly pipeline?: PipelineReport
+  readonly policies: ProjectPolicyFacts
+  readonly sourceFindings: ProjectSourceFindings
+  readonly legacyDiagnostics: readonly CheckDiagnostic[]
+}
+
+export type ProjectFactsSeed = Omit<ProjectFacts, "legacyDiagnostics">
+
+function freezeFindings<T extends object>(findings: readonly T[]): readonly T[] {
+  return Object.freeze(findings.map((finding) => Object.freeze({ ...finding })))
+}
+
+/**
+ * Publish the one immutable rule snapshot. The loader owns all mutable scan buffers; rules only see
+ * these copied, frozen collections. Policy/config objects remain caller-owned because assurance config
+ * can contain executable adapters and must not be frozen as a side effect of checking.
+ */
+export function freezeProjectFacts(
+  seed: ProjectFactsSeed,
+  legacyDiagnostics: readonly CheckDiagnostic[],
+): ProjectFacts {
+  const sourceFindings: ProjectSourceFindings = Object.freeze({
+    fetches: freezeFindings(seed.sourceFindings.fetches),
+    untypedClients: freezeFindings(seed.sourceFindings.untypedClients),
+    removedImports: freezeFindings(seed.sourceFindings.removedImports),
+    responseRoutes: freezeFindings(seed.sourceFindings.responseRoutes),
+    interpolatedSql: freezeFindings(seed.sourceFindings.interpolatedSql),
+  })
+  const policies = Object.freeze({
+    ...seed.policies,
+    rulePacks: Object.freeze([...seed.policies.rulePacks]),
+  })
+  return Object.freeze({
+    ...seed,
+    source: Object.freeze(seed.source),
+    routes: freezeFindings(seed.routes),
+    importGraph: freezeFindings(seed.importGraph),
+    packages: Object.freeze({
+      doctor: seed.packages.doctor,
+      manifestDrift: freezeFindings(seed.packages.manifestDrift),
+    }),
+    ...(seed.pipeline === undefined ? {} : { pipeline: Object.freeze(seed.pipeline) }),
+    policies,
+    sourceFindings,
+    legacyDiagnostics: freezeFindings(legacyDiagnostics),
+  })
+}

package/src/rules/comment-markers.ts

@@ -24,3 +24,39 @@ export function commentBlockHasMarker(
   }
   return false
 }
+
+/**
+ * The justification a reason-carrying marker (`<marker>: <reason>`) supplies, or `undefined` when the
+ * marker is absent or written with no reason after its colon. Searches the same trailing-comment /
+ * comment-block-above region as {@link commentBlockHasMarker}, so a hatch that demands a written reason
+ * can require this to be non-empty before it silences anything - an empty `<marker>:` never suppresses,
+ * which is how "reason mandatory" is enforced without a second diagnostic.
+ */
+export function commentBlockMarkerReason(
+  lines: readonly string[],
+  line: number,
+  marker: string,
+): string | undefined {
+  const reasonOn = (text: string): string | undefined => {
+    const at = text.indexOf(marker)
+    if (at < 0) return undefined
+    const rest = text.slice(at + marker.length)
+    const colon = rest.indexOf(":")
+    if (colon < 0) return undefined
+    // A block comment's `*/` closer is not part of the reason.
+    const reason = rest
+      .slice(colon + 1)
+      .replace(/\*\/\s*$/, "")
+      .trim()
+    return reason.length > 0 ? reason : undefined
+  }
+  const onLine = reasonOn(lines[line - 1] ?? "")
+  if (onLine !== undefined) return onLine
+  for (let i = line - 2; i >= 0; i--) {
+    const text = lines[i] ?? ""
+    if (!COMMENT_LINE.test(text)) return undefined
+    const reason = reasonOn(text)
+    if (reason !== undefined) return reason
+  }
+  return undefined
+}