Remove baseUrl requirement for tsconfig path aliases

[meyer]

As of TypeScript 4.1, baseUrl is not required for tsconfig path aliases to work. baseUrl is required in Astro, however.

The TypeScript docs actively discourage people from setting baseUrl:

baseUrl was designed for use with AMD module loaders. If you aren’t using an AMD module loader, you probably shouldn’t use baseUrl. Since TypeScript 4.1, baseUrl is no longer required to use paths and should not be used just to set the directory paths values are resolved from.

In this PR I’ve removed the explicit baseUrl check and early return. Rather than bailing if baseUrl is not set, Astro now defaults to a baseUrl of '.'. This aligns with TypeScript’s path resolving functionality. You can read more in the TypeScript documentation page about module resolution: https://www.typescriptlang.org/docs/handbook/modules/reference.html#relationship-to-baseurl

Changes

• baseUrl now defaults to '.' if baseUrl is not set in your tsconfig file

Testing

I copied the alias-tsconfig fixture over to a new fixture, alias-tsconfig-no-baseurl.

Docs

I removed usage of baseUrl from the README for vite-plugin-config-alias.

[changeset-bot]

🦋 Changeset detected

Latest commit: 18723ab

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[codspeed-hq]

CodSpeed Performance Report

Merging #12731 will not alter performance

_{Comparing meyer:remove-baseurl-requirement (18723ab) with main (e3bfd93)}

Summary

✅ 4 untouched benchmarks