Skip to content

fix(deps): patch renderer Vitest mocker vulnerabilities - #284

Merged
Angnuo (angnuoli) merged 1 commit into
mainfrom
codex/vitest-mocker-security-update
Oct 28, 2026
Merged

Angnuo (angnuoli) merged 1 commit into
mainfrom
codex/vitest-mocker-security-update

Conversation

@angnuoli

Copy link
Copy Markdown
Collaborator

Summary

Validation

  • Normal npm ci in desktop, including its renderer postinstall: passed.
  • Separate normal npm ci in desktop/renderer: passed. Neither command uses --ignore-scripts; existing npm install-script approval policy is unchanged.
  • Exact SHA-256 manifest/lock hashes remain unchanged after both installs.
  • All eight downloaded tarballs match the expected SHA-512 lock integrity. Their metadata and all 221 published files match the installed packages.
  • Structural lock comparison confirms only the root Vitest range and the eight approved package entries changed. Published mock redirect allowlist and string serialization hardening are present.
  • Full renderer tests before and after the update: 55 files, 827 passed, 2 skipped.
  • Desktop Node regression suite: 18 passed.
  • npm run build in desktop (TypeScript plus renderer production build): passed using Vite 6.4.4.
  • Renderer lint: no errors, 83 existing warnings; findings exactly match the fresh baseline.
  • Desktop lint: the same 2 existing errors and 57 warnings as the baseline (no-control-regex in chat-attachments.ts, unused readJsonObject in openclaw-upgrade-recovery.ts). No unrelated lint changes.
  • Lockfile-only npm audit: the two Vitest-related moderate findings disappear (7 findings before, 5 after); the five unrelated high findings are unchanged and are not addressed by this scoped PR.
  • git diff --check: passed.

Limitations

  • Full Electron-driven desktop tests and an installer smoke test are not claimed: the current npm install-script policy does not install the Electron runtime. The policy was not weakened to make validation pass.
  • Production build reports annotation/chunk-size warnings; build succeeds.
  • Original developer checkout and its two pre-existing local changes are untouched.

Merge

@angnuoli
Angnuo (angnuoli) merged commit f12f0a1 into main Oct 28, 2026
1 of 2 checks passed
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