mirror of
https://github.com/Comfy-Org/ComfyUI_frontend.git
synced 2026-07-17 09:18:26 +00:00
## Summary Adds a team-gated, label-triggered multi-model Cursor review as a **thin caller** for the reusable workflow in `Comfy-Org/github-workflows` — the single source of truth for the panel, judge, prompts, and scripts. This repo carries only the ~50-line caller, so there's no review logic to drift out of sync. ## Changes - **What**: `.github/workflows/pr-cursor-review.yaml` triggers on the `cursor-review` label and calls `Comfy-Org/github-workflows/.github/workflows/cursor-review.yml`, pinned to `047ca48` (github-workflows#9, current main). Inheriting the reusable workflow brings severity badges, line-anchored inline comments, diff-size caps, prompt-injection hardening, and optional Slack DMs. - **Config**: `diff_excludes` restated (overriding replaces the default wholesale) with this repo's heavy paths added (Playwright snapshots, generated manager types). Judge and panel both default to Opus 4.8 via the reusable workflow — no overrides needed. ## Review Focus - **Access control (the point).** Two layers, no allowlist: (1) only triage+ users can apply a label in a public repo; (2) the reusable workflow's secret-bearing jobs don't run on fork PRs, so `CURSOR_API_KEY` is reachable only on internal branches. - **Replaces a standalone draft.** Earlier revisions of this branch carried a self-contained workflow + review/judge scripts; that duplicated the reusable workflow, so it's been swapped for the thin caller. ## Prerequisites (already done) - `CURSOR_API_KEY` secret set on this repo. - `cursor-review` label created. - `SLACK_BOT_TOKEN` already present (enables the DM feature). --------- Co-authored-by: GitHub Action <action@github.com>
GitHub Workflows
Naming Convention
Workflow files follow a consistent naming pattern: <prefix>-<descriptive-name>.yaml
Category Prefixes
| Prefix | Purpose | Example |
|---|---|---|
ci- |
Testing, linting, validation | ci-tests-e2e.yaml |
release- |
Version management, publishing | release-version-bump.yaml |
pr- |
PR automation (triggered by labels) | pr-claude-review.yaml |
api- |
External Api type generation | api-update-registry-api-types.yaml |
i18n- |
Internationalization updates | i18n-update-core.yaml |
Documentation
Each workflow file contains comments explaining its purpose, triggers, and behavior. For specific details about what each workflow does, refer to the comments at the top of each .yaml file.
For GitHub Actions documentation, see Events that trigger workflows.