• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix-invalid-name-eth-call

Reviewing files that changed from the base of the PR and between d63f67e and 124bcc8.

• CHANGELOG.md (1 hunks)
• src/rpc/methods/eth/types.rs (1 hunks)

• GitHub Check: Build Ubuntu
• GitHub Check: cargo-publish-dry-run
• GitHub Check: Build MacOS
• GitHub Check: tests
• GitHub Check: tests-release
• GitHub Check: Build forest binaries on Linux AMD64
• GitHub Check: All lint checks

CHANGELOG.md (1)

58-59: LGTM! Clear changelog entry.

The changelog entry accurately documents the change and follows the established format.

src/rpc/methods/eth/types.rs (1)

317-319: LGTM! Clean implementation of field alias for interoperability.

The addition of the alias = "input" attribute correctly allows deserialization from either "input" or "data" field names while maintaining backward compatibility by keeping "data" as the serialization key. The comment clearly explains the rationale.

Please verify that there is test coverage for the new alias. Consider adding a test that deserializes an EthCallMessage with the "input" field to ensure the alias works as expected:

#[test]
fn test_eth_call_message_input_alias() {
    // Test deserialization with "input" field
    let json_with_input = r#"{"to":"0x1234567890123456789012345678901234567890","input":"0x1234"}"#;
    let msg: EthCallMessage = serde_json::from_str(json_with_input).unwrap();
    assert!(msg.data.is_some());
    
    // Test deserialization with "data" field still works
    let json_with_data = r#"{"to":"0x1234567890123456789012345678901234567890","data":"0x1234"}"#;
    let msg: EthCallMessage = serde_json::from_str(json_with_data).unwrap();
    assert!(msg.data.is_some());
}

Also confirm that eth_estimateGas mentioned in the changelog uses the same EthCallMessage struct.