Add improve-loop skill for systematic shell feature review#75

Merged

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

alex/improve_loop

Mar 13, 2026

Commits on Mar 12, 2026

Add improve-loop skill for systematic shell feature review

AlexandreYang
and
claude
committed
Increase improve-loop max iterations to 50

AlexandreYang
and
claude
committed
Add PR comment steps, randomize target order in improve-loop skill

AlexandreYang
and
claude
committed
Show current target name in Step 2 task subject during improve-loop

AlexandreYang
and
claude
committed
[field_splitting] Add test documenting consecutive non-whitespace IFS delimiter limitation

AlexandreYang
and
claude
committed
batch 50
AlexandreYang
committed
[inline_var] Use defer for variable restoration to protect against panics

AlexandreYang
and
claude
committed
[improve] batch 6: fix printf %q length modifier bug, add grep --help

AlexandreYang
and
claude
committed

Commits on Mar 13, 2026

[ls] Add --help flag (long-only since -h is --human-readable)

AlexandreYang
and
claude
committed
update .claude/skills/improve-loop/SKILL.md
AlexandreYang
committed
[improve] batch 1: fix readonly guards, parse error exit code, expandErr, ls joinPath

AlexandreYang
and
claude
committed
[improve] batch 2: add test coverage for line_continuation and cmd_separator

AlexandreYang
and
claude
committed
[exit] Fix bash compat: exit with invalid arg should not terminate shell

AlexandreYang
and
claude
committed
[improve] batch 5: fix field_splitting dead test, printf --help, pipe fd safety

AlexandreYang
and
claude
committed
[loopctl] Fix bash compat: break/continue with too many args should exit shell

AlexandreYang
and
claude
committed
[improve] batch 10: fix empty script CLI, loopctl exit code bash compat

AlexandreYang
and
claude
committed
[exit] Fix misleading doc comment about invalid arg behavior

AlexandreYang
and
claude
committed
[strings] Add missing --help scenario test

AlexandreYang
and
claude
committed
[iter 1] Address PR review comments: add expandErr test and fix stderr assertion

AlexandreYang
and
claude
committed
[iter 2] Fix exit non-numeric arg, printf --help exit code, redundant readonly check

AlexandreYang
and
claude
committed
[iter 3] Fix exit error test descriptions and remove skip_assert_against_bash

AlexandreYang
and
claude
committed
[iter 4] Fix break/continue non-numeric arg exit code: 128 → 2 (bash compat)

AlexandreYang
and
claude
committed
[iter 4] Fix break/continue invalid arg exit code to match bash (128, not 2)

AlexandreYang
and
claude
committed
[iter 5] Fix pipe reader deadlock: close pr before wg.Wait()

AlexandreYang
and
claude
committed
[iter 6] Fix if_then_semicolons test description: if/then/fi not if/then/else/fi

AlexandreYang
and
claude
committed
[iter 8] [ls] Gate backslash path separator handling to Windows only

AlexandreYang
and
claude
committed
Merge main into alex/improve_loop

AlexandreYang
and
claude
committed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Add improve-loop skill for systematic shell feature review#75

Add improve-loop skill for systematic shell feature review#75
gh-worker-dd-mergequeue-cf854d[bot] merged 27 commits intomainfrom
alex/improve_loop

Commits on Mar 12, 2026

Add improve-loop skill for systematic shell feature review

Increase improve-loop max iterations to 50

Add PR comment steps, randomize target order in improve-loop skill

Show current target name in Step 2 task subject during improve-loop

[field_splitting] Add test documenting consecutive non-whitespace IFS delimiter limitation

batch 50

[inline_var] Use defer for variable restoration to protect against panics

[improve] batch 6: fix printf %q length modifier bug, add grep --help

Commits on Mar 13, 2026

[ls] Add --help flag (long-only since -h is --human-readable)

update .claude/skills/improve-loop/SKILL.md

[improve] batch 1: fix readonly guards, parse error exit code, expandErr, ls joinPath

[improve] batch 2: add test coverage for line_continuation and cmd_separator

[exit] Fix bash compat: exit with invalid arg should not terminate shell

[improve] batch 5: fix field_splitting dead test, printf --help, pipe fd safety

[loopctl] Fix bash compat: break/continue with too many args should exit shell

[improve] batch 10: fix empty script CLI, loopctl exit code bash compat

[exit] Fix misleading doc comment about invalid arg behavior

[strings] Add missing --help scenario test

[iter 1] Address PR review comments: add expandErr test and fix stderr assertion

[iter 2] Fix exit non-numeric arg, printf --help exit code, redundant readonly check

[iter 3] Fix exit error test descriptions and remove skip_assert_against_bash

[iter 4] Fix break/continue non-numeric arg exit code: 128 → 2 (bash compat)

[iter 4] Fix break/continue invalid arg exit code to match bash (128, not 2)

[iter 5] Fix pipe reader deadlock: close pr before wg.Wait()

[iter 6] Fix if_then_semicolons test description: if/then/fi not if/then/else/fi

[iter 8] [ls] Gate backslash path separator handling to Windows only

Merge main into alex/improve_loop