fix(operations): contain agent run-log text inside the "פלט" popup #327

chaim · 2026-06-20T18:32:12Z

chaim commented

2026-06-20 18:32:12 +00:00

מה

תיקון: טקסט פלט-הסוכן ב-popup "פלט" (דף /operations) בורח מגבולות החלון.

למה

ה-RunLogDialog השתמש ב-Radix ScrollArea. ה-Viewport עוטף את הילדים ב-div פנימי עם display:table שמתכווץ לרוחב ה-token הארוך ביותר שאי-אפשר לשבור — מזהי toolu_… ונתיבי /home/chaim/… ב-JSON הגולמי של הלוג. תא-הטבלה מתרחב במקום להגביל רוחב, כך ש-whitespace-pre-wrap break-words לא קיבל רוחב לשבור מולו והטקסט גלש מעבר לגבולות הדיאלוג.

איך

החלפת ה-ScrollArea ב-div גלילה רגיל מוגבל-רוחב (h-[60vh] w-full overflow-y-auto overflow-x-hidden).
break-words → break-all כדי שגם מזהים/נתיבים ארוכים-ללא-רווח יישברו לשורות.
הסרת import לא-בשימוש של ScrollArea.

תיקון ויזואלי טהור — נכנס תחת החריג של שער-העיצוב (אינו משנה את העיצוב המכוון, אלא מחזיר את ההתנהגות הנכונה: הטקסט בתוך החלון). tsc --noEmit + eslint נקיים.

Invariants

ללא נגיעה ב-invariant — שינוי הצגתי בלבד ב-web-ui (אין מסלול מקביל/G2, אין שער-פלטפורמה/G12, אין בליעת-שגיאות §6).

🤖 Generated with Claude Code

## מה תיקון: טקסט פלט-הסוכן ב-popup "פלט" (דף /operations) בורח מגבולות החלון. ## למה ה-`RunLogDialog` השתמש ב-Radix `ScrollArea`. ה-Viewport עוטף את הילדים ב-div פנימי עם `display:table` שמתכווץ לרוחב ה-token הארוך ביותר שאי-אפשר לשבור — מזהי `toolu_…` ונתיבי `/home/chaim/…` ב-JSON הגולמי של הלוג. תא-הטבלה מתרחב במקום להגביל רוחב, כך ש-`whitespace-pre-wrap break-words` לא קיבל רוחב לשבור מולו והטקסט גלש מעבר לגבולות הדיאלוג. ## איך - החלפת ה-`ScrollArea` ב-div גלילה רגיל מוגבל-רוחב (`h-[60vh] w-full overflow-y-auto overflow-x-hidden`). - `break-words` → `break-all` כדי שגם מזהים/נתיבים ארוכים-ללא-רווח יישברו לשורות. - הסרת import לא-בשימוש של `ScrollArea`. תיקון ויזואלי טהור — נכנס תחת החריג של שער-העיצוב (אינו משנה את העיצוב המכוון, אלא מחזיר את ההתנהגות הנכונה: הטקסט בתוך החלון). `tsc --noEmit` + `eslint` נקיים. ## Invariants ללא נגיעה ב-invariant — שינוי הצגתי בלבד ב-web-ui (אין מסלול מקביל/G2, אין שער-פלטפורמה/G12, אין בליעת-שגיאות §6). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

chaim added 1 commit 2026-06-20 18:32:12 +00:00

fix(operations): contain agent run-log text inside the "פלט" popup

G12 Leak-Guard / leak-guard (pull_request) Successful in 4s

Details

Lint — undefined names / undefined-names (pull_request) Successful in 11s

Details

38234d9b4f

The RunLogDialog rendered the live agent log inside a Radix ScrollArea.
Radix wraps Viewport children in an inner `display:table` div that
shrink-wraps to the widest unbreakable token — the long `toolu_…` IDs and
`/home/chaim/…` file paths in the raw JSON log. That table cell expands
instead of constraining width, so `whitespace-pre-wrap break-words` never
had a width to wrap against and the text spilled out past the dialog
borders.

Replace the ScrollArea with a plain width-constrained scrollable div and
switch break-words → break-all so the long unbreakable IDs/paths wrap too.
Pure presentational fix (no invariant surface).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

chaim merged commit c9970a5955 into main

2026-06-20 18:32:17 +00:00

chaim deleted branch worktree-fix-runlog-overflow

2026-06-20 18:32:17 +00:00

chaim referenced this issue from a commit

2026-06-20 18:32:17 +00:00

Merge pull request 'fix(operations): contain agent run-log text inside the "פלט" popup' (#327) from worktree-fix-runlog-overflow into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: ezer-mishpati/legal-ai#327