Skip to content
This repository was archived by the owner on Oct 10, 2026. It is now read-only.

chore: track upstream orchestrator V2 PR #2829 - #223

Merged
cursor[bot] merged 1 commit into
mainfrom
codex/track-orchestrator-2829-a13c
Aug 8, 2026
Merged

cursor[bot] merged 1 commit into
mainfrom
codex/track-orchestrator-2829-a13c

Conversation

@aaditagrawal

@aaditagrawal aaditagrawal commented Aug 8, 2026 •

Copy link
Copy Markdown
Owner

Summary

Start tracking upstream pingdotgg/t3code#2829 (feat(orchestrator): introduce new orchestrator) via the existing fork PR-track machinery.

  • Updates config/upstream-pr-tracks.json to follow #2829 on codex/pr-2829-track
  • Retires stale adapter tracks (#179 / #295 / #364) that already landed or closed upstream
  • Documents the track in docs/custom-alpha-workflow.md

How to refresh

npm run sync:upstream-prs
# or: node scripts/sync-upstream-pr-tracks.mjs

That fetches upstream pull/2829/head into codex/pr-2829-track and prints unique commits / diff stats vs main (and codex/alpha if present).

Notes

  • #2829 is still open, XXL (~190k+/80k-), and currently CONFLICTING with upstream main.
  • This PR only adds tracking metadata — it does not port orchestration V2 onto the fork yet.
  • The tracking tip is also pushed to origin/codex/pr-2829-track for shared refresh/compare.

Test plan

  • npm run sync:upstream-prs refreshes codex/pr-2829-track
  • Config JSON remains valid
  • Docs list matches config/upstream-pr-tracks.json
Open in Web Open in Cursor 

Summary by CodeRabbit

  • Documentation

    • Updated the custom workflow documentation to reflect the current upstream orchestrator V2 development track.
    • Clarified that previously tracked pull requests have been completed or closed upstream.
  • Chores

    • Refreshed internal pull request tracking information by adding the active orchestrator work and removing retired entries.

Replace retired adapter PR tracks with the long-running orchestration
rewrite so `sync:upstream-prs` keeps a local `codex/pr-2829-track` tip
for future ports.

Co-authored-by: aaditagrawal <aaditagrawal@users.noreply.github.com>
@github-actions github-actions Bot added vouch:trusted PR author is trusted by repo permissions or the VOUCHED list. size:S 10-29 effective changed lines (test files excluded in mixed PRs). labels Aug 8, 2026
@coderabbitai

coderabbitai Bot commented Aug 8, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: c814f7d6-98e0-494f-9479-8fef33954f3e

📥 Commits

Reviewing files that changed from the base of the PR and between 8dd08b6 and 282a294.

📒 Files selected for processing (2)
  • config/upstream-pr-tracks.json
  • docs/custom-alpha-workflow.md

📝 Walkthrough

Walkthrough

The tracked upstream PR list now includes PR #2829 for the orchestrator V2 rewrite. The workflow documentation records this active track and marks PRs #179, #295, and #364 as retired.

Changes

Upstream PR tracking

Layer / File(s) Summary
Update tracked upstream PRs
config/upstream-pr-tracks.json, docs/custom-alpha-workflow.md
The configuration now tracks PR #2829 with its branch, URL, and status notes. The workflow documentation records PR #2829 as active and marks older tracks as merged or closed.

Estimated code review effort: 1 (Trivial) | ~5 minutes

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the metadata change: tracking upstream orchestrator V2 PR #2829.
Description check ✅ Passed The description explains what changed, why, scope, refresh steps, and test plan, but it uses a Summary section instead of the template headings.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/track-orchestrator-2829-a13c

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@cursor
cursor Bot merged commit 012cdd1 into main Aug 8, 2026
12 checks passed
@github-actions

github-actions Bot commented Aug 8, 2026

Copy link
Copy Markdown

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider Metric Main baseline This PR Impact PR ceiling
Codex Total thread wire 11.4 KiB 11.4 KiB −6 B (−0.1%) 15.1 KiB ✅
Codex Thread snapshot wire 5.5 KiB 5.5 KiB +1 B (+0.0%) 7.3 KiB ✅
Codex Live turn WebSocket wire 5.9 KiB 5.9 KiB −7 B (−0.1%) 7.8 KiB ✅
Codex Live turn WebSocket decoded 49.8 KiB 49.8 KiB 0 B (0.0%) 66.4 KiB ✅
Codex Live turn messages 17 17 0 (0.0%) 21 ✅
Claude Total thread wire 11.4 KiB 11.4 KiB −29 B (−0.2%) 15.1 KiB ✅
Claude Thread snapshot wire 5.5 KiB 5.5 KiB −2 B (−0.0%) 7.3 KiB ✅
Claude Live turn WebSocket wire 5.9 KiB 5.9 KiB −27 B (−0.4%) 7.8 KiB ✅
Claude Live turn WebSocket decoded 50.6 KiB 50.6 KiB 0 B (0.0%) 66.4 KiB ✅
Claude Live turn messages 17 17 0 (0.0%) 21 ✅

Baseline: 8dd08b6 · PR result: 282a294 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

  • Codex decoded thread snapshot: 94.7 KiB
  • Claude decoded thread snapshot: 95.4 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size:S 10-29 effective changed lines (test files excluded in mixed PRs). vouch:trusted PR author is trusted by repo permissions or the VOUCHED list.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants