Skip to content

remove tsconfig aliases#1054

Merged
kentcdodds merged 2 commits intomainfrom
ts-alias
Oct 23, 2025
Merged

remove tsconfig aliases#1054
kentcdodds merged 2 commits intomainfrom
ts-alias

Conversation

@kentcdodds
Copy link
Copy Markdown
Member

@kentcdodds kentcdodds commented Oct 23, 2025

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.

jansepke and others added 2 commits October 23, 2025 12:03
as typescript reuses package.json import config
@kentcdodds kentcdodds mentioned this pull request Oct 23, 2025
1 task
@kentcdodds kentcdodds merged commit dafaa96 into main Oct 23, 2025
7 checks passed
@kody-bot kody-bot deleted the ts-alias branch March 24, 2026 21:05
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.

2 participants