create-agdf 0.20.1 → 0.30.0

package/generated/plugins/copilot/agdf/runtime/create-agdf/runtime/core/lib/skill-dispatch/delivery-run-assignment.js

@@ -1,16 +1,16 @@
-import { discoverRuns } from "../control-state/run-state-reader.js";
+import { discoverRuns, readSelectedRun } from "../control-state/run-state-reader.js";
 import { assertCanonicalRunStore } from "../control-state/run-store-inspection.js";
 import { artefactFileDigest, runSealState } from "../control-state/run-seal.js";
 import { containedRegularFile } from "../control-state/contained-file.js";
-import { duplicateArtefactRowTypes, parseControlState } from "../control-state/run-state-parser.js";
-import { closeoutArtefacts, internalStepArtefacts, readArtefactHeading, userGateOrder } from "../control-evaluation/run-state.js";
+import { duplicateArtefactRowTypes } from "../control-state/run-state-parser.js";
+import { prepareRunContentState, readArtefactHeading } from "../control-evaluation/run-state.js";
 import { buildRunCandidates } from "../interaction-presentation.js";
 
 // Scope matching belongs to the coding agent. This leaf supplies canonical evidence only;
 // neither candidate count, recency nor AGDF_RUN_ID binds a delivery request to a run.
-export function readDeliveryRunInventory(targetDir) {
+export function readDeliveryRunInventory(targetDir, { runId } = {}) {
   assertCanonicalRunStore(targetDir);
-  const runs = discoverRuns(targetDir);
+  const runs = runId ? [readSelectedRun(targetDir, runId)] : discoverRuns(targetDir);
   const findings = runs.filter((run) => !run.valid)
     .flatMap((run) => run.findings.map((finding) => ({ ...finding, run_id: run.run_id })));
   const candidates = [];
@@ -24,11 +24,7 @@ export function readDeliveryRunInventory(targetDir) {
       findings.push({ code: "AGDF_RUN_ASSIGNMENT_SEAL_INVALID", run_id: run.run_id, path: run.path, seal_status: seal.status });
       continue;
     }
-    const state = parseControlState(run.content, {
-      userGates: userGateOrder,
-      internalSteps: [...internalStepArtefacts],
-      closeoutArtefacts: [...closeoutArtefacts],
-    });
+    const state = prepareRunContentState(targetDir, run.content, run.path);
     const ur = state.artefacts.get("UR");
     const urPath = ur?.path || null;
     if ((!urPath && (state.current_gate !== "UR" || state.approvals.get("UR")?.status === "approved"))

package/generated/plugins/copilot/agdf/runtime/create-agdf/runtime/core/lib/skill-dispatch/mcp-continuation.js

@@ -0,0 +1,64 @@
+import { readRunState } from '../control-evaluation/run-state.js';
+import { inspectArtefactRecordingMode } from '../control-state/run-artefact-recording.js';
+import { resolveControlCommandTarget } from '../control-state/approval-command-contract.js';
+
+// Transport guidance over existing routes; code, tests and Git remain external.
+export function mcpControlContinuation(input, result, availableGroups = []) {
+  // A new Run over MCP is created by agdf_run/run.create; point at its contract instead of a retry.
+  if (result.terminal && result.diagnostics?.[0]?.code === 'AGDF_NEW_RUN_OPERATION_REQUIRED' && result.recovery
+      && availableGroups.includes('agdf_run') && result.target?.governance_target)
+    return { ...result, recovery: { ...result.recovery, next_calls: [{ tool: 'agdf_inspect', arguments: { operation: 'operation-contract',
+      module: 'agdf_run/run.create', presentation_language: input.presentationLanguage, working_directory: input.workingDirectory,
+      primary_target: result.target.governance_target, target_source: input.targetSource } }] } };
+  if (result.terminal || !result.continuation || !result.target?.governance_target) return result;
+  const continuation = result.continuation, root = result.target.governance_target;
+  const run = continuation.run_id ?? result.control?.run_id, revision = continuation.revision_id ?? result.control?.revision_id;
+  const targetId = resolveControlCommandTarget(root).target_id;
+  const common = { presentation_language: input.presentationLanguage, working_directory: input.workingDirectory,
+    primary_target: root, target_source: input.targetSource };
+  const bound = { ...common, run_id: run, expected_revision_id: revision };
+  const inspect = (operation, extra = {}, bind = true) => ({ tool: 'agdf_inspect', arguments: { operation, ...(bind ? bound : common), ...extra } });
+  const { steps, template_path, artifact_path, ...retained } = continuation;
+  let recording;
+  if (['prd_definition', 'sd_definition', 'tp_definition'].includes(continuation.phase))
+    recording = inspectArtefactRecordingMode(readRunState(root, { runId: run, ignoreRunIdEnv: true }), result.control.current_gate);
+  let nextCalls, requiredGroups = [], instruction = continuation.instruction ?? '';
+  if (continuation.phase === 'planned_ur_definition') {
+    requiredGroups = ['agdf_record'];
+    nextCalls = continuation.next_calls;
+    instruction += ' Use planned_material.stage and the planning scope; do not substitute Run material or a shell writer.';
+  } else if (continuation.phase === 'run_missing' || continuation.phase === 'resolve_delivery_run') {
+    requiredGroups = ['agdf_run'];
+    nextCalls = [inspect('operation-contract', { module: 'agdf_run/run.create' }, false)];
+    instruction = continuation.phase === 'resolve_delivery_run'
+      ? instruction.replace('choose an unused run_id and redispatch with intake: true and intake_mode: new', 'call agdf_run/run.create with a new original operation UUID and semantic title; resume its returned Run/revision')
+      : 'Create independent work via agdf_run/run.create with the requested semantic title and one original UUID. Resume returned Run/revision. No validator or user ID question.';
+  } else if (run && revision) {
+    const phase = continuation.phase;
+    const kind = phase === 'presentation_required' ? 'presentation.prepare'
+      : phase === 'compact_closeout' || phase === 'closeout' ? 'closeout.record'
+      : phase === 'ur_definition' && continuation.draft_registered ? 'ur_draft.replace'
+      : ['prd_definition', 'sd_definition', 'tp_definition', 'required_gate_artifact'].includes(phase) ? 'artefact.record'
+      : phase === 'verified_change_record' || phase === 'compact_implementation' && continuation.mode === 'verified_change' ? 'verified_change.record' : 'step.record';
+    const group = kind === 'presentation.prepare' ? 'agdf_prepare' : kind === 'closeout.record' ? 'agdf_closeout'
+      : kind === 'ur_draft.replace' ? 'agdf_revise' : 'agdf_record';
+    requiredGroups = kind === 'presentation.prepare' ? [group] : [group, 'agdf_record'];
+    nextCalls = [inspect('authoring-inputs'), inspect('operation-contract', { module: `${group}/${kind}` })];
+    instruction = phase === 'presentation_required'
+      ? 'Prepare this gate via agdf_prepare/presentation.prepare, then presentation.deliver: default markdown, optional non-primary decision-view. Forward selected text once. Wait for a NEW deliberate response; earlier replies cannot approve it.'
+      : `${instruction} Typed MCP: read every authoring-inputs page for current templates/sources. Stage UTF-8 via agdf_record/material.stage: begin, parts <=16 KiB, seal. Use returned references with ${group}/${kind}; operation-contract supplies fields. Attest mappings only after source review. All four modes: control uses MCP; editing/tests/Git stay external. Redispatch this Run/returned revision. Use the inspected current control operation when these groups are available.`;
+    if (continuation.mode === 'verified_change') instruction = instruction.replace(/through the existing record\/run-update owner/gu, 'through verified_change.record phase execution')
+      .replace(/Complete the linked Verified Change Mini-Closeout.*?Git-aware CLI evaluation\./u, 'Stage the linked Verified Change Mini-Closeout and use closeout.record with its exact confirmation; the Core checks actual Git observations.');
+  }
+  const missing = [...new Set(requiredGroups)].filter(group => !availableGroups.includes(group));
+  if (missing.length && continuation.phase === 'planned_ur_definition') return { ...result,
+    continuation: { ...continuation, control_transport: 'unavailable', missing_mcp_groups: missing,
+      instruction: 'Planning Core capability is unavailable in this host. Enable the compatible AGDF Core record group before any mutation. No shell-writing or Run-intake fallback.' } };
+  if (missing.length) return { ...result, presentation: continuation.phase === 'presentation_required' ? null : result.presentation,
+    continuation: { ...continuation, control_transport: 'cli_fallback', missing_mcp_groups: missing,
+      transport_reason: 'required_mcp_group_unavailable' } };
+  return { ...result, presentation: continuation.phase === 'presentation_required' ? null : result.presentation,
+    continuation: { ...retained, instruction, control_transport: 'typed_mcp', external_execution: ['source edits', 'actual tests', 'Git operations'],
+      ...(recording ? { recording, instruction: instruction + ' ' + recording.instruction } : {}),
+      ...(nextCalls ? { next_calls: nextCalls } : {}) } };
+}

package/generated/plugins/copilot/agdf/runtime/create-agdf/runtime/core/lib/skill-dispatch/prd-definition.js

@@ -1,12 +1,13 @@
+import { runDocumentPath } from '../control-state/run-identity.js';
 import { extractField } from "../control-evaluation/verified-change.js";
 import { ARTEFACT_TEMPLATE_IDS, resolveArtefactTemplatePath } from "../resources/artefact-templates.js";
 import { resolveArtifactPresentationLanguages } from "../resources/context.js";
 
 // Pure routing over already evaluated control and contained source facts; never a writer or gate.
-export function prdDefinitionPhase(targetDir, control, input, sources) {
+export function prdDefinitionEligible(control, input) {
   const state = control?.status_card?.runState;
   if (control?.next_operation?.type === "reassess_source_analysis") return null;
-  if (input.ur_action !== undefined || input.sd_action !== undefined || input.tp_action !== undefined || !input.run_id || control?.status_card?.run_id !== input.run_id || !state || !sources
+  if (input.ur_action !== undefined || input.sd_action !== undefined || input.tp_action !== undefined || !input.run_id || control?.status_card?.run_id !== input.run_id || !state
       || control.current_gate !== "PRD" || !["open", "blocked"].includes(control.status)
       || !["none", "Approval: PRD"].includes(control.missing_approval)
       || !["none", "AGDF_PRD_DECISIONS_OPEN"].includes(control.blocking_reason)
@@ -17,15 +18,23 @@ export function prdDefinitionPhase(targetDir, control, input, sources) {
       || !["done", "not_applicable"].includes(state.artefacts.get("Brownfield Review")?.status)
       || !["structured_delivery", "structured_slice"].includes(state.mode_slice_decision?.decision)) return null;
   if (input.skill_id !== "prd-definition" && !(input.intake || input.continue_delivery)) return null;
-  const revisionId = extractField(state.content ?? "", "revision_id");
-  if (!revisionId) return null;
-  const artifactPath = `.agdf/control/artefacts/${input.run_id}/PRD.md`;
+  const artifactPath = `${runDocumentPath(input.run_id, '')}/PRD.md`;
   const artefact = state.artefacts.get("PRD");
   const registered = artefact?.status === "draft";
   if (artefact?.status === "approved" || (registered
       && String(artefact.path ?? "").replace(/^`|`$/gu, "") !== artifactPath)) return null;
   if (registered && input.skill_id !== "prd-definition" && input.prd_action !== "revise"
       && control.prd_readiness?.ready !== false) return null;
+  return true;
+}
+
+export function prdDefinitionPhase(targetDir, control, input, sources) {
+  if (!sources || !prdDefinitionEligible(control, input)) return null;
+  const state = control.status_card.runState;
+  const revisionId = extractField(state.content ?? "", "revision_id");
+  if (!revisionId) return null;
+  const artifactPath = `${runDocumentPath(input.run_id, '')}/PRD.md`;
+  const registered = state.artefacts.get("PRD")?.status === "draft";
   return Object.freeze({
     phase: "prd_definition", skill_id: "prd-definition", governance_target: targetDir,
     run_id: input.run_id, revision_id: revisionId,

package/generated/plugins/copilot/agdf/runtime/create-agdf/runtime/core/lib/skill-dispatch/sd-definition.js

@@ -1,3 +1,4 @@
+import { runDocumentPath } from '../control-state/run-identity.js';
 import { extractField } from "../control-evaluation/verified-change.js";
 import { ARTEFACT_TEMPLATE_IDS, resolveArtefactTemplatePath } from "../resources/artefact-templates.js";
 import { resolveArtifactPresentationLanguages } from "../resources/context.js";
@@ -20,7 +21,7 @@ export function sdDefinitionPhase(targetDir, control, input, sources) {
   if (input.skill_id !== "sd-definition" && !(input.intake || input.continue_delivery)) return null;
   const revisionId = extractField(state.content ?? "", "revision_id");
   if (!revisionId) return null;
-  const artifactPath = `.agdf/control/artefacts/${input.run_id}/SD.md`;
+  const artifactPath = `${runDocumentPath(input.run_id, '')}/SD.md`;
   const artefact = state.artefacts.get("SD");
   const registered = artefact?.status === "draft";
   if (artefact?.status === "approved" || (registered

package/generated/plugins/copilot/agdf/runtime/create-agdf/runtime/core/lib/skill-dispatch/service.js

@@ -1,7 +1,9 @@
+import { runDocumentPath } from '../control-state/run-identity.js';
 import { inspectDefinitionSources } from "../control-evaluation/definition-sources.js";
 import { prdDefinitionPhase } from "./prd-definition.js";
 import { sdDefinitionPhase } from "./sd-definition.js";
 import { tpDefinitionPhase } from "./tp-definition.js";
+import { inspectAuthoringInputs } from '../control-inspect/authoring-inputs.js';
 import { compactContinuation } from "./compact-continuation.js";
 import process from "node:process";
 import { extractField } from "../control-evaluation/verified-change.js";
@@ -47,21 +49,28 @@ function judgementTemplateId(skillId, currentGate) {
 }
 
 class SkillDispatchRuntimeError extends Error {
-  constructor(code, details = "") {
+  constructor(code, details = "", cause = null) {
     super(code);
     this.name = "SkillDispatchRuntimeError";
     this.code = code;
     this.details = details;
+    this.cause_code = cause;
   }
 }
 
+// Only a bare AGDF finding code is safe to surface; free-form messages may carry paths or content.
+const AGDF_CAUSE_PATTERN = /^AGDF_[A-Z0-9_]+$/u;
+const agdfCause = (error) => [error?.code, error?.message].find((value) => typeof value === "string" && AGDF_CAUSE_PATTERN.test(value)) ?? null;
+const blockingFindingCodes = (control) => [...new Set((control?.doctor_report?.findings ?? [])
+  .filter((finding) => finding.severity === "block").map((finding) => finding.code).filter(Boolean))];
+
 const presentationRecovery = (control) => String(control?.presentation_diagnostics?.approval_presentation_recovery ?? "").trim();
 
 function runDispatchStage(code, callback) {
   try {
     return callback();
-  } catch {
-    throw new SkillDispatchRuntimeError(code);
+  } catch (error) {
+    throw new SkillDispatchRuntimeError(code, "", agdfCause(error));
   }
 }
 
@@ -207,14 +216,14 @@ function controlSnapshot(report, { includeCandidateRuns = false } = {}) {
   });
 }
 
-export function createSkillDispatchFailure({ skill, runtime, timing, code, details, interactionLocales,
+export function createSkillDispatchFailure({ skill, runtime, timing, code, details, cause, interactionLocales,
   presentationLanguage, renderRecovery = renderSkillDispatchRecovery }) {
   const result = baseResult({ outcome: "evaluator_error", terminal: true, skill, runtime, timing });
   let action;
   try { action = renderRecovery({ code }, { registry: interactionLocales, requestedLocale: presentationLanguage }); }
   catch { action = null; }
   result.recovery = { action: [action ?? "Repair the installed locale registry and retry once.", details].filter(Boolean).join("\n\n") };
-  result.diagnostics = [{ code: artefactTemplateErrors.has(code) ? code : `dispatch_${code}` }];
+  result.diagnostics = [{ code: artefactTemplateErrors.has(code) ? code : `dispatch_${code}`, ...(cause ? { cause } : {}) }];
   return bindHostAction(result);
 }
 
@@ -287,6 +296,26 @@ export function createSkillDispatchService(dependencies = {}) {
       }
 
       const controlStarted = now();
+      if (input.planning) {
+        validateControlReadBoundary?.(target.governance_target);
+        const source = inspectAuthoringInputs(target.governance_target, input);
+        if (!source.available) throw Error(source.reason);
+        const result = baseResult({ outcome: 'skill_continuation', terminal: false, skill, runtime, timing });
+        result.target = target;
+        result.control = { scope: 'planning', planning: input.planning, current_gate: null, authorizes: false };
+        result.continuation = { phase: 'planned_ur_definition', skill_id: 'ur-definition', governance_target: target.governance_target,
+          planning: input.planning, ...resolveArtifactPresentationLanguages(target.governance_target, input.presentation_language),
+          ...(dependencies.readSkillRuntimeContracts ? { runtime_contracts: dependencies.readSkillRuntimeContracts('ur-definition') } : {}),
+          instruction: 'Use the shared ur-definition semantics in planning mode only. Read all current authoring-inputs pages and the exact planned writer contract. Capture or refine one unapproved UR and minimal metadata; incomplete drafts are permitted. Stage only planning-scoped UR material. No Run, PRD, SD, TP, gate presentation or approval is created. Return the saved source and remaining clarification. Start delivery only on a later explicit source-bound handoff request.',
+          next_calls: [{ tool: 'agdf_inspect', arguments: { operation: 'authoring-inputs', working_directory: input.working_directory,
+            primary_target: target.governance_target, target_source: input.target_source, presentation_language: input.presentation_language, planning: input.planning } },
+          { tool: 'agdf_inspect', arguments: { operation: 'operation-contract', module: `agdf_record/${source.recording.kind}`,
+            working_directory: input.working_directory, primary_target: target.governance_target, target_source: input.target_source, presentation_language: input.presentation_language } }],
+        };
+        timing.control_ms = round(milliseconds(controlStarted, now())); timing.total_ms = round(milliseconds(started, now()));
+        timing.wrapper_ms = round(wrapperMilliseconds(now, env));
+        return bindHostAction(result);
+      }
       const assignmentResult = (candidates, reason) => {
         const result = baseResult({ outcome: "intake_continuation", terminal: false, skill, runtime, timing });
         result.target = target;
@@ -300,7 +329,7 @@ export function createSkillDispatchService(dependencies = {}) {
         let candidates;
         try {
           validateControlReadBoundary?.(target.governance_target);
-          candidates = readRunInventory(target.governance_target);
+          candidates = readRunInventory(target.governance_target, input.run_id ? { runId: input.run_id } : {});
         } catch (error) {
           const code = error.code === "AGDF_CANONICAL_SCAFFOLD_REQUIRED" ? "intake_scaffold_required"
             : error.code === "AGDF_RUN_ASSIGNMENT_INVENTORY_INVALID" ? "run_assignment_inventory_invalid" : null;
@@ -341,15 +370,20 @@ export function createSkillDispatchService(dependencies = {}) {
         : null;
       } catch (error) {
         const code = error.message === "AGDF_RUN_COLLISION" ? "AGDF_RUN_COLLISION"
-          : error.code === "AGDF_CANONICAL_SCAFFOLD_REQUIRED" ? "AGDF_CANONICAL_SCAFFOLD_REQUIRED" : null;
-        // Only the two known intake refusals are reported as such; anything else keeps its real class.
-        if (!code) throw new SkillDispatchRuntimeError(DISPATCH_RECOVERY.control_evaluation_failed);
+          : error.code === "AGDF_CANONICAL_SCAFFOLD_REQUIRED" ? "AGDF_CANONICAL_SCAFFOLD_REQUIRED"
+            : error.message === "AGDF_NEW_RUN_OPERATION_REQUIRED" ? "AGDF_NEW_RUN_OPERATION_REQUIRED" : null;
+        // Only the known intake refusals are reported as such; anything else keeps its real class.
+        if (!code) throw new SkillDispatchRuntimeError(DISPATCH_RECOVERY.control_evaluation_failed, "", agdfCause(error));
         const result = baseResult({ outcome: "control_result", terminal: true, skill, runtime, timing });
         result.target = target;
         result.control = controlSnapshot(control);
-        result.diagnostics = [{ code }];
+        // The expected run-selection blockers explain why doctor reports the target as blocked.
+        const blockers = blockingFindingCodes(control);
+        result.diagnostics = [{ code, ...(blockers.length ? { blockers } : {}),
+          ...(code === "AGDF_NEW_RUN_OPERATION_REQUIRED" ? { run_id: input.run_id, expected_run_id: "run-<uuid>" } : {}) }];
         result.recovery = { action: renderRecovery({ code: code === "AGDF_RUN_COLLISION" ? DISPATCH_RECOVERY.intake_run_collision
-          : DISPATCH_RECOVERY.intake_scaffold_required }, { registry: rawInput.interactionLocales, requestedLocale: input.presentation_language }) };
+          : code === "AGDF_NEW_RUN_OPERATION_REQUIRED" ? DISPATCH_RECOVERY.intake_run_id_invalid
+            : DISPATCH_RECOVERY.intake_scaffold_required }, { registry: rawInput.interactionLocales, requestedLocale: input.presentation_language }) };
         return bindHostAction(result);
       }
       const urPhase = input.prd_action || input.sd_action || input.tp_action ? null : urDefinitionPhase(target.governance_target, control, input);
@@ -638,7 +672,7 @@ export function createSkillDispatchService(dependencies = {}) {
           run_id: result.control.run_id,
           revision_id: result.control.revision_id,
           presentation_language: input.presentation_language,
-          artifact_path: `.agdf/control/artefacts/${result.control.run_id}/VERIFIED_CHANGE.md`,
+          artifact_path: `${runDocumentPath(result.control.run_id, '')}/VERIFIED_CHANGE.md`,
           template_path: resolveArtefactTemplatePath(ARTEFACT_TEMPLATE_IDS.VERIFIED_CHANGE),
           instruction: "Create or repair the compact Verified Change record only through its existing gate-check/evaluator owner. Use the supplied plugin-owned template and canonical run-local artifact path, capture the required baseline, and prove every eligibility condition before implementation. Redispatch gate-check for the same run; never add another skill or route.",
           ...(dependencies.readSkillRuntimeContracts ? {
@@ -686,7 +720,7 @@ export function createSkillDispatchService(dependencies = {}) {
           result.continuation = Object.freeze({ phase: 'required_gate_artifact', skill_id: owner,
             governance_target: target.governance_target, run_id: result.control.run_id, revision_id: result.control.revision_id,
             presentation_language: input.presentation_language,
-            ...(!orientationOnly && unapprovedQa ? { artifact_path: `.agdf/control/artefacts/${result.control.run_id}/QA_REPORT.md`,
+            ...(!orientationOnly && unapprovedQa ? { artifact_path: `${runDocumentPath(result.control.run_id, '')}/QA_REPORT.md`,
               template_path: resolveArtefactTemplatePath(ARTEFACT_TEMPLATE_IDS.QA_REPORT) } : {}),
             instruction: `${control.presentation_diagnostics.approval_presentation_recovery}\nComplete only this bound Run's permitted unapproved content through its existing writer; preserve approved intent and all protected sources. Record the current revision and redispatch the same Run once. An unchanged failure stops; no earlier reply is approval.`,
             ...(dependencies.readSkillRuntimeContracts ? { runtime_contracts: runDispatchStage(DISPATCH_RECOVERY.runtime_contracts_unavailable, () => dependencies.readSkillRuntimeContracts(owner)) } : {}),
@@ -750,7 +784,7 @@ export function createSkillDispatchService(dependencies = {}) {
       const nextSkillId = control.status_card?.next_skill;
       const routeNextSkill = input.continue_delivery && !control.approval_presentation?.markdown
         && !control.completion?.fulfilled
-        && extractField(control.status_card?.runState?.content ?? "", "lifecycle") !== "completed"
+        && !["completed", "superseded", "abandoned"].includes(extractField(control.status_card?.runState?.content ?? "", "lifecycle"))
         && nextSkillId && nextSkillId !== "none"
         && (control.current_gate !== "QA" || control.missing_approval === "Approval: QA");
       if (routeNextSkill) skill = buildSkillDispatchRegistry(dependencies.pluginDefinition?.skillSet ?? rawInput.skillSet).get(nextSkillId) ?? skill;
@@ -827,6 +861,7 @@ export function createSkillDispatchService(dependencies = {}) {
       timing.total_ms = round(milliseconds(started, now()));
       timing.wrapper_ms = round(wrapperMilliseconds(now, env));
       return createSkillDispatchFailure({ skill, runtime, timing, code: recoveryCode, details: error?.details,
+        cause: error instanceof SkillDispatchRuntimeError ? error.cause_code : agdfCause(error),
         interactionLocales: rawInput.interactionLocales, presentationLanguage: input.presentation_language, renderRecovery });
     }
   };

package/generated/plugins/copilot/agdf/runtime/create-agdf/runtime/core/lib/skill-dispatch/tp-definition.js

@@ -1,3 +1,4 @@
+import { runDocumentPath } from '../control-state/run-identity.js';
 import { extractField } from "../control-evaluation/verified-change.js";
 import { ARTEFACT_TEMPLATE_IDS, resolveArtefactTemplatePath } from "../resources/artefact-templates.js";
 import { resolveArtifactPresentationLanguages } from "../resources/context.js";
@@ -22,7 +23,7 @@ export function tpDefinitionPhase(targetDir, control, input, sources) {
   if (input.skill_id !== "tp-definition" && !(input.intake || input.continue_delivery)) return null;
   const revisionId = extractField(state.content ?? "", "revision_id");
   if (!revisionId) return null;
-  const artifactPath = `.agdf/control/artefacts/${input.run_id}/TP.md`;
+  const artifactPath = `${runDocumentPath(input.run_id, '')}/TP.md`;
   const artefact = state.artefacts.get("TP");
   const registered = artefact?.status === "draft";
   if (artefact?.status === "approved" || (registered

package/generated/plugins/copilot/agdf/runtime/runtime-manifest.json

@@ -2,7 +2,7 @@
   "schema_version": "1",
   "generated": true,
   "source": "create-agdf/cli",
-  "version": "0.20.1",
+  "version": "0.30.0",
   "entrypoint": "create-agdf/bin/agdf-validator.js",
-  "digest": "9687032d64deec3ddc7461f0410c8462543c79dc94e87e4010e55eaaf1af7a8f"
+  "digest": "11079daafd9169912eb4f6cc66d5ba75c27acfa761634e17479baf1c553a815f"
 }

package/generated/plugins/copilot/agdf/skills/agdf-brownfield-analysis/SKILL.md

@@ -49,9 +49,10 @@ Then pick one catalog route; non-authorizing, downstream checks remain.
 
 ## Executable Dispatch
 
-Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `brownfield-analysis`, `presentation_language`, `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
+Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `brownfield-analysis`, `presentation_language` (optional override), `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
 If absent/failing: supplied schema-2 executable, child-only environment, immutable argv_prefix, declared arguments; `--skill brownfield-analysis`, language and working directory.
-For `--language`: Required presentation language for the latest natural-language user request as one well-formed BCP 47 tag. If the request explicitly asks for a response language, use that tag; otherwise use the dominant request language. Use en when mixed or ambiguous. A valid unsupported tag renders through the complete English pack. Missing or invalid input fails before governance evaluation.
+MCP: MCP reply-language override (BCP 47). Omit for a matching configured preference; otherwise supply the current request's language (explicit instruction, then dominant language, en if ambiguous). Invalid explicit tags fail; valid unsupported tags use the complete English pack. Core input remains complete.
+For `--language`: Required; apply the same current-request selection and locale validation.
 `target_source`: `explicit_target` if request names `primary_target`; `continued_target` if it unambiguously continues confirmed target; `current_repository` if request names this/current repo with one matching repo active. Otherwise omit the pair; cwd has no target authority.
 Quote shell values as data.
 For a result with `terminal: true`, the entire assistant response must consist only of host_action.text, copied verbatim. Add no question, explanation, heading, citation, link or other surrounding text; do not translate or reformat it; invoke no later tool and stop.
@@ -59,8 +60,10 @@ On skill_continuation use only its target/control. Without that tool and a valid
 or help retries. Dispatch never authorizes.
 
 Brownfield-specific output must make evidence, missing existing-system view, parallel-structure risk, reuse strategy, and the minimal next step visible.
-When `.agdf/control/` is present, persist or link `post_ur_review` output under `.agdf/control/artefacts/<key>/BROWNFIELD_REVIEW.md`,
-then record the Mode/Slice Decision with the validator's `run-step --step route --route <mode> --reason --evidence`.
+When `.agdf/control/` is present, persist or link `post_ur_review` output under `.agdf/control/runs/<run_id>/documents/BROWNFIELD_REVIEW.md`,
+then use the bound MCP continuation: stage the reviewed material and record the Mode/Slice
+Decision through `agdf_record/step.record` (step `route`). Only when that group is unavailable,
+use the supplied validator's `run-step --step route --route <mode> --reason --evidence`.
 
 ## Modes
 
@@ -198,7 +201,7 @@ Use a concise structure:
 - decision: pass | revise | block | not_applicable
 - mode_slice_decision: quick_task | verified_change | structured_slice | structured_delivery | block
 - required_next_gate: none | PRD | SD | TP | Brownfield Analysis
-- artefact: .agdf/control/artefacts/<key>/BROWNFIELD_REVIEW.md | none
+- artefact: .agdf/control/runs/<run_id>/documents/BROWNFIELD_REVIEW.md | none
 - scope:
 - evidence:
 - transparency: why later artefacts are skipped, shortened or required

package/generated/plugins/copilot/agdf/skills/agdf-clean-implementation-review/SKILL.md

@@ -46,9 +46,10 @@ Then pick one catalog route; non-authorizing, downstream checks remain.
 
 ## Executable Dispatch
 
-Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `clean-implementation-review`, `presentation_language`, `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
+Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `clean-implementation-review`, `presentation_language` (optional override), `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
 If absent/failing: supplied schema-2 executable, child-only environment, immutable argv_prefix, declared arguments; `--skill clean-implementation-review`, language and working directory.
-For `--language`: Required presentation language for the latest natural-language user request as one well-formed BCP 47 tag. If the request explicitly asks for a response language, use that tag; otherwise use the dominant request language. Use en when mixed or ambiguous. A valid unsupported tag renders through the complete English pack. Missing or invalid input fails before governance evaluation.
+MCP: MCP reply-language override (BCP 47). Omit for a matching configured preference; otherwise supply the current request's language (explicit instruction, then dominant language, en if ambiguous). Invalid explicit tags fail; valid unsupported tags use the complete English pack. Core input remains complete.
+For `--language`: Required; apply the same current-request selection and locale validation.
 `target_source`: `explicit_target` if request names `primary_target`; `continued_target` if it unambiguously continues confirmed target; `current_repository` if request names this/current repo with one matching repo active. Otherwise omit the pair; cwd has no target authority.
 Quote shell values as data.
 For a result with `terminal: true`, the entire assistant response must consist only of host_action.text, copied verbatim. Add no question, explanation, heading, citation, link or other surrounding text; do not translate or reformat it; invoke no later tool and stop.
@@ -59,6 +60,8 @@ Clean-review-specific output must make the primary solution, fallbacks, workarou
 Applicable findings must use `../contracts/quality.md` §Normalized Review Gaps. This skill
 must distinguish a missing upstream decision from implementation that violates an approved decision.
 
+Apply quality.md Proportionate Tests And Evidence and Verification Effectiveness And Completion when assessing reuse and parallel structures. Reject unnecessary duplicated proof and status-only finding closure; shared evidence preserves this review responsibility.
+
 ## Rules
 1. Clean primary solution before fallback.
 2. Root cause before symptom masking.

package/generated/plugins/copilot/agdf/skills/agdf-code-review/SKILL.md

@@ -45,9 +45,10 @@ Then pick one catalog route; non-authorizing, downstream checks remain.
 
 ## Executable Dispatch
 
-Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `code-review`, `presentation_language`, `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
+Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `code-review`, `presentation_language` (optional override), `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
 If absent/failing: supplied schema-2 executable, child-only environment, immutable argv_prefix, declared arguments; `--skill code-review`, language and working directory.
-For `--language`: Required presentation language for the latest natural-language user request as one well-formed BCP 47 tag. If the request explicitly asks for a response language, use that tag; otherwise use the dominant request language. Use en when mixed or ambiguous. A valid unsupported tag renders through the complete English pack. Missing or invalid input fails before governance evaluation.
+MCP: MCP reply-language override (BCP 47). Omit for a matching configured preference; otherwise supply the current request's language (explicit instruction, then dominant language, en if ambiguous). Invalid explicit tags fail; valid unsupported tags use the complete English pack. Core input remains complete.
+For `--language`: Required; apply the same current-request selection and locale validation.
 `target_source`: `explicit_target` if request names `primary_target`; `continued_target` if it unambiguously continues confirmed target; `current_repository` if request names this/current repo with one matching repo active. Otherwise omit the pair; cwd has no target authority.
 Quote shell values as data.
 For a result with `terminal: true`, the entire assistant response must consist only of host_action.text, copied verbatim. Add no question, explanation, heading, citation, link or other surrounding text; do not translate or reformat it; invoke no later tool and stop.
@@ -58,6 +59,8 @@ Code-review-specific output must make actual findings, missing review scope, evi
 Applicable findings must use `../contracts/quality.md` §Normalized Review Gaps. Concrete diff
 defects remain Code Review findings; missing upstream constraints route to their authoritative owner.
 
+Apply quality.md Proportionate Tests And Evidence and Verification Effectiveness And Completion to actual changed behavior: check requirement-derived expectations, meaningful defect detection and targeted re-verification of findings, including relevant side effects or justified alternatives.
+
 ## Rules
 1. Review the actual diff and impacted code, not intent alone.
 2. Findings require concrete evidence.
@@ -110,7 +113,8 @@ Use this compact structure:
 
 ```text
 ## Code Review
-Record the decision with the validator's `run-step --step review --decision <pass|revise|block> --evidence`.
+Stage the actual report and record through bound `agdf_record/step.record` (step `review`).
+If that MCP group is unavailable, use the supplied validator's `run-step --step review --decision <pass|revise|block> --evidence`.
 
 - decision: pass | revise | block | not_applicable
 - findings:

package/generated/plugins/copilot/agdf/skills/agdf-delivery-closeout/SKILL.md

@@ -49,9 +49,10 @@ Then pick one catalog route; non-authorizing, downstream checks remain.
 
 ## Executable Dispatch
 
-Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `delivery-closeout`, `presentation_language`, `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
+Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `delivery-closeout`, `presentation_language` (optional override), `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
 If absent/failing: supplied schema-2 executable, child-only environment, immutable argv_prefix, declared arguments; `--skill delivery-closeout`, language and working directory.
-For `--language`: Required presentation language for the latest natural-language user request as one well-formed BCP 47 tag. If the request explicitly asks for a response language, use that tag; otherwise use the dominant request language. Use en when mixed or ambiguous. A valid unsupported tag renders through the complete English pack. Missing or invalid input fails before governance evaluation.
+MCP: MCP reply-language override (BCP 47). Omit for a matching configured preference; otherwise supply the current request's language (explicit instruction, then dominant language, en if ambiguous). Invalid explicit tags fail; valid unsupported tags use the complete English pack. Core input remains complete.
+For `--language`: Required; apply the same current-request selection and locale validation.
 `target_source`: `explicit_target` if request names `primary_target`; `continued_target` if it unambiguously continues confirmed target; `current_repository` if request names this/current repo with one matching repo active. Otherwise omit the pair; cwd has no target authority.
 Quote shell values as data.
 For a result with `terminal: true`, the entire assistant response must consist only of host_action.text, copied verbatim. Add no question, explanation, heading, citation, link or other surrounding text; do not translate or reformat it; invoke no later tool and stop.

package/generated/plugins/copilot/agdf/skills/agdf-delivery-path-search/SKILL.md

@@ -42,9 +42,10 @@ Then pick one catalog route; non-authorizing, downstream checks remain.
 
 ## Executable Dispatch
 
-Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `delivery-path-search`, `presentation_language`, `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
+Use listed MCP `agdf_dispatch` (load deferred; host prefix allowed): `skill_id` `delivery-path-search`, `presentation_language` (optional override), `working_directory`, and only bound `target_source`/`primary_target` and `run_id`. Never search unlisted tools.
 If absent/failing: supplied schema-2 executable, child-only environment, immutable argv_prefix, declared arguments; `--skill delivery-path-search`, language and working directory.
-For `--language`: Required presentation language for the latest natural-language user request as one well-formed BCP 47 tag. If the request explicitly asks for a response language, use that tag; otherwise use the dominant request language. Use en when mixed or ambiguous. A valid unsupported tag renders through the complete English pack. Missing or invalid input fails before governance evaluation.
+MCP: MCP reply-language override (BCP 47). Omit for a matching configured preference; otherwise supply the current request's language (explicit instruction, then dominant language, en if ambiguous). Invalid explicit tags fail; valid unsupported tags use the complete English pack. Core input remains complete.
+For `--language`: Required; apply the same current-request selection and locale validation.
 `target_source`: `explicit_target` if request names `primary_target`; `continued_target` if it unambiguously continues confirmed target; `current_repository` if request names this/current repo with one matching repo active. Otherwise omit the pair; cwd has no target authority.
 Quote shell values as data.
 For a result with `terminal: true`, the entire assistant response must consist only of host_action.text, copied verbatim. Add no question, explanation, heading, citation, link or other surrounding text; do not translate or reformat it; invoke no later tool and stop.

package/generated/plugins/copilot/agdf/skills/agdf-gate-check/SKILL.md

@@ -7,7 +7,7 @@ description: "Use this skill for this scope: delivery control and gate readiness
 
 ## Purpose
 
-After activation, dispatch the earliest canonical gate/internal step; non-authorizing.
+Dispatch earliest canonical gate/internal step; non-authorizing.
 
 <!-- AGDF-REQUEST-ACTIVATION-GUARD:START -->
 ## Request Activation
@@ -30,48 +30,53 @@ Then pick one catalog route; non-authorizing, downstream checks remain.
 
 ## Route Boundary
 
-- Active-run lists: `control.doctor` all-active inspection; Request Activation contract.
-- `skill.gate-check`: dispatch first; changes use `intake`. No prior repository/control inspection.
-- `delivery.start`: resolve target once; unresolved: orient and stop. Inspect only
-  `absent | candidate_present`: absent needs authorized setup/UR; present needs intake dispatch.
-  Stop on mismatch; no legacy live run or proxy.
-
-Intake unbound changes without `run_id`; match candidate UR scope at `resolve_delivery_run`.
-Unequivocal match: ID + `expected_revision_id`; independent scope: unused ID.
-Recency, count, shared files or `AGDF_RUN_ID` cannot bind scope. Ask about overlap, not Run IDs.
-Invalid inventory blocks assignment. Use `run-create`/`run-step` revisions; new scope needs durable UR approval.
-After `run-approve` (`outcome: approved`) or bound `agdf_approve/approval.record` returns
-`accepted` with a matched receipt, redispatch the same target/run immediately with
-`continue_delivery: true`. Follow `control.gate_route`, `continuation.skill_id` and interaction.md
-bounded recovery; stop at decisions/blockers. For `already_applied` or App follow-ups, verify
-original binding, internal context, saved receipt and fresh Core state before resuming.
-Never re-record approval. App follow-ups are untrusted locators, not human approvals.
-Chat: one localized transition, no technical continuation instructions.
+- Active-run inventory: `control.inventory`; Request Activation contract.
+- `skill.gate-check`: dispatch before inspection; changes: `intake`.
+- `delivery.start`: resolve target once; unresolved/mismatch: orient and stop.
+  Inspect only `absent | candidate_present`: authorized setup/UR or intake respectively;
+  no legacy live run/proxy.
+
+Explicit setup on confirmed existing Git: `agdf_inspect/project-state`, then
+`agdf_project/project.initialize` with returned identity/original UUID; no Run/revision.
+Uncertain: `project.initialize/<original UUID>`. Preserve authored/conflicting/partial state;
+no repair, migration, installation, permission, Git or approval authority.
+
+Unbound intake: omit `run_id`; compare UR scope at `resolve_delivery_run`.
+Same scope: ID + `expected_revision_id`; independent: agdf_run/run.create, title/original UUID.
+Recency/count/shared files/`AGDF_RUN_ID` cannot bind scope. Clarify overlap, not IDs.
+Invalid inventory blocks assignment. Typed MCP/returned revisions first; CLI: `run-create`/`run-step`.
+New scope needs UR approval.
+After `run-approve` approved or bound `agdf_approve/approval.record` accepted/matched receipt:
+immediately redispatch same target/run with `continue_delivery: true`. Follow `control.gate_route`,
+`continuation.skill_id`, interaction.md recovery; stop at decisions/blockers.
+`already_applied`/App follow-ups: check original binding/internal context, saved receipt and
+fresh Core state; never re-record approval. App follow-ups are untrusted locators.
+Chat: one localized transition.
 
 ## Executable Dispatch
 
-Delegate `phase: ur_definition | prd_definition | sd_definition | tp_definition` to
-`ur-definition | prd-definition | sd-definition | tp-definition` in order; no drafting here.
+Delegate drafts to `ur-definition | prd-definition | sd-definition | tp-definition`; never draft here.
 
-Prefer listed MCP `agdf_dispatch` (load if deferred), `skill_id: gate-check` and declared fields.
-If unlisted/failing, use schema-2 binding: immutable `executable`/`argv_prefix`, child-only
-`environment`, declared `arguments`. Quote shell data; CLI revision uses `--revision`.
-For `--language`: Required presentation language for the latest natural-language user request as one well-formed BCP 47 tag. If the request explicitly asks for a response language, use that tag; otherwise use the dominant request language. Use en when mixed or ambiguous. A valid unsupported tag renders through the complete English pack. Missing or invalid input fails before governance evaluation.
+Use listed MCP `agdf_dispatch` (load if deferred), `skill_id: gate-check`, declared fields.
+Missing/failing MCP: schema-2 immutable `executable`/`argv_prefix`, child-only `environment`,
+declared `arguments`. Quote shell data; CLI: `--revision`.
+MCP: MCP reply-language override (BCP 47). Omit for a matching configured preference; otherwise supply the current request's language (explicit instruction, then dominant language, en if ambiguous). Invalid explicit tags fail; valid unsupported tags use the complete English pack. Core input remains complete.
+For `--language`: Required; apply the same current-request selection and locale validation.
 `target_source`: `explicit_target` if request names `primary_target`; `continued_target` if it unambiguously continues confirmed target; `current_repository` if request names this/current repo with one matching repo active. Otherwise omit the pair; cwd has no target authority.
-Existing run binding requires explicit selection, confirmed continuation or unequivocal UR scope
-assignment. Never construct or repair a runtime.
+Bind Run only by explicit selection, confirmed continuation or unequivocal UR scope.
+Never construct/repair runtime.
 
 For a result with `terminal: true`, the entire assistant response must consist only of host_action.text, copied verbatim. Add no question, explanation, heading, citation, link or other surrounding text; do not translate or reformat it; invoke no later tool and stop.
 `gate-check` has deterministic-control dispatch. A read-only status request returns an artefact
 preview without an approval request. Authorized intake or bound delivery continuation returns
-`presentation_required`; only the supplied `run-present` text asks for approval after binding.
+`presentation_required`; MCP: agdf_prepare then presentation.deliver; forward its selected text once. CLI fallback: run-present.
 `host_action.text` contains the preview or recovery for terminal results.
 Dispatch is non-authorizing. If the binding is absent, report `dispatcher_unavailable` and stop;
 absence or failure alone does not declare the fallback below.
 
 ## Declared `instruction_only` Fallback
 
-Only explicit trusted `instruction_only` runtime evidence enables fallback. Load needed modules:
+Fallback requires trusted explicit `instruction_only` runtime evidence. Load only needed modules:
 
 - `../contracts/task-target-resolution.md`
 - `../contracts/gate-transition.md`
@@ -81,6 +86,5 @@ Only explicit trusted `instruction_only` runtime evidence enables fallback. Load
 - `../contracts/modes.md`
 - `../contracts/quality.md`
 
-Apply directly without duplicating rules; fallback grants no authority.
-`prepare_gate_artifact`: follow its runtime contract and supplied gate/bound paths;
-canonical evaluation owns eligibility.
+Apply without duplication/authority. `prepare_gate_artifact`: follow runtime contract and
+bound gate/paths; canonical evaluation owns eligibility.