Fix missing migration is_starred -> is_highligted

[ildyria]

Summary by CodeRabbit

• Bug Fixes
  • Corrected album data values to ensure consistent sorting and highlighting behavior across the app. This improves reliability of album ordering and presentation after data updates.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: bd963f6a-a108-47c7-81bd-383882c8c898

📥 Commits

Reviewing files that changed from the base of the PR and between 55c2473 and 6968d1f.

📒 Files selected for processing (1)

• database/migrations/2026_03_15_000005_fix_is_starred_remaining_values.php

🚧 Files skipped from review as they are similar to previous changes (1)

• database/migrations/2026_03_15_000005_fix_is_starred_remaining_values.php

---

📝 Walkthrough

Walkthrough

Adds a new Laravel migration that updates base_albums.sorting_col by converting values from "is_starred" to "is_highlighted". Both up() and down() methods perform this same update rather than reversing it.

Changes

Cohort / File(s)	Summary
Database Migration database/migrations/2026_03_15_000005_fix_is_starred_remaining_values.php	New anonymous Migration class. up() updates sorting_col values from "is_starred" to "is_highlighted" in base_albums. down() contains the same update (does not revert the change), which may require review.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 I hopped through rows and fixed a name with cheer,
From "starred" to "highlighted" — now it's clear.
Two tiny methods, both doing the same,
A rabbit's patch to tidy up the game. ✨

🚥 Pre-merge checks | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

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

📝 Coding Plan

• Generate coding plan for human review comments

---

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.}

[coderabbitai]

Actionable comments posted: 1

🧹 Nitpick comments (1)

database/migrations/2026_03_15_000005_fix_is_starred_remaining_values.php (1)

21-28: The down() method is identical to up() and effectively does nothing after migration.

The down() method searches for is_starred values that will no longer exist after up() runs. While this is intentionally non-reversible (since is_starred is no longer a valid ColumnSortingType enum value per the relevant code snippets), the identical code is confusing without explanation.

Consider adding a clarifying comment:

📝 Suggested documentation improvement

 	public function down(): void
 	{
+		// Intentionally non-reversible: 'is_starred' is no longer a valid ColumnSortingType enum value.
+		// Rolling back would break album sorting functionality.
 		DB::table('base_albums')
 			->where('sorting_col', '=', 'is_starred')
 			->update([
 				'sorting_col' => 'is_highlighted',
 			]);
 	}

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: f47e96a7-2070-48a6-959f-158bf57cf984

📥 Commits

Reviewing files that changed from the base of the PR and between f3cd8ef and 55c2473.

📒 Files selected for processing (1)

• database/migrations/2026_03_15_000005_fix_is_starred_remaining_values.php