fix(openclaw): slim agents, fix free fallback, unblock CI - #2244
Conversation
Keep only main + ACP agents, register cliproxy free by bare id, and fix Shell/Nix/Lua/Upgrade failures blocking main. Co-authored-by: Cursor <cursoragent@cursor.com>
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (8)
Disabled knowledge base sources:
📝 WalkthroughSummary by CodeRabbit
WalkthroughChangesOpenClaw configuration
Neovim reliability
moshi-hook package update
dotagents revision
Dotfiles updater specification
Estimated code review effort: 3 (Moderate) | ~20 minutes Possibly related PRs
Poem
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
![indent-zero[bot]](https://avatars.githubusercontent.com/in/1426746?s=60&v=4){kind=small}
|
|
||
| It 'contains no configured group identifiers' | ||
| When run bash -c "! grep -Eq '@g\\.us|\\\"kind\\\": \\\"group\\\"' '$PWD/config/openclaw/openclaw.template.json'" | ||
| When run bash -c "! rg -q '@g\\.us|\"kind\": \"group\"' '$PWD/config/openclaw/openclaw.template.json'" |
There was a problem hiding this comment.
Undeclared rg dependency: This assertion now depends on ripgrep, but devenv.nix only lists shellspec/shellcheck — not pkgs.ripgrep. It works in CI because ubuntu-latest ships /usr/bin/rg and impure nix develop inherits system PATH, but a pure Nix shell or a minimal dev system will fail with rg: command not found. Consider adding pkgs.ripgrep to devenv.nix, or keeping grep -Eq and just cleaning the escaping that was tripping ShellSpec's stderr.
| }) | ||
| telescope.load_extension("gh") | ||
| telescope.load_extension("fzf") | ||
| pcall(telescope.load_extension, "fzf") |
There was a problem hiding this comment.
Silent fzf failure: With bare pcall, if the fzf native library is missing or fails to build the user gets a degraded sorter with no notification. Consider capturing the result and warning:
local ok, err = pcall(telescope.load_extension, "fzf")
if not ok then
vim.notify("telescope: fzf extension not loaded: " .. tostring(err), vim.log.levels.WARN)
endThis preserves the CI unblock while making real regressions visible.
1 participant
Summary
cliproxy/freefallback: model catalog id was wronglycliproxy/free(API 502); use barefreemain+ ACP only (amp,factory,codex); remap GitHub hook tomainopenclaw_hydrate_spec.shdotfiles_updater_spec.shfzfextension load withpcalluntil native lib is builtdotagentsfor removed upstreamask-questions-if-underspecified(fixes Upgrade skills-install)Test plan
shell-testno longer warns on openclaw group-id grep)skills-installcompletes)freeafter deepseek-pro 429Made with Cursor
Summary by cubic
Fixes OpenClaw model fallback by using
freeinstead ofcliproxy/free, and trims agents tomain+ ACP (amp,factory,codex) to unblock CI. Remaps the GitHub hook tomainand keepsdeepseek-flashin the fallback chain.Bug Fixes
freefor the fallback model to avoid 502s.telescopefzfload withpcalluntil the native lib is available.grep -E; switch ShellSpec group-id check toripgrepto silence stderr.dotfiles_updater_spec.shto fixtreefmtdrift.Dependencies
dotagentsto pick up removal of upstreamask-questions-if-underspecified(fixes Upgradeskills-install).moshi-hookto0.2.70with new hashes.Written for commit c767300. Summary will update on new commits.