Repository navigation
build(deps-dev): update typescript to 6.0.3 - #352
Conversation
|
@codex review |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
Codex Review: Didn't find any major issues. Bravo. Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
This PR updates TypeScript from 5.7.2 to 6.0.3, the latest 6.x release. TypeScript 7 waits until the tools support it. #351 shows that TypeScript 7 breaks CI: it removes the JS API, so
vue-tscfails withERR_PACKAGE_PATH_NOT_EXPORTED: './lib/tsc', and the builds andastro checkfail too.typescriptto the pnpm catalog. The root,examples/astroandexamples/nextusecatalog:.typescript7 and later. Remove this rule whenvue-tsc,vite-plugin-dtsand@astrojs/checksupport TypeScript 7.allowImportingTsExtensionsto the roottsconfig.json.tsc -p .reported TS5097 for the.tsimport path invitest.config.ts, also with TypeScript 5.7.2. The test tsconfigs already set this option, and the base config setsnoEmit.TypeScript 6 changes two defaults that affect the build. The 6.0 release notes list both:
noUncheckedSideEffectImportsis nowtrue. The compat package type-checks the react sources through the tsconfig paths, but itsincludedid not havepackages/react/src/reveal-style.d.ts. In CI,distdoes not exist yet, soimport "#reveal-style.css"failed with TS2882. Both compat tsconfigs now include the declaration.rootDiris now the tsconfig folder. The paths add the core sources to each adapter, sovite-plugin-dtslogged TS6059 for all 4 adapters, but the build did not fail.tsconfig.base.jsonnow setsrootDirto the repo root, which contains all the sources.The
distoutput does not change. With TypeScript 6, the build makes the same 49 files, and each.d.tsfile is byte-identical to the TypeScript 5.7.2 output.Checklist
nextbranch.!and aBREAKING CHANGE:footer.distfolders, as in CI:REVEAL_BROWSERS=chromium pnpm test: 635 passed, 28 skipped, no type errorspnpm test:package: 50 passed, no TypeScript diagnostics in the build log, publint and attw find no problemstsc -p packages/<package>/testfor core, react, react-awesome-reveal and astro,tsc -p packages/test-utils,vue-tsc -p testfor vue, andtsc -p .astro checkinexamples/astro,tsc --noEmitinexamples/nextandexamples/react,vue-tscinexamples/vuepnpm lintandpnpm format:check