Skip to content

fix(deps): clear Dependabot and full-audit vulnerabilities - #131

Open
Amir Bredy (ABMFST) wants to merge 8 commits into
mainfrom
fix/dependabot-xmldom-20260915
Open

Amir Bredy (ABMFST) wants to merge 8 commits into
mainfrom
fix/dependabot-xmldom-20260915

Conversation

@ABMFST

@ABMFST Amir Bredy (ABMFST) commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix all nine currently open Dependabot alerts: require patched @xmldom/xmldom 0.8.15+ and update Electron within 42.x to 42.11.3, replacing the unpatched extract-zip dependency with Electron-maintained extraction.
  • Also patch the newer brace-expansion and nanoid denial-of-service advisories found by the full audit: brace-expansion 1.1.18 / 2.1.4 / 5.0.9 and nanoid 3.3.19 (minimum 3.3.18).
  • Add regression coverage for every resolved copy and Electron workspace/lockfile consistency; update both changelogs.

Validation

  • GitHub-hosted dependency refresh and focused regression tests passed: https://github.com/Azure/ConfigForge/actions/runs/34940224546
  • Full npm audit including development dependencies: 0 vulnerabilities.
  • Public-registry policy and cross-platform optional binary preservation verified. The temporary refresh workflow is removed from the final diff.
  • All required PR checks passed on fb76d70: full suite, lint, Node 22/24 builds, Electron smoke and CodeQL. No npm commands or npm installs were run locally.

Flavor and release flow

Merge status

Required independent approving review is pending. The normal merge attempt was blocked by branch policy; no protection settings were changed.

Amir Bredy and others added 8 commits September 14, 2026 23:30
Generate lockfile and version evidence on a read-only GitHub-hosted runner; no local npm execution.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 48993ee6-f068-4c4a-b317-2cd940fab804
Use public registry metadata from the hosted refresh, preserve optional platform entries, and guard every resolved xmldom copy against security regressions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 48993ee6-f068-4c4a-b317-2cd940fab804
Stay on the supported Electron major and remove the unpatched extract-zip dependency through its upstream-maintained replacement.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Copilot-Session: 48993ee6-f068-4c4a-b317-2cd940fab804
Generate the patched dependency lockfile on a hosted runner and retain audit evidence.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Use a hosted Node 24 install to replace stale npm 10 workspace lock records and preserve native optional package metadata.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Cover the additional denial-of-service advisories reported by the complete hosted npm audit without changing dependency major versions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Refresh overridden packages across all workspaces and assert security floors for brace-expansion, nanoid, xmldom and Electron installation tooling.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Record the clean npm 11 dependency graph and remove the temporary refresh workflow. Preserve cross-platform optional binaries and public-registry resolutions.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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