feat(project-files): add cross-owner copy and bulk transfers by mzxchandra · Pull Request #9018 · simstudioai/sim · GitHub
Skip to content

feat(project-files): add cross-owner copy and bulk transfers - #9018

Merged
mzxchandra merged 5 commits into
stagingfrom
codex/project-files-bulk
Oct 15, 2026
Merged

feat(project-files): add cross-owner copy and bulk transfers#9018
mzxchandra merged 5 commits into
stagingfrom
codex/project-files-bulk

Conversation

@mzxchandra

@mzxchandra mzxchandra commented Oct 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Summary

Validation

  • 29 focused PostgreSQL content and authorization checks passed.
  • 41 focused app unit tests and 32 CLI tests passed; scoped TypeScript check passed.
  • 23 local app/relay HTTP and browser acceptance checks passed across delivery, rendered archives and cross-owner copy.
  • Regenerated API/CLI metadata after synchronization. Hosted CI must validate the new head.

Stack

#9015–#9017 are merged. This PR targets staging; #9019 and #9034 remain downstream and need synchronization after this PR lands.

@vercel

vercel Bot commented Oct 14, 2026 •
edited
Loading

Copy link
Copy Markdown

This was referenced Oct 14, 2026
Merged
Merged
Merged
Merged
Closed
@mzxchandra mzxchandra mentioned this pull request Oct 14, 2026
Merged
7 tasks done
@mzxchandra
mzxchandra force-pushed the codex/project-files-rendering branch 3 times, most recently from 428b57c to 2042762 Compare October 14, 2026 23:55
This was referenced Oct 15, 2026
Merged
Merged
@mzxchandra
mzxchandra force-pushed the codex/project-files-rendering branch 4 times, most recently from 46eef76 to 8fc8b2e Compare October 15, 2026 00:31
Base automatically changed from codex/project-files-rendering to staging October 15, 2026 00:59
@mzxchandra
mzxchandra force-pushed the codex/project-files-bulk branch from 6e3b83b to 3d4f3b0 Compare October 15, 2026 01:07
@mzxchandra

mzxchandra commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

@mzxchandra

mzxchandra commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@mzxchandra I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 15, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 68 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/lib/projects/files/api/download-presenter.ts
Comment thread apps/sim/lib/projects/files/application/extract.ts
Comment thread apps/sim/lib/projects/files/application/downloads.ts
Comment thread apps/sim/lib/workspace-files/application/copy-file-items.ts
Comment thread apps/sim/app/api/v2/projects/[projectId]/files/bulk-download/route.ts
Comment thread apps/sim/lib/workspace-files/download-selection.ts
Comment thread apps/sim/lib/api/contracts/v2/openapi/files-audit.ts Outdated
Comment thread apps/sim/lib/workspace-files/limits.ts
Comment thread apps/sim/lib/workspace-files/archive-extraction.ts
Comment thread apps/sim/lib/workspace-files/application/download-workspace-file-items.ts Outdated
@greptile-apps

greptile-apps Bot commented Oct 15, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

RetriggerView in GreptileConfidence Score: 5/5

[High impact] The reviewed changes appear safe to merge.

Summary

Adds copies between workspace and Project owners, bounded ZIP extraction, recursive downloads, and Markdown snapshot export.

  • Copied files get new identities in the destination owner.
  • Project file selections download as ZIP archives.
  • Markdown exports use the visible snapshot and its Project assets.
  • Project ZIP files unpack into a new sibling folder.

Diagram

 
%%{init: {'theme': 'neutral'}}%%
flowchart TD
  A[Copy request] --> B[Check source read and destination write access]
  B --> C[Capture selected files and folders]
  C --> D[Stage copied bytes]
  D --> E[Recheck access and source selection]
  E --> F[Create destination tree and update storage usage]
  F --> G[Return new identities and notify destination]
  E -->|Rejected| H[Discard staged bytes]
Loading

Reviews (4) · Last reviewed commit: "fix(files): recheck archive admission an..." · Reviewed by Greptile

greptile-apps[bot]
greptile-apps Bot reviewed Oct 15, 2026
View reviewed changes
Comment thread apps/sim/lib/workspace-files/copy.ts Outdated
Comment thread apps/sim/app/api/projects/[id]/files/download/route.ts
@mzxchandra

mzxchandra commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

@mzxchandra

mzxchandra commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@mzxchandra I have started the AI code review. It will take a few minutes to complete.

cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 15, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/lib/workspace-files/application/download-workspace-file-items.ts Outdated
Comment thread apps/sim/lib/workspace-files/copy.ts Outdated
@mzxchandra

mzxchandra commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

@greptile

@mzxchandra

mzxchandra commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

@cubic-dev-ai review this PR

@cubic-dev-ai

cubic-dev-ai Bot commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

@cubic-dev-ai review this PR

@mzxchandra I have started the AI code review. It will take a few minutes to complete.

@waleedlatif1 waleedlatif1 mentioned this pull request Oct 15, 2026
Merged
6 tasks done
cubic-dev-ai[bot]
cubic-dev-ai Bot reviewed Oct 15, 2026
View reviewed changes

@cubic-dev-ai cubic-dev-ai Bot left a comment •
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 69 files

Reply with feedback, questions, or to request a fix.

Turn on auto-fix | Re-trigger cubic

Comment thread apps/sim/lib/workspace-files/copy.ts
@mzxchandra
mzxchandra marked this pull request as ready for review October 15, 2026 01:47
@mzxchandra
mzxchandra merged commit e111f4f into staging Oct 15, 2026
54 checks passed
@mzxchandra
mzxchandra deleted the codex/project-files-bulk branch October 15, 2026 01:47
@mzxchandra

mzxchandra commented Oct 15, 2026

Copy link
Copy Markdown
Contributor Author

@waleedlatif1 waleedlatif1 mentioned this pull request Oct 15, 2026
Merged

This branch was previously deployed

1 inactive deployment
Preview — 63fee573 Deployed Oct 15, 2026 by vercel[bot]
View deployment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@greptile-apps greptile-apps[bot] greptile-apps[bot] left review comments
@cubic-dev-ai cubic-dev-ai[bot] cubic-dev-ai[bot] left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@mzxchandra