Skip to content

fix(dashboard): pin curated Lucide imports - #7338

Merged
houko merged 3 commits into
mainfrom
audit/dashboard-lucide-public
Aug 15, 2026
Merged

houko merged 3 commits into
mainfrom
audit/dashboard-lucide-public

Conversation

@houko

@houko houko commented Aug 14, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • pin lucide-react to the exact version used by the curated internal-path barrel
  • document the intentional version coupling beside the mappings
  • import and validate every curated icon in a smoke test
  • retain the existing tree-shaken production bundle instead of routing through the much larger public root chunk

Verification

  • corepack pnpm install --frozen-lockfile
  • corepack pnpm test (100 files, 1042 tests)
  • corepack pnpm lint
  • corepack pnpm typecheck
  • corepack pnpm build (Lucide chunk 42.58 KB / 14.03 KB gzip)
  • git diff --check origin/main...HEAD

Out of scope

  • changing the curated icon set or aliases

@github-actions github-actions Bot added no-rust-required This task does not require Rust knowledge size/S 10-49 lines changed labels Aug 14, 2026
@houko
houko enabled auto-merge (squash) August 15, 2026 00:07
@houko
houko merged commit 2c1b49f into main Aug 15, 2026
37 checks passed
@houko
houko deleted the audit/dashboard-lucide-public branch August 15, 2026 00:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-rust-required This task does not require Rust knowledge size/S 10-49 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant