Edit InterpolateHtmlPlugin hook (#6448)

[GuiHash]

Fixes#6448

[olessavluk]

Hi @GuiHash
I have tried to use your patch but result is the same. Could you please create example repo and steps on how to test your change?

What I have done:

1. yarn add file:/Users/olessavluk/tmp/create-react-app/packages/react-dev-utils/ - install patched package
2. yarn why react-dev-utils - to verify that I have only one package installed
3. open file less ./node_modules/react-dev-utils/InterpolateHtmlPlugin.js and verify that .afterTemplateExecution.tap( is present in this file
4. run yarn buid
5. less ./build/index.html and still see old value <script>var isProduction=!1</script>

[GuiHash]

Hi @olessavluk,

No need to create an example repo.
Here are the steps to test the changes :

1. clone repository : git clone https://github.com/GuiHash/create-react-app
2. go to directory cd create-react-app/packages/react-dev-utils
3. create link for react-dev-utils : yarn link
4. go to your bugged repository : cd your-repo
5. link react-dev-utils : yarn link "react-dev-utils"
6. run yarn build
7. 🚀

[olessavluk]

Hi @GuiHash

Thanks for providing steps to test this package.
I have created test repo and tried to this steps, but got an error - https://asciinema.org/a/fCpTGKHHOb6amRLDTUmrOySe7

Error: Cannot find module 'react-dev-utils/ForkTsCheckerWebpackPlugin'

~/t/cra-env-interpolation (master|✔) $ yarn build
yarn run v1.13.0
$ react-scripts build
internal/modules/cjs/loader.js:584
throw err;
^
Error: Cannot find module 'react-dev-utils/ForkTsCheckerWebpackPlugin'
at Function.Module._resolveFilename (internal/modules/cjs/loader.js:582:15)
at Function.Module._load (internal/modules/cjs/loader.js:508:25)
at Module.require (internal/modules/cjs/loader.js:637:17)
at require (internal/modules/cjs/helpers.js:22:18)
at Object.<anonymous> (/Users/olessavluk/tmp/cra-env-interpolation/node_modules/react-scripts/config/webpack.config.js:32:36)
at Module._compile (internal/modules/cjs/loader.js:701:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
at Module.load (internal/modules/cjs/loader.js:600:32)
at tryModuleLoad (internal/modules/cjs/loader.js:539:12)
at Function.Module._load (internal/modules/cjs/loader.js:531:3)
at Module.require (internal/modules/cjs/loader.js:637:17)
at require (internal/modules/cjs/helpers.js:22:18)
at Object.<anonymous> (/Users/olessavluk/tmp/cra-env-interpolation/node_modules/react-scripts/scripts/build.js:39:23)
at Module._compile (internal/modules/cjs/loader.js:701:30)
at Object.Module._extensions..js (internal/modules/cjs/loader.js:712:10)
at Module.load (internal/modules/cjs/loader.js:600:32)
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

Also when I tried to use your changes via yarn add file://... result have not changed, variables still injected after optimizations.

[GuiHash]

@olessavluk,

your react-scripts dependency must be in version 2.1.5 to work !

[olessavluk]

@GuiHash yes, now it works, index.html is correctly optimized. Thanks for the explanation!

[iansu]

Thanks!