chore(deps)(deps): bump js-yaml from 4.1.1 to 5.2.1 - #76
Closed
dependabot[bot] wants to merge 1 commit into
Closed
Conversation
Contributor
Author
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
There was a problem hiding this comment.
No security findings for the changed files.
Reviewed changes:
package.json:27,apps/api/package.json:31, andpackages/core/package.json:24update directjs-yamldependencies from^4.1.1to^5.2.0.pnpm-lock.yamlupdates the corresponding lockfile entries fromjs-yaml@4.1.1tojs-yaml@5.2.0with the sameargparse@2.0.1dependency shape.
Threat surfaces checked against this diff:
- MCP endpoint auth: no MCP route or token/project-scoping code changed.
- Query execution sandboxing: no
execute_query, DuckDB, view-scoping, timeout, or result-limit code changed. - Admin auth / Better Auth: no session secret, CSRF, cookie, or auth route code changed.
- API input validation: no Hono route handlers, Zod schemas, MongoDB query construction, or DuckDB SQL construction changed.
- Environment secrets: no
.env*files or secret-handling/logging code changed. - Dependency exposure: ran
pnpm audit --prod. It does not report ajs-yamladvisory for the bumped version. The audit still reports existing transitive advisories for packages such asxlsx,@xmldom/xmldom,ws, andform-data, but the affected lockfile entries are present in both the base and head and are not introduced by this PR.
Sent by Cursor Automation: archmax Security Review
Docker image readydocker pull ghcr.io/archmaxai/archmax:pr-76 |
Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.1.1 to 5.2.1. - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@4.1.1...5.2.1) --- updated-dependencies: - dependency-name: js-yaml dependency-version: 5.2.0 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
force-pushed
the
dependabot/npm_and_yarn/js-yaml-5.2.0
branch
from
July 20, 2026 14:07
e074cea to
fbd8fd2
Compare
Contributor
Author
|
Superseded by #85. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.


Bumps js-yaml from 4.1.1 to 5.2.1.
Changelog
Sourced from js-yaml's changelog.
... (truncated)
Commits
ac16b425.2.1 released4a864e5Deps bump39f3211!!omap: addMapsupport and remove quadratic complexityff17f1eChangelog update8ed15f1deps bump1a562dcFix changelog linkc28ed5e5.2.0 released125cd5aAddmaxAliasesoption3105455ReplacemaxMergeSeqLengthoption withmaxTotalMergeKeys(more robust)39d00d6numbers: Drop boxed numbers support, simplify .identify() checks, clarify rou...Note
Medium Risk
Major-version YAML parse/dump library used on semantic-model and publish paths; typings stay on v4 and v5 includes breaking schema/quoting behavior despite security improvements.
Overview
Bumps
js-yamlfrom 4.1.1 to 5.2.1 in the repo root,apps/api, andpackages/core, with matchingpnpm-lock.yamlupdates (direct deps move to 5.2.1; some transitive copies stay on 4.3.0).There are no application code changes—only dependency versions. Call sites still use the default import with
yaml.load/yaml.dumpfor semantic-model and migration YAML (e.g.packages/corepublish/semantic-model services andapps/apimigrate scripts).@types/js-yamlremains at ^4.0.9.The upgrade is a major release (schema/quoting behavior and new loader limits such as
maxTotalMergeKeys), so behavior of parsed or emitted dataset YAML could shift if inputs hit edge cases; the stated motivation in the upstream changelog includes security fixes around merge/alias handling.Reviewed by Cursor Bugbot for commit fbd8fd2. Bugbot is set up for automated code reviews on this repo. Configure here.