Skip to content

Fix Claude auth normalization and Studio flags - #27

Merged
armand0e merged 1 commit into
mainfrom
codex/pr18-safety-fixes
Jul 10, 2026
Merged

Fix Claude auth normalization and Studio flags#27
armand0e merged 1 commit into
mainfrom
codex/pr18-safety-fixes

Conversation

@armand0e

Copy link
Copy Markdown
Collaborator

Follow-up to #18 for the two unresolved P2 review findings that could not be pushed to the contributor fork:

  • normalize configured Claude OAuth placeholders before enabling subscription auth or validating api.base_url
  • share Claude effort/fallback flag construction with Studio native terminal sessions
  • add regression coverage for both paths

Validation:

  • pytest -q tests/test_config.py tests/test_runner.py tests/test_studio.py — 275 passed
  • full suite on the combined Add Claude Code subscription auth and passthroughs #18 + fixes tree with a fixed wide console — 570 passed, 25 skipped
  • ruff check on changed files — passed
  • wheel build — passed

@cursor

cursor Bot commented Jul 10, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@armand0e
armand0e merged commit 39ec4b8 into main Jul 10, 2026
3 checks passed
@armand0e
armand0e deleted the codex/pr18-safety-fixes branch July 10, 2026 06:57
@CompactAIOfficial CompactAIOfficial added bug Something isn't working triaged labels Aug 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working triaged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants