Skip to content

chore(ts): TypeScript 6.0.3, explicit types ["node","jest"], keep tests out of graphile-plugin-utils dist - #1843

Merged
pyramation merged 1 commit into
mainfrom
chore/ts6-types-node
Sep 17, 2026
Merged

pyramation merged 1 commit into
mainfrom
chore/ts6-types-node

Conversation

@pyramation

Copy link
Copy Markdown
Contributor

Summary

TS 6 baseline for the monorepo (TS 7 prep, no emit change to dist/):

  • typescript^6.0.3 everywhere (root + 12 package.json that pinned ^5.0.0/^5.1.6/^5.8.3/^5.9.3). ts-jest@29.4.12 peers <7, @typescript-eslint@8.66 peers <6.1 — both fine.
  • Root tsconfig.json: "types": ["node", "jest"]. TS 6 (warn) / TS 7 (hard) stop auto-loading every @types/* under typeRoots; without this 39/125 packages lose process/Buffer/describe under tsc 7. typeRoots is kept for the ambient stubs in ./types.
  • graphile/graphile-plugin-utils/tsconfig.json: exclude **/*.test.* / **/*.spec.* — it was emitting dist/__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, full pnpm -r test with Postgres + RustFS + fixtures:install (all pass except graphile-llm's real-Ollama suite, which needs a local Ollama), lerna version dry run (43 packages bump cleanly), npm pack --dry-run of graphile-plugin-utils dist/ 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

@devin-ai-integration

Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment, CI, and merge conflict monitoring

@tenki-reviewer

tenki-reviewer Bot commented Sep 17, 2026

Copy link
Copy Markdown

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 tsconfig.json adds a types: [node, jest] compiler option and graphile/graphile-plugin-utils/tsconfig.json adds an exclude field. All changes are version/range edits plus compiler-option tuning; no source logic was modified.

Files Change
Root package.json, tsconfig.json Bumps typescript to 6.0.3 and adds types: [node, jest] to the project compiler options.
sdk/*, graphql/*, examples/* package.json (9 files) Aligns the workspace's shared typescript dependency on the 6.0.3 range.
graphile/graphile-plugin-utils/tsconfig.json Adds an exclude field to the package build config.

Reviewed commit: b93646e

@socket-security

Copy link
Copy Markdown

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addedtypescript@​6.0.31001009010090

View full report

@pyramation
pyramation merged commit c2d11a2 into main Sep 17, 2026
22 checks passed
@pyramation
pyramation deleted the chore/ts6-types-node branch September 17, 2026 07:08
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.

1 participant