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

fix(appraiser-facts): valid Paperclip priority enum (normal→medium) #37

Merged
chaim merged 1 commits from fix/appraiser-facts-priority-enum into main 2026-06-02 11:49:24 +00:00
Conversation 0 Commits 1 Files Changed 1 +4 -1
chaim commented 2026-06-02 11:49:18 +00:00
Owner
Copy Link

הבעיה

לחיצה על "חלץ עובדות שמאיות עכשיו" החזירה HTTP 500 (POST /api/cases/{n}/extract-appraiser-facts).

שורש

wake_analyst_for_appraiser_facts יוצר issue-בן ב-Paperclip עם priority: "normal", אבל ISSUE_PRIORITIES של Paperclip הוא critical|high|medium|low בלבד. ה-createChildIssueSchema (Zod) דוחה את הערך עם 400, ו-pc_request.raise_for_status() הופך אותו ל-500 שמוצג ליו"ר.

POST http://10.0.1.1:3100/api/issues/{main_issue_id}/children → 400 Bad Request

התיקון

"normal" → "medium" ב-web/paperclip_client.py:1086 (המופע היחיד בריפו). status:"in_progress" תקין ולא נגע.

🤖 Generated with Claude Code

## הבעיה לחיצה על "חלץ עובדות שמאיות עכשיו" החזירה HTTP 500 (`POST /api/cases/{n}/extract-appraiser-facts`). ## שורש `wake_analyst_for_appraiser_facts` יוצר issue-בן ב-Paperclip עם `priority: "normal"`, אבל `ISSUE_PRIORITIES` של Paperclip הוא `critical|high|medium|low` בלבד. ה-`createChildIssueSchema` (Zod) דוחה את הערך עם 400, ו-`pc_request.raise_for_status()` הופך אותו ל-500 שמוצג ליו"ר. ``` POST http://10.0.1.1:3100/api/issues/{main_issue_id}/children → 400 Bad Request ``` ## התיקון `"normal"` → `"medium"` ב-[web/paperclip_client.py:1086](web/paperclip_client.py#L1086) (המופע היחיד בריפו). `status:"in_progress"` תקין ולא נגע. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
chaim added 1 commit 2026-06-02 11:49:19 +00:00
fix(appraiser-facts): valid Paperclip priority enum (normal→medium) 5d63a903ce 
The 'חלץ עובדות שמאיות עכשיו' button returned HTTP 500. Root cause:
wake_analyst_for_appraiser_facts POSTs a child issue to Paperclip with
priority='normal', but Paperclip's ISSUE_PRIORITIES enum is only
critical|high|medium|low. createChildIssueSchema (Zod) rejects 'normal'
with 400 Bad Request; pc_request raise_for_status() turns it into a 500
surfaced to the chair. Fixed to 'medium' (the sole non-normal occurrence
in the repo).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
chaim merged commit 77ed0361b7 into main 2026-06-02 11:49:24 +00:00
chaim deleted branch fix/appraiser-facts-priority-enum 2026-06-02 11:49:24 +00:00
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#37
Delete Branch "fix/appraiser-facts-priority-enum"

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?