refactor: Bump yaml from 2.8.3 to 2.9.0 - #10625
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. Our CI and AI review are safeguards, not development tools. If many issues are flagged, rethink your development approach. Invest more effort in planning and design rather than using review cycles to fix low-quality code. |
|
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)
Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. 📝 WalkthroughWalkthroughThe pull request upgrades the ChangesYAML dependency upgrade
Estimated code review effort: 1 (Trivial) | ~5 minutes Merge Risk: ⚪ Minimal · up to This PR only updates a development dependency and its lockfile entry, without changing published runtime behavior or the public API; no actionable merge-blocking risk remains beyond normal checks and review. Suggested reviewers: 🚥 Pre-merge checks | ✅ 7✅ Passed checks (7 passed)
Full details: Linked Issues checkExplanation The changes satisfy issue Full details: Docstring CoverageExplanation No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (2 skipped: 2 unsupported.) Full details: Security CheckExplanation PASS — The exact PR commit changes only Full details: Engage In Review FeedbackExplanation No review feedback required engagement. The checked PR is ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## alpha #10625 +/- ##
==========================================
+ Coverage 93.78% 93.79% +0.01%
==========================================
Files 192 192
Lines 16834 16834
Branches 248 248
==========================================
+ Hits 15788 15790 +2
+ Misses 1025 1023 -2
Partials 21 21 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
c66fb7e to
68476e2
Compare
|
@coderabbitai review |
|
|
@coderabbitai full review |
✅ Action performedFull review finished. |
Pull Request
Issue
Closes #10597
This PR replaces the original Dependabot pull request #10597. Dependabot branches live in the upstream repository and cannot be modified directly, so the update is re-created here from a fork to allow the branch to be rebased and re-run against CI.
Approach
Bumps the
yamldevelopment dependency from2.8.3to2.9.0.yamlis adevDependencyonly. It is used by the CI tooling inci/CiVersionCheck.js, which reads the GitHub Actions workflow YAML files and parses them (yaml.parse) to verify that the Node.js, MongoDB and PostgreSQL versions declared in the CI matrix stay in sync with the versions supported by Parse Server. It is not part of the published package runtime and has no effect on Parse Server's public API.Upstream changes included in this bump (
2.8.4and2.9.0):2.9.0— fix: avoid callingArray.prototype.push.apply()with a large source array.2.9.0— fix(lexer): avoid recursive calls that may exhaust the call stack.2.9.0— documentation change only: the claim thatparseDocument()andparseAllDocuments()"never throw" has been removed; upstream will from now on treat call-stack-exhaustion errors triggered by malicious input as ordinary bugs rather than security vulnerabilities. This is a policy and docs change upstream, not an API change.2.8.4— fix: disable alias resolution withmaxAliasCount: 0; handle invalid unicode escapes; applyminFractionDigitsonly to decimal strings.No breaking changes are documented for this range, and none of the fixed behaviours are relied upon by the CI version check.
Scope of the diff: only
package.jsonandpackage-lock.jsonare touched. The lockfile churn is confined to theyamlsubtree (the version,resolvedURL andintegrityhash of theyamlentries); no other dependency, transitive tree or lockfile section is modified.Tasks
2.8.3and2.9.0; none apply to this bump.No new tests and no documentation changes are required: this is a version-only bump of a development dependency and contains no changes to Parse Server source code, behaviour or public API.
Summary by CodeRabbit