Skip to content

update cardano test command#93

Merged
hadi-saleh14 merged 1 commit intomainfrom
update-cardano-docs
Dec 30, 2025
Merged

update cardano test command#93
hadi-saleh14 merged 1 commit intomainfrom
update-cardano-docs

Conversation

@hadi-saleh14
Copy link
Contributor

@hadi-saleh14 hadi-saleh14 commented Dec 30, 2025

Note

Docs

  • Updates content/docs/zkfetch/cardano.md Testing section to use a single npm run test command instead of multiple test scripts

Dependencies

  • Bumps next to 15.4.8 and updates related @next/* SWC binaries in package-lock.json
  • Normalizes lockfile metadata (many packages flipped from devOptional to dev)

Written by Cursor Bugbot for commit 9390dad. This will update automatically on new commits. Configure here.

Summary by CodeRabbit

Release Notes

  • Documentation
    • Simplified test command examples in Cardano documentation to use a single generic npm run test command for easier reference.

✏️ Tip: You can customize this high-level summary in your review settings.

@vercel
Copy link
Contributor

vercel bot commented Dec 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
docs Ready Ready Preview, Comment Dec 30, 2025 5:12pm

@coderabbitai
Copy link

coderabbitai bot commented Dec 30, 2025

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The PR simplifies the Cardano zkFetch documentation by consolidating test command examples from multiple granular commands (test:all, test:deno, test:node) to a single unified npm run test command in quickstart and testing sections.

Changes

Cohort / File(s) Summary
Documentation - Test Command Consolidation
content/docs/zkfetch/cardano.md
Updated test command references from granular npm run test:all, npm run test:deno, and npm run test:node to unified npm run test in quickstart and testing sections.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • add cardano zkfetch quickstart #63 — Prior PR that introduced the granular test commands (test:all, test:deno, test:node) that this PR consolidates into a single unified command.

Suggested reviewers

  • TheLedgerOfJoudi

Poem

🐰 Hop, skip, and a test command made clear,
One script to rule them all, we cheer!
No more granular paths to explore,
Just npm run test forevermore!


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 98b864d and 9390dad.

⛔ Files ignored due to path filters (1)
  • package-lock.json is excluded by !**/package-lock.json
📒 Files selected for processing (1)
  • content/docs/zkfetch/cardano.md

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.

@hadi-saleh14 hadi-saleh14 merged commit f957a7c into main Dec 30, 2025
3 of 4 checks passed
Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

"version": "15.4.8",
"resolved": "https://registry.npmjs.org/next/-/next-15.4.8.tgz",
"integrity": "sha512-jwOXTz/bo0Pvlf20FSb6VXVeWRssA2vbvq9SdrOPEg9x8E1B27C2rQtvriAn600o9hH61kjrVRexEffv3JybuA==",
"deprecated": "This version has a security vulnerability. Please upgrade to a patched version. See https://nextjs.org/blog/security-update-2025-12-11 for more details.",
Copy link

Choose a reason for hiding this comment

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

Next.js 15.4.8 has known security vulnerabilities

The commit updates to next@15.4.8 which is marked as deprecated due to security vulnerabilities. The package-lock.json itself contains the deprecation notice referencing the December 11, 2025 security update. This version is affected by CVE-2025-66478 (React2Shell) and additional vulnerabilities CVE-2025-55184 and CVE-2025-55183. The current secure version is 16.1.1 according to npm security advisories.

Fix in Cursor Fix in Web

@hadi-saleh14 hadi-saleh14 deleted the update-cardano-docs branch January 12, 2026 11:41
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