chore(ts): TypeScript 6.0.3, explicit types ["node","jest"], keep tests out of graphile-plugin-utils dist - #1843
Conversation
…ts from graphile-plugin-utils dist
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
Review complete. No issues found — approved ✅. This PR upgrades the TypeScript compiler dependency from scattered 5.x ranges to 6.0.3 across 11 packages in the Constructive monorepo, and makes two tsconfig adjustments: the root
Reviewed commit: b93646e |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Summary
TS 6 baseline for the monorepo (TS 7 prep, no emit change to
dist/):typescript→^6.0.3everywhere (root + 12 package.json that pinned^5.0.0/^5.1.6/^5.8.3/^5.9.3).ts-jest@29.4.12peers<7,@typescript-eslint@8.66peers<6.1— both fine.tsconfig.json:"types": ["node", "jest"]. TS 6 (warn) / TS 7 (hard) stop auto-loading every@types/*undertypeRoots; without this 39/125 packages loseprocess/Buffer/describeunder tsc 7.typeRootsis kept for the ambient stubs in./types.graphile/graphile-plugin-utils/tsconfig.json:exclude**/*.test.*/**/*.spec.*— it was emittingdist/__tests__/*.test.js(+dist/esm/__tests__) into the published package. Same fix as graphile-llm in chore(ts): moduleResolution node -> nodenext (CJS) / bundler (ESM) #1834.Verified locally under tsc 6.0.3:
pnpm -r build(0 errors),pnpm lint, fullpnpm -r testwith Postgres + RustFS +fixtures:install(all pass except graphile-llm's real-Ollama suite, which needs a local Ollama),lerna versiondry run (43 packages bump cleanly),npm pack --dry-runof graphile-plugin-utilsdist/contains no test files.Link to Devin session: https://app.devin.ai/sessions/d34558e831ad4bbf986f64cc4bd509bd
Open in Devin Desktop: https://app.devin.ai/desktop/session/d34558e831ad4bbf986f64cc4bd509bd?variant=devin
Requested by: @pyramation