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

Plugin status-sync מכוון ל-issue-ילד ומחזיר done→in_progress #446

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

סנכרון-הסטטוס של פלאגין legal-ai (marcusgroup.legal-ai) על תיק שטיוטתו מוכנה מפרסם '📋 טיוטה מוכנה' ומסמן in_progress על issue-ילד מקושר, ומחזיר sub-tasks שנסגרו (done) חזרה ל-in_progress. בנוסף, ה-issue הראשי של התיק עלול לא להיות מקושר-פלאגין כלל.

הקשר ופירוט

אובחן 2026-06-30 על תיק 8125-09-24 / issue CMPA-140 (5abf9850, ילד של הראשי 802efd7d).

תסמין: CMPA-140 חזר שוב ושוב ל-in_progress אחרי סגירה ל-done. activity_log: actor_type=plugin (53461b5a marcusgroup.legal-ai), action=issue.updated patch={status:in_progress} _previous={status:done}, ומיד issue.comment.created 'טיוטה מוכנה — בדוק ושלח לדפנה'.

שורש:

  1. הסנכרון בוחר issue מקושר-לתיק לפי plugin_state.state_key='legal-case-number' ומכוון ל-issue-ילד (sub-task שכבר נסגר), לא ל-issue הראשי/הביקורת.
  2. ה-issue הראשי 802efd7d (in_review אצל חיים — נתיב-הביקורת האמיתי) אינו מקושר-פלאגין לתיק (אין לו רשומת legal-case-number ב-plugin_state). לתיק 8125-09-24 מקושרים ~8 issues-ילדים ישנים, כמה מהם תקועים in_progress.
  3. כך כל סגירת done של ילד נדרסת ע"י הסנכרון הבא.

לא לולאת-recovery של סוכן (אין שיוך-סוכן; אין wakeup-recovery מאז 24.6) — לכן לא חוסם ולא מבזבז heartbeats. אפקט: רעש-לוח + sub-tasks שלא נשארים סגורים.

תיקון מוצע (לבחון):

  • הסנכרון יכוון ל-issue הראשי/פתוח של התיק (parent_id is null או issue-ביקורת), לא ל-sub-task שנסגר; ו/או ידלג על issues במצב done/cancelled.
  • לוודא שה-issue הראשי מקבל את קישור legal-case-number ב-plugin_state בעת יצירת התיק/קישורו.
  • אופציונלי: ניקוי הקישורים הישנים של 8125-09-24 ל-sub-tasks שנסגרו.

מגע-קוד: plugin-legal-ai (סנכרון-סטטוס / comment routing) + web/agent_platform_port.py (G12). spec-first: X15/X7. worktree+PR.

Acceptance Criteria

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

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

## What & Why סנכרון-הסטטוס של פלאגין legal-ai (marcusgroup.legal-ai) על תיק שטיוטתו מוכנה מפרסם '📋 טיוטה מוכנה' ומסמן in_progress על issue-ילד מקושר, ומחזיר sub-tasks שנסגרו (done) חזרה ל-in_progress. בנוסף, ה-issue הראשי של התיק עלול לא להיות מקושר-פלאגין כלל. ## הקשר ופירוט אובחן 2026-06-30 על תיק 8125-09-24 / issue CMPA-140 (5abf9850, ילד של הראשי 802efd7d). תסמין: CMPA-140 חזר שוב ושוב ל-in_progress אחרי סגירה ל-done. activity_log: actor_type=plugin (53461b5a marcusgroup.legal-ai), action=issue.updated patch={status:in_progress} _previous={status:done}, ומיד issue.comment.created 'טיוטה מוכנה — בדוק ושלח לדפנה'. שורש: 1. הסנכרון בוחר issue מקושר-לתיק לפי plugin_state.state_key='legal-case-number' ומכוון ל-issue-ילד (sub-task שכבר נסגר), לא ל-issue הראשי/הביקורת. 2. ה-issue הראשי 802efd7d (in_review אצל חיים — נתיב-הביקורת האמיתי) **אינו** מקושר-פלאגין לתיק (אין לו רשומת legal-case-number ב-plugin_state). לתיק 8125-09-24 מקושרים ~8 issues-ילדים ישנים, כמה מהם תקועים in_progress. 3. כך כל סגירת done של ילד נדרסת ע"י הסנכרון הבא. לא לולאת-recovery של סוכן (אין שיוך-סוכן; אין wakeup-recovery מאז 24.6) — לכן לא חוסם ולא מבזבז heartbeats. אפקט: רעש-לוח + sub-tasks שלא נשארים סגורים. תיקון מוצע (לבחון): - הסנכרון יכוון ל-issue הראשי/פתוח של התיק (parent_id is null או issue-ביקורת), לא ל-sub-task שנסגר; ו/או ידלג על issues במצב done/cancelled. - לוודא שה-issue הראשי מקבל את קישור legal-case-number ב-plugin_state בעת יצירת התיק/קישורו. - אופציונלי: ניקוי הקישורים הישנים של 8125-09-24 ל-sub-tasks שנסגרו. מגע-קוד: plugin-legal-ai (סנכרון-סטטוס / comment routing) + web/agent_platform_port.py (G12). spec-first: X15/X7. worktree+PR. ## Acceptance Criteria > ⚠️ **טרם הוגדרו.** המשימה הועברה מ-TaskMaster ללא `testStrategy`. > לפני לקיחתה — להגדיר AC ולסמן `status:ready` (כלל התבנית: issue ללא AC אינו pickable). --- <sub>הועבר מ-TaskMaster (tag `legal-ai`, id **209**, status היה `pending`) ב-2026-08-05. הפניות `(#209)` בהודעות-commit ישנות מתייחסות למזהה ה-TaskMaster, לא למספר ה-issue הזה.</sub>
chaim added the priority:p3-lowarea:agentstype:bug 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: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#446
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?