Skip to content

Conversation

@mssabr01
Copy link

@mssabr01 mssabr01 commented Sep 5, 2025

User description

updated dependencies to remove vulns. Removed NP, seems like our scripts were not using it


PR Type

enhancement, dependencies


Description

  • Updated several development dependencies to newer versions

  • Removed unused np package from devDependencies

  • Addressed potential security vulnerabilities via dependency updates

  • Ensured compatibility with Node.js 20.19.3


Changes walkthrough 📝

Relevant files
Dependencies
package.json
Update and clean up devDependencies in package.json           

package.json

  • Removed np from devDependencies as it was unused
  • Upgraded typescript from 4.4.4 to 4.9.4
  • Upgraded ts-jest from 27.0.5 to 29.4.1
  • Upgraded @types/jest from 27.0.1 to 29.5.14
  • Upgraded jest from 27.1.1 to 29.7.0
  • Maintained Node.js engine requirement at 20.19.3
  • +4/-5     

    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

    github-actions bot commented Sep 5, 2025

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

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

    Dependency Update Validation

    Ensure that all updated dependencies, especially major version bumps (e.g., jest, ts-jest, @types/jest, typescript), are compatible with the existing codebase and do not introduce breaking changes or regressions.

    "typescript": "4.9.4",
    "gts": "3.1.1",
    "eslint": "8.31.0",
    "@typescript-eslint/eslint-plugin": "5.48.0",
    "@typescript-eslint/parser": "5.48.0",
    "eslint-config-prettier": "8.6.0",
    "eslint-plugin-no-unsanitized": "4.0.2",
    "eslint-plugin-security": "1.5.0",
    "eslint-plugin-xss": "0.1.12",
    "typescript-eslint": "0.0.1-alpha.0",
    "ts-jest": "29.4.1",
    "@types/jest": "29.5.14",
    "jest": "29.7.0",
    Linear Issue Linkage

    Confirm that the PR is properly linked to a Linear issue, as required by the workflow. The branch name includes 'SHARD-2643', but ensure this is reflected in the PR description, title, or commit messages as well.

    {

    @mssabr01 mssabr01 changed the title updated dependencies to remove vulns. Removed NP, seems like our scri… SHARD-2643 updated dependencies Sep 5, 2025
    @github-actions
    Copy link

    github-actions bot commented Sep 5, 2025

    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

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    2 participants