Skip to content

Refactor: clutter less#135

Merged
Kuriel23 merged 5 commits into
developfrom
refactor/clutter-less
Jul 12, 2026
Merged

Refactor: clutter less#135
Kuriel23 merged 5 commits into
developfrom
refactor/clutter-less

Conversation

@Kuriel23

Copy link
Copy Markdown
Contributor

Description

Acknowledgements

Screenshots

Proof showing the creation/modification is working as expected

@coderabbitai

coderabbitai Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Warning

Review limit reached

@Kuriel23, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 18 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 8ce9f2de-4557-48f8-82ba-e5e9ebbaf4b6

📥 Commits

Reviewing files that changed from the base of the PR and between 84b1c22 and 861ed58.

📒 Files selected for processing (6)
  • src/routes/anime/$slug.tsx
  • src/routes/book/$slug.tsx
  • src/routes/game/$slug.tsx
  • src/routes/manga/$slug.tsx
  • src/routes/movie/$slug.tsx
  • src/routes/tv/$slug.tsx
📝 Walkthrough

Walkthrough

The PR adds authenticated custom-list management to anime, book, game, manga, movie, and TV detail pages, including list membership queries, toggles, and create-and-add flows. Detail-page tabs and layouts are reorganized, reviews move into standalone sections, and media displays gain updated carousel and image-zoom behavior. Shared styling updates refine layout positioning, emoji rendering, font loading, and transitions.

Estimated code review effort: 4 (Complex) | ~60 minutes

Sequence Diagram(s)

sequenceDiagram
  participant User
  participant DetailsPage
  participant ReactQuery
  participant ListsAPI
  User->>DetailsPage: open Lists control
  DetailsPage->>ReactQuery: load lists and membership
  ReactQuery->>ListsAPI: request list data
  ListsAPI-->>ReactQuery: return lists and memberships
  ReactQuery-->>DetailsPage: render checkbox states
  User->>DetailsPage: toggle or create list
  DetailsPage->>ListsAPI: update membership
  ListsAPI-->>ReactQuery: return mutation result
  ReactQuery-->>DetailsPage: invalidate and refresh queries
Loading

Possibly related PRs

  • TrackGeek/web#53: Introduces the ReviewItem fields used by the reorganized review rendering.
  • TrackGeek/web#117: Overlaps with the game details relations and dialog restructuring.
  • TrackGeek/web#121: Shares the DetailsPageLayout component changed in this PR.
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is too vague and does not describe the main changes in the PR. Use a specific title that mentions the core changes, such as custom lists and details-page layout refactoring.
✅ Passed checks (2 passed)
Check name Status Explanation
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/clutter-less

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

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 7

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
src/routes/game/$slug.tsx (1)

672-697: 🎯 Functional Correctness | 🟠 Major | ⚡ Quick win

Render the fetched game reviews here The current branch still shows a hardcoded ReviewItem when reviews.total >= 1; map reviews.items like the TV route so real game reviews appear.


ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 9e857fd4-a47b-4d45-b5c8-c72723cc1c7f

📥 Commits

Reviewing files that changed from the base of the PR and between eaf1a0c and 84b1c22.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (10)
  • index.html
  • src/components/pages/details/details-page-layout.tsx
  • src/components/pages/feed/activity-item.tsx
  • src/components/ui/image-zoom.tsx
  • src/routes/anime/$slug.tsx
  • src/routes/book/$slug.tsx
  • src/routes/game/$slug.tsx
  • src/routes/manga/$slug.tsx
  • src/routes/movie/$slug.tsx
  • src/routes/tv/$slug.tsx
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

  • TrackGeek/api (manual)

Comment thread src/routes/anime/$slug.tsx
Comment thread src/routes/game/$slug.tsx
Comment thread src/routes/game/$slug.tsx
Comment thread src/routes/manga/$slug.tsx Outdated
Comment thread src/routes/movie/$slug.tsx
Comment thread src/routes/tv/$slug.tsx
Comment thread src/routes/tv/$slug.tsx
@Kuriel23 Kuriel23 merged commit df3c577 into develop Jul 12, 2026
5 checks passed
@Kuriel23 Kuriel23 deleted the refactor/clutter-less branch July 12, 2026 16:30
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