Skip to content

feat(artifacts): move files within workspace folders#113

Merged
juacker merged 2 commits into
mainfrom
feat/same-workspace-artifact-move
Jul 22, 2026
Merged

feat(artifacts): move files within workspace folders#113
juacker merged 2 commits into
mainfrom
feat/same-workspace-artifact-move

Conversation

@juacker

@juacker juacker commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • add a guarded workspace_move_path command for same-workspace artifact moves into folders
  • make artifact folder rows accept same-workspace move drops and expand after 2 seconds of drag hover
  • use atomic no-overwrite rename primitives, including Windows MoveFileExW(..., 0), so collision handling cannot replace existing files

Verification

  • cargo fmt --manifest-path src-tauri/Cargo.toml -- --check
  • npm run typecheck
  • npm run lint
  • cargo test --manifest-path src-tauri/Cargo.toml commands::workspace::tests -- --nocapture
  • git diff --check

Review

  • Codex bounded final review: production_quality
  • Minimax follow-up final review: production_quality
  • Claude final review after login fix/restart: production_quality

@juacker
juacker marked this pull request as ready for review July 22, 2026 16:10
@juacker
juacker merged commit 00daf96 into main Jul 22, 2026
2 checks 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.

1 participant