chore: update CLIProxyAPI v7 replace - #758
Conversation
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthrough更新 ChangesGo 模块更新
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested reviewers: Poem Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@go.mod`:
- Line 128: Update the go.mod replacement for
github.com/router-for-me/CLIProxyAPI/v7 to a compatible revision that exports
sdk/exec.XAIExecutor and sdk/exec.NewXAIExecutor, preserving the existing
references in cliproxyapi_grok adapter.go; alternatively, rewrite that adapter
to use the replacement SDK’s equivalent exported API and verify the project
builds.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 8c298961-8fa6-437d-948c-00b0ac333722
⛔ Files ignored due to path filters (1)
go.sumis excluded by!**/*.sum
📒 Files selected for processing (1)
go.mod
📜 Review details
⏰ Context from checks skipped due to timeout. (1)
- GitHub Check: playwright
⚠️ CI failures not shown inline (4)
GitHub Actions: E2E Tests / 0_e2e.txt: chore: update CLIProxyAPI v7 replace
Conclusion: failure
##[group]Run go test -v -count=1 -timeout 300s -race ./tests/e2e/...
�[36;1mgo test -v -count=1 -timeout 300s -race ./tests/e2e/...�[0m
shell: /usr/bin/bash -e {0}
##[endgroup]
go: downloading github.com/golang-jwt/jwt/v5 v5.3.1
go: downloading github.com/tidwall/gjson v1.18.0
go: downloading golang.org/x/crypto v0.54.0
go: downloading github.com/gorilla/websocket v1.5.3
go: downloading github.com/gin-gonic/gin v1.10.1
go: downloading github.com/awsl-project/CLIProxyAPI/v7 v7.0.0-20260728045746-2dcc3bab4250
go: downloading github.com/tidwall/sjson v1.2.5
go: downloading github.com/andybalholm/brotli v1.2.0
go: downloading github.com/google/uuid v1.6.0
go: downloading github.com/klauspost/compress v1.18.3
go: downloading github.com/redis/go-redis/v9 v9.19.0
go: downloading github.com/glebarez/sqlite v1.11.0
go: downloading github.com/go-sql-driver/mysql v1.8.1
go: downloading golang.org/x/sync v0.22.0
go: downloading gorm.io/driver/mysql v1.6.0
go: downloading gorm.io/driver/postgres v1.6.0
go: downloading gorm.io/gorm v1.31.1
go: downloading github.com/go-webauthn/webauthn v0.16.0
go: downloading github.com/tidwall/match v1.1.1
go: downloading github.com/tidwall/pretty v1.2.0
go: downloading github.com/gin-contrib/sse v0.1.0
go: downloading github.com/mattn/go-isatty v0.0.20
go: downloading golang.org/x/net v0.57.0
go: downloading github.com/sirupsen/logrus v1.9.3
go: downloading github.com/aws/aws-sdk-go-v2 v1.41.5
go: downloading github.com/aws/aws-sdk-go-v2/credentials v1.19.14
go: downloading github.com/bytedance/sonic v1.15.0
go: downloading github.com/glebarez/go-sqlite v1.21.2
go: downloading modernc.org/sqlite v1.23.1
go: downloading filippo.io/edwards25519 v1.1.1
go: downloading github.com/jackc/pgx/v5 v5.9.2
go: downloading github.com/jinzhu/now v1.1.5
go: downloading github.com/jinzhu/inflection v1.0.0
go: downloading golang.org/x/text v0.40.0
go: downloading github.com/go-viper/mapstructure/v2 v2.5.0
go: download...
GitHub Actions: E2E Tests / e2e: chore: update CLIProxyAPI v7 replace
Conclusion: failure
##[group]Run go test -v -count=1 -timeout 300s -race ./tests/e2e/...
�[36;1mgo test -v -count=1 -timeout 300s -race ./tests/e2e/...�[0m
shell: /usr/bin/bash -e {0}
##[endgroup]
go: downloading github.com/golang-jwt/jwt/v5 v5.3.1
go: downloading github.com/tidwall/gjson v1.18.0
go: downloading golang.org/x/crypto v0.54.0
go: downloading github.com/gorilla/websocket v1.5.3
go: downloading github.com/gin-gonic/gin v1.10.1
go: downloading github.com/awsl-project/CLIProxyAPI/v7 v7.0.0-20260728045746-2dcc3bab4250
go: downloading github.com/tidwall/sjson v1.2.5
go: downloading github.com/andybalholm/brotli v1.2.0
go: downloading github.com/google/uuid v1.6.0
go: downloading github.com/klauspost/compress v1.18.3
go: downloading github.com/redis/go-redis/v9 v9.19.0
go: downloading github.com/glebarez/sqlite v1.11.0
go: downloading github.com/go-sql-driver/mysql v1.8.1
go: downloading golang.org/x/sync v0.22.0
go: downloading gorm.io/driver/mysql v1.6.0
go: downloading gorm.io/driver/postgres v1.6.0
go: downloading gorm.io/gorm v1.31.1
go: downloading github.com/go-webauthn/webauthn v0.16.0
go: downloading github.com/tidwall/match v1.1.1
go: downloading github.com/tidwall/pretty v1.2.0
go: downloading github.com/gin-contrib/sse v0.1.0
go: downloading github.com/mattn/go-isatty v0.0.20
go: downloading golang.org/x/net v0.57.0
go: downloading github.com/sirupsen/logrus v1.9.3
go: downloading github.com/aws/aws-sdk-go-v2 v1.41.5
go: downloading github.com/aws/aws-sdk-go-v2/credentials v1.19.14
go: downloading github.com/bytedance/sonic v1.15.0
go: downloading github.com/glebarez/go-sqlite v1.21.2
go: downloading modernc.org/sqlite v1.23.1
go: downloading filippo.io/edwards25519 v1.1.1
go: downloading github.com/jackc/pgx/v5 v5.9.2
go: downloading github.com/jinzhu/now v1.1.5
go: downloading github.com/jinzhu/inflection v1.0.0
go: downloading golang.org/x/text v0.40.0
go: downloading github.com/go-viper/mapstructure/v2 v2.5.0
go: download...
GitHub Actions: PR Checks / 1_Backend Checks.txt: chore: update CLIProxyAPI v7 replace
Conclusion: failure
##[group]Run go vet ./cmd/... ./internal/...
�[36;1mgo vet ./cmd/... ./internal/...�[0m
shell: /usr/bin/bash -e {0}
##[endgroup]
go: downloading gopkg.in/yaml.v3 v3.0.1
go: downloading github.com/gorilla/websocket v1.5.3
go: downloading github.com/tidwall/gjson v1.18.0
go: downloading github.com/spf13/cobra v1.10.2
go: downloading golang.org/x/term v0.45.0
go: downloading github.com/aws/aws-sdk-go-v2 v1.41.5
go: downloading github.com/google/uuid v1.6.0
go: downloading github.com/aws/aws-sdk-go-v2/credentials v1.19.14
go: downloading github.com/tidwall/sjson v1.2.5
go: downloading github.com/awsl-project/CLIProxyAPI/v7 v7.0.0-20260728045746-2dcc3bab4250
go: downloading github.com/gin-gonic/gin v1.10.1
go: downloading github.com/andybalholm/brotli v1.2.0
go: downloading github.com/klauspost/compress v1.18.3
go: downloading github.com/bytedance/sonic v1.15.0
go: downloading github.com/redis/go-redis/v9 v9.19.0
go: downloading github.com/wailsapp/wails/v2 v2.11.0
go: downloading golang.org/x/crypto v0.54.0
go: downloading github.com/go-webauthn/webauthn v0.16.0
go: downloading github.com/golang-jwt/jwt/v5 v5.3.1
go: downloading github.com/glebarez/sqlite v1.11.0
go: downloading github.com/go-sql-driver/mysql v1.8.1
go: downloading golang.org/x/sync v0.22.0
go: downloading gorm.io/driver/mysql v1.6.0
go: downloading gorm.io/driver/postgres v1.6.0
go: downloading gorm.io/gorm v1.31.1
go: downloading github.com/alicebob/miniredis/v2 v2.38.0
go: downloading github.com/DATA-DOG/go-sqlmock v1.5.2
go: downloading github.com/tidwall/match v1.1.1
go: downloading github.com/tidwall/pretty v1.2.0
go: downloading golang.org/x/sys v0.47.0
go: downloading github.com/spf13/pflag v1.0.9
go: downloading github.com/gin-contrib/sse v0.1.0
go: downloading github.com/mattn/go-isatty v0.0.20
go: downloading golang.org/x/net v0.57.0
go: downloading github.com/aws/smithy-go v1.24.2
go: downloading github.com/sirupsen/logrus v1.9.3
go: downloading git...
GitHub Actions: PR Checks / Backend Checks: chore: update CLIProxyAPI v7 replace
Conclusion: failure
##[group]Run go vet ./cmd/... ./internal/...
�[36;1mgo vet ./cmd/... ./internal/...�[0m
shell: /usr/bin/bash -e {0}
##[endgroup]
go: downloading gopkg.in/yaml.v3 v3.0.1
go: downloading github.com/gorilla/websocket v1.5.3
go: downloading github.com/tidwall/gjson v1.18.0
go: downloading github.com/spf13/cobra v1.10.2
go: downloading golang.org/x/term v0.45.0
go: downloading github.com/aws/aws-sdk-go-v2 v1.41.5
go: downloading github.com/google/uuid v1.6.0
go: downloading github.com/aws/aws-sdk-go-v2/credentials v1.19.14
go: downloading github.com/tidwall/sjson v1.2.5
go: downloading github.com/awsl-project/CLIProxyAPI/v7 v7.0.0-20260728045746-2dcc3bab4250
go: downloading github.com/gin-gonic/gin v1.10.1
go: downloading github.com/andybalholm/brotli v1.2.0
go: downloading github.com/klauspost/compress v1.18.3
go: downloading github.com/bytedance/sonic v1.15.0
go: downloading github.com/redis/go-redis/v9 v9.19.0
go: downloading github.com/wailsapp/wails/v2 v2.11.0
go: downloading golang.org/x/crypto v0.54.0
go: downloading github.com/go-webauthn/webauthn v0.16.0
go: downloading github.com/golang-jwt/jwt/v5 v5.3.1
go: downloading github.com/glebarez/sqlite v1.11.0
go: downloading github.com/go-sql-driver/mysql v1.8.1
go: downloading golang.org/x/sync v0.22.0
go: downloading gorm.io/driver/mysql v1.6.0
go: downloading gorm.io/driver/postgres v1.6.0
go: downloading gorm.io/gorm v1.31.1
go: downloading github.com/alicebob/miniredis/v2 v2.38.0
go: downloading github.com/DATA-DOG/go-sqlmock v1.5.2
go: downloading github.com/tidwall/match v1.1.1
go: downloading github.com/tidwall/pretty v1.2.0
go: downloading golang.org/x/sys v0.47.0
go: downloading github.com/spf13/pflag v1.0.9
go: downloading github.com/gin-contrib/sse v0.1.0
go: downloading github.com/mattn/go-isatty v0.0.20
go: downloading golang.org/x/net v0.57.0
go: downloading github.com/aws/smithy-go v1.24.2
go: downloading github.com/sirupsen/logrus v1.9.3
go: downloading git...
🧰 Additional context used
🪛 golangci-lint (2.12.2)
go.mod
[error] 26-26: : # github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok [github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok.test]
internal/adapter/provider/cliproxyapi_grok/adapter.go:26:17: undefined: exec.XAIExecutor
internal/adapter/provider/cliproxyapi_grok/adapter.go:104:18: undefined: exec.NewXAIExecutor
(typecheck)
🔇 Additional comments (1)
go.mod (1)
24-26: LGTM!Also applies to: 35-36, 104-104, 116-116, 118-119
1702996 to
4fa1c49
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@go.mod`:
- Line 128: Update the CLIProxyAPI/v7 replacement in go.mod to use a
commit/version that still exports sdk/exec.XAIExecutor and
sdk/exec.NewXAIExecutor, preserving compatibility with cliproxyapi_grok;
alternatively, migrate that adapter to the replacement’s new API before
retaining the current target.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 25deaad9-f796-4f4c-82e7-ff643d23bb70
⛔ Files ignored due to path filters (1)
go.sumis excluded by!**/*.sum
📒 Files selected for processing (1)
go.mod
📜 Review details
🧰 Additional context used
🪛 golangci-lint (2.12.2)
go.mod
[error] 26-26: : # github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok [github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok.test]
internal/adapter/provider/cliproxyapi_grok/adapter.go:26:17: undefined: exec.XAIExecutor
internal/adapter/provider/cliproxyapi_grok/adapter.go:104:18: undefined: exec.NewXAIExecutor
(typecheck)
🔇 Additional comments (1)
go.mod (1)
24-26: LGTM!Also applies to: 35-36, 46-46, 97-104, 116-119
4fa1c49 to
f8a6487
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@go.mod`:
- Line 128: Update the CLIProxyAPI/v7 replace target in go.mod to a compatible
revision that still exports sdk/exec.XAIExecutor and sdk/exec.NewXAIExecutor,
and verify both symbols exist before finalizing the dependency change. Do not
retain the current incompatible revision unless cliproxyapi_grok is migrated to
its replacement API.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 955ddb0b-469a-4841-9cd7-6337ebbe7d66
⛔ Files ignored due to path filters (1)
go.sumis excluded by!**/*.sum
📒 Files selected for processing (1)
go.mod
📜 Review details
🧰 Additional context used
🪛 golangci-lint (2.12.2)
go.mod
[error] 26-26: : # github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok [github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok.test]
internal/adapter/provider/cliproxyapi_grok/adapter.go:26:17: undefined: exec.XAIExecutor
internal/adapter/provider/cliproxyapi_grok/adapter.go:104:18: undefined: exec.NewXAIExecutor
(typecheck)
🔇 Additional comments (1)
go.mod (1)
24-26: LGTM!Also applies to: 35-36, 97-104, 116-119
f8a6487 to
903616a
Compare
There was a problem hiding this comment.
♻️ Duplicate comments (1)
go.mod (1)
128-128: 🎯 Functional Correctness | 🔴 Critical | ⚡ Quick win修复 CLIProxyAPI/v7 替换目标的 API 不兼容问题。
Line 128 指向
v7.0.0-20260730045150-d9fadb1301dc。静态检查确认该版本未导出exec.XAIExecutor和exec.NewXAIExecutor,但internal/adapter/provider/cliproxyapi_grok仍使用这两个符号。因此项目无法通过类型检查。请改用仍导出这两个符号的兼容 revision,或先迁移 adapter 到替换模块提供的 API。该问题与此前 Line 128 的评论相同,当前仍未解决。
请在合并前执行:
#!/usr/bin/env bash set -euo pipefail target='v7.0.0-20260730045150-d9fadb1301dc' dep_json="$(go mod download -json "github.com/awsl-project/CLIProxyAPI/v7@$target")" dep_dir="$(printf '%s\n' "$dep_json" | jq -r '.Dir // empty')" test -n "$dep_dir" test -d "$dep_dir/sdk/exec" rg -n '\b(XAIExecutor|NewXAIExecutor)\b' "$dep_dir/sdk/exec" rg -n '\b(XAIExecutor|NewXAIExecutor)\b' internal/adapter/provider/cliproxyapi_grok🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@go.mod` at line 128, Update the go.mod replacement for github.com/router-for-me/CLIProxyAPI/v7 so the selected github.com/awsl-project/CLIProxyAPI/v7 revision exports exec.XAIExecutor and exec.NewXAIExecutor used by internal/adapter/provider/cliproxyapi_grok; choose a compatible revision, or migrate that adapter to the replacement module’s available API, then verify the project type-checks.Source: Linters/SAST tools
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Duplicate comments:
In `@go.mod`:
- Line 128: Update the go.mod replacement for
github.com/router-for-me/CLIProxyAPI/v7 so the selected
github.com/awsl-project/CLIProxyAPI/v7 revision exports exec.XAIExecutor and
exec.NewXAIExecutor used by internal/adapter/provider/cliproxyapi_grok; choose a
compatible revision, or migrate that adapter to the replacement module’s
available API, then verify the project type-checks.
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 9dd8ca36-22a0-4166-9bae-273b069b0aa1
⛔ Files ignored due to path filters (1)
go.sumis excluded by!**/*.sum
📒 Files selected for processing (1)
go.mod
📜 Review details
🧰 Additional context used
🪛 golangci-lint (2.12.2)
go.mod
[error] 26-26: : # github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok [github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok.test]
internal/adapter/provider/cliproxyapi_grok/adapter.go:26:17: undefined: exec.XAIExecutor
internal/adapter/provider/cliproxyapi_grok/adapter.go:104:18: undefined: exec.NewXAIExecutor
(typecheck)
🔇 Additional comments (1)
go.mod (1)
24-26: LGTM!Also applies to: 35-36, 46-46, 97-104, 116-119
903616a to
3989f52
Compare
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@go.mod`:
- Line 128: 更新 go.mod 中 CLIProxyAPI/v7 的 replace 目标,改用仍导出 exec.XAIExecutor 与
exec.NewXAIExecutor 的 revision;如果必须保留当前 revision,则迁移 cliproxyapi_grok adapter
以使用其新 API。完成后验证目标模块的 sdk/exec 导出这两个符号,并确认 cliproxyapi_grok adapter 可成功构建。
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro Plus
Run ID: 506fc4e6-439e-4575-98d0-5e130c86f62f
⛔ Files ignored due to path filters (1)
go.sumis excluded by!**/*.sum
📒 Files selected for processing (1)
go.mod
📜 Review details
🧰 Additional context used
🪛 golangci-lint (2.12.2)
go.mod
[error] 26-26: : # github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok [github.com/awsl-project/maxx/internal/adapter/provider/cliproxyapi_grok.test]
internal/adapter/provider/cliproxyapi_grok/adapter.go:26:17: undefined: exec.XAIExecutor
internal/adapter/provider/cliproxyapi_grok/adapter.go:104:18: undefined: exec.NewXAIExecutor
(typecheck)
9b8a4b8 to
acd4a0f
Compare
acd4a0f to
4bb4515
Compare
Automated CLIProxyAPI v7 dependency update by CLIProxyAPI Sync workflow.
Summary by CodeRabbit