[P0] 🔥 core issue #890 needs to be addressed in sdk 🔥

[frankiebee]

An issue was misunderstood involving entropy-core issue #890 and a fix was attempted in the cli. The true fix needs to happen in the sdk. This will require a hot-fix patch into cli.

Closing this issue should require preserving the original subgroup list and reversing this order. Gotch-yas: i will not be requiring a full test suite to close this issue since their are to many places this could go wrong.

• The math could be a NaN how do we pick an index from their. Do we throw an error? one could throw an error but their isnt really anything the user could do to fix this error 😢 other then try again (i guess?)
• The subgroups could be reorder in transit (i.e; polkadot-js or substrate pulling from storage and formating on it's way to/threw rpc) but we wont know that till we try to sign.
• it's not happening on every signature which might suggest the math for picking index but really thats a recursive thought.

Conclusion try to resubmit once in reverse if it throws again throw error with the chosen index

[frankiebee]

seen in the wild here: https://discord.com/channels/852509982582767677/852509983085690912/1250819152613212234

that first error is a misunderstanding of the problem in the cli