[PR #111] [MERGED] ci: enrol in the team-gated multi-model Cursor review #111

New issue

Closed

opened 2026-10-31 22:04:39 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-31 22:04:39 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/Comfy-Org/comfy-typescript-sdk/pull/111
Author: @mattmillerai
Created: 8/31/2026
Status: ✅ Merged
Merged: 8/31/2026
Merged by: @mattmillerai

Base: main ← Head: matt/enrol-cursor-review

📝 Commits (3)

9641740 ci: enrol in the team-gated multi-model Cursor review
a2b5f36 ci: allowlist the public Comfy-Org/github-workflows repo in the hygiene scanner
6b557c9 ci: scope the generated-globs assets entry to the one binary it means

📊 Changes

2 files changed (+82 additions, -0 deletions)

View changed files

➕ .github/workflows/ci-cursor-review.yml (+75 -0)
📝 scripts/check-public-repo-hygiene.mjs (+7 -0)

📄 Description

What

Ports comfy-python-sdk's ci-cursor-review.yml — the thin, label-triggered caller for the reusable Comfy-Org/github-workflows cursor-review workflow (same SHA pin, ffcc3f5). Repo-specific change: extra_generated_globs swaps the python generated file for this repo's src/low/generated/** (machine-produced by @hey-api/openapi-ts from the vendored spec), so regenerations stay out of both the panel diff and the size cap while the hand-written src/low/src/sdk files stay reviewable.

Access control is unchanged from the pattern: label application needs triage+, and fork PRs never see CURSOR_API_KEY (org-level secret).

Verified

Hygiene scanner clean
Workflow is a verbatim port apart from the documented globs block; the reusable workflow owns gate/panel/judge behavior

🤖 Generated with Claude Code

_{🔄 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/comfy-typescript-sdk/pull/111 **Author:** [@mattmillerai](https://github.com/mattmillerai) **Created:** 8/31/2026 **Status:** ✅ Merged **Merged:** 8/31/2026 **Merged by:** [@mattmillerai](https://github.com/mattmillerai) **Base:** `main` ← **Head:** `matt/enrol-cursor-review` --- ### 📝 Commits (3) - [`9641740`](https://github.com/Comfy-Org/comfy-typescript-sdk/commit/9641740d6f1db0c77f98882082acf111d6d83cda) ci: enrol in the team-gated multi-model Cursor review - [`a2b5f36`](https://github.com/Comfy-Org/comfy-typescript-sdk/commit/a2b5f36cb897563462523f7ccff00237f2526cb6) ci: allowlist the public Comfy-Org/github-workflows repo in the hygiene scanner - [`6b557c9`](https://github.com/Comfy-Org/comfy-typescript-sdk/commit/6b557c95bb71029fa199f0cf31c67b679e0118ab) ci: scope the generated-globs assets entry to the one binary it means ### 📊 Changes **2 files changed** (+82 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `.github/workflows/ci-cursor-review.yml` (+75 -0) 📝 `scripts/check-public-repo-hygiene.mjs` (+7 -0) </details> ### 📄 Description ## What Ports comfy-python-sdk's `ci-cursor-review.yml` — the thin, label-triggered caller for the reusable `Comfy-Org/github-workflows` cursor-review workflow (same SHA pin, `ffcc3f5`). Repo-specific change: `extra_generated_globs` swaps the python generated file for this repo's `src/low/generated/**` (machine-produced by @hey-api/openapi-ts from the vendored spec), so regenerations stay out of both the panel diff and the size cap while the hand-written `src/low`/`src/sdk` files stay reviewable. Access control is unchanged from the pattern: label application needs triage+, and fork PRs never see `CURSOR_API_KEY` (org-level secret). ## Verified - Hygiene scanner clean - Workflow is a verbatim port apart from the documented globs block; the reusable workflow owns gate/panel/judge behavior 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-31 22:04:40 +00:00