Pass Konami sequence element counts

[mjc]

Passes element counts, rather than byte sizes, when initializing the three footpad gesture sequences.

Why

konami_init expects a uint8_t element count. The old code passed sizeof(sequence), which is the array byte size. After a valid gesture was consumed, the state machine could therefore continue indexing past the end of the sequence.

Changes

• Pass the element count for the flywheel, headlights-on, and headlights-off sequences.
• Convert each derived count explicitly at the uint8_t API boundary.
• Assert at compile time that every fixed sequence fits that representation.

Scope

Only initialization of the three fixed gestures changes. Their contents, timing, and valid state-machine behavior are otherwise unchanged.

Hilariously, it seems possible to do a TAS-style physical reproduction of this but there is no way I am going to spend the time to try. I did get strange behavior after enough trying to trigger it manually though.

[Copilot]

Pull request overview

Fixes Konami footpad gesture initialization to pass sequence element counts (not byte sizes) into the Konami state machine, preventing potential out-of-bounds indexing after a gesture is consumed.

Changes:

• Update flywheel Konami gesture initialization to pass array element count.
• Update headlights-on Konami gesture initialization to pass array element count.
• Update headlights-off Konami gesture initialization to pass array element count.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[mjc]

Added array_size helper and addressed the other feedback.

[mjc]

second force push was just formatting.