Open Code Review: alcance, filtros y llamadas al modelo
Crear un explorador de alcance de revisión de solo lectura
Convierte el selector en un ejercicio que explique exclusiones y distinga la cobertura prevista de la revisión ejecutada.
Qué aprenderás
- Muestra el motivo de cada exclusión
- Separa cobertura prevista y ejecutada
- Define aceptación antes de añadir gráficos
Antes de empezar
- Cambios Git y comparación desde merge-base
- Uso básico de CLI y credenciales de modelos
Elegir el alcance, explicar exclusiones y preparar una prueba controlada distinguiendo inspección y ejecución.
Conclusiones clave
- El explorador puede enseñar filtros sin acceso a un modelo.
- Selección y ejecución necesitan estados distintos.
- La extensión propuesta no es un compromiso upstream.
Muestra el motivo de cada exclusión
Una extensión útil mostraría cada ruta modificada junto a su decisión y el origen de esa decisión. En renombres, presenta rutas antigua y nueva: el selector fijado comprueba ambas para patrones secretos. Vincula cada explicación con la revisión de código inspeccionada.
Empieza con una interfaz de solo lectura y decisiones sintéticas o artefactos depurados de vista previa. No necesita escritura sobre el repositorio ni credenciales del modelo. Esto permite aprender selección sin añadir otra vía para publicar comentarios o fusionar cambios.
Separa cobertura prevista y ejecutada
Usa paneles distintos para archivos seleccionados y revisiones completadas. Un archivo puede estar seleccionado y quedar pendiente por un fallo del proveedor o presupuesto. Mostrar ambos estados evita interpretar una selección en verde como revisión terminada.
Añade ejemplos de archivo normal, eliminado, ruta secreta y plantilla. .ssh/.env.example ayuda a explicar por qué una excepción de nombre base no resuelve la ruta completa. Identifica cada ejemplo como sintético e indica la función que produjo el resultado.
Define aceptación antes de añadir gráficos
La primera condición es que cada decisión mostrada coincida con el artefacto y la interpretación de la fuente fijada. Prueba resultados vacíos, motivos desconocidos y ejecuciones incompletas. Representa rutas y descripciones como texto, no como HTML confiable.
Añade visualización cuando aclare una relación, como las dos rutas de un renombre. Un flujo SVG compacto basta para el primer ejercicio. Esta propuesta es un proyecto didáctico propio, no una función implementada ni una hoja de ruta anunciada por upstream.
Cómo elegir
| Criterio | Opción A | Opción B |
|---|---|---|
| Best when | You need predictable behavior and easy auditing | You need adaptive optimization and have reliable telemetry |
| Main risk | May leave performance on the table | Can become difficult to explain or debug |
Pasos de implementación
- 1
Define un artefacto de solo lectura con revisión y motivo de selección.
- 2
Muestra decisiones sintéticas separadas de los resultados de ejecución.
- 3
Prueba estados desconocidos y textos no confiables antes de añadir efectos.
Ejemplo para copiar
{
"propuesta": true,
"revision": "189be5b024d3309dd10fdc8cd8ee31b2530c210b",
"entrada": "artefacto sintético de selección",
"rutaAnterior": ".env",
"rutaNueva": "config.txt",
"motivo": "ruta secreta",
"ejecucion": "sin iniciar",
"soloLectura": true
}Preguntas frecuentes
¿Necesita Three.js?
No. Una tabla y un flujo pequeño explican las relaciones actuales. Añade gráficos solo para una pregunta concreta.
¿La función ya está publicada upstream?
El capítulo propone un ejercicio independiente y no afirma una publicación upstream.
Fuentes
- Open Code Review / internal/agent/selection.goFuente verificada 2026-09-18
- Open Code Review / internal/config/allowlist/secret_path.goFuente verificada 2026-09-18
- Open Code Review / internal/config/allowlist/secret_path_test.goFuente verificada 2026-09-18
- Open Code Review / pages/src/content/docs/en/architecture.mdFuente verificada 2026-09-18