fix(agentic-ide): align Serena setup for both CLIs#154
Merged
Conversation
Serena 1.2.0 removed the old disable-memory flags, so the setup docs needed to point at the supported config model. The Copilot-side flow also needed to stop implying a Claude-style setup path and instead show the Copilot registration flow directly. * remove deprecated Serena flags from both setup docs * document ~/.serena/serena_config.yml base_modes: [no-memories] * simplify the Copilot setup command to the Copilot CLI registration path * bump agentic-ide plugin versions to 1.0.5 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Updates the agentic-ide setup documentation to reflect Serena’s post-1.2.0 mode-based configuration (replacing removed CLI flags) and aligns the Copilot CLI registration flow around copilot mcp add. Also bumps the plugin version in both Claude and Copilot plugin manifests.
Changes:
- Replace deprecated Serena flags (
--disable-memories,--disable-onboarding) with~/.serena/serena_config.ymlusingbase_modes: [no-memories]. - Simplify Copilot CLI MCP registration examples (and remove the alternate JSON config examples).
- Bump
agentic-ideplugin versions to1.0.5in both manifests.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| plugins-copilot/agentic-ide/commands/setup.md | Updates Copilot setup steps for Serena (modes) and streamlines MCP registration instructions. |
| plugins-copilot/agentic-ide/.claude-plugin/plugin.json | Bumps Copilot plugin manifest version to 1.0.5. |
| plugins-claude/agentic-ide/skills/setup/SKILL.md | Updates Claude setup docs to remove deprecated Serena flags and reference mode config. |
| plugins-claude/agentic-ide/.claude-plugin/plugin.json | Bumps Claude plugin manifest version to 1.0.5. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+66
to
+69
| cat > ~/.serena/serena_config.yml <<'YAML' | ||
| base_modes: | ||
| - no-memories | ||
| YAML |
Comment on lines
109
to
112
| ``` | ||
|
|
||
| Or add it to `~/.copilot/mcp-config.json`: | ||
|
|
||
| ```json | ||
| { | ||
| "mcpServers": { | ||
| "semgrep": { | ||
| "type": "stdio", | ||
| "command": "semgrep-mcp", | ||
| "args": [], | ||
| "tools": ["*"], | ||
| "env": {} | ||
| } | ||
| } | ||
| } | ||
| ``` | ||
|
|
||
| Optional: add `SEMGREP_APP_TOKEN` to `env` to enable `semgrep_findings` (pulls from Semgrep AppSec Platform). Generate at <https://semgrep.dev/orgs/-/settings/tokens>. All local-scan tools work without it. | ||
|
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Update the agentic-ide setup docs to use Serena's supported mode-based configuration instead of removed flags, and make the Copilot CLI setup path explicitly register Copilot rather than implying a Claude-only flow.
Changes
--disable-memories/--disable-onboardingreferences~/.serena/serena_config.ymlwithbase_modes: [no-memories]copilot mcp addagentic-ideplugin versions in both manifestsTesting
.github/scripts/validate-plugins.shCloses #147