[PR #58] Add native cursor-review XL mode #67

New issue

Open

opened 2026-10-26 22:35:53 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-26 22:35:53 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/github-workflows/pull/58
Author: @synap5e
Created: 7/22/2026
Status: 🔄 Open

Base: main ← Head: agent/cursor-review-xl

📝 Commits (1)

f060e1d feat(cursor-review): add XL size-cap bypass

📊 Changes

4 files changed (+100 additions, -27 deletions)

View changed files

📝 .github/cursor-review/README.md (+19 -8)
📝 .github/cursor-review/post-review.py (+1 -1)
📝 .github/workflows/cursor-review.yml (+79 -17)
📝 README.md (+1 -1)

📄 Description

Summary

add cursor-review-xl as a standalone trigger for the reusable Cursor Review workflow
bypass only the diff-size cap in XL mode while preserving exclusions, line-count reporting, idempotency, and failure handling
make XL win when both review labels are present
centralize concurrency so XL replaces an in-flight standard panel without unrelated label events cancelling reviews
document the trigger and update retry guidance for either active review label

Why

Large-review behavior belongs in the reusable workflow so consumers keep thin callers and do not need to duplicate trigger or concurrency expressions.

Validation

python3 -m unittest discover -s .github/cursor-review/tests -p 'test_*.py' -v (40 tests)
actionlint .github/workflows/cursor-review.yml
git diff --check
exercised the embedded trigger gate across 11 standard, XL, both-label, skip-removal, unrelated-label, removal, and label-free event cases

Rollout

After merge, consumers create the cursor-review-xl label and update their pinned reusable-workflow SHA.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/Comfy-Org/github-workflows/pull/58 **Author:** [@synap5e](https://github.com/synap5e) **Created:** 7/22/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `agent/cursor-review-xl` --- ### 📝 Commits (1) - [`f060e1d`](https://github.com/Comfy-Org/github-workflows/commit/f060e1da7a86cd2ee91f730a60390b781715043a) feat(cursor-review): add XL size-cap bypass ### 📊 Changes **4 files changed** (+100 additions, -27 deletions) <details> <summary>View changed files</summary> 📝 `.github/cursor-review/README.md` (+19 -8) 📝 `.github/cursor-review/post-review.py` (+1 -1) 📝 `.github/workflows/cursor-review.yml` (+79 -17) 📝 `README.md` (+1 -1) </details> ### 📄 Description ## Summary - add `cursor-review-xl` as a standalone trigger for the reusable Cursor Review workflow - bypass only the diff-size cap in XL mode while preserving exclusions, line-count reporting, idempotency, and failure handling - make XL win when both review labels are present - centralize concurrency so XL replaces an in-flight standard panel without unrelated label events cancelling reviews - document the trigger and update retry guidance for either active review label ## Why Large-review behavior belongs in the reusable workflow so consumers keep thin callers and do not need to duplicate trigger or concurrency expressions. ## Validation - `python3 -m unittest discover -s .github/cursor-review/tests -p 'test_*.py' -v` (40 tests) - `actionlint .github/workflows/cursor-review.yml` - `git diff --check` - exercised the embedded trigger gate across 11 standard, XL, both-label, skip-removal, unrelated-label, removal, and label-free event cases ## Rollout After merge, consumers create the `cursor-review-xl` label and update their pinned reusable-workflow SHA. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi added the

pull-request

label

2026-10-26 22:35:53 +00:00

bsakaguchi referenced this issue

2026-10-26 22:35:59 +00:00

[PR #67] [MERGED] feat(groom): builder authors the PR body per the comfy-pr convention (BE-4346) #76