Skip to content

Delete migration asset directories during baseline cleanup#15

Merged
rwdaigle merged 2 commits intomainfrom
ryan/fix-baseline-flag
Feb 4, 2026
Merged

Delete migration asset directories during baseline cleanup#15
rwdaigle merged 2 commits intomainfrom
ryan/fix-baseline-flag

Conversation

@rwdaigle
Copy link
Contributor

@rwdaigle rwdaigle commented Feb 4, 2026

Summary

When baselining migrations, the tool now automatically deletes companion asset directories (named after the migration ID, e.g., 1fb2g-init/ for migration 1fb2g-init.sh). This prevents asset folders from accumulating when old migrations are cleaned up, and the output clearly indicates which migrations have associated directories being deleted.

Test Plan

  • Run cargo nextest run to verify all 49 tests pass
  • Test baseline with migrations that have asset directories
  • Test baseline with migrations without asset directories
  • Verify --dry-run output accurately reports what would be deleted
  • Verify --keep flag preserves both files and directories

🤖 Generated with Claude Code


Open with Devin

rwdaigle and others added 2 commits February 4, 2026 10:04
When baselining, also delete any companion directories named after the migration ID (e.g., 1fb2g-init/ for migration 1fb2g-init.sh). This prevents asset folders from accumulating when old migrations are cleaned up. The output clearly indicates which migrations have associated asset directories being deleted.

Co-Authored-By: Claude Haiku 4.5 <noreply@anthropic.com>
- Version bump: 0.4.1 → 0.5.0 (minor for new asset directory deletion feature)
- Fix cargo fmt issue in baseline.rs

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@rwdaigle rwdaigle merged commit 34b1d20 into main Feb 4, 2026
4 checks passed
@rwdaigle rwdaigle deleted the ryan/fix-baseline-flag branch February 4, 2026 15:07
Copy link

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 4 additional flags.

Open in Devin Review

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