fix: resolve Analyze (Go) undefined config.Config in cliproxyctl by KooshaPari · Pull Request #634 · KooshaPari/cliproxyapi-plusplus

KooshaPari · 2026-02-26T22:13:48Z

Summary:\n- switch cmd/cliproxyctl/main.go import from pkg/llmproxy/config to internal/config\n- align command executor signatures with the Config type used by pkg/llmproxy/cmd\n\nWhy:\n- PR #617 fails Analyze (Go) with undefined: config.Config at cmd/cliproxyctl/main.go:33:22

- #210: Add cmd to Bash required fields for Ampcode compatibility - #206: Remove type uppercasing that breaks nullable type arrays Fixes #210 Fixes #206

Cherry-picked from merge/1698-strip-empty-messages-openai-to-claude into aligned base

(cherry picked from commit a172fad)

(cherry picked from commit 3473184)

Co-authored-by: Codex <noreply@openai.com>

gemini-code-assist · 2026-02-26T22:13:51Z

Warning

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

coderabbitai · 2026-02-26T22:14:00Z

Important

Review skipped

Too many files!

This PR contains 300 files, which is 150 over the limit of 150.

📥 Commits

Reviewing files that changed from the base of the PR and between f20d681 and cf31ac7.

📒 Files selected for processing (300)

.github/workflows/codeql.yml
.github/workflows/docs.yml
.github/workflows/pr-path-guard.yml
.github/workflows/pr-test-build.yml
.github/workflows/vitepress-pages.yml
.worktrees/config/m/config-build/active/internal/config/sdk_config.go
cliproxyctl/main.go
cliproxyctl/main_test.go
cmd/cliproxyctl/main.go
cmd/cliproxyctl/main_test.go
cmd/codegen/main.go
cmd/server/config_validate.go
cmd/server/main.go
examples/custom-provider/main.go
examples/http-request/main.go
examples/translator/main.go
go.mod
internal/access/config_access/provider.go
internal/access/reconcile.go
internal/api/handlers/management/api_tools.go
internal/api/handlers/management/api_tools_test.go
internal/api/handlers/management/auth_files.go
internal/api/handlers/management/config_basic.go
internal/api/handlers/management/config_lists.go
internal/api/handlers/management/handler.go
internal/api/handlers/management/logs.go
internal/api/handlers/management/model_definitions.go
internal/api/handlers/management/usage.go
internal/api/handlers/management/vertex_import.go
internal/api/middleware/request_logging.go
internal/api/middleware/response_writer.go
internal/api/modules/amp/amp.go
internal/api/modules/amp/amp_test.go
internal/api/modules/amp/fallback_handlers.go
internal/api/modules/amp/fallback_handlers_test.go
internal/api/modules/amp/model_mapping.go
internal/api/modules/amp/model_mapping_test.go
internal/api/modules/amp/proxy_test.go
internal/api/modules/amp/routes.go
internal/api/modules/amp/routes_test.go
internal/api/modules/amp/secret.go
internal/api/modules/amp/secret_test.go
internal/api/modules/modules.go
internal/api/server.go
internal/api/server_test.go
internal/auth/antigravity/auth.go
internal/auth/claude/anthropic_auth.go
internal/auth/claude/token.go
internal/auth/claude/utls_transport.go
internal/auth/codex/openai_auth.go
internal/auth/codex/token.go
internal/auth/copilot/copilot_auth.go
internal/auth/copilot/oauth.go
internal/auth/copilot/token.go
internal/auth/gemini/gemini_auth.go
internal/auth/gemini/gemini_token.go
internal/auth/iflow/iflow_auth.go
internal/auth/iflow/iflow_token.go
internal/auth/kilo/kilo_token.go
internal/auth/kimi/kimi.go
internal/auth/kimi/token.go
internal/auth/kiro/aws_auth.go
internal/auth/kiro/background_refresh.go
internal/auth/kiro/codewhisperer_client.go
internal/auth/kiro/oauth.go
internal/auth/kiro/oauth_web.go
internal/auth/kiro/refresh_manager.go
internal/auth/kiro/social_auth.go
internal/auth/kiro/sso_oidc.go
internal/auth/kiro/usage_checker.go
internal/auth/qwen/qwen_auth.go
internal/auth/qwen/qwen_token.go
internal/auth/vertex/vertex_credentials.go
internal/cmd/anthropic_login.go
internal/cmd/antigravity_login.go
internal/cmd/auth_manager.go
internal/cmd/github_copilot_login.go
internal/cmd/iflow_cookie.go
internal/cmd/iflow_login.go
internal/cmd/kilo_login.go
internal/cmd/kimi_login.go
internal/cmd/kiro_login.go
internal/cmd/login.go
internal/cmd/openai_device_login.go
internal/cmd/openai_login.go
internal/cmd/qwen_login.go
internal/cmd/run.go
internal/cmd/vertex_import.go
internal/config/config.go
internal/config/config_generated.go
internal/config/provider_registry_generated.go
internal/interfaces/types.go
internal/logging/gin_logger.go
internal/logging/global_logger.go
internal/logging/request_logger.go
internal/managementasset/updater.go
internal/registry/model_registry.go
internal/runtime/executor/aistudio_executor.go
internal/runtime/executor/antigravity_executor.go
internal/runtime/executor/antigravity_executor_buildrequest_test.go
internal/runtime/executor/claude_executor.go
internal/runtime/executor/claude_executor_test.go
internal/runtime/executor/codex_executor.go
internal/runtime/executor/codex_websockets_executor.go
internal/runtime/executor/gemini_cli_executor.go
internal/runtime/executor/gemini_executor.go
internal/runtime/executor/gemini_vertex_executor.go
internal/runtime/executor/github_copilot_executor.go
internal/runtime/executor/github_copilot_executor_test.go
internal/runtime/executor/iflow_executor.go
internal/runtime/executor/iflow_executor_test.go
internal/runtime/executor/kilo_executor.go
internal/runtime/executor/kimi_executor.go
internal/runtime/executor/kiro_executor.go
internal/runtime/executor/logging_helpers.go
internal/runtime/executor/openai_compat_executor.go
internal/runtime/executor/openai_compat_executor_compact_test.go
internal/runtime/executor/payload_helpers.go
internal/runtime/executor/proxy_helpers.go
internal/runtime/executor/qwen_executor.go
internal/runtime/executor/qwen_executor_test.go
internal/runtime/executor/thinking_providers.go
internal/runtime/executor/usage_helpers.go
internal/store/gitstore.go
internal/store/objectstore.go
internal/store/postgresstore.go
internal/thinking/apply.go
internal/thinking/convert.go
internal/thinking/provider/antigravity/apply.go
internal/thinking/provider/claude/apply.go
internal/thinking/provider/codex/apply.go
internal/thinking/provider/gemini/apply.go
internal/thinking/provider/geminicli/apply.go
internal/thinking/provider/iflow/apply.go
internal/thinking/provider/kimi/apply.go
internal/thinking/provider/openai/apply.go
internal/thinking/types.go
internal/thinking/validate.go
internal/translator/antigravity/claude/antigravity_claude_request.go
internal/translator/antigravity/claude/antigravity_claude_request_test.go
internal/translator/antigravity/claude/antigravity_claude_response.go
internal/translator/antigravity/claude/antigravity_claude_response_test.go
internal/translator/antigravity/claude/init.go
internal/translator/antigravity/gemini/antigravity_gemini_request.go
internal/translator/antigravity/gemini/init.go
internal/translator/antigravity/openai/chat-completions/antigravity_openai_request.go
internal/translator/antigravity/openai/chat-completions/antigravity_openai_response.go
internal/translator/antigravity/openai/chat-completions/init.go
internal/translator/antigravity/openai/responses/antigravity_openai-responses_request.go
internal/translator/antigravity/openai/responses/antigravity_openai-responses_response.go
internal/translator/antigravity/openai/responses/init.go
internal/translator/claude/gemini-cli/claude_gemini-cli_request.go
internal/translator/claude/gemini-cli/claude_gemini-cli_response.go
internal/translator/claude/gemini-cli/init.go
internal/translator/claude/gemini/claude_gemini_request.go
internal/translator/claude/gemini/init.go
internal/translator/claude/openai/chat-completions/claude_openai_request.go
internal/translator/claude/openai/chat-completions/init.go
internal/translator/claude/openai/responses/claude_openai-responses_request.go
internal/translator/claude/openai/responses/init.go
internal/translator/codex/claude/codex_claude_request.go
internal/translator/codex/claude/init.go
internal/translator/codex/gemini-cli/codex_gemini-cli_request.go
internal/translator/codex/gemini-cli/codex_gemini-cli_response.go
internal/translator/codex/gemini-cli/init.go
internal/translator/codex/gemini/codex_gemini_request.go
internal/translator/codex/gemini/init.go
internal/translator/codex/openai/chat-completions/init.go
internal/translator/codex/openai/responses/init.go
internal/translator/gemini-cli/claude/gemini-cli_claude_request.go
internal/translator/gemini-cli/claude/init.go
internal/translator/gemini-cli/gemini/gemini-cli_gemini_request.go
internal/translator/gemini-cli/gemini/init.go
internal/translator/gemini-cli/openai/chat-completions/gemini-cli_openai_request.go
internal/translator/gemini-cli/openai/chat-completions/gemini-cli_openai_response.go
internal/translator/gemini-cli/openai/chat-completions/init.go
internal/translator/gemini-cli/openai/responses/gemini-cli_openai-responses_request.go
internal/translator/gemini-cli/openai/responses/gemini-cli_openai-responses_response.go
internal/translator/gemini-cli/openai/responses/init.go
internal/translator/gemini/claude/gemini_claude_request.go
internal/translator/gemini/claude/init.go
internal/translator/gemini/gemini-cli/gemini_gemini-cli_request.go
internal/translator/gemini/gemini-cli/init.go
internal/translator/gemini/gemini/gemini_gemini_request.go
internal/translator/gemini/gemini/init.go
internal/translator/gemini/openai/chat-completions/gemini_openai_request.go
internal/translator/gemini/openai/chat-completions/init.go
internal/translator/gemini/openai/responses/gemini_openai-responses_request.go
internal/translator/gemini/openai/responses/init.go
internal/translator/init.go
internal/translator/kiro/claude/init.go
internal/translator/kiro/claude/kiro_claude_request.go
internal/translator/kiro/claude/kiro_claude_response.go
internal/translator/kiro/claude/kiro_claude_stream.go
internal/translator/kiro/claude/kiro_claude_tools.go
internal/translator/kiro/claude/kiro_websearch_handler.go
internal/translator/kiro/claude/tool_compression.go
internal/translator/kiro/claude/truncation_detector.go
internal/translator/kiro/openai/init.go
internal/translator/kiro/openai/kiro_openai.go
internal/translator/kiro/openai/kiro_openai_request.go
internal/translator/kiro/openai/kiro_openai_response.go
internal/translator/kiro/openai/kiro_openai_stream.go
internal/translator/openai/claude/init.go
internal/translator/openai/claude/openai_claude_request.go
internal/translator/openai/claude/openai_claude_response.go
internal/translator/openai/gemini-cli/init.go
internal/translator/openai/gemini-cli/openai_gemini_request.go
internal/translator/openai/gemini-cli/openai_gemini_response.go
internal/translator/openai/gemini/init.go
internal/translator/openai/gemini/openai_gemini_request.go
internal/translator/openai/openai/chat-completions/init.go
internal/translator/openai/openai/responses/init.go
internal/translator/translator/translator.go
internal/usage/logger_plugin.go
internal/util/provider.go
internal/util/proxy.go
internal/util/util.go
internal/watcher/clients.go
internal/watcher/config_reload.go
internal/watcher/diff/auth_diff.go
internal/watcher/diff/config_diff.go
internal/watcher/diff/config_diff_test.go
internal/watcher/diff/model_hash.go
internal/watcher/diff/model_hash_test.go
internal/watcher/diff/models_summary.go
internal/watcher/diff/oauth_excluded.go
internal/watcher/diff/oauth_excluded_test.go
internal/watcher/diff/oauth_model_alias.go
internal/watcher/diff/openai_compat.go
internal/watcher/diff/openai_compat_test.go
internal/watcher/dispatcher.go
internal/watcher/events.go
internal/watcher/synthesizer/config.go
internal/watcher/synthesizer/config_test.go
internal/watcher/synthesizer/context.go
internal/watcher/synthesizer/file.go
internal/watcher/synthesizer/file_test.go
internal/watcher/synthesizer/helpers.go
internal/watcher/synthesizer/helpers_test.go
internal/watcher/synthesizer/interface.go
internal/watcher/watcher.go
internal/watcher/watcher_test.go
pkg/llmproxy/access/config_access/provider.go
pkg/llmproxy/access/config_access/provider_test.go
pkg/llmproxy/access/reconcile.go
pkg/llmproxy/api/aliases.go
pkg/llmproxy/api/handlers/management/api_tools.go
pkg/llmproxy/api/handlers/management/api_tools_test.go
pkg/llmproxy/api/handlers/management/auth_files.go
pkg/llmproxy/api/handlers/management/config_basic.go
pkg/llmproxy/api/handlers/management/config_lists.go
pkg/llmproxy/api/handlers/management/handler.go
pkg/llmproxy/api/handlers/management/logs.go
pkg/llmproxy/api/handlers/management/management_auth_test.go
pkg/llmproxy/api/handlers/management/management_basic_test.go
pkg/llmproxy/api/handlers/management/management_extra_test.go
pkg/llmproxy/api/handlers/management/management_fields_test.go
pkg/llmproxy/api/handlers/management/management_modelstates_test.go
pkg/llmproxy/api/handlers/management/model_definitions.go
pkg/llmproxy/api/handlers/management/routing_select.go
pkg/llmproxy/api/handlers/management/usage.go
pkg/llmproxy/api/handlers/management/vertex_import.go
pkg/llmproxy/api/handlers/routing_handler.go
pkg/llmproxy/api/middleware/request_logging.go
pkg/llmproxy/api/middleware/request_logging_test.go
pkg/llmproxy/api/middleware/response_writer.go
pkg/llmproxy/api/middleware/response_writer_test.go
pkg/llmproxy/api/modules/amp/amp.go
pkg/llmproxy/api/modules/amp/amp_test.go
pkg/llmproxy/api/modules/amp/fallback_handlers.go
pkg/llmproxy/api/modules/amp/fallback_handlers_test.go
pkg/llmproxy/api/modules/amp/model_mapping.go
pkg/llmproxy/api/modules/amp/model_mapping_test.go
pkg/llmproxy/api/modules/amp/proxy_test.go
pkg/llmproxy/api/modules/amp/routes.go
pkg/llmproxy/api/modules/amp/routes_test.go
pkg/llmproxy/api/modules/amp/secret.go
pkg/llmproxy/api/modules/amp/secret_test.go
pkg/llmproxy/api/modules/modules.go
pkg/llmproxy/auth/antigravity/auth.go
pkg/llmproxy/auth/claude/anthropic_auth.go
pkg/llmproxy/auth/claude/token.go
pkg/llmproxy/auth/claude/utls_transport.go
pkg/llmproxy/auth/codex/openai_auth.go
pkg/llmproxy/auth/codex/openai_auth_test.go
pkg/llmproxy/auth/codex/token.go
pkg/llmproxy/auth/copilot/copilot_auth.go
pkg/llmproxy/auth/copilot/copilot_auth_test.go
pkg/llmproxy/auth/copilot/copilot_extra_test.go
pkg/llmproxy/auth/copilot/oauth.go
pkg/llmproxy/auth/copilot/token.go
pkg/llmproxy/auth/diff/auth_diff.go
pkg/llmproxy/auth/diff/config_diff.go
pkg/llmproxy/auth/diff/config_diff_test.go
pkg/llmproxy/auth/diff/diff_generated.go
pkg/llmproxy/auth/diff/model_hash.go
pkg/llmproxy/auth/diff/model_hash_test.go
pkg/llmproxy/auth/diff/models_summary.go
pkg/llmproxy/auth/diff/oauth_excluded.go

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch codex/pr617-analyze-go-config-fix

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.}

KooshaPari · 2026-02-27T08:03:46Z

Closing: duplicate branch with #638, both conflicting

KooshaPari and others added 12 commits February 26, 2026 02:22

chore: align module path to kooshapari fork

1509a48

fix: resolve cliproxyctl delegate build regressions

38320e6

ci: allow translator kiro websearch hotfix file in path guard

56117f8

fix: resolve executor compile regressions

4740b95

ci: branch-scope build and codeql for migrated router compatibility

6932352

fix: multiple issues

4d359cc

- #210: Add cmd to Bash required fields for Ampcode compatibility - #206: Remove type uppercasing that breaks nullable type arrays Fixes #210 Fixes #206

Strip empty messages on translation from openai to claude

a741598

Cherry-picked from merge/1698-strip-empty-messages-openai-to-claude into aligned base

Merge: fix/circular-import-config and refactor/consolidation

7244955

(cherry picked from commit a172fad)

fix(ci): align sdk config types and include auto-merge workflow

ede7dfd

(cherry picked from commit 3473184)

fix: resolve cliproxyctl delegate build regressions

3288641

fix: clean duplicate structs/tests and harden auth region/path handling

a1a96d2

fix(cliproxyctl): point CLI command wiring to internal config

cf31ac7

Co-authored-by: Codex <noreply@openai.com>

KooshaPari closed this Feb 27, 2026

KooshaPari deleted the codex/pr617-analyze-go-config-fix branch February 27, 2026 09:24

KooshaPari restored the codex/pr617-analyze-go-config-fix branch February 27, 2026 09:44

KooshaPari deleted the codex/pr617-analyze-go-config-fix branch February 27, 2026 09:47

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix: resolve Analyze (Go) undefined config.Config in cliproxyctl#634

fix: resolve Analyze (Go) undefined config.Config in cliproxyctl#634
KooshaPari wants to merge 12 commits intomainfrom
codex/pr617-analyze-go-config-fix

KooshaPari commented Feb 26, 2026

Uh oh!

gemini-code-assist bot commented Feb 26, 2026

Uh oh!

coderabbitai bot commented Feb 26, 2026

Review skipped

Uh oh!

KooshaPari commented Feb 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

KooshaPari commented Feb 26, 2026

Uh oh!

gemini-code-assist bot commented Feb 26, 2026

Uh oh!

coderabbitai bot commented Feb 26, 2026

Review skipped

Uh oh!

KooshaPari commented Feb 27, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant