feat: add current-session steer command#76934
Conversation
|
Codex review: needs maintainer review before merge. Summary Reproducibility: not applicable. This is a new command feature rather than a reported bug; the review path is static diff inspection plus the focused command tests listed in the PR body. Next step before merge Security Review detailsBest possible solution: Land this after normal maintainer approval and required checks if the product decision to repurpose top-level Do we have a high-confidence way to reproduce the issue? Not applicable. This is a new command feature rather than a reported bug; the review path is static diff inspection plus the focused command tests listed in the PR body. Is this the best way to solve the issue? Yes. The proposed implementation is narrow: it reuses the existing active-run registry and queueing APIs, leaves child-run steering under subagent dispatch, and updates the registry/docs/tests for the changed command contract. What I checked:
Likely related people:
Codex review notes: model gpt-5.5, reasoning high; reviewed against 5355ef0f080b. |
a76acf9 to
27c3633
Compare
|
Landed via rebase onto
|
Summary
/steer <message>and/tell <message>handling for the current session active run./subagents steer <id|#> <message>.Verification
pnpm test src/auto-reply/reply/commands-steer.test.ts src/auto-reply/reply/commands-subagents-routing.test.ts src/auto-reply/reply/commands-subagents-steer.test.ts src/auto-reply/commands-registry.test.tspnpm exec oxfmt --check --threads=1 src/auto-reply/reply/commands-steer.ts src/auto-reply/reply/commands-steer.runtime.ts src/auto-reply/reply/commands-steer.test.ts src/auto-reply/reply/commands-handlers.runtime.ts src/auto-reply/reply/commands-subagents/shared.ts src/auto-reply/reply/commands-subagents-dispatch.ts src/auto-reply/reply/commands-subagents-routing.test.ts src/auto-reply/reply/commands-subagents-steer.test.ts src/auto-reply/commands-registry.shared.tsgit diff --checkpnpm check:docstbx_01kqqw2v7dnhcycta1f2yhwexe:pnpm check:changedpassed (syncDelegated=true, exit 0, 1m41s total)