@warpgogol/forge 5.3.3 → 6.0.0

package/src/onboarding/create.ts

@@ -9,7 +9,6 @@
 </non-goals>
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
-  <item>RFC-0643: pass profileId to runInit so forge.yaml gets a `profile` field.</item>
   <item>RFC-0664: scaffold memory layer (.agents/memory/) after init.</item>
   <item>RFC-0877: in-place mode only — --in-place flag required, no subdirectory creation, name derived from folder, strict empty-directory check (only .git/ tolerated).</item>
   <item>RFC-1097: step 6 — compass.migrate codemod run
@@ -18,7 +17,8 @@ Mechanical v1 to v2 header migration across the workspace: 942 files rewritten
   <item>RFC-1097: sweep — packages/forge + services clean
 
 Sweep batch 2: real KEY_DECISIONS on 10 files, expanded purposes (CONTRACT-02/PURPOSE-02), headers on mission/index + gen-upstreams, sanitizeItemText in summary.record (literal Compass tags corrupted history), excludedPaths for wrangler types, test-fixtures testPattern. forge+services now 0 diagnostics under --mode error.</item>
-  <history>RFC-0544, RFC-0548, RFC-0550, RFC-0640</history>
+  <item>RFC-1224: preserve operator forge.yaml content on upgrade, promote adrImplementStamp binding</item>
+  <history>RFC-0544, RFC-0548, RFC-0550, RFC-0640, RFC-0643, RFC-1224</history>
 </CHANGE_SUMMARY>
 */
 
@@ -32,6 +32,7 @@ import { scaffoldMemoryLayer } from "./memory-scaffold.ts";
 import {
   loadForgeConfig,
   serializeForgeConfig,
+  applyForgeYamlPatches,
   resolveForgeRoot,
   resolvePackageManager,
   applyCliBindingDefaults,
@@ -346,15 +347,30 @@ export async function runCreate(
     const forgeYamlPath = path.join(targetDir, "forge.yaml");
     if (await fio.exists(forgeYamlPath)) {
       try {
-        const config = loadForgeConfig(targetDir);
         const pm = resolvePackageManager(packageManager);
-        config.project.packageManager = pm;
-        if (config.bindings) {
-          config.bindings.commands = applyCliBindingDefaults(pm);
+        const commandDefaults = applyCliBindingDefaults(pm);
+        // RFC-1224: patch the document init just wrote instead of re-serializing —
+        // preserves comments/formatting and any schema-unknown sections.
+        const raw = await fio.readFile(forgeYamlPath);
+        const patched = applyForgeYamlPatches(raw, [
+          { path: ["project", "packageManager"], value: pm },
+          ...Object.entries(commandDefaults).map(([key, value]) => ({
+            path: ["bindings", "commands", key],
+            value,
+          })),
+        ]);
+        if (patched !== null) {
+          await fio.writeFile(forgeYamlPath, patched);
+        } else {
+          const config = loadForgeConfig(targetDir);
+          config.project.packageManager = pm;
+          if (config.bindings) {
+            config.bindings.commands = commandDefaults;
+          }
+          // RFC-1118: serializeForgeConfig writes the declared profile id verbatim —
+          // never the resolved StackProfile object, never drops an unresolvable id.
+          await fio.writeFile(forgeYamlPath, stringifyYaml(serializeForgeConfig(config)));
         }
-        // RFC-1118: serializeForgeConfig writes the declared profile id verbatim —
-        // never the resolved StackProfile object, never drops an unresolvable id.
-        await fio.writeFile(forgeYamlPath, stringifyYaml(serializeForgeConfig(config)));
       } catch {
         // Post-processing is best-effort — init already wrote a valid config
       }

package/src/onboarding/doctor.ts

@@ -8,7 +8,6 @@ Checks for forge.yaml, AGENTS.md, PREFERENCES.md, .agents/skills/, docs/rfcs/,
 </non-goals>
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
-  <item>RFC-0664: added memory-layer health check (budget usage, gitignore coverage, daily-file leak risk).</item>
   <item>RFC-0704: added independent-version-packages check — validates that paths in independentVersionPackages exist and contain package.json.</item>
   <item>RFC-0941: added pack-manifests advisory check — validates forge.plugin.yaml existence and schema for each declared skill pack.</item>
   <item>RFC-1097: step 6 — compass.migrate codemod run
@@ -17,7 +16,9 @@ Mechanical v1 to v2 header migration across the workspace: 942 files rewritten
   <item>RFC-1097: sweep — packages/forge + services clean
 
 Sweep batch 2: real KEY_DECISIONS on 10 files, expanded purposes (CONTRACT-02/PURPOSE-02), headers on mission/index + gen-upstreams, sanitizeItemText in summary.record (literal Compass tags corrupted history), excludedPaths for wrangler types, test-fixtures testPattern. forge+services now 0 diagnostics under --mode error.</item>
-  <history>RFC-0391, RFC-0393, RFC-0524, RFC-0539, RFC-0540, RFC-0611, RFC-0640, RFC-0660, RFC-0661, RFC-0663, RFC-0675</history>
+  <item>RFC-1224: preserve operator forge.yaml content on upgrade, promote adrImplementStamp binding</item>
+  <item>RFC-1226: skills-unmanaged check — warn on SKILL.md-bearing dirs without .forge-managed marker</item>
+  <history>RFC-0391, RFC-0393, RFC-0524, RFC-0539, RFC-0540, RFC-0611, RFC-0640, RFC-0660, RFC-0661, RFC-0663, RFC-0664, RFC-0675, RFC-1224</history>
 </CHANGE_SUMMARY>
 */
 
@@ -54,6 +55,7 @@ import { compactMemoryMd } from "./memory-compact.ts";
 import { checkInvariants } from "./invariant-engine.ts";
 import type { InvariantViolation } from "./invariant-engine.ts";
 import { checkNpmToken, workshopNeedsWarpgogolToken } from "./npm-token-check.ts";
+import { SKILL_MARKER_FILE } from "./skill-markers.ts";
 import { execSync } from "../utils/sync-fs.ts";
 import * as fs from "../utils/sync-fs.ts";
 import type { ProfilePrerequisite } from "../profiles/profile-schema.ts";
@@ -142,6 +144,7 @@ const BINDING_COMMAND_KEYS = [
   "commands.validateRfc",
   "commands.validateAdr",
   "commands.implementStamp",
+  "commands.adrImplementStamp",
   "commands.typecheck",
   "commands.test",
   "commands.scopedBuild",
@@ -762,6 +765,49 @@ async function checkPackSkills(workspaceRoot: string, io: WorkspaceIO): Promise<
   };
 }
 
+// ---------------------------------------------------------------------------
+// Unmanaged skill directories (RFC-1226)
+// ---------------------------------------------------------------------------
+
+// Visibility only: a dir bearing SKILL.md without .forge-managed is
+// consumer-owned per RFC-1154 — report it, never prune or block.
+async function checkUnmanagedSkillDirs(workspaceRoot: string, io: WorkspaceIO): Promise<DoctorCheck> {
+  let config;
+  try {
+    config = loadForgeConfig(workspaceRoot);
+  } catch {
+    return { name: "skills-unmanaged", status: "pass", message: "No forge.yaml — unmanaged skill dir check skipped" };
+  }
+
+  const skillsDir = join(workspaceRoot, config.paths.skillsDir);
+  let entries;
+  try {
+    entries = await io.readdir(skillsDir);
+  } catch {
+    return { name: "skills-unmanaged", status: "pass", message: "skillsDir absent — nothing to scan" };
+  }
+
+  const unmanaged: string[] = [];
+  for (const entry of entries) {
+    if (!entry.isDirectory) continue;
+    const dir = join(skillsDir, entry.name);
+    // Support dirs (_shared/, knowledge-only dirs) are not skills — skip.
+    if (!(await io.exists(join(dir, "SKILL.md")))) continue;
+    if (await io.exists(join(dir, SKILL_MARKER_FILE))) continue;
+    unmanaged.push(entry.name);
+  }
+
+  if (unmanaged.length === 0) {
+    return { name: "skills-unmanaged", status: "pass", message: "all skill dirs forge-managed" };
+  }
+
+  return {
+    name: "skills-unmanaged",
+    status: "warn",
+    message: `${unmanaged.length} unmanaged skill dir(s): ${unmanaged.join(", ")} — consumer-owned (no ${SKILL_MARKER_FILE} marker). Port into a declared skillPacks pack to make them forge-managed, or leave as-is.`,
+  };
+}
+
 // ---------------------------------------------------------------------------
 // Pack manifest diagnostics (RFC-0941)
 // ---------------------------------------------------------------------------
@@ -1455,6 +1501,9 @@ export async function runDoctor(
   const packManifestCheck = await checkPackManifests(workspaceRoot, fio);
   checks.push(packManifestCheck);
 
+  // RFC-1226: List unmanaged skill dirs under skillsDir (visibility only)
+  checks.push(await checkUnmanagedSkillDirs(workspaceRoot, fio));
+
   // RFC-0704: Check independent version packages — paths must exist and contain package.json
   const independentCheck = await checkIndependentVersionPackages(workspaceRoot, fio);
   checks.push(independentCheck);

package/src/onboarding/init.ts

@@ -461,9 +461,9 @@ export function runInit(
     }
   }
 
-  // 5. Copy neutral RFC and ADR templates into the project's docs directories.
-  // Templates are copied from the forge package so new projects can create
-  // RFCs and ADRs without falling back to the forge package's internal copy.
+  // 5. Copy neutral RFC, ADR, and plan templates into the project's docs
+  // directories. Templates are copied from the forge package so new projects
+  // can create documents without falling back to the package's internal copy.
   // Existing templates are never overwritten.
   const templatesToCopy = [
     {
@@ -476,6 +476,11 @@ export function runInit(
       destRel: `${config.paths.adrsDir}/adr-0000-template.md`,
       label: "docs/adrs/adr-0000-template.md",
     },
+    {
+      src: path.join(forgeRoot, "os", "plan", "plan-0000-template.md"),
+      destRel: `${config.paths.plansDir}/plan-0000-template.md`,
+      label: "docs/plans/plan-0000-template.md",
+    },
   ];
   for (const { src, destRel, label } of templatesToCopy) {
     const destPath = path.join(workspaceRoot, destRel);

package/src/onboarding/upgrade.ts

@@ -11,7 +11,6 @@ With --update-npm, also updates @warpgogol/forge from npm before syncing (skippe
 </non-goals>
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
-  <item>RFC-0611: added nested AGENTS.md generation after skill sync.</item>
   <item>RFC-0663: added syncSharedKnowledge step to sync shared knowledge layer to .agents/skills/shared-knowledge/.</item>
   <item>RFC-0664: added scaffoldMemoryLayer step to scaffold .agents/memory/ and .gitignore block.</item>
   <item>RFC-1097: step 6 — compass.migrate codemod run
@@ -20,7 +19,8 @@ Mechanical v1 to v2 header migration across the workspace: 942 files rewritten
   <item>RFC-1097: sweep — werkstatt-engine clean
 
 Sweep batch 4: 73 Compass headers on headerless engine files (certification, component-runtime, isolation, evolution, testing), real KEY_DECISIONS on 75 files (kernel, cache, dht, swim, gitmesh, runtime), ~80 purpose expansions (CONTRACT-02/PURPOSE-02), non-goals on 13 CONTRACT-03 files, CS-07 history literal fix repo-wide (253 files). Policy: .template.ts/.template.astro excludedPaths. werkstatt-engine now 0 diagnostics.</item>
-  <history>RFC-0543</history>
+  <item>RFC-1224: preserve operator forge.yaml content on upgrade, promote adrImplementStamp binding</item>
+  <history>RFC-0543, RFC-0611, RFC-1224</history>
 </CHANGE_SUMMARY>
 */
 
@@ -35,10 +35,12 @@ import {
   resolveForgeRoot,
   loadForgeConfig,
   serializeForgeConfig,
+  applyForgeYamlPatches,
   resolvePmRunner,
   resolvePmInstall,
   type ForgeConfig,
   type ForgeBindings,
+  type ForgeYamlPatch,
 } from "../config/forge-config.ts";
 import { FORGE_SKILLS, discoverPackSkills } from "../registry.ts";
 import { syncKnowledgeFile } from "../knowledge/index.ts";
@@ -452,18 +454,38 @@ async function updateSyncedVersion(
   workspaceRoot: string,
   config: ForgeConfig,
   version: string,
+  bindingsAdded: { key: string; value: string }[],
   dryRun: boolean,
   io: WorkspaceIO,
+  warn: (msg: string) => void,
 ): Promise<void> {
   if (dryRun) return;
 
-  // Set the syncedVersion on the in-memory config (bindings already updated)
+  // Keep the in-memory config truthful for downstream steps (doctor).
   config.forge = { syncedVersion: version };
 
-  // Write the full config back to forge.yaml
+  // RFC-1224: patch the existing document — syncedVersion plus newly added
+  // binding defaults — instead of re-serializing, so consumer-owned keys,
+  // sections, comments and ordering survive the rewrite.
+  const patches: ForgeYamlPatch[] = [
+    { path: ["forge", "syncedVersion"], value: version },
+    ...bindingsAdded.map((b) => ({
+      path: ["bindings", ...b.key.split(".")],
+      value: b.value,
+    })),
+  ];
+  const forgeYamlPath = path.join(workspaceRoot, "forge.yaml");
+  const raw = await io.readFile(forgeYamlPath).catch(() => null);
+  const patched = raw === null ? null : applyForgeYamlPatches(raw, patches);
+  if (patched !== null) {
+    await io.writeFile(forgeYamlPath, patched);
+    return;
+  }
+  warn(
+    "forge.upgrade: forge.yaml patch failed — falling back to full serialization (schema-unknown keys may be dropped)",
+  );
   // RFC-1118: serializeForgeConfig writes the declared profile id verbatim —
   // never the resolved StackProfile object, never drops an unresolvable id.
-  const forgeYamlPath = path.join(workspaceRoot, "forge.yaml");
   await io.writeFile(forgeYamlPath, stringifyYaml(serializeForgeConfig(config)));
 }
 
@@ -689,7 +711,15 @@ export async function runUpgrade(
 
   // Step 5: Update forge.syncedVersion
   if (!isDryRun) {
-    await updateSyncedVersion(workspaceRoot, config, toVersion, false, fio);
+    await updateSyncedVersion(
+      workspaceRoot,
+      config,
+      toVersion,
+      bindingsAdded,
+      false,
+      fio,
+      (msg) => context.logger.warn(msg),
+    );
   }
 
   // Step 5b: Generate nested AGENTS.md (RFC-0611). RFC-1153: the write path

package/src/pipeline-status.ts

@@ -19,9 +19,24 @@ os/ or kernel imports.</purpose>
   <item>RFC-1140: steps 1-4 — shared resolver, queue module, registration
 
 Extract pipeline-status derivation into packages/forge/src/pipeline-status.ts, refactor rfc.pipeline.status onto it, add os/queue module with queue.validate command, register in WORKSHOP_MODULE_MAP.forge + bin/cli.ts + package.json exports.</item>
+  <item>RFC-1250: decision-aware queue report — QueueItemReport gains kind +
+  decisions counts; `next` exclusion of parked/blocked items is computed by
+  the queue.validate caller (the resolver stays ledger-agnostic).</item>
   <item>RFC-1140: step 6 — cover shared resolver and queue.validate
 
 Add pipeline-status.test.ts (derivation matrix, order, next selection, ADR, skipped) and queue-validate.test.ts (schema errors, unknown/dup ids, id-filename mismatch, empty items, dependsOn warning, JSON shape). Fix queue derivation: pipelineStep is the stage after the last completed stage, not first incomplete — handles non-contiguous artifacts.</item>
+  <item>RFC-1250: review wave — QUEUE-07 gates un-parked opens only, ledger binding checks, fail-closed loader
+
+REVIEW-RFC-1250-01 findings: QUEUE-07 no longer fires on parked entries
+(the park is the containment — a parked queue stays resumable and the
+decision window arbitrates it); loader fails open only on ENOENT —
+other read errors are QUEUE-01; ledger gains queue/id-stem binding
+(QUEUE-02), Q-N uniqueness (QUEUE-05), and QUEUE-08 hygiene warnings
+(missing answers, foreign doc ids); next excludes QUEUE-07-blocked
+items; top-level decision totals added. Orchestrator pre-flight treats
+QUEUE-07 as the window agenda — structural errors still stop the batch;
+maturation skips parked/deferred items; uncovered imperative ask sites
+gain collect riders (ADR code-trace, NC markers, audit-verdict guard).</item>
 </CHANGE_SUMMARY>
 */
 
@@ -244,6 +259,10 @@ export interface QueueItemReport {
   status: QueueItemStatus;
   /** First incomplete pipeline stage — present only when in-progress. */
   pipelineStep?: PipelineStage;
+  /** Document kind — needed by queue.validate for QUEUE-07 (any non-terminal ADR is implementable). */
+  kind?: QueueDocumentKind;
+  /** Decision-ledger counts (RFC-1250) — present when the queue carries a sibling ledger. */
+  decisions?: { open: number; deferred: number; autoResolved: number };
 }
 
 /**
@@ -266,20 +285,20 @@ export async function deriveQueueItemStatus(
   const implementedAt = fm["implementedAt"] ? String(fm["implementedAt"]) : undefined;
 
   if (status === "implemented" || implementedAt) {
-    return { id: doc.id, status: "implemented" };
+    return { id: doc.id, status: "implemented", kind: doc.kind };
   }
   if (status === "rejected" || status === "superseded") {
-    return { id: doc.id, status: "skipped" };
+    return { id: doc.id, status: "skipped", kind: doc.kind };
   }
 
   // ADR pipeline is create → implement — no intermediate artifacts exist.
   if (doc.kind === "adr") {
-    return { id: doc.id, status: "pending" };
+    return { id: doc.id, status: "pending", kind: doc.kind };
   }
 
   // RFC accepted means the plan step is done by definition.
   if (status === "accepted") {
-    return { id: doc.id, status: "in-progress", pipelineStep: "implement" };
+    return { id: doc.id, status: "in-progress", pipelineStep: "implement", kind: doc.kind };
   }
 
   const enhancedAt = fm["enhancedAt"] ? String(fm["enhancedAt"]) : undefined;
@@ -302,13 +321,13 @@ export async function deriveQueueItemStatus(
     if (stages[i]!.done) lastDone = i;
   }
   if (lastDone === -1) {
-    return { id: doc.id, status: "pending" };
+    return { id: doc.id, status: "pending", kind: doc.kind };
   }
   const nextStage = stages[lastDone + 1];
   if (!nextStage) {
-    return { id: doc.id, status: "pending" };
+    return { id: doc.id, status: "pending", kind: doc.kind };
   }
-  return { id: doc.id, status: "in-progress", pipelineStep: nextStage.stage };
+  return { id: doc.id, status: "in-progress", pipelineStep: nextStage.stage, kind: doc.kind };
 }
 
 export interface QueueReport {
@@ -317,7 +336,12 @@ export interface QueueReport {
   next: string | null;
 }
 
-/** Derive the full queue report, preserving manifest order. */
+/**
+ * Derive the full queue report, preserving manifest order. `next` selects
+ * the first pending/in-progress item; callers that need ledger-aware
+ * exclusion (RFC-1250 parked/QUEUE-07-blocked items) re-filter the returned
+ * items — the resolver itself stays ledger-agnostic.
+ */
 export async function deriveQueueReport(
   workspaceRoot: string,
   itemIds: string[],

package/src/tests/bindings-schema.test.ts

@@ -7,6 +7,8 @@
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
   <item>RFC-0639: initial test suite for semantic command keys, terminology resolution, and applyCliBindingDefaults.</item>
+  <item>RFC-1224: adrImplementStamp schema/default coverage; consumer-defined command key passthrough.</item>
+  <item>RFC-1224: preserve operator forge.yaml content on upgrade, promote adrImplementStamp binding</item>
 </CHANGE_SUMMARY>
 */
 
@@ -174,12 +176,14 @@ describe("RFC-0639: Terminology schema change", () => {
 });
 
 describe("RFC-0639: applyCliBindingDefaults", () => {
-  test("returns 13 keys with semantic keys as null", () => {
+  test("returns 14 keys with semantic keys as null", () => {
     const commands = applyCliBindingDefaults("pnpm");
     // CLI-backed keys are non-null
     expect(commands.validateRfc).not.toBeNull();
     expect(commands.validateAdr).not.toBeNull();
     expect(commands.implementStamp).not.toBeNull();
+    // RFC-1224: adr.implement.stamp promoted to a first-class default binding
+    expect(commands.adrImplementStamp).not.toBeNull();
     expect(commands.specValidate).not.toBeNull();
     expect(commands.sessionSave).not.toBeNull();
     // Software-domain keys are null
@@ -312,3 +316,51 @@ describe("RFC-0639: resolveTerminology", () => {
     expect(resolveTerminology(config, undefined, "source")).toBe("source file");
   });
 });
+
+describe("RFC-1224: promoted + consumer-defined command keys", () => {
+  test("forgeBindingsSchema accepts adrImplementStamp", () => {
+    const result = forgeBindingsSchema.safeParse({
+      schema: "forge/bindings@1",
+      commands: {
+        adrImplementStamp:
+          "pnpm exec forge adr.implement.stamp --id {id} --implementation-commit {commit}",
+      },
+      paths: {
+        invariantsFile: null,
+        compassDocs: [],
+        reviewsDir: null,
+        handoffsDir: null,
+        sessionsDir: null,
+      },
+    });
+    expect(result.success).toBe(true);
+    if (result.success) {
+      expect(result.data.commands.adrImplementStamp).toBe(
+        "pnpm exec forge adr.implement.stamp --id {id} --implementation-commit {commit}",
+      );
+    }
+  });
+
+  test("forgeBindingsSchema retains consumer-defined command keys", () => {
+    const result = forgeBindingsSchema.safeParse({
+      schema: "forge/bindings@1",
+      commands: {
+        validateRfc: null,
+        createRfc: "pnpm exec forge rfc.create",
+      },
+      paths: {
+        invariantsFile: null,
+        compassDocs: [],
+        reviewsDir: null,
+        handoffsDir: null,
+        sessionsDir: null,
+      },
+    });
+    expect(result.success).toBe(true);
+    if (result.success) {
+      expect((result.data.commands as Record<string, unknown>)["createRfc"]).toBe(
+        "pnpm exec forge rfc.create",
+      );
+    }
+  });
+});

package/src/tests/cli-flags.test.ts

@@ -143,6 +143,26 @@ describe("resolveCliFlags — strict (schema-carrying) path", () => {
     expect(flags["files"]).toEqual(["a", "b"]);
     expect(errorRuleIds(diagnostics)).toEqual([]);
   });
+
+  test("string[] flag consumes consecutive bare tokens as a value list", () => {
+    const cmd = command({
+      files: { kind: "string[]", description: "Files." },
+      id: { kind: "string", description: "Id." },
+    });
+    const { flags, diagnostics } = resolveCliFlags(["--files", "a", "b", "c", "--id", "x"], cmd);
+    expect(flags["files"]).toEqual(["a", "b", "c"]);
+    expect(flags["id"]).toBe("x");
+    expect(errorRuleIds(diagnostics)).toEqual([]);
+  });
+
+  test("string[] flag with no value still emits KERNEL-FLAG-02", () => {
+    const cmd = command({
+      files: { kind: "string[]", description: "Files." },
+      json2: { kind: "boolean", description: "Switch." },
+    });
+    const { diagnostics } = resolveCliFlags(["--files", "--json2"], cmd);
+    expect(errorRuleIds(diagnostics)).toEqual(["KERNEL-FLAG-02"]);
+  });
 });
 
 describe("resolveCliFlags — legacy (schema-less) path", () => {

package/src/tests/cli-output.test.ts

@@ -8,6 +8,12 @@ renderNextSteps, renderIdeRecommendation, generateHelp.</purpose>
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
   <item>RFC-0542: initial CLI output rendering tests.</item>
+  <item>RFC-1231: step 1 — rename supportsAllSites to acceptsAllFlag
+
+Mechanical sweep: the field only ever gated --all argv acceptance; fan-out
+follows the parsed selector. Guard renamed assertAllSitesAllowed ->
+assertAllFlagAccepted, message updated. 417 declaration sites + type
+surfaces (KernelCommandMetadata, ForgeCommandMetadata) in one atomic pass.</item>
 </CHANGE_SUMMARY>
 */
 
@@ -41,7 +47,7 @@ class MockRegistry implements CommandRegistry, ForgeModuleRegistry {
       provider: "workspace" as const,
       mutatesState: c.mutatesState,
       requiresNetwork: c.requiresNetwork,
-      supportsAllSites: c.supportsAllSites,
+      acceptsAllFlag: c.acceptsAllFlag,
       timeoutMs: c.timeoutMs,
       expectedDurationMs: c.expectedDurationMs,
       longRunning: c.longRunning,
@@ -112,7 +118,7 @@ describe("generateHelp", () => {
       name: "forge.init",
       description: "Deploy forge into a project.",
       scope: "workspace",
-      supportsAllSites: false,
+      acceptsAllFlag: false,
       flags: {},
       cacheable: false,
       execute: async () => ({ exitCode: 0 }),
@@ -121,7 +127,7 @@ describe("generateHelp", () => {
       name: "forge.doctor",
       description: "Diagnose forge state in an existing project.",
       scope: "workspace",
-      supportsAllSites: false,
+      acceptsAllFlag: false,
       flags: {},
       cacheable: false,
       execute: async () => ({ exitCode: 0 }),
@@ -138,7 +144,7 @@ describe("generateHelp", () => {
       name: "forge.doctor",
       description: "Diagnose forge state.",
       scope: "workspace",
-      supportsAllSites: false,
+      acceptsAllFlag: false,
       flags: {},
       cacheable: false,
       execute: async () => ({ exitCode: 0 }),
@@ -147,7 +153,7 @@ describe("generateHelp", () => {
       name: "forge.init",
       description: "Deploy forge.",
       scope: "workspace",
-      supportsAllSites: false,
+      acceptsAllFlag: false,
       flags: {},
       cacheable: false,
       execute: async () => ({ exitCode: 0 }),
@@ -167,7 +173,7 @@ describe("generateHelp", () => {
       name: "forge.init",
       description: "Deploy forge.",
       scope: "workspace",
-      supportsAllSites: false,
+      acceptsAllFlag: false,
       flags: {},
       cacheable: false,
       execute: async () => ({ exitCode: 0 }),
@@ -184,7 +190,7 @@ describe("generateHelp", () => {
       name: "forge.init",
       description: "Deploy forge.",
       scope: "workspace",
-      supportsAllSites: false,
+      acceptsAllFlag: false,
       flags: {},
       cacheable: false,
       execute: async () => ({ exitCode: 0 }),

package/src/tests/doctor-bindings.test.ts

@@ -4,6 +4,8 @@
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
   <item>RFC-0540: initial doctor binding notices tests.</item>
+  <item>RFC-1224: adrImplementStamp added to notice inventory and fixtures.</item>
+  <item>RFC-1224: preserve operator forge.yaml content on upgrade, promote adrImplementStamp binding</item>
 </CHANGE_SUMMARY>
 */
 
@@ -58,7 +60,7 @@ async function setupMinimalProject(dir: string): Promise<void> {
   await writeFile(join(dir, "PREFERENCES.md"), "# Test", "utf8");
 }
 
-test("doctor emits defaultable-binding-null notices for all 5 forge-CLI bindings when null", async () => {
+test("doctor emits defaultable-binding-null notices for all 6 forge-CLI bindings when null", async () => {
   await setupMinimalProject(tempDir);
   await writeForgeYaml(
     tempDir,
@@ -96,11 +98,12 @@ bindings:
 
   const result = await runDoctor({ argv: [], flags: {} }, mockContext(tempDir));
   const notices = result.data?.bindings?.notices ?? [];
-  expect(notices).toHaveLength(5);
+  expect(notices).toHaveLength(6);
   const keys = notices.map((n) => n.key);
   expect(keys).toContain("commands.validateRfc");
   expect(keys).toContain("commands.validateAdr");
   expect(keys).toContain("commands.implementStamp");
+  expect(keys).toContain("commands.adrImplementStamp");
   expect(keys).toContain("commands.specValidate");
   expect(keys).toContain("commands.sessionSave");
   for (const notice of notices) {
@@ -131,6 +134,7 @@ bindings:
     validateRfc: "custom rfc validate command"
     validateAdr: "custom adr validate command"
     implementStamp: "custom stamp command"
+    adrImplementStamp: "custom adr stamp command"
     typecheck: null
     test: null
     scopedBuild: null
@@ -172,6 +176,7 @@ bindings:
     validateRfc: "npx forge rfc.validate --id {id} --json"
     validateAdr: "npx forge adr.validate --id {id} --json"
     implementStamp: "npx forge rfc.implement.stamp --id {id} --implementation-commit {commit}"
+    adrImplementStamp: "npx forge adr.implement.stamp --id {id} --implementation-commit {commit}"
     typecheck: null
     test: null
     scopedBuild: null