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

chore(case-ui): הסרת כפתור "פתח עורך החלטה" הכפול + regen api types #391

Merged
chaim merged 1 commits from worktree-remove-decision-editor-btn into main 2026-07-05 07:19:44 +00:00
Conversation 0 Commits 1 Files Changed 2 +73 -7
chaim commented 2026-07-05 07:19:35 +00:00
Owner
Copy Link

מה ולמה

הכפתור "פתח עורך החלטה" בבאנר-התיק ביצע רק setTab("decision") — כפילות מדויקת של טאב "ההחלטה" שכבר קיים ברצועת-הטאבים (עורך-ההחלטה אוחד לתוך הטאב; /compose נמחק). שריד שאיבד משמעות אחרי איחוד-המסכים. הוסר לבקשת היו"ר.

בנוסף: npm run api:types — רענון src/lib/api/types.ts מול הסכמה החיה. קולט את שדות DocumentPatchRequest (is_post_hearing/protocol_scope) שנוספו ב-#390 (כבר פרוס בייצור — אומת מול /openapi.json).

Invariants — הצהרה

  • נוגע / מקיים:
    • G2 (מקור-אמת יחיד) — הסרת נתיב-ניווט כפול לטאב-ההחלטה; רצועת-הטאבים נשארת המסלול היחיד. types.ts מיושר לסכמה החיה (מקור-האמת).
    • אין נגיעה ב-Backend / Paperclip (G12) / תוכן-משפטי (G11).

אימות

  • npx tsc --noEmit ✓ (גם אחרי regen של types.ts) · npx eslint על הדף ✓.
  • setTab/Button עדיין בשימוש במקומות אחרים — אין imports יתומים.
  • ⚠️ אין הרצת-E2E מקומית (Docker/Coolify בלבד); שינוי ויזואלי טהור (הסרת אלמנט כפול), אושר ישירות ע"י היו"ר.

🤖 Generated with Claude Code

## מה ולמה הכפתור **"פתח עורך החלטה"** בבאנר-התיק ביצע רק `setTab("decision")` — כפילות מדויקת של טאב **"ההחלטה"** שכבר קיים ברצועת-הטאבים (עורך-ההחלטה אוחד לתוך הטאב; `/compose` נמחק). שריד שאיבד משמעות אחרי איחוד-המסכים. הוסר לבקשת היו"ר. בנוסף: `npm run api:types` — רענון `src/lib/api/types.ts` מול הסכמה החיה. קולט את שדות `DocumentPatchRequest` (`is_post_hearing`/`protocol_scope`) שנוספו ב-#390 (כבר פרוס בייצור — אומת מול `/openapi.json`). ## Invariants — הצהרה - **נוגע / מקיים:** - **G2 (מקור-אמת יחיד)** — הסרת נתיב-ניווט כפול לטאב-ההחלטה; רצועת-הטאבים נשארת המסלול היחיד. `types.ts` מיושר לסכמה החיה (מקור-האמת). - אין נגיעה ב-Backend / Paperclip (G12) / תוכן-משפטי (G11). ## אימות - `npx tsc --noEmit` ✓ (גם אחרי regen של types.ts) · `npx eslint` על הדף ✓. - `setTab`/`Button` עדיין בשימוש במקומות אחרים — אין imports יתומים. - ⚠️ אין הרצת-E2E מקומית (Docker/Coolify בלבד); שינוי ויזואלי טהור (הסרת אלמנט כפול), אושר ישירות ע"י היו"ר. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
chaim added 1 commit 2026-07-05 07:19:35 +00:00
chore(case-ui): הסרת כפתור "פתח עורך החלטה" הכפול + regen api types
All checks were successful
G12 Leak-Guard / leak-guard (pull_request) Successful in 4s
Details
Lint — undefined names / undefined-names (pull_request) Successful in 10s
Details
93cce0505e 
הכפתור בבאנר-התיק רק ביצע setTab("decision") — כפילות מדויקת של טאב
"ההחלטה" שכבר קיים ברצועת-הטאבים (עורך-ההחלטה אוחד לתוכה, /compose נמחק).
שריד שאיבד משמעות אחרי איחוד-המסכים; הוסר.

בנוסף: regen של src/lib/api/types.ts מול הסכמה החיה — קולט את שדות
DocumentPatchRequest (is_post_hearing/protocol_scope) שנוספו ב-#390.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
chaim merged commit acb887845c into main 2026-07-05 07:19:44 +00:00
chaim deleted branch worktree-remove-decision-editor-btn 2026-07-05 07:19:44 +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#391
Delete Branch "worktree-remove-decision-editor-btn"

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?