Feat: Create page components around data/state layer for Sandbox details/inspect#79
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
from
3018bb7 to
d20d57c
Compare
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
from
d20d57c to
42134d8
Compare
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
3 times, most recently
from
0edfd68 to
e95e447
Compare
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
from
e95e447 to
e16ef9f
Compare
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
from
e16ef9f to
6fb546f
Compare
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
from
8268ed7 to
6fb546f
Compare
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
2 times, most recently
from
8c8e3a5 to
0e710ed
Compare
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
from
0e710ed to
34caf3e
Compare
ben-fornefeld
force-pushed
the
create-page-layout-e2b-2464
branch
from
bcd4dca to
ea991c2
Compare
…the sandbox is no longer running
…o display alias when available
…larity; enhance TableRow component with prefetch option and import order adjustments
…tress test script generation and execution with detailed logging for better error handling and metrics reporting
…ving redundant toast notifications; enhance SandboxInspectFilesystem and NotFound components with improved layout and visual elements
…LayoutEffect with useEffect; utilize useMemo for start and end date calculations to improve performance and readability
…ameters and timeouts for improved testing efficiency; ensure secure connections in sandbox manager and auth actions
…lculation and improving readability; replace direct sandboxInfo access with memoized value
…anager by removing unused error handling logic and improving code clarity
…reamline error handling in pipeError method; improve StoppedBanner component with last updated display and code organization
…d on 'show' state instead of a hardcoded true value
…lesystem watcher parameters in SandboxManager for improved performance and reliability
…e layout of NotFound component for better user experience
… errors and updating error messages for better clarity
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR creates page components around the data/state layer for Sandbox details/inspect functionality, utilizing Next.js SSR and streaming to preload data and connect the frontend to the new state/API layer.
- Adds comprehensive sandbox inspect functionality with filesystem exploration and real-time updates
- Implements server-side data preloading and client-side state management for sandbox details
- Creates responsive UI components for file inspection, resource monitoring, and sandbox management
Reviewed Changes
Copilot reviewed 79 out of 82 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| src/features/dashboard/sandbox/ | New sandbox detail pages with inspect functionality and context providers |
| src/ui/primitives/ | Enhanced UI components with animations, styling updates, and new props |
| src/server/sandboxes/ | Server actions for sandbox data fetching and state management |
| src/types/filesystem.ts | Type definitions for filesystem operations and events |
| src/configs/ | New configuration constants for versioning, URLs, and intervals |
| src/app/dashboard/[teamIdOrSlug]/sandboxes/[sandboxId]/ | New page routes for sandbox details and inspect views |
Comments suppressed due to low confidence (1)
src/features/dashboard/sandbox/inspect/filesystem.tsx:42
- There's an extra space in the closing div tag. It should be
</div>instead of</ div>.
</ div>
…r enhanced configuration options
…ving conditional classNames for improved readability
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Member
Author
|
@jakubno i addressed copilot review |
…dev/dashboard into create-page-layout-e2b-2464
… call conditions for better stability
jakubno
reviewed
Jul 31, 2025
…t for improved data fetching and state management
…dir' in sandbox components for consistency and clarity
jakubno
approved these changes
Jul 31, 2025
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.
This pr connects the new state/api layer changes from #78 to the user interface. It utilizes next.js ssr and streaming features to:
layout.tsx/inspect/page.tsxand hydrate client/inspect/page.tsxclient initializes inspect state & establishes connection to the sandbox viae2bsdk, as implemented in Feat: Application layer for sandbox details/inspect view #78UI States
Sandbox running, default inspect view
Sandbox running, user tries to set an invalid root path
Sandbox running, but watcher could not be created (no live updates)
Sandbox stopped, local state remaining inspect view
Sandbox stopped, user tries to change root path of local state