Skip to content

fix: consolidate security updates and repair stale-lock recovery - #205

Merged
siracusa5 merged 3 commits into
mainfrom
codex/security-dependency-sweep
Oct 26, 2026
Merged

siracusa5 merged 3 commits into
mainfrom
codex/security-dependency-sweep

Conversation

@siracusa5

@siracusa5 siracusa5 commented Oct 25, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Consolidates all six open Dependabot PRs and refreshes the three app dependency trees to resolve the current security findings. Fresh full-tree npm audits report zero vulnerabilities for console, site, and desktop; the separate desktop production audit is also clean.

Supersedes #193, #195, #198, #202, #203, and #204, which are closed in favor of this combined PR.

Changes

Affected area

  • Core engine / MCP / write path
  • Console
  • Site
  • Playground or local demo surfaces
  • Docs / specs / contributor workflow
  • Packs

Desktop runtime and build tooling are also updated.

Validation

All local checks used Node 22.23.4.

  • npm test: 76/76 root suites, including retrieval eval, package/release checks, and scale/lifecycle tests.
  • npm --prefix apps/console run typecheck
  • npm --prefix apps/console test: 789 tests.
  • npm --prefix apps/console run build:live: production renderer build.
  • npm --prefix apps/site run build: 41 pages; install-page and hidden-commerce verification.
  • Fresh desktop npm ci && npm test: 186 tests, including settings, quit persistence, and window lifecycle.
  • Desktop navigation and CLI-status checks.
  • Sandboxed Electron boot/token smoke with temporary app data: authenticated 200, unauthenticated 401.
  • Desktop 3,000-document isolation check: 6 ms main-loop lag (400 ms limit).
  • Fresh full-tree npm audits for all three apps and separate desktop --omit=dev audit: zero findings.
  • Independent code, security, and documentation review: no outstanding blockers.
  • Release-workflow and distribution-artifact checks after the Node setup correction: 31 tests.
  • Manifest-lock tests: 22 pass; the live-replacement regression failed against the original code.
  • GitHub CI checks pass on ed5cae3: required gate, full engine suites on Node 22/24, Windows engine, console, site, Mac/Linux desktop, CodeQL, and DCO.

Compatibility

  • Root commands still work. App toolchains now require Node 22.23.4 or a compatible newer runtime; the dependency-free engine floor remains Node 22.
  • No root npm dependencies were added.
  • Generated files remain ignored.
  • Desktop package and lockfile root versions remain 0.10.0.

Electron moves to 44.9.3 and Vitest to 5.0.5, covering the pending major updates with their latest compatible patches. No release or deployment is performed here; packaged-app fixes reach users through the next signed app release. Patched GitHub alerts close after merge and dependency-graph refresh.

The existing lock protocol also has a separate three-contender parking/restoration risk documented in the audit note. This PR makes the measured owner-generation correction; it does not claim to redesign the complete acquisition/reclamation protocol.

Sign-off

Signed-off-by: John Siracusa <siracusa5@users.noreply.github.com>
Signed-off-by: John Siracusa <siracusa5@users.noreply.github.com>
Signed-off-by: John Siracusa <siracusa5@users.noreply.github.com>
@siracusa5 siracusa5 changed the title fix(deps): consolidate security patches and pending dependency updates fix: consolidate security updates and repair stale-lock recovery Oct 26, 2026
@siracusa5
siracusa5 merged commit af5ca0a into main Oct 26, 2026
11 checks passed
@siracusa5
siracusa5 deleted the codex/security-dependency-sweep branch October 26, 2026 19:00
@siracusa5 siracusa5 mentioned this pull request Oct 28, 2026
5 tasks done
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