Chore/build and config cleanup - #413
Merged
Merged
Conversation
…options Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
…move wxt to devDependencies Drop dompurify, react-ga4, and workbox-background-sync — none are referenced anywhere in the codebase. Add framer-motion as an explicit dependency: src/common/motion.tsx and src/pages/root.tsx import it directly, but it was only resolving as a hoisted transitive dependency. Move wxt to devDependencies since it's build-only tooling that doesn't ship in the built extension. Also drop autoprefixer, postcss, globals, and @types/styled-jsx, which aren't referenced by any config or source file.
… passes No behavioral changes. Running `biome check --write` across the touched directories for the commits above also reformatted a handful of unrelated files it happened to pass over: quote-style normalization, multi-line call wrapping, a couple of missing trailing newlines, a redundant `!!` double-negation, and one `@ts-ignore` -> `@ts-expect-error` (Biome's stricter, more specific suppression form). Isolated here so the functional commits stay readable.
sajjadmrx
approved these changes
Aug 8, 2026
sajjadmrx
approved these changes
Aug 8, 2026
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 free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Fix broken production build config: wxt.config.ts set tryCatchDeoptimization: false under rollupOptions.treeshake, a Rollup-only option that WXT's rolldown bundler rejects ("Expected never but received ..."). Removed it, along with moduleSideEffects: false (too aggressive — overrides the safe default for deps like the webextension polyfill and workbox that rely on side effects for ~0.008% bundle savings).
Added a @/styles path alias for upcoming style work.
Enabled Tailwind directive parsing and CSS formatting in Biome.
Added .gitignore entry for a local docs/ folder and a .gitattributes to normalize line endings to LF.
Dependency cleanup: removed dompurify, react-ga4, workbox-background-sync, postcss, autoprefixer, globals, and @types/styled-jsx — none are referenced anywhere in the codebase. Added framer-motion as an explicit dependency (it was only resolving as a hoisted transitive dep before). Moved wxt to devDependencies since it's build-only tooling.
Applied a handful of incidental Biome formatting fixes (quote style, trailing newlines, a redundant !!, @ts-ignore → @ts-expect-error) picked up while touching the files above.