feat: add t3 project move command with --dry-run flag#2467
feat: add t3 project move command with --dry-run flag#2467shoaib050326 wants to merge 1 commit intopingdotgg:mainfrom
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
ApprovabilityVerdict: Needs human review This PR introduces a new CLI command ( You can customize Macroscope's approvability policy. Learn more. |
55b0f5a to
1d722c2
Compare
1d722c2 to
36025a7
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 36025a7. Configure here.
- Adds t3 project move <project> <path> to physically move workspace dirs - Adds --dry-run flag that previews without making changes - Validates source exists, target parent exists+writable, target not taken - Reuses existing project.meta.update orchestration command for metadata - Adds 4 tests: success, dry-run no-op, missing source, existing target Closes pingdotgg#2358
36025a7 to
381a39e
Compare

Summary
t3 project move <project> <path>command that physically moves a project workspace directory and updates the workspace root in the orchestration system--dry-runflag that previews the move operation without making any changesChanges
apps/server/src/cli.ts: NewprojectMoveCommandfollowing existing patterns (add/remove/rename)apps/server/src/cli.test.ts: 4 new tests covering success, dry-run no-op, missing source, and existing targetTesting
Risk/Rollback
projectMoveCommandand revertprojectCommandsubcommands arrayCloses #2358
Note
Medium Risk
Adds a new command that renames directories on disk and updates persisted project metadata; mistakes could move/delete user workspaces or leave state inconsistent, though validations and rollback reduce this risk.
Overview
Adds a new
t3 project move <project> <path>subcommand that physically renames the project workspace directory and then dispatchesproject.meta.updateto persist the newworkspaceRoot(with rollback if the metadata update fails).Includes
--dry-runoutput-only mode and preflight checks for missing source, non-existent/non-writable target parent, existing target path, and collisions with another active project’sworkspaceRoot. CLI tests are extended to cover the happy path, dry-run no-op, and key rejection/error cases.Reviewed by Cursor Bugbot for commit 381a39e. Bugbot is set up for automated code reviews on this repo. Configure here.
Note
Add
project movesubcommand to thet3CLI with--dry-runsupportt3 project move <project> <path>command in cli.ts that renames a project's workspace directory and updates itsworkspaceRootmetadata via aproject.meta.updatedispatch.--dry-runflag prints a preview of the source and target paths without making any filesystem or metadata changes.Macroscope summarized 381a39e.