Skip to content

Fix Claude and Codex follow-up progress reporting - #16217

Merged
seemeroland merged 1 commit into
masterfrom
roland/fix-claude-code-inprogress
Oct 1, 2026
Merged

seemeroland merged 1 commit into
masterfrom
roland/fix-claude-code-inprogress

Conversation

@seemeroland

@seemeroland seemeroland commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

Description

Explicit prompt submissions now bypass cached INPROGRESS deduplication for Claude Code and plugin-backed Codex. A harness can call finish-task before its stop hook fires, leaving the server completed while the client cache still records INPROGRESS. A follow-up in that interval must report progress again rather than trust the cache.

Forced updates retain their boundaries through the serialized queue, including behind in-flight requests. Other status updates and conversation-token deduplication keep their existing behavior. No server or plugin changes are required.

This addresses suppressed progress reporting only. The original turn's late stop can still close a newer follow-up; turn ordering remains separate work. Codex's legacy OSC 9 fallback is unchanged because it emits no prompt-start event.

Linked Issue

Investigated from staging run 01a0f2e7-b6d5-7579-bfd2-f7a6a071d7d9.

  • The linked issue is labeled ready-to-spec or ready-to-implement. No tracked issue is linked; the run above is the incident reference.
  • Where appropriate, screenshots or a short video of the implementation are included below. No visual capture; this changes lifecycle reporting.

Testing

The original issue isn't easily reproducible due to timing, but tested locally with e2e tests a few times and tests passed on the hot follow up

Agent Mode

  • Warp Agent Mode - This PR was created via Warp's AI Agent Mode

Changelog Entries for Stable

CHANGELOG-OZ: Claude Code and Codex follow-ups now report resumed activity even when the client retains a stale in-progress state.

Agent artifacts

Investigation and implementation conversation

Co-Authored-By: Warp agent@warp.dev

Always report INPROGRESS for explicit Claude and Codex prompt submissions, preserving prompt boundaries through the task update queue even when cached state is stale after finish-task.

Co-Authored-By: Warp <agent@warp.dev>
@cla-bot cla-bot Bot added the cla-signed label Sep 30, 2026
@seemeroland
seemeroland marked this pull request as ready for review September 30, 2026 22:05
@warp-for-oss

warp-for-oss Bot commented Sep 30, 2026 •

Copy link
Copy Markdown
Contributor

@seemeroland

I'm starting a first review of this pull request.

You can view the conversation on Warp.

I completed the review and no human review was requested for this pull request.

Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@warp-for-oss warp-for-oss Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview

This PR threads prompt-submit status context through CLI agent session updates so explicit Claude Code and plugin-backed Codex follow-ups can force a fresh InProgress task update even when the local delivered-state cache still says InProgress. The queue changes preserve existing token and normal progress deduplication while adding focused tests for cached progress, in-flight updates, ordering, and token behavior.

Concerns

No blocking concerns found. The added/changed comments explain non-obvious cache and lifecycle behavior rather than restating the code, and the new coverage is at the appropriate unit/model level for the changed queue and event-propagation logic. No approved spec context was available, and the security pass found no security-relevant issues.

Verdict

Found: 0 critical, 0 important, 0 suggestions

Approve

Comment /warp-agent-review on this pull request to retrigger a review (up to 3 times on the same pull request).

Powered by Oz

@harryalbert harryalbert left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@seemeroland
seemeroland merged commit db1c7a5 into master Oct 1, 2026
59 of 60 checks passed
@seemeroland
seemeroland deleted the roland/fix-claude-code-inprogress branch October 1, 2026 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants