Skip to content

chore(deps): upgrade to TypeScript 7.0.1-rc#585

Open
travis-hoover-glean wants to merge 1 commit into
mainfrom
upgrade-typescript-7
Open

chore(deps): upgrade to TypeScript 7.0.1-rc#585
travis-hoover-glean wants to merge 1 commit into
mainfrom
upgrade-typescript-7

Conversation

@travis-hoover-glean

Copy link
Copy Markdown
Contributor

Summary

Upgrades typescript from ~5.9.3 to the 7.0.1-rc native (Go) compiler.

TypeScript 7.0 removes the baseUrl option (hard error TS5102). baseUrl was set in three places: our root tsconfig.json, the theme package tsconfig, and @docusaurus/tsconfig (the upstream base we extends). Because an inherited option can't be unset via extends, the base config is patched to drop baseUrl + paths, and paths: { "@site/*": ["./*"] } is re-declared locally in the root config where the alias is used.

Changes

  • package.json: typescript7.0.1-rc; added pnpm.patchedDependencies
  • patches/@docusaurus__tsconfig@3.10.1.patch: removes baseUrl + paths from the base config
  • tsconfig.json: drop baseUrl; re-declare paths: { "@site/*": ["./*"] }
  • packages/docusaurus-theme-glean/tsconfig.json: drop baseUrl

Validation

  • pnpm build (CI gate): passes (27/27)
  • pnpm format:check, pnpm sidebar:check: pass
  • tsc: 0 TS5102 baseUrl errors; @site/* resolves
  • changelog-generator tsc emit: 0 errors (~5× faster: 1.9s → 0.37s)

Notes

  • The site build (pnpm build) transpiles via swc/esbuild and never invokes tsc, so this is not on the build critical path; the win is faster typecheck + editor.
  • tsc --noEmit now reports more errors (26 → 39) purely because TS 7 is stricter (verified: removing baseUrl is behavior-neutral under 5.9.3). These are pre-existing latent issues (null/undefined narrowing, an implicit-any) and tsc is not a CI gate. Can be cleaned up in a follow-up.
  • The patch is a temporary bridge until @docusaurus/tsconfig ships without baseUrl; remove patches/ + the patchedDependencies entry then.

Warning

typescript@7.0.1-rc is a release candidate. No stable programmatic API ships until 7.1; nothing here consumes the TS API.

Bumps typescript from ~5.9.3 to the 7.0.1-rc native (Go) compiler.

TypeScript 7.0 removes the `baseUrl` option (hard error TS5102). It is set
in three places here: our root tsconfig, the theme package tsconfig, and
@docusaurus/tsconfig (the upstream base we extend). Since an inherited
option can't be unset via `extends`, the base config is patched (pnpm
patch) to drop `baseUrl` + `paths`, and `paths: { "@site/*": ["./*"] }` is
re-declared locally in the root config where the alias is used.

Validation:
- pnpm build (CI gate): passes
- pnpm format:check / sidebar:check: pass
- tsc: 0 TS5102 errors; @site/* resolves
- changelog-generator tsc emit: 0 errors

The patch is a temporary bridge until @docusaurus/tsconfig ships without
baseUrl; remove patches/ and the patchedDependencies entry at that point.
@vercel

vercel Bot commented Jun 19, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
glean-developer-site Ready Ready Preview, Comment Jun 19, 2026 5:57pm

Request Review

@travis-hoover-glean travis-hoover-glean marked this pull request as ready for review June 19, 2026 20:17
@travis-hoover-glean travis-hoover-glean requested a review from a team as a code owner June 19, 2026 20:17
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