Skip to content

feat: enrich validation logs, pin mega-kona, add justfile - #189

Open
claymega wants to merge 3 commits into
feat/support_kona_validatorfrom
clay/feat/add_log
Open

feat: enrich validation logs, pin mega-kona, add justfile#189
claymega wants to merge 3 commits into
feat/support_kona_validatorfrom
clay/feat/add_log

Conversation

@claymega

Copy link
Copy Markdown

Summary

  • Enrich successful block validation debug logs with block hash, tx/gas/size, salt KV count, and per-stage timing stats.
  • Pin mega-kona dependencies to rev 20500df (devnet Rex6 schedule) instead of floating develop.
  • Add a justfile recipe to build the opt-in kona-validator binary.

Test plan

  • cargo check -p stateless-validator
  • Confirm validation success logs include the new fields at debug level
  • just build-kona-validator (or equivalent cargo command) builds successfully with network access to mega-kona

claymega and others added 3 commits August 3, 2026 08:07
Pin kona workspace deps to 20500dfb so chain id 6344 (pnet/devnet)
loads rex_6_time and SequencerRegistry v2 upgrade can succeed.
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@github-actions

github-actions Bot commented Aug 24, 2026

Copy link
Copy Markdown

Claude review status

Living comment — rewritten in place. The review workflow keeps this single comment up to date instead of posting a new one each round, so it always describes the latest reviewed commit and the earlier text is intentionally gone. No reply is needed here; answer findings and questions in the review threads it links to.

🛠️ Review did not finish

Attempted head 7ee6a2d4 · updated 2026-08-24T06:51:31+00:00

This round did not publish: MODEL_ACTION_FAILED in phase review_retry. Anything listed below is from the last round that did. Re-run the workflow or push a new commit to try again.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 7ee6a2d4dd

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread Cargo.toml
kona-mpt = { git = "https://github.com/megaeth-labs/mega-kona.git", branch = "develop", package = "kona-mpt", default-features = false }
kona-proof = { git = "https://github.com/megaeth-labs/mega-kona.git", branch = "develop", package = "kona-proof", default-features = false }
kona-registry = { git = "https://github.com/megaeth-labs/mega-kona.git", branch = "develop", package = "kona-registry", default-features = false }
kona-driver = { git = "https://github.com/megaeth-labs/mega-kona.git", rev = "20500dfb6c14fb0ed20ccd2a2a0dab388907c5b6", package = "kona-driver", default-features = false }

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Validate the newly introduced Rex6 schedule

When chain 6344 supplies a missing or mismatched rex6Time, this pin makes Kona load and execute the registry's new rex_6_time, but the local MegaethGenesisHardforks still stops at Rex5 and check_hardfork_consistency likewise compares only through rex_5. Startup therefore accepts a registry/genesis consensus-schedule skew and validates using the registry rather than the operator-supplied genesis; update the local MegaEVM/chain-spec representation and consistency check alongside this pin.

Useful? React with 👍 / 👎.

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.

3 participants