Skip to content

Slice 15 — Drag-and-drop import to folder (#16) - #37

Merged
cbaucom merged 3 commits into
mainfrom
feat/16-drag-drop-import
Jun 11, 2026
Merged

cbaucom merged 3 commits into
mainfrom
feat/16-drag-drop-import

Conversation

@cbaucom

@cbaucom cbaucom commented Jun 11, 2026

Copy link
Copy Markdown
Owner

Closes #16

Summary

Adds a pure importDroppedMediaFiles module in project-store that validates file types before writing, auto-suffixes duplicate filenames (photo-1.jpg), and returns imported slides plus skipped entries. The editor shell wires window drag-and-drop when a folder is open: imported media appends to the storyboard grid and unsupported types show a dismissible amber notice.

Acceptance criteria

  • Given dropped media files, when the copy completes, then the files exist in the folder and appear at the end of the grid.
  • Given a dropped file whose name already exists, when imported, then it is saved under a suffixed name and both remain playable.
  • Given an unsupported file type, when dropped, then it is skipped with a visible notice and no partial writes remain.

Test plan

  • pnpm test — import-media unit tests cover all three acceptance criteria
  • pnpm build && pnpm lint
  • Playwright smoke (node scripts/playwright-issue-16-smoke.mjs) — drop batch adds imported.jpg, suffixes duplicate to photo-1.jpg, shows notice for notes.txt (see issue-16-drag-drop-import.png)

Code review

Score: 5/5 — APPROVE

No blocking findings. Pure import logic lives in project-store with in-memory FS fakes; editor shell stays thin (DropImportLayer + useProject.importDroppedFiles).

Made with Cursor

cbaucom and others added 3 commits June 11, 2026 09:44
Resolve duplicate filenames with numeric suffixes and skip unsupported types before writing.

Co-authored-by: Cursor <cursoragent@cursor.com>
Append imported slides to the storyboard and show a dismissible notice for skipped file types.

Co-authored-by: Cursor <cursoragent@cursor.com>
Verifies appended slides, duplicate suffixing, and unsupported-file notice.

Co-authored-by: Cursor <cursoragent@cursor.com>
@cbaucom
cbaucom merged commit fed56da into main Jun 11, 2026
1 check passed
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.

Slice 15 — Drag-and-drop import to folder

1 participant