Added unit test to detect orphaned migration folders

[rob-ghost]

ref https://linear.app/ghost/issue/BER-3318/

knex-migrator silently skips migration folders where the folder version exceeds ghostVersion.safe (the major.minor extracted from package.json). If a developer creates a migration folder for the next minor version without bumping package.json to an rc, those migrations will never run.

This adds a unit test that scans all version folders under migrations/versions/ and fails when any folder exceeds the current safe version. It uses the same regex as @tryghost/version and the same semver.gt/semver.coerce comparison that knex-migrator uses internally. The error message tells the developer exactly what to do: run slimer migration (which handles the bump automatically) or manually bump package.json to the next minor rc.

This is part 3 of 3 for the rc version bump initiative (BER-3315). Part 2 (BER-3317, super-slimer auto-bump) ensures slimer migration bumps package.json to an rc automatically when creating a migration. This test is the safety net that catches cases where someone bypasses slimer and creates a migration folder manually.

[coderabbitai]

Walkthrough

A new unit test file is added to validate migration version consistency. The test loads the current package.json version, derives a safe version using regex, and enumerates migration version folders that match a numeric dot-numeric pattern. It uses semver.coerce to compare each folder version against the safe version and asserts that no migration folders have versions greater than the current package.json version. An error message provides guidance on resolving any version mismatches by running a migration or updating package.json.

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title directly and accurately describes the main change: adding a unit test to detect orphaned migration folders, which matches the changeset's core purpose.
Description check	✅ Passed	The description is well-detailed and clearly explains the purpose, implementation, and rationale for the test, including its role in the broader RC version bump initiative.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch feat/ber-3318-orphaned-migration-test

---

No actionable comments were generated in the recent review. 🎉

🧹 Recent nitpick comments

ghost/core/test/unit/server/data/migrations/version-consistency.test.js (1)

9-9: Nit: the first capture group in the regex is optional, which could produce a dotless safeVersion.

If pkg.version were ever a single number (e.g. "5"), the match would return "5" without a dot. semver.coerce("5") still yields 5.0.0 so the comparison would still work, but every X.Y folder would appear orphaned. This is extremely unlikely for Ghost's package.json, just noting for completeness — no action needed.

---

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.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}