Skip to content

Integrate @sailkit/lighthouse for 404 page#21

Open
joshribakoff wants to merge 3 commits intomainfrom
lighthouse-404
Open

Integrate @sailkit/lighthouse for 404 page#21
joshribakoff wants to merge 3 commits intomainfrom
lighthouse-404

Conversation

@joshribakoff
Copy link
Owner

Summary

  • Replaces inline fuzzy matching code with @sailkit/lighthouse NotFound component
  • Reduces 404.astro from ~385 lines to ~66 lines
  • Maintains same functionality: fuzzy matching, auto-redirect, suggestions
  • CSS variables override to match site theme

Dependencies

  • Depends on: sailkit#8 (lighthouse implementation)

Test plan

  • Build passes
  • Manual testing of 404 page with various non-existent URLs
  • Verify auto-redirect works for high-confidence matches
  • Verify suggestions list displays for multiple matches

🤖 Generated with Claude Code

Replaces inline fuzzy matching with lighthouse NotFound component.

- Reduce 404.astro from ~385 lines to ~66 lines
- Use lighthouse's drop-in NotFound component
- Override CSS variables to match site theme
- Keep same functionality: fuzzy matching, auto-redirect, suggestions

Depends on: sailkit#8

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@vercel
Copy link

vercel bot commented Dec 15, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
fightingwithai Ready Ready Preview, Comment Dec 20, 2025 7:30pm

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