fix(pipeline): final_halacha_pipeline מעביר no_capture ל-hpa.main (#133) #231

chaim · 2026-06-12T10:32:14Z

chaim commented

2026-06-12 10:32:14 +00:00

הבאג

FU-1 (#214) הוסיף ל-halacha_panel_approve.py את הדגל --no-capture ואת השימוש if not args.no_capture בשלב-הלכידה. אבל final_halacha_pipeline.py (כפתור "הרץ הלכות") קורא ל-hpa.main() עם Namespace שנבנה ביד (limit/concurrency/apply בלבד) — בלי no_capture. לכן הרצת-הצינור קרסה ב-AttributeError בדיוק בשלב שמירת-הסבבים, אחרי שה-apply כבר רץ → 0 סבבים נלכדו לתיק.

התיקון

שורה אחת: no_capture=False ב-Namespace שהצינור מרכיב. אומת מקצה-לקצה על 8174-12-24 → captured 49 panel rounds, errors=0.

Audit (לבקשת היו"ר)

סרקתי את כל הקוד ל-call-sites שבונים Namespace ביד לקריאת main() של סקריפט אחר — בדיוק מחלקת-הבאג הזו. רק 2 קיימים:

call-site	יעד	מצב
`final_halacha_pipeline.py:134`	`halacha_panel_approve`	תוקן כאן
`final_learning_pipeline.py:147`	`style_lesson_panel`	תקין — כל 5 ה-args ש-`main` קורא (case/pair_id/apply/limit/concurrency) מסופקים

אין באגים נוספים מהמחלקה הזו.

לקח

פרמטר חדש לפונקציה שנקראת גם מ-CLI וגם מ-call-site שבונה Namespace ביד → חייב לעדכן את שני הנתיבים. נתיב-ה-CLI נבדק ב-FU-1; נתיב-הצינור לא הורץ-מחדש אז ההגדרה-החסרה צצה רק בהרצה מלאה.

## הבאג FU-1 (#214) הוסיף ל-`halacha_panel_approve.py` את הדגל `--no-capture` ואת השימוש `if not args.no_capture` בשלב-הלכידה. אבל `final_halacha_pipeline.py` (כפתור "הרץ הלכות") קורא ל-`hpa.main()` עם `Namespace` שנבנה **ביד** (`limit`/`concurrency`/`apply` בלבד) — בלי `no_capture`. לכן הרצת-הצינור קרסה ב-`AttributeError` בדיוק בשלב שמירת-הסבבים, **אחרי** שה-apply כבר רץ → 0 סבבים נלכדו לתיק. ## התיקון שורה אחת: `no_capture=False` ב-Namespace שהצינור מרכיב. אומת מקצה-לקצה על **8174-12-24** → `captured 49 panel rounds, errors=0`. ## Audit (לבקשת היו"ר) סרקתי את כל הקוד ל-call-sites שבונים `Namespace` ביד לקריאת `main()` של סקריפט אחר — בדיוק מחלקת-הבאג הזו. **רק 2 קיימים:** | call-site | יעד | מצב | |---|---|---| | `final_halacha_pipeline.py:134` | `halacha_panel_approve` | **תוקן כאן** | | `final_learning_pipeline.py:147` | `style_lesson_panel` | **תקין** — כל 5 ה-args ש-`main` קורא (case/pair_id/apply/limit/concurrency) מסופקים | אין באגים נוספים מהמחלקה הזו. ## לקח פרמטר חדש לפונקציה שנקראת גם מ-CLI וגם מ-call-site שבונה Namespace ביד → חייב לעדכן את **שני** הנתיבים. נתיב-ה-CLI נבדק ב-FU-1; נתיב-הצינור לא הורץ-מחדש אז ההגדרה-החסרה צצה רק בהרצה מלאה.

chaim added 1 commit 2026-06-12 10:32:14 +00:00

fix(pipeline): final_halacha_pipeline מעביר no_capture ל-hpa.main (#133 )

G12 Leak-Guard / leak-guard (pull_request) Successful in 5s

Details

ab1e72f0cc

FU-1 (#214) הוסיף ל-halacha_panel_approve.py את הדגל --no-capture
ואת השימוש `if not args.no_capture` בשלב-הלכידה. אבל
final_halacha_pipeline.py קורא ל-hpa.main() עם Namespace שנבנה ביד
(limit/concurrency/apply בלבד) — בלי no_capture. לכן הרצת הצינור
("הרץ הלכות") קרסה ב-AttributeError בדיוק בשלב שמירת-הסבבים, אחרי
שה-apply כבר רץ → 0 סבבים נלכדו לתיק.

תוקן: הוספת `no_capture=False` ל-Namespace. אומת מקצה-לקצה על
8174-12-24 → "captured 49 panel rounds, errors=0".

audit: רק 2 מקומות בונים Namespace ביד לקריאת main() של סקריפט אחר —
זה (תוקן), ו-final_learning_pipeline.py→style_lesson_panel (נבדק, כל
ה-args מסופקים, תקין). אין באגים נוספים מהמחלקה הזו.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

chaim merged commit 51f9d9d309 into main

2026-06-12 10:32:20 +00:00

chaim deleted branch worktree-pipeline-nocapture-fix

2026-06-12 10:32:20 +00:00

chaim referenced this issue from a commit

2026-06-12 10:32:22 +00:00

Merge pull request 'fix(pipeline): final_halacha_pipeline מעביר no_capture ל-hpa.main (#133)' (#231) from worktree-pipeline-nocapture-fix into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: ezer-mishpati/legal-ai#231