@akira-tl/forgerelay 1.1.2 → 1.2.0

package/scripts/ci/architecture.mjs

@@ -9,11 +9,12 @@ const MAX_DIRECT_FILES = 8;
 const MAX_DIRECT_DIRS = 8;
 const LINE_LIMIT_EXTENSIONS = new Set([".ts", ".tsx", ".js", ".mjs", ".cjs", ".css"]);
 
-// Append-only versioned archives grow by design and must keep their canonical
-// flat paths for release tooling and stable links. They remain subject to all
-// other architecture checks, including code line limits where applicable.
+// Versioned public contract directories grow by design and must keep their
+// canonical flat paths for release tooling, schema URLs, and stable links. They
+// remain subject to all other architecture checks, including code line limits.
 const DIRECT_FILE_LIMIT_EXEMPT_DIRS = new Set([
   "docs/releases",
+  "schemas/v1",
 ]);
 
 // Repository-root protocol files are intentionally discoverable by Git, npm,
@@ -70,6 +71,8 @@ for (const allowed of ROOT_PROTOCOL_FILES) {
   // Presence is allowed, not required; this loop documents the intentional set.
 }
 
+checkConfigV2Contraction(violations);
+
 const directories = new Set([...directFiles.keys(), ...directDirs.keys()]);
 for (const directory of [...directories].sort()) {
   const files = directFiles.get(directory) ?? 0;
@@ -77,7 +80,7 @@ for (const directory of [...directories].sort()) {
   if (directory !== "." && !DIRECT_FILE_LIMIT_EXEMPT_DIRS.has(directory) && files > MAX_DIRECT_FILES) {
     violations.push(`${directory}: ${files} direct files > ${MAX_DIRECT_FILES}`);
   }
-  if (dirs > MAX_DIRECT_DIRS) {
+  if (directory !== "." && dirs > MAX_DIRECT_DIRS) {
     violations.push(`${directory}: ${dirs} direct directories > ${MAX_DIRECT_DIRS}`);
   }
 }
@@ -105,6 +108,53 @@ function gitTrackedFiles() {
   return result.stdout.split("\0").filter(Boolean);
 }
 
+function checkConfigV2Contraction(violations) {
+  const rules = [
+    {
+      path: "src/runtime/config/config.ts",
+      forbidden: [
+        /\bconfig\.(?:mcpServers|languageServers|hooks)\b/,
+        /\b(?:mcpServers|languageServers|hooks):\s*(?:ExternalMcpServersConfig|LanguageServerConfigInput|HookConfig)\b/,
+        /\b(?:parseExternalMcpServers|parseHookConfig|mergeHookConfigs)\b/,
+        /\bfiles\.hooks\b/,
+      ],
+      reason: "ServerConfig must not materialize migrated domain snapshots outside Config System v2",
+    },
+    {
+      path: "src/cli/config/inspect.ts",
+      forbidden: [/\blegacyServers\s*:/, /\blegacyUser\s*:/, /\bparseHookConfig\b/, /\bmergeHookConfigs\b/],
+      reason: "config inspection must consume unified domain resolvers rather than rebuild legacy precedence",
+    },
+    {
+      path: "src/mcp/hooks/hook-cli.ts",
+      forbidden: [/\bloadForgeRelayFiles\b/, /\bparseHookConfig\b/, /\bmergeHookConfigs\b/],
+      reason: "Hook CLI must consume the unified Hook resolver rather than load legacy sources directly",
+    },
+    {
+      path: "src/lsp/runtime/manager.ts",
+      forbidden: [/\bglobalConfig\s*:/, /\bconfig\.languageServers\b/],
+      reason: "Language Server runtime must resolve configuration on demand through Config System v2",
+    },
+    {
+      path: "src/mcp/operations/external-mcp/external-mcp-runtime.ts",
+      forbidden: [/\blegacyServers\s*:/, /\bconfig\.mcpServers\b/],
+      reason: "External MCP runtime must resolve configuration on demand through Config System v2",
+    },
+    {
+      path: "src/runtime/config/inspection/live.ts",
+      forbidden: [/\blegacyServers\s*:/, /\blegacyUser\s*:/, /\bconfig\.(?:mcpServers|languageServers|hooks)\b/],
+      reason: "live diagnostics must consume unified domain resolvers rather than startup snapshots",
+    },
+  ];
+
+  for (const rule of rules) {
+    const text = readFileSync(rule.path, "utf8");
+    for (const pattern of rule.forbidden) {
+      if (pattern.test(text)) violations.push(`${rule.path}: ${rule.reason} (${pattern})`);
+    }
+  }
+}
+
 function normalizeDirectory(directory) {
   return directory === "" ? "." : directory.replaceAll("\\", "/");
 }

package/scripts/ci/cmd-acceptance.mjs

@@ -6,6 +6,7 @@ import { mkdtemp, mkdir, readFile, rm, writeFile } from "node:fs/promises";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
 import { spawnSync } from "node:child_process";
+import { resolveAcceptancePrefix, resolveAcceptanceTarball } from "./gates/acceptance-artifact.mjs";
 
 if (process.platform !== "win32") {
   console.log("cmd.exe packaged acceptance skipped outside Windows.");
@@ -181,7 +182,8 @@ async function exercisePtyLifecycle(runtime) {
 
 async function exercisePackagedCmdShim(cmd) {
   const artifactDir = join(root, "artifact");
-  const prefix = join(root, "prefix");
+  const sharedPrefix = resolveAcceptancePrefix(process.cwd());
+  const prefix = sharedPrefix ?? join(root, "prefix");
   const configDir = join(root, "config");
   const stateDir = join(root, "state");
   await Promise.all([
@@ -191,14 +193,12 @@ async function exercisePackagedCmdShim(cmd) {
     mkdir(stateDir, { recursive: true }),
   ]);
 
-  const packed = runNpm(["pack", "--json", "--pack-destination", artifactDir]);
-  const packResult = JSON.parse(packed.stdout);
-  const filename = packResult?.[0]?.filename;
-  if (!filename) throw new Error(`npm pack did not report a package filename: ${packed.stdout}`);
-  const tarball = join(artifactDir, filename);
-  assert.ok(existsSync(tarball), `packed artifact is missing: ${tarball}`);
-
-  runNpm(["install", "--global", "--prefix", prefix, tarball]);
+  if (!sharedPrefix) {
+    const tarball = resolveAcceptanceTarball({ repoRoot: process.cwd(), artifactDir, npmCli });
+    runNpm(["install", "--global", "--prefix", prefix, tarball]);
+  } else {
+    console.log(`Using prepared acceptance install: ${prefix}`);
+  }
   const shim = join(prefix, "forgerelay.cmd");
   assert.ok(existsSync(shim), `npm did not create the cmd launcher shim: ${shim}`);