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

block_writer שומר טקסט-שיחה של המודל לתוך תוכן הבלוק — נדרש שער-ניקוי בכתיבה #455

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

What & Why

write_and_store_block שומר את פלט-ה-LLM הגולמי ל-decision_blocks ללא כל ניקוי. כשהמודל עוטף את התוצר בפתיח ("אכתוב את הפרק מחדש לפי הוראת היו״ר...") או בדוח-סיכום נלווה (טבלת סעיפים + "שמרתי על הפרדה מלאה...") — הטקסט הזה נכנס לבלוק, זולג ל-drafts/decision.md דרך _update_draft_file, ומשם ל-DOCX שהיו״ר מקבלת.

נצפה בפועל (16.07.26, ערר 1043-02-26):

  1. block-zayin — הרצת write_block החזירה 7,872 מילים שכללו פתיח + דוח-סיכום בסוף. נוקה ידנית ונשמר מחדש דרך save_block_content (7,697 מילים נקיות).
  2. block-tet — זיהום קיים כבר בטיוטה שהיו״ר קראה: "מצב הנתונים בתיק 1043-02-26 כרגע אינו מאפשר בלוק ט 'אמיתי':", "להלן טיוטת-עבודה...", "הערה למנסח:", ו-14 placeholders מסוג [תוקף — יושלם] / נ.. ה-DOCX המיוצא מציג את שורת-המטא כ-Heading 3.
  3. block-chet — "הערה: הרחבתי מעט את סעיף 7.4...".

ההסתברות לזיהום עולה ככל שההנחיות ארוכות/דיאלוגיות יותר.

הפתרון המוצע — שער-ניקוי בנקודת-הכתיבה (quality-at-source, G4), לא תיקון בקריאה (G1):

  • פונקציית _strip_model_chatter(content, block_id) ב-block_writer, שתופעל ב-write_and_store_block לפני store_block: חיתוך כל מה שלפני כותרת-הבלוק הראשונה (^#{1,3}\s), וחיתוך זנב-מטא (מפריד --- ואחריו טקסט בגוף ראשון/טבלת-סיכום).
  • אין בליעה שקטה: אם נחתך טקסט — לרשום WARNING עם מספר התווים שנחתכו ולהחזיר שדה chatter_stripped בתוצאה.
  • לחזק את הפרומפט: "החזר אך ורק את גוף הבלוק, בלי פתיח, בלי דוח-סיכום ובלי פנייה למזמין".
  • שער QA: legal-qa ייכשל על בלוק שמכיל סמני-שיחה (גוף ראשון "אכתוב/שמרתי/הרחבתי", "הערה למנסח", "טיוטת-עבודה").

הערה נלווית: save_block_content מסמן generation_type='claude-code'/model_used='claude-code' — כך שניקוי ידני של פלט-Opus מאבד את הפרובננס האמיתי (X8). כדאי לאפשר שימור-פרובננס בשמירה מנוקה.

Acceptance Criteria

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

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

## What & Why write_and_store_block שומר את פלט-ה-LLM הגולמי ל-decision_blocks ללא כל ניקוי. כשהמודל עוטף את התוצר בפתיח ("אכתוב את הפרק מחדש לפי הוראת היו״ר...") או בדוח-סיכום נלווה (טבלת סעיפים + "שמרתי על הפרדה מלאה...") — הטקסט הזה נכנס לבלוק, זולג ל-drafts/decision.md דרך _update_draft_file, ומשם ל-DOCX שהיו״ר מקבלת. נצפה בפועל (16.07.26, ערר 1043-02-26): 1. block-zayin — הרצת write_block החזירה 7,872 מילים שכללו פתיח + דוח-סיכום בסוף. נוקה ידנית ונשמר מחדש דרך save_block_content (7,697 מילים נקיות). 2. block-tet — זיהום קיים כבר בטיוטה שהיו״ר קראה: "**מצב הנתונים בתיק 1043-02-26 כרגע אינו מאפשר בלוק ט 'אמיתי':**", "להלן **טיוטת-עבודה**...", "הערה למנסח:", ו-14 placeholders מסוג [תוקף — יושלם] / **נ.**. ה-DOCX המיוצא מציג את שורת-המטא כ-Heading 3. 3. block-chet — "הערה: הרחבתי מעט את סעיף 7.4...". ההסתברות לזיהום עולה ככל שההנחיות ארוכות/דיאלוגיות יותר. הפתרון המוצע — שער-ניקוי בנקודת-הכתיבה (quality-at-source, G4), לא תיקון בקריאה (G1): - פונקציית _strip_model_chatter(content, block_id) ב-block_writer, שתופעל ב-write_and_store_block לפני store_block: חיתוך כל מה שלפני כותרת-הבלוק הראשונה (^#{1,3}\s), וחיתוך זנב-מטא (מפריד --- ואחריו טקסט בגוף ראשון/טבלת-סיכום). - אין בליעה שקטה: אם נחתך טקסט — לרשום WARNING עם מספר התווים שנחתכו ולהחזיר שדה chatter_stripped בתוצאה. - לחזק את הפרומפט: "החזר אך ורק את גוף הבלוק, בלי פתיח, בלי דוח-סיכום ובלי פנייה למזמין". - שער QA: legal-qa ייכשל על בלוק שמכיל סמני-שיחה (גוף ראשון "אכתוב/שמרתי/הרחבתי", "הערה למנסח", "טיוטת-עבודה"). הערה נלווית: save_block_content מסמן generation_type='claude-code'/model_used='claude-code' — כך שניקוי ידני של פלט-Opus מאבד את הפרובננס האמיתי (X8). כדאי לאפשר שימור-פרובננס בשמירה מנוקה. ## Acceptance Criteria > ⚠️ **טרם הוגדרו.** המשימה הועברה מ-TaskMaster ללא `testStrategy`. > לפני לקיחתה — להגדיר AC ולסמן `status:ready` (כלל התבנית: issue ללא AC אינו pickable). --- <sub>הועבר מ-TaskMaster (tag `legal-ai`, id **229**, status היה `pending`) ב-2026-08-05. הפניות `(#229)` בהודעות-commit ישנות מתייחסות למזהה ה-TaskMaster, לא למספר ה-issue הזה.</sub>
chaim added the priority:p1-higharea:writingtype:bug labels 2026-08-05 11:01:19 +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:writing 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#455
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?