fix(frontend): align on Tailwind v4 + flat typescript-eslint import#13
Merged
Conversation
added 7 commits
July 22, 2026 15:33
The initial commit accidentally shipped Tailwind v3 config (postcss + autoprefixer + tailwind.config.ts + @tailwind directives) while the README, docs and other starters advertise Tailwind v4. It also imported 'typescript-eslint' (unified) but only depended on the legacy @typescript-eslint/* split packages, causing CI to fail at 'npm run lint'. This aligns T01 with T02-T15: @tailwindcss/vite plugin + @import 'tailwindcss' + typescript-eslint + globals + eslint parser wiring for .svelte files, plus a .prettierignore so prettier doesn't scan .svelte-kit outputs.
Three fixes: - api.ts: use PUBLIC_API_BASE (Vite exposes PUBLIC_ vars), drop the broken `process` typeof check. - i18n index: type dictionaries as `Record<Lang, Dict>` so the getter returns Dict, not the en|fr union. - api.test.ts: guard against .mock.calls being empty before indexing.
The previous fr.ts had `as const` inferred literal types that didn't match en.ts's literals, so `Record<Lang, Dict>` rejected the fr entry with 'string literal EN not assignable to FR'. Extract a shared Dict interface into types.ts and type both dictionaries as Dict — this widens the fields to plain 'string' and satisfies svelte-check strict mode.
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
1 participant
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.
The initial commit accidentally shipped Tailwind v3 config (postcss + autoprefixer + tailwind.config.ts + @tailwind directives) while the README, docs and other starters advertise Tailwind v4. It also imported
typescript-eslint(unified) but only depended on the legacy@typescript-eslint/*split packages, causing CI to fail atnpm run lint.This aligns T01 with T02-T15:
@tailwindcss/viteplugin +@import 'tailwindcss'+ drop postcss/autoprefixertypescript-eslint(unified) +globals+ eslint parser wiring for .svelte files.prettierignoreso prettier doesn't scan.svelte-kitoutputs