fix: las rutas del diff solo se leen de la cabecera #16

Merged

julio.caicedo merged 1 commit from fix/rutas-solo-de-la-cabecera into main

2026-10-23 14:56:48 +00:00

julio.caicedo commented

2026-10-23 14:56:02 +00:00

Owner

Sigue a #15. diffPath y diffLines buscaban --- a/ y +++ en toda la sección, también dentro de los hunks, donde el texto lo escribe el PR:

Esconder un borrado: en un archivo borrado, una línea eliminada con el texto -- a/go.sum aparece como --- a/go.sum. diffPath tomaba la sección por go.sum y TrimDiff la sacaba de la revisión.
Mover comentarios: una línea agregada ++ b/otro.go aparece como +++ b/otro.go. diffLines cambiaba de archivo y los comentarios siguientes caían en el equivocado.

Ahora la ruta solo se lee en la cabecera, entre diff --git y el primer @@. Dentro de un hunk toda línea lleva su prefijo (+, -, espacio), así que ni diff --git ni @@ pueden aparecer al principio de una línea de contenido.

TestTrimDiffReadsPathsOnlyFromTheHeader y TestDiffLinesIgnoresHeadersInsideHunks fallan con el código anterior y pasan con este.

🤖 Generated with Claude Code

Sigue a #15. `diffPath` y `diffLines` buscaban `--- a/` y `+++ ` en toda la sección, también dentro de los hunks, donde el texto lo escribe el PR: - **Esconder un borrado:** en un archivo borrado, una línea eliminada con el texto `-- a/go.sum` aparece como `--- a/go.sum`. `diffPath` tomaba la sección por `go.sum` y `TrimDiff` la sacaba de la revisión. - **Mover comentarios:** una línea agregada `++ b/otro.go` aparece como `+++ b/otro.go`. `diffLines` cambiaba de archivo y los comentarios siguientes caían en el equivocado. Ahora la ruta solo se lee en la cabecera, entre `diff --git` y el primer `@@`. Dentro de un hunk toda línea lleva su prefijo (`+`, `-`, espacio), así que ni `diff --git` ni `@@` pueden aparecer al principio de una línea de contenido. `TestTrimDiffReadsPathsOnlyFromTheHeader` y `TestDiffLinesIgnoresHeadersInsideHunks` fallan con el código anterior y pasan con este. 🤖 Generated with [Claude Code](https://claude.com/claude-code)

julio.caicedo added 1 commit

2026-10-23 14:56:02 +00:00

fix: las rutas del diff solo se leen de la cabecera

Go / Test (pull_request) Successful in 41s

Details

d9f14b8f3c

Una línea borrada '-- a/go.sum' aparece como '--- a/go.sum' y hacía pasar
un archivo borrado por go.sum para sacarlo de la revisión. Una línea
agregada '++ b/x' movía los comentarios a otro archivo. La ruta se lee
solo antes del primer '@@'.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

julio.caicedo merged commit 601a192e01 into main

2026-10-23 14:56:48 +00:00

julio.caicedo deleted branch fix/rutas-solo-de-la-cabecera

2026-10-23 14:56:48 +00:00

julio.caicedo referenced this pull request from a commit

2026-10-23 14:56:49 +00:00

Merge pull request 'fix: las rutas del diff solo se leen de la cabecera' (#16) from fix/rutas-solo-de-la-cabecera into main