@warpgogol/forge 4.2.1 → 4.2.3

package/skills/fo/fo-compass-annotate/templates/header-format.md

@@ -1,13 +1,16 @@
-# Canonical Compass Header Format
+# Canonical Compass Header Format (v2, RFC-1094)
 
 This is the canonical format for Compass source headers. Check the project's invariants file for the canonical Compass markup rule.
 
-## Two-block contract
+## Three-block contract
 
-Every non-trivial source file in `apps/`, `packages/`, or `services/` must carry:
+Every non-trivial source file in `apps/`, `packages/`, or `services/` must carry, in this canonical order:
 
 1. `MODULE_CONTRACT` — describes the file's purpose and boundaries.
-2. `CHANGE_SUMMARY` — chronological list of changes with RFC/ticket references.
+2. `KEY_DECISIONS` — current-state design truths (required on medium/high riskClass files, optional on low).
+3. `CHANGE_SUMMARY` — compressed change history: the 5 newest governance-ID items plus a `<history>` ID tail.
+
+Blocks out of this order are a `COMPASS-ORDER-01` violation.
 
 ## MODULE_CONTRACT
 
@@ -22,24 +25,42 @@ Every non-trivial source file in `apps/`, `packages/`, or `services/` must carry
 
 ### Rules
 
-- `<purpose>` must be specific to this file, under 100 characters.
+- `<purpose>` must be specific to this file, under 100 characters, and contain at least one file-derived token (filename stem segment, parent-directory segment for generic stems, or an exported symbol name). Boilerplate openers ("This file…", "Utility…", "TODO…") are `COMPASS-PURPOSE-01`; a purpose with no file-derived token is `COMPASS-PURPOSE-02`.
 - `<non-goals>` must have at least one `<item>`.
 - Do not include `@ai-invariant` lines in `MODULE_CONTRACT` — they are inline comments in the file body.
 
+## KEY_DECISIONS
+
+```
+<KEY_DECISIONS>
+  <item>Validation is deterministic regex parsing — no LLM in the check path.</item>
+  <item>Warnings never fail the build during the migration window.</item>
+</KEY_DECISIONS>
+```
+
+### Rules
+
+- Content: current-state design truths of the file — invariants, chosen mechanisms, forbidden approaches. Rewritten in place when behavior changes; never appended chronologically.
+- Required on files whose `riskClass` is `medium` or `high` (`COMPASS-KD-01`); optional on `low`. When present on a low-risk file, all item rules still apply.
+- Limits: 1–7 `<item>` entries (`COMPASS-KD-04`), each ≤ 20 words (`COMPASS-KD-03`).
+- An item MUST NOT begin with a governance-ID prefix (`RFC-`, `ADR-`, ticket pattern) — history belongs in `CHANGE_SUMMARY` (`COMPASS-KD-05`).
+- An empty block or `TODO` placeholder items are a violation (`COMPASS-KD-02`).
+
 ## CHANGE_SUMMARY
 
 ```
 <CHANGE_SUMMARY>
-  <item>RFC-XXXX: one-line description of the change.</item>
+  <item>RFC-1094: adopted v2 three-block contract.</item>
+  <history>RFC-0348, RFC-0349, RFC-0538</history>
 </CHANGE_SUMMARY>
 ```
 
 ### Rules
 
-- Each `<item>` should reference an RFC id, ticket id, or ADR id (protected items).
-- Items without an RFC/ticket reference are "unprotected" and may be trimmed by `compass.summary.trim`.
-- Maximum 30 total items per file (enforced by `compass.changesummary.validate`).
-- Boilerplate items (e.g., "Initial creation", "Compass scaffolding") are automatically removed by `compass.summary.trim`.
+- At most 5 `<item>` entries, each `ID: one-line description` (`COMPASS-CS-05`). The retained items are the 5 newest — the chronological tail.
+- Every `<item>` MUST carry a governance ID (`RFC-XXXX`, `ADR-XXXX`, ticket pattern); items without an ID are violations (`COMPASS-CS-06`) and will be deleted by `compass.migrate` (sibling RFC, not yet implemented).
+- Older IDs collapse into a single `<history>` element: comma-separated, deduplicated, per-namespace ascending numeric order (`COMPASS-CS-07`). `<history>` carries IDs only — no descriptions — and has no length cap.
+- An empty `CHANGE_SUMMARY` (no items, no history) is legal for files never touched by a governance-referencing commit.
 
 ## Comment syntax by extension
 
@@ -47,8 +68,9 @@ See `reference/comment-styles.md` for the correct comment syntax per file extens
 
 ## Forbidden blocks
 
-The following legacy Compass blocks are forbidden and must be removed:
+The following legacy Compass blocks are forbidden and must be removed (`COMPASS-FORBIDDEN-01`, always an error regardless of `--mode`):
 
+- `MODULE_MAP`, `keywords`, `responsibilities`, `COMPASS_BLOCK` — removed by RFC-0348.
 - `GRACE_MODULE_CONTRACT` — use `MODULE_CONTRACT` instead.
 - `GRACE_CHANGE_SUMMARY` — use `CHANGE_SUMMARY` instead.
 - `AI_INVARIANTS` — use inline `// @ai-invariant` comments instead.

package/skills/fo/fo-compass-annotate/templates/header-prompt.md

@@ -19,9 +19,17 @@ Generate a `MODULE_CONTRACT` block with:
 - The purpose must be specific to this file, not generic.
 - Non-goals must describe real boundaries the file respects.
 - Do not use vague phrases like "utility functions" or "helper code".
-- Do not reference the file name in the purpose.
+- The purpose MUST contain at least one file-derived token — a filename stem segment, a parent-directory segment for generic stems, or an exported symbol name (COMPASS-PURPOSE-02). Do not open with boilerplate like "This file" or "Utility" (COMPASS-PURPOSE-01).
 - Keep the purpose under 100 characters.
 
+## KEY_DECISIONS (v2, RFC-1094)
+
+When the file's `riskClass` is `medium` or `high`, also generate a `KEY_DECISIONS` block between `MODULE_CONTRACT` and `CHANGE_SUMMARY`:
+
+- 1–7 `<item>` entries, each ≤ 20 words.
+- Content is current-state design truth: invariants, chosen mechanisms, forbidden approaches — "current truth, not history".
+- Items MUST NOT start with a governance-ID prefix (`RFC-`, `ADR-`, ticket) — history belongs in `CHANGE_SUMMARY`.
+
 ## Example
 
 ```
@@ -32,4 +40,7 @@ Generate a `MODULE_CONTRACT` block with:
   <item>Do not delete protected (RFC/code-referencing) items under any circumstance.</item>
 </non-goals>
 </MODULE_CONTRACT>
+<KEY_DECISIONS>
+  <item>Classification is deterministic regex — no LLM in the validate path.</item>
+</KEY_DECISIONS>
 ```

package/src/tests/create.test.ts

@@ -64,7 +64,7 @@ test("forge create --in-place scaffolds in cwd with forge.yaml and docs dirs", a
   expect(existsSync(join(tempDir, "package.json"))).toBe(true);
   expect(existsSync(join(tempDir, "scripts", "clean.mjs"))).toBe(true);
   expect(existsSync(join(tempDir, "hooks", "pre-user-prompt-wrapper.mjs"))).toBe(true);
-}, 30000);
+}, 120000);
 
 test("forge create --in-place refuses when forge artifacts already exist", async () => {
   await writeFile(join(tempDir, "forge.yaml"), "existing: true", "utf8");
@@ -102,7 +102,7 @@ test("forge create --in-place tolerates only .git directory in target directory"
   expect(result.data?.status).toBe("pass");
   expect(existsSync(join(tempDir, "forge.yaml"))).toBe(true);
   expect(existsSync(join(tempDir, ".git"))).toBe(true);
-}, 30000);
+}, 120000);
 
 test("forge create fails on non-kebab-case name", async () => {
   const result = await runCreate(
@@ -141,7 +141,7 @@ test("forge create --in-place --profile forge-shell uses forge-shell profile", a
   );
   expect(result.exitCode).toBe(0);
   expect(result.data?.profile).toBe("forge-shell");
-}, 30000);
+}, 120000);
 
 test("forge create --package-manager npm writes npm into forge.yaml with npx bindings", async () => {
   const result = await runCreate(
@@ -162,7 +162,7 @@ test("forge create --package-manager npm writes npm into forge.yaml with npx bin
   expect(config.project.packageManager).toBe("npm");
   expect(config.bindings?.commands.validateRfc).toContain("npx");
   expect(config.bindings?.commands.validateRfc).toContain("forge rfc.validate");
-}, 30000);
+}, 120000);
 
 test("forge create --package-manager pnpm writes pnpm into forge.yaml", async () => {
   const result = await runCreate(
@@ -182,7 +182,7 @@ test("forge create --package-manager pnpm writes pnpm into forge.yaml", async ()
   const config = loadForgeConfig(tempDir);
   expect(config.project.packageManager).toBe("pnpm");
   expect(config.bindings?.commands.validateRfc).toContain("pnpm exec");
-}, 30000);
+}, 120000);
 
 test("forge.yaml has non-null forge-CLI bindings and null stack bindings", async () => {
   await runCreate(
@@ -199,7 +199,7 @@ test("forge.yaml has non-null forge-CLI bindings and null stack bindings", async
   expect(config.bindings?.commands.typecheck).toBeNull();
   expect(config.bindings?.commands.test).toBeNull();
   expect(config.bindings?.commands.scopedBuild).toBeNull();
-}, 30000);
+}, 120000);
 
 test("forge.yaml has forge.syncedVersion set", async () => {
   await runCreate(
@@ -209,7 +209,7 @@ test("forge.yaml has forge.syncedVersion set", async () => {
 
   const config = loadForgeConfig(tempDir);
   expect(config.forge?.syncedVersion).toBeTruthy();
-}, 30000);
+}, 120000);
 
 test("result includes nextSteps with forge-bootstrap", async () => {
   const result = await runCreate(
@@ -220,7 +220,7 @@ test("result includes nextSteps with forge-bootstrap", async () => {
   expect(result.nextSteps).toBeDefined();
   expect(result.nextSteps?.some((s) => s.action.includes("forge-bootstrap"))).toBe(true);
   expect(result.nextSteps?.some((s) => s.kind === "required")).toBe(true);
-}, 30000);
+}, 120000);
 
 test("forge create generates AGENTS.md with behavioral layer (RFC-0548)", async () => {
   const result = await runCreate(
@@ -237,7 +237,7 @@ test("forge create generates AGENTS.md with behavioral layer (RFC-0548)", async
   expect(agentsMd).toContain("<!-- forge:begin behavioral-layer -->");
   expect(agentsMd).toContain("<!-- forge:end behavioral-layer -->");
   expect(agentsMd).toContain("### Intent-to-skill routing");
-}, 30000);
+}, 120000);
 
 test("forge create writes NEXT_STEPS.md with greenfield and transplant guidance (RFC-0550)", async () => {
   const result = await runCreate(
@@ -255,7 +255,7 @@ test("forge create writes NEXT_STEPS.md with greenfield and transplant guidance
   expect(nextSteps).toContain("forge-bootstrap");
   expect(nextSteps).toContain("language");
   expect(result.data?.filesCreated).toContain("NEXT_STEPS.md");
-}, 30000);
+}, 120000);
 
 test("forge create root package.json has scripts and replaced project name", async () => {
   const result = await runCreate(
@@ -272,7 +272,7 @@ test("forge create root package.json has scripts and replaced project name", asy
   expect(pkgJson.scripts["format:check"]).toBe("prettier --check .");
   expect(pkgJson.scripts.test).toBe("vitest run --passWithNoTests");
   expect(pkgJson.scripts["upgrade-packages"]).toBe("pnpm up");
-}, 30000);
+}, 120000);
 
 test("forge create --in-place derives name from folder name when --name omitted", async () => {
   const namedDir = join(tempDir, "my-derived-project");
@@ -288,7 +288,7 @@ test("forge create --in-place derives name from folder name when --name omitted"
   const { readFile: readFileAsync } = await import("node:fs/promises");
   const pkgJson = JSON.parse(await readFileAsync(join(namedDir, "package.json"), "utf8"));
   expect(pkgJson.name).toBe("my-derived-project");
-}, 30000);
+}, 120000);
 
 test("forge create --profile godot-csharp writes compass.fileExtensions into forge.yaml", async () => {
   const result = await runCreate(
@@ -305,7 +305,7 @@ test("forge create --profile godot-csharp writes compass.fileExtensions into for
   expect(forgeYaml).toContain(".tscn");
   expect(forgeYaml).toContain(".tres");
   expect(forgeYaml).toContain(".gd");
-}, 30000);
+}, 120000);
 
 test("forge create --in-place setup-git-guards.sh contains shell auto-detection (RFC-1019)", async () => {
   const result = await runCreate(
@@ -319,7 +319,7 @@ test("forge create --in-place setup-git-guards.sh contains shell auto-detection
   expect(setupScript).toContain(".zshenv");
   expect(setupScript).toContain(".bashrc");
   expect(setupScript).toContain("profile_file");
-}, 30000);
+}, 120000);
 
 test("forge create --in-place .windsurf/hooks.json calls wrapper not bash directly (RFC-1019)", async () => {
   const result = await runCreate(
@@ -332,7 +332,7 @@ test("forge create --in-place .windsurf/hooks.json calls wrapper not bash direct
   const hooksJson = await readFileAsync(join(tempDir, ".windsurf", "hooks.json"), "utf8");
   expect(hooksJson).toContain("pre-user-prompt-wrapper.mjs");
   expect(hooksJson).not.toContain("2>/dev/null || true");
-}, 30000);
+}, 120000);
 
 test("forge create --in-place pins pre-user-prompt-wrapper.mjs in pinned.yaml (RFC-1019)", async () => {
   const result = await runCreate(
@@ -344,7 +344,7 @@ test("forge create --in-place pins pre-user-prompt-wrapper.mjs in pinned.yaml (R
   const { readFile: readFileAsync } = await import("node:fs/promises");
   const pinnedYaml = await readFileAsync(join(tempDir, ".forge", "pinned.yaml"), "utf8");
   expect(pinnedYaml).toContain("pre-user-prompt-wrapper.mjs");
-}, 30000);
+}, 120000);
 
 test("forge create --profile phaser-turborepo writes compass.fileExtensions into forge.yaml", async () => {
   const result = await runCreate(
@@ -357,4 +357,4 @@ test("forge create --profile phaser-turborepo writes compass.fileExtensions into
   const forgeYaml = await readFileAsync(join(tempDir, "forge.yaml"), "utf8");
   expect(forgeYaml).toContain("compass:");
   expect(forgeYaml).toContain("fileExtensions:");
-}, 30000);
+}, 120000);

package/src/tests/file-size-lint.test.ts

@@ -0,0 +1,181 @@
+/*
+<MODULE_CONTRACT>
+<purpose>Unit tests for forge.file-size.lint — SIZE-01 rule from RFC-1088.</purpose>
+</MODULE_CONTRACT>
+<CHANGE_SUMMARY>
+  <item>RFC-1088: initial tests for portable file-size lint handler.</item>
+</CHANGE_SUMMARY>
+*/
+
+import { describe, it, expect, beforeEach, afterEach } from "vitest";
+import { mkdtemp, mkdir, writeFile, rm, readFile } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import { join } from "node:path";
+import { runFileSizeLint, countLines } from "../../os/core/handlers/file-size-lint.ts";
+import type { ForgeRuntimeContext, ForgeLogger } from "../types.ts";
+
+const logger: ForgeLogger = {
+  section() {},
+  info() {},
+  warn() {},
+  error() {},
+  success() {},
+};
+
+function mockContext(workspaceRoot: string): ForgeRuntimeContext {
+  return {
+    workspaceRoot,
+    logger,
+    dryRun: false,
+    outputFormat: "json",
+  };
+}
+
+function input(flags: Record<string, unknown> = {}): { argv: string[]; flags: Record<string, unknown> } {
+  return { argv: [], flags };
+}
+
+describe("countLines (RFC-1088)", () => {
+  it("counts physical lines including a trailing newline", () => {
+    expect(countLines("a\nb\nc\n")).toBe(4);
+  });
+
+  it("returns 0 for an empty source", () => {
+    expect(countLines("")).toBe(0);
+  });
+});
+
+describe("runFileSizeLint (RFC-1088, command-level)", () => {
+  let tempDir: string;
+
+  beforeEach(async () => {
+    tempDir = await mkdtemp(join(tmpdir(), "forge-file-size-lint-"));
+  });
+
+  afterEach(async () => {
+    await rm(tempDir, { recursive: true, force: true });
+  });
+
+  it("SIZE-01 (warning): flags a 700-line file as warning (601-1200 tier)", async () => {
+    await mkdir(join(tempDir, "packages", "some-pkg", "src"), { recursive: true });
+    const big = Array.from({ length: 700 }, (_, i) => `const x${i} = ${i};`).join("\n");
+    await writeFile(join(tempDir, "packages", "some-pkg", "src", "big.ts"), big, "utf8");
+
+    const result = await runFileSizeLint(input(), mockContext(tempDir));
+    expect(result.exitCode ?? 0).toBe(0);
+
+    const diagnostics = (
+      result.data as { diagnostics: Array<{ ruleId: string; severity: string; file: string }> }
+    ).diagnostics;
+    const hit = diagnostics.find((d) => d.ruleId === "SIZE-01" && d.file.endsWith("big.ts"));
+    expect(hit).toBeDefined();
+    expect(hit!.severity).toBe("warning");
+  });
+
+  it("SIZE-01 (error): flags a 1300-line file as error (1200+ tier)", async () => {
+    await mkdir(join(tempDir, "packages", "some-pkg", "src"), { recursive: true });
+    const huge = Array.from({ length: 1300 }, (_, i) => `const x${i} = ${i};`).join("\n");
+    await writeFile(join(tempDir, "packages", "some-pkg", "src", "huge.ts"), huge, "utf8");
+
+    const result = await runFileSizeLint(input(), mockContext(tempDir));
+    expect(result.exitCode).toBe(1);
+
+    const diagnostics = (
+      result.data as { diagnostics: Array<{ ruleId: string; severity: string; file: string }> }
+    ).diagnostics;
+    const hit = diagnostics.find((d) => d.ruleId === "SIZE-01" && d.file.endsWith("huge.ts"));
+    expect(hit).toBeDefined();
+    expect(hit!.severity).toBe("error");
+  });
+
+  it("passes without diagnostics when every file is under the threshold", async () => {
+    await mkdir(join(tempDir, "packages", "some-pkg", "src"), { recursive: true });
+    await writeFile(
+      join(tempDir, "packages", "some-pkg", "src", "small.ts"),
+      "export const x = 1;\n",
+      "utf8",
+    );
+
+    const result = await runFileSizeLint(input(), mockContext(tempDir));
+    expect(result.exitCode ?? 0).toBe(0);
+
+    const diagnostics = (result.data as { diagnostics: Array<{ ruleId: string }> }).diagnostics;
+    expect(diagnostics.length).toBe(0);
+  });
+
+  it("--write-baseline writes a YAML baseline to workspace root", async () => {
+    await mkdir(join(tempDir, "packages", "some-pkg", "src"), { recursive: true });
+    const big = Array.from({ length: 700 }, (_, i) => `const x${i} = ${i};`).join("\n");
+    await writeFile(join(tempDir, "packages", "some-pkg", "src", "big.ts"), big, "utf8");
+
+    const result = await runFileSizeLint(
+      input({ "write-baseline": true }),
+      mockContext(tempDir),
+    );
+    expect(result.exitCode ?? 0).toBe(0);
+
+    const baselineRaw = await readFile(
+      join(tempDir, "file-size-lint.baseline.yaml"),
+      "utf8",
+    );
+    expect(baselineRaw).toContain("schemaVersion");
+    expect(baselineRaw).toContain("threshold: 600");
+    expect(baselineRaw).toContain("big.ts");
+  });
+
+  it("baseline suppresses files within their ceiling", async () => {
+    await mkdir(join(tempDir, "packages", "some-pkg", "src"), { recursive: true });
+    const big = Array.from({ length: 700 }, (_, i) => `const x${i} = ${i};`).join("\n");
+    await writeFile(join(tempDir, "packages", "some-pkg", "src", "big.ts"), big, "utf8");
+
+    await writeFile(
+      join(tempDir, "file-size-lint.baseline.yaml"),
+      `meta:\n  schemaVersion: 1\n  threshold: 600\nceilings:\n  packages/some-pkg/src/big.ts: 700\n`,
+      "utf8",
+    );
+
+    const result = await runFileSizeLint(input(), mockContext(tempDir));
+    expect(result.exitCode ?? 0).toBe(0);
+
+    const diagnostics = (result.data as { diagnostics: Array<{ ruleId: string }> }).diagnostics;
+    expect(diagnostics.length).toBe(0);
+  });
+
+  it("baseline flags files that grew beyond their ceiling", async () => {
+    await mkdir(join(tempDir, "packages", "some-pkg", "src"), { recursive: true });
+    const big = Array.from({ length: 800 }, (_, i) => `const x${i} = ${i};`).join("\n");
+    await writeFile(join(tempDir, "packages", "some-pkg", "src", "big.ts"), big, "utf8");
+
+    await writeFile(
+      join(tempDir, "file-size-lint.baseline.yaml"),
+      `meta:\n  schemaVersion: 1\n  threshold: 600\nceilings:\n  packages/some-pkg/src/big.ts: 700\n`,
+      "utf8",
+    );
+
+    const result = await runFileSizeLint(input(), mockContext(tempDir));
+    expect(result.exitCode ?? 0).toBe(0);
+
+    const diagnostics = (
+      result.data as { diagnostics: Array<{ ruleId: string; message: string }> }
+    ).diagnostics;
+    const hit = diagnostics.find((d) => d.ruleId === "SIZE-01");
+    expect(hit).toBeDefined();
+    expect(hit!.message).toContain("grew from the baselined 700 to 800");
+  });
+
+  it("--baseline-path overrides the default baseline location", async () => {
+    await mkdir(join(tempDir, "packages", "some-pkg", "src"), { recursive: true });
+    const big = Array.from({ length: 700 }, (_, i) => `const x${i} = ${i};`).join("\n");
+    await writeFile(join(tempDir, "packages", "some-pkg", "src", "big.ts"), big, "utf8");
+
+    const customPath = "custom-baseline.yaml";
+    const result = await runFileSizeLint(
+      input({ "write-baseline": true, "baseline-path": customPath }),
+      mockContext(tempDir),
+    );
+    expect(result.exitCode ?? 0).toBe(0);
+
+    const baselineRaw = await readFile(join(tempDir, customPath), "utf8");
+    expect(baselineRaw).toContain("big.ts");
+  });
+});

package/src/tests/public-surface.test.ts

@@ -4,6 +4,7 @@
 </MODULE_CONTRACT>
 <CHANGE_SUMMARY>
   <item>RFC-1080: initial public-surface validator tests covering all 5 rules, exit codes, and monorepo resolution.</item>
+  <item>RFC-1088: raise SURFACE-01 threshold from 300 to 600 lines.</item>
 </CHANGE_SUMMARY>
 */
 
@@ -101,7 +102,7 @@ async function setupMonorepo(
   await setupStandalone(join(dir, "packages", "forge"), opts);
 }
 
-test("SURFACE-01: README under 300 lines passes", async () => {
+test("SURFACE-01: README under 600 lines passes", async () => {
   await setupStandalone(tempDir, { readmeLines: 100 });
   const result = await runPublicSurfaceValidate({} as never, mockContext(tempDir));
   const checks = result.data?.checks ?? [];
@@ -109,12 +110,20 @@ test("SURFACE-01: README under 300 lines passes", async () => {
   expect(surface01?.status, "README with 100 lines should pass SURFACE-01").toBe("pass");
 });
 
-test("SURFACE-01: README over 300 lines warns", async () => {
-  await setupStandalone(tempDir, { readmeLines: 301 });
+test("SURFACE-01: README at exactly 600 lines passes", async () => {
+  await setupStandalone(tempDir, { readmeLines: 600 });
   const result = await runPublicSurfaceValidate({} as never, mockContext(tempDir));
   const checks = result.data?.checks ?? [];
   const surface01 = checks.find((c) => c.rule === "SURFACE-01");
-  expect(surface01?.status, "README with 301 lines should warn on SURFACE-01").toBe("warn");
+  expect(surface01?.status, "README with 600 lines should pass SURFACE-01").toBe("pass");
+});
+
+test("SURFACE-01: README over 600 lines warns", async () => {
+  await setupStandalone(tempDir, { readmeLines: 601 });
+  const result = await runPublicSurfaceValidate({} as never, mockContext(tempDir));
+  const checks = result.data?.checks ?? [];
+  const surface01 = checks.find((c) => c.rule === "SURFACE-01");
+  expect(surface01?.status, "README with 601 lines should warn on SURFACE-01").toBe("warn");
 });
 
 test("SURFACE-02: Node version match passes", async () => {

package/src/tests/scaffold-project.test.ts

@@ -60,7 +60,7 @@ test("forge.scaffold tolerates non-empty directory (RFC-0877 in-place mode)", as
   expect(result.data?.status).toBe("pass");
   expect(existsSync(join(tempDir, "package.json"))).toBe(true);
   expect(existsSync(join(tempDir, "some-file.txt"))).toBe(true);
-}, 30000);
+}, 120000);
 
 test("forge.scaffold fails on missing --profile", async () => {
   const result = await runScaffoldProject(