Home Original page

remove tsconfig aliases by kentcdodds · Pull Request #1054 · epicweb-dev/epic-stack

as typescript reuses package.json import config

Checklist

  • Docs updated

solves #1029

Originally opened by @jansepke as #1030, but moved to this because I needed to make changes and I couldn't push to that PR because it was opened from @jansepke's main branch rather than a separate branch.

Note

Removes tsconfig path aliases in favor of package.json "imports", updates import specifiers across the app/tests, and documents the change with a new ADR.

  • Build/TypeScript:
    • Remove compilerOptions.paths aliases from tsconfig.json.
    • Update internal imports to include .ts/.tsx extensions throughout the codebase.
  • Docs:
    • Add docs/decisions/046-remove-path-aliases.md documenting the move to package.json "imports".
    • Mark docs/decisions/031-imports.md as superseded and revise guidance.
  • App:
    • Update imports in app/components/error-boundary.tsx, app/routes/admin/cache/index.tsx, and app/routes/admin/cache/sqlite.server.ts.
  • Tests/Seed:
    • Adjust imports in tests/e2e/onboarding.test.ts, tests/playwright-utils.ts, app/utils/auth.server.test.ts, and prisma/seed.ts to new specifiers.

Written by Cursor Bugbot for commit 522a85b. This will update automatically on new commits. Configure here.