[yordis]

• A wake could describe checks or conflicts for a commit that had already been replaced by a newer push, sending the agent to fix problems that no longer exist.
• The agent needs to know which commit the findings are about so it can tell stale results from current ones.

[cursor]

PR Summary

Medium Risk
Changes orchestration watch sweeps and agent wake content; incorrect behavior could miss wakes or send agents to fix obsolete commits, but scope is limited to PR monitoring.

Overview
Fixes pull request watches waking the agent with stale host data or missing conflict news after the PR head moves.

The watch reactor now always invalidates cached PR reads before each planned detail/activity fetch (not only when a watch fingerprint exists), so sweeps do not reuse data from an older push or from a viewer read.

Conflict handling in evaluatePullRequestWatch treats a push as resetting mergeability uncertainty: a conflict on a new head is reported even if the previous head was already conflicting, and unknown mergeability no longer keeps the old head’s conflict flag across a head change.

Wake copy now states which short SHA the findings refer to and tells the agent to confirm the PR head is still that commit before acting. Tests cover invalidate-then-detail when fingerprint is null, conflict-on-replaced-head, the new message line, and reactor mocks that stub invalidate.

^{Reviewed by Cursor Bugbot for commit 2ea27de. Bugbot is set up for automated code reviews on this repo. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 21e8cf5. Configure here.}

[coderabbitai]

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 39 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration

• Source mode: Auto
• Repository: TrogonStack/t3code
• Branch: yordis/fix-watch-wakes-follow-head
• File: .coderabbit.config.ts
• Commit: 2ea27de
• Layers: Repository TS config
• Review profile: CHILL
• Plan: Advanced
• Run ID: 17739969-7a12-4ddc-af64-a329746ffa26

Commits

Reviewing files that changed from the base of the PR and between e46e6db and 2ea27de.

Files selected for processing: 7

• apps/server/src/orchestration-v2/Orchestrator.ts
• apps/server/src/orchestration-v2/PullRequestWatchReactor.test.ts
• apps/server/src/orchestration-v2/PullRequestWatchReactor.ts
• apps/server/src/orchestration-v2/pullRequestWatch.test.ts
• apps/server/src/orchestration-v2/pullRequestWatch.ts
• apps/server/src/orchestration-v2/runtimeLayer.test.ts
• evals/mcp-tool-use/cases/pull-requests.ndjson

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

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

• X
• Mastodon
• Reddit
• LinkedIn

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