Implement sed builtin command#56

Merged

gh-worker-dd-mergequeue-cf854d[bot] merged 35 commits intomainfrom

Mar 12, 2026

Commits on Mar 12, 2026

update .claude/skills/code-review/SKILL.md
AlexandreYang
committed
Address review comments: fix 8 sed bugs for bash compatibility

AlexandreYang
and
claude
committed
Revert "update .claude/skills/code-review/SKILL.md"
AlexandreYang
committed
Revert "update .claude/skills/code-review/SKILL.md"
AlexandreYang
committed
Address review comments: fix 7 more sed bugs for bash compatibility

AlexandreYang
and
claude
committed
Skip TestPentestDevNull on Windows where /dev/null is unavailable

AlexandreYang
and
claude
committed
Address review comments: fix q+append flush, reject s///0, add pentest coverage

AlexandreYang
and
claude
committed
Fix 3 review comments: l command octal escapes, \a\b\f\r in s///, totalRead newline counting

AlexandreYang
and
claude
committed
Address review comments: fix 6 sed bugs for bash compatibility

AlexandreYang
and
claude
committed
Address review comments: fix 6 more sed bugs for bash compatibility

AlexandreYang
and
claude
committed
Update review-fix-loop Step 3: verify @codex replied to latest review request

AlexandreYang
and
claude
committed
Address review comments: fix 0,/re/ range, $ multi-file, step.yaml

AlexandreYang
and
claude
committed
Address review comments: fix branch propagation, invalid backrefs, non-special escapes

AlexandreYang
and
claude
committed
Update review-fix-loop Step 3: verify @codex replied to latest review request

AlexandreYang
and
claude
committed
Address review comments: fix n/N isLastFile propagation, remove unnecessary skip_assert

AlexandreYang
and
claude
committed
Merge branch 'main' into alex/sed
AlexandreYang
authored
Merge branch 'main' into alex/sed
AlexandreYang
committed
Address PR review comments: fix 5 issues in sed builtin

AlexandreYang
and
claude
committed
Merge branch 'main' into alex/sed
AlexandreYang
committed
Address PR review comments: 4 improvements to sed builtin

AlexandreYang
and
claude
committed
Remove unused findLabel and found functions after labelMap refactor

AlexandreYang
and
claude
committed
Fix backref followed by digit in sed replacement (e.g. \10 -> group 1 + literal 0)

AlexandreYang
and
claude
committed
Reject extra characters after q/Q command to match GNU sed

AlexandreYang
and
claude
committed
Allow # inline comments after s/// flags (GNU sed compat)

AlexandreYang
and
claude
committed
Fix CRLF preservation and append queue flush before N command

AlexandreYang
and
claude
committed
Add bytes.IndexByte to builtin import allowlist

AlexandreYang
and
claude
committed
Remove unused transFrom/transTo fields from sedCmd

AlexandreYang
and
claude
committed
Merge main into alex/sed, resolve conflicts

AlexandreYang
and
claude
committed
Merge branch 'main' into alex/sed
AlexandreYang
committed
[iter 1] Address review comments: bash compatibility fixes for sed parser

AlexandreYang
and
claude
committed
[iter 1] Fix gofmt formatting in sed_pentest_test.go

AlexandreYang
and
claude
committed