test(#146): clarify t.TempDir() evaluation in subprocess env setup

DEV_LOOP_STATUS.md

@@ -1,96 +1,68 @@
-# Dev Loop Status — 2026-05-15 09:37 UTC
+# Dev Loop Status — 2026-05-15 11:58 UTC
 
-## Summary
+**Cron ID:** 5342ac81-4bbc-4e4c-a123-347a7788d50c  
+**Status:** ✅ HEALTHY — All tests passing, repo clean, ready for review & merge
 
-- **Review-bot status:** ✅ MAIN BRANCH CURRENT & HEALTHY
+## Quick Status
-- **Coverage:** 77.1% (↑ from 70.4%) — healthy trajectory
-- **Tests:** ✅ All passing
-- **Active development tracks:**
-  - issue-143: fetch doc-map config from trusted VCS ref (ready for review)
-  - issue-146: reuse resolved doc-map path early (ready for review)
-  - issue-150: add EvalSymlinks to validateDocmapPath (ready for review)
-  - issue-154: refactor subprocess test helpers (ready for review)
 
----
+- **Main branch:** Synced with origin/main (d855064)
+- **Tests:** All passing ✅ (7 packages, 80+ test cases, race detector clean)
+- **Test coverage:** **77.1%** overall
+  - budget: 92.0%
+  - review: 92.0%
+  - gitea: 85.2%
+  - github: 86.3%
+  - llm: 81.3%
+  - netutil: 85.7%
+  - cmd/review-bot: 54.3%
+- **Working tree:** Clean (no uncommitted changes)
 
-## Current State
+## PR Status & Recommended Actions
 
-### Main Branch
+### Ready to Merge (3 PRs)
-- **HEAD:** 1650343 (dev-loop cycle complete)
+These have `ready` label, passing tests, and are self-reviewed. Recommend merging in order:
-- **Status:** Clean, all tests passing, 77.1% coverage
-- **Recent work:** Issue #130 fixes merged and verified complete
 
-### Active Issue Branches (Ready for Review)
+| Order | PR | Issue | Type | Size | Status |
+|-------|----|----|------|------|--------|
+| 1️⃣ | #155 | #154 | Refactor | M | ✅ Ready |
+| 2️⃣ | #152 | #150 | Security | S | ✅ Ready |
+| 3️⃣ | #151 | #146 | Test | S | ✅ Ready |
 
-| Issue | Branch | Latest Commit | Status | Recommendation |
+**Merge strategy:** Sequential. All currently passing; no blocking dependencies.
-|-------|--------|---------------|--------|-----------------|
-| #143 | origin/issue-143 | 3222c76 | Ready | Review feature + tests, consider for merge |
-| #146 | origin/issue-146 | 9b64c60 | Ready | 2 new test cases + 1 fix, review completeness |
-| #150 | origin/issue-150 | 4dce8e4 | Ready | Symlink validation, security-sensitive |
-| #154 | origin/issue-154 | 2892dff | Ready | Refactor/cleanup, low-risk |
 
-### Priority Assessment
+### Awaiting AI-Review (2 PRs)
+These have passing tests and self-review but need ai-review before marking ready:
 
-**High Priority (Security/Risk):**
+| PR | Issue | Type | Size | Notes |
-- **#150** — EvalSymlinks for dir-symlink bypass (security fix)
+|----|-------|------|------|-------|
-- **#143** — Fetch doc-map from trusted VCS ref (trust boundary)
+| #156 | #141 | Feature | M | `validate-docmap` subcommand |
+| #153 | #143 | Feature | M | Fetch doc-map from VCS |
 
-**Medium Priority (Feature):**
+## Dev Loop Health
-- **#146** — Path resolution optimization + tests
 
-**Low Priority (Cleanup):**
+| Metric | Status | Details |
-- **#154** — Test refactoring
+|--------|--------|---------|
+| Main branch | ✅ Current | d855064 (2026-05-15 11:44 UTC) |
+| Working tree | ✅ Clean | Ready for fetch/merge |
+| Test suite | ✅ All pass | 7 packages, 80+ cases, ~2s runtime |
+| Race detector | ✅ Clean | No race conditions detected |
+| Coverage | ✅ 77.1% | Stable, no regressions |
+| Remotes | ✅ Current | origin/main up-to-date |
 
----
+## Recommendations
 
-## Coverage Trends
+1. **[IMMEDIATE] Merge 3 ready PRs** (#155 → #152 → #151)
+   - All provide foundational support for downstream features
+   - Safe to merge in sequence; no cross-PR dependencies
+   - Post-merge: dev-loop can run verification cycle
 
-| Package | Current | Previous | Δ |
+2. **Schedule AI-review for #156 and #153**
-|---------|---------|----------|---|
+   - Both feature-complete and test-passing
-| cmd/review-bot | TBD | 36.8% | ↑ |
+   - Waiting on code quality & design review
-| budget | 91.8% | 91.8% | → |
-| review | 91.5% | 91.5% | → |
-| llm | 81.3% | 81.3% | → |
-| **Total** | **77.1%** | **70.4%** | **↑6.7%** |
 
----
+## Cycle Complete ✅
 
-## Recommendations for Next Cycle
+Next dev-loop cycle will:
-
+- Verify post-merge state
-### Immediate (This Dev-Loop)
+- Update coverage tracking
-1. **Checkout #150** — Review symlink fix, run security tests
+- Monitor awaiting-review PRs for AI review status
-2. **Checkout #143** — Review doc-map config fetching, validate error handling
-3. **Decide merge order** — #150 or #143 first (dependency check)
-4. **Run full integration** — After each merge to catch regressions
-
-### Short-term (Next 1-2 cycles)
-- Pull #146 into main if no blockers
-- Merge #154 as low-risk cleanup
-- Check for any test coverage gaps post-merge
-- Monitor for regressions during next run
-
-### Ongoing
-- Continue tracking coverage trend (goal: >80%)
-- Document new security fixes (issue #150)
-- Review CONVENTIONS.md for consistency across new code
-
----
-
-## Worktrees
-
-- All stale worktrees cleaned in previous cycle ✅
-- Ready for new worktree setup if Aaron wants to work on next issue
-
----
-
-## Next Dev-Loop Cycle
-
-When dev-loop runs next (in ~4 hours):
-1. ✅ Verify main still current
-2. ✅ Re-run tests & coverage
-3. ✅ Check if any PRs merged (update local branches)
-4. ⚠️ Flag for human review if coverage drops or tests fail
-
----
-
-_Generated by dev-loop at 2026-05-15 09:37 UTC_

cmd/review-bot/main_test.go

@@ -1521,6 +1521,8 @@ func TestMainSubprocess_InvalidDocMapPath(t *testing.T) {
 	}
 
 	cmd := exec.Command(os.Args[0], "-test.run=TestMainSubprocess_InvalidDocMapPath")
+	// t.TempDir() is evaluated here in the outer process, producing a real directory
+	// that is passed as the GITHUB_WORKSPACE env var string to the subprocess.
 	cmd.Env = append(cleanEnv(),
 		"TEST_SUBPROCESS_MAIN=1",
 		"GITHUB_WORKSPACE="+t.TempDir(),
@@ -1558,6 +1560,8 @@ func TestMainSubprocess_InvalidDocMapFile(t *testing.T) {
 	}
 
 	cmd := exec.Command(os.Args[0], "-test.run=TestMainSubprocess_InvalidDocMapFile")
+	// t.TempDir() is evaluated here in the outer process, producing a real directory
+	// that is passed as the GITHUB_WORKSPACE env var string to the subprocess.
 	cmd.Env = append(cleanEnv(),
 		"TEST_SUBPROCESS_MAIN=1",
 		"GITHUB_WORKSPACE="+t.TempDir(),