@moretea-labs/forge 1.8.1 → 1.8.2

package/scripts/check-mcp-compatibility.ts

@@ -1,47 +1,45 @@
 import { readFileSync } from 'fs';
 import { FORGE_TOOL_SURFACE, FORGE_VERSION, forgeToolSurfaceFingerprint } from '../src/cli/controller/runtime-config';
-import { injectAutomationEnvelopeFields } from '../adapters/mcp/runtime-gateway/automation-receipt-adapter';
 import { injectCognitionSettlementFields } from '../adapters/mcp/runtime-gateway/cognition-settlement';
-import { injectDurableCommandFields } from '../src/runtime/gateway/mcp/router';
-import { runtimePolicy } from '../src/cli/mcp/multi-repository';
-import { buildMcpToolDefinitions } from '../src/cli/mcp/tools';
-import { accessToolDefinitions } from '../src/cli/mcp/access-tools';
-import { repositoryToolDefinitions } from '../src/cli/mcp/repository-tools';
-import { runtimeToolDefinitions } from '../src/runtime/gateway/mcp/runtime-tools';
-import { executionToolDefinitions } from '../src/runtime/gateway/mcp/execution-tools';
-import { processToolDefinitions } from '../src/runtime/gateway/mcp/process-tools';
-import {
-  ADVANCED_CONTROLLER_TOOL_NAMES,
-  CORE_CONTROLLER_TOOL_NAMES,
-  DEFAULT_CONTROLLER_TOOL_NAMES,
-  PREFERRED_FACADE_TOOL_NAMES,
-  STABLE_CONTROLLER_TOOL_NAMES,
-} from '../src/cli/mcp/toolset';
+import { injectDurableCommandFields } from '../adapters/mcp/runtime-gateway/router';
+import { runtimePolicy } from '../adapters/mcp/multi-repository';
+import { buildMcpToolDefinitions } from '../adapters/mcp/tool-mapping/tools';
+import { controllerExposureSnapshot } from '../adapters/mcp/toolset';
+import type { MultiRepositoryMcpToolContext } from '../adapters/mcp/multi-repository';
+import { accessToolDefinitions } from '../adapters/mcp/tool-mapping/access-tools';
+import { repositoryToolDefinitions } from '../adapters/mcp/tool-mapping/repository-tools';
+import { runtimeToolDefinitions } from '../adapters/mcp/runtime-gateway/runtime-tools';
+import { executionToolDefinitions } from '../adapters/mcp/runtime-gateway/execution-tools';
+import { processToolDefinitions } from '../adapters/mcp/runtime-gateway/process-tools';
+import { PREFERRED_CONTROLLER_TOOL_NAMES, PREFERRED_FACADE_TOOL_NAMES } from '../adapters/mcp/toolset';
 
-// The served ChatGPT schema is intentionally much smaller than the exhaustive
-// compatibility catalog. Keep a tight budget so accidental tool additions are
-// caught before they become discovery latency and schema-cache churn.
-const MAX_DEFAULT_TOOL_COUNT = 24;
+// The preferred model-routing list remains small; actual tools/list always
+// exposes the complete authoritative registry, without exposure modes.
+const MAX_PREFERRED_TOOL_COUNT = 24;
+const EXPECTED_EXPOSED_NAME_FINGERPRINT = '10db066a55335555';
 
 // #197 decomposition authority: the public/frozen Tool Contract is an explicit
 // baseline, independent from whichever internal adapter currently defines a
 // tool. Adapter extraction may move implementation ownership without silently
 // changing names, descriptions, input schemas, or annotations.
-const EXPECTED_STABLE_CONTROLLER_TOOL_NAMES = [
+const EXPECTED_PREFERRED_CONTROLLER_TOOL_NAMES = [
   'rh_access', 'rh_status', 'rh_inbox', 'rh_context', 'rh_work', 'capability_execute',
-  'repository_list', 'repository_get', 'repository_register', 'repository_command_execute',
+  'repository_list', 'repository_get', 'repository_register', 'execute_command',
   'read_repository_file', 'repository_safe_patch_apply', 'run_check', 'plugin_action_execute',
-  'process_exec', 'process_get', 'process_wait', 'process_logs', 'process_cancel', 'result_read', 'result_search',
+  'process_get', 'process_wait', 'process_logs', 'process_cancel', 'result_read', 'result_search',
 ] as const;
 // The stable bounded surface now includes the generic typed capability executor
 // already shipped on main. V3 Work objective relations intentionally extend only
 // the existing rh_work input schema; both changes must invalidate stale Connector
 // schema snapshots without adding a parallel execution or graph authority.
-const EXPECTED_STABLE_TOOL_NAME_FINGERPRINT = 'b25da75c1f774e66';
+const EXPECTED_PREFERRED_TOOL_NAME_FINGERPRINT = 'f22e635a213b45fa';
 // Every public object-schema tool carries shared transport metadata only through
-// the final tools/list materialization path. Automation and Cognition settlement
-// revisions intentionally invalidate stale Connector schema snapshots.
-const EXPECTED_STABLE_TOOL_SCHEMA_FINGERPRINT = '643521037647c28a';
+// the final tools/list materialization path. Tool schema revisions
+// intentionally invalidate stale Connector schema snapshots.
+// 2026-10-15: rh_context gained model-authored search_terms in 5a0f8d839.
+// Removing only that property from the current frozen surface reproduces the
+// preceding 9bd588d157f66fdf fingerprint; the new surface is intentional.
+const EXPECTED_PREFERRED_TOOL_SCHEMA_FINGERPRINT = '793a6c1d07070dc2';
 
 const policy = runtimePolicy(process.cwd(), {
   profile: 'controller',
@@ -63,34 +61,19 @@ const sourceGroups = Object.fromEntries(
 const allDefinitions = Object.values(sourceDefinitionGroups).flat();
 const definitionByName = new Map(allDefinitions.map((tool) => [tool.name, tool]));
 const fullNames = [...new Set(Object.values(sourceGroups).flat())];
-const defaultNames: string[] = [...DEFAULT_CONTROLLER_TOOL_NAMES];
-const coreNames: string[] = [...CORE_CONTROLLER_TOOL_NAMES];
-const advancedNames: string[] = [...ADVANCED_CONTROLLER_TOOL_NAMES];
-const catalogNames: string[] = [...STABLE_CONTROLLER_TOOL_NAMES];
+const defaultNames: string[] = [...PREFERRED_CONTROLLER_TOOL_NAMES];
 const preferredNames: string[] = [...PREFERRED_FACADE_TOOL_NAMES];
 const defaultFingerprint = forgeToolSurfaceFingerprint(defaultNames);
 const stableDefinitions = defaultNames
   .map((name) => definitionByName.get(name)).filter((tool) => tool !== undefined)
-  .map((tool) => injectCognitionSettlementFields(injectAutomationEnvelopeFields(injectDurableCommandFields(tool))));
+  .map((tool) => injectCognitionSettlementFields(injectDurableCommandFields(tool)));
 const stableSchemaFingerprint = forgeToolSurfaceFingerprint(stableDefinitions);
-const catalogFingerprint = forgeToolSurfaceFingerprint(catalogNames);
 const fullFingerprint = forgeToolSurfaceFingerprint(fullNames);
 const duplicateDefault = defaultNames.filter((name, index) => defaultNames.indexOf(name) !== index);
 const missingDefault = defaultNames.filter((name) => !fullNames.includes(name));
-const missingCatalog = catalogNames.filter((name) => !fullNames.includes(name));
 const sourceCollisions = Object.entries(sourceGroups).flatMap(([group, names], groupIndex, entries) =>
   names.filter((name) => entries.slice(0, groupIndex).some(([, earlier]) => earlier.includes(name)))
     .map((name) => `${group}:${name}`));
-const currentToolNames = new Set([
-  ...sourceGroups.runtime,
-  ...sourceGroups.execution,
-  ...sourceGroups.process,
-  ...sourceGroups.access,
-  ...sourceGroups.repository,
-]);
-const legacyHandlerSource = readFileSync(new URL('../src/cli/mcp/legacy-tool-service.ts', import.meta.url), 'utf8');
-const legacyHandlerNames = [...legacyHandlerSource.matchAll(/case\s+["']([^"']+)["']\s*:/g)].map((match) => match[1]);
-const legacyHandlerCollisions = [...new Set(legacyHandlerNames.filter((name) => currentToolNames.has(name)))].sort();
 const recoveryMcpSource = readFileSync(new URL('../src/runtime/standalone-recovery/mcp-server.ts', import.meta.url), 'utf8');
 const recoveryStatelessMarkers = [
   'createMcpHandler',
@@ -116,37 +99,38 @@ for (const marker of recoverySessionAuthorityMarkers) {
     failures.push(`Standalone Recovery MCP must not own transport-session state: found ${marker}`);
   }
 }
-if (defaultNames.join('\n') !== EXPECTED_STABLE_CONTROLLER_TOOL_NAMES.join('\n')) {
-  failures.push(`stable ChatGPT Tool Contract names changed: ${defaultNames.join(', ')}`);
+if (defaultNames.join('\n') !== EXPECTED_PREFERRED_CONTROLLER_TOOL_NAMES.join('\n')) {
+  failures.push(`preferred-name schema contract names changed: ${defaultNames.join(', ')}`);
 }
-if (defaultFingerprint !== EXPECTED_STABLE_TOOL_NAME_FINGERPRINT) {
-  failures.push(`stable ChatGPT tool-name fingerprint changed: ${defaultFingerprint} != ${EXPECTED_STABLE_TOOL_NAME_FINGERPRINT}`);
+if (defaultFingerprint !== EXPECTED_PREFERRED_TOOL_NAME_FINGERPRINT) {
+  failures.push(`stable ChatGPT tool-name fingerprint changed: ${defaultFingerprint} != ${EXPECTED_PREFERRED_TOOL_NAME_FINGERPRINT}`);
 }
 if (stableDefinitions.length !== defaultNames.length) {
-  failures.push(`stable ChatGPT Tool Contract definitions are incomplete: ${stableDefinitions.length}/${defaultNames.length}`);
-} else if (stableSchemaFingerprint !== EXPECTED_STABLE_TOOL_SCHEMA_FINGERPRINT) {
-  failures.push(`stable ChatGPT Tool Contract schema fingerprint changed: ${stableSchemaFingerprint} != ${EXPECTED_STABLE_TOOL_SCHEMA_FINGERPRINT}`);
+  failures.push(`preferred-name schema contract definitions are incomplete: ${stableDefinitions.length}/${defaultNames.length}`);
+} else if (stableSchemaFingerprint !== EXPECTED_PREFERRED_TOOL_SCHEMA_FINGERPRINT) {
+  failures.push(`preferred-name schema contract schema fingerprint changed: ${stableSchemaFingerprint} != ${EXPECTED_PREFERRED_TOOL_SCHEMA_FINGERPRINT}`);
 }
-if (defaultNames.length > MAX_DEFAULT_TOOL_COUNT) {
-  failures.push(`default ChatGPT tools/list exceeds the schema budget: ${defaultNames.length} > ${MAX_DEFAULT_TOOL_COUNT}`);
+if (defaultNames.length > MAX_PREFERRED_TOOL_COUNT) {
+  failures.push(`preferred controller tool list exceeds its advisory size budget: ${defaultNames.length} > ${MAX_PREFERRED_TOOL_COUNT}`);
 }
-if (duplicateDefault.length) failures.push(`default duplicate names: ${[...new Set(duplicateDefault)].join(', ')}`);
-if (missingDefault.length) failures.push(`default tools missing from registered definitions: ${missingDefault.join(', ')}`);
-if (missingCatalog.length) failures.push(`compatibility catalog tools missing from registered definitions: ${missingCatalog.join(', ')}`);
+if (duplicateDefault.length) failures.push(`preferred-name duplicate names: ${[...new Set(duplicateDefault)].join(', ')}`);
+if (missingDefault.length) failures.push(`preferred tools missing from registered definitions: ${missingDefault.join(', ')}`);
 if (sourceCollisions.length) failures.push(`tool schema authority collisions: ${sourceCollisions.join(', ')}`);
-if (legacyHandlerCollisions.length) failures.push(`legacy execution authority collisions: ${legacyHandlerCollisions.join(', ')}`);
-if (coreNames.join('\n') !== defaultNames.join('\n')) {
-  failures.push('core surface must alias the bounded default ChatGPT surface');
-}
-if (advancedNames.join('\n') !== defaultNames.join('\n')) {
-  failures.push('advanced surface must alias the bounded default ChatGPT surface');
-}
 for (const name of ['rh_access', 'rh_status', 'rh_inbox', 'rh_context', 'rh_work']) {
   if (!preferredNames.includes(name)) failures.push(`preferred facade surface missing: ${name}`);
-  if (!defaultNames.includes(name)) failures.push(`default surface missing facade tool: ${name}`);
+  if (!defaultNames.includes(name)) failures.push(`preferred list missing facade tool: ${name}`);
 }
 if (fullNames.length < defaultNames.length) {
-  failures.push(`full compatibility surface is smaller than default surface: ${fullNames.length} < ${defaultNames.length}`);
+  failures.push('complete MCP registry is missing preferred capability names');
+}
+if (fullFingerprint !== EXPECTED_EXPOSED_NAME_FINGERPRINT) {
+  failures.push(`complete MCP tool-name fingerprint changed: ${fullFingerprint} != ${EXPECTED_EXPOSED_NAME_FINGERPRINT}`);
+}
+const projectedNames = controllerExposureSnapshot({
+  policy, toolset: 'full', controllerHome: process.cwd(), repoRoot: process.cwd(),
+} as MultiRepositoryMcpToolContext).actualToolNames.sort();
+if (projectedNames.join('\n') !== [...fullNames].sort().join('\n')) {
+  failures.push('controller_capabilities expectation differs from the complete MCP registry');
 }
 
 const rhWorkDefinition = runtimeToolDefinitions.find((tool) => tool.name === 'rh_work');
@@ -170,16 +154,13 @@ console.log(JSON.stringify({
   status: 'ok',
   toolSurface: FORGE_TOOL_SURFACE,
   version: FORGE_VERSION,
-  stableToolCount: defaultNames.length,
-  stableFingerprint: defaultFingerprint,
-  stableSchemaFingerprint,
-  defaultToolBudget: MAX_DEFAULT_TOOL_COUNT,
-  compatibilityCatalogToolCount: catalogNames.length,
-  compatibilityCatalogFingerprint: catalogFingerprint,
-  fullCompatibilityToolCount: fullNames.length,
-  fullCompatibilityFingerprint: fullFingerprint,
+  preferredToolCount: defaultNames.length,
+  preferredFingerprint: defaultFingerprint,
+  preferredSchemaFingerprint: stableSchemaFingerprint,
+  preferredToolBudget: MAX_PREFERRED_TOOL_COUNT,
+  exposedToolCount: fullNames.length,
+  exposedNameFingerprint: fullFingerprint,
   sourceToolCounts: Object.fromEntries(Object.entries(sourceGroups).map(([name, tools]) => [name, tools.length])),
   sourceCollisions: [...new Set(sourceCollisions)].sort(),
-  legacyHandlerCollisions,
   accessModeChangesSchema: false,
 }, null, 2));

package/scripts/check-package-identity.mjs

@@ -8,7 +8,6 @@ const scriptDir = dirname(fileURLToPath(import.meta.url));
 const root = resolve(scriptDir, "..");
 const pkg = JSON.parse(readFileSync(resolve(root, "package.json"), "utf8"));
 const lock = JSON.parse(readFileSync(resolve(root, "package-lock.json"), "utf8"));
-const skill = JSON.parse(readFileSync(resolve(root, "assets/skill-version.json"), "utf8"));
 
 const expectedName = "@moretea-labs/forge";
 const requiredFiles = [
@@ -62,10 +61,6 @@ function requireReusableTarballPublisher() {
 if (pkg.name !== expectedName) fail(`package.json name is ${pkg.name}`);
 const versionMatch = String(pkg.version ?? "").match(/^(\d+\.\d+\.\d+)(?:-rc\.(\d+))?$/);
 if (!versionMatch) fail(`package.json version is not a stable or rc semantic version: ${pkg.version}`);
-const coreVersion = versionMatch[1];
-if (coreVersion !== skill.version || coreVersion !== skill.templateVersion) {
-  fail(`package core version ${coreVersion} must match workflow versions ${skill.version}/${skill.templateVersion}`);
-}
 if (pkg.publishConfig?.access !== "public") fail("publishConfig.access must be public");
 if (pkg.publishConfig?.provenance !== true) fail("publishConfig.provenance must be true");
 if (pkg.publishConfig?.tag !== undefined) fail("publishConfig.tag must be omitted");