fix(agents): allow Cursor product repository work#2410
Conversation
@coderabbitai review |
|
✅ Action performedReview finished.
|
|
Warning Review limit reached
Next review available in: 2 minutes Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available. How can I continue?After more reviews become available, a review can be triggered using the To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews. How do review limits work?CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability. For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window. Please refer docs for additional details. Review details⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (2)
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. Comment |
@codex review |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9935bee027
ℹ️ 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".
| > - **Product repositories are in scope when present in the cloud workspace.** Do not mistake an | ||
| > empty monorepo submodule directory for a lane boundary: a current Cursor tick demonstrably opened | ||
| > and delivered a `cursor/*` PR in `devantler-tech/platform-template` on 2026-07-22. Resolve the | ||
| > current checkout against the contract's Portfolio map, work in that product repository, and push | ||
| > the branch to that repository's origin. A repository absent from the workspace is an honest |
There was a problem hiding this comment.
Scan Cursor claims before enabling product work
When Cursor selects a product issue, it cannot self-assign, so until its draft exists the only visible claim is the pushed cursor/* branch. The checked .claude/agents/portfolio-surveyor.md still scans branches only for assigned-but-PR-less issues and filters them with grep '^claude/'; local siblings will therefore report that issue as actionable and can start a duplicate implementation during this window. Extend the surveyor and its contract test to discover cursor/* claims before widening Cursor to every product repository.
AGENTS.md reference: AGENTS.md:L362-L371
Useful? React with 👍 / 👎.
Closing this implementation after current-head Codex review found a valid coordination regression: widening the Cursor lane to product repositories increases the scope in which local siblings cannot see its pre-PR The obvious surveyor-only fix was already rejected in #2300 because a branch without Cursor's unavailable assignment has no valid lease clock and can park work indefinitely. The durable, fail-closed prerequisite is the lane-neutral arbitration in #2302. This branch remains recoverable, but it must not be promoted ahead of that contract. |
Why
The versioned Cursor loader still classified product-repository work as unavailable. Live behavior now disproves that deployment fact:
app/cursoropened and delivered platform-template#111 from acursor/*branch on 2026-07-22.What changed
Verification
portfolio-surveyor.test.shfailed withCursor loader still treats its empty boot checkout as a product-repository boundarybefore the loader change..claude/scripts/portfolio-surveyor.test.shbash -n .claude/scripts/portfolio-surveyor.test.shgit diff --checkFixes #2395