General-purpose coding policy for Baruch's AI agents
—
—
Does it follow best practices?
Run evals on this skill
Adds up to 20 points to the overall score
View guide
Low
Low-risk findings worth noting
except Exception:, or the analogous form in other languagesexcept BaseException: (or its equivalent that traps interrupts); KeyboardInterrupt and SystemExit must propagate so processes stay killableouter-boundary-process-contract# noqa: BLE001 on the except Exception: line// eslint-disable-next-line directly above the catchset -euo pipefail (except under the carve-out below)|| true, no || :, no 2>/dev/null standing in for a handlerif or case on its exit code, never blanket suppressiongrep exits 1 on no-match and 2 on error, and || true collapses both, so an unreadable file or a bad regex reads as "nothing found"cmd 2>/dev/null || { echo "<actionable message>" >&2; exit 1; } replaces a worse message with a better oneEXIT trap's final command status becomes the script's exit status — end cleanup handlers with return 0 so cleanup never rewrites the outcomeset -eset -uo pipefail — only -e is droppedPATH, a value that defaults to something dangerous — carries its own explicit failure check. A temp dir used only as a path for later file operations already fails loudly on first use and needs noneset -euo pipefail|| true on a source of the script under testset -euo pipefail in the harness's shell, and its entry-point guard then returns non-zero under the -e it just enabledsource itself, never an assertion or a command under testset +e follows on the next line, restoring the harness's own disciplineset -e carve-out above|| true in a harness still converts to an explicit exit-code checkapp init to create one".tessl-plugin
rules
skills
adopt-fork-pr
install-reviewer
migrate-to-plugin