feature: Public API for companion review plugins?
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 15/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- lua, neovim
- Área
- api, developer-experience
Línea de trabajo
This is an API design question, not a bug report. Start with the existing CodeDiffOpen, CodeDiffClose and CodeDiffFileSelect events and the public navigation functions the issue mentions, and check which of the three requested capabilities (reading the active comparison, jumping to a line or range, and a ready/refresh event) they already cover. Done means a maintainer decides the public surface, and the issue has no comments or decision yet, so wait for that before writing code.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I have a fork with AI summaries for what changed and why, plus line-by-line explanations of selected code. I'd like to move those into a separate companion plugin and add a guided review that walks through related changes in order, using CodeDiff as the diff viewer.
The main thing I'm trying to figure out is how to do that without depending on CodeDiff's internal modules. Right now my changes use session state, diff buffers, and refresh internals.
Would you be open to a small public API that lets a companion plugin:
- Read the active comparison: repository, revisions, selected file, old/new buffers, and changed ranges.
- Select a file and jump to a line or range on either side of the diff.
- Get an event after the diff is ready or refreshed, so explanations can be updated or cleared when the code changes.
I saw the existing CodeDiffOpen, CodeDiffClose, and CodeDiffFileSelect events and the public navigation functions. Is there already a supported way to cover the rest, or would adding a few extension points make sense? Of course, I'm happy to help work on this api, I don't expect you to just do it for me.
The companion would handle the AI connection, prompts, explanations, and tour state. CodeDiff would handle displaying the diff. I'd like to get your thoughts on the approach before building more around the internals.
- Lenguaje dominante
- Lua
- Estrellas
- 1.6k
- Forks
- 104
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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 esmuellert/codediff.nvim
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
esmuellert/codediff.nvim#582 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
esmuellert/codediff.nvim#579 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
esmuellert/codediff.nvim#575 ·
-
Bug: explorer.focus_on_select = false ignored for added/untracked/deleted files in inline layoutAbiertoType/Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
esmuellert/codediff.nvim#567 · 1 reacción ·
-
Type/Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
esmuellert/codediff.nvim#565 · 1 comentario ·
Todos los issues de esmuellert/codediff.nvim
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
-
good first issue
Dificultad 2/5 Medio día Aptitud para principiantes 74/100
whoismjanyway/sereno#14 ·
-
luaebpf: the by-example page's intro names two outcomes of a lowering and omits the stop with the default verdictPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abiertodocumentation severity: low
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
luainkernel/lunatik#2231 ·
Los mantenedores suelen responder en 1 día
-
bug triage
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
Cidan/BetterBags#1239 ·
Los mantenedores suelen responder en 1 día