chore(npm): do not npmignore angular/dist + unlock npm (#805)

https://coveord.atlassian.net/browse/CDX-1010

<!-- For Coveo Employees only. Fill this section.

CDX-1010

-->

## Proposed changes

I discussed with a developer from the npm team here
npm/npm-packlist#106 and got a good
understanding of their philosophy moving forward with the concept of
workspaces and their multi-level ignore files.

In short, npm will try to acquire an ignorefile at each directory level
and concatenate them. The peculiarity (at least IMO), is that if it
acquire a npmignore first, this will have no incidence for the
acquisition of the other ignorefile. So, we can end up with a mix of
`gitignore` and `npmignore`. I discussed this behavior with the npm
team, and they are kinda stuck, so its their least worst option here, so
we just have to adapt and make it work.

This is what this PR is about.

## Testing

 - Ran `npm publish --dry-run` on every package and checked them out.
 
 ## Prerequisites
 
 - [x] npm/npm-packlist#108
 - [ ] `npm-cli` released with ☝️

.github/actions/e2e-clean/action.yml

@@ -11,10 +11,6 @@ runs:
       with:
         node-version: 16
         cache: 'npm'
-    - name: Install locked NPM version.
-      shell: bash
-      # TODO: CDX-1010 | Remove in CDX-1010.
-      run: npm i -g npm@8.10.0
     - run: npm ci
       shell: bash
     - name: Decrypt config

.github/actions/e2e-run/action.yml

@@ -42,10 +42,6 @@ runs:
     - name: Install Mitmproxy through pip
       shell: bash
       run: pip install -r packages/cli-e2e/mitmproxy/requirements.txt
-    - name: Install locked NPM version.
-      shell: bash
-      # TODO: CDX-1010 | Remove in CDX-1010.
-      run: npm i -g npm@8.10.0
     - name: Decrypt config
       shell: bash
       working-directory: packages/cli-e2e

.github/workflows/build.yml

@@ -36,9 +36,7 @@ jobs:
         with:
           node-version: 16
           cache: 'npm'
-      - name: Install locked NPM version.
-        # TODO: CDX-1010 | Remove in CDX-1010.
-        run: npm i -g npm@8.10.0
+
       - name: Setup repo
         run: npm ci
       - name: Check linting
@@ -66,9 +64,6 @@ jobs:
         with:
           node-version: ${{matrix.node}}
           cache: 'npm'
-      - name: Install locked NPM version.
-        # TODO: CDX-1010 | Remove in CDX-1010.
-        run: npm i -g npm@8.10.0
       - name: Setup repo
         run: npm ci
       - name: Build
@@ -89,9 +84,6 @@ jobs:
         with:
           node-version: ${{matrix.node}}
           cache: 'npm'
-      - name: Install locked NPM version.
-        # TODO: CDX-1010 | Remove in CDX-1010.
-        run: npm i -g npm@8.10.0
       - uses: ./.github/actions/e2e-login
         id: setup
   e2e-setup-verdaccio:
@@ -126,9 +118,6 @@ jobs:
         with:
           path: packages\cli-e2e\verdaccio
           key: verdaccio-${{matrix.os}}-${{ github.sha }}-${{ github.run_attempt }}
-      - name: Install locked NPM version.
-        # TODO: CDX-1010 | Remove in CDX-1010.
-        run: npm i -g npm@8.10.0
       - run: npm ci
       - name: Build
         run: npm run build

.github/workflows/daily-e2e.yml

@@ -39,9 +39,7 @@ jobs:
         with:
           node-version: ${{matrix.node}}
           cache: 'npm'
-      - name: Install locked NPM version.
-        # TODO: CDX-1010 | Remove in CDX-1010.
-        run: npm i -g npm@8.10.0
+
       - name: Install Coveo CLI
         run: npm i -g @coveo/cli@latest
       - uses: ./.github/actions/e2e-login

.github/workflows/release.yml

@@ -60,8 +60,6 @@ jobs:
         with:
           node-version: '16'
           registry-url: 'https://registry.npmjs.org'
-      # TODO: CDX-1010 | Remove in CDX-1010.
-      - run: npm i -g npm@8.10.0
       - name: Install dependencies
         run: npm ci
       - name: Setup credentials

packages/angular/.npmignore

@@ -1,2 +1,4 @@
 # Ignores TypeScript files, but keeps definitions.
 !*.d.ts
+# Do not ignore dist.
+!dist

packages/cli/core/src/lib/flags/platformCommonFlags.ts

@@ -9,7 +9,7 @@ import {
 export const withRegion = (withDefault = true) => ({
   region: Flags.string({
     char: 'r',
-    options: Object.entries(Region).flatMap(regionEntry=>regionEntry),
+    options: Object.entries(Region).flatMap((regionEntry) => regionEntry),
     parse: (input) => Promise.resolve(input.toLowerCase()),
     default: withDefault ? DEFAULT_REGION : undefined,
     description: