[Yash-Singh1]

Problem

The sidebar can keep showing a pull request as a draft after it becomes ready. Its saved status only refreshed when the PR changed between open, closed, or merged, so changes to isDraft were missed.

Change

Track draft status alongside the PR state when detecting changes. Refresh linked thread snapshots when draft status changes, and include draft status in reports from connected environments.

Scope and approval

This is a focused fix for an obvious stale sidebar status. The thread details menu already reads the current draft status, but the sidebar's saved snapshot did not update when only that status changed.

Verification

Ran 162 focused tests, package typechecks for the server, client runtime, and contracts, and lint on the changed files. Lint reported existing warnings. Browser verification was not run.

[macroscopeapp]

Approvability

Verdict: Approved at 1e4d9ce

Macroscope's review found this PR approvable — This is a narrowly scoped fix to propagate pull request draft-status transitions through the existing server, routing, and synchronization paths. The contract addition is optional and backward-compatible, and the production behavior is covered by focused tests without changing defaults or static-analysis configuration.

No code changes detected at b6f33dc. Prior analysis still applies.

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

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Path: .coderabbit.config.ts
• Review profile: CHILL
• Plan: Team
• Run ID: 781a0ed8-3bf7-427b-9bc7-281826f77634

📥 Commits

Reviewing files that changed from the base of the PR and between 96f09c6 and b6f33dc.

📒 Files selected for processing: 8

• apps/server/src/git/GitManager.test.ts
• apps/server/src/git/GitManager.ts
• apps/server/src/orchestration-v2/PullRequestSyncReactor.test.ts
• apps/server/src/pullRequest/PullRequestService.test.ts
• apps/server/src/pullRequest/PullRequestService.ts
• packages/client-runtime/src/state/pullRequestRouting.ts
• packages/client-runtime/src/state/pullRequests.test.ts
• packages/contracts/src/pullRequest.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.

📝 Walkthrough

Pull request state reporting and change detection now include optional draft status. The server publishes changes when draft status changes, deduplicates reports using both state and draft status, and synchronizes updated status.

Changes: Draft-status propagation

Layer / File(s)	Summary
Status contract and routed propagation packages/contracts/src/pullRequest.ts, packages/client-runtime/src/state/pullRequestRouting.ts, packages/client-runtime/src/state/pullRequests.test.ts	The report-state contract accepts optional isDraft. Client routing forwards it when present, and detail-read tests cover absent, true, and false values.
Server-side draft change detection apps/server/src/pullRequest/PullRequestService.ts, apps/server/src/git/GitManager.ts, apps/server/src/pullRequest/PullRequestService.test.ts, apps/server/src/git/GitManager.test.ts	Pull request detail readings and Git lookup state include draft status. The server publishes notifications when state or draft status changes. Tests cover draft transitions and stale reads.
State report deduplication and sync apps/server/src/pullRequest/PullRequestService.ts, apps/server/src/pullRequest/PullRequestService.test.ts, apps/server/src/orchestration-v2/PullRequestSyncReactor.test.ts	The server deduplicates reports using both state and draft status. Reactor tests verify cache invalidation and synchronization with the updated draft status.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Change: Bug fix

Sequence Diagram(s)

sequenceDiagram
  participant ClientRuntime
  participant PullRequestService
  participant PullRequestSyncReactor
  participant SyncCommand
  ClientRuntime->>PullRequestService: Report state with optional isDraft
  PullRequestService->>PullRequestSyncReactor: Publish state-change event
  PullRequestSyncReactor->>PullRequestSyncReactor: Invalidate pull request cache
  PullRequestSyncReactor->>SyncCommand: Dispatch updated isDraft value

Loading

Suggested reviewers: t3dotgg, flamboh

Merge Risk: ⚪ Minimal · up to b6f33

Draft-status propagation is mergeable after normal checks; no outstanding issue is supported by the reviewed evidence.

Pre-merge checks: 4

Check name	Status	Explanation
Title check		The title is concise, uses the repository's conventional format, and clearly describes syncing pull request draft status changes.
Description check		The description includes all required sections. It explains the stale-status problem, the implementation change, the focused scope justification, and the verification performed, including limitations.
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	5.0 KiB	5.0 KiB	+7 B (+0.1%)	6.8 KiB	✅
Codex	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Codex	Live turn WebSocket wire	1.2 KiB	1.2 KiB	+7 B (+0.6%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.9 KiB	20.9 KiB	0 B (0.0%)	29.3 KiB	✅
Codex	Live turn messages	2	2	0 (0.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	−7 B (−0.1%)	6.8 KiB	✅
Claude	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Claude	Live turn WebSocket wire	1.2 KiB	1.2 KiB	−7 B (−0.6%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.2 KiB	21.2 KiB	0 B (0.0%)	29.3 KiB	✅
Claude	Live turn messages	2	2	0 (0.0%)	8	✅

Baseline: 96f09c6 · PR result: b6f33dc · 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: 108.5 KiB
• Claude decoded thread snapshot: 108.8 KiB

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