Skip to content

Don't set finalized block below head - #81

Merged
phearnot merged 6 commits into
mainfrom
fix/no-finalized-below-head
Oct 14, 2026
Merged

phearnot merged 6 commits into
mainfrom
fix/no-finalized-below-head

Conversation

@phearnot

Copy link
Copy Markdown
Contributor

No description provided.

phearnot and others added 6 commits October 14, 2026 13:00
Docker Compose builds via buildx bake, which rejects `network: host`
without an explicit --allow=network.host entitlement, failing the
Local Network tests job. The build only fetches Foundry and pip
packages, which works on the default build network.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
upload-artifact rejects names containing "/", so test logs were lost
for branches like fix/... The run attempt keeps names unique on re-runs.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
forge script keeps broadcast state per script and chain id in
contracts/eth/cache and now refuses to run while another process
modifies it ("another process is already modifying recovery state").
Suites run in parallel JVMs with the same chain id, so a concurrent
deployment failed silently and the suite aborted after waiting for the
contracts. Deployments now take an OS-level file lock, and a non-zero
forge exit code fails the setup right away.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
After the setup miner leaves, the acting miner can become the
designated miner of an epoch in which the setup miner already mined.
waitForMinerEpoch returned there, the tests then waited for the next
epoch to register their block, and failed when another miner was
designated in it ("is not allowed to mine in 5 epoch"). Wait for an
epoch of the acting miner without registered blocks instead.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@phearnot
phearnot merged commit 34f7fbc into main Oct 14, 2026
3 checks passed
@phearnot
phearnot deleted the fix/no-finalized-below-head branch October 14, 2026 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant