chore(deps): consolidate dependency and toolchain upgrades#1225
Conversation
|
@copilot Install dependencies is failing, pnpm v11 requires |
There was a problem hiding this comment.
Pull request overview
This PR consolidates dependency and toolchain upgrades across the pnpm monorepo, including Node/pnpm tool versions, TypeScript, and several key runtime/test dependencies.
Changes:
- Upgraded toolchain/runtime config (mise) and repository TypeScript configuration (incl. TS 6 deprecation handling).
- Refreshed dependency versions and lockfile (TypeScript 6, Vitest, GraphQL, AWS SDK, @types/node).
- Minor typing improvement in RabbitMQ connection getter.
Reviewed changes
Copilot reviewed 10 out of 12 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| tsconfig.spec.json | Adds explicit node/jest types for Jest + ts-jest compilation. |
| tsconfig.json | Adds TS deprecation suppression config and restricts ambient types to node. |
| pnpm-lock.yaml | Lockfile refresh reflecting upgraded TypeScript/Vitest/GraphQL/AWS SDK dependency graph. |
| packages/testing/ts-jest/package.json | Bumps AWS S3 client version used in the ts-jest testing package. |
| packages/rabbitmq/src/amqp/connection.ts | Adds an explicit return type for managedChannels getter. |
| packages/graphql-request/package.json | Bumps graphql dependency to ^16.14.0. |
| package.json | Bumps TypeScript to 6.0.3, updates @types/node, and Vitest optional deps. |
| mise.toml | Updates pinned Node and pnpm versions used by CI/local via mise. |
| integration/rabbitmq/tsconfig.json | Adds ignoreDeprecations setting for TS 6 behavior. |
| integration/rabbitmq/package.json | Bumps TypeScript to 6.0.3 for the RabbitMQ integration project. |
| integration/google-cloud-pubsub/tsconfig.json | Adds ignoreDeprecations setting for TS 6 behavior. |
| integration/google-cloud-pubsub/package.json | Bumps TypeScript to 6.0.3 for the Pub/Sub integration project. |
Implemented in Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
|
@copilot Review the last CI run, it appears that sinon/some test types are not available to some of the libraries. It can be related to the typescript upgrade |
Investigated the latest failed CI run ( |
| "target": "es2020", | ||
| "sourceMap": true, | ||
| "outDir": "./dist", | ||
| "baseUrl": "./", |
There was a problem hiding this comment.
@copilot is this still required the baseUrl removals?
There was a problem hiding this comment.
They are not strictly required for TS 5.9.3, but keeping the baseUrl removals is still safe and keeps the integration tsconfigs compatible with TS 6 deprecation rules. They don’t change runtime behavior in this repo’s current import setup. No UI changes to screenshot for this config question.
|
@copilot CI is failing on typecheck, run typecheck before commit and verify that all libraries compile just fine |
Addressed in
Checked the latest CI run ( |
|
@copilot Close all the open dependantbot/renovate PRs (dependencies) that this PR will address |
Verified on |
I can’t directly close other PRs from this environment (the available GitHub tools here are read-only for PR state changes), but I verified this branch ( |
Uh oh!
There was an error while loading. Please reload this page.