Skip to content

add favicon, error page, remove garu tokenizer - #13

Merged
seokju-na merged 2 commits into
mainfrom
favicon-and-more
Jul 11, 2026
Merged

add favicon, error page, remove garu tokenizer#13
seokju-na merged 2 commits into
mainfrom
favicon-and-more

Conversation

@seokju-na

@seokju-na seokju-na commented Jul 11, 2026

Copy link
Copy Markdown
Member

Summary by cubic

Added site icons and a localized global error page with Sentry reporting. Replaced the garu-based Korean search with a lightweight in-house tokenizer to shrink the Worker bundle and simplify the search API.

  • New Features

    • Added favicons and Apple touch icon; linked in __root head tags.
    • Global error page with locale-aware UI, retry via router.invalidate(), home action, and Sentry event ID. Shows raw error message only in dev.
  • Refactors

    • Replaced garu-ko/garu-orama-tokenizer with a small Korean tokenizer for @orama/orama that strips common particles/endings and unions with English tokens; no longer forwards language to the base tokenizer to avoid Orama LANGUAGE_NOT_SUPPORTED.
    • Simplified search server: per-locale map uses the custom ko tokenizer and builds once without dynamic imports.
    • Removed garu-specific Vite aliases/plugins, garu-runtime.d.ts, and related deps; cleaned package.json/yarn.lock; dropped WORKS.md.
    • Added *.env to .gitignore.

Written for commit 36e9824. Summary will update on new commits.

Review in cubic

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jul 11, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
webview-bundle-website 36e9824 Commit Preview URL

Branch Preview URL
Jul 11 2026, 12:02 PM

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All reported issues were addressed across 16 files

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread src/lib/korean-tokenizer.ts Outdated
Comment thread src/layouts/ErrorPage.tsx Outdated
…validate

- korean-tokenizer: Orama's tokenizer throws LANGUAGE_NOT_SUPPORTED for any
  language other than its own, so passing our `korean` through to the `english`
  base would break as soon as a caller supplies one (Orama leaves it undefined
  today, which is why it appeared to work).
- ErrorPage: `reset` only clears the boundary, which re-renders the same stored
  loader error. `router.invalidate()` re-runs the loaders and bumps `loadedAt`,
  the key the boundary resets on, so the button actually retries.
- Drop WORKS.md, which `oxfmt --check` was failing on in CI.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@seokju-na
seokju-na merged commit 4f393b8 into main Jul 11, 2026
4 checks passed
@seokju-na
seokju-na deleted the favicon-and-more branch July 11, 2026 12:03
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