Skip to content

fix: remove extra argument in KVArchiverDataStore constructor call#21361

Merged
spalladino merged 1 commit intomerge-train/spartanfrom
claudebox/fix-archiver-build-error
Mar 11, 2026
Merged

fix: remove extra argument in KVArchiverDataStore constructor call#21361
spalladino merged 1 commit intomerge-train/spartanfrom
claudebox/fix-archiver-build-error

Conversation

@AztecBot
Copy link
Collaborator

Summary

  • Fixes yarn build error (TS2554: Expected 1-2 arguments, but got 3) in archiver-misc.test.ts
  • The KVArchiverDataStore constructor was changed to accept only (db, logsMaxPageSize?) but the test still passed a third { epochDuration } argument

Test plan

  • CI build should pass with this fix

ClaudeBox log: https://claudebox.work/s/b321e7afd3132cf4?run=1

@AztecBot AztecBot added the claudebox Owned by claudebox. it can push to this PR. label Mar 11, 2026
@spalladino spalladino marked this pull request as ready for review March 11, 2026 13:42
@spalladino spalladino enabled auto-merge (squash) March 11, 2026 13:42
@spalladino spalladino merged commit 4f980f1 into merge-train/spartan Mar 11, 2026
25 checks passed
@spalladino spalladino deleted the claudebox/fix-archiver-build-error branch March 11, 2026 13:59
github-merge-queue bot pushed a commit that referenced this pull request Mar 11, 2026
BEGIN_COMMIT_OVERRIDE
fix: (A-623) increase committee timeout in scenario smoke test (#21193)
feat: orchestrator enqueues via serial queue (#21247)
feat: rollup mana limit gas validation (#21219)
fix: make e2e HA test more deterministic (#21199)
chore: fix chonk_browser lint warning (#21265)
chore: deploy SPONSORED_FPC in test networks (#21254)
fix: (A-635) e2e bot flake on nonce mismatch (#21288)
chore: deflake duplicate attestations and proposals slash tests (#21294)
fix(sequencer): fix log when not enough txs (#21297)
chore: send env var to pods (#21307)
fix: Simulate gas in n tps test. Set min txs per block to 1 (#21312)
fix: update dependabot dependencies (#21238)
test: run nightly bench of block capacity (#20726)
fix: update block_capacity test to use new send() result types (#21345)
fix(node): fix index misalignment in findLeavesIndexes (#21327)
fix(log): do not log validation error if unregistered handler (#21111)
fix: limit parallel blocks in prover to max AVM parallel simulations
(#21320)
fix: use native sha256 to speed up proving job id generation (#21292)
chore: remove v4-devnet-1 (#21044)
fix(validator): wait for l1 sync before processing block proposals
(#21336)
fix(txpool): cap priority fee with max fees when computing priority
(#21279)
chore: Properly compute finalized block (#21156)
fix: remove extra argument in KVArchiverDataStore constructor call
(#21361)
chore: revert l2 slot time 72 -> 36 on scenario network (#21291)
fix(archiver): do not error if proposed block matches checkpointed
(#21367)
fix(claude): rule to not append echo exit (#21368)
chore: reduce severity of errors due to HA node not acquiring signature
(#21311)
fix: make reqresp batch retry test deterministic (#21322)
fix: (A-643) add buffer to maxFeePerBlobGas for gas estimation and fix
bump loop truncation (#21323)
fix(e2e): use L2 priority fee in deploy_method same-block test (#21373)
fix: reqresp flake & add logging (#21334)
END_COMMIT_OVERRIDE
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