@linimin/pi-letscook 0.1.54 → 0.1.55

package/extensions/completion/types.ts

@@ -85,117 +85,3 @@ export type CompletionStatusSurface = {
 	liveStateDeltas?: string[];
 	liveDetailsLines?: string[];
 };
-
-export type NaturalLanguageCookTriggerMode = "off" | "router" | "auto";
-export type CookTriggerClassifierDecision = "offer_workflow" | "normal_prompt" | "unclear";
-export type CookTriggerWorkflowBias = "startup" | "resume" | "refocus" | "next_round" | "unknown";
-
-export type CookTriggerClassification = {
-	decision: CookTriggerClassifierDecision;
-	confidence: number;
-	workflowBias: CookTriggerWorkflowBias;
-	reason: string;
-	focusHint?: string;
-	evidence: string[];
-	riskFlags: string[];
-};
-
-export type CookTriggerConfirmationAction = "start_workflow" | "send_as_normal_chat" | "cancel";
-export type CookTriggerClarificationAction =
-	| "route_startup"
-	| "route_resume"
-	| "route_refocus"
-	| "route_next_round"
-	| "send_as_normal_chat"
-	| "cancel";
-export type CookTriggerRecoveryAction = "retry_routing" | "send_as_normal_chat" | "cancel";
-export type CookTriggerAdoptedArtifactKind = "recent_plan" | "repo_markdown";
-
-export type CookTriggerConfirmationActionItem = {
-	id: CookTriggerConfirmationAction;
-	label: string;
-	description: string;
-};
-
-export type CookTriggerClarificationActionItem = {
-	id: CookTriggerClarificationAction;
-	label: string;
-	description: string;
-};
-
-export type CookTriggerConfirmationLayout = {
-	title: string;
-	intro: string;
-	evidenceHeading?: string;
-	evidenceBody?: string;
-	riskHeading?: string;
-	riskBody?: string;
-	focusHintHeading?: string;
-	focusHintBody?: string;
-	actionsHeading: string;
-	actions: CookTriggerConfirmationActionItem[];
-	footer: string;
-};
-
-export type CookTriggerClarificationCapsule = {
-	goal?: string;
-	scope?: string[];
-	nonGoal?: string[];
-	doneWhen?: string[];
-	selectedWorkflowBias: CookTriggerWorkflowBias;
-	reason: string;
-};
-
-export type CookTriggerAdoptedArtifact = {
-	kind: CookTriggerAdoptedArtifactKind;
-	basis: "explicit_user_adoption";
-	title: string;
-	path?: string;
-	preview?: string;
-};
-
-export type CookNaturalLanguageHandoff = {
-	preferredRoutingBias?: CookTriggerWorkflowBias;
-	triggerText?: string;
-	hintText?: string;
-	clarificationCapsule?: CookTriggerClarificationCapsule;
-	adoptedArtifact?: CookTriggerAdoptedArtifact;
-};
-
-export type CookTriggerClarificationLayout = {
-	title: string;
-	intro: string;
-	currentMissionHeading?: string;
-	currentMissionBody?: string;
-	candidateMissionHeading?: string;
-	candidateMissionBody?: string;
-	adoptedArtifactHeading?: string;
-	adoptedArtifactBody?: string;
-	actionsHeading: string;
-	actions: CookTriggerClarificationActionItem[];
-	footer: string;
-};
-
-export type CookTriggerRecoveryActionItem = {
-	id: CookTriggerRecoveryAction;
-	label: string;
-	description: string;
-};
-
-export type CookTriggerRecoveryLayout = {
-	title: string;
-	intro: string;
-	failureHeading?: string;
-	failureBody?: string;
-	actionsHeading: string;
-	actions: CookTriggerRecoveryActionItem[];
-	footer: string;
-};
-
-export type CookTriggerDecision = {
-	mode: NaturalLanguageCookTriggerMode;
-	action: "continue" | "handled" | "routed_to_cook";
-	reason: string;
-	classification?: CookTriggerClassification;
-	bypassReason?: string;
-};

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linimin/pi-letscook",
-  "version": "0.1.54",
+  "version": "0.1.55",
   "description": "Pi package for long-running completion workflows with canonical .agent state, role-based subagents, continuity, and verification helpers.",
   "license": "MIT",
   "private": false,
@@ -13,7 +13,14 @@
   "bugs": {
     "url": "https://github.com/linimin/pi-letscook/issues"
   },
-  "keywords": ["pi-package", "pi", "workflow", "completion", "agent", "subagent"],
+  "keywords": [
+    "pi-package",
+    "pi",
+    "workflow",
+    "completion",
+    "agent",
+    "subagent"
+  ],
   "files": [
     "extensions",
     "skills",
@@ -43,7 +50,11 @@
     "typebox": "*"
   },
   "pi": {
-    "extensions": ["./extensions/completion"],
-    "skills": ["./skills"]
+    "extensions": [
+      "./extensions/completion"
+    ],
+    "skills": [
+      "./skills"
+    ]
   }
 }

package/scripts/active-slice-contract-test.sh

@@ -82,9 +82,13 @@ assertIncludes('extensions/completion/prompt-surfaces.ts', 'Active slice contrac
 assertIncludes('extensions/completion/index.ts', 'Canonical active-slice contract drift is currently: ${activeContractDrift}');
 assertIncludes('extensions/completion/prompt-surfaces.ts', '`active_slice_contract_drift_fields: ${args.activeSliceContractDrift}`');
 assertIncludes('extensions/completion/index.ts', 'treat .agent/active-slice.json as the canonical implementation contract');
+assertIncludes('.agent/verify_completion_control_plane.sh', 'const REQUIRED_TRACKED_CONTRACT_FILES = [');
+assertIncludes('.agent/verify_completion_control_plane.sh', 'Required tracked completion contract file is missing from git index:');
 assertIncludes('.agent/verify_completion_control_plane.sh', "const planMirrorFields = ['locked_notes', 'must_fix_findings', 'implementation_surfaces', 'verification_commands', 'basis_commit', 'remaining_contract_ids_before', 'release_blocker_count_before', 'high_value_gap_count_before'];");
 assertIncludes('.agent/verify_completion_control_plane.sh', 'slice_id must match a slice in .agent/plan.json when status carries an exact handoff');
 assertIncludes('.agent/verify_completion_control_plane.sh', '.agent/active-slice.json must match the selected .agent/plan.json slice across: ');
+assertIncludes('.agent/verify_completion_control_plane.sh', '.agent/active-slice.json implementation_surfaces must cover every tracked file changed from basis_commit to current HEAD; missing: ');
+assertIncludes('scripts/release-check.sh', 'git ls-files --error-unmatch .agent/README.md .agent/mission.md .agent/profile.json .agent/verify_completion_stop.sh .agent/verify_completion_control_plane.sh >/dev/null');
 NODE
 
 ROOT="$TMPDIR/repo"
@@ -102,11 +106,24 @@ PI_COMPLETION_SKIP_DRIVER_KICKOFF=1 \
 pi --session "$BOOTSTRAP_SESSION" -e "$PKG_ROOT" -p "/cook" \
   >"$TMPDIR/pi-active-slice-bootstrap.out" 2>"$TMPDIR/pi-active-slice-bootstrap.err"
 
-python3 - <<'PY'
+git config user.name 'Completion Fixture'
+git config user.email 'completion-fixture@example.com'
+git add .
+git commit -q -m 'fixture basis'
+BASIS_SHA="$(git rev-parse HEAD)"
+printf '\nSlice surface fixture change.\n' >> README.md
+git add README.md
+git commit -q -m 'fixture slice change'
+HEAD_SHA="$(git rev-parse HEAD)"
+
+BASIS_SHA="$BASIS_SHA" HEAD_SHA="$HEAD_SHA" python3 - <<'PY'
 import json
+import os
 from pathlib import Path
 
 mission = 'Exercise active-slice contract parity.'
+basis_sha = os.environ['BASIS_SHA']
+head_sha = os.environ['HEAD_SHA']
 task_type = 'completion-workflow'
 evaluation_profile = 'completion-rubric-v1'
 verification_commands = [
@@ -157,8 +174,8 @@ state = {
     'last_reground_at': '2026-05-03T00:00:00Z',
     'last_auditor_verdict': None,
     'contract_status': 'selected_slice_pending_implementation',
-    'latest_completed_slice': 'fixturebasis',
-    'latest_verified_slice': 'fixturebasis',
+    'latest_completed_slice': head_sha,
+    'latest_verified_slice': head_sha,
 }
 plan = {
     'schema_version': 1,
@@ -182,7 +199,7 @@ plan = {
             'must_fix_findings': must_fix_findings,
             'implementation_surfaces': implementation_surfaces,
             'verification_commands': verification_commands,
-            'basis_commit': 'fixturebasis',
+            'basis_commit': basis_sha,
             'remaining_contract_ids_before': remaining_contracts,
             'release_blocker_count_before': 0,
             'high_value_gap_count_before': 2,
@@ -204,7 +221,7 @@ active = {
     'must_fix_findings': must_fix_findings,
     'implementation_surfaces': implementation_surfaces,
     'verification_commands': verification_commands,
-    'basis_commit': 'fixturebasis',
+    'basis_commit': basis_sha,
     'remaining_contract_ids_before': remaining_contracts,
     'release_blocker_count_before': 0,
     'high_value_gap_count_before': 2,
@@ -223,7 +240,7 @@ Path('.agent/verification-evidence.json').write_text(json.dumps({
     'goal': active['goal'],
     'contract_ids': active['contract_ids'],
     'basis_commit': active['basis_commit'],
-    'head_sha': active['basis_commit'],
+    'head_sha': head_sha,
     'verification_commands': verification_commands,
     'outcome': 'passed',
     'recorded_at': '2026-05-03T00:00:00Z',
@@ -247,6 +264,19 @@ PY
 
 bash .agent/verify_completion_control_plane.sh >/dev/null
 
+git rm --cached -q .agent/profile.json
+if bash .agent/verify_completion_control_plane.sh >"$TMPDIR/untracked-contract.out" 2>"$TMPDIR/untracked-contract.err"; then
+  echo 'expected verifier to fail when a required tracked .agent contract file becomes untracked' >&2
+  exit 1
+fi
+if ! grep -q 'Required tracked completion contract file is missing from git index: .agent/profile.json' "$TMPDIR/untracked-contract.err"; then
+  echo 'expected verifier failure output to mention the untracked .agent/profile.json contract file' >&2
+  cat "$TMPDIR/untracked-contract.err" >&2
+  exit 1
+fi
+git add .agent/profile.json
+bash .agent/verify_completion_control_plane.sh >/dev/null
+
 python3 - <<'PY'
 import copy
 import json
@@ -254,7 +284,9 @@ import subprocess
 from pathlib import Path
 
 plan_path = Path('.agent/plan.json')
+active_path = Path('.agent/active-slice.json')
 base_plan = json.loads(plan_path.read_text())
+base_active = json.loads(active_path.read_text())
 
 cases = [
     ('slice_id', lambda slice: slice.__setitem__('slice_id', 'different-slice')),
@@ -285,6 +317,29 @@ for label, mutate in cases:
     assert label in combined, f'expected verifier output to mention {label}, got: {combined}'
 
 plan_path.write_text(json.dumps(base_plan, indent=2) + '\n')
+active_path.write_text(json.dumps(base_active, indent=2) + '\n')
+
+surface_drift_plan = copy.deepcopy(base_plan)
+surface_drift_active = copy.deepcopy(base_active)
+surface_drift_plan['candidate_slices'][0]['implementation_surfaces'] = [
+    surface for surface in surface_drift_plan['candidate_slices'][0]['implementation_surfaces']
+    if surface != 'README.md'
+]
+surface_drift_active['implementation_surfaces'] = [
+    surface for surface in surface_drift_active['implementation_surfaces']
+    if surface != 'README.md'
+]
+plan_path.write_text(json.dumps(surface_drift_plan, indent=2) + '\n')
+active_path.write_text(json.dumps(surface_drift_active, indent=2) + '\n')
+result = subprocess.run(['bash', '.agent/verify_completion_control_plane.sh'], capture_output=True, text=True)
+combined = (result.stdout or '') + (result.stderr or '')
+assert result.returncode != 0, 'expected verifier failure when implementation_surfaces omit a tracked basis-to-HEAD diff file'
+assert 'implementation_surfaces must cover every tracked file changed from basis_commit to current HEAD; missing: README.md' in combined, combined
+
+plan_path.write_text(json.dumps(base_plan, indent=2) + '\n')
+active_path.write_text(json.dumps(base_active, indent=2) + '\n')
 PY
 
+bash .agent/verify_completion_control_plane.sh >/dev/null
+
 echo "active-slice contract test passed: $TMPDIR"

package/scripts/context-proposal-test.sh

@@ -654,6 +654,16 @@ assert plan['evaluation_profile'] == expected_eval_profile, 'plan.json evaluatio
 assert active['mission_anchor'] == mission, 'active-slice.json mission_anchor mismatch after analyst-derived bootstrap'
 assert active['task_type'] == expected_task_type, 'active-slice.json task_type mismatch after analyst-derived bootstrap'
 assert active['evaluation_profile'] == expected_eval_profile, 'active-slice.json evaluation_profile mismatch after analyst-derived bootstrap'
+brief = state['advisory_startup_brief']
+assert brief['kind'] == 'startup_brief', 'state.json should preserve the confirmed startup brief as advisory intake'
+assert brief['mission'] == mission, 'advisory startup brief mission should match the accepted mission anchor'
+assert brief['scope'] == ['Keep the non-running completion widget.', 'Suppress the widget while a completion role is active.'], 'advisory startup brief should preserve scope items separately from canonical planning state'
+assert brief['constraints'] == ['Do not reintroduce any other completion status surface.'], 'advisory startup brief should preserve constraints separately from canonical planning state'
+assert brief['acceptance'] == ['Update README to match the shipped behavior.', 'Keep observability regression coverage truthful.'], 'advisory startup brief should preserve acceptance separately from canonical planning state'
+assert brief['risks'] == ['Stale widget-removal discussion could broaden the startup plan if it gets treated as mission text.'], 'advisory startup brief should preserve derived risks'
+assert brief['notes'] == ['Keep critique separate from the mission anchor so startup analysis does not rewrite the workflow goal.', 'Possible noise: older widget restyle ideas'], 'advisory startup brief should preserve operator notes and possible-noise cautions'
+assert plan['candidate_slices'] == [], 'startup brief should remain advisory intake only until regrounder owns plan selection'
+assert active['status'] == 'idle', 'startup brief should not become the active-slice source before regrounder runs'
 assert proposal['mission'] == mission, 'discussion-only proposal snapshot should keep the inferred mission anchor'
 assert proposal['analysis']['taskType'] == expected_task_type, 'discussion-only proposal snapshot should expose task_type hints separately'
 assert proposal['analysis']['evaluationProfile'] == expected_eval_profile, 'discussion-only proposal snapshot should expose evaluation_profile hints separately'
@@ -1080,6 +1090,7 @@ assert profile['evaluation_profile'] == expected_eval_profile, 'profile.json eva
 assert state['mission_anchor'] == mission, 'state.json mission_anchor mismatch after starting the next workflow round'
 assert state['task_type'] == expected_task_type, 'state.json task_type mismatch after starting the next workflow round'
 assert state['evaluation_profile'] == expected_eval_profile, 'state.json evaluation_profile mismatch after starting the next workflow round'
+assert state['advisory_startup_brief']['mission'] == mission, 'next-round startup should preserve the confirmed startup brief as advisory intake'
 assert plan['mission_anchor'] == mission, 'plan.json mission_anchor mismatch after starting the next workflow round'
 assert plan['task_type'] == expected_task_type, 'plan.json task_type mismatch after starting the next workflow round'
 assert plan['evaluation_profile'] == expected_eval_profile, 'plan.json evaluation_profile mismatch after starting the next workflow round'
@@ -1101,8 +1112,7 @@ assert plan['plan_basis'] == 'user_refocus', 'plan_basis should reset to user_re
 assert active['status'] == 'idle', 'active-slice should reset to idle for the next workflow round'
 PY
 
-# Active workflow: /cook <hint> should bias active-workflow proposal derivation,
-# still route through the chooser, and leave canonical state unchanged when the user cancels.
+# Active workflow: inline `/cook` arguments should fail closed immediately and leave canonical state unchanged.
 ACTIVE_INLINE_REJECTION_ROUTING="$TMPDIR/context-proposal-active-inline-arg-routing.json"
 ACTIVE_INLINE_REJECTION_PROPOSAL="$TMPDIR/context-proposal-active-inline-arg-proposal.json"
 ACTIVE_INLINE_REJECTION_CHOOSER="$TMPDIR/context-proposal-active-inline-arg-chooser.json"
@@ -1123,9 +1133,6 @@ tracked = [
 Path(sys.argv[1]).write_text(json.dumps({path.name: path.read_text() for path in tracked}, indent=2) + '\n')
 PY
 
-PI_COMPLETION_EXISTING_WORKFLOW_ACTION=refocus \
-PI_COMPLETION_CONTEXT_PROPOSAL_ACTION=cancel \
-PI_COMPLETION_CONTEXT_PROPOSAL_ANALYST_OUTPUT='{"mission":"Replacement mission for the active workflow.","scope":["Review the replacement through the existing workflow chooser first."],"constraints":["Do not rewrite canonical state before the final Start confirmation."],"acceptance":["Show the final replacement proposal only after the chooser selects refocus."],"task_type":"completion-workflow","evaluation_profile":"completion-rubric-v1","confidence":0.9}' \
 PI_COMPLETION_TEST_CONTEXT_PROPOSAL_PATH="$ACTIVE_INLINE_REJECTION_PROPOSAL" \
 PI_COMPLETION_TEST_ACTIVE_WORKFLOW_ROUTING_PATH="$ACTIVE_INLINE_REJECTION_ROUTING" \
 PI_COMPLETION_TEST_EXISTING_WORKFLOW_CHOOSER_PATH="$ACTIVE_INLINE_REJECTION_CHOOSER" \
@@ -1138,9 +1145,9 @@ import sys
 from pathlib import Path
 
 output = Path(sys.argv[1]).read_text() + Path(sys.argv[2]).read_text()
-routing = json.loads(Path(sys.argv[3]).read_text())
+routing = Path(sys.argv[3])
 proposal = Path(sys.argv[4])
-chooser = json.loads(Path(sys.argv[5]).read_text())
+chooser = Path(sys.argv[5])
 before = json.loads(Path(sys.argv[6]).read_text())
 tracked = [
     Path('.agent/mission.md'),
@@ -1151,17 +1158,15 @@ tracked = [
     Path('.agent/verification-evidence.json'),
 ]
 
-assert routing['action'] == 'refocus', 'active /cook <hint> should run active-workflow routing'
-assert routing['proposedMissionAnchor'] == 'Replacement mission for the active workflow.', 'active /cook <hint> should bias toward the hinted replacement mission'
-assert json.loads(proposal.read_text())['mission'] == 'Replacement mission for the active workflow.', 'active /cook <hint> should carry the hinted mission into the final replacement proposal'
-assert chooser['choices'][1].startswith('Start new workflow from recent discussion'), 'active /cook <hint> should open the existing-workflow chooser'
-assert 'Cancelled replacement workflow proposal.' in output or 'Cancelled existing workflow confirmation.' in output, 'active /cook <hint> cancel should report cancellation'
+assert not routing.exists(), 'active /cook inline-args rejection should not run active-workflow routing'
+assert not proposal.exists(), 'active /cook inline-args rejection should not emit a replacement startup-brief proposal'
+assert not chooser.exists(), 'active /cook inline-args rejection should not open the existing-workflow chooser'
+assert '/cook no longer accepts inline arguments.' in output, 'active /cook inline-args rejection should explain the bare-only entry contract'
 after = {path.name: path.read_text() for path in tracked}
-assert before == after, 'active /cook <hint> cancel should leave canonical files unchanged'
+assert before == after, 'active /cook inline-args rejection should leave canonical files unchanged'
 PY
 
-# Completed workflow: /cook <hint> should bias next-round proposal derivation and still leave canonical state
-# unchanged when the user cancels the approval-only proposal.
+# Completed workflow: inline `/cook` arguments should also fail closed before any next-round proposal derivation.
 mark_done
 
 DONE_INLINE_REJECTION_ROUTING="$TMPDIR/context-proposal-done-inline-arg-routing.json"
@@ -1184,8 +1189,6 @@ tracked = [
 Path(sys.argv[1]).write_text(json.dumps({path.name: path.read_text() for path in tracked}, indent=2) + '\n')
 PY
 
-PI_COMPLETION_CONTEXT_PROPOSAL_ACTION=cancel \
-PI_COMPLETION_CONTEXT_PROPOSAL_ANALYST_OUTPUT='{"mission":"Update README guidance for the next workflow round.","scope":["Refresh README guidance for /cook hint-driven startup behavior."],"constraints":["Do not rewrite canonical state before Start is accepted."],"acceptance":["Keep the next-round proposal scoped to README updates only."],"task_type":"completion-workflow","evaluation_profile":"completion-rubric-v1","confidence":0.9}' \
 PI_COMPLETION_TEST_CONTEXT_PROPOSAL_PATH="$DONE_INLINE_REJECTION_PROPOSAL" \
 PI_COMPLETION_TEST_ACTIVE_WORKFLOW_ROUTING_PATH="$DONE_INLINE_REJECTION_ROUTING" \
 PI_COMPLETION_TEST_EXISTING_WORKFLOW_CHOOSER_PATH="$DONE_INLINE_REJECTION_CHOOSER" \
@@ -1199,7 +1202,7 @@ from pathlib import Path
 
 output = Path(sys.argv[1]).read_text() + Path(sys.argv[2]).read_text()
 routing = Path(sys.argv[3])
-proposal = json.loads(Path(sys.argv[4]).read_text())
+proposal = Path(sys.argv[4])
 chooser = Path(sys.argv[5])
 before = json.loads(Path(sys.argv[6]).read_text())
 tracked = [
@@ -1211,14 +1214,14 @@ tracked = [
     Path('.agent/verification-evidence.json'),
 ]
 state_before = json.loads(before['state.json'])
-assert state_before['current_phase'] == 'done', 'done /cook <hint> should start from a completed workflow'
-assert state_before['project_done'] is True, 'done /cook <hint> should start from project_done=true'
-assert not routing.exists(), 'done /cook <hint> should not run active-workflow routing while starting the next round'
-assert proposal['mission'] == 'Update README guidance for the next workflow round.', 'done /cook <hint> should bias next-round mission derivation toward the hint'
-assert not chooser.exists(), 'done /cook <hint> should not open the existing-workflow chooser when starting the next round'
-assert 'Cancelled next workflow round proposal.' in output, 'done /cook <hint> cancel should report next-round proposal cancellation'
+assert state_before['current_phase'] == 'done', 'done /cook inline-args rejection should start from a completed workflow'
+assert state_before['project_done'] is True, 'done /cook inline-args rejection should start from project_done=true'
+assert not routing.exists(), 'done /cook inline-args rejection should not run active-workflow routing while starting the next round'
+assert not proposal.exists(), 'done /cook inline-args rejection should not emit a next-round startup-brief proposal'
+assert not chooser.exists(), 'done /cook inline-args rejection should not open the existing-workflow chooser when starting the next round'
+assert '/cook no longer accepts inline arguments.' in output, 'done /cook inline-args rejection should explain the bare-only entry contract'
 after = {path.name: path.read_text() for path in tracked}
-assert before == after, 'done /cook <hint> cancel should leave canonical files unchanged'
+assert before == after, 'done /cook inline-args rejection should leave canonical files unchanged'
 PY
 
 # Completed workflow again: /cook with no goal should be able to use model-assisted
@@ -1296,15 +1299,15 @@ from pathlib import Path
 snapshot = json.loads(Path(sys.argv[1]).read_text())
 state = json.loads(Path('.agent/state.json').read_text())
 
-assert snapshot['proposalHeading'] == 'Proposed workflow', 'custom confirmation snapshot should expose a dedicated proposal section'
-assert snapshot['critiqueHeading'] == 'Critique and risks', 'custom confirmation snapshot should expose critique separately from the proposal body'
+assert snapshot['proposalHeading'] == 'Startup brief', 'custom confirmation snapshot should expose a dedicated startup-brief section'
+assert snapshot['critiqueHeading'] == 'Notes and risks', 'custom confirmation snapshot should expose notes separately from the startup-brief body'
 assert snapshot['routingHeading'] == 'Routing recommendations', 'custom confirmation snapshot should expose routing recommendations separately from the proposal body'
 assert 'approval-only' in snapshot['intro'], 'custom confirmation intro should explain the approval-only gate'
 assert state['task_type'] == 'completion-workflow', 'start action should preserve canonical task_type'
 assert state['evaluation_profile'] == 'completion-rubric-v1', 'start action should preserve canonical evaluation_profile'
 assert 'Mission\nReplace the crowded selector with a clearer action layout.' in snapshot['proposalBody'], 'proposal body should be captured separately from the action list'
-assert 'Keep critique details separate from the approval-only proposal summary.' not in snapshot['proposalBody'], 'critique notes should not be embedded in the proposal body'
-assert 'Critique\n- Keep critique details separate from the approval-only proposal summary.' in snapshot['critiqueBody'], 'critique section should render accepted critique notes separately'
+assert 'Keep critique details separate from the approval-only proposal summary.' not in snapshot['proposalBody'], 'critique notes should not be embedded in the startup-brief body'
+assert 'Critique\n- Keep critique details separate from the approval-only proposal summary.' in snapshot['critiqueBody'], 'notes section should render accepted critique notes separately'
 assert 'Risks\n- Bundling critique into the action list would make the confirmation harder to scan.' in snapshot['critiqueBody'], 'critique section should render risk notes separately'
 assert 'Possible noise\n- old selector wording' in snapshot['critiqueBody'], 'critique section should render possible-noise notes separately'
 assert '- task_type: completion-workflow' in snapshot['routingBody'], 'routing section should render the recommended task_type'
@@ -1316,6 +1319,7 @@ for action in snapshot['actions']:
     assert 'Replace the crowded selector with a clearer action layout.' not in action['label'], 'proposal mission should not be embedded in action labels'
     assert 'Separate proposal text from actions.' not in action['description'], 'proposal scope should not be embedded in action descriptions'
 assert state['mission_anchor'] == 'Replace the crowded selector with a clearer action layout.', 'start action should still accept the proposed mission'
+assert state['advisory_startup_brief']['mission'] == 'Replace the crowded selector with a clearer action layout.', 'start action should preserve the confirmed startup brief canonically'
 assert state['continuation_reason'].startswith('User started workflow via /cook:'), 'start action should persist the startup routing outcome in continuation_reason'
 assert 'Keep critique details separate from the approval-only proposal summary.' in state['continuation_reason'], 'start action should persist the accepted critique outcome canonically'
 PY

package/scripts/legacy-cleanup-test.sh

@@ -54,6 +54,13 @@ assertIncludes('extensions/completion/role-runner.ts', 'async function runContex
 assertIncludes('extensions/completion/prompt-surfaces.ts', 'export function buildSystemReminder(');
 assertIncludes('extensions/completion/prompt-surfaces.ts', 'export function buildResumeCapsule(');
 
+if (fs.existsSync('extensions/completion/input-routing.ts')) {
+  throw new Error('extensions/completion/input-routing.ts should be removed after explicit-/cook cleanup');
+}
+if (fs.existsSync('scripts/cook-trigger-routing-test.sh')) {
+  throw new Error('scripts/cook-trigger-routing-test.sh should be removed after explicit-/cook cleanup');
+}
+
 assertIncludes('extensions/completion/index.ts', 'scaffoldCompletionFiles as scaffoldCompletionFilesOnDisk');
 assertIncludes('extensions/completion/index.ts', 'return await scaffoldCompletionFilesOnDisk(root, missionAnchor, {');
 assertIncludes('extensions/completion/index.ts', 'applyLiveRoleEvent,');
@@ -102,6 +109,10 @@ assertNotIncludes('extensions/completion/index.ts', 'async function analyzeConte
 assertNotIncludes('extensions/completion/index.ts', 'function deriveMissionAnchor(');
 assertNotIncludes('extensions/completion/index.ts', 'function buildSystemReminder(');
 assertNotIncludes('extensions/completion/index.ts', 'function buildResumeCapsule(');
+assertNotIncludes('extensions/completion/role-runner.ts', 'classifyCookTriggerIntentWithAgent(');
+assertNotIncludes('extensions/completion/role-runner.ts', 'COOK_TRIGGER_CLASSIFIER_SYSTEM_PROMPT');
+assertNotIncludes('extensions/completion/prompt-surfaces.ts', 'buildCookTriggerConfirmationLayout(');
+assertNotIncludes('extensions/completion/prompt-surfaces.ts', 'buildNaturalLanguageHandoffMetadataLines(');
 NODE
 
 echo "legacy cleanup test passed: $ROOT"

package/scripts/refocus-test.sh

@@ -102,8 +102,6 @@ tracked = [
 Path(sys.argv[1]).write_text(json.dumps({path.name: path.read_text() for path in tracked}, indent=2) + '\n')
 PY
 
-PI_COMPLETION_EXISTING_WORKFLOW_ACTION=cancel \
-PI_COMPLETION_CONTEXT_PROPOSAL_ANALYST_OUTPUT='{"mission":"Replacement mission that should stay in the main chat.","scope":["Replace the current workflow only after explicit approval."],"constraints":["Do not rewrite canonical state before the final Start confirmation."],"acceptance":["Surface the chooser before any replacement workflow rewrite."],"task_type":"completion-workflow","evaluation_profile":"completion-rubric-v1","confidence":0.9}' \
 PI_COMPLETION_TEST_ACTIVE_WORKFLOW_ROUTING_PATH="$INLINE_REJECTION_ROUTING" \
 PI_COMPLETION_TEST_CONTEXT_PROPOSAL_PATH="$INLINE_REJECTION_PROPOSAL" \
 PI_COMPLETION_TEST_EXISTING_WORKFLOW_CHOOSER_PATH="$INLINE_REJECTION_CHOOSER" \
@@ -117,9 +115,9 @@ import sys
 from pathlib import Path
 
 output = Path(sys.argv[1]).read_text() + Path(sys.argv[2]).read_text()
-routing = json.loads(Path(sys.argv[3]).read_text())
+routing = Path(sys.argv[3])
 proposal = Path(sys.argv[4])
-chooser = json.loads(Path(sys.argv[5]).read_text())
+chooser = Path(sys.argv[5])
 initial_mission = sys.argv[6]
 before = json.loads(Path(sys.argv[7]).read_text())
 tracked = [
@@ -131,14 +129,13 @@ tracked = [
     Path('.agent/verification-evidence.json'),
 ]
 current_state = json.loads(before['state.json'])
-assert current_state['mission_anchor'] == initial_mission, 'active /cook <hint> should start from the current mission anchor'
-assert routing['action'] == 'refocus', 'active /cook <hint> should route through active-workflow replacement assessment'
-assert routing['proposedMissionAnchor'] == 'Replacement mission that should stay in the main chat.', 'active /cook <hint> should preserve the hinted replacement mission'
-assert not proposal.exists(), 'active /cook <hint> chooser cancel should not open final proposal confirmation'
-assert chooser['choices'][1].startswith('Start new workflow from recent discussion'), 'active /cook <hint> should open the existing-workflow chooser'
-assert 'Cancelled existing workflow confirmation.' in output, 'active /cook <hint> chooser cancel should report cancellation'
+assert current_state['mission_anchor'] == initial_mission, 'active /cook inline-args rejection should start from the current mission anchor'
+assert not routing.exists(), 'active /cook inline-args rejection should not run active-workflow routing'
+assert not proposal.exists(), 'active /cook inline-args rejection should not open final startup-brief confirmation'
+assert not chooser.exists(), 'active /cook inline-args rejection should not open the existing-workflow chooser'
+assert '/cook no longer accepts inline arguments.' in output, 'active /cook inline-args rejection should explain the bare-only entry contract'
 after = {path.name: path.read_text() for path in tracked}
-assert before == after, 'active /cook <hint> chooser cancel should leave canonical files unchanged'
+assert before == after, 'active /cook inline-args rejection should leave canonical files unchanged'
 PY
 
 SESSION_INITIAL_REFOCUS="$TMPDIR/session-initial-bare-refocus.jsonl"
@@ -175,6 +172,7 @@ assert profile['evaluation_profile'] == expected_eval_profile, 'profile.json eva
 assert state['mission_anchor'] == new_anchor, 'state.json mission_anchor mismatch after refocus'
 assert state['task_type'] == expected_task_type, 'state.json task_type mismatch after refocus'
 assert state['evaluation_profile'] == expected_eval_profile, 'state.json evaluation_profile mismatch after refocus'
+assert state['advisory_startup_brief']['mission'] == new_anchor, 'refocus should preserve the confirmed startup brief as advisory intake'
 assert plan['mission_anchor'] == new_anchor, 'plan.json mission_anchor mismatch after refocus'
 assert plan['task_type'] == expected_task_type, 'plan.json task_type mismatch after refocus'
 assert plan['evaluation_profile'] == expected_eval_profile, 'plan.json evaluation_profile mismatch after refocus'
@@ -338,6 +336,7 @@ assert profile['evaluation_profile'] == expected_eval_profile, 'profile.json eva
 assert state['mission_anchor'] == new_anchor, 'state.json mission_anchor mismatch after bare refocus'
 assert state['task_type'] == expected_task_type, 'state.json task_type mismatch after bare refocus'
 assert state['evaluation_profile'] == expected_eval_profile, 'state.json evaluation_profile mismatch after bare refocus'
+assert state['advisory_startup_brief']['mission'] == new_anchor, 'bare refocus should preserve the confirmed startup brief as advisory intake'
 assert plan['mission_anchor'] == new_anchor, 'plan.json mission_anchor mismatch after bare refocus'
 assert plan['task_type'] == expected_task_type, 'plan.json task_type mismatch after bare refocus'
 assert plan['evaluation_profile'] == expected_eval_profile, 'plan.json evaluation_profile mismatch after bare refocus'

package/scripts/release-check.sh

@@ -4,8 +4,9 @@ set -euo pipefail
 ROOT="$(cd "$(dirname "${BASH_SOURCE[0]}")/.." && pwd)"
 cd "$ROOT"
 
-echo "[release-check] running control-plane validation, explicit-/cook parity, startup/refocus/context regressions, canonical evidence artifact, active-slice contract, observability, legacy cleanup, evaluator calibration, and rubric contract coverage"
+echo "[release-check] running control-plane validation, tracked .agent contract coverage, slice-surface parity, explicit-/cook parity, startup/refocus/context regressions, canonical evidence artifact, active-slice contract, observability, legacy cleanup, evaluator calibration, and rubric contract coverage"
 bash .agent/verify_completion_control_plane.sh
+git ls-files --error-unmatch .agent/README.md .agent/mission.md .agent/profile.json .agent/verify_completion_stop.sh .agent/verify_completion_control_plane.sh >/dev/null
 
 echo "[release-check] verifying public /cook parity and explicit-entry docs/help"
 python3 - <<'PY'
@@ -15,24 +16,24 @@ from pathlib import Path
 checks = {
     "README.md": [
         "`/cook` is the explicit workflow boundary for starting, continuing, refocusing, or beginning the next round of long-running repo work.",
-        "Only explicit `/cook` or `/cook <hint>` enters the workflow. Ordinary prompts stay in the main chat and go straight to the primary agent.",
+        "Only explicit `/cook` enters the workflow. Ordinary prompts stay in the main chat and go straight to the primary agent.",
         "`/cook` is the canonical workflow boundary and manual entry point",
         "Discuss the concrete repo change in the main chat, then run `/cook`",
+        "The confirmed startup brief is also preserved there as advisory intake for later re-grounding.",
     ],
     "CHANGELOG.md": [
-        "removed workflow-aware prompt interception so only explicit `/cook` or `/cook <hint>` enters the workflow; ordinary prompts now always stay on the main chat path",
-        "updated docs and release checks to describe explicit `/cook` entry instead of router-managed natural-language takeover",
+        "made `/cook` derive a confirmable startup brief from recent discussion before any canonical workflow rewrite, then preserve the confirmed brief in canonical state as advisory intake for later re-grounding",
+        "removed inline `/cook` arguments from the shipped entry path again so explicit bare `/cook` is the only public command, and fail closed when recent discussion is insufficient or unreliable",
     ],
     "extensions/completion/index.ts": [
-        'description: "/cook workflow: start, continue, refocus, or start the next round from an explicit /cook command"',
-        'const COOK_BARE_ONLY_GUIDANCE =',
-        '"/cook is the canonical workflow boundary. Discuss the concrete repo changes in the main chat, then run /cook when you want to start, continue, refocus, or begin the next workflow round."',
-        '"/cook failed closed because recent discussion did not produce a clear execution-ready Mission/Scope/Constraints/Acceptance proposal for concrete repo changes. Clarify the concrete repo changes in the main chat and rerun /cook."',
+        'description: "/cook workflow: derive a startup brief from recent discussion, then start, continue, refocus, or start the next round from the explicit /cook command"',
+        '"/cook failed closed because recent discussion did not produce a clear execution-ready startup brief with Mission/Scope/Constraints/Acceptance for concrete repo changes. Clarify the concrete repo changes in the main chat and rerun /cook."',
     ],
 }
 
 forbidden = {
     "README.md": [
+        "`/cook <hint>`",
         "Natural-language routing is optional and shipped in two modes",
         "PI_COMPLETION_TRIGGER_MODE",
         "workflow-aware router",
@@ -41,9 +42,8 @@ forbidden = {
     ],
     "CHANGELOG.md": ["compatibility" + " shim"],
     "extensions/completion/index.ts": [
-        'description: "/cook workflow: start, continue, refocus, or start the next round; /cook stays canonical while natural-language routing can be off or router"',
-        '"/cook remains the canonical workflow boundary. Natural-language routing can stay off or run in router mode to review each non-bypass user turn before implementation starts, but the shared /cook flow still owns mission selection and confirmation."',
-        '"/cook failed closed because recent discussion did not produce a clear execution-ready Mission/Scope/Constraints/Acceptance proposal for concrete repo changes. Router mode only offers the same /cook flow, and router recovery only replays to normal chat when you explicitly choose Send as normal chat, so clarify the concrete repo changes in the main chat and rerun /cook."',
+        'description: "/cook workflow: start, continue, refocus, or start the next round from an explicit /cook command"',
+        '"/cook failed closed because recent discussion did not produce a clear execution-ready Mission/Scope/Constraints/Acceptance proposal for concrete repo changes. Clarify the concrete repo changes in the main chat and rerun /cook."',
         'handleCookNaturalLanguageTrigger',
     ],
 }

package/scripts/role-runner-contract-test.sh

@@ -21,10 +21,9 @@ const assertNotIncludes = (file, snippet) => {
   }
 };
 
-assertIncludes('extensions/completion/role-runner.ts', 'import { completionRootKey, findCompletionRoot, findRepoRoot, loadCompletionDataForReminder } from "./state-store";');
+assertIncludes('extensions/completion/role-runner.ts', 'import { completionRootKey, findCompletionRoot, findRepoRoot } from "./state-store";');
 assertIncludes('extensions/completion/role-runner.ts', 'import { parseReportFields, transcribeRoleOutput, type TranscriptionResult } from "./transcription";');
 assertIncludes('extensions/completion/role-runner.ts', 'const agent = await loadAgentDefinition(params.root, params.role);');
-assertIncludes('extensions/completion/role-runner.ts', 'await loadCompletionDataForReminder(params.root);');
 assertIncludes('extensions/completion/role-runner.ts', 'const systemPromptTemp = await writeTempFile(params.root, "pi-completion-role-", agent.systemPrompt);');
 assertIncludes('extensions/completion/role-runner.ts', 'const reportFields = parseReportFields(output);');
 assertIncludes('extensions/completion/role-runner.ts', 'const transcription = exitCode === 0 ? await transcribeRoleOutput(params.role, params.root, output, reportFields) : undefined;');

package/scripts/rubric-contract-test.sh

@@ -83,7 +83,6 @@ assertIncludes('extensions/completion/prompt-surfaces.ts', '`- task_type: ${deps
 assertIncludes('extensions/completion/prompt-surfaces.ts', '`- evaluation_profile: ${deps.currentEvaluationProfile(snapshot) ?? "(missing)"}`');
 assertIncludes('extensions/completion/prompt-surfaces.ts', 'Canonical evaluation handoff for ${role}:');
 assertIncludes('extensions/completion/index.ts', 'buildEvaluationRoleReminderText(snapshot, nextRole)');
-assertIncludes('extensions/completion/index.ts', 'import { parseFirstNumber, parseYesNo } from "./transcription";');
 assertIncludes('extensions/completion/role-runner.ts', 'import { parseReportFields, transcribeRoleOutput, type TranscriptionResult } from "./transcription";');
 assertIncludes('extensions/completion/transcription.ts', 'roleReporting.transcribeCanonicalRoleReport');
 assertIncludes('extensions/completion/role-reporting.js', 'Missing Rubric heading for ${role}.');