Repository navigation
Electron suite: 7 pre-existing failures on main (verify-gate, checkpoints, web, store) #536
Description
Found while verifying a merge against origin/main (2026-08-18, origin/main at 9517511). These fail identically on pristine origin/main, so they are not merge artifacts.
electron/test/verify-gate.test.js— 0 pass / 4 fail (whole file red)electron/test/checkpoints.test.js— 1 fail: "runner auto-checkpoint on successful turn" throwsworktreeBase is not configuredfrommaterializePendingWorktree(runner.js) — the fixture'screateRunnergets nouserDataPath, and the Worktree creation failure must never silently run the agent in the main checkout #511 fail-closed path now requires itelectron/test/web.test.js— 1 failelectron/test/store.test.js— 1 fail at store.test.js:106: thread object now carriesreviewAcceptedHunks: []which the expected fixture doesn't include
Repro: npm run test:electron on main.
Activity
currentbits commentedon Aug 20, 2026
Fixed on
coder/fork-electron-suite-7-pre-existing-failu-b31efc. These were fixture/assertion drift, not product bugs.verify-gate (4) + checkpoints (1):
createRunneromitteduserDataPath. After #511,materializePendingWorktreethrowsworktreeBase is not configuredfor any thread withworktreePath. Tests now passuserDataPath: fx.tmpDirso rematerialize usestmpDir/worktrees(same asmakeWorktreeFixture).web.test.js (2):
contextMenu:showis registered inregisterIpcoutsideIPC_HANDLERSon purpose (nativeMenu.popup; the web bridge must not pop a menu). The ownership test skips it and still assertsipcMain.handle("contextMenu:show").package-app.shcopieswsviaROOT_NM_PKGS+node_modules/$pkg, not a literalnode_modules/wspath (that string lives inverify-package.sh).store.test.js: already green on this tree — the round-trip fixture includes
reviewAcceptedHunks: [](andworktreeRetention: 10after #559).Verified: those four files 78/78. Full
npm run test:electronno longer reports any of the original seven; remaining reds are the known env suite (orch-serverneedsmemory-server/node_modules).