Skip to content
This repository was archived by the owner on Mar 13, 2026. It is now read-only.

feat: op-celestia-indexer support for 4844 blob indexing#530

Merged
tuxcanfly merged 15 commits intocelestia-developfrom
n/4844-blob-indexing
Dec 19, 2025
Merged

feat: op-celestia-indexer support for 4844 blob indexing#530
tuxcanfly merged 15 commits intocelestia-developfrom
n/4844-blob-indexing

Conversation

@nuke-web3
Copy link
Copy Markdown
Member

@nuke-web3 nuke-web3 commented Dec 3, 2025

resolves #519

@nuke-web3 nuke-web3 self-assigned this Dec 3, 2025
nuke-web3
nuke-web3 commented Dec 3, 2025
View reviewed changes
Comment thread README.md
nuke-web3
nuke-web3 commented Dec 3, 2025
View reviewed changes
Comment thread op-celestia/flags/flags.go
github-advanced-security[bot]
github-advanced-security AI found potential problems Dec 3, 2025
View reviewed changes
Comment thread .github/workflows/docker.yml Fixed
Comment thread .github/workflows/docker.yml Fixed
Comment thread .github/workflows/fork-sync.yml Fixed
Comment thread .github/workflows/fork-sync.yml Fixed
Comment thread .github/workflows/validate.yml Fixed
tuxcanfly and others added 8 commits December 3, 2025 16:28
@tuxcanfly @nuke-web3
da: use celestia as da
977b3ed 
build(deps): update go-da to v0.1.0

build(deps): update go-da to v0.2.0; add ctx

da: add cli flags for da config

da: add get timeout to da client

build(deps): bump local-celestia-devnet to v0.12.7

fix(cli): fix da rpc check

fix(cli): check return err

fix(da): blob data source reuse DataFromEVMTransactions

da: upgrade to go-da v0.5.0

feat(op-batcher): support disable eth fallback

chore(op-batcher): refactor EthFallbackDisabledFlagName

da: use env auth token if set

feat: enable multi frame txs and frame size setting for calldata txs

chore: add log for multiframetxs setting

da: try using blobdata for eth fallback

da: add gas price flag (#451)

da: reuse useblobs for multiframetxs (#452)

Add Github CI (#472)

Cleanup Github CI workflow (#478)

feat: allow to toggle Celestia in op-batcher and op-node (#498)

feat: add op-celestia indexer (#503)

fix: init op-node client

da: use celestia-node client (#504)

da: fix batcher submit (#511)

feat(op-celestia/indexer): add indexed l2 block status (#514)

da: optionally support new tx client (#517)

da: fix celestia config

da: fix celestia enabled

da: update celestia-node v0.26.4

da: log celestia config
@nuke-web3
feat: add EIP4844 indexing for blobs to Celestia indexer.
e8d0a2e
@nuke-web3
fix: log unsupported AltDA types, better comments on spec
eb85d28
@nuke-web3
fix merge artifacts
7765a14
@nuke-web3
fix just test , nits
919dce3
@nuke-web3
rm duplicate fn
f9aa896
@nuke-web3
fix: op-node builds
fb6de84
@nuke-web3
BREAKING API CHANGE: different type for calldata vs blob in API response
fc9bf19
@nuke-web3 nuke-web3 force-pushed the n/4844-blob-indexing branch from ba3b790 to fc9bf19 Compare December 3, 2025 23:29
nuke-web3
nuke-web3 commented Dec 3, 2025
View reviewed changes
Comment thread op-celestia/indexer/rpc/api.go Outdated
nuke-web3
nuke-web3 commented Dec 3, 2025
View reviewed changes
Comment thread op-celestia/indexer/store/store_test.go
nuke-web3
nuke-web3 commented Dec 9, 2025
View reviewed changes
Comment thread op-celestia/indexer/service.go
@nuke-web3
Copy link
Copy Markdown
Member Author

nuke-web3 commented Dec 9, 2025

TIL https://getfoundry.sh/anvil/reference/#supported-beacon-node-api-endpoints so we should be able to test against anvil + celestia devnet image 🤞 - I will report back

@nuke-web3
Copy link
Copy Markdown
Member Author

nuke-web3 commented Dec 10, 2025

Added some tests such that just test is happy. It might be unit tests for storage & API tests are redundant.... open to better ideas!

Now need integration tests against real devnet 🔜

tuxcanfly
tuxcanfly suggested changes Dec 10, 2025
View reviewed changes
Comment thread op-celestia/indexer/config.go
Comment thread op-celestia/flags/flags.go
Comment thread op-celestia/readme.md
@tuxcanfly tuxcanfly marked this pull request as ready for review December 19, 2025 21:06
tuxcanfly
tuxcanfly approved these changes Dec 19, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@tuxcanfly tuxcanfly left a comment

Choose a reason for hiding this comment

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

OK. Ran unit tests and seeing a failure which seems to be unrelated to blobs, but other blob tests look good:

=== RUN   TestIndexer_E2E_CalldataAndCelestia
    harness_test.go:160: [hint] calldata tx included in L1 block 703 (0xa2e657240289a5900eb34b1be2e33f570d7016714993474ae82713b247f766c4)
    harness_test.go:164: [hint] submitted celestia blob at height=9290241 namespace=0000000000000000000000000000000000000001010101010101010101 commitment=2b783b93b670df59c7b4fae8fda05e3b88a906bd99c5bec40d6129f12c809a32
    harness_test.go:168: [hint] alt-da commitment tx included in L1 block 706 (0xad8e5c8b3252dd2d3e9a89a755bc36df989796fe99289b8003b1c599cf655e00)
    harness_test.go:171: indexer did not store any locations in time.
        Current store status: indexed_blocks=0 last_indexed_block=716
        Hints:
        - Ensure geth is producing blocks (mining/automine).
        - Ensure the batch inbox address matches the tx 'to' address: 0x8943545177806ED17B9F23F0a21ee5948eCaa776
        - Check indexer logs for batch/frame decode errors.

Would revert the readme change of response type back to "ethereum", as we're differentiating blobs based on other fields now.

@tuxcanfly tuxcanfly merged this pull request into celestia-develop Dec 19, 2025
6 of 17 checks passed
@tuxcanfly tuxcanfly deleted the n/4844-blob-indexing branch December 19, 2025 21:10
tuxcanfly added a commit that referenced this pull request Dec 19, 2025
@tuxcanfly
da: use celestia as da
7151a46 
build(deps): update go-da to v0.1.0

build(deps): update go-da to v0.2.0; add ctx

da: add cli flags for da config

da: add get timeout to da client

build(deps): bump local-celestia-devnet to v0.12.7

fix(cli): fix da rpc check

fix(cli): check return err

fix(da): blob data source reuse DataFromEVMTransactions

da: upgrade to go-da v0.5.0

feat(op-batcher): support disable eth fallback

chore(op-batcher): refactor EthFallbackDisabledFlagName

da: use env auth token if set

feat: enable multi frame txs and frame size setting for calldata txs

chore: add log for multiframetxs setting

da: try using blobdata for eth fallback

da: add gas price flag (#451)

da: reuse useblobs for multiframetxs (#452)

Add Github CI (#472)

Cleanup Github CI workflow (#478)

feat: allow to toggle Celestia in op-batcher and op-node (#498)

feat: add op-celestia indexer (#503)

fix: init op-node client

da: use celestia-node client (#504)

da: fix batcher submit (#511)

feat(op-celestia/indexer): add indexed l2 block status (#514)

da: optionally support new tx client (#517)

da: fix celestia config

da: fix celestia enabled

da: update celestia-node v0.26.4

da: log celestia config

gh: delete dependabot

feat: `op-celestia-indexer` support for 4844 blob indexing (#530)

Co-authored-by: Javed Khan <tuxcanfly@gmail.com>
This was referenced Jan 23, 2026
Merged
Closed
@nuke-web3 nuke-web3 mentioned this pull request Feb 12, 2026
Open
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

1 more reviewer

@tuxcanfly tuxcanfly tuxcanfly approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@nuke-web3 nuke-web3

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Indexer drops batches when batcher falls back to EIP-4844 blobs

3 participants

@nuke-web3 @tuxcanfly @github-advanced-security