Repository navigation
Conversation
- A Warnings tab lists the findings Solid reports on its diagnostics channel, such as a signal read outside a tracking scope. - Equal findings are one row with a count. The tab shows the total. - Rows can be filtered by severity, or searched by code, message and owner. - The detail shows the kind, owner path, node and when it was seen. It links to the owner in the ownership tree, the node in the reactivity graph, and the code's section of the repair guide. - The copy button puts a report of the finding on the clipboard for an agent. - The panel listens from the moment the toolbar mounts, so findings from the first render are kept. - Owner paths start at the app's first component, without the toolbar's own owners. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- The panel lists errors, warnings and info, so a tab named after one of its severities read oddly next to its own filters. - The tab, its empty state, the clear and copy buttons, and the report title say "issue" now. - The code moves from warnings/ to issues/, with the data attributes, the icon and the report renamed to match. - The severity filters still read Errors, Warnings and Info. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
lxsmnsyc
force-pushed
the
feat/warnings-panel
branch
from
October 27, 2026 08:49
f8b3815 to
f900c3c
Compare
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
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.
Adds an Issues tab that lists what the Solid runtime reports on
OBSERVE.diagnostics, such as a signal read outside a tracking scope. Today these findings only appear in the console. This PR is based on #27, since the panel uses its copy button.Changes
DEV.guideUrl.onSettled, so findings from the first render are kept even while the dock is closed.<DevToolbar> › <Errored> › … › <AppScope>) are dropped.Findings from server rendering (
SSR_*) are reported on the server, so they do not reach this panel.Testing
src/dev-toolbar/issues/tracker.test.tscover grouping, the limit, owner path trimming and the report.STRICT_READ_UNTRACKED, then checks the count, the trimmed owner path, the severity filter, the repair guide link, the jump to<DraftReader>in the ownership tree, and clearing.pnpm test,pnpm typecheck,pnpm format:check,pnpm build,pnpm checkand the full e2e suite pass locally.🤖 Generated with Claude Code