feat(devtools): structured rebuild error panel (amicode #1022) - #340
Merged
Conversation
Replace the single-span truncated-stderr error display in Developer
Tools with a structured error panel:
Controller (developer-tools-controller.ts):
- New RebuildErrorInfo type: { message, fix[], detail? }
- rebuildError signal changed from string to RebuildErrorInfo
- Accepts both structured (new) and flat-string (legacy) errors from
the bridge for backward compatibility
- Timeout error now carries fix steps
Component (developer-tools.tsx):
- RebuildStatusIndicator split into RebuildStatusBadge (inline dot +
label in heading, shows only 'Failed') and RebuildErrorPanel (block
below rebuild buttons)
- RebuildErrorPanel renders: error message (red, bold), numbered fix
steps (ordered list), collapsible detail (Show/Hide details toggle
with monospace pre block)
CSS (amicode.css):
- .devtools-error-panel: tinted danger background with border
- .devtools-error-message: flexbox with status dot
- .devtools-error-fix-steps: numbered list with secondary text
- .devtools-error-detail-toggle: minimal button
- .devtools-error-detail: monospace scrollable pre block
All styles use existing theme variables (--text-danger-base,
--surface-danger-strong, --text-secondary-base) for light/dark compat.
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 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 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of harmoniqs/amicode#1016 / harmoniqs/amicode#1022.
Replaces the single-span truncated-stderr error display in Developer Tools with a structured error panel that shows classified errors with fix steps.
Changes
Controller (
developer-tools-controller.ts)RebuildErrorInfotype:{ message, fix[], detail? }rebuildErrorsignal changed fromstringtoRebuildErrorInfoComponent (
developer-tools.tsx)RebuildStatusIndicatorsplit into:RebuildStatusBadge— inline dot + label in the heading (shows only "Failed")RebuildErrorPanel— block below rebuild buttons with:<ol>)<pre>)CSS (
amicode.css).devtools-error-panel: tinted danger background.devtools-error-fix-steps: numbered list.devtools-error-detail: monospace scrollable pre blockCompanion PR
The extension-side error catalog and bridge changes are in harmoniqs/amicode#1033.