isAddress() does not check checksum if passed-in address is lower-case

[kunstmusik]

Check existing issues

• I checked there isn't already an issue for the bug I encountered.

Viem Version

2.21.7

Current Behavior

If an address is all lower-case, it will not have it's checksum checked. The problematic code is here:

viem/src/utils/address/isAddress.ts

Line 33 in 2159ca0

if (address.toLowerCase() === address) return true

Expected Behavior

Passing in any address will have checksum checked when strict is true.

Steps To Reproduce

1. Pass in eth address with .toLowerCase() called upon it.

Link to Minimal Reproducible Example

https://stackblitz.com/edit/viem-getting-started-x9vfv5fu?file=index.ts,package.json

Anything else?

No response

[jxom]

This is correct behavior.

[kunstmusik]

How is this the correct behavior? Docs for the function say "Enables strict mode. Whether or not to compare the address against its checksum." and here it is bypassing that with strict = true. If the behavior is correct, it would be worth fixing the documentation to explain this case.

[github-actions]

This issue has been locked since it has been closed for more than 14 days.

If you found a concrete bug or regression related to it, please open a new bug report with a reproduction against the latest Viem version. If you have any questions or comments you can create a new discussion thread.