Repository navigation
Patch dev SQLite build dependency to Undici 6.29.1 - #194
Conversation
🧪 Comprehensive Test Suite
Full-stack smoke gate runs in the CI workflow. |
🧪 Comprehensive Test Suite
Full-stack smoke gate runs in the CI workflow. |
|
Integration preflight: exact head Independent agent review cleared the exact head and documentation-only provenance from runtime head Final CI 40567729657 and Node matrix/installer 40567729457 passed at that head. Actual smoke-job logs show 6 smoke, 42 new-user, 27 mobile, 3 living-graph and 3 performance cases passed, with no retries/failures/skips in those sequences. Build, types/lint, all unit suites, real-Neo4j MCP contract, security scan and showcase passed. Production-main/nightly smoke is intentionally inapplicable to this dev PR. The earlier canceled runtime-head run is not used as final CI evidence. Local clean Node 22.18.0/npm 10.8.2 install, 163 harness cases, real SQLite bootstrap/restart, all builds/types/lint passed. Fresh production audit has no high/critical entries; five moderate entries remain separately tracked by #186. The isolated native fixture was disposable; no shared database or deployed service was changed. Proceeding with the owner-authorized administrative merge into dev only, matching the exact head, with all protection settings unchanged. Post-merge branch CI and remote parent/tree verification remain required before integration closeout. This operation supplies neither a non-author approval nor main/release/deployment authority. |
|
Post-merge verification is complete for dev Both actual branch workflows passed: CI 40573195798 and Node 22.18/24 + installer 40573195765. All applicable unit/build/types/lint/security/real-Neo4j/browser/showcase gates are successful; the main/nightly production job remains inapplicable to dev. Administrative integration used the standing dev-only authorization, with protection settings unchanged. This is not a non-author approval, main promotion, release or deployment. |
devstill installs Undici 6.28.1 through SQLite's optional node-gyp dependency, failing the production audit and blocking #192. Backport the already reviewed compatible 6.29.1 patch from #191 (8a9c7512b5dedf4020d117887674d378de85667b) without bringing main-promotion workflow changes into this fix.Only the Undici lockfile record changes. The regression checks every locked copy and the actual SQLite → node-gyp consumer resolution against its declared range. The index conflict was resolved by retaining dev's entries and adding only this lesson; main-only lesson links were not imported.
Validation at
765cf1985c389f8850a13df338d56c8113350ce7with Node 22.18.0 / npm 10.8.2:npm ci --legacy-peer-deps --no-audit --no-fund: passed.npm audit --omit=dev --audit-level=high: zero high/critical, five moderate package findings remain (Resolve the remaining production moderate dependency families in scoped PRs #186); Undici is absent.This does not update Node's bundled fetch or claim an exploit reproduction, hosted parity, deployment, or resolution of the remaining dependency families. Historical October 24 evidence is clearly dated; current backport evidence is separate. Part of #193; unblocks #192 and does not close #186.