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

באג עמידות: ריצת-heartbeat כפולה על אותו issue דורסת טיוטת-קובץ תוך כדי כתיבה #444

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

What & Why

בעת עבודה על CMPA-116 (ערר 8125-09-24, כתיבת בלוק י+יא), ריצת-heartbeat שנייה של אותו סוכן (run 6f48d781) עשתה checkout ל-issue בזמן שקריאת legal-writer subagent הארוכה (~573s) של ריצה ראשונה (397fd2e8) עוד רצה. הריצה השנייה כתבה טיוטת-דיון מתחרה (גרסה תֵמָטית, headers לא-קנוניים) ל-data/cases/8125-09-24/drafts/decision.md ב-10:18, ודרסה את גרסת ה-writer הקנונית (## י. דיון). התאוששתי דרך git (commit 81d948b), שחזרתי, validate 7/7, וייצאתי DOCX יציב. אך הריצה השנייה תפסה את ה-ownership lock ולכן הריצה הראשונה לא יכלה לעדכן comment/status (Issue run ownership conflict).

הקשר ופירוט

שורש: (1) drafts/decision.md מבוסס-קובץ ולא-נעול — שתי ריצות כותבות לאותו path. (2) ה-platform מאפשר checkout שני ל-issue שכבר בריצה ארוכה (אין mutual-exclusion ברמת issue בזמן ריצת subagent ארוכה). (3) validate_decision מפענח את הקובץ לפי headers קנוניים (## י., ## יא.) — גרסה לא-קנונית נכשלת ב-claims_coverage/structural_integrity. כיווני-תיקון לבדיקה: נעילת-issue אמיתית בזמן ריצה פעילה / זיהוי double-checkout ודחיית הריצה הישנה מוקדם; כתיבת בלוקים ל-DB (save_block_content) ולא רק לקובץ כדי שהקובץ לא יהיה ה-SoT היחיד; auto-commit שלא ידרוס שינויים לא-ממוזגים. ראה reference_paperclip_recovery_loops + feedback_shared_worktree_branch_race. ארטיפקטים: exports/החלטה-דיון-והכרעה-v1.docx (גרסה A תקפה), git 81d948b (A), git 42bae60 (B המתחרה).

Acceptance Criteria

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

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

## What & Why בעת עבודה על CMPA-116 (ערר 8125-09-24, כתיבת בלוק י+יא), ריצת-heartbeat שנייה של אותו סוכן (run 6f48d781) עשתה checkout ל-issue בזמן שקריאת legal-writer subagent הארוכה (~573s) של ריצה ראשונה (397fd2e8) עוד רצה. הריצה השנייה כתבה טיוטת-דיון מתחרה (גרסה תֵמָטית, headers לא-קנוניים) ל-data/cases/8125-09-24/drafts/decision.md ב-10:18, ודרסה את גרסת ה-writer הקנונית (## י. דיון). התאוששתי דרך git (commit 81d948b), שחזרתי, validate 7/7, וייצאתי DOCX יציב. אך הריצה השנייה תפסה את ה-ownership lock ולכן הריצה הראשונה לא יכלה לעדכן comment/status (Issue run ownership conflict). ## הקשר ופירוט שורש: (1) drafts/decision.md מבוסס-קובץ ולא-נעול — שתי ריצות כותבות לאותו path. (2) ה-platform מאפשר checkout שני ל-issue שכבר בריצה ארוכה (אין mutual-exclusion ברמת issue בזמן ריצת subagent ארוכה). (3) validate_decision מפענח את הקובץ לפי headers קנוניים (## י., ## יא.) — גרסה לא-קנונית נכשלת ב-claims_coverage/structural_integrity. כיווני-תיקון לבדיקה: נעילת-issue אמיתית בזמן ריצה פעילה / זיהוי double-checkout ודחיית הריצה הישנה מוקדם; כתיבת בלוקים ל-DB (save_block_content) ולא רק לקובץ כדי שהקובץ לא יהיה ה-SoT היחיד; auto-commit שלא ידרוס שינויים לא-ממוזגים. ראה reference_paperclip_recovery_loops + feedback_shared_worktree_branch_race. ארטיפקטים: exports/החלטה-דיון-והכרעה-v1.docx (גרסה A תקפה), git 81d948b (A), git 42bae60 (B המתחרה). ## Acceptance Criteria > ⚠️ **טרם הוגדרו.** המשימה הועברה מ-TaskMaster ללא `testStrategy`. > לפני לקיחתה — להגדיר AC ולסמן `status:ready` (כלל התבנית: issue ללא AC אינו pickable). --- <sub>הועבר מ-TaskMaster (tag `legal-ai`, id **155**, status היה `pending`) ב-2026-08-05. הפניות `(#155)` בהודעות-commit ישנות מתייחסות למזהה ה-TaskMaster, לא למספר ה-issue הזה.</sub>
chaim added the type:bugpriority:p1-higharea:agents labels 2026-08-05 11:01:15 +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:agents priority:p1-high 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#444
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?