Hyperapp: More than one action breaks actions!

[zaceno]

Sorry guys, I know we're chomping at the bit 1.0, but I found this really wierd & serious bug...

See this codepen: https://codepen.io/zaceno/pen/JMWjwQ

Try typing in the input field. Notice that it works. Now: uncomment L12 ( otherAction: _ => ({otherVal: true}))

💥

...the input field breaks. From my debugging, it looks like the set action isn't called even.

(... if this is a user error, I'm going to be very embarrassed...)

[mietennet]

if i'm change the order of the actions it works again... wired

[SkaterDad]

Definitely looks like a bug with 0.18.1.

If you change the script reference to https://unpkg.com/[email protected] it works.

[whaaaley]

Looks like the otherAction is getting called on accident somehow.
I added a state tree to this example. We can see it add otherValue: https://codepen.io/dustindowell/pen/aEJObB

[selfup]

🤔

[okwolf]

I noticed that the counter example now only calls the up action. Adding a NOP action breaks the example entirely 😮

Switching back to 0.18.0 fixes both of these issues for me... 🤔

[jorgebucaran]

@zaceno @okwolf @whaaaley @selfup Can you reproduce this locally or could it be only a CodePen issue?

My local examples work as usual. 🤔

[jorgebucaran]

I think this could be a bug in uglify-js, we're going to need to downgrade it. 🎓

[okwolf]

@JorgeBucaran confirmed that it works with the non-uglified bundle for 0.18.1. How bizarre...

[jorgebucaran]

Maybe related. https://github.com/mishoo/UglifyJS2/issues/2663

[jorgebucaran]

Fixed in 0.18.3! 🎉