shellcheck: fix issues in audit.sh #15

Merged

bc1bb merged 1 commit from fix-shellcheck-audit-sh into master

2026-05-17 23:12:38 +02:00

Claude commented

2026-05-17 22:29:15 +02:00

Collaborator

Fixes #14

Changes

SC2001 (line 167) — replaced echo "$ref" | sed 's|refs/remotes/origin/||' with parameter expansion echo "${ref#refs/remotes/origin/}".
SC2086 (lines 291/293/315/317) — converted mode_args and CLAUDE_EXTRA_ARGS to arrays so they expand quoted as "${mode_args[@]}" / "${extra_args[@]}" without losing word splitting where intended.
SC2188 (line 441) — prefixed the empty truncating redirect > "$STATE_DIR/${repo_name}.state" with : so the redirection has an explicit no-op command.

Verification

bash -n audit.sh passes. Re-run of the shellcheck CI on this branch should now report 0 issues for audit.sh.

Fixes #14 ## Changes - **SC2001 (line 167)** — replaced `echo "$ref" | sed 's|refs/remotes/origin/||'` with parameter expansion `echo "${ref#refs/remotes/origin/}"`. - **SC2086 (lines 291/293/315/317)** — converted `mode_args` and `CLAUDE_EXTRA_ARGS` to arrays so they expand quoted as `"${mode_args[@]}"` / `"${extra_args[@]}"` without losing word splitting where intended. - **SC2188 (line 441)** — prefixed the empty truncating redirect `> "$STATE_DIR/${repo_name}.state"` with `:` so the redirection has an explicit no-op command. ## Verification `bash -n audit.sh` passes. Re-run of the shellcheck CI on this branch should now report 0 issues for `audit.sh`.

Claude added 1 commit

2026-05-17 22:29:15 +02:00

shellcheck: fix issues in audit.sh

ShellCheck / shellcheck (push) Successful in 6s

Details

ShellCheck / shellcheck (pull_request) Successful in 6s

Details

3cb4f61a57

- SC2001 line 167: use parameter expansion instead of sed
- SC2086 lines 291/293/315/317: word-split into arrays for safe quoted expansion
- SC2188 line 441: prefix truncating redirect with `:`

Closes #14