Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
b615013e63 | ||
|
|
e4e63cdbb7 |
43
docs/issue-648-verification.md
Normal file
43
docs/issue-648-verification.md
Normal file
@@ -0,0 +1,43 @@
|
||||
# Issue #648 Verification
|
||||
|
||||
## Status: ✅ ALREADY IMPLEMENTED
|
||||
|
||||
`timmy-home#648` asked for a durable session harvest report for 2026-04-14.
|
||||
That repo-side deliverable is already present on `main`.
|
||||
|
||||
## Acceptance Criteria Check
|
||||
|
||||
1. ✅ Durable report artifact exists
|
||||
- Evidence: `reports/production/2026-04-14-session-harvest-report.md`
|
||||
2. ✅ Report preserves the original session ledger and names issue-body drift
|
||||
- Evidence: the report includes `## Delivered PR Ledger`, `## Triage Actions`, `## Blocked / Skip Items`, and `## Current Totals`
|
||||
3. ✅ Regression coverage already exists on `main`
|
||||
- Evidence: `tests/test_session_harvest_report_2026_04_14.py`
|
||||
4. ✅ Fresh verification passed from a new clone
|
||||
- Evidence: `python3 -m pytest tests/test_session_harvest_report_2026_04_14.py -q` → `4 passed in 0.03s`
|
||||
|
||||
## Evidence
|
||||
|
||||
### Existing report artifact on main
|
||||
- `reports/production/2026-04-14-session-harvest-report.md`
|
||||
- The report explicitly references `Source issue: timmy-home#648`
|
||||
- The report already records the delivered PR ledger, issue-body drift, triage actions, blocked items, and verified totals
|
||||
|
||||
### Existing regression test on main
|
||||
- `tests/test_session_harvest_report_2026_04_14.py`
|
||||
- The test already locks the report path, required headings, verified PR tokens, and follow-up issue state changes
|
||||
|
||||
## Verification Run
|
||||
|
||||
From a fresh clone on branch `fix/648`, before adding this verification note:
|
||||
|
||||
```text
|
||||
python3 -m pytest tests/test_session_harvest_report_2026_04_14.py -q
|
||||
.... [100%]
|
||||
4 passed in 0.03s
|
||||
```
|
||||
|
||||
## Recommendation
|
||||
|
||||
Close issue #648 as already implemented on `main`.
|
||||
This PR only adds the verification note so the open issue can be closed without redoing the report work.
|
||||
25
tests/test_issue_648_verification.py
Normal file
25
tests/test_issue_648_verification.py
Normal file
@@ -0,0 +1,25 @@
|
||||
from pathlib import Path
|
||||
|
||||
DOC = Path('docs/issue-648-verification.md')
|
||||
|
||||
|
||||
def read_doc() -> str:
|
||||
assert DOC.exists(), 'verification doc for issue #648 must exist'
|
||||
return DOC.read_text(encoding='utf-8')
|
||||
|
||||
|
||||
def test_verification_doc_exists_for_issue_648():
|
||||
assert DOC.exists(), 'verification doc for issue #648 must exist'
|
||||
|
||||
|
||||
def test_verification_doc_captures_existing_report_evidence():
|
||||
text = read_doc()
|
||||
for token in [
|
||||
'# Issue #648 Verification',
|
||||
'Status: ✅ ALREADY IMPLEMENTED',
|
||||
'reports/production/2026-04-14-session-harvest-report.md',
|
||||
'tests/test_session_harvest_report_2026_04_14.py',
|
||||
'4 passed',
|
||||
'Close issue #648',
|
||||
]:
|
||||
assert token in text
|
||||
Reference in New Issue
Block a user