create-agdf 0.20.2 → 0.31.0

package/lib/runtime/control-command-service.js

@@ -1,12 +1,27 @@
-import { executeApprovalCommand } from "#agdf-core/control-state/approval-command.js";
-import { evaluateGateCheck } from "#agdf-core/control-evaluation/gate-check.js";
-import { cliGitObservation } from "../control-evaluation/git-observation.js";
-import { pluginDefinition } from "./control-context.js";
+import { executeApprovalCommand } from '#agdf-core/control-state/approval-command.js';
+import { validateApprovalCommand, resolveControlCommandTarget } from '#agdf-core/control-state/approval-command-contract.js';
+import { createCoreServices } from '#agdf-core';
+import { evaluateGateCheck } from '#agdf-core/control-evaluation/gate-check.js';
+import { cliGitObservation } from '../control-evaluation/git-observation.js';
+import { pluginDefinition, resources } from './control-context.js';
 
 const evaluateLocalGate = (root, selection) => evaluateGateCheck(root, selection, cliGitObservation);
+const invalidResult = (root, command) => executeApprovalCommand(root, command, {
+  evaluateGateCheck: evaluateLocalGate, packageVersion: pluginDefinition.version,
+});
 
 export function recordGateApprovalCommand(root, command) {
-  return executeApprovalCommand(root, command, {
-    evaluateGateCheck: evaluateLocalGate, packageVersion: pluginDefinition.version,
-  });
+  // Preserve the public action's strict descriptor validation before copying
+  // any field. Valid calls share CLI/MCP normalization and original UUID proof.
+  if (validateApprovalCommand(command)) return invalidResult(root, command);
+  let target;
+  try { target = resolveControlCommandTarget(root); } catch { return invalidResult(root, command); }
+  const input = { working_directory: process.cwd(), primary_target: target.root, target_source: 'explicit_target',
+    target_id: command.target_id, run_id: command.run_id, expected_revision_id: command.expected_revision_id,
+    operation_id: command.operation_id, assurance: command.assurance,
+    command: { kind: 'approval.record', payload: { gate: command.gate, presentation_id: command.presentation_id, response: command.response } } };
+  const result = createCoreServices({ resources, observers: { gitObservation: cliGitObservation } }).command('agdf_approve', input);
+  // The action facade keeps its existing result observations. The canonical
+  // command owner supplies its current receipt and replay, never a second writer.
+  return result.family_result?.operation_id === command.operation_id ? result.family_result : result;
 }

package/lib/runtime/control-context.js

@@ -5,7 +5,14 @@ const generatedURL = new URL("../../generated/", import.meta.url);
 const contractsURL = coreResources.packageRoot === fileURLToPath(packageURL)
   ? pathToFileURL(`${coreResources.contractsRoot}/`)
   : new URL("plugins/agdf/meta/contracts/", generatedURL);
-export const resources = createResourceContext({ packageURL, generatedURL, contractsURL });
+// Projected packages bind Core and CLI to the same immutable resource files.
+// Reuse that already-validated data instead of parsing/freezing it twice. Source
+// compositions with different roots still construct their own bound context;
+// CLI's existing null template-root contract and fresh runtime digests remain.
+const sameResources = coreResources.packageRoot === fileURLToPath(packageURL)
+  && coreResources.generatedRoot === fileURLToPath(generatedURL);
+export const resources = sameResources ? Object.freeze({ ...coreResources, templatesRoot: null })
+  : createResourceContext({ packageURL, generatedURL, contractsURL });
 export const { packageRoot, generatedRoot, pluginDefinition, interactionLocales } = resources;
 export const configuredLanguage = value => configured(value, resources);
 export const resolveConfiguredChatLanguage = target => resolveLanguage(target, resources);

package/lib/runtime/validator-application.js

@@ -4,7 +4,7 @@ import { validateCommandOptions, commandGroupRoutes } from "../cli/command-regis
 import { CliUsageError, parseArgs } from "../cli/parse-args.js";
 import { pluginDefinition } from "../cli/runtime-context.js";
 
-const validatorCommands = new Set(["target-check", "skill-dispatch", "doctor", "gate-check", "delivery-map", "delivery-path-search", "contract", "control-maintenance", "run-create", "run-present", "run-update", "run-revise", "run-step", "run-approve", ...Object.keys(commandGroupRoutes)]);
+const validatorCommands = new Set(["project-state", "operation-status", "run-view", "target-check", "skill-dispatch", "doctor", "gate-check", "delivery-map", "inventory", "delivery-path-search", "contract", "control-maintenance", "run-create", "run-present", "run-revise", "run-step", "run-approve", "run-close", ...Object.keys(commandGroupRoutes)]);
 
 export async function runValidatorCli(argv = process.argv.slice(2), adapters = {}) {
   const io = adapters.io ?? console;
@@ -22,7 +22,7 @@ export async function runValidatorCli(argv = process.argv.slice(2), adapters = {
     return error.exitCode;
   }
   if (parsed.kind === "help") {
-    io.log("AGDF surface-local validator: target-check | skill-dispatch | doctor | gate-check | delivery-map | delivery-path-search | contract | control-maintenance | run-create | run-present | run-update | run-revise | run-step | run-approve");
+    io.log("AGDF surface-local validator: project-state | operation-status | run-view | target-check | skill-dispatch | doctor | gate-check | delivery-map | inventory | delivery-path-search | contract | control-maintenance | run-create | run-present | run-revise | run-step | run-approve | run-close");
     return 0;
   }
   let options;