Skip to content

Conversation

@IsaacYangSLA
Copy link
Collaborator

Description

Update the npm dependencies in examples/tutorials/self-paced-training

Types of changes

  • Non-breaking change (fix or new feature that would not break existing functionality).
  • Breaking change (fix or new feature that would cause existing functionality to change).
  • New tests added to cover the changes.
  • Quick tests passed locally by running ./runtest.sh.
  • In-line docstrings updated.
  • Documentation updated.

@IsaacYangSLA
Copy link
Collaborator Author

/build

@greptile-apps
Copy link
Contributor

greptile-apps bot commented Dec 9, 2025

Greptile Overview

Greptile Summary

Updates npm dependencies in the medical devices analytics dashboard example to more specific versions and fixes vendor package path references from ./vendor to vendor.

Key Changes:

  • Pinned React ecosystem dependencies to specific patch versions (react@^18.3.1, react-dom@^18.3.1)
  • Updated react-day-picker from ^9.4.2 to ^9.12.0
  • Updated recharts from ^2.14.1 to ^2.15.4
  • Pinned TypeScript type definitions to specific versions (@types/node@^20.19.26, @types/react@^18.3.27, etc.)
  • Updated tailwindcss from ^3.4.1 to ^3.4.18
  • Corrected all @kui-react/* and @kui-shared/* local file paths by removing the ./ prefix
  • Regenerated pnpm-lock.yaml with updated dependency resolution

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk - it's a straightforward dependency update with path corrections
  • The changes are low-risk maintenance updates that pin dependencies to specific versions and fix path references. All updates are minor/patch version bumps within compatible ranges. The vendor packages exist in the repository, and the lockfile was properly regenerated
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
examples/tutorials/self-paced-training/part-5_federated_learning_applications_in_industries/chapter-11_federated_learning_in_healthcare_lifescience/11.3_medical_devices/code/visualization/frontend/analytics-dashboard/package.json 5/5 Updated dependency versions to specific patch versions and changed file paths from ./vendor to vendor for local packages
examples/tutorials/self-paced-training/part-5_federated_learning_applications_in_industries/chapter-11_federated_learning_in_healthcare_lifescience/11.3_medical_devices/code/visualization/frontend/analytics-dashboard/pnpm-lock.yaml 5/5 Regenerated lockfile reflecting updated dependency versions and resolved transitive dependencies

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant PM as Package Manager (pnpm)
    participant PJ as package.json
    participant Lock as pnpm-lock.yaml
    participant Vendor as Local Vendor Packages
    participant NPM as NPM Registry

    Dev->>PJ: Update dependency versions
    Note over PJ: react: ^18 → ^18.3.1<br/>react-day-picker: ^9.4.2 → ^9.12.0<br/>recharts: ^2.14.1 → ^2.15.4<br/>@types/* versions pinned<br/>tailwindcss: ^3.4.1 → ^3.4.18
    Dev->>PJ: Fix vendor paths (./vendor → vendor)
    Dev->>PM: Run pnpm install
    PM->>Vendor: Resolve local file: dependencies
    Vendor-->>PM: Return local packages
    PM->>NPM: Fetch updated packages
    NPM-->>PM: Return [email protected], [email protected]
    NPM-->>PM: Return [email protected]
    NPM-->>PM: Return [email protected]
    NPM-->>PM: Return devDependencies updates
    PM->>Lock: Regenerate lockfile
    Note over Lock: Update dependency tree<br/>Resolve transitive deps<br/>984 additions, 1091 deletions
    Lock-->>PM: Lockfile updated
    PM-->>Dev: Installation complete
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

Copilot AI review requested due to automatic review settings December 10, 2025 22:02
@IsaacYangSLA IsaacYangSLA merged commit 2ef4034 into main Dec 10, 2025
40 checks passed
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +12 to +26
"@kui-react/app-bar": "file:vendor/npm-packages/kui-react-app-bar-4.131.6.tgz",
"@kui-react/button": "file:vendor/npm-packages/kui-react-button-4.131.6.tgz",
"@kui-react/card": "file:vendor/npm-packages/kui-react-card-4.131.6.tgz",
"@kui-react/checkbox": "file:vendor/npm-packages/kui-react-checkbox-4.131.6.tgz",
"@kui-react/datepicker": "file:vendor/npm-packages/kui-react-datepicker-4.131.6.tgz",
"@kui-react/flex": "file:vendor/npm-packages/kui-react-flex-4.131.6.tgz",
"@kui-react/horizontal-nav": "file:vendor/npm-packages/kui-react-horizontal-nav-4.131.6.tgz",
"@kui-react/label": "file:vendor/npm-packages/kui-react-label-4.131.6.tgz",
"@kui-react/radio": "file:vendor/npm-packages/kui-react-radio-4.131.6.tgz",
"@kui-react/select": "file:vendor/npm-packages/kui-react-select-4.131.6.tgz",
"@kui-react/table": "file:vendor/npm-packages/kui-react-table-4.131.6.tgz",
"@kui-react/text": "file:vendor/npm-packages/kui-react-text-4.131.6.tgz",
"@kui-react/text-input": "file:vendor/npm-packages/kui-react-text-input-4.131.6.tgz",
"@kui-react/theme": "file:vendor/npm-packages/kui-react-theme-4.131.6.tgz",
"@kui-shared/styles": "file:vendor/npm-packages/kui-shared-styles-4.131.6.tgz",
Copy link

Copilot AI Dec 10, 2025

Choose a reason for hiding this comment

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

The local package paths have been changed from file:./vendor to file:vendor. While both paths may work, this change affects how npm/pnpm resolves these local dependencies. Ensure that the vendor directory is at the same level as package.json and that this path change has been tested in the build environment. The ./ prefix is more explicit about relative path resolution from the current directory.

Suggested change
"@kui-react/app-bar": "file:vendor/npm-packages/kui-react-app-bar-4.131.6.tgz",
"@kui-react/button": "file:vendor/npm-packages/kui-react-button-4.131.6.tgz",
"@kui-react/card": "file:vendor/npm-packages/kui-react-card-4.131.6.tgz",
"@kui-react/checkbox": "file:vendor/npm-packages/kui-react-checkbox-4.131.6.tgz",
"@kui-react/datepicker": "file:vendor/npm-packages/kui-react-datepicker-4.131.6.tgz",
"@kui-react/flex": "file:vendor/npm-packages/kui-react-flex-4.131.6.tgz",
"@kui-react/horizontal-nav": "file:vendor/npm-packages/kui-react-horizontal-nav-4.131.6.tgz",
"@kui-react/label": "file:vendor/npm-packages/kui-react-label-4.131.6.tgz",
"@kui-react/radio": "file:vendor/npm-packages/kui-react-radio-4.131.6.tgz",
"@kui-react/select": "file:vendor/npm-packages/kui-react-select-4.131.6.tgz",
"@kui-react/table": "file:vendor/npm-packages/kui-react-table-4.131.6.tgz",
"@kui-react/text": "file:vendor/npm-packages/kui-react-text-4.131.6.tgz",
"@kui-react/text-input": "file:vendor/npm-packages/kui-react-text-input-4.131.6.tgz",
"@kui-react/theme": "file:vendor/npm-packages/kui-react-theme-4.131.6.tgz",
"@kui-shared/styles": "file:vendor/npm-packages/kui-shared-styles-4.131.6.tgz",
"@kui-react/app-bar": "file:./vendor/npm-packages/kui-react-app-bar-4.131.6.tgz",
"@kui-react/button": "file:./vendor/npm-packages/kui-react-button-4.131.6.tgz",
"@kui-react/card": "file:./vendor/npm-packages/kui-react-card-4.131.6.tgz",
"@kui-react/checkbox": "file:./vendor/npm-packages/kui-react-checkbox-4.131.6.tgz",
"@kui-react/datepicker": "file:./vendor/npm-packages/kui-react-datepicker-4.131.6.tgz",
"@kui-react/flex": "file:./vendor/npm-packages/kui-react-flex-4.131.6.tgz",
"@kui-react/horizontal-nav": "file:./vendor/npm-packages/kui-react-horizontal-nav-4.131.6.tgz",
"@kui-react/label": "file:./vendor/npm-packages/kui-react-label-4.131.6.tgz",
"@kui-react/radio": "file:./vendor/npm-packages/kui-react-radio-4.131.6.tgz",
"@kui-react/select": "file:./vendor/npm-packages/kui-react-select-4.131.6.tgz",
"@kui-react/table": "file:./vendor/npm-packages/kui-react-table-4.131.6.tgz",
"@kui-react/text": "file:./vendor/npm-packages/kui-react-text-4.131.6.tgz",
"@kui-react/text-input": "file:./vendor/npm-packages/kui-react-text-input-4.131.6.tgz",
"@kui-react/theme": "file:./vendor/npm-packages/kui-react-theme-4.131.6.tgz",
"@kui-shared/styles": "file:./vendor/npm-packages/kui-shared-styles-4.131.6.tgz",

Copilot uses AI. Check for mistakes.
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

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.

2 participants