FLO-162: Dependency hygiene — drop web-vitals boilerplate, patch yaml advisory#4
Conversation
- Remove unused web-vitals dep + src/reportWebVitals.ts (CRA leftover on the removed v2 onCLS/getFID API; not called for any real metrics). - Drop the reportWebVitals import/call from src/index.tsx. - Bump yaml ^2.8.2 -> ^2.8.3 (resolves 2.9.0) to clear GHSA-48c2-rrv3-qjmp (moderate: stack overflow via deeply nested collections). - @types/node was already ^22 (handled in FLO-158), so no change needed. Verification: yarn typecheck / build / test all green (117 tests); yarn npm audit reports no advisories; app loads and board renders. Co-Authored-By: Paperclip <noreply@paperclip.ing>
|
Warning Review limit reached
Your plan currently allows 1 review/hour. Refill in 48 minutes and 38 seconds. Your organization has run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After more review capacity refills, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than trial, open-source, and free plans. In all cases, review capacity refills continuously over time. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
✨ 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 |
Summary
Final cleanup in the modernization chain from FLO-156 — clears the residual stale dependencies left after the CRA removal.
web-vitals+ deletesrc/reportWebVitals.tsand its import/call insrc/index.tsx. It was unused CRA boilerplate on the removed v2getCLS/getFIDAPI — never wired to any real metrics sink. (If perf metrics are wanted later, that's a separate ticket to addweb-vitals@5with the currentonCLS/onINPAPI.)yaml^2.8.2→^2.8.3(resolves 2.9.0) to clear GHSA-48c2-rrv3-qjmp (moderate: stack overflow via deeply nested collections).@types/nodewas already aligned to^22in FLO-158, so no change needed there.Verification
🤖 Generated with Claude Code