fix: drop gold-set card from chair approval center (data/ not in image) #20

Merged
chaim merged 1 commits from fix/chair-pending-drop-goldset-card into main 2026-05-31 15:41:40 +00:00
Owner

כרטיס ה-gold-set ב-/api/chair/pending קרא data/eval/gold-set.jsonl, אבל .dockerignore מוציא את data/ מ-build context — הקובץ אף פעם לא בקונטיינר והכרטיס פשוט לא הופיע (4 במקום 5). בנדול eval-data ל-image = שכבה שגויה. ה-gold-set הוא משימה חד-פעמית, לא תור-אישורים חוזר — נסקר ישירות מול היו"ר (#63), לא על הלוח החי. הלוח מחזיר כעת את 4 השערים החזקים מבוססי-DB (הלכות, פסיקה חסרה, הערות, qa_failed). מסיר גם את ה-best-effort read + import מיותר.

🤖 Generated with Claude Code

כרטיס ה-gold-set ב-`/api/chair/pending` קרא `data/eval/gold-set.jsonl`, אבל `.dockerignore` מוציא את `data/` מ-build context — הקובץ אף פעם לא בקונטיינר והכרטיס פשוט לא הופיע (4 במקום 5). בנדול eval-data ל-image = שכבה שגויה. ה-gold-set הוא משימה חד-פעמית, לא תור-אישורים חוזר — נסקר ישירות מול היו"ר (#63), לא על הלוח החי. הלוח מחזיר כעת את 4 השערים החזקים מבוססי-DB (הלכות, פסיקה חסרה, הערות, qa_failed). מסיר גם את ה-best-effort read + import מיותר. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
chaim added 1 commit 2026-05-31 15:41:31 +00:00
The gold-set card read data/eval/gold-set.jsonl, but .dockerignore excludes
data/ from the build context, so the file is never in the container and the
card silently never rendered. Baking eval data into the image is the wrong
layering (data/ is runtime volumes). The gold-set review is a one-time task,
not a recurring chair queue, so it doesn't belong on the live board — it's
tracked via task #63 and reviewed directly with the chair. The board now
returns the 4 robust DB-backed gates (halachot, missing precedents, feedback,
qa_failed). Removes the best-effort file read + its unused Path import.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
chaim merged commit 83d6b5ecf0 into main 2026-05-31 15:41:40 +00:00
chaim deleted branch fix/chair-pending-drop-goldset-card 2026-05-31 15:41:40 +00:00
Sign in to join this conversation.
No Reviewers
No Label
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: ezer-mishpati/legal-ai#20