Question RE: adding/removing hooks... about react-refresh-webpack-plugin

[pmmmwh]

Hey! Thanks for the issue.

I just shipped v0.1.0 (on npm) - I tested with the latest master and I wasn't able to reproduce what you're mentioning. Can you try it out and see if it works?

[drather19]

I tried again with the latest released version and on master (nice overlay!), but do still see things error out when I add/remove hooks. Sorry for not adding a simple test, but simply adding/remove the following from a React.FC triggers the error overlay:

const callback = React.useCallback(() => {}, []);

It's not a big deal at this point (just an extra reload I can trigger by making more edits). I was mainly curious about whether this is the expected normal behavior.

Thanks!

[pmmmwh]

I tried again with the latest released version and on master (nice overlay!), but do still see things error out when I add/remove hooks. Sorry for not adding a simple test, but simply adding/remove the following from a React.FC triggers the error overlay:

const callback = React.useCallback(() => {}, []);

It's not a big deal at this point (just an extra reload I can trigger by making more edits). I was mainly curious about whether this is the expected normal behavior.

Thanks!

I'll have to check if I did something wrong. I am pretty sure state/reducer/refs work, let me check.

[gaearon]

I was mainly curious about whether this is the expected normal behavior.

No

[pmmmwh]

I tried again with the latest released version and on master (nice overlay!), but do still see things error out when I add/remove hooks. Sorry for not adding a simple test, but simply adding/remove the following from a React.FC triggers the error overlay:

const callback = React.useCallback(() => {}, []);

It's not a big deal at this point (just an extra reload I can trigger by making more edits). I was mainly curious about whether this is the expected normal behavior.

Thanks!

I couldn't reproduce this in a normal webpack app. I guess it might have something to do with integration between this and CRA. #7

[drather19]

As mentioned in #25, I can resolve this in my CRA app by setting webpackConfig.optimization.runtimeChunk = false; via craco. Now everything seems to be working like a charm!

[pmmmwh]

Fixed in v0.1.3 🎉