Skip to content

Conversation

@q1blue
Copy link
Contributor

@q1blue q1blue commented Oct 28, 2025

snyk-top-banner

Snyk has created this PR to fix 2 vulnerabilities in the npm dependencies of this project.

Snyk changed the following file(s):

  • package.json
  • package-lock.json

Vulnerabilities that will be fixed with an upgrade:

Issue Score
high severity Regular Expression Denial of Service (ReDoS)
SNYK-JS-TRIM-1017038
  169  
medium severity Regular Expression Denial of Service (ReDoS)
SNYK-JS-OCTOKITREQUEST-8730853
  67  

Important

  • Check the changes in this PR to ensure they won't cause issues with your project.
  • Max score is 1000. Note that the real score may have changed since the PR was raised.
  • This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.

For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic


Learn how to fix vulnerabilities with free interactive lessons:

🦉 Regular Expression Denial of Service (ReDoS)


EntelligenceAI PR Summary

This PR updates two major dependencies: @docusaurus/preset-classic from v2.4.1 to v3.0.0 and octokit from v2.0.14 to v3.0.0. These updates include breaking changes that require additional compatibility updates and security fixes.

@socket-security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updated@​docusaurus/​preset-classic@​2.4.1 ⏵ 3.0.0991007099 +1100

View full report

@socket-security
Copy link

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn High
caniuse-lite@1.0.30001751 has a License Policy Violation.

License: CC-BY-4.0 (npm metadata)

License: CC-BY-4.0 (package/LICENSE)

License: CC-BY-4.0 (package/package.json)

From: package-lock.jsonnpm/@svgr/webpack@6.5.1npm/@docusaurus/core@2.4.1npm/@docusaurus/plugin-content-docs@2.4.1npm/@docusaurus/preset-classic@3.0.0npm/caniuse-lite@1.0.30001751

ℹ Read more on: This package | This alert | What is a license policy violation?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Find a package that does not violate your license policy or adjust your policy to allow this package's license.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/caniuse-lite@1.0.30001751. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@guardrails
Copy link

guardrails bot commented Oct 28, 2025

⚠️ We detected 9 security issues in this pull request:

Vulnerable Libraries (9)
Severity Details
High pkg:npm/@docusaurus/core@2.4.1 upgrade to: > 2.4.1
High pkg:npm/@docusaurus/plugin-google-gtag@2.4.1 upgrade to: > 2.4.1
High pkg:npm/@docusaurus/plugin-content-docs@2.4.1 upgrade to: > 2.4.1
High pkg:npm/eslint@8.33.0 upgrade to: > 8.33.0
High pkg:npm/@svgr/webpack@6.5.1 upgrade to: > 6.5.1
Medium pkg:npm/prismjs@1.29.0 (t) - no patch available
High pkg:npm/octokit@3.0.0 upgrade to: 11.1.2,12.0.3,14.0.2,3.1.2,12.3.3,9.26.3,10.9.2
High pkg:npm/@docusaurus/plugin-ideal-image@2.4.1 upgrade to: > 2.4.1
High pkg:npm/@docusaurus/preset-classic@3.0.0 upgrade to: > 3.0.0

More info on how to fix Vulnerable Libraries in JavaScript.


👉 Go to the dashboard for detailed results.

📥 Happy? Share your feedback with us.

@cloudflare-workers-and-pages
Copy link

cloudflare-workers-and-pages bot commented Oct 28, 2025

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
crystal-alchemist-blue-block-1596 5625377 Oct 28 2025, 01:53 AM

@entelligence-ai-pr-reviews
Copy link

📝 Walkthrough

This PR performs major version upgrades for two critical dependencies: @docusaurus/preset-classic (v2.4.1 → v3.0.0) and octokit (v2.0.14 → v3.0.0). These updates involve significant breaking changes that require careful migration steps and additional compatibility updates.

The PR includes a complete dependency tree refresh with numerous transitive dependency updates, as evidenced by the massive changes in the lock file (49,204 lines changed). However, several critical issues need to be addressed before merging, including a security vulnerability in octokit (CVE-2023-50728), Node.js runtime compatibility requirements, and necessary complementary dependency updates.


📊 Changes

File Change
package.json Updated @docusaurus/preset-classic from ^2.4.1 to ^3.0.0
package.json Updated octokit from ^2.0.14 to ^3.0.0
package-lock.json Massive regeneration with 49,204 lines changed

🔒 Security Highlights

  • 🚨 CVE-2023-50728: octokit versions < 3.1.2 contain a DoS vulnerability allowing remote attackers to crash the application
  • 🔍 Hundreds of new transitive dependencies introduced without security review
  • 🛡️ Recommendation to update octokit to 3.1.2+ immediately and run npm audit
  • 🔐 Implement dependency scanning in CI/CD pipeline

🔒 Security Analysis

  • Vulnerabilities: 0
  • Bugs: 0
  • Code Smells: 0
  • Security Hotspots: 0

Caution

5 comments are outside the diff range and can't be posted inline due to platform limitations.

⚠️ View Outside Diff Range Comments (5)
🔴 Critical High Priority  ·  4 issues
package.json  ·  4 comments

1. Lines 14-15 · Security

octokit v3.0.0 contains a critical security vulnerability (CVE-2023-50728) that allows remote attackers to perform DoS attacks

Committable Code Suggestion
‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.
"octokit": "^3.1.2",

2. Lines 14 · Correctness

@docusaurus/preset-classic v3.0.0 requires Node.js 18+ but no runtime version validation is performed


3. Lines 15 · Correctness

octokit v3.0.0 requires Node.js 18+, dropping support for Node.js 14 and 16


4. Lines 13-14 · Correctness

Missing migration steps for Docusaurus v2 to v3 (MDX syntax validation, Prism React Renderer config updates)

🟡 Medium Medium Priority  ·  1 issue
package.json  ·  1 comment

1. Lines 15 · Correctness

@mdx-js/react ^1.6.22 is incompatible with Docusaurus v3, needs upgrade to ^3.0.0

Committable Code Suggestion
‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.
"@mdx-js/react": "^3.0.0",

▶️AI Code Reviews for VS Code, Cursor, Windsurf
Install the extension

Note for Windsurf Please change the default marketplace provider to the following in the windsurf settings:

Marketplace Extension Gallery Service URL: https://marketplace.visualstudio.com/_apis/public/gallery

Marketplace Gallery Item URL: https://marketplace.visualstudio.com/items

Entelligence.ai can learn from your feedback. Simply add 👍 / 👎 emojis to teach it your preferences. More shortcuts below

Emoji Descriptions:

  • ⚠️ Potential Issue - May require further investigation.
  • 🔒 Security Vulnerability - Fix to ensure system safety.
  • 💻 Code Improvement - Suggestions to enhance code quality.
  • 🔨 Refactor Suggestion - Recommendations for restructuring code.
  • ℹ️ Others - General comments and information.

Interact with the Bot:

  • Send a message or request using the format:
    @entelligenceai + *your message*
Example: @entelligenceai Can you suggest improvements for this code?
  • Help the Bot learn by providing feedback on its responses.
    @entelligenceai + *feedback*
Example: @entelligenceai Do not comment on `save_auth` function !

Also you can trigger various commands with the bot by doing
@entelligenceai command

The current supported commands are

  1. config - shows the current config
  2. retrigger_review - retriggers the review

More commands to be added soon.

Comment on lines 22 to 23
"@mdx-js/react": "^1.6.22",
"@svgr/webpack": "^6.5.1",

Choose a reason for hiding this comment

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

Correctness: React ^17.0.1 is incompatible with Docusaurus v3, which expects React 18

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"@mdx-js/react": "^1.6.22",
"@svgr/webpack": "^6.5.1",
"react": "^18.2.0",
"react-dom": "^18.2.0",

"@docusaurus/plugin-google-gtag": "^2.4.1",
"@docusaurus/plugin-ideal-image": "^2.4.1",
"@docusaurus/preset-classic": "^2.4.1",
"@docusaurus/preset-classic": "^3.0.0",

Choose a reason for hiding this comment

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

Correctness: prism-react-renderer ^1.2.1 is incompatible with Docusaurus v3, needs update to ^2.1.0

📝 Committable Code Suggestion

‼️ Ensure you review the code suggestion before committing it to the branch. Make sure it replaces the highlighted code, contains no missing lines, and has no issues with indentation.

Suggested change
"@docusaurus/preset-classic": "^3.0.0",
"prism-react-renderer": "^2.1.0",

Comment on lines 18 to 23
"@docusaurus/plugin-content-docs": "^2.4.1",
"@docusaurus/plugin-google-gtag": "^2.4.1",
"@docusaurus/plugin-ideal-image": "^2.4.1",
"@docusaurus/preset-classic": "^2.4.1",
"@docusaurus/preset-classic": "^3.0.0",
"@mdx-js/react": "^1.6.22",
"@svgr/webpack": "^6.5.1",

Choose a reason for hiding this comment

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

Correctness: Untested breaking API changes in both dependencies could fail at runtime

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.

3 participants