Store initialization action should have a type field

[roblox]

Store initialization action should have a type field

[LPGhatguy]

Ooh, that's an oversight.

Should the initial action go through middleware? If so, I wonder if it should be a string like @@INIT like Redux to make it serializable.

[AmaranthineCodices]

I'm honestly not sure. I think it depends on how crucial we consider the initialization action. Middleware can outright cancel an action (in my implementation, at least), or tamper with it in any number of ways. If the initialization action is considered a "absolutely must be dispatched" sort of thing, then no, the initialization action should not go through the middleware.

[LPGhatguy]

I dug through all of the conversations the Redux team had about their initial action dispatching and how it relates to middleware, and I don´t think it needs to run through middleware.

In the worst case, if your middleware needs some sort of state, you can instantiate it:

local middlewareInstance = middleware({ options })
local store = Store.new(reducer, nil, { middlewareInstance })

I'll go ahead and fix this, since it's pretty easy to patch up and shouldn't cause conflicts with any existing PRs.