pi-agenticoding/03: restore post-handoff proceed

Author: grzegorznowak

CHANGELOG.md

@@ -9,7 +9,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Changed
 
-- **Breaking:** handoff no longer auto-sends `Proceed.` after compaction and no longer supports configurable auto-resume. The superseded `handoff.resumeBehavior` (`"wait"`/`"proceed"`) setting is ignored.
+- Handoff keeps auto-sending `Proceed.` after successful compaction, matching core Pi behavior, while the superseded `handoff.resumeBehavior` (`"wait"`/`"proceed"`) setting is ignored and no longer configures continuation.
 - Added `handoff.automaticEnabled` raw settings support with JSON boolean values. Missing settings default to automatic handoff enabled; `false` removes the agent-facing handoff tool and handoff-call guidance during normal turns while preserving explicit `/handoff <direction>`.
 - Added the extension-owned `/agenticoding-settings` TUI panel for automatic handoff availability. TUI saves are global-only to `~/.pi/agent/settings.json`, preserve unrelated settings keys, persist real booleans, and visibly warn when a project override masks the global value.
 - Manual `/handoff <direction>` now waits behind the scenes when the assistant is streaming, then enables the `handoff` tool and starts a fresh handoff turn once idle. Visible stale-cleanup diagnostics have been removed from this flow, including TUI warnings and `agenticoding-handoff-diagnostic` conversation messages.

README.md

@@ -52,7 +52,7 @@ That's it. Your agent now has `spawn`, `notebook_write`, `notebook_read`, `noteb
 |---------|-------------------|
 | **Context usage %** | `ctx 65%` in status bar — green < 30%, yellow < 50%, orange < 70%, red ≥ 70% |
 | **Notebook count** | 📒 `3` when pages exist, dim `📒 0` when empty |
-| **`/handoff` command** | Explicit manual pivot — agent drafts brief, compacts context, then waits for your next input |
+| **`/handoff` command** | Explicit manual pivot — agent drafts brief, compacts context, then Pi sends `Proceed.` in the fresh context |
 | **`/agenticoding-settings` command** | TUI panel for global `handoff.automaticEnabled`, with project override warnings |
 | **`/notebook` command** | Overlay showing all notebook pages with previews |
 | **Auto-rehydration** | Notebook pages survive session restarts |
@@ -117,7 +117,7 @@ A sparse pocket notebook the agent curates while working. After discovering some
 
 When context degrades or the job changes, the agent saves reusable state to the notebook, writes a focused brief preserving what's still missing, and restarts clean. The new context starts with the brief front-and-center, all notebook pages accessible, and zero noise.
 
-By default, automatic handoff is enabled: the agent can see the `handoff` tool and may use it at context/job boundaries. Handoff completion always waits for your next explicit input; there is no configurable auto-`Proceed.` behavior.
+By default, automatic handoff is enabled: the agent can see the `handoff` tool and may use it at context/job boundaries. After successful handoff compaction, Pi auto-sends `Proceed.` so the fresh context continues immediately; this continuation is fixed, not configurable.
 
 To make handoff human-driven only, set `handoff.automaticEnabled` to `false` in raw Pi settings JSON. Supported persisted values are JSON booleans `true` and `false`; missing settings default to `true`.
 
@@ -127,11 +127,11 @@ To make handoff human-driven only, set `handoff.automaticEnabled` to `false` in
 }
 ```
 
-Settings are read from `~/.pi/agent/settings.json` and `<project>/.pi/settings.json`, with project settings overriding global settings. When automatic handoff is disabled, the agent-facing `handoff` tool and handoff-call guidance are removed from normal turns. The explicit operator command `/handoff <direction>` still works from idle or busy prompts: if the assistant is streaming, it waits behind the scenes until the current run is idle, temporarily enables the tool for a fresh requested handoff turn, compacts, restores the disabled state, and then waits for your next input.
+Settings are read from `~/.pi/agent/settings.json` and `<project>/.pi/settings.json`, with project settings overriding global settings. When automatic handoff is disabled, the agent-facing `handoff` tool and handoff-call guidance are removed from normal turns. The explicit operator command `/handoff <direction>` still works from idle or busy prompts: if the assistant is streaming, it waits behind the scenes until the current run is idle, temporarily enables the tool for a fresh requested handoff turn, compacts, restores the disabled state, and auto-sends `Proceed.` after successful compaction.
 
 Run `/agenticoding-settings` to change the global value from the TUI. It saves global-only to `~/.pi/agent/settings.json`, preserves unrelated JSON keys, shows the effective runtime value separately, and warns when a project override masks the global value. Setting changes affect future fresh agent turns; manual `/handoff` uses that same rule by waiting for idle before enabling the tool and starting its own fresh turn. Edit or remove project overrides manually.
 
-Migration note: the superseded PR-only `handoff.resumeBehavior` (`"wait"`/`"proceed"`) setting is ignored and cannot trigger automatic continuation. Remove it when convenient.
+Migration note: the superseded PR-only `handoff.resumeBehavior` (`"wait"`/`"proceed"`) setting is ignored and cannot change the fixed post-compaction `Proceed.` continuation. Remove it when convenient.
 
 **Rule of thumb:** The notebook holds reusable learned knowledge. Handoff carries the remaining situational context.
 

agenticoding.test.ts

@@ -408,7 +408,7 @@ test("/handoff requires a direction", async () => {
 	assert.deepEqual(pi.sentUserMessages, []);
 });
 
-test("handoff automatic setting defaults to enabled without automatic continuation", async () => {
+test("handoff automatic setting defaults to enabled with post-compaction Proceed", async () => {
 	const pi = new MockPi();
 	const state = createState();
 	state.notebookPages.set("auth-refresh", "sensitive notebook body");
@@ -441,19 +441,19 @@ test("handoff automatic setting defaults to enabled without automatic continuati
 	assert.equal(result.terminate, true);
 
 	compactOptions.onComplete({});
-	assert.deepEqual(pi.sentUserMessages, []);
+	assert.deepEqual(pi.sentUserMessages, [{ content: "Proceed.", options: undefined }]);
 	assert.ok(pi.activeTools.includes("handoff"));
 });
 
-test("handoff automatic setting true keeps handoff active without automatic continuation", async () => {
+test("handoff automatic setting true keeps handoff active and proceeds after compaction", async () => {
 	const result = await runHandoffResumeScenario({
 		globalSettings: { handoff: { automaticEnabled: false } },
 		projectSettings: { handoff: { automaticEnabled: true } },
 	});
 
 	assert.ok(result.compactOptions);
 	result.compactOptions.onComplete({});
-	assert.deepEqual(result.sentUserMessages, []);
+	assert.deepEqual(result.sentUserMessages, [{ content: "Proceed.", options: undefined }]);
 	assert.deepEqual(result.notifications, []);
 	assert.ok(result.activeTools.includes("handoff"));
 });
@@ -558,14 +558,14 @@ test("handoff automatic setting non-ENOENT read errors are treated as invalid so
 	});
 });
 
-test("handoff resumeBehavior is ignored and cannot trigger automatic continuation", async () => {
+test("handoff resumeBehavior is ignored and completion still uses fixed Proceed", async () => {
 	const result = await runHandoffResumeScenario({
 		globalSettings: { handoff: { resumeBehavior: "proceed" } },
 	});
 
 	assert.ok(result.compactOptions);
 	result.compactOptions.onComplete({});
-	assert.deepEqual(result.sentUserMessages, []);
+	assert.deepEqual(result.sentUserMessages, [{ content: "Proceed.", options: undefined }]);
 	assert.deepEqual(result.notifications, []);
 });
 
@@ -593,6 +593,9 @@ test("manual slash handoff temporarily activates handoff when automatic handoff
 		});
 		assert.ok(compactOptions);
 		assert.equal(state.pendingRequestedHandoff?.toolCalled, true);
+
+		compactOptions.onComplete({});
+		assert.equal(pi.sentUserMessages.at(-1)?.content, "Proceed.");
 	});
 });
 

handoff/tool.ts

@@ -98,7 +98,9 @@ export function registerHandoffTool(
 				manualRequest.toolCalled = true;
 			}
 			ctx.compact({
-				onComplete: () => {},
+				onComplete: () => {
+					pi.sendUserMessage("Proceed.");
+				},
 				onError: () => {
 					state.pendingHandoff = null;
 					state.pendingRequestedHandoff = null;

settings.ts

@@ -316,7 +316,7 @@ export function getAgenticodingSettingsDisplayLines(model: AgenticodingSettingsM
 		`Supported values: true, false. Default: true (automatic handoff enabled).`,
 		`When false, the agent-facing handoff tool is inactive for normal turns; manual /handoff <direction> still works from an idle prompt.`,
 		`Setting changes affect future fresh agent turns; in-flight queued follow-ups keep their existing tool schema.`,
-		`Handoff completion waits for your next explicit input; handoff.resumeBehavior is ignored and no automatic Proceed. message is sent.`,
+		`After successful handoff compaction, Pi auto-sends Proceed.; handoff.resumeBehavior is ignored and cannot change that behavior.`,
 		`Global settings: ${model.state.global.path} (${model.state.global.invalid ? "invalid JSON" : describeValue(model.state.global.automaticEnabled)})`,
 		`Project settings: ${model.state.project.path} (${model.state.project.invalid ? "invalid JSON" : describeValue(model.state.project.automaticEnabled)})`,
 		`TUI saves are global-only; project settings override global settings at runtime.`,