Skip to content

fix: lock markdownlint to patched reviewed installs - #528

Merged
omkhar merged 2 commits into
mainfrom
dependabot/npm_and_yarn/tools/markdownlint/js-yaml-5.2.2
Jul 29, 2026
Merged

fix: lock markdownlint to patched reviewed installs#528
omkhar merged 2 commits into
mainfrom
dependabot/npm_and_yarn/tools/markdownlint/js-yaml-5.2.2

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • bump js-yaml to 5.2.2 and brace-expansion to 5.0.8 in the reviewed markdownlint lockfile
  • replace the same-version global markdownlint install with repository-local npm ci --ignore-scripts --omit=dev
  • require a byte-identical lock stamp before host or validator markdownlint execution
  • keep the local dependency tree out of Git, Docker context, and repository/document scans

Security

Closes Dependabot alerts #20 and #21 and prevents a stale same-version global or local markdownlint tree from retaining vulnerable transitives. Fresh validator workspaces use only the image-owned locked install; missing or stale installs fail closed.

Validation

  • npm audit --omit=dev: zero vulnerabilities
  • full host repository validation: 17 scenarios passed, 5 certification-only skips
  • clean-workspace validator image and shared validate lane passed, including host invariants and install artifact build
  • validator-backed docs lane passed
  • go test ./internal/metadatautil ./internal/testkit
  • pin-policy mutation and stale-lock behavioral negative tests passed
  • three-lens adversarial post-fix review: no actionable findings

Single-maintainer workflow; no independent human approval is claimed.

Bumps [js-yaml](https://github.com/nodeca/js-yaml) from 5.2.1 to 5.2.2.
- [Changelog](https://github.com/nodeca/js-yaml/blob/master/CHANGELOG.md)
- [Commits](nodeca/js-yaml@5.2.1...5.2.2)

---
updated-dependencies:
- dependency-name: js-yaml
  dependency-version: 5.2.2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot
dependabot Bot requested a review from omkhar as a code owner July 29, 2026 13:37
@dependabot dependabot Bot added dependencies Dependency updates npm NPM dependency updates labels Jul 29, 2026
…space certification and peer review pass; security tooling fix)
@omkhar omkhar changed the title chore: bump js-yaml from 5.2.1 to 5.2.2 in /tools/markdownlint fix: lock markdownlint to patched reviewed installs Jul 29, 2026
@omkhar

omkhar commented Jul 29, 2026

Copy link
Copy Markdown
Owner

@codex review

@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review: Didn't find any major issues. You're on a roll.

Reviewed commit: 0ed0c8471f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@omkhar

omkhar commented Jul 29, 2026

Copy link
Copy Markdown
Owner

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 0ed0c8471f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread CONTRIBUTING.md
@omkhar
omkhar merged commit 9558d9a into main Jul 29, 2026
15 of 16 checks passed
@omkhar
omkhar deleted the dependabot/npm_and_yarn/tools/markdownlint/js-yaml-5.2.2 branch July 29, 2026 14:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Dependency updates npm NPM dependency updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant