Skip to content

Conversation

@rezzmah
Copy link

@rezzmah rezzmah commented Jul 18, 2025

Waiting on paradigmxyz#17441

rezzmah added 5 commits July 16, 2025 20:48
Enable CLI to work with custom block header types by requiring
From<alloy_consensus::Header> conversion. This allows chains to
use custom headers while maintaining CLI compatibility through
standard Ethereum header conversion.

Changes:
- Make init_state command generic over BlockHeader types
- Add From<Header> trait bound for custom headers
- Update header factory to use generic conversion
- Fix header field access to use trait methods
Clean up code formatting and import organization:
- Fix multi-line type parameter formatting
- Add missing trailing comma in where clause
- Reorganize imports for better readability
- Format constraint bounds consistently

This addresses formatting issues from the previous commit.
The function only needs Decodable trait to read RLP-encoded headers.
BlockHeader constraint was redundant since the function doesn't use
any BlockHeader methods - it only decodes the header from RLP bytes.
@github-actions
Copy link

Your PR title doesn't follow the Conventional Commit guidelines.

Example of valid titles:

  • feat: add new user login
  • fix: correct button size
  • docs: update README

Usage:

  • feat: Introduces a new feature
  • fix: Patches a bug
  • chore: General maintenance tasks or updates
  • test: Adding new tests or modifying existing tests
  • bench: Adding new benchmarks or modifying existing benchmarks
  • perf: Performance improvements
  • refactor: Changes to improve code structure
  • docs: Documentation updates
  • ci: Changes to CI/CD configurations
  • revert: Reverts a previously merged PR
  • deps: Updates dependencies

Breaking Changes

Breaking changes are noted by using an exclamation mark. For example:

  • feat!: changed the API
  • chore(node)!: Removed unused public function

Help

For more information, follow the guidelines here: https://www.conventionalcommits.org/en/v1.0.0/

@rezzmah rezzmah changed the title Rezbera/cli custom headers Temporary PR till Upstream Merges Jul 18, 2025
@rezzmah rezzmah changed the title Temporary PR till Upstream Merges temp Jul 18, 2025
@rezzmah rezzmah closed this Jul 19, 2025
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.

2 participants