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

FU-6 — שערי-QA נאכפים בקוד (GAP-15/16) #9

Closed
chaim wants to merge 0 commits from fix/fu6-qa-gates into main
pull from: fix/fu6-qa-gates
merge into: ezer-mishpati:main
Conversation 1 Commits - Files Changed -
chaim commented 2026-05-30 18:06:42 +00:00
Owner
Copy Link

FU-6 (תת-פרויקט 3, יחידה קריטית, ללא תלות)

מיישם את INV-QA3/INV-EX3 מהספ.

  • GAP-15: export_docx חוסם ייצוא כשיש כשל-QA קריטי — מגדר על-בסיס qa_results השמורות (SELECT זול, ללא הרצת-LLM חוזרת); חוסם גם אם QA מעולם לא רץ. ה-endpoint /api/cases/{n}/export-docx מחזיר 409 בחסימה (ה-UI מציג שגיאה במקום toast-הצלחה כוזב).
  • GAP-16: תיקון עקביות — fallback של בלוק-ו ריק היה severity=critical אך passed=True; עכשיו warning.

אימות: טסט offline tests/test_export_qa_gate.py (5/5, לא-ואקואי) + ביקורת-קוד עצמאית (עמודות qa_results אומתו, שער לפני side-effects, מסלול-UI מכוסה).

השפעה: ייצוא של תיק שלא עבר QA ייחסם (הקשחה מכוונת). הסוכן legal-exporter כבר מריץ validate_decision לפני ייצוא, אז הזרימה הרגילה לא נשברת.

פריסה: נוגע ב-web/app.py (קונטיינר → Coolify) + mcp-server (editable → סוכנים auto-load + restart chat-service).

🤖 Generated with Claude Code

## FU-6 (תת-פרויקט 3, יחידה קריטית, ללא תלות) מיישם את INV-QA3/INV-EX3 מהספ. - **GAP-15:** `export_docx` חוסם ייצוא כשיש כשל-QA קריטי — מגדר על-בסיס `qa_results` השמורות (SELECT זול, ללא הרצת-LLM חוזרת); חוסם גם אם QA מעולם לא רץ. ה-endpoint `/api/cases/{n}/export-docx` מחזיר **409** בחסימה (ה-UI מציג שגיאה במקום toast-הצלחה כוזב). - **GAP-16:** תיקון עקביות — fallback של בלוק-ו ריק היה severity=critical אך passed=True; עכשיו warning. **אימות:** טסט offline `tests/test_export_qa_gate.py` (5/5, לא-ואקואי) + ביקורת-קוד עצמאית (עמודות qa_results אומתו, שער לפני side-effects, מסלול-UI מכוסה). **השפעה:** ייצוא של תיק שלא עבר QA ייחסם (הקשחה מכוונת). הסוכן legal-exporter כבר מריץ validate_decision לפני ייצוא, אז הזרימה הרגילה לא נשברת. **פריסה:** נוגע ב-web/app.py (קונטיינר → Coolify) + mcp-server (editable → סוכנים auto-load + restart chat-service). 🤖 Generated with Claude Code
chaim added 2 commits 2026-05-30 18:06:42 +00:00
fix(qa): enforce critical-QA gate on export + fix neutral_background critical-but-passed (GAP-15/16, INV-QA3/EX3) 084b31cd9b 
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
fix(api): export endpoint returns 409 when QA gate blocks (FU-6 UX — avoid false success toast) a61495f5ef 
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
chaim commented 2026-05-30 19:41:42 +00:00
Author
Owner
Copy Link

נסגר: כל הקומיטים של ענף זה כבר מוכלים במלואם ב-main (מוזגו מוקדם יותר). אין diff למזג — סגירה לניקוי, התוכן כבר משולב.

נסגר: כל הקומיטים של ענף זה כבר מוכלים במלואם ב-main (מוזגו מוקדם יותר). אין diff למזג — סגירה לניקוי, התוכן כבר משולב.
chaim closed this pull request 2026-05-30 19:41:42 +00:00

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No Label
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#9
Delete Branch "fix/fu6-qa-gates"

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?