Fix BytesN parsing when using valid hex values.

[fnando]

What

Properly parse BytesN when using valid hex values.

Why

Fix #2245.

Known limitations

N/A

[Copilot]

Pull request overview

This PR fixes a bug where BytesN parsing fails when using valid all-digit hexadecimal values. The issue occurred because when serde_json::from_str successfully parsed an all-digit hex string as a JSON number, it wasn't being converted back to a string for Bytes and BytesN types, leading to incorrect parsing downstream.

Changes:

• Added ScType::Bytes and ScType::BytesN(_) to the list of types that should always be treated as strings in the success branch of from_string() method
• Added a new test case test_bytesn_32_conversion_all_digit_hex to verify the fix works for the specific failing case from issue #2245

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

[leighmcculloch]

Looks good, one comment inline because this shadows a code path that becomes dead and wondering if there's an error relating to that.