Update to webp-hero 0.0.0-dev.26

[alexisbmills]

Update to the latest version of webp-hero.

[alexisbmills]

Currently blocked by error, after updating, when running demo:

WARNING in ./node_modules/webp-hero/libwebp/dist/webp.js
Module not found: Error: Can't resolve 'crypto' in 'ngx-webp-polyfill\node_modules\webp-hero\libwebp\dist'


ERROR in ./node_modules/webp-hero/libwebp/dist/webp.js
Module not found: Error: Can't resolve 'fs' in 'ngx-webp-polyfill\node_modules\webp-hero\libwebp\dist'

ERROR in ./node_modules/webp-hero/libwebp/dist/webp.js
Module not found: Error: Can't resolve 'path' in 'ngx-webp-polyfill\node_modules\webp-hero\libwebp\dist'

Possible workarounds include:

• package.json Angular error during compilation - Can't resolve 'crypto' iden3/snarkjs#17 (comment)
• monkey patching build config Webpack Config is stripping out Crypto angular/angular-cli#1548 (comment)

Both workarounds need to be implemented in the consumer application (not in ngx-webp-polyfill) which is not ideal :(

[chase-moskal]

@alexisbmills

it looks like you're using libwebp directly, so be sure to take a peek at the new webp-hero readme section about that

it's possible that instead of using
  webp-hero/libwebp/dist/webp.js

you may need to use
  webp-hero/libwebp/dist/webp.cjs.js

see webp-hero/libwebp/dist/

[alexisbmills]

Hi @chase-moskal.

Currently we go as deep as importing WebpMachine - import { WebpMachine } from 'webp-hero'; which has import {Webp} from "../libwebp/dist/webp.js" (https://github.com/chase-moskal/webp-hero/blob/master/source/webp-machine.ts#L2).

It's not ideal for this project to be using CJS, in terms of bundle size, although I haven't done a comparison of the ESM/CJS bundle results. Reference https://angular.io/guide/build#configuring-commonjs-dependencies.

[chase-moskal]

@alexisbmills oh, i see

this issue might be a regression in webp-hero. i think it may be this issue

i'm thinking in webp-hero, we'd need to add ENVIRONMENT = web somewhere like here in the dockerfile or maybe here in cmakelists

[chase-moskal]

@alexisbmills i may have found the problem. i reopened chase-moskal/webp-hero#40 about it and explained more in my post there