Skip to content

fix(auth): align codex sdk import types#625

Merged
KooshaPari merged 1 commit intomainfrom
fix/pr617-main-ci-fix3-sdk-auth-types
Feb 26, 2026
Merged

fix(auth): align codex sdk import types#625
KooshaPari merged 1 commit intomainfrom
fix/pr617-main-ci-fix3-sdk-auth-types

Conversation

@KooshaPari
Copy link
Copy Markdown
Owner

@KooshaPari KooshaPari commented Feb 26, 2026

Summary\n- switch to \n- remove internal/pkg codex type mismatch in build path

Summary by CodeRabbit

  • Chores
    • Reorganized internal dependencies to improve code structure and maintainability.

Use the llmproxy codex auth package in both login paths so buildAuthRecord receives consistent types.\n\nCo-authored-by: Codex <noreply@openai.com>
@gemini-code-assist
Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@KooshaPari KooshaPari merged commit 04fea06 into main Feb 26, 2026
1 of 4 checks passed
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Feb 26, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3513a01 and c1e01ce.

📒 Files selected for processing (1)
  • sdk/auth/codex.go

📝 Walkthrough

Walkthrough

The Codex package import in sdk/auth/codex.go is reorganized from the internal module path to the external llmproxy package path. The go.mod file is updated to reflect this change. No functional modifications are made.

Changes

Cohort / File(s) Summary
Import Path Reorganization
sdk/auth/codex.go, go.mod
Updated Codex package import from internal (v6/internal/auth/codex) to external location (v6/pkg/llmproxy/auth/codex). No behavioral changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 The Codex hops from shadows deep,
Out to where the packages keep,
From internal vaults to pkgs bright,
Reorganized, all is right!

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/pr617-main-ci-fix3-sdk-auth-types

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 golangci-lint (2.5.0)

Error: can't load config: the Go language version (go1.25) used to build golangci-lint is lower than the targeted Go version (1.26.0)
The command is terminated due to an error: can't load config: the Go language version (go1.25) used to build golangci-lint is lower than the targeted Go version (1.26.0)


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

This was referenced Feb 26, 2026
@KooshaPari KooshaPari deleted the fix/pr617-main-ci-fix3-sdk-auth-types branch February 27, 2026 09:25
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