chore(deps): bump lucide-react to ^1.22.0 - #43
Open
datj9 wants to merge 1 commit into
Open
Conversation
The lucide-react package crossed its 1.0 milestone. Icon names and import paths used in this codebase (Cloud, BookOpen, ChevronRight, Star, etc.) remain unchanged across the boundary, so this is a no-touch bump for src/. Lint and 'next build' pass.
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 freeto 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
Audit of the cloudcompass dependency tree against the latest stable releases. The only outdated production dep with a meaningful gap is
lucide-react; everything else is already on the latest stable, has a caret range that picks up the latest patch on next install, or is a major bump intentionally deferred (see below).Bumped
lucide-react^0.577.0^1.22.0The
lucide-reactpackage crossed its 1.0 milestone. All icon names used in this codebase (Cloud,BookOpen,ChevronRight,Star,ArrowRight,CheckCircle2,XCircle,Search,Menu,X,Award,Copy,Check,Terminal,FileCode2,FileJson,FileText,Database,Lightbulb,Target,DollarSign,HelpCircle,Clock,AlertTriangle,AlertCircle,MinusCircle,Circle,RotateCcw,ArrowLeft,BarChart2,Lock,Zap,GitCompare) and their import paths ("lucide-react") are unchanged across the boundary.Affected files (verified no source change needed)
src/app/page.tsxsrc/app/learn/page.tsxsrc/app/learn/[cloud]/page.tsxsrc/app/learn/[cloud]/[topic]/page.tsxsrc/app/practice/page.tsxsrc/app/practice/[labId]/page.tsxsrc/app/certifications/page.tsxsrc/app/certifications/[certId]/page.tsxsrc/app/certifications/[certId]/[domainId]/page.tsxsrc/app/certifications/[certId]/[domainId]/quiz/QuizPageClient.tsxsrc/app/certifications/[certId]/dashboard/DashboardClient.tsxsrc/app/compare/page.tsxsrc/app/search/page.tsxsrc/components/ConfidenceRating.tsxsrc/components/MarkAsReadButton.tsxsrc/components/Navbar.tsxsrc/components/DomainNav.tsxsrc/components/QuizRunner.tsxsrc/components/ExamInfoCard.tsxsrc/components/CodeBlock.tsxDeferred (worth a follow-up, not in this PR)
These are flagged so the reviewer knows they were considered:
eslint-config-next@16.2.9advertised peer (typescript >=3.3.1is permissive but the toolchain hasn't been exercised against TS 6). Skipped pending an ecosystem-readiness review.^20.19.0 || ^22.13.0 || >=24; current dev environment is Node 22, so passable. Deferred becauseeslint-config-nextadvertises onlyeslint >=9.0.0and has not been re-tested against ESLint 10 here.^4.3.1range already resolves to 4.3.2 on a fresh install, so no package.json change is needed.Skipped audit items
npm auditflags a moderatepostcss <8.5.10advisory reachable only via a transitive dep ofnext(node_modules/next/node_modules/postcss). The "fix" would downgradenextto 9.x, which is not acceptable. Left alone, to be resolved when Next ships a patchedpostcssin its next release.Test plan
npm install— succeeds, lockfile updated, 0 errorsnpm run lint— passes (5 pre-existing unused-import warnings, unrelated to this change)npm run build— full static export succeeds across all routes (/,/learn/**,/certifications/**,/practice/**,/compare,/search,/sitemap.xml)npm run dev) — not run in CI; recommend a quick visual pass before mergeGenerated by Claude Code