Bump deps, drop react-tooltip, fix devDep categorization#81
Conversation
- Bump @fluffylabs/shared-ui 0.6.3 -> 0.8.0 - Bump lucide-react 0.577.0 -> 1.8.0 (major) - Bump @biomejs/biome 2.4.8 -> 2.4.12 (+ schema URL) - Move @biomejs/biome, @tailwindcss/vite, tailwindcss to devDependencies - Replace react-tooltip with WithTooltip from @fluffylabs/shared-ui in DiffHighlight - npm audit fix (resolves vite + picomatch advisories) Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
✅ Deploy Preview for fluffy-codec ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughUpdated Biome schema version from 2.4.8 to 2.4.12 in configuration. Reorganized package dependencies: moved Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Summary
@fluffylabs/shared-ui0.6.3 → 0.8.0,lucide-react0.577 → 1.8.0 (major),@biomejs/biome2.4.8 → 2.4.12.npm audit fixalso bumpedviteandpicomatchtransitively, clearing 2 high-severity advisories (0 remaining).@biomejs/biome,@tailwindcss/vite, andtailwindcssfromdependenciestodevDependencies— they're build/lint tooling, not runtime code.react-tooltip: its single use inDiffHighlight.tsxis now served byWithTooltipfrom@fluffylabs/shared-ui(which we already depend on). ~40 lines of tooltip boilerplate removed, no new code.Behavior note
The diff tooltip previously used per-type background colors (blue/red/green) on the tooltip popover. With
WithTooltipthe popover uses shared-ui's default styling; the colored background on the visible text segment is preserved, so the type signal is still there.Test plan
npm run qa— cleannpm run test— 152/152 passnpm run build— succeedsnpm audit— 0 vulnerabilitieslucide-reacticons still export under v1.8.0 (UploadIcon,InfoIcon,X,ChevronDownIcon,Settings)🤖 Generated with Claude Code