refactor: Bump marked from 17.0.3 to 17.0.5#3306
refactor: Bump marked from 17.0.3 to 17.0.5#3306mtrezza merged 1 commit intoparse-community:alphafrom
Conversation
|
🚀 Thanks for opening this pull request! We appreciate your effort in improving the project. Please let us know once your pull request is ready for review. Tip
Note Please respond to review comments from AI agents just like you would to comments from a human reviewer. Let the reviewer resolve their own comments, unless they have reviewed and accepted your commit, or agreed with your explanation for why the feedback was incorrect. Caution Pull requests must be written using an AI agent with human supervision. Pull requests written entirely by a human will likely be rejected, because of lower code quality, higher review effort and the higher risk of introducing bugs. Please note that AI review comments on this pull request alone do not satisfy this requirement. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (2)
📝 WalkthroughWalkthroughBumped the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5 | ❌ 2❌ Failed checks (2 inconclusive)
✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 Checkov (3.2.510)package.json2026-04-01 15:24:46,576 [MainThread ] [ERROR] Template file not found: package.json ... [truncated 2547 characters] ... [MainThread ] [WARNI] Secret scanning: could not process file package.json 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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@package-lock.json`:
- Line 90: The lockfile was corrupted during the marked bump (entry "marked":
"17.0.5") and is missing required packages like `@codemirror/`*, `@lezer/`*,
pg-query-stream, pg-cursor, style-mod, and w3c-keyname; regenerate a clean
lockfile by deleting package-lock.json and node_modules and running a fresh
install (e.g., rm -rf node_modules package-lock.json && npm install) so the
package-lock reflects all transitive deps including the marked update.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: a93c57bf-7272-4e39-b425-74eba411946d
📒 Files selected for processing (2)
package-lock.jsonpackage.json
| "jest-environment-jsdom": "30.0.5", | ||
| "madge": "8.0.0", | ||
| "marked": "17.0.3", | ||
| "marked": "17.0.5", |
There was a problem hiding this comment.
Critical: Pipeline failure indicates corrupted lockfile.
While the marked version update to 17.0.5 is correct and addresses the security objectives, the pipeline is failing because this lockfile is missing required packages:
@codemirror/* packages (language, state, view)@lezer/* packages (common, highlight, lr)- PostgreSQL packages (pg-query-stream, pg-cursor)
- style-mod, w3c-keyname
These unrelated packages should not have been removed during a marked version bump. The lockfile needs to be regenerated properly.
🔧 Fix the lockfile
To fix this issue, regenerate the lockfile cleanly:
# Remove existing lockfile and node_modules
rm -rf node_modules package-lock.json
# Reinstall to generate a clean lockfile
npm installThis will ensure all required dependencies (including the marked update) are properly reflected in the lockfile.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@package-lock.json` at line 90, The lockfile was corrupted during the marked
bump (entry "marked": "17.0.5") and is missing required packages like
`@codemirror/`*, `@lezer/`*, pg-query-stream, pg-cursor, style-mod, and w3c-keyname;
regenerate a clean lockfile by deleting package-lock.json and node_modules and
running a fresh install (e.g., rm -rf node_modules package-lock.json && npm
install) so the package-lock reflects all transitive deps including the marked
update.
There was a problem hiding this comment.
The lock file was regenerated from a clean baseline. All CI checks including NPM Lock File Version are now passing, confirming the lock file is correct.
There was a problem hiding this comment.
@mtrezza, thanks for confirming! Glad to hear the lockfile was regenerated cleanly and all CI checks — including the NPM Lock File Version check — are now passing. The concern is resolved. ✅
🐇 🎉
🧠 Learnings used
Learnt from: mtrezza
Repo: parse-community/parse-dashboard PR: 3252
File: changelogs/CHANGELOG_alpha.md:0-0
Timestamp: 2026-03-06T18:23:39.621Z
Learning: In the parse-community/parse-dashboard repository, changelog entries (e.g. changes to `changelogs/CHANGELOG_alpha.md` or `CHANGELOG.md`) are NOT allowed in pull requests. Changelogs are managed automatically by semantic-release. Any PR that includes manual changelog modifications must be rejected and the contributor must be asked to remove the changelog changes.
If you found this review helpful, would you consider giving us a shout-out on X?
Thank you for using CodeRabbit!
e521928 to
efbf566
Compare
Lock file was regenerated from clean baseline. All CI checks pass.
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Bump
markedfrom 17.0.3 to 17.0.5 (patch).Changes (bug fixes only):
Closes #3300
Summary by CodeRabbit