Hacktoberfest 2026:维护者为十月标记出来的 issue,仍然开放、适合新手。 浏览 Hacktoberfest issue

[correctness][confirmed-defect][builtins/wc] wc pads regular-file stdin as though it were a pipe

未关闭 适合新手
#660 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
78/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
活跃
技术栈
go
领域
cli

调研方向

stdin 格式化的决定位于 builtins/wc/wc.go:只要使用 stdin,hasNonRegular 就会被设置(约第 176 行),而该标志会强制最小宽度为 7(约第 216 行)。在临时目录中运行复现命令 wc -lc < zz_correctness_scan_input,并与 bash -c 的输出进行比较,然后确认 hasNonRegular 只作用于管道。完成的标准是:对普通文件重定向时输出 2 8,管道行为保持不变,并且 tests/scenarios/cmd/wc/formatting/ 下有一个场景覆盖该情况。

由索引模型根据 Issue 内容生成。

描述

Generated by the rshell behavior correctness scanner.

  • Contract: BEH-007
  • Classification: confirmed-defect
  • Impact: low
  • Target: builtins/wc
  • Scanned commit: 514d1cb8188fda6b1accca5955dfab3b0e8aa34a

wc -lc < file adds seven-column padding instead of the GNU regular-file formatting. Counts are correct; deterministic stdout bytes are not.

Contract sources
  • [SHELL_FEATURES.md:58](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/SHELL_FEATURES.md#L58): Supported surface; Bash/GNU behavior applies where no intentional divergence is documented.
Implementation references
  • [builtins/wc/wc.go:176](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/builtins/wc/wc.go#L176): hasNonRegular is set merely because stdin is used.
  • [builtins/wc/wc.go:216](https://github.com/datadog/rshell/blob/514d1cb8188fda6b1accca5955dfab3b0e8aa34a/builtins/wc/wc.go#L216): That flag forces minimum width seven even for a regular-file redirect.

Fixture setup

Use a fresh empty working directory for each run. Build from /workspace/repo first. Create these UTF-8 files with exactly the following contents (JSON string notation): {"zz_correctness_scan_input": "one\ntwo\n"}. Run the CLI and reference commands with that directory as cwd.

Reproducer

wc -lc < zz_correctness_scan_input

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 'wc -lc < zz_correctness_scan_input '
  • LC_ALL=C.UTF-8 /usr/bin/bash --noprofile --norc -c 'wc -lc < zz_correctness_scan_input '
  • 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:

2 8

stderr:


Actual

Exit code: 0

stdout:

      2       8

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_wc_seekable_format.yaml (removed after evidence capture)
  • Suggested regression-test location: tests/scenarios/cmd/wc/formatting/

Fingerprint: fcf7755d6ac7070d8f8b00c3ee045a2e875d56641b19c176402e3797e485b1f1

主要语言
Go
星标
31
派生
3
平均合并
2 天 9 小时
30 天内合并 PR
14

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

DataDog/rshell 的其他 Issue

查看 DataDog/rshell 的全部 Issue

相似的 Issue

更多 Go Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。