Skip to content

Feature/update deps#408

Open
TriPSs wants to merge 9 commits intomasterfrom
feature/update-deps
Open

Feature/update deps#408
TriPSs wants to merge 9 commits intomasterfrom
feature/update-deps

Conversation

@TriPSs
Copy link
Owner

@TriPSs TriPSs commented Nov 27, 2025

Summary by CodeRabbit

  • Chores

    • Updated Yarn package manager to 4.12.0.
    • Upgraded numerous dependencies and dev tooling (frameworks, GraphQL, DB drivers, docs, linters/formatters, test tooling) for stability, performance, and security.
  • Style

    • Reformatted several public type and class declarations for consistent multiline readability (no behavioral or API changes).

TriPSs added 2 commits October 6, 2025 09:31
- Bumped `@nestjs/common`, `@nestjs/core`, and other `@nestjs` packages to latest minor versions.
- Upgraded devDependencies like `graphql`, `mysql2`, `prettier`, and various others.
- Synchronized updates in `yarn.lock`.
- Updated package manager to `yarn@4.12.0`.
@coderabbitai
Copy link

coderabbitai bot commented Nov 27, 2025

📝 Walkthrough

Walkthrough

The pull request upgrades Yarn from version 4.4.0 to 4.12.0 in configuration and multiple npm package dependencies across the monorepo (NestJS, Apollo, Docusaurus, GraphQL, and database drivers). Additionally, class and interface type declarations across TypeScript files are reformatted to split generic parameters and inheritance clauses across multiple lines for improved readability.

Changes

Cohort / File(s) Summary
Configuration & Dependency Updates
​.yarnrc.yml, package.json
Yarn path updated to yarn-4.12.0.cjs; packageManager set to yarn@4.12.0; numerous dependency version bumps across NestJS, Apollo, Docusaurus, GraphQL, Typegoose, DB drivers, tooling, and devDependencies. No structural or runtime logic changes.
Core Type Formatting
packages/core/src/assemblers/abstract.assembler.ts, packages/core/src/services/assembler-query.service.ts
Reflowed/split generic type parameter lists and class declarations across multiple lines; only formatting changes, no API or behavioral changes.
Query-GraphQL Loaders
packages/query-graphql/src/loader/aggregate-relations.loader.ts, packages/query-graphql/src/loader/count-relations.loader.ts, packages/query-graphql/src/loader/find-relations.loader.ts, packages/query-graphql/src/loader/query-relations.loader.ts
Multiline reformatting of implements/generic clauses for loader classes; no semantic changes to types, interfaces, or behavior.
Query-GraphQL Resolvers
packages/query-graphql/src/resolvers/crud.resolver.ts, packages/query-graphql/src/resolvers/read.resolver.ts
Interface and resolver generic/extends clauses reformatted to multi-line layout; signatures remain semantically identical though one file flagged as higher review attention due to broader public declarations.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Poem

🐰 Hop along with cleaner lines!
Dependencies bloom like springtime vines,
Yarn ascends to 4.12 delight,
Types now formatted, tidy and bright!

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Title check ❓ Inconclusive The title 'Feature/update deps' is vague and generic. It uses non-descriptive terminology ('deps' is an abbreviation) that fails to convey specific information about the changeset's primary updates. Provide a more descriptive title such as 'Update dependencies and dev tooling to latest versions' or 'Bump NestJS, Apollo, and related packages to 2025 versions' to clearly communicate the scope and purpose of the changes.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/update-deps

No actionable comments were generated in the recent review. 🎉


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

- Adjusted TypeScript patch hash in `yarn.lock` to synchronize dependency resolution.
@nx-cloud
Copy link

nx-cloud bot commented Nov 27, 2025

🤖 Nx Cloud AI Fix Eligible

An automatically generated fix could have helped fix failing tasks for this run, but Self-healing CI is disabled for this workspace. Visit workspace settings to enable it and get automatic fixes in future runs.

To disable these notifications, a workspace admin can disable them in workspace settings.


View your CI Pipeline Execution ↗ for commit 1b04aaa

Command Status Duration Result
nx run-many --target=test --all ❌ Failed 2m 26s View ↗
nx run-many --target=e2e --all ❌ Failed 1m 55s View ↗
nx run-many --target=lint --all ❌ Failed 50s View ↗
nx run-many --target=build --all ✅ Succeeded <1s View ↗
nx run workspace:version ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-10 08:16:02 UTC

- Updated `mongoose` dependency to version 8.20.1 in `package.json`.
- Updated associated dependencies: `mongodb` to 6.20.0 and `bson` to 6.10.4.
- Synchronized changes in `yarn.lock` to reflect updated resolutions.
… resolvers, and services

- Refactored generic type declarations for better readability in multiple files.
- Updated formatting in `packages/query-graphql` and `packages/core` modules.
@pkg-pr-new
Copy link

pkg-pr-new bot commented Feb 10, 2026

Open in StackBlitz

@ptc-org/nestjs-query-core

npm i https://pkg.pr.new/TriPSs/nestjs-query/@ptc-org/nestjs-query-core@408

@ptc-org/nestjs-query-graphql

npm i https://pkg.pr.new/TriPSs/nestjs-query/@ptc-org/nestjs-query-graphql@408

@ptc-org/nestjs-query-mikro-orm

npm i https://pkg.pr.new/TriPSs/nestjs-query/@ptc-org/nestjs-query-mikro-orm@408

@ptc-org/nestjs-query-mongoose

npm i https://pkg.pr.new/TriPSs/nestjs-query/@ptc-org/nestjs-query-mongoose@408

@ptc-org/nestjs-query-sequelize

npm i https://pkg.pr.new/TriPSs/nestjs-query/@ptc-org/nestjs-query-sequelize@408

@ptc-org/nestjs-query-typegoose

npm i https://pkg.pr.new/TriPSs/nestjs-query/@ptc-org/nestjs-query-typegoose@408

@ptc-org/nestjs-query-typeorm

npm i https://pkg.pr.new/TriPSs/nestjs-query/@ptc-org/nestjs-query-typeorm@408

commit: 1b04aaa

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
package.json (1)

13-16: ⚠️ Potential issue | 🟠 Major

resolutions override the updated devDependencies for Apollo packages.

The resolutions block still pins @apollo/gateway to 2.10.1 and @apollo/subgraph to 2.2.3, but the devDependencies were bumped to 2.12.1 for both. Yarn resolutions take precedence, so the actual installed versions will be the old pinned ones — the devDependency bumps on lines 39 and 41 are effectively dead.

Either update the resolutions to match, or remove them if the pinning is no longer needed.

Proposed fix (update resolutions to match)
   "resolutions": {
-    "@apollo/gateway": "2.10.1",
-    "@apollo/subgraph": "2.2.3"
+    "@apollo/gateway": "2.12.1",
+    "@apollo/subgraph": "2.12.1"
   },

Also applies to: 39-41

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