ERROR in ./node_modules/image-size/lib/index.js

Plotly.js version: 1.58.4 Platform: mac bis sur

When upgrading plotly from 1.52.2 to 1.58.4 a new error appears when trying to run it in a react webpage:

ERROR in ./node_modules/image-size/lib/index.js
Module not found: Error: Can't resolve 'fs' in '/local/home/xxx/node_modules/image-size/lib'
 @ ./node_modules/image-size/lib/index.js 3:9-22
 @ ./node_modules/plotly.js/src/traces/image/helpers.js
 @ ./node_modules/plotly.js/src/traces/image/calc.js
 @ ./node_modules/plotly.js/src/traces/image/index.js
 @ ./node_modules/plotly.js/lib/image.js
 @ ./node_modules/plotly.js/lib/index.js
 @ ./src/util/ui/MultiChart.js
 @ ./src/xxx/xxx.js
 @ ./src/App.js
 @ ./src/index.development.js

Is there some new dependency which needs to be added?

[bishonen]

Wasn’t quite sure how you meant to change the encoding (in the header of the index.html ?). Therefore I searched, found and followed the advice from someone who already had this issue. They used the minified version which turned out to be working fine.

With this in place, I can gladly confirm that the latest build does not throw any ‘fs’ related errors anymore!

Compiled successfully.

Thanks to all for the quick turnaround. This is in part what makes plotly a great library (on which one can rely upon). Thanks

[archmoj]

All of my initial guesses proved false. The dist version of image-size has require('fs'). I can’t find that Plotly runs the brfs transform when bundling, which would browserify fs calls:

https://github.com/plotly/plotly.js/blob/3a501ef02964c6eb7c5f5e790a0696fab3738da2/tasks/util/browserify_wrapper.js#L44

In short, it’s not clear to me.

@rreusser simply FYI we are using image-size@0.7.5 not image-size@@0.9.3 which liked above. Suggestions are welcome : )