chore: update - #658
Conversation
|
You do not have enough credits to review this pull request. Please purchase more credits to continue. |
Summary of ChangesHello @shunkakinoki, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed! This pull request performs a routine configuration update. It specifically modifies a model identifier within the default settings for Highlights
Using Gemini Code AssistThe full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips. Invoking Gemini You can request assistance from Gemini at any point by creating a comment using either
Customization To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a Limitations & Feedback Gemini Code Assist may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments. If you're interested in giving your feedback about your experience with Gemini Code Assist for Github and other Google products, sign up here. You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension. Footnotes
|
Mesa DescriptionTL;DRUpdated clawdbot's configuration and default model, migrated kyber config to a new schema, switched to the upstream nix-clawdbot overlay, and refreshed project dependencies. What changed?
Description generated by Mesa. Update settings |
|
Note Other AI code review bot(s) detectedCodeRabbit has detected other AI code review bot(s) in this pull request and will avoid duplicating their findings in the review comments. This may lead to a less comprehensive review. 📝 WalkthroughWalkthroughUpdated submodule pointer for Changes
Sequence Diagram(s)sequenceDiagram
participant Host as Host (Kyber)
participant Activation as Home Activation
participant Secrets as Secrets Store
participant FS as Filesystem (clawdbot.json)
participant Service as Clawdbot Service
Host->>Activation: trigger activations (post-boot / config apply)
Activation->>Secrets: read "gateway-token" and "telegram-token"
Activation->>FS: inject gateway.auth.token into clawdbot.json
Activation->>FS: inject channels.telegram.botToken into clawdbot.json
Activation->>FS: remove deprecated keys (e.g., messages.queue.byProvider)
Activation->>Service: signal/reload clawdbot service
Estimated code review effort🎯 4 (Complex) | ⏱️ ~45 minutes Possibly related PRs
Suggested labels
Poem
🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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.
Pull request overview
This PR updates the configuration for the clawdbot module by changing the model provider and version, and updates a subproject commit reference.
Changes:
- Updated the AI model reference from
anthropic/claude-opus-4-5tocliproxy/claude-opus-4-5-20251101 - Updated the dotagents subproject commit hash
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| home-manager/modules/clawdbot/default.nix | Updates the default model configuration to use a different provider and versioned model |
| dotagents | Updates the subproject commit reference |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the default clawdbot model to one served by cliproxy. My review includes one suggestion to improve maintainability by adding a comment to clarify the date in the new model name, which appears to be a version or expiration date. This will help future developers understand the configuration.
| # Default settings | ||
| defaults = { | ||
| model = "anthropic/claude-opus-4-5"; | ||
| model = "cliproxy/claude-opus-4-5-20251101"; |
There was a problem hiding this comment.
The model name cliproxy/claude-opus-4-5-20251101 appears to include a version identifier that resembles a date. If this identifier signifies an expiration date or a specific version lifetime, it would be beneficial for future maintenance to add a comment explaining its meaning. This will help prevent unexpected failures and guide developers on necessary updates in the future.
80f850c to
c95b483
Compare
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
There was a problem hiding this comment.
1 issue found across 2 files (changes from recent commits).
Prompt for AI agents (all issues)
Check if these issues are valid — if so, understand the root cause of each and fix them.
<file name="overlays/default.nix">
<violation number="1" location="overlays/default.nix:9">
P2: Placeholder pnpmDepsHash will trigger a Nix fixed-output hash mismatch and break builds until replaced with the real hash.</violation>
</file>
Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.
- Update nix-clawdbot input to 978b03d (2026-01-24) - Remove custom clawdbot source override - now included upstream - Simplify overlays/default.nix to use default nix-clawdbot overlay
The memory plugin slot defaults to "memory-core" when undefined or empty. Setting it to "none" explicitly disables the slot, which is the magic keyword recognized by clawdbot's normalizeSlotValue function.
Updates clawdbot-gateway to version 2026.1.24-0 which includes fixes from upstream.
Summary
Test plan
Summary by cubic
Updated clawdbot’s default model to cliproxy/claude-opus-4-5-20251101 and migrated kyber config to the new schema (token injection, byChannel queue, Telegram under channels, bridge removed). Switched to the default nix-clawdbot overlay with the upstream fix, removed our custom override, included the memory plugin fix, disabled the memory plugin slot, cleaned deprecated config keys; also bumped the dotagents submodule and refreshed Rust dependencies in Cargo.lock.
Written for commit 0aa0529. Summary will update on new commits.