Skip to content

chore(kv-store): disable browser tests on v4-next to unblock staging CI#22805

Merged
nchamo merged 1 commit intobackport-to-v4-next-stagingfrom
claudebox/backport-22693-disable-kv-store-browser-tests
Apr 27, 2026
Merged

chore(kv-store): disable browser tests on v4-next to unblock staging CI#22805
nchamo merged 1 commit intobackport-to-v4-next-stagingfrom
claudebox/backport-22693-disable-kv-store-browser-tests

Conversation

@AztecBot
Copy link
Copy Markdown
Collaborator

What

Cherry-picks the last commit of #22693 (f0f0c6977b95114aeb2b7204414941adfff86efe, "disable kv-store browser tests to unblock CI") onto v4-next.

yarn-project/kv-store/package.json       | 2 +-
yarn-project/kv-store/package.local.json | 2 +-

Drops && yarn test:browser from the kv-store test script so CI runs node tests only.

Why

The staging backport PR #22763 is failing ci on the kv-store browser-test step. The failure is the well-known sqlite-opfs hang originally fixed in #22693: the run consistently dies after sqlite-opfs/multi_map.test.ts > AztecMultiMap > multiple keys are independent, with timeout 600s killing bash 10 minutes later. v4-next does not yet have #22693, so the staging branch reproduces the hang.

Per Martin's call in Slack, only this last commit (the simple test script change) is needed on v4-next — the other commits in #22693 (worker.ts pool-skip + vitest gate) aren't required just to unblock CI.

Once this lands on v4-next, re-running CI on #22763 should pass.

ClaudeBox log: https://claudebox.work/s/c257cb057ea53390?run=3

@AztecBot AztecBot added ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR. labels Apr 27, 2026
@nchamo nchamo self-assigned this Apr 27, 2026
@nchamo nchamo marked this pull request as ready for review April 27, 2026 19:00
@nchamo nchamo enabled auto-merge April 27, 2026 19:01
@nchamo nchamo merged commit 76c72f8 into backport-to-v4-next-staging Apr 27, 2026
30 of 36 checks passed
@nchamo nchamo deleted the claudebox/backport-22693-disable-kv-store-browser-tests branch April 27, 2026 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-draft Run CI on draft PRs. claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants