Fix: minified react error on inspect routing

[ben-fornefeld]

Note

Refactors the sandbox inspect route to a client-driven view, adds client directives across inspect/UI modules, switches to motion/react, and makes JsonPopover button customizable.

• Inspect:
  • Client View Extraction: Replaces page-level composition with SandboxInspectView that wraps SandboxInspectProvider, Filesystem, and Viewer.
  • Client Components: Adds 'use client' to dir.tsx, file.tsx, node.tsx, parent-dir-item.tsx, and the new view.tsx.
  • Motion: Switches imports to motion/react in frame.tsx and stopped-banner.tsx.
• Header/UI:
  • JsonPopover: Converted to client component and supports buttonProps to customize trigger; metadata.tsx updated to use it instead of a direct Button.
• Metrics:
  • Dependency Fix: Includes lastMetrics in total useMemo deps in resource-usage-client.tsx.
• Clients:
  • Marks Supabase browser client (src/lib/clients/supabase/client.ts) as a client module.

^{Written by Cursor Bugbot for commit b126ae2. This will update automatically on new commits. Configure here.}

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
web	Ready	Preview	Comment	Nov 21, 2025 1:56pm
web-juliett	Ready	Preview	Comment	Nov 21, 2025 1:56pm

[cursor]

Bug: Event handlers in buttonProps override critical popover handlers

The {...buttonProps} spread at line 52 occurs after hardcoded onClick and onDoubleClick handlers (lines 42-50). If buttonProps contains these event properties, they will silently override the essential handlers that call stopPropagation(), preventDefault(), and set isOpen(true). This breaks the popover's functionality. The handlers should be merged or buttonProps should be placed before the hardcoded handlers to allow them to take precedence.

 

[ben-fornefeld]

expected