Skip to content

chore: removing unused dependencies#1443

Merged
neelneelneel merged 1 commit intodevfrom
1438-remove-extra-dependencies
Jul 7, 2025
Merged

chore: removing unused dependencies#1443
neelneelneel merged 1 commit intodevfrom
1438-remove-extra-dependencies

Conversation

@neelneelneel
Copy link
Copy Markdown
Contributor

Description

Changes Made

How to Test

  1. Steps to reproduce/test the behavior
  2. Expected outcomes

Notes

@neelneelneel neelneelneel requested a review from a team as a code owner July 7, 2025 21:10
@neelneelneel neelneelneel linked an issue Jul 7, 2025 that may be closed by this pull request
1 task
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 7, 2025

@CodiumAI-Agent /describe

@neelneelneel neelneelneel merged commit d7a79d8 into dev Jul 7, 2025
4 checks passed
@neelneelneel neelneelneel deleted the 1438-remove-extra-dependencies branch July 7, 2025 21:10
@github-actions
Copy link
Copy Markdown

github-actions bot commented Jul 7, 2025

@CodiumAI-Agent /update_changelog

@QodoAI-Agent
Copy link
Copy Markdown

Title

chore: removing unused dependencies


User description

Description

Changes Made

How to Test

  1. Steps to reproduce/test the behavior
  2. Expected outcomes

Notes


PR Type

Other


Description

  • Remove unused dependencies across packages

  • Add missing deps: @mui/x-date-pickers, dayjs, mermaid

  • Remove obsolete Rollup and tooling plugins

  • Clean up root and lib package.json configs


Changes walkthrough 📝

Relevant files
Configuration changes
6 files
package.json
Update renderer deps and remove plugins                                   
+3/-4     
package.json
Remove unused SDK dependencies                                                     
+0/-2     
package.json
Prune UI package cross-env dep                                                     
+0/-1     
package.json
Clean up root dev and runtime deps                                             
+1/-17   
package.json
Drop unused CLI dependencies                                                         
+0/-2     
package.json
Remove unused client dependencies                                               
+0/-3     
Additional files
1 files
pnpm-lock.yaml +122/-3137

Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link
    Copy Markdown

    github-actions bot commented Jul 7, 2025

    @CodiumAI-Agent /review

    @QodoAI-Agent
    Copy link
    Copy Markdown

    Changelog updates: 🔄

    2025-07-07 *

    Changed

    • Removed unused dependencies across multiple packages (renderer, sdk, ui, client, cli)

    to commit the new content to the CHANGELOG.md file, please type:
    '/update_changelog --pr_update_changelog.push_changelog_changes=true'

    @github-actions
    Copy link
    Copy Markdown

    github-actions bot commented Jul 7, 2025

    @CodiumAI-Agent /improve

    @QodoAI-Agent
    Copy link
    Copy Markdown

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    MUI Version Compatibility

    Ensure that the added @mui/x-date-pickers version 6.8.0 is compatible with the existing MUI core and @mui/x-data-grid versions to avoid mismatched peer dependency issues.

    "@mui/x-date-pickers": "6.8.0",
    Bundling Configuration

    Verify that the newly added mermaid dependency is properly handled by the Rollup configuration or requires additional plugins to bundle correctly.

    "mermaid": "^10.9.1",
    Duplicate Dependency

    The dayjs dependency appears in multiple packages; consider hoisting or ensuring consistent versions to prevent multiple instances in the final bundle.

    "dayjs": "1.11.8",

    @QodoAI-Agent
    Copy link
    Copy Markdown

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    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.

    Remove extra dependencies

    2 participants