fix: refresh burn lane audit for #662

docs/LAB_003_BATTERY_DISCONNECT_PACKET.md

@@ -1,74 +0,0 @@
-# LAB-003 — Truck Battery Disconnect Install Packet
-
-No battery disconnect switch has been purchased or installed yet.
-This packet turns the issue into a field-ready purchase / install / validation checklist while preserving what still requires live work.
-
-## Candidate Store Run
-
-- AutoZone — Newport or Claremont
-- Advance Auto Parts — Newport or Claremont
-- O'Reilly Auto Parts — Newport or Claremont
-
-## Required Items
-
-- battery terminal disconnect switch
-- terminal shim/post riser if needed
-
-## Selection Criteria
-
-- Fits the truck battery post without forcing the clamp
-- Mounts on the negative battery terminal
-- Physically secure once tightened
-- no special tools required to operate
-
-## Live Purchase State
-
-- Store selected: pending
-- Part selected: pending
-- Part cost: pending purchase
-
-## Installation Target
-
-- Install location: negative battery terminal
-- Ready to operate without tools: yes
-
-## Install Checklist
-
-- [ ] Verify the truck is off and keys are removed before touching the battery
-- [ ] Confirm the disconnect fits the negative battery terminal before final tightening
-- [ ] Install the disconnect on the negative battery terminal
-- [ ] Tighten until physically secure with no terminal wobble
-- [ ] Verify the disconnect can be opened and closed by hand
-
-## Validation Checklist
-
-- [ ] Leave the truck parked with the disconnect opened for at least 24 hours
-- [ ] Reconnect the switch by hand the next day
-- [ ] Truck starts reliably after sitting 24+ hours with switch disconnected
-- [ ] Receipt or photo of installed switch uploaded to this issue
-
-## Overnight Verification Log
-
-- Install completed: False
-- Physically secure: False
-- Overnight disconnect duration: pending
-- Truck started after disconnect: pending
-- Receipt / photo path: pending
-
-## Battery Replacement Fallback
-
-If the truck still fails the overnight test after the disconnect install, replace battery and re-run the 24-hour validation.
-
-## Missing Live Fields
-
-- store_selected
-- part_name
-- install_completed
-- physically_secure
-- overnight_test_hours
-- truck_started_after_disconnect
-- receipt_or_photo_path
-
-## Honest next step
-
-Buy the disconnect switch, install it on the negative battery terminal, leave the truck disconnected for 24+ hours, and only close the issue after receipt/photo evidence and the overnight start result are attached.

reports/production/2026-04-16-burn-lane-empty-audit.md

@@ -1,6 +1,6 @@
 # Burn Lane Empty Audit — timmy-home #662
 
-Generated: 2026-04-16T01:22:37Z
+Generated: 2026-04-17T03:42:50Z
 Source issue: `[ops] Burn lane empty — all open issues triaged (2026-04-14)`
 
 ## Source Snapshot
@@ -11,9 +11,9 @@ Issue #662 is an operational status note, not a normal feature request. Its body
 
 - Referenced issues audited: 42
 - Already closed: 30
-- Open but likely closure candidates (merged PR found): 0
-- Open with active PRs: 12
-- Open / needs manual review: 0
+- Open but likely closure candidates (merged PR found): 1
+- Open with active PRs: 0
+- Open / needs manual review: 11
 
 ## Issue Body Drift
 
@@ -21,56 +21,56 @@ The body of #662 is not current truth. It mixes closed issues, open issues, rang
 
 | Issue | State | Classification | PR Summary |
 |---|---|---|---|
-| #579 | closed | already closed | closed PR #644, closed PR #640, closed PR #635, closed PR #620 |
-| #648 | open | active pr | open PR #731 |
+| #579 | closed | already closed | closed PR #644, closed PR #643, closed PR #640, closed PR #635, closed PR #620 |
+| #648 | open | needs manual review | closed PR #731 |
 | #647 | closed | already closed | issue already closed |
 | #619 | closed | already closed | issue already closed |
 | #616 | closed | already closed | issue already closed |
 | #614 | closed | already closed | issue already closed |
 | #613 | closed | already closed | issue already closed |
 | #660 | closed | already closed | issue already closed |
-| #659 | closed | already closed | issue already closed |
+| #659 | closed | already closed | closed PR #660 |
 | #658 | closed | already closed | issue already closed |
 | #657 | closed | already closed | issue already closed |
 | #656 | closed | already closed | closed PR #658 |
 | #655 | closed | already closed | issue already closed |
 | #654 | closed | already closed | closed PR #661 |
-| #653 | closed | already closed | issue already closed |
-| #652 | closed | already closed | merged PR #657 |
-| #651 | closed | already closed | issue already closed |
-| #650 | closed | already closed | merged PR #654 |
-| #649 | closed | already closed | issue already closed |
-| #646 | closed | already closed | issue already closed |
-| #582 | open | active pr | open PR #738 |
+| #653 | closed | already closed | closed PR #660, closed PR #655 |
+| #652 | closed | already closed | closed PR #660, merged PR #657, closed PR #655 |
+| #651 | closed | already closed | closed PR #655 |
+| #650 | closed | already closed | closed PR #661, closed PR #660, merged PR #654, closed PR #651 |
+| #649 | closed | already closed | closed PR #660, merged PR #657, closed PR #651 |
+| #646 | closed | already closed | closed PR #655, closed PR #651 |
+| #582 | open | closure candidate | merged PR #641, merged PR #639, merged PR #637, merged PR #631, merged PR #630 |
 | #627 | closed | already closed | issue already closed |
 | #631 | closed | already closed | issue already closed |
 | #632 | closed | already closed | issue already closed |
 | #634 | closed | already closed | issue already closed |
 | #639 | closed | already closed | issue already closed |
 | #641 | closed | already closed | issue already closed |
-| #575 | closed | already closed | merged PR #656 |
-| #576 | closed | already closed | closed PR #663, closed PR #660, closed PR #655, closed PR #651, closed PR #646, closed PR #642, closed PR #633 |
+| #575 | closed | already closed | closed PR #658, merged PR #656 |
+| #576 | closed | already closed | merged PR #664, closed PR #663, closed PR #660, closed PR #655, merged PR #654, closed PR #651, closed PR #646, closed PR #642, closed PR #633 |
 | #578 | closed | already closed | merged PR #638, closed PR #636 |
 | #636 | closed | already closed | issue already closed |
 | #638 | closed | already closed | issue already closed |
-| #547 | open | active pr | open PR #730 |
-| #548 | open | active pr | open PR #712 |
-| #549 | open | active pr | open PR #729 |
-| #550 | open | active pr | open PR #727 |
-| #551 | open | active pr | open PR #725 |
-| #552 | open | active pr | open PR #724 |
-| #553 | open | active pr | open PR #722 |
-| #562 | open | active pr | open PR #718 |
-| #544 | open | active pr | open PR #732 |
-| #545 | open | active pr | open PR #719 |
+| #547 | open | needs manual review | closed PR #730 |
+| #548 | open | needs manual review | closed PR #712 |
+| #549 | open | needs manual review | closed PR #729 |
+| #550 | open | needs manual review | closed PR #727 |
+| #551 | open | needs manual review | closed PR #725 |
+| #552 | open | needs manual review | closed PR #724 |
+| #553 | open | needs manual review | closed PR #722 |
+| #562 | open | needs manual review | closed PR #718 |
+| #544 | open | needs manual review | closed PR #732 |
+| #545 | open | needs manual review | closed PR #719 |
 
 ## Closure Candidates
 
 These issues are still open but already have merged PR evidence in the forge and should be reviewed for bulk closure.
 
-| None |
-|---|
-| None |
+| Issue | State | Classification | PR Summary |
+|---|---|---|---|
+| #582 | open | closure candidate | merged PR #641, merged PR #639, merged PR #637, merged PR #631, merged PR #630 |
 
 ## Still Open / Needs Manual Review
 
@@ -78,18 +78,17 @@ These issues either have no matching PR signal or still have an active PR / ambi
 
 | Issue | State | Classification | PR Summary |
 |---|---|---|---|
-| #648 | open | active pr | open PR #731 |
-| #582 | open | active pr | open PR #738 |
-| #547 | open | active pr | open PR #730 |
-| #548 | open | active pr | open PR #712 |
-| #549 | open | active pr | open PR #729 |
-| #550 | open | active pr | open PR #727 |
-| #551 | open | active pr | open PR #725 |
-| #552 | open | active pr | open PR #724 |
-| #553 | open | active pr | open PR #722 |
-| #562 | open | active pr | open PR #718 |
-| #544 | open | active pr | open PR #732 |
-| #545 | open | active pr | open PR #719 |
+| #648 | open | needs manual review | closed PR #731 |
+| #547 | open | needs manual review | closed PR #730 |
+| #548 | open | needs manual review | closed PR #712 |
+| #549 | open | needs manual review | closed PR #729 |
+| #550 | open | needs manual review | closed PR #727 |
+| #551 | open | needs manual review | closed PR #725 |
+| #552 | open | needs manual review | closed PR #724 |
+| #553 | open | needs manual review | closed PR #722 |
+| #562 | open | needs manual review | closed PR #718 |
+| #544 | open | needs manual review | closed PR #732 |
+| #545 | open | needs manual review | closed PR #719 |
 
 ## Recommendation
 

scripts/burn_lane_issue_audit.py

@@ -23,6 +23,7 @@ class PullSummary:
     state: str
     merged: bool
     head: str
+    body: str
     url: str
 
 
@@ -75,7 +76,8 @@ def api_get(path: str, token: str):
 def collect_pull_summaries(repo: str, token: str) -> list[PullSummary]:
     pulls: list[PullSummary] = []
     for state in ("open", "closed"):
-        for page in range(1, 6):
+        page = 1
+        while True:
             batch = api_get(f"/repos/{ORG}/{repo}/pulls?state={state}&limit=100&page={page}", token)
             if not batch:
                 break
@@ -87,18 +89,18 @@ def collect_pull_summaries(repo: str, token: str) -> list[PullSummary]:
                         state=pr.get("state") or state,
                         merged=bool(pr.get("merged")),
                         head=(pr.get("head") or {}).get("ref") or "",
+                        body=pr.get("body") or "",
                         url=pr.get("html_url") or pr.get("url") or "",
                     )
                 )
-            if len(batch) < 100:
-                break
+            page += 1
     return pulls
 
 
 def match_prs(issue_num: int, pulls: Iterable[PullSummary]) -> list[PullSummary]:
     matches: list[PullSummary] = []
     for pr in pulls:
-        text = f"{pr.title} {pr.head}"
+        text = f"{pr.title} {pr.head} {pr.body}"
         if f"#{issue_num}" in text or pr.head == f"fix/{issue_num}" or f"/{issue_num}" in pr.head or f"-{issue_num}" in pr.head:
             matches.append(pr)
     return matches
@@ -116,12 +118,16 @@ def classify_issue(issue: dict, related_prs: list[PullSummary]) -> IssueAuditRow
     else:
         merged = [pr for pr in related_prs if pr.merged]
         open_prs = [pr for pr in related_prs if pr.state == "open"]
+        closed_unmerged = [pr for pr in related_prs if pr.state != "open" and not pr.merged]
         if merged:
             classification = "closure_candidate"
             pr_summary = summarize_prs(merged)
         elif open_prs:
             classification = "active_pr"
             pr_summary = summarize_prs(open_prs)
+        elif closed_unmerged:
+            classification = "needs_manual_review"
+            pr_summary = summarize_prs(closed_unmerged)
         else:
             classification = "needs_manual_review"
             pr_summary = "no matching PR found"

scripts/lab_003_battery_disconnect_packet.py

@@ -1,267 +0,0 @@
-#!/usr/bin/env python3
-"""Prepare a field-ready install packet for LAB-003 truck battery disconnect work."""
-
-from __future__ import annotations
-
-import argparse
-import json
-from pathlib import Path
-from typing import Any
-
-
-CANDIDATE_STORES = [
-    "AutoZone — Newport or Claremont",
-    "Advance Auto Parts — Newport or Claremont",
-    "O'Reilly Auto Parts — Newport or Claremont",
-]
-
-REQUIRED_ITEMS = [
-    "battery terminal disconnect switch",
-    "terminal shim/post riser if needed",
-]
-
-SELECTION_CRITERIA = [
-    "Fits the truck battery post without forcing the clamp",
-    "Mounts on the negative battery terminal",
-    "Physically secure once tightened",
-    "no special tools required to operate",
-]
-
-INSTALL_CHECKLIST = [
-    "Verify the truck is off and keys are removed before touching the battery",
-    "Confirm the disconnect fits the negative battery terminal before final tightening",
-    "Install the disconnect on the negative battery terminal",
-    "Tighten until physically secure with no terminal wobble",
-    "Verify the disconnect can be opened and closed by hand",
-]
-
-VALIDATION_CHECKLIST = [
-    "Leave the truck parked with the disconnect opened for at least 24 hours",
-    "Reconnect the switch by hand the next day",
-    "Truck starts reliably after sitting 24+ hours with switch disconnected",
-    "Receipt or photo of installed switch uploaded to this issue",
-]
-
-BATTERY_REPLACEMENT_FOLLOWUP = (
-    "If the truck still fails the overnight test after the disconnect install, "
-    "replace battery and re-run the 24-hour validation."
-)
-
-
-def _as_bool(value: Any) -> bool | None:
-    if value is None:
-        return None
-    if isinstance(value, bool):
-        return value
-    text = str(value).strip().lower()
-    if text in {"1", "true", "yes", "y"}:
-        return True
-    if text in {"0", "false", "no", "n"}:
-        return False
-    return None
-
-
-def build_packet(details: dict[str, Any]) -> dict[str, Any]:
-    store_selected = (details.get("store_selected") or "").strip()
-    part_name = (details.get("part_name") or "").strip()
-    receipt_or_photo_path = (details.get("receipt_or_photo_path") or "").strip()
-    install_completed = _as_bool(details.get("install_completed"))
-    physically_secure = _as_bool(details.get("physically_secure"))
-    truck_started = _as_bool(details.get("truck_started_after_disconnect"))
-    replacement_needed = _as_bool(details.get("replacement_battery_needed"))
-    overnight_test_hours = details.get("overnight_test_hours")
-    part_cost_usd = details.get("part_cost_usd")
-
-    try:
-        overnight_test_hours = int(overnight_test_hours) if overnight_test_hours is not None else None
-    except (TypeError, ValueError):
-        overnight_test_hours = None
-
-    try:
-        part_cost_usd = float(part_cost_usd) if part_cost_usd is not None else None
-    except (TypeError, ValueError):
-        part_cost_usd = None
-
-    missing_fields: list[str] = []
-    if not store_selected:
-        missing_fields.append("store_selected")
-    if not part_name:
-        missing_fields.append("part_name")
-    if install_completed is not True:
-        missing_fields.append("install_completed")
-    if physically_secure is not True:
-        missing_fields.append("physically_secure")
-    if overnight_test_hours is None:
-        missing_fields.append("overnight_test_hours")
-    if truck_started is None:
-        missing_fields.append("truck_started_after_disconnect")
-    if not receipt_or_photo_path:
-        missing_fields.append("receipt_or_photo_path")
-
-    ready_to_operate_without_tools = True
-
-    if replacement_needed is True or truck_started is False:
-        status = "battery_replace_candidate"
-    elif not store_selected or not part_name:
-        status = "pending_parts_run"
-    elif install_completed is not True:
-        status = "pending_install"
-    elif physically_secure is not True or overnight_test_hours is None or truck_started is None or not receipt_or_photo_path:
-        status = "overnight_validation"
-    elif overnight_test_hours >= 24 and truck_started is True:
-        status = "verified"
-    else:
-        status = "overnight_validation"
-
-    return {
-        "candidate_stores": list(CANDIDATE_STORES),
-        "required_items": list(REQUIRED_ITEMS),
-        "selection_criteria": list(SELECTION_CRITERIA),
-        "install_target": "negative battery terminal",
-        "install_checklist": list(INSTALL_CHECKLIST),
-        "validation_checklist": list(VALIDATION_CHECKLIST),
-        "store_selected": store_selected,
-        "part_name": part_name,
-        "part_cost_usd": part_cost_usd,
-        "install_completed": install_completed,
-        "physically_secure": physically_secure,
-        "overnight_test_hours": overnight_test_hours,
-        "truck_started_after_disconnect": truck_started,
-        "receipt_or_photo_path": receipt_or_photo_path,
-        "ready_to_operate_without_tools": ready_to_operate_without_tools,
-        "missing_fields": missing_fields,
-        "battery_replacement_followup": BATTERY_REPLACEMENT_FOLLOWUP,
-        "status": status,
-    }
-
-
-def render_markdown(packet: dict[str, Any]) -> str:
-    part_cost = packet["part_cost_usd"]
-    cost_line = f"${part_cost:.2f}" if isinstance(part_cost, (int, float)) else "pending purchase"
-    overnight = packet["overnight_test_hours"]
-    overnight_line = f"{overnight} hours" if overnight is not None else "pending"
-    started = packet["truck_started_after_disconnect"]
-    if started is True:
-        started_line = "yes"
-    elif started is False:
-        started_line = "no"
-    else:
-        started_line = "pending"
-
-    lines = [
-        "# LAB-003 — Truck Battery Disconnect Install Packet",
-        "",
-        "No battery disconnect switch has been purchased or installed yet.",
-        "This packet turns the issue into a field-ready purchase / install / validation checklist while preserving what still requires live work.",
-        "",
-        "## Candidate Store Run",
-        "",
-    ]
-    lines.extend(f"- {store}" for store in packet["candidate_stores"])
-    lines.extend([
-        "",
-        "## Required Items",
-        "",
-    ])
-    lines.extend(f"- {item}" for item in packet["required_items"])
-    lines.extend([
-        "",
-        "## Selection Criteria",
-        "",
-    ])
-    lines.extend(f"- {item}" for item in packet["selection_criteria"])
-    lines.extend([
-        "",
-        "## Live Purchase State",
-        "",
-        f"- Store selected: {packet['store_selected'] or 'pending'}",
-        f"- Part selected: {packet['part_name'] or 'pending'}",
-        f"- Part cost: {cost_line}",
-        "",
-        "## Installation Target",
-        "",
-        f"- Install location: {packet['install_target']}",
-        f"- Ready to operate without tools: {'yes' if packet['ready_to_operate_without_tools'] else 'no'}",
-        "",
-        "## Install Checklist",
-        "",
-    ])
-    lines.extend(f"- [ ] {item}" for item in packet["install_checklist"])
-    lines.extend([
-        "",
-        "## Validation Checklist",
-        "",
-    ])
-    lines.extend(f"- [ ] {item}" for item in packet["validation_checklist"])
-    lines.extend([
-        "",
-        "## Overnight Verification Log",
-        "",
-        f"- Install completed: {packet['install_completed'] if packet['install_completed'] is not None else 'pending'}",
-        f"- Physically secure: {packet['physically_secure'] if packet['physically_secure'] is not None else 'pending'}",
-        f"- Overnight disconnect duration: {overnight_line}",
-        f"- Truck started after disconnect: {started_line}",
-        f"- Receipt / photo path: {packet['receipt_or_photo_path'] or 'pending'}",
-        "",
-        "## Battery Replacement Fallback",
-        "",
-        packet['battery_replacement_followup'],
-        "",
-        "## Missing Live Fields",
-        "",
-    ])
-    if packet["missing_fields"]:
-        lines.extend(f"- {field}" for field in packet["missing_fields"])
-    else:
-        lines.append("- none")
-    lines.extend([
-        "",
-        "## Honest next step",
-        "",
-        "Buy the disconnect switch, install it on the negative battery terminal, leave the truck disconnected for 24+ hours, and only close the issue after receipt/photo evidence and the overnight start result are attached.",
-        "",
-    ])
-    return "\n".join(lines)
-
-
-def main() -> None:
-    parser = argparse.ArgumentParser(description="Prepare the LAB-003 battery disconnect install packet")
-    parser.add_argument("--store-selected", default="")
-    parser.add_argument("--part-name", default="")
-    parser.add_argument("--part-cost-usd", type=float, default=None)
-    parser.add_argument("--install-completed", action="store_true")
-    parser.add_argument("--physically-secure", action="store_true")
-    parser.add_argument("--overnight-test-hours", type=int, default=None)
-    parser.add_argument("--truck-started-after-disconnect", choices=["yes", "no"], default=None)
-    parser.add_argument("--receipt-or-photo-path", default="")
-    parser.add_argument("--replacement-battery-needed", action="store_true")
-    parser.add_argument("--output", default=None)
-    parser.add_argument("--json", action="store_true")
-    args = parser.parse_args()
-
-    packet = build_packet(
-        {
-            "store_selected": args.store_selected,
-            "part_name": args.part_name,
-            "part_cost_usd": args.part_cost_usd,
-            "install_completed": args.install_completed,
-            "physically_secure": args.physically_secure,
-            "overnight_test_hours": args.overnight_test_hours,
-            "truck_started_after_disconnect": args.truck_started_after_disconnect,
-            "receipt_or_photo_path": args.receipt_or_photo_path,
-            "replacement_battery_needed": args.replacement_battery_needed,
-        }
-    )
-    rendered = json.dumps(packet, indent=2) if args.json else render_markdown(packet)
-
-    if args.output:
-        output_path = Path(args.output).expanduser()
-        output_path.parent.mkdir(parents=True, exist_ok=True)
-        output_path.write_text(rendered, encoding="utf-8")
-        print(f"Battery disconnect packet written to {output_path}")
-    else:
-        print(rendered)
-
-
-if __name__ == "__main__":
-    main()

tests/test_burn_lane_issue_audit.py

@@ -1,6 +1,13 @@
 from pathlib import Path
 
-from scripts.burn_lane_issue_audit import extract_issue_numbers, render_report
+from scripts.burn_lane_issue_audit import (
+    PullSummary,
+    classify_issue,
+    collect_pull_summaries,
+    extract_issue_numbers,
+    match_prs,
+    render_report,
+)
 
 
 def test_extract_issue_numbers_handles_ranges_and_literals() -> None:
@@ -14,6 +21,99 @@ def test_extract_issue_numbers_handles_ranges_and_literals() -> None:
     assert extract_issue_numbers(body) == [579, 660, 659, 658, 582, 627, 631, 547, 546, 545]
 
 
+def test_match_prs_detects_issue_ref_in_pr_body() -> None:
+    pulls = [
+        PullSummary(
+            number=731,
+            title="docs: verify session harvest report",
+            state="open",
+            merged=False,
+            head="fix/session-harvest-report",
+            body="Refs #648",
+            url="https://forge.example/pr/731",
+        ),
+        PullSummary(
+            number=732,
+            title="unrelated",
+            state="open",
+            merged=False,
+            head="fix/unrelated",
+            body="Refs #700",
+            url="https://forge.example/pr/732",
+        ),
+    ]
+
+    assert [pr.number for pr in match_prs(648, pulls)] == [731]
+
+
+
+def test_open_issue_with_closed_unmerged_pr_stays_manual_review_with_history() -> None:
+    issue = {
+        "number": 648,
+        "title": "session harvest report",
+        "state": "open",
+        "html_url": "https://forge.example/issues/648",
+    }
+    row = classify_issue(
+        issue,
+        [
+            PullSummary(
+                number=731,
+                title="docs: add session harvest report",
+                state="closed",
+                merged=False,
+                head="fix/648",
+                body="Closes #648",
+                url="https://forge.example/pr/731",
+            )
+        ],
+    )
+
+    assert row.classification == "needs_manual_review"
+    assert row.pr_summary == "closed PR #731"
+
+
+
+def test_collect_pull_summaries_pages_until_empty(monkeypatch) -> None:
+    def fake_api_get(path: str, token: str):
+        if "state=open" in path:
+            return []
+        page = int(path.split("page=")[1])
+        if page <= 5:
+            return [
+                {
+                    "number": page * 1000 + i,
+                    "title": f"page {page} pr {i}",
+                    "state": "closed",
+                    "merged": False,
+                    "head": {"ref": f"fix/{page}-{i}"},
+                    "body": f"Refs #{page * 1000 + i}",
+                    "html_url": f"https://forge.example/pr/{page * 1000 + i}",
+                }
+                for i in range(100)
+            ]
+        if page == 6:
+            return [
+                {
+                    "number": 900,
+                    "title": "late page pr",
+                    "state": "closed",
+                    "merged": False,
+                    "head": {"ref": "fix/900"},
+                    "body": "Refs #900",
+                    "html_url": "https://forge.example/pr/900",
+                }
+            ]
+        return []
+
+    monkeypatch.setattr("scripts.burn_lane_issue_audit.api_get", fake_api_get)
+
+    pulls = collect_pull_summaries("timmy-home", "token")
+
+    assert any(pr.number == 900 for pr in pulls)
+
+
+
 def test_render_report_calls_out_drift_and_candidates() -> None:
     rows = [
         {

tests/test_lab_003_battery_disconnect_packet.py

@@ -1,88 +0,0 @@
-from pathlib import Path
-import importlib.util
-import unittest
-
-
-ROOT = Path(__file__).resolve().parent.parent
-SCRIPT_PATH = ROOT / "scripts" / "lab_003_battery_disconnect_packet.py"
-DOC_PATH = ROOT / "docs" / "LAB_003_BATTERY_DISCONNECT_PACKET.md"
-
-
-def load_module(path: Path, name: str):
-    assert path.exists(), f"missing {path.relative_to(ROOT)}"
-    spec = importlib.util.spec_from_file_location(name, path)
-    assert spec and spec.loader
-    module = importlib.util.module_from_spec(spec)
-    spec.loader.exec_module(module)
-    return module
-
-
-class TestLab003BatteryDisconnectPacket(unittest.TestCase):
-    def test_packet_defaults_to_parts_run_and_tracks_issue_specific_requirements(self):
-        mod = load_module(SCRIPT_PATH, "lab_003_battery_disconnect_packet")
-        packet = mod.build_packet({})
-
-        self.assertEqual(packet["status"], "pending_parts_run")
-        self.assertEqual(packet["install_target"], "negative battery terminal")
-        self.assertIn("battery terminal disconnect switch", packet["required_items"])
-        self.assertIn("terminal shim/post riser if needed", packet["required_items"])
-        self.assertIn("AutoZone", packet["candidate_stores"][0])
-        self.assertIn("no special tools required to operate", packet["selection_criteria"]) 
-        self.assertIn("overnight_test_hours", packet["missing_fields"]) 
-        self.assertIn("receipt_or_photo_path", packet["missing_fields"]) 
-
-    def test_packet_marks_verified_after_successful_24h_validation_with_proof(self):
-        mod = load_module(SCRIPT_PATH, "lab_003_battery_disconnect_packet")
-        packet = mod.build_packet(
-            {
-                "store_selected": "AutoZone - Newport",
-                "part_name": "Knob-style battery disconnect switch",
-                "part_cost_usd": 24.99,
-                "install_completed": True,
-                "physically_secure": True,
-                "overnight_test_hours": 26,
-                "truck_started_after_disconnect": True,
-                "receipt_or_photo_path": "evidence/lab-003-installed-switch.jpg",
-            }
-        )
-
-        self.assertEqual(packet["status"], "verified")
-        self.assertEqual(packet["missing_fields"], [])
-        self.assertTrue(packet["ready_to_operate_without_tools"])
-
-    def test_packet_flags_battery_replace_candidate_when_overnight_test_fails(self):
-        mod = load_module(SCRIPT_PATH, "lab_003_battery_disconnect_packet")
-        packet = mod.build_packet(
-            {
-                "store_selected": "O'Reilly - Claremont",
-                "part_name": "Knob-style battery disconnect switch",
-                "install_completed": True,
-                "physically_secure": True,
-                "overnight_test_hours": 24,
-                "truck_started_after_disconnect": False,
-            }
-        )
-
-        self.assertEqual(packet["status"], "battery_replace_candidate")
-        self.assertIn("battery_replacement_followup", packet)
-        self.assertIn("replace battery", packet["battery_replacement_followup"].lower())
-
-    def test_repo_contains_grounded_lab_003_packet_doc(self):
-        self.assertTrue(DOC_PATH.exists(), "missing committed LAB-003 packet doc")
-        text = DOC_PATH.read_text(encoding="utf-8")
-        for snippet in (
-            "# LAB-003 — Truck Battery Disconnect Install Packet",
-            "No battery disconnect switch has been purchased or installed yet.",
-            "negative battery terminal",
-            "AutoZone",
-            "Advance",
-            "O'Reilly",
-            "terminal shim/post riser if needed",
-            "Truck starts reliably after sitting 24+ hours with switch disconnected",
-            "Receipt or photo of installed switch uploaded to this issue",
-        ):
-            self.assertIn(snippet, text)
-
-
-if __name__ == "__main__":
-    unittest.main()