docs(cli): clarify managed MCP path needs no endpoint (O-3)#1881
docs(cli): clarify managed MCP path needs no endpoint (O-3)#1881nicolasduransluke wants to merge 1 commit into
Conversation
cotctl mcp install uses a built-in default endpoint; only the manual claude mcp add path requires a URL. (Addresses review note O-3.) Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
![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 MCP integration documentation in both English and Spanish to clarify that the managed path does not require a manual endpoint by default. Feedback on the Spanish translation suggests using 'voseo' conjugation and replacing 'default' with 'predeterminado' to maintain consistency with the rest of the localized documentation.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| <div className="alert alert--info"> | ||
|
|
||
| **¿Dónde está el endpoint?** La URL del endpoint MCP es específica de tu entorno de Cotalker — tu contacto de Cotalker te la puede proveer. `cotctl mcp install` maneja los detalles de conexión por vos, así que el comando gestionado es la ruta recomendada. | ||
| **La vía gestionada no necesita endpoint.** `cotctl mcp install` ya conoce el endpoint de documentación de Cotalker por defecto, así que no tienes que indicarlo — solo ejecútalo. Solo necesitas una URL explícita si registras el servidor manualmente con `claude mcp add` (abajo), o si quieres apuntar a un entorno distinto del default (puedes sobrescribirlo con la variable de entorno `COTCTL_MCP_URL`). |
There was a problem hiding this comment.
Para mantener la consistencia con el resto de la documentación en español (que utiliza de manera uniforme el voseo, por ejemplo: creás, preferís, Reemplazá, Podés), se sugiere adaptar este párrafo al mismo estilo de redacción. Además, se recomienda cambiar "default" por "predeterminado" para evitar el uso innecesario de anglicismos y mantener la coherencia con "por defecto" mencionado anteriormente.
| **La vía gestionada no necesita endpoint.** `cotctl mcp install` ya conoce el endpoint de documentación de Cotalker por defecto, así que no tienes que indicarlo — solo ejecútalo. Solo necesitas una URL explícita si registras el servidor manualmente con `claude mcp add` (abajo), o si quieres apuntar a un entorno distinto del default (puedes sobrescribirlo con la variable de entorno `COTCTL_MCP_URL`). | |
| **La vía gestionada no necesita endpoint.** `cotctl mcp install` ya conoce el endpoint de documentación de Cotalker por defecto, así que no tenés que indicarlo — solo ejecutalo. Solo necesitás una URL explícita si registrás el servidor manualmente con `claude mcp add` (abajo), o si querés apuntar a un entorno distinto al predeterminado (podés sobrescribirlo con la variable de entorno `COTCTL_MCP_URL`). |
ccarrascor
left a comment
ccarrascor
left a comment
There was a problem hiding this comment.
Veredicto
Solicito cambios — los hechos técnicos están correctos (verificados contra el código de cotctl) y la redacción es clara, pero antes de aprobar falta adjuntar al PR una captura del resultado renderizado del recuadro alert--info (EN y ES) para confirmar visualmente que el markdown en negrita y el admonition se ven correctos en la página publicada. Además hay un anglicismo de estilo (default) a corregir (ver inline).
Resumen
PR de documentación de 2 archivos / +2 −2: reescribe una línea del recuadro alert--info en docs/developer/cli/mcp-integration.md y su traducción ES, aclarando que cotctl mcp install ya trae el endpoint por defecto y que solo la vía manual claude mcp add o un entorno no-default requieren URL (COTCTL_MCP_URL como override). La aclaración es correcta y útil.
Hallazgos
- Bloqueantes: 1 — falta adjuntar la captura/preview del recuadro renderizado.
- Inexactitudes: 0 — todos los hechos verificados contra
cotctl. - Sugerencias: 2 (comentadas inline) + 1 nota menor fuera del diff.
Bloqueante — preview renderizado
Antes de aprobar hace falta adjuntar al PR una captura del recuadro alert--info ya renderizado (página de Docusaurus), tanto en EN como en ES, para confirmar que el texto en negrita y el admonition se ven correctos. Hoy no hay ninguna imagen ni preview en la descripción ni en los comentarios del PR.
Inexactitudes
Ninguna. Verificado contra el código real de cotctl:
cotctl mcp installusaoptions.url ?? getMcpUrl()(src/commands/mcp.ts:35) → sin--urlusa el default; «no necesita endpoint» es correcto.- Default
https://llm.cotalker.com/mcp(src/lib/mcp-url.generated.ts:3). - Override
COTCTL_MCP_URLcon nombre exacto (src/lib/mcp-config.ts:10).
Revisiones previas consideradas
- Gemini Code Assist (review COMMENTED + inline en ES:25) planteó (a) pasar la línea a voseo y (b) cambiar
defaultporpredeterminado. Coincido en (b) (ver inline). En (a) coincido en el síntoma (el archivo queda con registro mezclado) pero no en la dirección: la norma de neutralidad LATAM prefiere tú/impersonal, así que la línea nueva (en tú/impersonal) es la correcta y no debe pasarse a voseo; lo que se desvía es el resto del archivo, que idealmente se migra a tú/impersonal en un cleanup aparte (fuera de este diff).
Cosas bien resueltas
- El texto nuevo es más claro y accionable que el anterior y elimina la fricción de pedir un endpoint que el comando ya conoce.
- Documenta el override real (
COTCTL_MCP_URL) con el nombre exacto, verificable. - EN y ES se actualizaron en el mismo PR — no quedó la traducción desfasada.
Riesgos y notas adicionales
- El cambio no toca estructura MDX, enlaces ni anclas, así que no afecta
npm run build. El doc sigue cableado ensidebars.js:370. - Nota menor (fuera del diff): el comando crea el server con nombre
cotalker-ragpor defecto (cotctl src/commands/mcp.ts:29), mientras el ejemplo manual del doc usacotalker-docs(:33). Estado preexistente, no introducido por este PR — solo a tener en el radar si se quiere alinear nomenclatura.
| <div className="alert alert--info"> | ||
|
|
||
| **¿Dónde está el endpoint?** La URL del endpoint MCP es específica de tu entorno de Cotalker — tu contacto de Cotalker te la puede proveer. `cotctl mcp install` maneja los detalles de conexión por vos, así que el comando gestionado es la ruta recomendada. | ||
| **La vía gestionada no necesita endpoint.** `cotctl mcp install` ya conoce el endpoint de documentación de Cotalker por defecto, así que no tienes que indicarlo — solo ejecútalo. Solo necesitas una URL explícita si registras el servidor manualmente con `claude mcp add` (abajo), o si quieres apuntar a un entorno distinto del default (puedes sobrescribirlo con la variable de entorno `COTCTL_MCP_URL`). |
There was a problem hiding this comment.
Sugerencia: default es un anglicismo en este contexto en español; usá predeterminado (o «por defecto»). Aplicar también en la versión EN si corresponde para mantener la equivalencia.
| <div className="alert alert--info"> | ||
|
|
||
| **¿Dónde está el endpoint?** La URL del endpoint MCP es específica de tu entorno de Cotalker — tu contacto de Cotalker te la puede proveer. `cotctl mcp install` maneja los detalles de conexión por vos, así que el comando gestionado es la ruta recomendada. | ||
| **La vía gestionada no necesita endpoint.** `cotctl mcp install` ya conoce el endpoint de documentación de Cotalker por defecto, así que no tienes que indicarlo — solo ejecútalo. Solo necesitas una URL explícita si registras el servidor manualmente con `claude mcp add` (abajo), o si quieres apuntar a un entorno distinto del default (puedes sobrescribirlo con la variable de entorno `COTCTL_MCP_URL`). |
There was a problem hiding this comment.
Sugerencia (registro): esta línea está en tú/impersonal (no tienes que indicarlo, solo ejecútalo, solo necesitas), que es el registro neutro recomendado. El resto del archivo usa voseo (creás :7, Reemplazá :32, Podés :38/:45), así que el archivo queda con el registro mezclado. La línea nueva está bien así — no la pasen a voseo. Lo coherente a futuro es migrar el resto del archivo a tú/impersonal, pero eso excede este diff. Dejo la nota para que esta línea no se «corrija» hacia voseo por consistencia con el resto del archivo.
2 participants
Follow-up de la review de Leo (O-3): aclara que
cotctl mcp installya trae el endpoint por defecto (llm.cotalker.com), así que el partner no necesita indicarlo; solo la vía manualclaude mcp addrequiere URL. EN + ES (tuteo).Aparte — O-2 (binario standalone): no lo documenté a propósito. Los binarios del último GitHub Release son
release-0.6.1pero npm va en0.8.0→ documentar el binario mandaría a partners a una versión vieja. Es un gap del pipeline de releases de cotctl, no del doc.🤖 Generated with Claude Code