Skip to content

Fix/mobile tsconfig path aliases - #586

Open
Davoski1 wants to merge 3 commits into
Miracle656:mainfrom
Davoski1:fix/mobile-tsconfig-path-aliases
Open

Fix/mobile tsconfig path aliases#586
Davoski1 wants to merge 3 commits into
Miracle656:mainfrom
Davoski1:fix/mobile-tsconfig-path-aliases

Conversation

@Davoski1

Copy link
Copy Markdown

Summary

Configure the mobile TypeScript project to use the same @/ path alias as the web wallet by mapping "@/*": ["./*"]. This makes imports consistent across mobile and web and prevents import/type errors when moving code between the apps.

Changes

  • Update frontend/mobile/tsconfig.json:
    • Change "@/*" path mapping from ["./app/*"] to ["./*"].
    • Preserve existing "@/assets/*": ["./assets/*"] mapping.

Why

The web wallet uses "@/*": ["./*"], but the mobile app previously mapped @/* only to ./app/*. That made many @/... imports resolve differently on mobile vs web, causing type and resolution issues. Unifying the alias improves developer ergonomics and CI consistency.

Testing / Verification

  • Run tsc --noEmit in the frontend/mobile workspace (or rely on CI) to verify no resolution/type errors.
  • Build a development Expo run to ensure Metro/Babel resolves the alias as expected (Expo automatically respects path mappings when using a compatible Metro config or plugin).
    Closes 9. tsconfig + path aliases #437

@Davoski1
Davoski1 requested a review from Miracle656 as a code owner July 29, 2026 16:59
@vercel

vercel Bot commented Jul 29, 2026

Copy link
Copy Markdown

@rhoggs-bot-test-account is attempting to deploy a commit to the miracle656's projects Team on Vercel.

A member of the Team first needs to authorize it.

@drips-wave

drips-wave Bot commented Jul 29, 2026

Copy link
Copy Markdown

@Davoski1 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

9. tsconfig + path aliases

1 participant