Feature/fix/nx build config#43
Merged
mocolicious merged 24 commits intoAug 11, 2025
Merged
Conversation
feat(nx-cloud): set up nx workspace
Bumps [axios](https://github.com/axios/axios) from 0.27.2 to 0.30.0. - [Release notes](https://github.com/axios/axios/releases) - [Changelog](https://github.com/axios/axios/blob/v0.30.0/CHANGELOG.md) - [Commits](axios/axios@v0.27.2...v0.30.0) --- updated-dependencies: - dependency-name: axios dependency-version: 0.30.0 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@nestjs/common](https://github.com/nestjs/nest/tree/HEAD/packages/common) from 9.2.1 to 10.4.16. - [Release notes](https://github.com/nestjs/nest/releases) - [Commits](https://github.com/nestjs/nest/commits/v10.4.16/packages/common) --- updated-dependencies: - dependency-name: "@nestjs/common" dependency-version: 10.4.16 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [undici](https://github.com/nodejs/undici) from 5.3.0 to 5.29.0. - [Release notes](https://github.com/nodejs/undici/releases) - [Commits](nodejs/undici@v5.3.0...v5.29.0) --- updated-dependencies: - dependency-name: undici dependency-version: 5.29.0 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [@babel/runtime](https://github.com/babel/babel/tree/HEAD/packages/babel-runtime) from 7.18.3 to 7.27.3. - [Release notes](https://github.com/babel/babel/releases) - [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md) - [Commits](https://github.com/babel/babel/commits/v7.27.3/packages/babel-runtime) --- updated-dependencies: - dependency-name: "@babel/runtime" dependency-version: 7.27.3 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [http-proxy-middleware](https://github.com/chimurai/http-proxy-middleware) from 2.0.6 to 2.0.9. - [Release notes](https://github.com/chimurai/http-proxy-middleware/releases) - [Changelog](https://github.com/chimurai/http-proxy-middleware/blob/v2.0.9/CHANGELOG.md) - [Commits](chimurai/http-proxy-middleware@v2.0.6...v2.0.9) --- updated-dependencies: - dependency-name: http-proxy-middleware dependency-version: 2.0.9 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [base-x](https://github.com/cryptocoinjs/base-x) from 3.0.9 to 3.0.11. - [Commits](cryptocoinjs/base-x@v3.0.9...v3.0.11) --- updated-dependencies: - dependency-name: base-x dependency-version: 3.0.11 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…/axios-0.30.0 chore(deps): bump axios from 0.27.2 to 0.30.0
…/nestjs/common-10.4.16 chore(deps): bump @nestjs/common from 9.2.1 to 10.4.16
…/undici-5.29.0 chore(deps): bump undici from 5.3.0 to 5.29.0
…/babel/runtime-7.27.3 chore(deps): bump @babel/runtime from 7.18.3 to 7.27.3
…/http-proxy-middleware-2.0.9 chore(deps): bump http-proxy-middleware from 2.0.6 to 2.0.9
…/base-x-3.0.11 chore(deps): bump base-x from 3.0.9 to 3.0.11
Bumps [serialize-javascript](https://github.com/yahoo/serialize-javascript) from 6.0.0 to 6.0.2. - [Release notes](https://github.com/yahoo/serialize-javascript/releases) - [Commits](yahoo/serialize-javascript@v6.0.0...v6.0.2) --- updated-dependencies: - dependency-name: serialize-javascript dependency-version: 6.0.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
Bumps [systeminformation](https://github.com/sebhildebrandt/systeminformation) from 5.11.16 to 5.27.1. - [Changelog](https://github.com/sebhildebrandt/systeminformation/blob/master/CHANGELOG.md) - [Commits](sebhildebrandt/systeminformation@v5.11.16...v5.27.1) --- updated-dependencies: - dependency-name: systeminformation dependency-version: 5.27.1 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…/serialize-javascript-6.0.2 chore(deps): bump serialize-javascript from 6.0.0 to 6.0.2
…/systeminformation-5.27.1 chore(deps): bump systeminformation from 5.11.16 to 5.27.1
Bumps [rollup](https://github.com/rollup/rollup) from 2.75.0 to 2.79.2. - [Release notes](https://github.com/rollup/rollup/releases) - [Changelog](https://github.com/rollup/rollup/blob/master/CHANGELOG.md) - [Commits](rollup/rollup@v2.75.0...v2.79.2) --- updated-dependencies: - dependency-name: rollup dependency-version: 2.79.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com>
…/rollup-2.79.2 chore(deps): bump rollup from 2.75.0 to 2.79.2
…cache 📜 Changelog 1️⃣ Dependency Updates Updated and added necessary Nx packages: Replaced @nx/node executors with @nx/js:tsc and @nx/js:node. Installed missing dependencies like ts-loader. Verified and fixed versions for typescript, @typescript-eslint/*, and other related packages. Ensured compatibility for all @nx/* packages in resolutions. 2️⃣ Project Configuration Changes project.json: Replaced the old @nx/webpack:webpack executor with @nx/js:tsc for the build target. Updated the serve target to use @nx/js:node. Removed unused webpack-related options (e.g., target, compiler). Cleaned up build configurations for better compatibility with the latest Nx version. tsconfig.app.json: Fixed trailing comma in compiler options. Updated include and exclude patterns for better TypeScript coverage. tsconfig.base.json: Verified and retained proper paths mapping for libraries. Ensured include covers both apps/**/*.ts and libs/**/*.ts. tsconfig.json (API): Configured references properly to ensure TypeScript incremental builds. Kept it minimal and aligned with Nx structure. tsconfig.tools.json: Validated rootDir, outDir, and module settings for scripts. 3️⃣ Error Resolution Fixed @nx/node:node executor error by migrating to @nx/js:node. Fixed ts-loader missing dependency error by installing it locally. Resolved rootDir errors in TypeScript by adjusting configs and removing incorrect libs inclusion from tsconfig.app.json. Cleared Nx and TypeScript caches to eliminate stale build artifacts: rm -rf dist/ rm -rf node_modules/.cache 4️⃣ Final Verification Successfully built and served the API app using nx run api:build and nx serve api. Confirmed that TypeScript references and library imports from libs/ resolved correctly. Verified no stale cache or residual compilation issues after cleanup. ✅ Outcome: API project now builds and serves without errors using Nx, with clean and maintainable configuration files. Would you like me to generate this as a conventional commit with BREAKING CHANGE or feat/chore sections for a proper semantic version changelog (so it works well with tools like standard-version or release-please)?
There was a problem hiding this comment.
Pull Request Overview
This PR upgrades the project from Nx v15 to v21, updating packages, configurations, and fixing build compatibility issues. The upgrade includes major version updates for NestJS, TypeScript, cache-manager, and other core dependencies.
- Upgraded Nx from v15.4.5 to v21.3.11 with updated package references throughout
- Updated NestJS dependencies to v11 and TypeScript to v5.4.0
- Fixed cache-manager integration and Buffer usage for Node.js compatibility
Reviewed Changes
Copilot reviewed 254 out of 256 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Core dependency upgrades including Nx, NestJS, TypeScript, and cache-manager |
| tsconfig.base.json | Added include property and removed rootDir for better TypeScript configuration |
| All project.json files | Updated executor references from @nrwl/* to @nx/* |
| Cache service | Fixed cache-manager v7 compatibility and CACHE_MANAGER import |
| Buffer usage | Updated Buffer operations to use Uint8Array for Node.js compatibility |
@bbajwadev Im going to apply this suggestion, Kin-memo is obsolete so we should probably remove this feature in the release regardless. Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
mocolicious
approved these changes
Aug 10, 2025
mocolicious
merged commit 78f2907
into
kinnytips:fix-ui-libraries
6 of 12 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.