Input machine should not panic on invalid state transfer, especially in case of wormhole send. about magic-wormhole.rs HOT 5 CLOSED

Alternatively if we intend to keep input machine as is (meaning it will panic) then input machine should also be started when core is started.

from magic-wormhole.rs.

But note that if we try to do Input machine start then we need to integrate with Input machine every where. Else panic just moves to different place :)

from magic-wormhole.rs.

Yeah, I left a reminder to myself (the TODO comment on want_nameplate()) that turned out to be involved: the wordlist can arrive in states for which we don't want it. In fact, if set_code() is called, it can arrive before we even set the nameplate (although maybe we could fix that, by being more cautious about how we emit the other events).

I'm working on this now. Input should accept (and store) the wordlist in any state. I'm wrestling with whether to implement this by copying all the states (so we'd have a WantNameplateNoNameplatesNoWordlist, WantNameplateNoNameplatesHaveWordlist, WantNameplateHaveNameplatesNoWordlist, etc.. gross), or having a separate Option<Wordlist> on the Input struct (which I think copyninja had recommended before.. you were right :). I'll probably go with the latter.

I'm also changing the Boss machine to have a distinct Start event, because the most immediate problem with io/blocking is that it never initiates the websocket connection, because it isn't currently calling start. Since Start returns Actions, it should really be called in the same way as any other API call (so the caller is prepared to act upon the vector of actions that come back), so I'm wiring it up as a regular API event instead of a distinct API call. For io/blocking, after it sets up the thread and the channel, it'll send() a Start through the API channel, which should kick everything else off.

The application won't see this start event or API: it will be hidden inside the IO layer.

from magic-wormhole.rs.

I just landed the Start event. I'll work on the Input/wordlist stuff later today.

from magic-wormhole.rs.

During my refactorings of the core, the input listing feature got cut. Sorry about that. It'll probably have to be re-implemented completely from scratch again, so this issue is obsolete now.

from magic-wormhole.rs.