ezer-mishpati/legal-ai
1
0
Fork 0
Code Issues 29 Pull Requests 1 Actions Packages Projects Releases Wiki Activity

חילוץ-הלכות: אטומיות-מלאה ל-force re-extract (extract-then-swap / staging_run_id) #441

New Issue
Open
opened 2026-08-05 11:01:13 +00:00 by chaim · 0 comments
chaim commented 2026-08-05 11:01:13 +00:00
Owner
Copy Link

What & Why

follow-up מ-#144 (PR #267): force=True ב-halacha_extractor עדיין מוחק הלכות לא-מאושרות (reset_halacha_extraction) לפני שהריצה החדשה הצליחה — לא אטומי. #144 מיתן את האובדן-הקבוע (תיקון limit-notice ב-claude_session → chunk שנכשל נשאר un-checkpointed → resume משחזר), ו-PR #257 מיתן את ה-trigger (cooldown על 429-טרי), כך שאין יותר אובדן-קבוע — אך עדיין ייתכן מצב-ביניים זמני עם פחות הלכות עד ש-resume מסיים. אטומיות-מלאה (אפס-חלון): extract-to-staging-then-swap עם עמודת staging_run_id (מיגרציה אדיטיבית) + scope של dedup-on-insert ל-approved/published+staging במהלך force. נדחה מ-#144 כי דורש מיגרציית-סכמה + שינוי hot-path של store_halachot_for_chunk; עדיפות נמוכה כי האובדן-הקבוע כבר נמנע.

Acceptance Criteria

⚠️ טרם הוגדרו. המשימה הועברה מ-TaskMaster ללא testStrategy.
לפני לקיחתה — להגדיר AC ולסמן status:ready (כלל התבנית: issue ללא AC אינו pickable).

הועבר מ-TaskMaster (tag legal-ai, id 147, status היה pending) ב-2026-08-05. הפניות (#147) בהודעות-commit ישנות מתייחסות למזהה ה-TaskMaster, לא למספר ה-issue הזה.

## What & Why follow-up מ-#144 (PR #267): force=True ב-halacha_extractor עדיין מוחק הלכות לא-מאושרות (reset_halacha_extraction) *לפני* שהריצה החדשה הצליחה — לא אטומי. #144 מיתן את האובדן-הקבוע (תיקון limit-notice ב-claude_session → chunk שנכשל נשאר un-checkpointed → resume משחזר), ו-PR #257 מיתן את ה-trigger (cooldown על 429-טרי), כך שאין יותר אובדן-קבוע — אך עדיין ייתכן מצב-ביניים זמני עם פחות הלכות עד ש-resume מסיים. אטומיות-מלאה (אפס-חלון): extract-to-staging-then-swap עם עמודת staging_run_id (מיגרציה אדיטיבית) + scope של dedup-on-insert ל-approved/published+staging במהלך force. נדחה מ-#144 כי דורש מיגרציית-סכמה + שינוי hot-path של store_halachot_for_chunk; עדיפות נמוכה כי האובדן-הקבוע כבר נמנע. ## Acceptance Criteria > ⚠️ **טרם הוגדרו.** המשימה הועברה מ-TaskMaster ללא `testStrategy`. > לפני לקיחתה — להגדיר AC ולסמן `status:ready` (כלל התבנית: issue ללא AC אינו pickable). --- <sub>הועבר מ-TaskMaster (tag `legal-ai`, id **147**, status היה `pending`) ב-2026-08-05. הפניות `(#147)` בהודעות-commit ישנות מתייחסות למזהה ה-TaskMaster, לא למספר ה-issue הזה.</sub>
chaim added the type:bugarea:extractionpriority:p3-low labels 2026-08-05 11:01:13 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
area:agents
סוכני Paperclip, תזמור, שער-הפלטפורמה (X15)
 area:backend
FastAPI web/, endpoints
 area:ci
Gitea Actions, שערי-CI
 area:docs
ספ-המערכת, תיעוד, תכנון
 area:extraction
קליטה, OCR, חילוץ הלכות/מטא/טענות/תכניות
 area:frontend
web-ui/ (Next.js), RTL
 area:infra
Deploy, Coolify, MinIO, אחסון, גיבויים
 area:mcp
שרת ה-MCP וחוזה-הכלים (X9)
 area:retrieval
RAG, חיפוש, קורפוס, גרף-ציטוטים, יומונים
 area:security
אבטחה, סודות, PII, audit
 area:writing
12 הבלוקים, סגנון דפנה, QA, רכישת-סגנון
 priority:p0-blocker
Blocks everything
 priority:p1-high
High priority
 priority:p2-normal
Normal priority
 priority:p3-low
Low priority / nice to have
 size:l
a week+ — consider splitting
 size:m
2-3 days
 size:s
~1 day
 size:xs
< half a day
 status:blocked
Waiting on another issue/decision
 status:deferred
Decided to defer — parked backlog; not auto-picked
 status:good-first-issue
Small, self-contained starting point
 status:in-progress
Being worked on (a branch/worktree exists)
 status:in-review
A PR is open for this issue
 status:needs-decision
Requires a call before proceeding
 status:ready
Scoped + pickable (has acceptance criteria + size)
 type:bug
Something works incorrectly
 type:chore
Infra, deps, refactor, CI, docs
 type:feature
New capability
 type:spike
Time-boxed investigation -> a decision/doc
No Label area:extraction priority:p3-low type:bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
chaim
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ezer-mishpati/legal-ai#441
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?