Add shellcheck to pre-commit

Author: AlexWaygood

.pre-commit-config.yaml

@@ -128,5 +128,10 @@ repos:
           # but the integration only works if shellcheck is installed
           - "github.com/wasilibs/go-shellcheck/cmd/shellcheck@v0.10.0"
 
+  - repo: https://github.com/shellcheck-py/shellcheck-py
+    rev: v0.10.0.1
+    hooks:
+      - id: shellcheck
+
 ci:
   skip: [cargo-fmt, dev-generate-all]

fuzz/init-fuzzer.sh

@@ -3,7 +3,7 @@
 # https://stackoverflow.com/a/246128/3549270
 SCRIPT_DIR=$( cd -- "$( dirname -- "${BASH_SOURCE[0]}" )" &> /dev/null && pwd )
 
-cd "$SCRIPT_DIR"
+cd "$SCRIPT_DIR" || exit
 
 if ! cargo fuzz --help >&/dev/null; then
   echo "Installing cargo-fuzz..."
@@ -20,7 +20,7 @@ if [ ! -d corpus/common ]; then
   done
 
   (
-    cd corpus/common
+    cd corpus/common || exit
 
     read -p "Would you like to build a corpus from a python source code dataset? (this will take a long time!) [Y/n] " -n 1 -r
     echo

scripts/mypy_primer.sh

@@ -5,33 +5,33 @@ echo "Enabling mypy primer specific configuration overloads (see .github/mypy-pr
 mkdir -p ~/.config/ty
 cp .github/mypy-primer-ty.toml ~/.config/ty/ty.toml
 
-PRIMER_SELECTOR="$(paste -s -d'|' $PRIMER_SELECTOR)"
+PRIMER_SELECTOR="$(paste -s -d'|' "${PRIMER_SELECTOR}")"
 
 echo "new commit"
-git rev-list --format=%s --max-count=1 "$GITHUB_SHA"
+git rev-list --format=%s --max-count=1 "${GITHUB_SHA}"
 
-MERGE_BASE="$(git merge-base "$GITHUB_SHA" "origin/$GITHUB_BASE_REF")"
-git checkout -b base_commit "$MERGE_BASE"
+MERGE_BASE="$(git merge-base "${GITHUB_SHA}" "origin/${GITHUB_BASE_REF}")"
+git checkout -b base_commit "${MERGE_BASE}"
 echo "base commit"
 git rev-list --format=%s --max-count=1 base_commit
 
 cd ..
 
-echo "Project selector: $PRIMER_SELECTOR"
+echo "Project selector: ${PRIMER_SELECTOR}"
 # Allow the exit code to be 0 or 1, only fail for actual mypy_primer crashes/bugs
 uvx \
   --from="git+https://github.com/hauntsaninja/mypy_primer@59509d48de6da6aaa4e3a2f5e338769bc471f2d7" \
   mypy_primer \
   --repo ruff \
   --type-checker ty \
   --old base_commit \
-  --new "$GITHUB_SHA" \
+  --new "${GITHUB_SHA}" \
   --project-selector "/($PRIMER_SELECTOR)\$" \
   --output concise \
-  --debug > $DIFF_FILE || [ $? -eq 1 ]
+  --debug > "${DIFF_FILE}" || [ $? -eq 1 ]
 
 # Output diff with ANSI color codes
-cat $DIFF_FILE
+cat "${DIFF_FILE}"
 
 # Remove ANSI color codes before uploading
-sed -ie 's/\x1b\[[0-9;]*m//g' $DIFF_FILE
+sed -ie 's/\x1b\[[0-9;]*m//g' "${DIFF_FILE}"

scripts/release/bump.sh

@@ -1,3 +1,4 @@
 #!/usr/bin/env bash
 
+# shellcheck disable=SC2016
 echo 'This script has been removed, use `./scripts/release.sh` instead'