[AdeyaDev]

Two follow-ups on top of #35641. Both were fixed on #35622 after the commit this branch started from; these are smaller versions written for your code. One commit each, so you can merge this PR or cherry-pick either one.

1. A steer to a parent can complete a remote delegated task early (05918e3, +9/−4 + test)
RemoteDelegation's background follower calls PeerForwarding.waitForThread with the parent's scope, so it also watches the parent's steering. Steps:

1. A remote run finishes, but its delegated work is still running.
2. The remote wait reports completed, timedOut: true.
3. The user steers the parent.
4. The follower's wait returns completed with timedOut: false forced on, so the task completes early with the wrong reply.

Fix: only the agent-facing t3_thread_wait yields to input (yieldToInput = false by default, true from call).

2. OpenCode: a foreground subagent started after an accepted steer keeps the steer waiting (a9c6af6, +16/−12 + test)
steerTurn checks for active foreground calls only once, when the steer is accepted. If the model is still generating and then starts a foreground subagent in the same step, session.background is never sent. The steer sits in the inbox for the child's whole run.

Fix: the inline call moves into releaseForeground, which is also called from session.tool.called when a pending steer meets a new foreground call. It's forked because events are handled under the session lock. It also gets the REQUEST_REPLY_TIMEOUT the other requests in the file use; this was the only request without one.

Tests: the existing steering test now runs five variants, including "child starts after steer" and "background fails". There's a new PeerForwarding test for the follower case. Without the fixes, all four new cases fail; with them, the PeerForwarding and OpenCode2 adapter tests pass (182/182), and check:changed passes. The branch includes a merge of your current head (55d338a); the only conflict was the /background request assertion from main.

The forwarded-wait test fixture now releases its blocked remote wait when the test ends, rather than Effect.never. Without that, the new test hangs on teardown.

🤖 Generated with Claude Code

[macroscopeapp]

Approvability

Verdict: Approved at 82c3b4a

Macroscope's review found this PR approvable — Separates task-follower waits from agent steering and releases OpenCode foreground observation when a pending steer meets a newly started subagent. The changes are narrowly scoped to responsiveness, with bounded background requests and existing access controls preserved.

^{You can add or adjust custom eligibility rules. Learn more.}

[github-actions]

This PR changes the test suite by +1120 test cases (browser +103, unit +555, server +462).

New browser test files: apps/web/src/components/AppSidebarLayout.browser.test.tsx, apps/web/src/components/GitActionsControl.composition.browser.test.tsx, apps/web/src/components/RightPanelSheet.browser.test.tsx, apps/web/src/components/chat/ComposerPendingUserInputPanel.browser.test.tsx, apps/web/src/components/chat/DraftHeroHeadline.browser.test.tsx, apps/web/src/components/chat/MarkdownCodeBlock.browser.test.tsx, apps/web/src/components/chat/McpAppFrame.browser.test.tsx, apps/web/src/components/chat/useComposerStash.browser.test.tsx, apps/web/src/components/chat/useDraftEnvironmentRouting.browser.test.tsx, apps/web/src/components/cloud/CloudEnvironmentConnectList.browser.test.tsx, apps/web/src/components/diffs/DiffFileTree.browser.test.tsx, apps/web/src/components/diffs/StyledDiffCodeView.browser.test.tsx, apps/web/src/components/preview/ThreadPreviewMiniPlayer.browser.test.tsx, apps/web/src/components/settings/AddUsageLimitSourceDialog.browser.test.tsx, apps/web/src/components/settings/IntegrationsSettings.browser.test.tsx, apps/web/src/components/settings/LinkedEnvironmentsSettings.browser.test.tsx, apps/web/src/themeCss.browser.test.ts.
Each browser test file costs CI seconds of module loading before its first test runs; a unit test costs milliseconds.

Keep tests that would fail for a real reason, and test logic below the UI where it can be. See Testing.

[maria-rcks]

Note

Written by gpt-6.1-sol on behalf of Maria

merged current main without rebasing and resolved the peer-wait conflicts, preserving main's 45-second limit and the distinction between agent waits and task-follower waits. updated the context budget to the measured 49,732 bytes.

verified b1fb6fa9f6bd0fb6465a3fd5b3cbf50a9df1639d: blacksmith vp run check:changed github/main passed format, lint, knip, graphql, affected typechecks, and 328 tests. the previously failing workspace-search and pr-cache tests also pass. both independent reviewers approved this head. all 34 check contexts pass or are intentionally skipped; github reports no conflicts and an approved review decision.

the test-delta report includes main's tests because this stacked pr still targets factory/pr-35622; the takeover added no unrelated tests. live opencode provider integration and paid model evals remain unverified.