Skip to content

Remove deprecated baseUrl from tsconfig.app.json#2612

Merged
adamw merged 1 commit intomasterfrom
fix-tsconfig-baseurl-deprecation
Apr 1, 2026
Merged

Remove deprecated baseUrl from tsconfig.app.json#2612
adamw merged 1 commit intomasterfrom
fix-tsconfig-baseurl-deprecation

Conversation

@adamw
Copy link
Copy Markdown
Member

@adamw adamw commented Apr 1, 2026

Summary

  • Remove baseUrl: "." from tsconfig.app.json — it was deprecated in TypeScript 6.0 (TS5101 error)
  • Since baseUrl was ".", path resolution is unchanged — paths entries resolve relative to the tsconfig file by default
  • Fixes CI failure introduced by the TypeScript 5.9.3 → 6.0.2 bump (Bump typescript from 5.9.3 to 6.0.2 in /ui #2609)

Test plan

  • yarn tsc --noEmit -p tsconfig.app.json passes locally with no errors

🤖 Generated with Claude Code

TypeScript 6.0 deprecated `baseUrl`. Since it was set to `"."` (the
default for path resolution), removing it changes nothing but fixes the
TS5101 error that breaks CI.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 1, 2026 19:47
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Removes the deprecated compilerOptions.baseUrl: "." setting from ui/tsconfig.app.json to resolve the TypeScript 6.0 TS5101 configuration error while keeping path resolution behavior unchanged.

Changes:

  • Removed baseUrl: "." from ui/tsconfig.app.json to avoid TS 6.0 deprecation/error.
  • Kept existing paths alias (@/*./src/*) intact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@adamw adamw merged commit 52c8707 into master Apr 1, 2026
9 checks passed
@adamw adamw deleted the fix-tsconfig-baseurl-deprecation branch April 1, 2026 19:52
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