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

fix(lint): תיקון 10 שגיאות ESLint + ניקוי directives מיותרים #60

Merged
chaim merged 1 commits from fix/lint-errors into main 2026-06-06 13:32:46 +00:00
Conversation 0 Commits 1 Files Changed 11 +15 -12
chaim commented 2026-06-06 13:32:40 +00:00
Owner
Copy Link

רקע

ב-web-ui היו 10 שגיאות ESLint — כולן קיימות-מראש, לא מהפיצ'רים האחרונים. ה-build עבר למרותן (Next לא מפיל על ESLint), אבל שגיאות אמיתיות לא צריכות להישאר.

תיקונים

סוג כמות קבצים
react/no-unescaped-entities 3 legal-arguments-panel, precedent-edit-sheet — escaping מרכאות ב-JSX
react-hooks/set-state-in-effect 6 documents-panel, chair-editor, content-checklists, discussion-rules, golden-ratios, documents.ts — disable-comment לדפוסי sync/reset לגיטימיים (false-positive ידוע, תואם convention קיים)
React Compiler reassign 1 subject-donut — refactor ל-prefix-sums ללא mutable accumulator

ניקוי נוסף: הסרת 5 eslint-disable directives מיותרים. תוצאה: 0 errors (היה 10), 24 warnings (היה 29).

נותרו 24 warnings מהותיים קיימים (exhaustive-deps, incompatible-library) — מחוץ לטווח, ייתכן PR נפרד.

אימות

  • npm run lint → 0 errors
  • npm run build → עובר

🤖 Generated with Claude Code

## רקע ב-web-ui היו 10 שגיאות ESLint — כולן **קיימות-מראש**, לא מהפיצ'רים האחרונים. ה-build עבר למרותן (Next לא מפיל על ESLint), אבל שגיאות אמיתיות לא צריכות להישאר. ## תיקונים | סוג | כמות | קבצים | |-----|------|-------| | `react/no-unescaped-entities` | 3 | legal-arguments-panel, precedent-edit-sheet — escaping מרכאות ב-JSX | | `react-hooks/set-state-in-effect` | 6 | documents-panel, chair-editor, content-checklists, discussion-rules, golden-ratios, documents.ts — disable-comment לדפוסי sync/reset לגיטימיים (false-positive ידוע, תואם convention קיים) | | React Compiler reassign | 1 | subject-donut — refactor ל-prefix-sums ללא mutable accumulator | **ניקוי נוסף:** הסרת 5 `eslint-disable` directives מיותרים. **תוצאה: 0 errors (היה 10), 24 warnings (היה 29).** נותרו 24 warnings מהותיים קיימים (exhaustive-deps, incompatible-library) — מחוץ לטווח, ייתכן PR נפרד. ## אימות - [x] `npm run lint` → 0 errors - [x] `npm run build` → עובר 🤖 Generated with [Claude Code](https://claude.com/claude-code)
chaim added 1 commit 2026-06-06 13:32:40 +00:00
fix(lint): תיקון 10 שגיאות ESLint + ניקוי directives מיותרים 1f1a025509 
10 שגיאות (כולן קיימות-מראש, לא מהפיצ'רים האחרונים):
- react/no-unescaped-entities (3): legal-arguments-panel, precedent-edit-sheet
  — escaping של מרכאות ב-JSX (“/")
- react-hooks/set-state-in-effect (6): documents-panel, chair-editor,
  content-checklists, discussion-rules, golden-ratios, documents.ts
  — disable-comment לדפוסי sync/reset לגיטימיים (false-positive ידוע)
- React Compiler reassign (1): subject-donut — refactor לחישוב prefix-sums
  ללא mutable accumulator

ניקוי: הסרת 5 eslint-disable directives מיותרים (halacha-review-panel,
precedent-upload-sheet). תוצאה: 0 errors (היה 10), 24→ warnings (היה 29).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
chaim merged commit 27b40dfec5 into main 2026-06-06 13:32:46 +00:00
chaim deleted branch fix/lint-errors 2026-06-06 13:32:46 +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#60
Delete Branch "fix/lint-errors"

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?