From ab1e72f0cc94792e10fa9515a9e1afceab73ea66 Mon Sep 17 00:00:00 2001 From: Chaim Date: Fri, 12 Jun 2026 10:31:56 +0000 Subject: [PATCH] =?UTF-8?q?fix(pipeline):=20final=5Fhalacha=5Fpipeline=20?= =?UTF-8?q?=D7=9E=D7=A2=D7=91=D7=99=D7=A8=20no=5Fcapture=20=D7=9C-hpa.main?= =?UTF-8?q?=20(#133)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 --- scripts/final_halacha_pipeline.py | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/scripts/final_halacha_pipeline.py b/scripts/final_halacha_pipeline.py index 7c3918e..722d047 100644 --- a/scripts/final_halacha_pipeline.py +++ b/scripts/final_halacha_pipeline.py @@ -131,7 +131,11 @@ async def main(args: argparse.Namespace) -> int: print(f"[3/4] halacha_panel_approve {'--apply' if apply else '(dry-run)'} " f"(Opus+DeepSeek+Gemini)…", flush=True) import halacha_panel_approve as hpa - rc = await hpa.main(Namespace(limit=args.limit, concurrency=6, apply=apply)) + # no_capture=False → persist panel rounds (FU-1, #133). Without this key + # hpa.main() raises AttributeError at its capture step (the pipeline builds + # the Namespace by hand, so every hpa arg must be set explicitly). + rc = await hpa.main(Namespace( + limit=args.limit, concurrency=6, apply=apply, no_capture=False)) return {"panel_rc": rc or 0} steps = [ -- 2.49.1