Skip to content

Merge latest upstream#19

Merged
karlb merged 40 commits intodevelopfrom
karlb/merge-upstream2
Sep 19, 2025
Merged

Merge latest upstream#19
karlb merged 40 commits intodevelopfrom
karlb/merge-upstream2

Conversation

@karlb
Copy link
Copy Markdown

@karlb karlb commented Sep 16, 2025

I didn't do thorough testing yet, but locally it compiles, lints and tests pass, except for the two tests that require many env vars to be set (didn't try enough to set them correctly yet).

Closes https://github.com/celo-org/celo-blockchain-planning/issues/1228

fakedev9999 and others added 29 commits July 10, 2025 00:12
* ci: add link + spell checker

* ci: remove recursive submodules checkout

* docs: use different link for spn
…sts (succinctlabs#560)

* add : essential logs for range+agg proofs

* chg : if/else into match

---------

Co-authored-by: Yuwen Zhang <7747842+yuwen01@users.noreply.github.com>
* chore: bump kona

* refac: avoid allocations when handling input

* refac: avoid alloc + clone

* chore: nits
* feat: add game prove metrics

* feat: add fast finality proving limit

* chore: nit fixes
* feat: improve game resolution

* feat(fault-proof): separate bond claiming by mode

* docs: fix typo

* feat: set jemalloc as global allocator
* feat: update to support celestia indexer changes

See celestiaorg/optimism#502.
Now handles batcher that switch between DA layers.

* chore: fix logger init

* chore: bump hana for v4 compatibility

* chore: bump celestia range elf
* tests pass, fixes in

* proposer is running locally

* fix test, still broken though

* more detailed error

* jank proxy

* wrapper solution instead

* cleanup

* fix silly errors;

* tweak forge test

* docs for dgfProposeL2Output;

* spot=false

* bump elf

* nits + docs

* add audit + pin nightly toolchain
* chore: add features support to deploy-fdg-contracts in justfile

* feat(contracts): add missing getters for public values
* fix: end exclusive range

See https://docs.celestia.org/how-to-guides/blobstream-proof-queries.

* refactor(scripts): host method for starting block

* refac: handle when None returned

* change for fetch-l2oo-config as well
* feat: add cycles + prover gas metrics

Also improved logging for better vis and debugging.

* chore: make fmt happy

* chore: apply comments
* add dev section to readme

* doc pass
* fix(validity): request deadline

* fix(fault-proof): request deadline

* fix: add 1T gas limit

The DEFAULT_GAS_LIMIT is 1B.

* fix: add 4h timeout to agg proof request as well

---------

Co-authored-by: fakedev9999 <taehoon@succinct.xyz>
* docs: add missing CELESTIA_INDEXER_RPC

* feat: add op-celestia-indexer to docker compose

* docs: update docs

* fix healthcheck

* docs: add warning for proof window

* docs: udpate start l1 block and op-celestia-indexer
…naged instance (not naked proxies) (succinctlabs#606)

* fix(contracts): upgrade, missing proxy in calldata for proxy admin managed instance (not naked proxies)

* chore: forge fmt

* refac(contracts): use abi.encodeCall

---------

Co-authored-by: emiliano-conduitxyz <emiliano@conduit.xyz>
* feat: support eigenda

Bumps sp1, hana, hokulea for kona-client/v1.0.2 with MSRV 1.86.

* ci: install sp1 toolchain

* chore: bump elfs

* fix: write proof to stdin for proof agg

* fix: bump hokulea + hardcoded vkey + defer verification

* fmt

* bump elfs

* feat: docker for eigenda

* refac: extract proof bytes from witness data

* refac: remove executor placeholder

* chore: download srs from proxy image

* docs: add eigenda docs

* chore: bump hokulea

* make ci green

* skip canoe proof generation

* chore: bump hokulea

* bump hokulea to support mock mode for sp1-cc proving
…s#608)

* chore: remove unused code

* fix: reduce RPC concurrency to prevent rate limit errors
cursor[bot]

This comment was marked as outdated.

@seolaoh
Copy link
Copy Markdown
Collaborator

seolaoh commented Sep 17, 2025

We need to update the elf files when updating celo-kona version or op-succinct codes, see https://github.com/celo-org/op-succinct/blob/develop/book/advanced/verify-binaries.md to check how to build the elfs

cursor[bot]

This comment was marked as outdated.

@karlb karlb force-pushed the karlb/merge-upstream2 branch from 390862b to 8e9f803 Compare September 17, 2025 11:08
@palango
Copy link
Copy Markdown

palango commented Sep 17, 2025

There's a little fixup upstream we should probably include: https://github.com/succinctlabs/op-succinct/pull/610/files

@karlb
Copy link
Copy Markdown
Author

karlb commented Sep 17, 2025

There's a little fixup upstream we should probably include: https://github.com/succinctlabs/op-succinct/pull/610/files

I merged upstream/main again, so it that is included now.

@piersy
Copy link
Copy Markdown

piersy commented Sep 17, 2025

I'm currently testing the cost estimator on this, and facing some issues which seem to have been introduced by merging upstream.

This error

2025-09-16T22:20:51.859282Z DEBUG sending request method=eth_getBlockByHash id=59611
2025-09-16T22:20:52.007955Z DEBUG ReqwestTransport: received response from server status=200 OK url=https://internal-forno.celo-sepolia.celo-testnet.org/
2025-09-16T22:20:52.009004Z DEBUG ReqwestTransport: retrieved response body. Use `trace` for full body bytes=6430 url=https://internal-forno.celo-sepolia.celo-testnet.org/
2025-09-16T22:20:52.009709Z DEBUG ReqwestTransport: close time.busy=1.76ms time.idle=149ms url=https://internal-forno.celo-sepolia.celo-testnet.org/
2025-09-16T22:20:52.011356Z ERROR Failed to prefetch hint: deserialization error: data did not match any variant of untagged enum BlockTransactions at line 1 column 6186

See https://github.com/celo-org/celo-blockchain-planning/issues/1236

Copy link
Copy Markdown
Collaborator

@seolaoh seolaoh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't check the contract codes and @Mc01 's commits, but other than that LGTM

@karlb karlb merged commit bf0eb5d into develop Sep 19, 2025
6 checks passed
@karlb karlb deleted the karlb/merge-upstream2 branch September 19, 2025 09:33
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.

10 participants