Skip to content

Conversation

@FliPPeDround
Copy link
Member

@FliPPeDround FliPPeDround commented Sep 22, 2025

Update dependencies

Summary by CodeRabbit

  • Chores
    • Updated dependencies and devDependencies to latest patch/minor versions:
      • nutui-uniapp ^1.9.1
      • @uni-ku/root ^1.4.0
      • @uni-helper/vite-plugin-uni-pages ^0.3.15
      • @uni-helper/vite-plugin-uni-components ^0.2.1
      • @iconify-json/tabler ^1.2.23
      • eslint ^9.36.0
      • @types/node ^24.5.2
    • No user-facing behavior changes expected.

@coderabbitai
Copy link

coderabbitai bot commented Sep 22, 2025

Walkthrough

Multiple package.json files update dependency versions across UI templates, config templates, and plugin templates. No code, logic, or public API changes are introduced.

Changes

Cohort / File(s) Summary
UI templates
packages/core/template/UI/ano/package.json, packages/core/template/UI/nut/package.json
Bump @iconify-json/tabler ^1.2.22 → ^1.2.23; bump nutui-uniapp ^1.9.0 → ^1.9.1.
Config templates
packages/core/template/config/lint/package.json, packages/core/template/config/typescript/package.json
Bump eslint ^9.35.0 → ^9.36.0; bump @types/node ^24.5.1 → ^24.5.2.
Plugin templates
packages/core/template/plugin/import/package.json, packages/core/template/plugin/pages/package.json, packages/core/template/plugin/root/package.json
Bump @uni-helper/vite-plugin-uni-components ^0.2.0 → ^0.2.1; bump @uni-helper/vite-plugin-uni-pages ^0.3.13 → ^0.3.15; bump @uni-ku/root ^1.3.4 → ^1.4.0.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • chore: update deps #123 — Updates @uni-ku/root in plugin/root; same dependency touched here.
  • chore: update deps #127 — Bumps @types/node and @uni-helper/vite-plugin-uni-pages; overlaps with this PR’s config/typescript and plugin/pages updates.
  • chore: update deps #106 — Adjusts eslint and @types/node in config templates; same files and dependencies.

Suggested labels

size/M

Poem

I nibble versions, hop to new,
Patch by patch, the garden grew.
Icons shine, the lint is neat,
Plugins prance on lighter feet.
With tiny bumps, I thump—hooray!
The carrot builds will pass today. 🥕🐇

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "chore: update deps" accurately and succinctly captures the PR's primary change — multiple dependency version bumps across various package.json files as shown in the raw_summary. It is concise, follows conventional commit style, and directly relates to the changeset. Adding a narrow scope (for example indicating affected templates/plugins) could provide slightly more context but is not required.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/update-deps

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.


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.

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.

Actionable comments posted: 1

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 15c2d2e and 6f2dbb9.

⛔ Files ignored due to path filters (1)
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing (7)
  • packages/core/template/UI/ano/package.json (1 hunks)
  • packages/core/template/UI/nut/package.json (1 hunks)
  • packages/core/template/config/lint/package.json (1 hunks)
  • packages/core/template/config/typescript/package.json (1 hunks)
  • packages/core/template/plugin/import/package.json (1 hunks)
  • packages/core/template/plugin/pages/package.json (1 hunks)
  • packages/core/template/plugin/root/package.json (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (20)
  • GitHub Check: test (-t vitesse, 20.x, windows-latest)
  • GitHub Check: test (--ts, -e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p ...
  • GitHub Check: test (--ts, -e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p ...
  • GitHub Check: test (--ts, -e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p ...
  • GitHub Check: test (--ts, -e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p ...
  • GitHub Check: test (--ts, -e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p ...
  • GitHub Check: test (--ts, -e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p ...
  • GitHub Check: test (-e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p middle...
  • GitHub Check: test (-e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p middle...
  • GitHub Check: test (-e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p middle...
  • GitHub Check: test (--ts, -e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p ...
  • GitHub Check: test (-e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p middle...
  • GitHub Check: test (-e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p middle...
  • GitHub Check: test (-e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p middle...
  • GitHub Check: test (-e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p middle...
  • GitHub Check: test (-e, -p import -p pages -p layouts -p manifest -p filePlatform -p platformModifier -p middle...
  • GitHub Check: stable - aarch64-pc-windows-msvc - node@20
  • GitHub Check: stable - aarch64-apple-darwin - node@20
  • GitHub Check: stable - i686-pc-windows-msvc - node@20
  • GitHub Check: stable - x86_64-pc-windows-msvc - node@20
🔇 Additional comments (6)
packages/core/template/UI/ano/package.json (1)

6-6: LGTM: safe icon metadata bump.

No impact expected on runtime or build.

packages/core/template/config/lint/package.json (1)

8-8: ESLint 9.x bump — confirm Node engine & flat-config compatibility

npm shows eslint@^9.36.0 requires node "^18.18.0 || ^20.9.0 || >=21.1.0". Ensure packages/core/template/config/lint/package.json (and template engines/CI Node versions) meet that range and that any flat config remains valid; approve after confirmation.

packages/core/template/UI/nut/package.json (1)

3-3: Check NutUI peer deps & release notes.
packages/core/template/UI/nut/package.json (line 3) -> "nutui-uniapp": "^1.9.1".
Peer deps for nutui-uniapp@^1.9.1: vue >=3.2.0; @uni-helper/uni-app-types ^1.0.0-alpha.6.
Ensure the project’s Vue/uni-app versions satisfy these and review NutUI changelog/release notes for any CSS-variable changes/migration steps.

packages/core/template/plugin/root/package.json (1)

3-3: Validate @uni-ku/root requirements

  • npm view shows @uni-ku/root peerDependency: vite ^5.0.0 and no "engines" returned — check packages/core/template/plugin/root/package.json (line 3).
  • Action: ensure the template/project uses Vite ≥5 or pin to a compatible @uni-ku/root; review @uni-ku/root v1.4.0 changelog for breaking/minor-bump changes and confirm Node version expectations (add/update "engines" or docs if needed).
packages/core/template/plugin/pages/package.json (1)

3-3: Ensure Vite compatibility — plugin requires Vite ^5.0.0

File: packages/core/template/plugin/pages/package.json (lines 3-3)

"@uni-helper/vite-plugin-uni-pages": "^0.3.15"

npm shows peerDependencies: {"vite":"^5.0.0"}. Verify the repository's Vite version (and Uni runtime) is compatible with Vite ^5.0.0; if not, upgrade Vite or pin a plugin version that matches your Vite/Uni stack.

packages/core/template/plugin/import/package.json (1)

3-3: Verify peerDependencies for @uni-helper/vite-plugin-uni-components (^0.2.1)
npm view returned no output — unable to confirm peerDependencies/engines for ^0.2.1. Run locally: npm view @uni-helper/vite-plugin-uni-components@^0.2.1 peerDependencies engines --json or check the package's npm/GitHub page; ensure the plugin's peer deps are compatible with the project's Vite/Uni versions (packages/core/template/plugin/import/package.json, line 3).

},
"devDependencies": {
"@types/node": "^24.5.1",
"@types/node": "^24.5.2",
Copy link

Choose a reason for hiding this comment

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

🛠️ Refactor suggestion

🧩 Analysis chain

Align Node type defs with target runtime.

Make sure template “engines.node” (if any) matches @types/node major.


🏁 Script executed:

#!/bin/bash
# Inspect declared Node engines across templates (if present)
fd -t f package.json | xargs -I{} sh -c 'jq -r "[input_filename, (.engines.node // \"\")]|@tsv" < "{}"' | awk -F'\t' '$2!=""'

Length of output: 84


🏁 Script executed:

#!/bin/bash
set -euo pipefail
file='packages/core/template/config/typescript/package.json'

echo "CHECK: specific template package.json"
if [[ -f "$file" ]]; then
  engine=$(jq -r '.engines.node // empty' "$file")
  typesnode=$(jq -r '.devDependencies["@types/node"] // .dependencies["@types/node"] // empty' "$file")
  printf '%s\tengines:%s\t@types/node:%s\n' "$file" "${engine:-<NO_ENGINES>}" "${typesnode:-<NO_TYPES_NODE>}"
else
  echo "MISSING_FILE: $file"
fi

echo
echo "SCAN: all package.json files for engines.node and @types/node"
find . -type f -name package.json -print0 | while IFS= read -r -d '' f; do
  engine=$(jq -r '.engines.node // empty' "$f")
  typesnode=$(jq -r '.devDependencies["@types/node"] // .dependencies["@types/node"] // empty' "$f")
  if [[ -n "$engine" || -n "$typesnode" ]]; then
    printf '%s\tengines:%s\t@types/node:%s\n' "$f" "${engine:-<NO_ENGINES>}" "${typesnode:-<NO_TYPES_NODE>}"
  fi
done

Length of output: 862


Align Node engine with @types/node major

  • packages/core/template/config/typescript/package.json declares "@types/node": "^24.5.2" but has no "engines.node".
  • Repo root ./package.json uses "@types/node": "^18.19.50" and several gui packages set "engines": ">= 10" — Node targets are inconsistent.
  • Action: pick the intended Node runtime and either add/update "engines.node" in the template to match @types/node major 24, or change the template's @types/node to the repository's chosen Node major.

@FliPPeDround FliPPeDround merged commit 5ce6b7b into main Sep 22, 2025
46 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Oct 6, 2025
@coderabbitai coderabbitai bot mentioned this pull request Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants