@linimin/pi-letscook 0.1.72 → 0.1.73

package/CHANGELOG.md

@@ -1,12 +1,20 @@
 # Changelog
 
+## 0.1.73
+
+### Fixed
+
+- fixed `/cook` await-user-input resumptions so a user's exact reply in the active workflow can dispatch the mandatory follow-up completion role without forcing an extra `/cook` rerun
+- kept active-workflow completion-role gating strict for ordinary main-chat turns while adding regression coverage that fails if await-user-input replies lose their workflow dispatch rights
+
 ## 0.1.72
 
 ### Fixed
 
 - relaxed reviewer no-follow-up routing parsing so `Acceptable as-is: yes` now also accepts `none, proceed to completion-auditor` and `none - proceed to auditor` in addition to the original exact allowance, reducing avoidable completion transcription warnings without weakening the follow-up-slice guard
 - fixed completion-role continuation gating so an already-active `/cook` workflow with `continuation_policy: continue` can keep dispatching mandatory follow-up roles even when the harness no longer recognizes the current turn text as an explicit `/cook` or workflow-driver prompt, while still blocking ordinary main-chat turns from calling `completion_role`
-- added a dedicated `completion-role-gating-test` regression so release-check now fails if active-workflow continuation falls back to the old prompt-only dispatch gate or stops rejecting ordinary main-chat turns
+- fixed `/cook` await-user-input resumptions so a user's exact reply in the active workflow can dispatch the mandatory follow-up completion role without forcing an extra `/cook` rerun
+- added a dedicated `completion-role-gating-test` regression so release-check now fails if active-workflow continuation falls back to the old prompt-only dispatch gate, await-user-input replies lose workflow dispatch rights, or ordinary main-chat turns stop being rejected
 
 ## 0.1.71
 

package/extensions/completion/index.ts

@@ -274,6 +274,15 @@ function isOrdinaryMainChatTurnDuringActiveWorkflow(
 	return true;
 }
 
+function isAwaitingUserInputWorkflowReplyTurn(
+	snapshot: CompletionStateSnapshot | undefined,
+	ctx: { sessionManager?: any },
+): boolean {
+	if (!hasActiveWorkflowEntry(snapshot)) return false;
+	if (!isOrdinaryMainChatTurnDuringActiveWorkflow(snapshot, ctx)) return false;
+	return asString(snapshot?.state?.continuation_policy) === "await_user_input";
+}
+
 function isCompletionRoleDispatchAllowedTurn(
 	snapshot: CompletionStateSnapshot | undefined,
 	ctx: { sessionManager?: any },
@@ -281,6 +290,7 @@ function isCompletionRoleDispatchAllowedTurn(
 	if (hasCompletionRoutingActivation(snapshot)) return true;
 	if (!hasActiveWorkflowEntry(snapshot)) return false;
 	if (isCompletionWorkflowSessionTurn(snapshot, ctx)) return true;
+	if (isAwaitingUserInputWorkflowReplyTurn(snapshot, ctx)) return true;
 	if (isOrdinaryMainChatTurnDuringActiveWorkflow(snapshot, ctx)) return false;
 	return asString(snapshot?.state?.continuation_policy) === "continue";
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linimin/pi-letscook",
-  "version": "0.1.72",
+  "version": "0.1.73",
   "description": "Pi package for long-running completion workflows with canonical .agent state, role-based subagents, continuity, and verification helpers.",
   "license": "MIT",
   "private": false,

package/scripts/completion-role-gating-test.sh

@@ -23,13 +23,17 @@ const assertNotIncludes = (file, snippet) => {
 
 assertIncludes('extensions/completion/index.ts', 'function isOrdinaryMainChatTurnDuringActiveWorkflow(');
 assertIncludes('extensions/completion/index.ts', 'function isCompletionRoleDispatchAllowedTurn(');
+assertIncludes('extensions/completion/index.ts', 'function isAwaitingUserInputWorkflowReplyTurn(');
+assertIncludes('extensions/completion/index.ts', 'if (isAwaitingUserInputWorkflowReplyTurn(snapshot, ctx)) return true;');
 assertIncludes('extensions/completion/index.ts', 'if (isOrdinaryMainChatTurnDuringActiveWorkflow(snapshot, ctx)) return false;');
+assertIncludes('extensions/completion/index.ts', 'return asString(snapshot?.state?.continuation_policy) === "await_user_input";');
 assertIncludes('extensions/completion/index.ts', 'return asString(snapshot?.state?.continuation_policy) === "continue";');
 assertIncludes('extensions/completion/index.ts', 'const completionRoleDispatchAllowed = Boolean(role) || isCompletionRoleDispatchAllowedTurn(snapshot, ctx);');
 assertIncludes('extensions/completion/index.ts', 'if (isCookCommandTurn(ctx)) return false;');
 assertIncludes('extensions/completion/index.ts', 'if (isCompletionDriverPromptTurn(snapshot, ctx)) return false;');
 assertIncludes('extensions/completion/policy-guards.ts', 'return "completion_role may only be used from an active /cook workflow session.";');
 assertIncludes('CHANGELOG.md', 'fixed completion-role continuation gating so an already-active `/cook` workflow with `continuation_policy: continue` can keep dispatching mandatory follow-up roles');
+assertIncludes('CHANGELOG.md', 'fixed `/cook` await-user-input resumptions so a user\'s exact reply in the active workflow can dispatch the mandatory follow-up completion role without forcing an extra `/cook` rerun');
 
 assertNotIncludes(
   'extensions/completion/index.ts',
@@ -38,7 +42,11 @@ assertNotIncludes(
 
 const indexText = read('extensions/completion/index.ts');
 const ordinaryGuardIndex = indexText.indexOf('if (isOrdinaryMainChatTurnDuringActiveWorkflow(snapshot, ctx)) return false;');
+const awaitReplyAllowIndex = indexText.indexOf('if (isAwaitingUserInputWorkflowReplyTurn(snapshot, ctx)) return true;');
 const continueFallbackIndex = indexText.indexOf('return asString(snapshot?.state?.continuation_policy) === "continue";');
+if (awaitReplyAllowIndex === -1 || ordinaryGuardIndex === -1 || awaitReplyAllowIndex > ordinaryGuardIndex) {
+  throw new Error('extensions/completion/index.ts must allow active await_user_input reply turns before the ordinary main-chat rejection guard.');
+}
 if (ordinaryGuardIndex === -1 || continueFallbackIndex === -1 || ordinaryGuardIndex > continueFallbackIndex) {
   throw new Error('extensions/completion/index.ts must reject ordinary main-chat turns before allowing the continuation_policy=continue fallback.');
 }