Skip to content

Security and Code Scanning Fixes - CodeQL and dependabot for ainxt-code - #38

Open
patil-utkarsh wants to merge 9 commits into
npci:mainfrom
patil-utkarsh:main
Open

patil-utkarsh wants to merge 9 commits into
npci:mainfrom
patil-utkarsh:main

Conversation

@patil-utkarsh

@patil-utkarsh patil-utkarsh commented Oct 23, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Fixes all open Dependabot and CodeQL alerts in ainxt-code.

Dependency updates (dev-only, mostly lockfile changes)

  • brace-expansion → 1.1.21 / 2.1.7 / 5.0.12
  • markdown-it → 14.3.3
  • fast-uri → 3.1.8
  • serialize-javascript → 7.1.2
  • vite → ^6.4.5 and source-map-js → 1.2.2 (vscode-acp/webview-ui)

XSS fixes in the docs viewer (docs/index.html)

  • Mermaid diagram source stays HTML-escaped, and diagrams are rendered with mermaid.run() and securityLevel: 'strict' instead of innerHTML.
  • Diagram render errors and page error messages are shown as plain text (textContent), not parsed as HTML.

Clears 18 Dependabot alerts and 4 CodeQL alerts (js/xss-through-dom, js/double-escaping) on the fork.

Verification

  • CI: Lint, build, test, package and Mermaid diagrams render (DOC-021) both pass.
  • The fork shows 0 open Dependabot and CodeQL alerts.

Notes

  • The advanced CodeQL workflow was removed. This repo already uses CodeQL default setup, which rejects uploads from an advanced workflow.
  • Squash merge recommended, since the commits include merges of fork PRs.

dependabot Bot and others added 9 commits October 6, 2026 10:45
Bumps  and [brace-expansion](https://github.com/juliangruber/brace-expansion). These dependencies needed to be updated together.

Updates `brace-expansion` from 1.1.18 to 1.1.21
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@v1.1.18...v1.1.21)

Updates `brace-expansion` from 5.0.9 to 5.0.12
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@v1.1.18...v1.1.21)

Updates `brace-expansion` from 2.1.4 to 2.1.7
- [Release notes](https://github.com/juliangruber/brace-expansion/releases)
- [Commits](juliangruber/brace-expansion@v1.1.18...v1.1.21)

---
updated-dependencies:
- dependency-name: brace-expansion
  dependency-version: 1.1.21
  dependency-type: indirect
- dependency-name: brace-expansion
  dependency-version: 5.0.12
  dependency-type: indirect
- dependency-name: brace-expansion
  dependency-version: 2.1.7
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [source-map-js](https://github.com/7rulnik/source-map-js) from 1.2.1 to 1.2.2.
- [Release notes](https://github.com/7rulnik/source-map-js/releases)
- [Changelog](https://github.com/7rulnik/source-map-js/blob/main/CHANGELOG.md)
- [Commits](7rulnik/source-map-js@v1.2.1...v1.2.2)

---
updated-dependencies:
- dependency-name: source-map-js
  dependency-version: 1.2.2
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Updated CodeQL workflow configuration for better analysis.
…i-57637e47fe

Bump brace-expansion by dependabot
…de-acp/webview-ui/source-map-js-1.2.2

Bump source-map-js from 1.2.1 to 1.2.2 in /vscode-acp/webview-ui by dependabot
* Fix npm audit findings and CodeQL XSS alerts in docs viewer

* Render mermaid via mermaid.run to avoid re-parsing DOM text as HTML

* Show diagram source when mermaid fails to render

---------

Co-authored-by: patil-utkarsh <331620036+patil-utkarsh@users.noreply.github.com>
Bumps [markdown-it](https://github.com/markdown-it/markdown-it) from 14.3.1 to 14.3.3.
- [Changelog](https://github.com/markdown-it/markdown-it/blob/14.3.3/CHANGELOG.md)
- [Commits](markdown-it/markdown-it@14.3.1...14.3.3)

---
updated-dependencies:
- dependency-name: markdown-it
  dependency-version: 14.3.3
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite) from 6.4.3 to 6.4.5.
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v6.4.5/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v6.4.5/packages/vite)

---
updated-dependencies:
- dependency-name: vite
  dependency-version: 6.4.5
  dependency-type: direct:development
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
The upstream repository uses CodeQL default setup, which rejects SARIF uploads from advanced workflows. Drop the workflow so the PR's Analyze jobs no longer fail.

Signed-off-by: patil-utkarsh <patil.utkarsh@npci.org.in>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant