Skip to content

fix: bound getTxEffects oracle receipt reads#24640

Draft
AztecBot wants to merge 3 commits into
merge-train/fairies-v5from
cb/fix-gettxeffects-batch-read-concurrency
Draft

fix: bound getTxEffects oracle receipt reads#24640
AztecBot wants to merge 3 commits into
merge-train/fairies-v5from
cb/fix-gettxeffects-batch-read-concurrency

Conversation

@AztecBot

Copy link
Copy Markdown
Collaborator

Summary

  • Bound aztec_utl_getTxEffects receipt reads to 16 concurrent node requests instead of firing every unique hash at once.
  • Added a PXE oracle unit test that proves a 17-hash batch only starts the first 16 reads until one completes.

Merge queue investigation

Verification

  • git diff --check passed.
  • ./bootstrap.sh ci does not exist as a root bootstrap target on this branch; it exits with Unknown command: ci.
  • ./bootstrap.sh ci-fast was attempted locally because that is the failed merge-queue command, but this container has no Docker/Redis cache path and failed early in test_engine after only four early commands were written to /tmp/test_cmds; this is not the same failure as the remote timeout.
  • Focused Yarn tests are currently blocked because the Noir JS portal packages are missing and noir/bootstrap.sh cannot fetch just@1.42.4 from crates.io in this container (dns error, Label contains invalid characters: Err(Errors)).

Created by claudebox · group: slackbot

@AztecBot AztecBot added ci-draft Run CI on draft PRs. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR. labels Jul 10, 2026
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. ci-no-fail-fast Sets NO_FAIL_FAST in the CI so the run is not aborted on the first failure claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant