Add LSP diagnostics to completion requests
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 30/100
Línea de trabajo
The issue does not identify ECA files, tests, or entry points; first locate how completion requests are assembled and where LSP data is available. Resolve the open questions about which diagnostics to include, scope, and configurability before implementation; done means completion requests carry the agreed diagnostic context, with tests covering the chosen behavior.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
To upvote this issue, give it a thumbs up. See this list for the most upvoted issues.
Completion (aka next edit prediction, ai autocomplete, ai tab complete, etc) would benefit from LSP diagnostics.
Prior Art
Cursor
Info pulled from minified JS via an Agent. Take with a grain of salt.
Example JSON
The following JSON snippet is an example payload based on the keys in the bundle. This unfortunately was not logged or pulled from a running application.
{
"linterErrors": {
"relativeWorkspacePath": "src/parser.rs",
"errors": [
{
"message": "expected `;`, found `}`",
"source": "rustc",
"uri": "file:///project/src/parser.rs",
"severity": "ERROR",
"range": {
"startPosition": { "line": 42, "column": 5 },
"endPosition": { "line": 42, "column": 6 }
},
"relatedInformation": []
}
]
},
"lspSuggestedItems": { "suggestions": [{ "label": "parse_expr" }] },
"parameterHints": [
{
"label": "fn read(path: &Path) -> Result<String, Error>",
"documentation": "Reads the entire file into a string."
}
],
"lspContexts": []
}
(Real requests include many non-LSP fields:
currentFile, diff histories,workspaceId,codeResults, timing, etc.)
Scope
linterErrorsis for the current file only.additionalFiles(visible-range snippets, paths, open files, recently viewed, etc.) can include other buffers whenenableRvfTrackingis enabled.
Zed
Zed is open source! Much easier to read. Zed sends LSP context in the user prompt. It looks like LSP diagnostics get serialized to a token friendly format and embedded into the prompt.
Link to Diagnostic type def
Link to collect_diagnostics
Link to user prompt type def
Zed also sends related_files from go to definition (from users cursor) and type defintions.
Outstanding Questions
- What LSP info should be included in the completion request
- Should the selected diagnostics be configurable in ECA config?
- Would any other request types benefit from LSP information?
- Should diagnostics be limited to the file being editted or should it include full workspace diagnostics?
- It looks like Cursor and zed both support this but have booleans to enable. There is probably a reason for the boolean. May be good to test.
- Lenguaje dominante
- Clojure
- Estrellas
- 1k
- Forks
- 82
- Merge medio
- 3 d 39 min
- PR fusionados (30 d)
- 15
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 editor-code-assistant/eca
-
bug
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
editor-code-assistant/eca#655 ·
Los mantenedores suelen responder en 2 días
-
bug
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
editor-code-assistant/eca#652 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 12/100
editor-code-assistant/eca#648 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
editor-code-assistant/eca#644 · 1 comentario ·
Los mantenedores suelen responder en 2 días
-
Add the ability to see what changed in the system promptPosiblemente ocupada @zikajk la tomó hace 9 días. Abiertoenhancement
editor-code-assistant/eca#642 · 1 asignado ·
Los mantenedores suelen responder en 2 días
Todos los issues de editor-code-assistant/eca
Issues similares
-
Usage analytics `v_tables` ignores table metadata set by users (`metabase_table_user_settings`)Abierto.Needs Triage .Team/Graphy Priority:P3 Type:Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
metabase/metabase#84762 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
NoahTheDuke/splint#57 · 1 comentario ·
-
IOP MCP
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
calcit-lang/calcit#2133 ·
Los mantenedores suelen responder en 1 día