fix(pipeline): final_halacha_pipeline מעביר no_capture ל-hpa.main (#133)
All checks were successful
G12 Leak-Guard / leak-guard (pull_request) Successful in 5s
All checks were successful
G12 Leak-Guard / leak-guard (pull_request) Successful in 5s
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>
This commit is contained in:
@@ -131,7 +131,11 @@ async def main(args: argparse.Namespace) -> int:
|
|||||||
print(f"[3/4] halacha_panel_approve {'--apply' if apply else '(dry-run)'} "
|
print(f"[3/4] halacha_panel_approve {'--apply' if apply else '(dry-run)'} "
|
||||||
f"(Opus+DeepSeek+Gemini)…", flush=True)
|
f"(Opus+DeepSeek+Gemini)…", flush=True)
|
||||||
import halacha_panel_approve as hpa
|
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}
|
return {"panel_rc": rc or 0}
|
||||||
|
|
||||||
steps = [
|
steps = [
|
||||||
|
|||||||
Reference in New Issue
Block a user