chore: bump version, bun config, linting#150
Merged
Merged
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (2)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughThis PR modernizes Tailwind CSS utilities across the application and refactors the OG image generation route. Changes include converting leading-* to slash-based text sizing, replacing h-* w-* with size-* shorthand, simplifying positioning utilities, and switching from a custom OG helper to fumadocs-ui's DefaultImage component. Documentation examples are updated to reflect current API patterns. ChangesTailwind Modernization & OG Route Refactor
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ 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 |
There was a problem hiding this comment.
Pull request overview
This PR updates the repo’s Bun/Turbo-driven developer workflow by moving linting to the apps/www workspace, adding a Bun install policy (bunfig.toml), refreshing dependencies/lockfile, and applying widespread Tailwind v4-style class refactors across the docs site and registry UI.
Changes:
- Switch lint scripts to run within
apps/www, adjust ESLint configs (including re-enablingbetter-tailwindcssinapps/www). - Add
bunfig.tomlwithminimumReleaseAgepolicy and updatebun.lockwith dependency bumps. - Apply formatting/refactor changes across many UI components and docs (Tailwind utility updates, minor layout tweaks).
Reviewed changes
Copilot reviewed 41 out of 43 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Moves root linting to apps/www via Bun filters; trims root devDependencies; bumps Prettier. |
| eslint.config.mjs | Removes better-tailwindcss from the root ESLint base config. |
| bunfig.toml | Adds Bun install policy (minimumReleaseAge + excludes). |
| bun.lock | Updates lockfile to reflect dependency bumps and workspace devDependency relocation. |
| apps/www/scripts/registry-dev.mts | Updates usage docs to Bun-based invocation for the registry pipeline script. |
| apps/www/registry/default/ui/error-screen.tsx | Tailwind class refactor (text-xs/relaxed style). |
| apps/www/registry/default/blocks/linear-player/ui/toggle.tsx | Tailwind class wrapping refactor for state styles. |
| apps/www/registry/default/blocks/linear-player/components/timeline-slider-control.tsx | Tailwind utility refactor (size-2 usage). |
| apps/www/registry/default/blocks/linear-player/components/playlist.tsx | Adjusts thumbnail rounding class (rounded-sm). |
| apps/www/package.json | Adds lint scripts; switches registry scripts to Bun; bumps many dependencies/devDependencies. |
| apps/www/next-env.d.ts | Updates referenced Next-generated routes types path. |
| apps/www/eslint.config.mjs | Enables better-tailwindcss plugin in apps/www ESLint config. |
| apps/www/content/docs/components/media.mdx | Removes stray blank line in usage snippet. |
| apps/www/content/docs/components/fallback-poster.mdx | Removes stray blank line in usage snippet. |
| apps/www/content/docs/components/error-screen.mdx | Reflows Callout text for readability. |
| apps/www/content/docs/components/captions.mdx | Removes stray blank line in usage snippet. |
| apps/www/components/ui/tooltip.tsx | Tailwind calc formatting change for translate utility. |
| apps/www/components/ui/toggle-group.tsx | Splits long Tailwind lines for readability. |
| apps/www/components/ui/tabs.tsx | Tailwind line wrapping refactor for dark active state styles. |
| apps/www/components/ui/sidebar.tsx | Tailwind refactors + CSS var shorthand for offcanvas positioning and sizing. |
| apps/www/components/ui/select.tsx | Splits long dark-mode Tailwind line into multiple lines. |
| apps/www/components/ui/navigation-menu.tsx | Reorders Tailwind utilities (no behavior change intended). |
| apps/www/components/ui/kbd.tsx | Refactors selector utilities to in-data-* form. |
| apps/www/components/ui/item.tsx | Tailwind refactors including text-sm/normal and text-sm/snug. |
| apps/www/components/ui/field.tsx | Tailwind refactors including text-sm/normal and wrapping long responsive utilities. |
| apps/www/components/ui/dropdown-menu.tsx | Splits long Tailwind lines; refactors focus and destructive styles formatting. |
| apps/www/components/ui/breadcrumb.tsx | Changes wrapping utility for breadcrumb list styling. |
| apps/www/components/ui/badge.tsx | Tailwind indentation/formatting refactor in variant definitions. |
| apps/www/components/stream-panel/presets-overlay.tsx | Tailwind refactor to font-size/line-height shorthand (text-[10px]/4). |
| apps/www/components/preview-tab-component.tsx | Changes code pill rounding to rounded-sm. |
| apps/www/components/player-container.tsx | Tailwind refactors (text-xs/relaxed, size-3). |
| apps/www/components/nav-tabs.tsx | Tailwind refactor for icon sizing utility. |
| apps/www/components/hero.tsx | Tailwind refactor (text-sm/relaxed). |
| apps/www/components/hero-buttons.tsx | Tailwind refactor (size-4). |
| apps/www/components/header.tsx | Uses inset-x-0 instead of separate left/right. |
| apps/www/components/feature-trail-section.tsx | Consolidates sizing utilities to size-[...] and adjusts responsive sizing. |
| apps/www/components/codeblock.tsx | Changes max-height value for the code viewport container. |
| apps/www/components/cloud-search.tsx | Tailwind refactors and logical property changes (inset-e-*, sm:inset-e-2). |
| apps/www/components/blocks/sidebar.tsx | Tailwind refactor for z-index utility. |
| apps/www/components/block-viewer.tsx | Tailwind refactors (spacing, separators, background utilities, aspect ratio utility changes). |
| apps/www/components/ai/page-actions.tsx | Tailwind utility ordering changes for consistency. |
| apps/www/app/og/docs/[...slug]/route.tsx | Switches OG generator to fumadocs-ui/og default generator and updates route typing approach. |
| apps/www/app/(home)/page.tsx | Tailwind refactors using font-size/line-height shorthand (e.g., text-[28px]/7). |
Comment on lines
+17
to
+19
| plugins: { | ||
| "better-tailwindcss": pluginBetterTailwindcss, | ||
| }, |
Comment on lines
+11
to
+12
| "registry:build": "REGISTRY_HOST=https://limeplay.winoffrg.dev bun run ./scripts/registry-dev.mts", | ||
| "registry:dev": "REGISTRY_HOST=http://localhost:3000 bun --watch run ./scripts/registry-dev.mts --watch", |
Comment on lines
+14
to
+15
| * REGISTRY_HOST=http://localhost:3000 bun --watch run ./scripts/registry-dev.mts --watch | ||
| * REGISTRY_HOST=https://limeplay.winoffrg.dev bun run ./scripts/registry-dev.mts |
Comment on lines
17
to
20
| export async function GET( | ||
| _req: Request, | ||
| { params }: { params: Promise<{ slug: string[] }> } | ||
| { params }: RouteContext<"/og/docs/[...slug]"> | ||
| ) { |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/www/components/ui/dropdown-menu.tsx`:
- Line 309: The submenu chevron (ChevronRightIcon) lost RTL-aware flipping;
restore an RTL-aware class on the ChevronRightIcon element so it points left in
RTL layouts (e.g., replace the current className="ml-auto" with an RTL-aware
variant that keeps right-aligned layout in LTR and flips the icon in RTL, such
as adding rtl:mr-auto rtl:ml-0 and an RTL horizontal flip like rtl:scale-x-[-1]
on ChevronRightIcon).
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
Run ID: 3458b71a-54b8-4c1d-8adc-54c5f7340fec
⛔ Files ignored due to path filters (6)
apps/www/eslint.config.mjsis excluded by none and included by noneapps/www/next-env.d.tsis excluded by none and included by noneapps/www/package.jsonis excluded by none and included by nonebun.lockis excluded by!**/*.lockand included by noneeslint.config.mjsis excluded by none and included by nonepackage.jsonis excluded by none and included by none
📒 Files selected for processing (4)
apps/www/app/docs/docs.cssapps/www/components/blocks/sidebar.tsxapps/www/components/cloud-search.tsxapps/www/components/ui/dropdown-menu.tsx
| > | ||
| {children} | ||
| <ChevronRightIcon className="cn-rtl-flip ml-auto" /> | ||
| <ChevronRightIcon className="ml-auto" /> |
There was a problem hiding this comment.
Restore RTL-aware submenu chevron direction.
Line 309 drops RTL flipping, so submenu affordance points the wrong way in RTL layouts. Please keep RTL-aware flipping on this icon.
Suggested fix
- <ChevronRightIcon className="ml-auto" />
+ <ChevronRightIcon className="cn-rtl-flip ml-auto" />📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| <ChevronRightIcon className="ml-auto" /> | |
| <ChevronRightIcon className="cn-rtl-flip ml-auto" /> |
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@apps/www/components/ui/dropdown-menu.tsx` at line 309, The submenu chevron
(ChevronRightIcon) lost RTL-aware flipping; restore an RTL-aware class on the
ChevronRightIcon element so it points left in RTL layouts (e.g., replace the
current className="ml-auto" with an RTL-aware variant that keeps right-aligned
layout in LTR and flips the icon in RTL, such as adding rtl:mr-auto rtl:ml-0 and
an RTL horizontal flip like rtl:scale-x-[-1] on ChevronRightIcon).
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary by CodeRabbit
Style
h-4 w-4→size-4,text-sm leading-normal→text-sm/normal).Refactor
Documentation
Chores