Skip to content

fix(cursor): use resolved OAuth model for runs#132

Merged
kaitranntt merged 1 commit into
mainfrom
kai/fix/cursor-oauth-alias-124
Jul 10, 2026
Merged

fix(cursor): use resolved OAuth model for runs#132
kaitranntt merged 1 commit into
mainfrom
kai/fix/cursor-oauth-alias-124

Conversation

@kaitranntt

Copy link
Copy Markdown
Owner

Replaces the oauth-model-alias portion of contributor PR #124 (original commit 59b83e0) with a narrow, tested fix.

The conductor already resolves a Cursor OAuth alias into req.Model. Cursor Run construction now sends that resolved upstream model on every initial, auth-migration, and cold-resume path, while preserving the client alias in parsed.Model for OpenAI-compatible responses.

Coverage verifies alias override, empty and whitespace fallback, missing client model, and non-mutation. Full go test, build, formatting, diff check, and review passed locally.

Tool-stream changes from #124 are intentionally excluded: there is no safe reproducer and the contributor identified unresolved index/resume/DONE/timeout defects.

Replaces the oauth-model-alias portion of contributor PR #124 (59b83e0) with a focused boundary fix and regression coverage.
@kaitranntt kaitranntt merged commit 3aae95a into main Jul 10, 2026
2 checks passed
@kaitranntt kaitranntt deleted the kai/fix/cursor-oauth-alias-124 branch July 10, 2026 15:10
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.

1 participant