[correctness][confirmed-defect][builtins/uniq] uniq ignores an invalid count overwritten by a later flag
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
El flag se guarda como una sola cadena en builtins/uniq/uniq.go alrededor de la línea 133, y la validación en la línea 151 solo comprueba el valor final. Lee la configuración del flag y esa validación, luego ejecuta el reproductor printf 'a\na\n' | uniq -f wrong -f 0 para ver el código de salida. Está terminado cuando un valor anterior inválido de -f falla con el mensaje estilo GNU y salida 1, con un escenario en tests/scenarios/cmd/uniq/errors/.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Generated by the rshell behavior correctness scanner.
- Contract:
BEH-003 - Classification:
confirmed-defect - Impact:
low - Target:
builtins/uniq - Scanned commit:
514d1cb8188fda6b1accca5955dfab3b0e8aa34a
An invalid -f value must fail when encountered. Storing only the final value allows -f wrong -f 0 to silently succeed and process input.
Contract sources
[SHELL_FEATURES.md:55](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/SHELL_FEATURES.md#L55): Supported surface; Bash/GNU behavior applies where no intentional divergence is documented.
Implementation references
[builtins/uniq/uniq.go:133](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/builtins/uniq/uniq.go#L133): skip-fields is stored as a single StringP value.[builtins/uniq/uniq.go:151](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/builtins/uniq/uniq.go#L151): Validation checks only the final stored value.
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 'a\na\n' | uniq -f wrong -f 0
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 '"'"'a\na\n'"'"' | uniq -f wrong -f 0 'LC_ALL=C.UTF-8 /usr/bin/bash --noprofile --norc -c 'printf '"'"'a\na\n'"'"' | uniq -f wrong -f 0 'GOENV_VERSION=1.26.3 timeout 300s go test ./tests -run '^TestShellScenarios$/^.$/zz_correctness_scan_' -count=2 -v -timeout 300s
Expected
Exit code: 1
stdout:
stderr:
uniq: wrong: invalid number of fields to skip
Actual
Exit code: 0
stdout:
a
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_uniq_overridden_invalid_count.yaml (removed after evidence capture)
- Suggested regression-test location:
tests/scenarios/cmd/uniq/errors/
Fingerprint: 4f71efcff5df3d86c97abda5e62ade3317a632e940403ad1ba88ab9b1f83be7f
- 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
-
[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
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
Los mantenedores suelen responder en 1 día
Todos los issues de DataDog/rshell
Issues similares
-
bug performance scalability
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Davidjustice28/basildb#35 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
rhysd/actionlint#761 ·
-
bug needs-triage P2 tier:1-mechanical
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
krakenhavoc/cmd_and_ctrl#3652 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 66/100
puppe1990/amarra-cais#345 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
limitlessprojects/Go-Hybrid#50 ·
Los mantenedores suelen responder en 1 día