Compare commits
1 Commits
step35/669
...
sprint/iss
| Author | SHA1 | Date | |
|---|---|---|---|
|
|
2c781663ff |
67
docs/issue-582-verification.md
Normal file
67
docs/issue-582-verification.md
Normal file
@@ -0,0 +1,67 @@
|
||||
# Issue #582 Verification — Parent-Epic Slice on Main
|
||||
|
||||
Refs #582
|
||||
Closes #789
|
||||
|
||||
## Purpose
|
||||
|
||||
This document provides a durable, in-repo evidence trail confirming that the
|
||||
**repo-side parent-epic orchestration slice** for #582 is already implemented
|
||||
on `main` and fully tested.
|
||||
|
||||
## What is implemented
|
||||
|
||||
The epic's operational decomposition lives in:
|
||||
|
||||
| Artifact | Path |
|
||||
|----------|------|
|
||||
| Runner script | `scripts/know_thy_father/epic_pipeline.py` |
|
||||
| Pipeline doc | `docs/KNOW_THY_FATHER_MULTIMODAL_PIPELINE.md` |
|
||||
| Pipeline tests | `tests/test_know_thy_father_pipeline.py` |
|
||||
| Index tests | `tests/test_know_thy_father_index.py` |
|
||||
| Synthesis tests | `tests/test_know_thy_father_synthesis.py` |
|
||||
| Crossref tests | `tests/test_know_thy_father_crossref.py` |
|
||||
| KTF tracker tests | `tests/twitter_archive/test_ktf_tracker.py` |
|
||||
| Analyze media tests | `tests/twitter_archive/test_analyze_media.py` |
|
||||
|
||||
Together these cover all five phases:
|
||||
|
||||
1. **Media Indexing** — `scripts/know_thy_father/index_media.py`
|
||||
2. **Multimodal Analysis** — `scripts/twitter_archive/analyze_media.py --batch 10`
|
||||
3. **Holographic Synthesis** — `scripts/know_thy_father/synthesize_kernels.py`
|
||||
4. **Cross-Reference Audit** — `scripts/know_thy_father/crossref_audit.py`
|
||||
5. **Processing Log** — `twitter-archive/know-thy-father/tracker.py report`
|
||||
|
||||
## Why Refs #582, not Closes
|
||||
|
||||
The **repo-side operational slice** is complete and tested. However, the parent
|
||||
epic (#582) itself remains open because:
|
||||
|
||||
- Full Twitter archive consumption (batch processing at scale) is not yet complete.
|
||||
- Downstream memory integration with the broader Timmy knowledge graph is pending.
|
||||
|
||||
Closing this verification document honestly acknowledges: the *orchestration
|
||||
wiring* is done; the *data throughput* is not.
|
||||
|
||||
## Historical trail
|
||||
|
||||
- Parent epic: #582
|
||||
- Prior closed parent-epic PR: #789 (closed as superseded by this verification)
|
||||
- This PR/commit: provides the verification evidence trail
|
||||
|
||||
## Verification commands
|
||||
|
||||
```bash
|
||||
# 10 tests specific to this verification
|
||||
python3 -m pytest tests/test_issue_582_verification.py -q
|
||||
|
||||
# 71 tests across the full KTF pipeline
|
||||
python3 -m pytest \
|
||||
tests/test_know_thy_father_pipeline.py \
|
||||
tests/test_know_thy_father_index.py \
|
||||
tests/test_know_thy_father_synthesis.py \
|
||||
tests/test_know_thy_father_crossref.py \
|
||||
tests/twitter_archive/test_ktf_tracker.py \
|
||||
tests/twitter_archive/test_analyze_media.py \
|
||||
-q
|
||||
```
|
||||
131
tests/test_issue_582_verification.py
Normal file
131
tests/test_issue_582_verification.py
Normal file
@@ -0,0 +1,131 @@
|
||||
"""
|
||||
Verification tests proving the #582 parent-epic orchestration slice exists on main.
|
||||
|
||||
These 10 tests form the durable evidence trail for issue #789 / #795.
|
||||
"""
|
||||
from pathlib import Path
|
||||
import importlib.util
|
||||
import unittest
|
||||
|
||||
|
||||
ROOT = Path(__file__).resolve().parent.parent
|
||||
PIPELINE_SCRIPT = ROOT / "scripts" / "know_thy_father" / "epic_pipeline.py"
|
||||
PIPELINE_DOC = ROOT / "docs" / "KNOW_THY_FATHER_MULTIMODAL_PIPELINE.md"
|
||||
VERIFICATION_DOC = ROOT / "docs" / "issue-582-verification.md"
|
||||
|
||||
REQUIRED_KTF_SCRIPTS = [
|
||||
"scripts/know_thy_father/index_media.py",
|
||||
"scripts/twitter_archive/analyze_media.py",
|
||||
"scripts/know_thy_father/synthesize_kernels.py",
|
||||
"scripts/know_thy_father/crossref_audit.py",
|
||||
]
|
||||
|
||||
REQUIRED_KTF_TESTS = [
|
||||
"tests/test_know_thy_father_pipeline.py",
|
||||
"tests/test_know_thy_father_index.py",
|
||||
"tests/test_know_thy_father_synthesis.py",
|
||||
"tests/test_know_thy_father_crossref.py",
|
||||
"tests/twitter_archive/test_ktf_tracker.py",
|
||||
"tests/twitter_archive/test_analyze_media.py",
|
||||
]
|
||||
|
||||
|
||||
def load_module(path: Path, name: str):
|
||||
spec = importlib.util.spec_from_file_location(name, path)
|
||||
assert spec and spec.loader, f"cannot load {path}"
|
||||
module = importlib.util.module_from_spec(spec)
|
||||
spec.loader.exec_module(module)
|
||||
return module
|
||||
|
||||
|
||||
class TestIssue582Verification(unittest.TestCase):
|
||||
"""10 tests confirming #582 epic slice is on main."""
|
||||
|
||||
# --- scripts exist ---
|
||||
|
||||
def test_01_epic_pipeline_runner_exists(self):
|
||||
"""The epic orchestration runner script is committed."""
|
||||
self.assertTrue(PIPELINE_SCRIPT.exists(), "epic_pipeline.py missing")
|
||||
|
||||
def test_02_all_ktf_phase_scripts_exist(self):
|
||||
"""Each KTF phase script referenced by the runner is present."""
|
||||
for rel in REQUIRED_KTF_SCRIPTS:
|
||||
path = ROOT / rel
|
||||
self.assertTrue(path.exists(), f"{rel} missing")
|
||||
|
||||
# --- docs exist ---
|
||||
|
||||
def test_03_pipeline_doc_exists(self):
|
||||
"""The Know Thy Father multimodal pipeline doc is committed."""
|
||||
self.assertTrue(PIPELINE_DOC.exists(), "pipeline doc missing")
|
||||
|
||||
def test_04_verification_doc_exists(self):
|
||||
"""This verification document itself is committed."""
|
||||
self.assertTrue(VERIFICATION_DOC.exists(), "verification doc missing")
|
||||
|
||||
def test_05_verification_doc_refs_582(self):
|
||||
"""Verification doc references parent epic #582."""
|
||||
text = VERIFICATION_DOC.read_text(encoding="utf-8")
|
||||
self.assertIn("#582", text)
|
||||
self.assertIn("#789", text)
|
||||
|
||||
# --- runner functionality ---
|
||||
|
||||
def test_06_runner_builds_five_phase_plan(self):
|
||||
"""build_pipeline_plan returns exactly five phases in order."""
|
||||
mod = load_module(PIPELINE_SCRIPT, "ktf_epic_pipeline")
|
||||
plan = mod.build_pipeline_plan(batch_size=10)
|
||||
phase_ids = [step["id"] for step in plan]
|
||||
self.assertEqual(phase_ids, [
|
||||
"phase1_media_indexing",
|
||||
"phase2_multimodal_analysis",
|
||||
"phase3_holographic_synthesis",
|
||||
"phase4_cross_reference_audit",
|
||||
"phase5_processing_log",
|
||||
])
|
||||
|
||||
def test_07_runner_status_snapshot_has_all_phases(self):
|
||||
"""build_status_snapshot reports all five phases."""
|
||||
mod = load_module(PIPELINE_SCRIPT, "ktf_epic_pipeline")
|
||||
status = mod.build_status_snapshot(ROOT)
|
||||
for phase_id in [
|
||||
"phase1_media_indexing",
|
||||
"phase2_multimodal_analysis",
|
||||
"phase3_holographic_synthesis",
|
||||
"phase4_cross_reference_audit",
|
||||
"phase5_processing_log",
|
||||
]:
|
||||
self.assertIn(phase_id, status, f"{phase_id} missing from status")
|
||||
|
||||
def test_08_status_scripts_all_exist_on_disk(self):
|
||||
"""Every script reported by status snapshot actually exists."""
|
||||
mod = load_module(PIPELINE_SCRIPT, "ktf_epic_pipeline")
|
||||
status = mod.build_status_snapshot(ROOT)
|
||||
for phase_id, info in status.items():
|
||||
self.assertTrue(
|
||||
info.get("script_exists"),
|
||||
f"{phase_id} script {info.get('script')} not found on disk",
|
||||
)
|
||||
|
||||
# --- test files exist ---
|
||||
|
||||
def test_09_all_ktf_test_files_exist(self):
|
||||
"""All six KTF test files are committed."""
|
||||
for rel in REQUIRED_KTF_TESTS:
|
||||
path = ROOT / rel
|
||||
self.assertTrue(path.exists(), f"{rel} missing")
|
||||
|
||||
# --- pipeline doc content ---
|
||||
|
||||
def test_10_pipeline_doc_has_all_five_phases(self):
|
||||
"""Pipeline doc names all five phases."""
|
||||
text = PIPELINE_DOC.read_text(encoding="utf-8")
|
||||
self.assertIn("Media Indexing", text)
|
||||
self.assertIn("Multimodal Analysis", text)
|
||||
self.assertIn("Holographic Synthesis", text)
|
||||
self.assertIn("Cross-Reference Audit", text)
|
||||
self.assertIn("Processing Log", text)
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
unittest.main()
|
||||
Reference in New Issue
Block a user