fix: allow go install without replace#559
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
📜 Recent review details⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
🔇 Additional comments (3)
📝 Walkthrough总体概览此 PR 将项目的 CLIProxyAPI 依赖从 v6 升级至 v7。主要变更包括在 变更CLIProxyAPI v7 升级和 go.mod 清理
相关 PR
建议审核者
代码审查工作量评估🎯 1 (简单) | ⏱️ ~8 分钟 庆祝诗
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Tip 💬 Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. 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 |
|
Validation update:
Current merge blocker is repository review policy: |
Summary
Fixes #556.
go install github.com/awsl-project/maxx/cmd/maxx@mainfails today because the main module contains areplacedirective forgithub.com/router-for-me/CLIProxyAPI/v6. Go rejects modules withreplacedirectives when installing a package by version.This PR removes the
replacedirective by switching the CLIProxyAPI SDK imports to the canonicalawsl-projectmodule path and depending ongithub.com/awsl-project/CLIProxyAPI/v7directly.Dependency note: this uses
github.com/awsl-project/CLIProxyAPI/v7 v7.0.0-20260515045015-8112adc7fefb, from awsl-project/CLIProxyAPI#5 (fix/awsl-module-path-v7). That dependency PR changes the fork's module path/imports togithub.com/awsl-project/CLIProxyAPI/v7and keeps the exportedsdk/execpackage maxx uses.Validation
go test ./sdk/exec ./sdk/cliproxy/auth ./sdk/translatorinawsl-project/CLIProxyAPIat8112adc7fefb7bac02af03f496a7d9417e9b276bgo test ./internal/adapter/provider/cliproxyapi_codex ./internal/adapter/provider/cliproxyapi_antigravityGOBIN=$(mktemp -d) go install ./cmd/maxxgo install github.com/awsl-project/maxx/cmd/maxx@32332d4c886228b6286f7069773f0734059d275emaxx --versionoutput:maxx dev (commit: unknown, built: unknown)Summary by CodeRabbit
发布说明