fix(web,server): improve runtime error visibility in thread error banner#1202
fix(web,server): improve runtime error visibility in thread error banner#1202ecatuogno1 wants to merge 1 commit intopingdotgg:mainfrom
Conversation
Runtime errors (e.g. malformed local SKILL.md) were truncated twice: server-side to 180 chars, then UI-side to 3 lines via line-clamp-3. This made errors like invalid YAML parsing failures nearly unreadable. - Increase server-side runtime error truncation limit from 180 to 512 characters to preserve diagnostic detail - Expand UI error banner from line-clamp-3 to line-clamp-6 so longer error messages remain readable without dominating the viewport Closes pingdotgg#1084 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
📝 Coding Plan
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment Tip CodeRabbit can use TruffleHog to scan for secrets in your code with verification capabilities.Add a TruffleHog config file (e.g. trufflehog-config.yml, trufflehog.yml) to your project to customize detectors and scanning behavior. The tool runs only when a config file is present. |
|
Is there a chance we can get an example image attached? |
|
@binbandit Here’s the repro screenshot from #1084 showing the current truncated error banner state that this PR is fixing. This change keeps the same banner UI, but increases the visible clamp from 3 lines to 6 and preserves up to 512 chars server-side instead of 180, so the same error is much more readable. |
|
@binbandit Fresh after screenshot from this PR branch. This is the expanded thread error banner with the same kind of long malformed- |

Summary
line-clamp-3toline-clamp-6so longer error messages remain readable without dominating the viewportProblem
Runtime errors (e.g. malformed local
SKILL.mdwith invalid YAML frontmatter) are truncated twice: server-side to 180 chars, then UI-side to 3 lines vialine-clamp-3. This makes errors like invalid YAML parsing failures nearly unreadable — the user has to take a screenshot fast before the banner disappears, and even then the text is cut off.Test plan
.agent/skills/.../SKILL.md)bun run testinapps/server)Closes #1084
Note
Improve runtime error visibility in thread error banner
ThreadErrorBannerfrom 3 to 6 visible lines before clamping.512character limit totruncateDetailforruntime.erroractivity messages inProviderRuntimeIngestion.ts, replacing the previous default truncation length.Macroscope summarized 86b53ae.