fix(common): fix elementary type state mutability handling

[dk1a]

This is another old bug that never came up apparently
I only fix this for worldgen, system libraries still don't support this, see the TODO
(similar to #3660)

For system libs this can probably be solved by hacking renderAbiDecode to check for /payable(\[.*\])?/
A better solution would require refactoring contractToInterface to return more nuanced parameter data (types,names separately)

Anyways address payable[] isn't important to support, this change is more about removing incorrect assumptions from contractToInterface, and simplifying flattenTypeName

[changeset-bot]

⚠️ No Changeset found

Latest commit: 5994048

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR