[correctness][confirmed-defect][builtins/grep] grep -w reverses matches for punctuation-only patterns
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
- 72/100
Línea de trabajo
La lógica de palabra completa está en builtins/grep/grep.go alrededor de la línea 462, donde \b se aplica a ambos lados del patrón. Empieza leyendo esa comprobación de coincidencia y luego ejecuta el reproductor printf '@\nx@y\n' | grep -w @, que actualmente imprime x@y en lugar de @. Está terminado cuando se selecciona la línea @ aislada, no se selecciona x@y, y una prueba de escenario en tests/scenarios/cmd/grep/flags/ cubre patrones formados solo por signos de puntuación.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Generated by the rshell behavior correctness scanner.
- Contract:
BEH-002 - Classification:
confirmed-defect - Impact:
medium - Target:
builtins/grep - Scanned commit:
514d1cb8188fda6b1accca5955dfab3b0e8aa34a
Wrapping a pattern in word boundaries is not equivalent to requiring non-word neighbors. For @, rshell rejects the standalone line and accepts x@y, exactly reversing the expected selection.
Contract sources
[SHELL_FEATURES.md:23](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/SHELL_FEATURES.md#L23): Supported surface; Bash/GNU behavior applies where no intentional divergence is documented.
Implementation references
[builtins/grep/grep.go:462](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/builtins/grep/grep.go#L462): Implements whole-word matching with \b on both sides, which incorrectly requires word/non-word transitions for non-word endpoints.
Fixture setup
Use a fresh empty working directory for each run. Build from /workspace/repo first. No fixture files are needed. Run the CLI and reference commands with that directory as cwd.
Reproducer
printf '@\nx@y\n' | grep -w @
Commands used:
GOENV_VERSION=1.26.3 timeout 300s go build -o /tmp/zz_correctness_scan_rshell ./cmd/rshell/tmp/zz_correctness_scan_rshell --allow-all-commands --allowed-paths "$PWD" -c 'printf '"'"'@\nx@y\n'"'"' | grep -w @ 'LC_ALL=C.UTF-8 /usr/bin/bash --noprofile --norc -c 'printf '"'"'@\nx@y\n'"'"' | grep -w @ 'GOENV_VERSION=1.26.3 timeout 300s go test ./tests -run '^TestShellScenarios$/^.$/zz_correctness_scan_' -count=2 -v -timeout 300s
Expected
Exit code: 0
stdout:
@
stderr:
Actual
Exit code: 0
stdout:
x@y
stderr:
Verification
- Reproduction runs: 4
- Reference: Two fresh CLI fixtures and two fresh scenario fixtures; exploratory runs excluded. Native Bash 5.2.21, GNU coreutils 9.4, grep 3.11, sed 4.9, LC_ALL=C.UTF-8. Docker comparison attempted but image pull returned HTTP 502.
- Temporary test: tests/scenarios/zz_correctness_scan_grep_word_punctuation.yaml (removed after evidence capture)
- Suggested regression-test location:
tests/scenarios/cmd/grep/flags/
Fingerprint: 3a2d49617cf7c2be540db56be523d757cafe1f0e39c7c47a34e2c38f35f62a8a
- Lenguaje dominante
- Go
- Estrellas
- 31
- Forks
- 3
- Merge medio
- 2 d 9 h
- PR fusionados (30 d)
- 14
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una 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 DataDog/rshell
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 75/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
[correctness][confirmed-defect][builtins/wc] wc pads regular-file stdin as though it were a pipeAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
Los mantenedores suelen responder en 1 día
Todos los issues de DataDog/rshell
Issues similares
-
scaleway_lb_frontend: updating a frontend with external_acls = true deletes all scaleway_lb_aclAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
scaleway/terraform-provider-scaleway#4485 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día
-
Add SolentaPosiblemente ocupada @sharmao07 la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
slavakurilyak/awesome-ai-agents#760 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100