Skip to content

hotfix: fix claude token build error + codeql v4#196

Merged
KooshaPari merged 1 commit intomainfrom
codescan-hotfix-main
Feb 23, 2026
Merged

hotfix: fix claude token build error + codeql v4#196
KooshaPari merged 1 commit intomainfrom
codescan-hotfix-main

Conversation

@KooshaPari
Copy link
Copy Markdown
Owner

@KooshaPari KooshaPari commented Feb 23, 2026

Fixes post-merge CI failures from PR #195.\n\nChanges:\n- remove duplicate safePath redeclaration in pkg/llmproxy/auth/claude/token.go\n- upgrade CodeQL workflow actions from v3 to v4 (init, analyze)\n\nValidation:\n- go test ./pkg/llmproxy/auth/claude -count=1\n


Note

Low Risk
Low risk CI/config update plus a small Go fix removing a duplicate variable/path-sanitization function; runtime behavior should be unchanged aside from using the shared path validation helper.

Overview
Upgrades the CodeQL GitHub Actions workflow to github/codeql-action@v4 for both init and analyze.

Fixes a Go build issue in Claude token persistence by removing a duplicate safePath redeclaration and dropping the local sanitizeTokenFilePath helper in favor of the shared misc.ResolveSafeFilePath validation.

Written by Cursor Bugbot for commit 569957f. This will update automatically on new commits. Configure here.

@KooshaPari KooshaPari merged commit 37d8a39 into main Feb 23, 2026
2 of 4 checks passed
@KooshaPari KooshaPari deleted the codescan-hotfix-main branch February 23, 2026 03:25
@KooshaPari KooshaPari restored the codescan-hotfix-main branch February 23, 2026 04:55
@KooshaPari KooshaPari deleted the codescan-hotfix-main branch February 23, 2026 05:12
KooshaPari added a commit that referenced this pull request Feb 23, 2026
hotfix: fix claude token build error + codeql v4
KooshaPari pushed a commit that referenced this pull request Feb 27, 2026
GitHub Copilot now supports claude-opus-4.6 but it was missing from
the proxy's model definitions. Fixes #196.
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