fix(trace): attribute shell changes in a mixed Copilot pre-tool batch
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 48/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- go
- Área
- developer-experience, devtools
Línea de trabajo
Start by reading the shared pre-tool handler and the Copilot provider path that handles preToolUse batches; the issue does not name their files or tests. Trace how shell and file calls select snapshots, then check existing tests for mixed toolCalls[] batches. Done when one hook takes both snapshot types and shell changes in mixed batches are attributed to the AI.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Copilot CLI sends the preToolUse hook as a batch of calls (toolCalls[]). The Trace pre-tool handler takes one kind of snapshot for each hook input: a worktree snapshot for a shell tool, or file snapshots for a file tool. When one batch holds both a file tool (create, edit, apply_patch) and a shell tool (bash, powershell), the Copilot provider snapshots only the files, so the changes of the shell command are not attributed to the AI.
In 52 real batches of Copilot CLI 1.0.109, no batch mixed the two kinds. The fix needs the shared pre-tool handler to take both snapshots for one hook.
Follow-up of #3800.
- Lenguaje dominante
- Go
- Estrellas
- 582
- Forks
- 61
- Merge medio
- 12 h 47 min
- PR fusionados (30 d)
- 137
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Sin 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 chainloop-dev/chainloop
-
feature-parity trace-provider
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
chainloop-dev/chainloop#3802 ·
Los mantenedores suelen responder en 1 día
-
feat(trace): support GitHub Copilot (CLI and VS Code) in Chainloop TracePosiblemente ocupada @migmartri la tomó hoy. AbiertoFeature spec trace-provider
chainloop-dev/chainloop#3800 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
Allow enabling policy enforcement at the workflow contract levelPosiblemente ocupada @migmartri la tomó hace 10 días. Abiertocomponent/backend new-policy spec
chainloop-dev/chainloop#3762 · 1 comentario · 1 asignado ·
Los mantenedores suelen responder en 1 día
-
chainloop component/CLI performance trace-provider
Dificultad 4/5 3-5 días Aptitud para principiantes 62/100
chainloop-dev/chainloop#3756 ·
Los mantenedores suelen responder en 1 día
-
chainloop component/CLI performance trace-provider
Dificultad 2/5 1-3 horas Aptitud para principiantes 25/100
chainloop-dev/chainloop#3749 ·
Los mantenedores suelen responder en 1 día
Todos los issues de chainloop-dev/chainloop
Issues similares
-
GitHub webhook is not subscribed to release events, so `event: release` never triggersPosiblemente ocupada @qsun la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
woodpecker-ci/woodpecker#7542 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
zilliztech/milvus-backup#1386 ·
Los mantenedores suelen responder en 1 día
-
Pipelined flush no longer observes kill signals after #2042: flush backoffer is built with nil varsAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
model:sonnet runtime:claude security size:S
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
FootprintAI/Containarium#3624 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día