chore(config): update Claude and OpenCode plugin configurations - #527
Conversation
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 focuses on essential configuration updates for both Claude and OpenCode environments. The changes aim to keep the AI tools current by updating plugin versions, integrating a new plugin marketplace for Claude, and referencing a newer version of the GLM model for OpenCode, thereby enhancing their capabilities and ensuring access to the latest features. 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
|
|
Caution Review failedThe pull request is closed. 📝 WalkthroughSummary by CodeRabbitRelease Notes
✏️ Tip: You can customize this high-level summary in your review settings. WalkthroughThis pull request updates configuration files for Claude and OpenCode tools. It replaces a Claude plugin, adds a new marketplace-sourced plugin, configures a new marketplace source, updates the OpenRouter GLM model preset version, and adds a global safety-net plugin. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Poem
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro Disabled knowledge base sources:
📒 Files selected for processing (2)
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 |
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
![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 configurations for Claude and OpenCode. For Claude, it updates a plugin from ralph-wiggum to ralph-loop and adds a new marketplace configuration for acme-tools. For OpenCode, it corrects a GLM model reference from glm-4-6 to glm-4-7 to match its details and adds the cc-safety-net plugin. The changes are mostly straightforward and correct. I have added one comment regarding a potentially confusing name in config/claude/settings.json to improve long-term maintainability.
| "extraKnownMarketplaces": { | ||
| "acme-tools": { | ||
| "source": { | ||
| "source": "github", | ||
| "repo": "kenryu42/cc-marketplace" | ||
| } |
There was a problem hiding this comment.
The marketplace name acme-tools is generic and its connection to the source repository kenryu42/cc-marketplace isn't immediately obvious. This could be confusing for future developers maintaining this configuration.
To improve clarity, if the name acme-tools is intentional, please consider adding a comment explaining the relationship. For example:
"acme-tools": { // Marketplace for ACME's custom tools, hosted in a fork of cc-marketplace
"source": {
"source": "github",
"repo": "kenryu42/cc-marketplace"
}
}Alternatively, if the name isn't a hard requirement, renaming it to something more descriptive like kenryu42-cc-marketplace would also resolve the potential confusion.
There was a problem hiding this comment.
Pull request overview
This PR updates plugin configurations for Claude and OpenCode tools, replacing outdated plugins and adding new marketplace configurations.
Changes:
- Update OpenCode GLM model reference from glm-4-6 to glm-4-7 for consistency with the preset ID
- Add cc-safety-net plugin to OpenCode configuration
- Replace ralph-wiggum plugin with ralph-loop in Claude settings
- Add extraKnownMarketplaces configuration for acme-tools marketplace
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| config/opencode/opencode.jsonc | Updates GLM model key to glm-4-7 and adds cc-safety-net plugin |
| config/claude/settings.json | Replaces ralph-wiggum with ralph-loop plugin and adds acme-tools marketplace configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "safety-net@cc-marketplace": true | ||
| }, | ||
| "extraKnownMarketplaces": { | ||
| "acme-tools": { |
There was a problem hiding this comment.
The marketplace name in the "extraKnownMarketplaces" configuration is "acme-tools", but the repository being referenced is "kenryu42/cc-marketplace". This appears to be a mismatch - the marketplace name should reflect the actual marketplace being referenced. Consider either renaming the marketplace key to match the repository (e.g., "cc-marketplace") or verifying that "acme-tools" is the correct identifier for this marketplace in your configuration system.
| "acme-tools": { | |
| "cc-marketplace": { |
| "extraKnownMarketplaces": { | ||
| "acme-tools": { | ||
| "source": { | ||
| "source": "github", |
There was a problem hiding this comment.
The "source" object contains a nested "source" key with value "github", which appears redundant. Verify if this nested structure is intentional or if the configuration should have a different structure, such as using "type" or "provider" for the outer key instead of "source".
| "source": "github", | |
| "type": "github", |
![mesa-dot-dev[bot]](https://avatars.githubusercontent.com/in/1050077?s=60&v=4){kind=small}
There was a problem hiding this comment.
Performed full review of a6fe34c...ba5f091
Analysis
-
Supply Chain Security Risk - Addition of external GitHub marketplace dependency (
kenryu42/cc-marketplace) without apparent version pinning, integrity checks, or validation mechanisms introduces significant supply chain vulnerabilities. -
Plugin Naming Inconsistency - Different naming conventions across configurations ("safety-net@cc-marketplace" vs "cc-safety-net") suggest potential runtime errors and configuration management issues.
-
Version Management Concerns - GLM model update from version 4-6 to 4-7 lacks fallback mechanisms or migration paths if the newer model becomes unavailable.
-
Plugin Ecosystem Evolution - Transition from "ralph-wiggum" to "ralph-loop" indicates architectural shift from one-shot to loop-based patterns without clear transition guidance.
Tip
Help
Slash Commands:
/review- Request a full code review/review latest- Review only changes since the last review/describe- Generate PR description. This will update the PR body or issue comment depending on your configuration/help- Get help with Mesa commands and configuration options
2 files reviewed | 3 comments | Edit Agent Settings • Read Docs
| "plan-export@cc-marketplace": true, | ||
| "safety-net@cc-marketplace": true | ||
| }, | ||
| "extraKnownMarketplaces": { |
There was a problem hiding this comment.
Adding an external GitHub marketplace without version pinning or integrity verification creates a supply chain security risk. Consider: 1) Pinning to a specific commit hash or tag instead of the default branch, 2) Adding a verification mechanism to ensure plugin integrity, or 3) Documenting the security implications of using external marketplaces in your security policy.
Agent: 🏛 Architecture •
Prompt for Agent
Task: Address review feedback left on GitHub.
Repository: shunkakinoki/dotfiles#527
File: config/claude/settings.json#L17
Action: Open this file location in your editor, inspect the highlighted code, and resolve the issue described below.
Feedback:
Adding an external GitHub marketplace without version pinning or integrity verification creates a supply chain security risk. Consider: 1) Pinning to a specific commit hash or tag instead of the default branch, 2) Adding a verification mechanism to ensure plugin integrity, or 3) Documenting the security implications of using external marketplaces in your security policy.
| "serena@claude-plugins-official": true, | ||
| "typescript-lsp@claude-plugins-official": true, | ||
| "plan-export@cc-marketplace": true, | ||
| "safety-net@cc-marketplace": true |
There was a problem hiding this comment.
The naming convention safety-net@cc-marketplace in Claude's config differs from cc-safety-net in OpenCode's config (line 292). This inconsistency could indicate: 1) Different plugin systems with different naming patterns, or 2) A configuration error where the plugins don't actually reference the same functionality. Verify that this naming difference is intentional and both plugins serve their intended purpose.
Agent: 🏛 Architecture •
Prompt for Agent
Task: Address review feedback left on GitHub.
Repository: shunkakinoki/dotfiles#527
File: config/claude/settings.json#L15
Action: Open this file location in your editor, inspect the highlighted code, and resolve the issue described below.
Feedback:
The naming convention `safety-net@cc-marketplace` in Claude's config differs from `cc-safety-net` in OpenCode's config (line 292). This inconsistency could indicate: 1) Different plugin systems with different naming patterns, or 2) A configuration error where the plugins don't actually reference the same functionality. Verify that this naming difference is intentional and both plugins serve their intended purpose.
| }, | ||
| "models": { | ||
| "glm-4-6": { | ||
| "glm-4-7": { |
There was a problem hiding this comment.
The model key change from 'glm-4-6' to 'glm-4-7' removes the old configuration entirely. If this model reference is used elsewhere in your codebase or if the new model version is not yet available via OpenRouter, this could cause runtime failures. Consider: 1) Verifying the model is available in the OpenRouter API, 2) Adding a fallback mechanism, or 3) Temporarily keeping both versions during a transition period.
Agent: 🏛 Architecture •
Prompt for Agent
Task: Address review feedback left on GitHub.
Repository: shunkakinoki/dotfiles#527
File: config/opencode/opencode.jsonc#L199
Action: Open this file location in your editor, inspect the highlighted code, and resolve the issue described below.
Feedback:
The model key change from 'glm-4-6' to 'glm-4-7' removes the old configuration entirely. If this model reference is used elsewhere in your codebase or if the new model version is not yet available via OpenRouter, this could cause runtime failures. Consider: 1) Verifying the model is available in the OpenRouter API, 2) Adding a fallback mechanism, or 3) Temporarily keeping both versions during a transition period.
Summary
Summary by cubic
Updated Claude and OpenCode configs: swapped ralph-wiggum→ralph-loop, added cc-marketplace, enabled plan-export and cc-safety-net, and bumped the OpenCode GLM preset to glm-4-7.
Written for commit acf401d. Summary will update on new commits.