Skip to content

Report incremental command progress - #108

Merged
ariobarin merged 3 commits into
mainfrom
perf/command-progress
Jul 28, 2026
Merged

Report incremental command progress#108
ariobarin merged 3 commits into
mainfrom
perf/command-progress

Conversation

@ariobarin

Copy link
Copy Markdown
Owner

Summary

  • return bounded stdout and stderr while commands are still running
  • add monotonic command progress sequences and change-aware get_command waits
  • negotiate command-progress support during worker registration
  • preserve current-worker and legacy single-worker fallback during staggered rollout

Stack

Verification

  • git diff --check
  • npm run check
  • review loop pass 1: found capability-negotiation rollout issue
  • review loop pass 2: clean after capability gating and fallback tests

@vercel

vercel Bot commented Jul 27, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated (UTC)
glossa Ignored Ignored Preview Jul 28, 2026 3:56pm

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. What shall we delve into next?

Reviewed commit: 5cd522b1e6

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Base automatically changed from perf/explicit-command-routing to main July 28, 2026 15:28
@ariobarin
ariobarin force-pushed the perf/command-progress branch from 5cd522b to 2457dfd Compare July 28, 2026 15:35

Copy link
Copy Markdown
Owner Author

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 2457dfdc54

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/protocol.md

Copy link
Copy Markdown
Owner Author

@codex review

@ariobarin
ariobarin force-pushed the perf/command-progress branch from c87c8af to e1499c2 Compare July 28, 2026 15:56
@ariobarin
ariobarin merged commit bbc6c35 into main Jul 28, 2026
19 checks passed
@ariobarin
ariobarin deleted the perf/command-progress branch July 28, 2026 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant