GeezerVison

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

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview, Comment	May 15, 2026 6:02pm

PR Summary

Medium Risk
Medium risk: adds new workspace file folder APIs (move/bulk delete/download) and refactors multiple existing routes to call orchestration helpers, which could change behavior/error mapping across files, workflows, schedules, credentials, and MCP server management.

Overview
Adds workspace file folders + multi-select operations: new APIs to create/list/update/delete/restore file folders, move and bulk-archive file/folder selections, and download selected files/folders as a capped ZIP; upload/presign/register flows now accept folderId, and internal file-manage tooling supports path-based folder creation plus a move operation.

Refactors several existing endpoints (credentials, folders, workflows, schedules jobs, tables/knowledge/workflow restore, and mcp server/tool routes) to delegate logic to perform* orchestration functions with standardized error-code-to-HTTP mapping, updating/adding tests accordingly.

Improves UI/accessibility by adding ModalDescription to multiple modals, and enhances the Resource table component with shift/meta selection support, optional row drag/drop hooks, leading header actions, and percentage-based column sizing; adds new files UI components for a floating selection action bar and delete confirmation modal.

^{Reviewed by Cursor Bugbot for commit 62393f2. Configure here.}

Greptile Summary

This PR adds folder support and multiselect to the workspace files UI, introducing a new workspace_file_folders table, folder CRUD/move/restore API routes, a bulk-archive endpoint, a ZIP download endpoint, and updates the VFS to correctly group files under folder paths.

• Schema: Adds workspace_file_folders with a self-referential FK (SET NULL on delete) and replaces the flat workspace-file unique-name constraint with a folder-scoped partial unique index.
• API routes: New endpoints for folder CRUD (/folders, /folders/[folderId], /folders/[folderId]/restore), bulk archive (/bulk-archive), move (/move), and ZIP download (/download); advisory-lock transactions guard all mutation paths.
• VFS: materializeFiles now embeds folder paths into VFS paths and returns folderPath as a separate field so buildWorkspaceMd grouping works correctly; archived file folders are surfaced in recently-deleted.

Confidence Score: 4/5

The change is broadly well-constructed; one route handler maps a new error thrown by the folder-validation helper to the wrong HTTP status code.

The register route now passes folderId to registerUploadedWorkspaceFile, which calls assertWorkspaceFileFolderTarget. When the folder has been deleted between the presign and register steps, that helper throws 'Target folder not found' — but the catch block only maps 'not found in storage' to 404, so this new error surfaces as a 500. Every other new route in this PR handles the same error class cleanly.

apps/sim/app/api/workspaces/[id]/files/register/route.ts — the catch block needs to cover the new 'Target folder not found' error path introduced by the folder-aware registration flow.

Important Files Changed

_{Reviews (18): Last reviewed commit: "fix mcp server error code" | Re-trigger Greptile}

@greptile

bugbot run

@greptile

bugbot run

@greptile

bugbot run

bugbot run

@greptile

bugbot run

@greptile

bugbot run

bugbot run

@greptile

bugbot run

@greptile

bugbot run

@greptile

bugbot run

bugbot run

@greptile