Skip to content

Use org CURSOR_MODEL for Cursor code review workflows - #69

Merged
YoungDan merged 1 commit into
mainfrom
cursor/cursor-model-org-var-ddd7
Sep 3, 2026
Merged

YoungDan merged 1 commit into
mainfrom
cursor/cursor-model-org-var-ddd7

Conversation

@YoungDan

@YoungDan YoungDan commented Sep 2, 2026

Copy link
Copy Markdown
Member

Summary

Fixes the hardcoded Cursor model in automated PR review workflows. Both cursor-code-review.yml and review-fix.yml now pass vars.CURSOR_MODEL to the deslicer-code-harness reusables, with a credit-safe default of auto when the org variable is unset (same pattern as DAP and mcp-for-splunk).

Changes

  • cursor-code-review.yml: model: ${{ vars.CURSOR_MODEL || 'auto' }} instead of gpt-5.3-codex
  • review-fix.yml: same org var for @slicer-fix instead of gpt-5.2
  • .github/workflows/README.md: document CURSOR_MODEL as the org Actions variable source

Notes

  • Keeps the existing deslicer-code-harness @v1 reusable workflows — only the model input changes.
  • Does not touch install.sh, auth code, or other harness workflows (auto-fix, issue-triage).
  • actionlint passes locally on the changed workflow files.

Test plan

  • Workflow Syntax Check (actionlint) passes on this PR
  • Quality Gate unchanged (no Rust changes)
Open in Web Open in Cursor 

@YoungDan
YoungDan marked this pull request as ready for review September 3, 2026 06:44
@YoungDan
YoungDan requested a review from a team as a code owner September 3, 2026 06:44
Pass vars.CURSOR_MODEL (default auto) to the code-harness review and
review-fix reusable workflows instead of hardcoded model ids. Document
the org Actions variable in .github/workflows/README.md.

Co-authored-by: Daniel Young <YoungDan@users.noreply.github.com>
@YoungDan
YoungDan force-pushed the cursor/cursor-model-org-var-ddd7 branch from a82e0b2 to f2b1495 Compare September 3, 2026 07:31
@YoungDan
YoungDan merged commit 6204c7d into main Sep 3, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants