Fix plan expect handling on terminal navigation

Author: cursoragent

packages/agent/src/translator/browser-act.ts

@@ -101,7 +101,7 @@ export async function runBrowserAct(action: CuaActionBrowserAct, runtime: Browse
 	}
 
 	let finalExpectation: BrowserActExpectationEvidence | undefined;
-	const terminalNavigation = stopReason === "navigation" && stoppedAt === finalStepIndex && steps.at(-1)?.outcome === "worked";
+	const terminalNavigation = stopReason === "navigation" && stoppedAt === finalStepIndex;
 	if (action.expect && (!stopReason || terminalNavigation)) {
 		try {
 			const result = await runtime.wait(action.expect!, baseline, baseline.targetId, action.tab_id, action.timeout_ms, action.poll_ms);
@@ -163,7 +163,8 @@ export async function runBrowserAct(action: CuaActionBrowserAct, runtime: Browse
 	const verified = action.expect
 		? finalExpectation?.status === "newly_verified"
 		: steps.length === action.steps.length && steps.every((step) => step.outcome === "worked");
-	const verifiedNavigation = stopReason === "navigation" && steps.length === action.steps.length && steps.at(-1)?.outcome === "worked";
+	const verifiedNavigation = stopReason === "navigation" && steps.length === action.steps.length
+		&& (steps.at(-1)?.outcome === "worked" || finalExpectation?.status === "newly_verified");
 	return {
 		outcome: failed ? "didnt" : !timedOut && verified && (!stopReason || verifiedNavigation) ? "worked" : "unknown",
 		steps,

packages/agent/test/translator-browser.test.ts

@@ -185,6 +185,17 @@ describe("browser_act orchestration", () => {
 		expect(result).toMatchObject({ outcome: "worked", stopped_at: 0, stop_reason: "navigation", final_expectation: { status: "newly_verified" } });
 	});
 
+	it("evaluates plan expectations after terminal navigation without step expectations", async () => {
+		const before = observation("before");
+		const after = observation("after", 1);
+		const result = await runBrowserAct({
+			type: "browser_act",
+			steps: [{ type: "click", ref: "e1" }],
+			expect: { type: "url", contains: "after" },
+		}, runtime([before, before, after, after], [waitResult("newly_verified")]));
+		expect(result).toMatchObject({ outcome: "worked", stopped_at: 0, stop_reason: "navigation", final_expectation: { status: "newly_verified" } });
+	});
+
 	it("recollects a successor after a raced target boundary", async () => {
 		const result = await runBrowserAct({ type: "browser_act", steps: [{ type: "click", ref: "e1", expect: { type: "text", text: "Done" } }] }, runtime([
 			observation("before"), observation("before"), observation("after"), observation("stale"), observation("stable"),