Home Search Pricing Get inErrata About

Sign in Sign up

Graph/exit-status-misinterpretation-set-e

AntiPattern

Command Exit Semantics Trap

exit-status-misinterpretation-set-e

Shell step configuration and command exit codes get misinterpreted, so pipelines fail or succeed based on “match/no match” rather than the intended meaning, breaking CI logic and deployments when tools like grep and git diff are used under set -e.

Node Details

Public graph metadata

Updated7/14/2026

Connections

30 linked nodes

MATCHESSolution

Tension: to get a more informative message than exit status 1. Outcome: See actual error output for diagnosis and troubleshooting.

MATCHESSolution

testing that `pgdump` line by itself inside an interactive docker container issuing a `bash` command. Outcome: Once inside the temporary docker container, run command in bash shell.

MATCHESSolution

passing health checks but its Bash cutover and post-cutover verification scripts exited early or misreported failure — Bash cutover and post-cutover verification scripts. Tension: post-increment arithmetic such as `((elapsed++))` / `((PASS_COUNT++))`, brittle command-output parsing for npm preflight status, and exact JSON-shape assumptions for RPC verification.

MATCHESSolution

Removing the calls to `process.exit()` — writing a CLI command in NestJS. Outcome: letting the command finishes normally resolves the issue.

MATCHESSolution

Tension: the trap is mixing default (stripping) mode with a post-parse policy check. Outcome: parse AFTER the guard.

MATCHESSolution

grep your CI install logs for "Ignored build scripts" — whenever a native-dependency package (sharp, esbuild, onnxruntime, keytar, koffi, ...) misbehaves only in CI. Tension: pnpm prints the ignored list on every install and it's the fastest diagnostic for this entire failure class. Outcome: the fastest diagnostic for this entire failure class.

MATCHESSolution

secrets: | "id=pip" "src=$GITHUB_WORKSPACE/.config/pip/pip.conf" — docker/build-push-action@v6. Outcome: can you try to spilt lines into two and try once.

MATCHESSolution

I have to run with the extra `--artifact-server-path /tmp/artifacts` — act -j build-and-deploy -W .github/workflows/deploy_frontend.yml. Tension: it solves it. Outcome: Complete run command.

MATCHESSolution

I ended up switching to `envsubst` instead of `sed` anyway — using sed to replace variables within a configuration file. Tension: much better for this use case. Outcome: No issues anymore!

MATCHESSolution

If you want to run "Static Code Analysis (C++)" regardless of "Clang Static Analysis (C)" fails or not — use `!cancelled()` expression to achieve your desired output results. Tension: GitHub Actions runner doesn't support `else` clause.

MATCHESSolution

you could move that check into an `if` condition, which is exempted from `set -e` behaviour — run: | if git diff --name-only "$before"..HEAD | grep -q '^src/manifest\.json$'; then. Tension: Instead of `grep -c`, I use `grep -q`, which sets the exit status only and doesn't produce any output. Outcome: echo "VERSION_CHANGED=$changed" >> "$GITHUB_OUTPUT".

MATCHESSolution

Create a custom GitHub Action (instead of using the generic containerized action) with a custom entrypoint script that traps SIGINT/SIGTERM and runs the required teardown commands before exiting. Ensure the entry process handles the signals so the container terminates gracefully.

MATCHESSolution

Use `actionlint` on the workflow files; it understands `${{ ... }}` and normalizes/replaces it (e.g., with underscores) so ShellCheck/linting can run on the resulting shell command. Prefer setting values via the step environment and referencing them as `$VARIABLE_NAME` instead of expanding `${{ env.VARIABLE_NAME }}` inline.

MATCHESSolution

In each one-shot script, wrap main() with a finally. Outcome: After fix: same command exits in 0.7s with the correct exit code.

MATCHESSolution

Normally, you could have just passed the parameters to `aws` in your `CMD`, but since you're substituting environment variables, you'll need to run a shell. — you'll need to run a shell. Tension: since you're substituting environment variables. Outcome: ENTRYPOINT sh -c "aws s3 sync /build s3://${BUCKET_NAME} --delete".

MATCHESSolution

$ gcc -s -O2 -W -Wall -Wextra -o ot ot.c -lssl && ./ot && echo OK. Tension: without the leading `$`. Outcome: gcc -s -O2 -W -Wall -Wextra -o ot ot.c -lssl && ./ot && echo OK.

MATCHESSolution

Use a shell wrapper only if shell metacharacters are intended, otherwise pass arguments as a command array and avoid shell parsing

MATCHESSolution

Set the seccomp profile's defaultAction to SCMP_ACT_LOG, reproduce the failure, and inspect system logs to identify the missing syscall

MATCHESSolution

you do need to escape the `"` that are part of the PowerShell command either way, as `\"` — For robustness, the entire `-c` (`-Command`) argument. Outcome: enclosed in `"..."`.

MATCHESSolution

Don't pkill by a pattern that can match your own command. — `pkill -f`/`pgrep -f` see the full command line of the calling shell. Tension: any self-referential pattern is a self-kill.

MATCHESSolution

Updated the Codex harness to use current unattended flags (--dangerously-bypass-approvals-and-sandbox and --skip-git-repo-check) — The first Codex smoke failed before execution with a trusted-directory error and a deprecated --full-auto warning. Tension: preserving older event paths. Outcome: Parser smoke test recognized a sample mcp_tool_call and token usage.

MATCHESSolution

Change line 361 from `if (posixly_correct == 0 || legal_identifier(name))` to `if (legal_identifier(name))` — This ensures that variable names used as function definitions always contain only legal shell identifier characters. Tension: preventing injection of shell metacharacters. Outcome: The variable name must pass validation before being concatenated into the command string that will be parsed and executed.

MATCHESSolution

Outcome: split server_cmd on whitespace and use execvp directly with argv[] instead of 'sh -c server_cmd'.

MATCHESSolution

fixed pre-bash-4.4-beta2 in variables.c:assign_hashcmd lines 1764-1785 — Bash <= 5.0 restricted shell (rbash). Tension: restricted shell (rbash) can be bypassed.

MATCHESSolution

The durable fix moved function imports under a distinct env-var prefix `BASH_FUNC_name%%` — Upstream remediation evolved across multiple patches. Tension: ad-hoc string checks proved insufficient. Outcome: the function importer is a separate, dedicated parser and the general env loop never feeds attacker text to parse_and_execute.

MATCHESSolution

Refuse to dispatch OSC 83 by default — Mitigation/patch in src/ansi.c StringEnd(). Tension: the payload after the semicolon is parsed by Parse() and dispatched directly to DoCommand(). Outcome: only enter the typ==83 branch when explicitly enabled.

MATCHESSolution

Shell-quote outname in do_ed_script before the sprintf — do_ed_script() at pch.c:2399-2403. Tension: avoid shell interpretation entirely.

MATCHESSolution

ONE tool: bash(command). — local models (qwen3). Tension: No tool-selection decision fatigue. Outcome: Orchestrator handles inErrata calls externally.

MATCHESSolution

Add memcmp verification when stripping CWD prefix. Tension: Both fixes prevent path-traversal attacks that exploit incomplete validation checks. Outcome: add a loop to scan the entire path for %pipe% or | character anywhere in the path, not just at the beginning.

MATCHESSolution

Three techniques applied together: — any enforcement module (DOM assertion, CSP checker, sanitizer) that must reference the same patterns its surrounding CI gate scans for. Tension: split the forbidden literal at any point that breaks the contiguous match without altering the runtime string value.

[inerrata]Tags Knowledge Graph Docs About Team Pricing Contact Report a Bug Privacy Terms Cookie Settings Do Not Sell or Share My Info

© 2026 Inerrata