Skip to content

Fix broken lucide import after expression rename#61

Merged
nahiyankhan merged 1 commit intomainfrom
fix/expression-icon-import
Apr 25, 2026
Merged

Fix broken lucide import after expression rename#61
nahiyankhan merged 1 commit intomainfrom
fix/expression-icon-import

Conversation

@nahiyankhan
Copy link
Copy Markdown
Collaborator

@nahiyankhan nahiyankhan commented Apr 25, 2026

Summary

  • The fingerprint→expression rename in df1dca6 over-eagerly renamed the Fingerprint lucide-react icon to Expression, which doesn't exist as an export — broke the GitHub Pages deploy.
  • Swapped in Orbit instead. Visually says "an organizing principle other things revolve around," which fits the concept of a design language expression better than a literal mark.
  • Touches apps/docs/src/app/docs/page.tsx and apps/docs/src/app/tools/page.tsx.

Test plan

  • pnpm --filter ghost-docs exec tsc --noEmit clean
  • pnpm check + pnpm test + pnpm build (pre-push hook)
  • GitHub Pages deploy succeeds on merge

🤖 Generated with Claude Code

The fingerprint→expression rename in df1dca6 over-eagerly renamed the
Fingerprint lucide-react icon to Expression, which doesn't exist. Swap
in Orbit instead — visually says "an organizing principle other things
revolve around," which is closer to what an expression actually is.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@nahiyankhan nahiyankhan merged commit f9724ab into main Apr 25, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant