Skip to content

fix: Workspace UI errors not shown#9145

Merged
royendo merged 8 commits intomainfrom
fix-ui-errors
Apr 2, 2026
Merged

fix: Workspace UI errors not shown#9145
royendo merged 8 commits intomainfrom
fix-ui-errors

Conversation

@royendo
Copy link
Copy Markdown
Contributor

@royendo royendo commented Mar 30, 2026

Lots of changes and looks like this functionality was lots

Screenshot 2026-03-30 at 11 43 15 Screenshot 2026-03-30 at 11 44 11

Checklist:

  • Covered by tests
  • Ran it and it works as intended
  • Reviewed the diff before requesting a review
  • Checked for unhandled edge cases
  • Linked the issues it closes
  • Checked if the docs need to be updated. If so, create a separate Linear DOCS issue
  • Intend to cherry-pick into the release branch
  • I'm proud of this work!

royendo and others added 7 commits March 30, 2026 11:23
Add \$Slots interface so `let:rootCauseReconcileError` in CanvasWorkspace
and ExploreWorkspace passes TypeScript validation.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add `interface \$\$Slots` to declare the `rootCauseReconcileError` slot
  prop type so svelte-check accepts `let:rootCauseReconcileError` in consumers
- Add `varsIgnorePattern: "^\$\$"` to ESLint no-unused-vars so Svelte's
  reserved `\$\$`-prefixed type declarations (Slots, Props, Events) are not
  flagged as unused

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Remove the slot prop pattern from WorkspaceError (which required an
unidiomatic \$\$Slots declaration). Canvas and Explore now compute
rootCauseReconcileError in their own script for use in their content
area; WorkspaceError still owns the single error bar. TanStack Query
deduplicates the ListResources request so there is no extra network cost.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@royendo royendo requested a review from AdityaHegde March 30, 2026 17:24
@royendo royendo requested a review from AdityaHegde March 31, 2026 14:43
@royendo royendo merged commit 0866827 into main Apr 2, 2026
13 checks passed
@royendo royendo deleted the fix-ui-errors branch April 2, 2026 12:51
@royendo royendo mentioned this pull request Apr 2, 2026
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants