everything: misleading names in get-resource-links (isOdd) and copy-pasted JSDoc tool names
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 82/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- devtools, documentation
Línea de trabajo
Lee src/everything/tools/get-resource-links.ts, src/everything/tools/trigger-long-running-operation.ts y docs/features.md; los PR anteriores #4503 y #4769 se describen como prototipos para el código refactorizado. Cambia el nombre del flag engañoso, documenta la correspondencia entre los ID pares e impares y corrige cada nombre de herramienta de JSDoc. El trabajo estará terminado cuando el flag coincida con su comportamiento, la correspondencia esté documentada y ambos encabezados coincidan con las herramientas que registran sus archivos.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
Two small naming slips in the everything server, which people read to learn the protocol:
- In
src/everything/tools/get-resource-links.ts, the flagisOddis computed asresourceId % 2 === 0, so it is true for even IDs (which get Text resources). The behavior is right; the name says the opposite, anddocs/features.mddoesn't say which IDs map to which resource type. - The JSDoc header in
src/everything/tools/get-resource-links.tsnames theget-resource-referencetool (copied from its neighbor), and the one insrc/everything/tools/trigger-long-running-operation.tsmisspells its tool's name.
Expected
Rename the flag (for example to isText), document the even/odd mapping, and have each JSDoc header name the tool its file registers.
Prior work
- #4503 by @Joosboy
- #4769 by @simpleqt
The PR(s) above targeted the pre-refactor code and are closed as part of the backlog triage in #4875; this issue is where the work continues, with the PR(s) as a prototype. Thank you to their authors.
- Lenguaje dominante
- TypeScript
- Estrellas
- 91.4k
- Forks
- 11.8k
- Merge medio
- 7 h 36 min
- PR fusionados (30 d)
- 126
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de modelcontextprotocol/servers
-
bug v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
modelcontextprotocol/servers#5217 ·
Los mantenedores suelen responder en 1 día
-
filesystem: search_files does not expand non-wildcard excludePatterns like directory_tree doesAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
modelcontextprotocol/servers#5216 ·
Los mantenedores suelen responder en 1 día
-
bug v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 79/100
modelcontextprotocol/servers#5215 ·
Los mantenedores suelen responder en 1 día
-
documentation server-git server-time v2
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
modelcontextprotocol/servers#5213 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
documentation v2
Dificultad 1/5 1-3 horas Aptitud para principiantes 78/100
modelcontextprotocol/servers#5212 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de modelcontextprotocol/servers
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 72/100
zts212653/clowder-ai#1678 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Progressive-Victory/the-website#680 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100