Skip to content

fix(archiver): filter tagged log queries by block number (backport to v4)#21433

Merged
spalladino merged 4 commits intobackport-to-v4-stagingfrom
claudebox/17e0293698b5a7b9-4
Mar 12, 2026
Merged

fix(archiver): filter tagged log queries by block number (backport to v4)#21433
spalladino merged 4 commits intobackport-to-v4-stagingfrom
claudebox/17e0293698b5a7b9-4

Conversation

@AztecBot
Copy link
Collaborator

Summary

Backport of #21388 to v4.

Resolves the referenceBlock hash to a block number in the AztecNode and passes it down as upToBlockNumber so the LogStore stops returning logs from blocks beyond the client's sync point. Also adds an ordering check on log insertion to guard against out-of-order appends.

Conflict resolution

One conflict in yarn-project/archiver/src/errors.ts: the original PR added BlockAlreadyCheckpointedError which doesn't exist on v4. Kept only OutOfOrderLogInsertionError (the new class from this PR) and dropped the v4-irrelevant class.

Commits

  1. Cherry-pick with conflicts — raw cherry-pick attempt preserved for reviewer visibility
  2. Conflict resolution — resolved the errors.ts conflict

ClaudeBox log: https://claudebox.work/s/17e0293698b5a7b9?run=4

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 12, 2026
@spalladino spalladino marked this pull request as ready for review March 12, 2026 20:14
@spalladino spalladino enabled auto-merge (squash) March 12, 2026 20:21
@AztecBot
Copy link
Collaborator Author

Flakey Tests

🤖 says: This CI run detected 1 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/bae0e7bb52543320�bae0e7bb525433208;;�): yarn-project/end-to-end/scripts/run_test.sh ha src/composed/ha/e2e_ha_full.test.ts (73s) (code: 0)

@spalladino spalladino merged commit ee06f88 into backport-to-v4-staging Mar 12, 2026
9 checks passed
@spalladino spalladino deleted the claudebox/17e0293698b5a7b9-4 branch March 12, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

claudebox Owned by claudebox. it can push to this PR.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants