chore(deps)(deps): bump js-yaml from 4.1.1 to 5.2.0#76
Open
dependabot[bot] wants to merge 1 commit into
Open
Conversation
Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 4.1.1 to 5.2.0. - [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md) - [Commits](nodeca/js-yaml@4.1.1...5.2.0) --- 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>
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 |
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.0.
Changelog
Sourced from js-yaml's changelog.
... (truncated)
Commits
c28ed5e5.2.0 released125cd5aAddmaxAliasesoption3105455ReplacemaxMergeSeqLengthoption withmaxTotalMergeKeys(more robust)39d00d6numbers: Drop boxed numbers support, simplify .identify() checks, clarify rou...eb5cb5bfix: round-trip integers that stringify in exponential notation (#771)89024c4Update migration info, close #770f1e45cd5.1.0 released53b22beFix constructor coveragea1eaa2bFix quote style options and restore forceQuotes0532e7dAdd finalizers for immutable collection tagsDependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)Note
Medium Risk
Major-version dependency bump with no code changes in the diff; parsing/serialization behavior may change for YAML usage in api/core without compile-time signal from this PR alone.
Overview
Upgrades js-yaml from
^4.1.1to^5.2.0in the root workspace,apps/api, andpackages/core, with matching updates inpnpm-lock.yaml(direct deps resolve to 5.2.0; some transitive Astro-related packages move tojs-yaml@4.3.0).There are no application source changes in this PR—only dependency manifests and the lockfile. js-yaml v5 is a major release (TypeScript rewrite, schema/API changes per upstream migration notes), so runtime YAML
load/dumpbehavior could differ anywhere the library is used once installed.Reviewed by Cursor Bugbot for commit e074cea. Bugbot is set up for automated code reviews on this repo. Configure here.