apply_patch: say where an ambiguous hunk matches and when to use an @@ anchor
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
- 76/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
- Área
- tooling
Línea de trabajo
Empieza por src/main/tools/apply_patch.ts y busca dónde se construye el error de hunk ambiguo. Revisa las pruebas unitarias existentes y el punto de entrada de la descripción de la herramienta; después, añade cobertura para el mensaje y las indicaciones actualizados. La tarea estará completa cuando el error muestre algunos números de línea coincidentes, la descripción explique cuándo usar un ancla @@ o más contexto y la prueba unitaria pase.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
apply_patch rejects a hunk that matches several places with "a hunk matches N places. Add more context lines or an @@ anchor." (src/main/tools/apply_patch.ts). That is the right refusal, but the message gives no line numbers, so the agent has to guess what to add. In recent chats 4 of 12 apply_patch calls failed this way (CSS: 107 matches; a TypeScript file: 5; a test file: 2; the changelog: 2), each costing a round trip. edit_file did not fail once.
Proposal: list the matching line numbers (first few) in the error, and add a sentence to the tool description that repetitive files (CSS, changelogs, tests) need an @@ anchor or more context. Add a unit test for the new message.
- Lenguaje dominante
- TypeScript
- Estrellas
- 2
- Forks
- 2
- Merge medio
- 8 h 28 min
- PR fusionados (30 d)
- 26
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 PierrunoYT/patch
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
PierrunoYT/patch#316 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
PierrunoYT/patch#315 ·
Los mantenedores suelen responder en 1 día
-
enhancement priority: low severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
PierrunoYT/patch#305 ·
Los mantenedores suelen responder en 1 día
-
security: isAppPageUrl checks devRendererUrl with startsWith instead of comparing parsed URL originsAbiertopriority: low security severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
PierrunoYT/patch#300 ·
Los mantenedores suelen responder en 1 día
-
Claude Code chats: tool calls made inside Claude Code subagents leave cards that end as "(stopped)"Abiertobug priority: low severity: low
Dificultad 2/5 1-3 horas Aptitud para principiantes 83/100
PierrunoYT/patch#289 ·
Los mantenedores suelen responder en 1 día
Todos los issues de PierrunoYT/patch
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