From cb4a97e880ccca282efd51bae787f18884f94acf Mon Sep 17 00:00:00 2001
From: Lasse Benninga <devops.pipeline@example.com>
Date: Fri, 22 May 2026 13:47:20 +0200
Subject: [PATCH 1/3] feat(autograder): add shared grader_lib + code hygiene
 warnings
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Same grader_lib.sh as week-1 — shared bash library with pass/fail/warn
helpers and static-analysis checks from c55 PR review analysis.

New warning checks for week-2 specific patterns:
- check_silent_zero_in_except: AST-based detection of 'x = 0' in except
  blocks (reviewer c55#3: "sets price/revenue/vat to 0 instead of skipping
  the row — silently corrupts output")
- check_exception_logged: warns when except-block log call doesn't include
  the exception variable 'e' (reviewer c55#5,#7: "log the error type!")
- check_ruff F401/E302: unused imports + missing blank lines between
  functions (c55#3,#1: "This import isn't used" / "add new line after func")
- check_no_print_statements: logging over print()
- check_no_notimplemented: stubs left in (c55#3)
- check_gitignore_python: __pycache__/ and .env ignored
- AI_DEBUG.md traceback check
- Screenshot format: 7/20 pts (warn) for .jpg, full 10/20 for .png

Scoring ladder unchanged (60/20/20, passing=60).
---
 .hyf/grader_lib.sh | 250 +++++++++++++++++++++++++++++++++++++++++++++
 .hyf/test.sh       |  86 ++++++++++++----
 2 files changed, 318 insertions(+), 18 deletions(-)
 create mode 100755 .hyf/grader_lib.sh

diff --git a/.hyf/grader_lib.sh b/.hyf/grader_lib.sh
new file mode 100755
index 0000000..3478d4f
--- /dev/null
+++ b/.hyf/grader_lib.sh
@@ -0,0 +1,250 @@
+#!/usr/bin/env bash
+# grader_lib.sh — shared helpers for HYF Data Track autograders.
+# Source this at the top of test.sh:
+#   source "$(dirname "$0")/grader_lib.sh"
+#
+# Provides: pass(), fail(), warn(), print_results(), write_score(),
+# and a set of common static-analysis checks derived from recurring
+# PR review patterns across cohort c55.
+
+_grader_details=()
+
+pass() { _grader_details+=("✓ PASS  $1"); }
+fail() { _grader_details+=("✗ FAIL  $1"); }
+warn() { _grader_details+=("⚠ WARN  $1"); }
+
+print_results() {
+  local header="${1:-Autograder Results}"
+  echo ""
+  echo "=== $header ==="
+  for line in "${_grader_details[@]}"; do echo "  $line"; done
+  echo ""
+}
+
+write_score() {
+  # write_score <score> <passing> [<outfile>]
+  local score="$1"
+  local passing="$2"
+  local outfile="${3:-$(dirname "${BASH_SOURCE[0]}")/score.json}"
+  local pass_flag="false"
+  [[ "$score" -ge "$passing" ]] && pass_flag="true"
+  cat > "$outfile" << JSON
+{
+  "score": $score,
+  "pass": $pass_flag,
+  "passingScore": $passing
+}
+JSON
+  echo "Score: $score / 100  (passing: $passing)  pass=$pass_flag"
+}
+
+# ── Common static-analysis checks ────────────────────────────────────────────
+# Each function: returns 0 on pass, 1 on fail/warn (for caller logic).
+# All feedback goes through pass()/fail()/warn() so it appears in print_results.
+
+check_no_print_statements() {
+  # Usage: check_no_print_statements <dir> [label]
+  # Flags bare print() calls that should be logging calls.
+  local dir="${1:-.}"
+  local label="${2:-$dir}"
+  local found
+  found=$(grep -rn "^[[:space:]]*print(" "$dir" --include="*.py" 2>/dev/null | grep -v "# noqa" || true)
+  if [[ -n "$found" ]]; then
+    local count
+    count=$(echo "$found" | wc -l | tr -d ' ')
+    warn "$label: $count print() call(s) found — use logging.info/warning/error instead (see Week 1 Ch1)"
+    return 1
+  fi
+  return 0
+}
+
+check_no_notimplemented() {
+  # Usage: check_no_notimplemented <dir> [label]
+  # Flags NotImplementedError stubs left in after implementation.
+  local dir="${1:-.}"
+  local label="${2:-$dir}"
+  local found
+  found=$(grep -rn "raise NotImplementedError" "$dir" --include="*.py" 2>/dev/null || true)
+  if [[ -n "$found" ]]; then
+    fail "$label: raise NotImplementedError still present — remove stubs before submitting"
+    return 1
+  fi
+  return 0
+}
+
+check_no_relative_imports() {
+  # Usage: check_no_relative_imports <dir> [label]
+  # Flags `from .module import x` in scripts not inside a proper package.
+  # Relative imports break the grader: python3 src/cleaner.py fails with
+  # "attempted relative import with no known parent package".
+  local dir="${1:-.}"
+  local label="${2:-$dir}"
+  local found
+  found=$(grep -rn "^from \." "$dir" --include="*.py" 2>/dev/null || true)
+  if [[ -n "$found" ]]; then
+    fail "$label: relative import found (from .module) — use absolute: 'from src.module import x'"
+    return 1
+  fi
+  return 0
+}
+
+check_no_logging_in_utils() {
+  # Usage: check_no_logging_in_utils <utils_file>
+  # utils.py should be pure helpers; logging config belongs in the entry point.
+  local file="${1:-task-1/src/utils.py}"
+  if [[ ! -f "$file" ]]; then return 0; fi
+  if grep -qE "logging\.basicConfig|logging\.getLogger" "$file"; then
+    warn "$file: logging.basicConfig/getLogger found — logging setup belongs in cleaner.py or the entry-point, not in utils"
+    return 1
+  fi
+  return 0
+}
+
+check_gitignore_python() {
+  # Usage: check_gitignore_python [<gitignore_path>]
+  # Warns when Python cache patterns are absent from .gitignore.
+  local gi="${1:-.gitignore}"
+  if [[ ! -f "$gi" ]]; then
+    warn ".gitignore is missing — add one so __pycache__/ and *.pyc are not committed"
+    return 1
+  fi
+  local ok=true
+  if ! grep -q "__pycache__" "$gi"; then
+    warn ".gitignore missing __pycache__/ — Python bytecode cache dirs should not be committed"
+    ok=false
+  fi
+  if ! grep -qE "^\*\.pyc$|^.*\*\.pyc" "$gi"; then
+    warn ".gitignore missing *.pyc — compiled Python files should not be committed"
+    ok=false
+  fi
+  if ! grep -qE "^\.env$|^\.env\b" "$gi"; then
+    warn ".gitignore missing .env — secret files should not be committed"
+    ok=false
+  fi
+  [[ "$ok" = true ]] && pass ".gitignore correctly excludes __pycache__/, *.pyc, and .env"
+}
+
+check_screenshot_is_png() {
+  # Usage: check_screenshot_is_png <expected_path> [<wrong_ext_glob>]
+  # Awards full credit for .png, warns (and still credits) for .jpg/.jpeg,
+  # zero for missing. Matches the pattern flagged in c55 PR reviews.
+  local expected_png="$1"
+  local dir
+  dir="$(dirname "$expected_png")"
+  local base
+  base="$(basename "$expected_png" .png)"
+
+  if [[ -s "$expected_png" ]]; then
+    pass "screenshot is $expected_png (.png format ✓)"
+    return 0
+  fi
+  for ext in jpg jpeg; do
+    if [[ -s "$dir/$base.$ext" ]]; then
+      warn "screenshot is .$ext but should be .png — rename to $base.png (partial credit still given)"
+      return 1
+    fi
+  done
+  fail "screenshot missing: $expected_png not found"
+  return 2
+}
+
+check_silent_zero_in_except() {
+  # Usage: check_silent_zero_in_except <file>
+  # Detects the pattern: try: x = compute() / except: x = 0
+  # which silently corrupts data instead of skipping or raising.
+  local file="$1"
+  if [[ ! -f "$file" ]]; then return 0; fi
+  local found
+  found=$(python3 - "$file" 2>/dev/null << 'PY'
+import ast, sys
+try:
+    tree = ast.parse(open(sys.argv[1]).read())
+except SyntaxError:
+    sys.exit(0)
+for node in ast.walk(tree):
+    if isinstance(node, ast.ExceptHandler):
+        for stmt in node.body:
+            if isinstance(stmt, ast.Assign):
+                if isinstance(stmt.value, ast.Constant) and stmt.value.value == 0:
+                    print(f"line {stmt.lineno}: '{ast.unparse(stmt)}' — sets field to 0 in except block (silent data corruption)")
+PY
+)
+  if [[ -n "$found" ]]; then
+    warn "$file: silent 0-assignment in except block — skip the row or raise instead of setting to 0:\n    $found"
+    return 1
+  fi
+  return 0
+}
+
+check_exception_logged() {
+  # Usage: check_exception_logged <dir>
+  # Warns when except blocks log/print a message but don't include the
+  # exception variable (e, err, exc), meaning the error type is lost.
+  local dir="${1:-.}"
+  local found
+  found=$(python3 - "$dir" 2>/dev/null << 'PY'
+import ast, os, sys
+issues = []
+for root, _, files in os.walk(sys.argv[1]):
+    for fname in files:
+        if not fname.endswith(".py"):
+            continue
+        path = os.path.join(root, fname)
+        try:
+            tree = ast.parse(open(path).read())
+        except SyntaxError:
+            continue
+        for node in ast.walk(tree):
+            if not isinstance(node, ast.ExceptHandler):
+                continue
+            exc_var = node.name  # e.g. "e" in `except ValueError as e`
+            if not exc_var:
+                continue
+            for stmt in node.body:
+                for call in ast.walk(stmt):
+                    if not isinstance(call, ast.Call):
+                        continue
+                    # Is it a logging.* or print call?
+                    func = call.func
+                    is_log = (isinstance(func, ast.Attribute) and
+                              isinstance(func.value, ast.Name) and
+                              func.value.id == "logging")
+                    is_print = isinstance(func, ast.Name) and func.id == "print"
+                    if not (is_log or is_print):
+                        continue
+                    # Does the call reference the exception variable?
+                    src = ast.unparse(call)
+                    if exc_var not in src:
+                        issues.append(f"{path}:{call.lineno}: log message doesn't include exception variable '{exc_var}' — add it for easier debugging")
+if issues:
+    for i in issues[:3]:  # cap at 3 to keep output readable
+        print(i)
+PY
+)
+  if [[ -n "$found" ]]; then
+    warn "exception variable not included in log message (harder to debug):\n    $found"
+    return 1
+  fi
+  return 0
+}
+
+check_ruff() {
+  # Usage: check_ruff <dir> [<select>]
+  # Runs ruff on <dir> if available; warns on violations.
+  # Default select: F401 (unused imports), E302 (missing blank lines).
+  local dir="${1:-.}"
+  local select="${2:-F401,E302,E303}"
+  if ! command -v ruff &>/dev/null && ! python3 -m ruff --version &>/dev/null 2>&1; then
+    return 0  # ruff not installed — skip silently
+  fi
+  local out
+  out=$(python3 -m ruff check --select="$select" --output-format=text "$dir" 2>/dev/null || true)
+  if [[ -n "$out" ]]; then
+    local count
+    count=$(echo "$out" | grep -c "\.py:" || true)
+    warn "$dir: ruff found $count style issue(s) (unused imports / missing blank lines) — run 'ruff check $dir' to see details"
+    return 1
+  fi
+  pass "$dir: no ruff style issues (F401/E302/E303)"
+  return 0
+}
diff --git a/.hyf/test.sh b/.hyf/test.sh
index c786914..d01cce9 100755
--- a/.hyf/test.sh
+++ b/.hyf/test.sh
@@ -11,6 +11,9 @@ SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
 REPO_ROOT="$(cd "$SCRIPT_DIR/.." && pwd)"
 cd "$REPO_ROOT"
 
+# shellcheck source=.hyf/grader_lib.sh
+source "$SCRIPT_DIR/grader_lib.sh"
+
 PASSING=60
 
 # --- Task 1: Cleaner Pipeline (60 points) ---
@@ -186,40 +189,87 @@ fi
 # Screenshot is required (10 pts); blob_url.txt with a valid Azure Storage
 # URL earns the remaining 10 pts. Both checks live inside the screenshot
 # branch — no screenshot means 0/20 regardless of blob_url.txt.
+# Full credit for .png (the required format). Partial credit + warning for
+# .jpg/.jpeg (wrong extension — matches c55 review pattern).
 task3=0
 task3_msg="missing task-3/assets/azure_blob_week2.png (or .jpg/.jpeg)"
+screenshot_found=false
 for ext in png jpg jpeg; do
     if [ -s "task-3/assets/azure_blob_week2.$ext" ]; then
-        task3=10
+        screenshot_found=true
+        if [ "$ext" != "png" ]; then
+            warn "Task 3: screenshot is .$ext but the assignment requires .png — rename to azure_blob_week2.png"
+            task3=7  # partial credit for wrong format
+            task3_msg="azure_blob_week2.$ext present but wrong format — rename to .png"
+        else
+            task3=10
+            task3_msg="azure_blob_week2.png present"
+        fi
         if [ -s "task-3/assets/blob_url.txt" ]; then
             # Require at least <container>/<blob> after the host so a bare
             # storage-account root URL doesn't satisfy the check.
             if grep -qE "https://[a-z0-9]+\.blob\.core\.windows\.net/[^/]+/[^/]+" task-3/assets/blob_url.txt; then
-                task3=20
-                task3_msg="screenshot and blob URL both present"
+                task3=$((task3 + 10))
+                task3_msg="${task3_msg}; blob_url.txt valid"
             else
-                task3_msg="blob_url.txt present but URL format is wrong — expected https://<account>.blob.core.windows.net/<container>/<blob>"
+                task3_msg="${task3_msg}; blob_url.txt present but URL format is wrong — expected https://<account>.blob.core.windows.net/<container>/<blob>"
             fi
         else
-            task3_msg="screenshot present but task-3/assets/blob_url.txt is missing"
+            task3_msg="${task3_msg}; task-3/assets/blob_url.txt is missing"
         fi
         break
     fi
 done
 
 score=$((task1 + task2 + task3))
-if [ "$score" -ge "$PASSING" ]; then pass=true; else pass=false; fi
-
-cat > "$SCRIPT_DIR/score.json" <<EOF
-{
-  "score": $score,
-  "pass": $pass,
-  "passingScore": $PASSING
-}
-EOF
-
-echo "Task 1 (Cleaner Pipeline): $task1/60 — $task1_msg"
-echo "Task 2 (AI Debug Report):  $task2/20 — $task2_msg"
+
+# ── Code hygiene warnings (0 pts — informational only) ──────────────────────
+# These checks mirror recurring review comments from cohort c55. They do not
+# affect the score but surface issues the teacher would otherwise flag manually.
+echo ""
+echo "--- Code Hygiene (warnings only, do not affect score) ---"
+
+# print() should be replaced with logging.* — chapter mandates logging.
+check_no_print_statements "task-1/src" "task-1/src"
+
+# NotImplementedError stubs left in after implementation.
+# Reviewer comment: "please remove the NotImplementedError from your helpers
+# since you've already added the functionality".
+check_no_notimplemented "task-1/src" "task-1/src"
+
+# Silent 0-assignment in except blocks corrupts data silently.
+# Reviewer comment: "sets price/revenue/vat to 0 instead of skipping the row,
+# a bad price would silently corrupt the output rather than being dropped".
+check_silent_zero_in_except "task-1/src/transforms.py"
+
+# Exception variable not included in log message loses error context.
+# Reviewer comment: "I would log the error type for easier debug!"
+check_exception_logged "task-1/src"
+
+# Unused imports and missing blank lines — caught by ruff F401/E302.
+# Reviewer comments: "This import isn't used anywhere" and
+# "need to add new line after the end of each function".
+check_ruff "task-1/src" "F401,E302,E303"
+
+# AI_DEBUG.md should include the full traceback.
+# Reviewer comment: "Would be good if the full traceback error was pasted here".
+if [ -s task-2/AI_DEBUG.md ]; then
+    if ! grep -q "Traceback" task-2/AI_DEBUG.md; then
+        warn "AI_DEBUG.md: no 'Traceback' found — paste the full error traceback in the '## The Error' section"
+    fi
+fi
+
+# .gitignore should exclude Python cache files.
+check_gitignore_python ".gitignore"
+
+print_results "Week 2 Autograder"
+
+# ── Final score ──────────────────────────────────────────────────────────────
+write_score "$score" "$PASSING" "$SCRIPT_DIR/score.json"
+
+echo ""
+echo "Task 1 (Cleaner Pipeline):  $task1/60 — $task1_msg"
+echo "Task 2 (AI Debug Report):   $task2/20 — $task2_msg"
 echo "Task 3 (Azure Blob Upload): $task3/20 — $task3_msg"
 echo "----------------------------------------"
-echo "Total: $score/100 — pass=$pass (passing threshold: $PASSING)"
+echo "Total: $score/100 — (passing threshold: $PASSING)"

From 5cc550456af55d76ec58add916208ae85ec27be3 Mon Sep 17 00:00:00 2001
From: Lasse Benninga <devops.pipeline@example.com>
Date: Fri, 22 May 2026 13:54:22 +0200
Subject: [PATCH 2/3] fix(grader_lib): use if-then instead of [[]] && to avoid
 set -e exit

---
 .hyf/grader_lib.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.hyf/grader_lib.sh b/.hyf/grader_lib.sh
index 3478d4f..d383a4a 100755
--- a/.hyf/grader_lib.sh
+++ b/.hyf/grader_lib.sh
@@ -121,7 +121,7 @@ check_gitignore_python() {
     warn ".gitignore missing .env — secret files should not be committed"
     ok=false
   fi
-  [[ "$ok" = true ]] && pass ".gitignore correctly excludes __pycache__/, *.pyc, and .env"
+  if [[ "$ok" = true ]]; then pass ".gitignore correctly excludes __pycache__/, *.pyc, and .env"; fi
 }
 
 check_screenshot_is_png() {

From 87923efffc3fe6f46c2e33f8b326744a989761b3 Mon Sep 17 00:00:00 2001
From: Lasse Benninga <devops.pipeline@example.com>
Date: Fri, 22 May 2026 14:07:53 +0200
Subject: [PATCH 3/3] fix(grader_lib): all check_* functions always return 0
 (safe under set -e)

Same fix as week-1. Verified: scaffold 20/100 fail, solution 100/100 pass.
---
 .hyf/grader_lib.sh | 56 ++++++++++++++++++----------------------------
 1 file changed, 22 insertions(+), 34 deletions(-)

diff --git a/.hyf/grader_lib.sh b/.hyf/grader_lib.sh
index d383a4a..1871010 100755
--- a/.hyf/grader_lib.sh
+++ b/.hyf/grader_lib.sh
@@ -3,9 +3,9 @@
 # Source this at the top of test.sh:
 #   source "$(dirname "$0")/grader_lib.sh"
 #
-# Provides: pass(), fail(), warn(), print_results(), write_score(),
-# and a set of common static-analysis checks derived from recurring
-# PR review patterns across cohort c55.
+# Design rule: every check_* function always returns 0.
+# Results are communicated via pass()/fail()/warn() into _grader_details,
+# never via exit code. This makes bare calls safe under `set -euo pipefail`.
 
 _grader_details=()
 
@@ -39,8 +39,8 @@ JSON
 }
 
 # ── Common static-analysis checks ────────────────────────────────────────────
-# Each function: returns 0 on pass, 1 on fail/warn (for caller logic).
-# All feedback goes through pass()/fail()/warn() so it appears in print_results.
+# All check_* functions always return 0. Findings are recorded via
+# pass()/fail()/warn() and printed by print_results().
 
 check_no_print_statements() {
   # Usage: check_no_print_statements <dir> [label]
@@ -53,7 +53,6 @@ check_no_print_statements() {
     local count
     count=$(echo "$found" | wc -l | tr -d ' ')
     warn "$label: $count print() call(s) found — use logging.info/warning/error instead (see Week 1 Ch1)"
-    return 1
   fi
   return 0
 }
@@ -67,7 +66,6 @@ check_no_notimplemented() {
   found=$(grep -rn "raise NotImplementedError" "$dir" --include="*.py" 2>/dev/null || true)
   if [[ -n "$found" ]]; then
     fail "$label: raise NotImplementedError still present — remove stubs before submitting"
-    return 1
   fi
   return 0
 }
@@ -83,7 +81,6 @@ check_no_relative_imports() {
   found=$(grep -rn "^from \." "$dir" --include="*.py" 2>/dev/null || true)
   if [[ -n "$found" ]]; then
     fail "$label: relative import found (from .module) — use absolute: 'from src.module import x'"
-    return 1
   fi
   return 0
 }
@@ -95,7 +92,6 @@ check_no_logging_in_utils() {
   if [[ ! -f "$file" ]]; then return 0; fi
   if grep -qE "logging\.basicConfig|logging\.getLogger" "$file"; then
     warn "$file: logging.basicConfig/getLogger found — logging setup belongs in cleaner.py or the entry-point, not in utils"
-    return 1
   fi
   return 0
 }
@@ -106,7 +102,7 @@ check_gitignore_python() {
   local gi="${1:-.gitignore}"
   if [[ ! -f "$gi" ]]; then
     warn ".gitignore is missing — add one so __pycache__/ and *.pyc are not committed"
-    return 1
+    return 0
   fi
   local ok=true
   if ! grep -q "__pycache__" "$gi"; then
@@ -122,18 +118,17 @@ check_gitignore_python() {
     ok=false
   fi
   if [[ "$ok" = true ]]; then pass ".gitignore correctly excludes __pycache__/, *.pyc, and .env"; fi
+  return 0
 }
 
 check_screenshot_is_png() {
-  # Usage: check_screenshot_is_png <expected_path> [<wrong_ext_glob>]
-  # Awards full credit for .png, warns (and still credits) for .jpg/.jpeg,
-  # zero for missing. Matches the pattern flagged in c55 PR reviews.
+  # Usage: check_screenshot_is_png <expected_path>
+  # Awards full credit for .png, warns for .jpg/.jpeg, fails if missing.
   local expected_png="$1"
   local dir
   dir="$(dirname "$expected_png")"
   local base
   base="$(basename "$expected_png" .png)"
-
   if [[ -s "$expected_png" ]]; then
     pass "screenshot is $expected_png (.png format ✓)"
     return 0
@@ -141,17 +136,16 @@ check_screenshot_is_png() {
   for ext in jpg jpeg; do
     if [[ -s "$dir/$base.$ext" ]]; then
       warn "screenshot is .$ext but should be .png — rename to $base.png (partial credit still given)"
-      return 1
+      return 0
     fi
   done
   fail "screenshot missing: $expected_png not found"
-  return 2
+  return 0
 }
 
 check_silent_zero_in_except() {
   # Usage: check_silent_zero_in_except <file>
-  # Detects the pattern: try: x = compute() / except: x = 0
-  # which silently corrupts data instead of skipping or raising.
+  # Detects: try: x = compute() / except: x = 0 (silent data corruption).
   local file="$1"
   if [[ ! -f "$file" ]]; then return 0; fi
   local found
@@ -170,16 +164,14 @@ for node in ast.walk(tree):
 PY
 )
   if [[ -n "$found" ]]; then
-    warn "$file: silent 0-assignment in except block — skip the row or raise instead of setting to 0:\n    $found"
-    return 1
+    warn "$file: silent 0-assignment in except block — skip the row or raise instead:\n    $found"
   fi
   return 0
 }
 
 check_exception_logged() {
   # Usage: check_exception_logged <dir>
-  # Warns when except blocks log/print a message but don't include the
-  # exception variable (e, err, exc), meaning the error type is lost.
+  # Warns when an except block logs a message but omits the exception variable.
   local dir="${1:-.}"
   local found
   found=$(python3 - "$dir" 2>/dev/null << 'PY'
@@ -197,14 +189,13 @@ for root, _, files in os.walk(sys.argv[1]):
         for node in ast.walk(tree):
             if not isinstance(node, ast.ExceptHandler):
                 continue
-            exc_var = node.name  # e.g. "e" in `except ValueError as e`
+            exc_var = node.name
             if not exc_var:
                 continue
             for stmt in node.body:
                 for call in ast.walk(stmt):
                     if not isinstance(call, ast.Call):
                         continue
-                    # Is it a logging.* or print call?
                     func = call.func
                     is_log = (isinstance(func, ast.Attribute) and
                               isinstance(func.value, ast.Name) and
@@ -212,30 +203,27 @@ for root, _, files in os.walk(sys.argv[1]):
                     is_print = isinstance(func, ast.Name) and func.id == "print"
                     if not (is_log or is_print):
                         continue
-                    # Does the call reference the exception variable?
                     src = ast.unparse(call)
                     if exc_var not in src:
-                        issues.append(f"{path}:{call.lineno}: log message doesn't include exception variable '{exc_var}' — add it for easier debugging")
+                        issues.append(f"{path}:{call.lineno}: log message doesn't include '{exc_var}' — add it for easier debugging")
 if issues:
-    for i in issues[:3]:  # cap at 3 to keep output readable
+    for i in issues[:3]:
         print(i)
 PY
 )
   if [[ -n "$found" ]]; then
-    warn "exception variable not included in log message (harder to debug):\n    $found"
-    return 1
+    warn "exception variable missing from log message:\n    $found"
   fi
   return 0
 }
 
 check_ruff() {
   # Usage: check_ruff <dir> [<select>]
-  # Runs ruff on <dir> if available; warns on violations.
-  # Default select: F401 (unused imports), E302 (missing blank lines).
+  # Runs ruff if available; warns on F401 (unused imports) and E302/E303 (blank lines).
   local dir="${1:-.}"
   local select="${2:-F401,E302,E303}"
   if ! command -v ruff &>/dev/null && ! python3 -m ruff --version &>/dev/null 2>&1; then
-    return 0  # ruff not installed — skip silently
+    return 0
   fi
   local out
   out=$(python3 -m ruff check --select="$select" --output-format=text "$dir" 2>/dev/null || true)
@@ -243,8 +231,8 @@ check_ruff() {
     local count
     count=$(echo "$out" | grep -c "\.py:" || true)
     warn "$dir: ruff found $count style issue(s) (unused imports / missing blank lines) — run 'ruff check $dir' to see details"
-    return 1
+  else
+    pass "$dir: no ruff style issues (F401/E302/E303)"
   fi
-  pass "$dir: no ruff style issues (F401/E302/E303)"
   return 0
 }