FU-1: איחוד מסלול-הקליטה למסלול קנוני אחד (GAP-01/02/04/05) #11

chaim · 2026-05-30T19:30:50Z

chaim commented

2026-05-30 19:30:50 +00:00

Summary

מאחד את שני מסלולי ה-ingest המקבילים (ingest_precedent / ingest_internal_decision) למסלול קנוני יחיד services/ingest.py.

GAP-01 — פייפליין קנוני יחיד ingest_document(spec, ...) (Template Method); וריאציה לפי-סוג ב-IntakeSpec. שתי הפונקציות הציבוריות נשארות wrappers דקים — חתימות נשמרו, אפס שינוי לקוראים.
GAP-02 — המסלול הפנימי מתזמן עכשיו גם metadata extraction; צעד-חסר נעשה בלתי-אפשרי מבנית.
GAP-04 — ולידציית enums אחידה לשני הסוגים.
GAP-05 — staging/derivation/citation-guard/multimodal/fallback מאוחדים.

מספק INV-ING1/ING3/G2/G4.

גבולות

גבול FU-2 נקי: ה-DB-create נשאר נפרד (מנותב דרך spec.create_record). אין מיגרציה (0/56 internal חסרי-metadata).
אילוץ claude_session נשמר (אין import של מחלצי-LLM).
ההכרעה האדריכלית אומתה מול ≥3 מקורות. ספק: docs/superpowers/specs/2026-05-30-fu1-unified-ingest-design.md
42 החלטות-ועדה חסרות multimodal → TaskMaster #61.2 (FU-3).

Security

path-traversal שסומן ב-review: לא רגרסיה (verbatim מהישן), לא נצפה (file_path הוא server-controlled), תיקון ALLOWED_ROOT היה שובר workflow — מחוץ להיקף.

Test Plan

9 בדיקות offline חדשות (כולל regression GAP-02)
כל החבילה ירוקה — 77 passed
smoke-import, חתימות ציבוריות נשמרו
סקירת-קוד סופית עברה; 2 הערות תוקנו

🤖 Generated with Claude Code

## Summary מאחד את שני מסלולי ה-ingest המקבילים (`ingest_precedent` / `ingest_internal_decision`) למסלול קנוני יחיד `services/ingest.py`. - **GAP-01** — פייפליין קנוני יחיד `ingest_document(spec, ...)` (Template Method); וריאציה לפי-סוג ב-`IntakeSpec`. שתי הפונקציות הציבוריות נשארות wrappers דקים — חתימות נשמרו, אפס שינוי לקוראים. - **GAP-02** — המסלול הפנימי מתזמן עכשיו גם metadata extraction; צעד-חסר נעשה בלתי-אפשרי מבנית. - **GAP-04** — ולידציית enums אחידה לשני הסוגים. - **GAP-05** — staging/derivation/citation-guard/multimodal/fallback מאוחדים. מספק INV-ING1/ING3/G2/G4. ## גבולות - גבול FU-2 נקי: ה-DB-create נשאר נפרד (מנותב דרך `spec.create_record`). אין מיגרציה (0/56 internal חסרי-metadata). - אילוץ claude_session נשמר (אין import של מחלצי-LLM). - ההכרעה האדריכלית אומתה מול ≥3 מקורות. ספק: docs/superpowers/specs/2026-05-30-fu1-unified-ingest-design.md - 42 החלטות-ועדה חסרות multimodal → TaskMaster #61.2 (FU-3). ## Security path-traversal שסומן ב-review: לא רגרסיה (verbatim מהישן), לא נצפה (file_path הוא server-controlled), תיקון ALLOWED_ROOT היה שובר workflow — מחוץ להיקף. ## Test Plan - [x] 9 בדיקות offline חדשות (כולל regression GAP-02) - [x] כל החבילה ירוקה — 77 passed - [x] smoke-import, חתימות ציבוריות נשמרו - [x] סקירת-קוד סופית עברה; 2 הערות תוקנו 🤖 Generated with [Claude Code](https://claude.com/claude-code)

chaim added 9 commits 2026-05-30 19:30:50 +00:00

docs(spec): FU-1 unified-ingest design + FU-3 backfill task (#61.2) 357a5238c4

Design for unifying the two parallel ingest paths (ingest_precedent /
ingest_internal_decision) into one canonical pipeline parameterized by an
IntakeSpec config object — Template Method skeleton + Strategy injection.
Closes the GAP-02 root cause (missing metadata queue on internal path) by
making a skipped step structurally impossible.

Architecture choice verified against 3+ authoritative sources (refactoring.guru
Template-Method/Replace-Conditional, Fowler FlagArgument, Strategy pattern).
DB check (2026-05-30): no migration needed — 0/56 internal rows lack metadata,
0 invalid enums; multimodal backfill (42 rows) tracked as TaskMaster #61.2 / FU-3.

Covers GAP-01/02/04/05 · provides INV-ING1/ING3/G2/G4 · TaskMaster #59.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

docs(plan): FU-1 unified-ingest implementation plan (6 tasks, TDD) 15f42bc91c

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

test(ingest): failing tests for unified pipeline (FU-1) 9ae2d47d03

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(ingest): IntakeSpec + shared helpers for canonical pipeline (FU-1) d4663eba8f

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

feat(ingest): canonical ingest_document pipeline (FU-1) be4f7bbe99

refactor(ingest): ingest_precedent delegates to canonical pipeline (FU-1) d7eb1b2824

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

refactor(ingest): ingest_internal_decision delegates to canonical pipeline; queue metadata too (GAP-02, FU-1) 5104db8f4e

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

refactor(ingest): drop obsolete queue_halachot flag + dead imports (FU-1 review) 3c431403f6

pipeline always queues both extraction kinds (INV-ING3); remove the
now-meaningless queue_halachot param from ingest_internal_decision and
migrate_from_style_corpus. Also trim chunker/extractor/rerank from the
precedent_library module-top import (chunking/extraction moved to ingest.py).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

docs(spec): FU-1 documented drift notes + mark TaskMaster #59 done 90728ccb3e

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chaim merged commit 44ae739031 into main

2026-05-30 19:37:34 +00:00

chaim deleted branch fix/fu1-unified-ingest

2026-05-30 19:37:34 +00:00

chaim referenced this issue from a commit

2026-05-30 19:37:36 +00:00

Merge pull request 'FU-1: איחוד מסלול-הקליטה למסלול קנוני אחד (GAP-01/02/04/05)' (#11) from fix/fu1-unified-ingest into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: ezer-mishpati/legal-ai#11