Compare commits
2 Commits
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
16fcabb5fc | ||
|
|
72159c1714 |
69
docs/issue-675-verification.md
Normal file
69
docs/issue-675-verification.md
Normal file
@@ -0,0 +1,69 @@
|
||||
# Issue #675 Verification
|
||||
|
||||
## Status: ✅ ALREADY IMPLEMENTED
|
||||
|
||||
`the-testament-GENOME.md` is already present on `timmy-home/main` and already delivers the requested full codebase analysis for `Timmy_Foundation/the-testament`.
|
||||
|
||||
This PR does not regenerate the genome. It adds the missing regression coverage and documents the evidence so issue #675 can be closed cleanly.
|
||||
|
||||
## Acceptance Criteria Check
|
||||
|
||||
1. ✅ Full genome artifact exists
|
||||
- `the-testament-GENOME.md` exists at repo root
|
||||
- it includes the required analysis sections:
|
||||
- Project Overview
|
||||
- Architecture
|
||||
- Entry Points
|
||||
- Data Flow
|
||||
- Key Abstractions
|
||||
- API Surface
|
||||
- Test Coverage Gaps
|
||||
- Security Considerations
|
||||
|
||||
2. ✅ Genome is grounded in real target-repo verification
|
||||
- the artifact explicitly references:
|
||||
- `scripts/build-verify.py --json`
|
||||
- `bash scripts/smoke.sh`
|
||||
- `python3 compile_all.py --check`
|
||||
- it also names target-repo architecture surfaces like:
|
||||
- `website/index.html`
|
||||
- `game/the-door.py`
|
||||
- `scripts/index_generator.py`
|
||||
- `build/semantic_linker.py`
|
||||
|
||||
3. ✅ Concrete repo-specific findings are already captured
|
||||
- the artifact records the live manuscript counts:
|
||||
- `18,884` chapter words
|
||||
- `19,227` concatenated output words
|
||||
- it records the known `compile_all.py --check` failure
|
||||
- it links the follow-up bug filed in the target repo:
|
||||
- `https://forge.alexanderwhitestone.com/Timmy_Foundation/the-testament/issues/51`
|
||||
|
||||
4. ✅ Missing regression coverage added in this PR
|
||||
- `tests/test_the_testament_genome.py` now locks the artifact path, sections, and grounded findings
|
||||
|
||||
## Evidence
|
||||
|
||||
Fresh verification against `Timmy_Foundation/the-testament` from a clean clone at `/tmp/the-testament-675`:
|
||||
|
||||
```bash
|
||||
python3 scripts/build-verify.py --json
|
||||
bash scripts/smoke.sh
|
||||
python3 compile_all.py --check
|
||||
```
|
||||
|
||||
Observed results:
|
||||
- `scripts/build-verify.py --json` passed and reported 18 chapters
|
||||
- `bash scripts/smoke.sh` passed
|
||||
- `python3 compile_all.py --check` failed with the known qrcode version bug already documented by the genome artifact
|
||||
|
||||
Host-repo regression added and verified:
|
||||
|
||||
```bash
|
||||
python3 -m pytest tests/test_the_testament_genome.py -q
|
||||
```
|
||||
|
||||
## Recommendation
|
||||
|
||||
Close issue #675 as already implemented on `main`.
|
||||
The truthful delta remaining in `timmy-home` was regression coverage and verification, not a second rewrite of `the-testament-GENOME.md`.
|
||||
63
tests/test_the_testament_genome.py
Normal file
63
tests/test_the_testament_genome.py
Normal file
@@ -0,0 +1,63 @@
|
||||
from pathlib import Path
|
||||
|
||||
|
||||
GENOME = Path("the-testament-GENOME.md")
|
||||
VERIFICATION = Path("docs/issue-675-verification.md")
|
||||
|
||||
|
||||
def read_genome() -> str:
|
||||
assert GENOME.exists(), "the-testament-GENOME.md must exist at repo root"
|
||||
return GENOME.read_text(encoding="utf-8")
|
||||
|
||||
|
||||
def test_the_testament_genome_exists_with_required_sections() -> None:
|
||||
text = read_genome()
|
||||
for heading in [
|
||||
"# GENOME.md — the-testament",
|
||||
"## Project Overview",
|
||||
"## Architecture",
|
||||
"## Entry Points",
|
||||
"## Data Flow",
|
||||
"## Key Abstractions",
|
||||
"## API Surface",
|
||||
"## Test Coverage Gaps",
|
||||
"## Security Considerations",
|
||||
]:
|
||||
assert heading in text
|
||||
|
||||
|
||||
def test_the_testament_genome_captures_grounded_runtime_findings() -> None:
|
||||
text = read_genome()
|
||||
for token in [
|
||||
"```mermaid",
|
||||
"scripts/build-verify.py --json",
|
||||
"bash scripts/smoke.sh",
|
||||
"python3 compile_all.py --check",
|
||||
"qrcode",
|
||||
"website/index.html",
|
||||
"game/the-door.py",
|
||||
"scripts/index_generator.py",
|
||||
"build/semantic_linker.py",
|
||||
"18,884",
|
||||
"19,227",
|
||||
".gitea/workflows/build.yml",
|
||||
".gitea/workflows/smoke.yml",
|
||||
".gitea/workflows/validate.yml",
|
||||
"the-testament/issues/51",
|
||||
]:
|
||||
assert token in text
|
||||
|
||||
|
||||
def test_issue_675_verification_doc_exists_and_references_artifact() -> None:
|
||||
assert VERIFICATION.exists(), "docs/issue-675-verification.md must exist"
|
||||
text = VERIFICATION.read_text(encoding="utf-8")
|
||||
for token in [
|
||||
"# Issue #675 Verification",
|
||||
"Status: ✅ ALREADY IMPLEMENTED",
|
||||
"the-testament-GENOME.md",
|
||||
"tests/test_the_testament_genome.py",
|
||||
"scripts/build-verify.py --json",
|
||||
"bash scripts/smoke.sh",
|
||||
"python3 compile_all.py --check",
|
||||
]:
|
||||
assert token in text
|
||||
Reference in New Issue
Block a user