Skip to content

Conversation

@kentcdodds
Copy link
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
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.

3 participants