build(pnp): use rollup to bundle the library (#3931)

* build(pnp): use rollup to bundle the library

* chore: remove unnecessary `typescript` peer dependency

* chore: change references to `webpack` to `rollup` and `esbuild`

* docs: remove outdated Babel comment

* chore: update gitignore

* chore: remove unnecessary `typescript` dependencies

* ci: remove leftover build monitoring

Author: merceyz

.github/workflows/integration-workflow.yml

@@ -153,7 +153,6 @@ jobs:
 
     - name: 'Build bundle & plugins'
       run: |
-        export BUILD_MONITORING_ENABLED=1
         node --version
         node ./scripts/run-yarn.js build:pnp:hook
         node ./scripts/run-yarn.js build:cli
@@ -167,8 +166,6 @@ jobs:
         node ./scripts/run-yarn.js build:plugin-version
         node ./scripts/run-yarn.js build:plugin-workspace-tools
       shell: bash
-      env:
-        DD_API_KEY: ${{ secrets.DD_API_KEY }}
 
     - uses: actions/upload-artifact@v2
       with:

.github/workflows/pr-smart-merge.yml

@@ -45,13 +45,13 @@ jobs:
 
           if git diff --name-only --diff-filter=U | grep packages/yarnpkg-pnp/sources/hook.js; then
             git checkout --theirs packages/yarnpkg-pnp/sources/hook.js
-            yarn build:pnp:cjs
+            yarn build:pnp:hook
             SHOULD_RUN_AN_INSTALL=1
           fi
 
           if git diff --name-only --diff-filter=U | grep packages/yarnpkg-pnp/sources/esm-loader/built-loader.js; then
             git checkout --theirs packages/yarnpkg-pnp/sources/esm-loader/built-loader.js
-            yarn build:pnp:esm
+            yarn build:pnp:hook
             SHOULD_RUN_AN_INSTALL=1
           fi
 

.gitignore

@@ -34,9 +34,6 @@ package.tgz
 # Those packages are built inline and the JS files must not be checked-in
 /packages/yarnpkg-libui/sources/**/*.js
 
-# We check-in the PnP hook because it would be heavy to regenerate it before each bundle build
-!/packages/yarnpkg-pnp/lib/hook.js
-
 /vscode-case-study
 
 .idea