fix: adiciona servidores MCP faltantes nos configs de IDE - #4
fix: adiciona servidores MCP faltantes nos configs de IDE#4nikolasdehor wants to merge 2 commits into
Conversation
- Adiciona exa ao config do Claude Code e Cursor - Adiciona playwright ao config do Cursor (já presente no Claude) - Atualiza preset de referência para 'aios-full' - Corrige URLs da documentação (allfluence → SynkraAI) Agora os configs de IDE refletem todos os servidores disponíveis no ecossistema, consistente com os presets definidos. Closes #2
|
Warning Ignoring CodeRabbit configuration file changes. For security, only the configuration from the base branch is applied for open source repositories. 📝 WalkthroughWalkthroughThese changes add missing MCP server configurations (exa and playwright) to IDE sample config files, update presets from "aios-dev" to "aios-full", and modify the documentation URL to point to the SynkraAI MCP ecosystem repository; additionally, repository automation and guidance files ( ChangesMCP Server Configuration Updates
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 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 |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@ide-configs/cursor/sample-config.json`:
- Around line 10-21: The "playwright" MCP configuration references a
non-existent npm package "@anthropic/playwright-mcp" which will cause npx to
fail; locate the "playwright" JSON object (keys: "command" and "args") and
replace the package name in "args" with the correct, published package (or
remove the entry if there is no MCP) after verifying the package exists on the
npm registry; ensure the "args" value points to a valid package string and keep
the surrounding structure ("command": "npx") unchanged.
| }, | ||
| "playwright": { | ||
| "command": "npx", | ||
| "args": ["-y", "@anthropic/playwright-mcp"] | ||
| }, | ||
| "exa": { | ||
| "command": "npx", | ||
| "args": ["-y", "exa-mcp-server"], | ||
| "env": { | ||
| "EXA_API_KEY": "${EXA_API_KEY}" | ||
| } | ||
| } |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Verify the `@anthropic/playwright-mcp` package exists on npm
curl -s "https://registry.npmjs.org/@anthropic/playwright-mcp" | jq '{name: .name, version: ."dist-tags".latest, description: .description}'Repository: SynkraAI/mcp-ecosystem
Length of output: 126
🏁 Script executed:
#!/bin/bash
# Get the full response from npm registry for `@anthropic/playwright-mcp`
curl -s "https://registry.npmjs.org/@anthropic/playwright-mcp" | head -100
# Also check if the package exists at all by looking for error response
echo "---"
curl -s -w "\nHTTP Status: %{http_code}\n" "https://registry.npmjs.org/@anthropic/playwright-mcp"Repository: SynkraAI/mcp-ecosystem
Length of output: 129
The Playwright MCP package does not exist—configuration will fail.
The @anthropic/playwright-mcp package referenced in line 12 does not exist on the npm registry (returned HTTP 404). The npx command will fail when Cursor attempts to load this server. Verify the correct package name and update the configuration before merging.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@ide-configs/cursor/sample-config.json` around lines 10 - 21, The "playwright"
MCP configuration references a non-existent npm package
"@anthropic/playwright-mcp" which will cause npx to fail; locate the
"playwright" JSON object (keys: "command" and "args") and replace the package
name in "args" with the correct, published package (or remove the entry if there
is no MCP) after verifying the package exists on the npm registry; ensure the
"args" value points to a valid package string and keep the surrounding structure
("command": "npx") unchanged.
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
.github/copilot-instructions.md (1)
1-9: 💤 Low valueConsidere usar terminologia consistente em português.
Na linha 8, "mcp servers config" mistura inglês em um documento português. Para manter a consistência com
language: "pt-BR"definido em.coderabbit.yaml, considere: "configs de servidores MCP" ou "configuração de servidores MCP".🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.github/copilot-instructions.md around lines 1 - 9, The phrase "mcp servers config" in the Copilot instructions mixes English into a Portuguese document; update that exact text to a consistent Portuguese alternative such as "configs de servidores MCP" or "configuração de servidores MCP" (the string "mcp servers config" is the unique token to find and replace) and ensure it matches the language setting in .coderabbit.yaml (language: "pt-BR").
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.github/copilot-instructions.md:
- Around line 19-22: Update the folder structure text in
.github/copilot-instructions.md so it matches the README: replace the incorrect
`configs/` entry with the actual folders `presets/`, `servers/` and
`ide-configs/`, keep the existing `docker/`/`Dockerfile*` and
`docker-compose.yml` entries, and ensure the single-line descriptions mirror the
README wording to avoid inconsistency.
- Around line 29-32: Atualize a seção "## Critical Files" no arquivo
.github/copilot-instructions.md para refletir a estrutura real: substitua a
entrada `configs/*.json` pela lista correta `presets/*.json` e `servers/*.json`
(ou seja, ajuste o bloco que atualmente lista `docker-compose.yml`,
`configs/*.json`, `Dockerfile*`) para usar `presets/*.json` e `servers/*.json`
conforme o README; mantenha o formato de lista existente e preserve as outras
entradas inalteradas.
---
Nitpick comments:
In @.github/copilot-instructions.md:
- Around line 1-9: The phrase "mcp servers config" in the Copilot instructions
mixes English into a Portuguese document; update that exact text to a consistent
Portuguese alternative such as "configs de servidores MCP" or "configuração de
servidores MCP" (the string "mcp servers config" is the unique token to find and
replace) and ensure it matches the language setting in .coderabbit.yaml
(language: "pt-BR").
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 639aef2b-1beb-438c-88c7-309f2d659141
📒 Files selected for processing (2)
.coderabbit.yaml.github/copilot-instructions.md
| ## Folder Structure | ||
| - `configs/` ou raiz — JSONs de configuração MCP | ||
| - `docker/` ou `Dockerfile*` — definições de container | ||
| - `docker-compose.yml` — orquestração |
There was a problem hiding this comment.
Alinhe a estrutura de pastas com a organização real do repositório.
A descrição menciona configs/ (linha 20), mas o README.md documenta a estrutura real como presets/, servers/ e ide-configs/. Isso pode confundir desenvolvedores que seguem este guia.
📁 Estrutura correta baseada no README
## Folder Structure
-- `configs/` ou raiz — JSONs de configuração MCP
+- `presets/` — Configurações preset (aios-dev, aios-research, aios-full)
+- `servers/` — Configurações de servidores MCP individuais
+- `ide-configs/` — Configs específicos por IDE (claude/, cursor/)
- `docker/` ou `Dockerfile*` — definições de container
- `docker-compose.yml` — orquestraçãoBased on learnings and context from README.md that documents the actual folder structure: presets/, servers/, and ide-configs/.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ## Folder Structure | |
| - `configs/` ou raiz — JSONs de configuração MCP | |
| - `docker/` ou `Dockerfile*` — definições de container | |
| - `docker-compose.yml` — orquestração | |
| ## Folder Structure | |
| - `presets/` — Configurações preset (aios-dev, aios-research, aios-full) | |
| - `servers/` — Configurações de servidores MCP individuais | |
| - `ide-configs/` — Configs específicos por IDE (claude/, cursor/) | |
| - `docker/` ou `Dockerfile*` — definições de container | |
| - `docker-compose.yml` — orquestração |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.github/copilot-instructions.md around lines 19 - 22, Update the folder
structure text in .github/copilot-instructions.md so it matches the README:
replace the incorrect `configs/` entry with the actual folders `presets/`,
`servers/` and `ide-configs/`, keep the existing `docker/`/`Dockerfile*` and
`docker-compose.yml` entries, and ensure the single-line descriptions mirror the
README wording to avoid inconsistency.
| ## Critical Files | ||
| - `docker-compose.yml` — orquestração principal | ||
| - `configs/*.json` — definições MCP | ||
| - `Dockerfile*` — imagens custom |
There was a problem hiding this comment.
Corrija o caminho dos arquivos críticos para refletir a estrutura real.
A linha 31 menciona configs/*.json, mas a estrutura do repositório usa presets/*.json e servers/*.json conforme documentado no README.md.
🗂️ Caminhos corretos
## Critical Files
- `docker-compose.yml` — orquestração principal
-- `configs/*.json` — definições MCP
+- `presets/*.json` — configurações preset
+- `servers/*.json` — definições de servidores MCP
+- `ide-configs/*/sample-config.json` — exemplos para IDEs
- `Dockerfile*` — imagens customBased on learnings and context from README.md showing the actual directory structure with presets/ and servers/ folders.
📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| ## Critical Files | |
| - `docker-compose.yml` — orquestração principal | |
| - `configs/*.json` — definições MCP | |
| - `Dockerfile*` — imagens custom | |
| ## Critical Files | |
| - `docker-compose.yml` — orquestração principal | |
| - `presets/*.json` — configurações preset | |
| - `servers/*.json` — definições de servidores MCP | |
| - `ide-configs/*/sample-config.json` — exemplos para IDEs | |
| - `Dockerfile*` — imagens custom |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.github/copilot-instructions.md around lines 29 - 32, Atualize a seção "##
Critical Files" no arquivo .github/copilot-instructions.md para refletir a
estrutura real: substitua a entrada `configs/*.json` pela lista correta
`presets/*.json` e `servers/*.json` (ou seja, ajuste o bloco que atualmente
lista `docker-compose.yml`, `configs/*.json`, `Dockerfile*`) para usar
`presets/*.json` e `servers/*.json` conforme o README; mantenha o formato de
lista existente e preserve as outras entradas inalteradas.
1 participant
Resumo
exaao config do Claude Code e Cursorplaywrightao config do Cursor (já presente no Claude)aios-fullallfluence→SynkraAI)Problema
Os configs de IDE estavam incompletos em relação aos servidores disponíveis:
Verificação
python3 -m json.tool)env.EXA_API_KEYusa interpolação${EXA_API_KEY}consistente comservers/exa.jsonCloses #2
Summary by CodeRabbit