Fix #97: scope REPORT_DIR to the Test Reports section

[laird]

Closes #97

Root Cause

regression-test.sh selected the report directory with a global first match:

REPORT_DIR=$(grep "Location: " CLAUDE.md | sed '...' | head -1)

CLAUDE.md has two Location: lines and the Unit Tests one comes first, so REPORT_DIR became tests/test_*.sh and mkdir -p created a directory with that literal name.

Impact

Worse than a misplaced report — the directory shadows the project's own documented test glob. Once it exists:

for t in tests/test_*.sh; do bash "$t" || exit 1; done

yields the directory, bash exits 126, and the unit suite goes red with no failing test behind it. Reports also never reached docs/test/regression-reports/.

Solution

• Scope the lookup to the **Test Reports** section rather than taking the first global match.
• Reject a resolved REPORT_DIR that still contains glob metacharacters (*?[) — that is a parse failure, not a directory name — falling back to the default.

Changes Made

• plugins/autocoder/scripts/regression-test.sh — section-scoped lookup + metacharacter guard.
• tests/test_regression_report_dir.sh (new) — asserts no glob-named path is ever created and the report lands in docs/test/regression-reports/. Two scenarios: the real CLAUDE.md section ordering, and a Test Reports location that is itself a glob, so the guard cannot rot into dead code.

Verification

• Test written first, confirmed red: FAIL: regression-test.sh created a path literally named 'tests/test_*.sh'.
• Both guards proven non-vacuous by reverting each independently and observing the matching failure.
• 18/18 shell unit tests pass; bash -n and py_compile clean.
• End-to-end against the real CLAUDE.md: report now written to docs/test/regression-reports/, no stray directory.

Scope

Distinct from #73/#85 (unit-test counting and exit codes); this is the REPORT_DIR selection path only. Independent of #96 — both branch from master and touch different files.

Note: on this base the harness still reports 0/0 passed as green — that is the #73/#85 defect, unchanged here and still behind the blocked #74 stack.

🤖 Generated with Claude Code

[laird]

Superseded — master already fixed this (see #97, closed with the master line numbers). Closing without merging to avoid re-applying a stale branch's version of this fix over master's current one.