Rewrite internals for v2 release (#83)

Addresses #15, #35, #36, #47, #51, and #59

Author: mcous

.eslintrc.yml

@@ -7,7 +7,7 @@ extends:
   - "plugin:@typescript-eslint/strict"
   - "plugin:import/recommended"
   - "plugin:import/typescript"
-  - "plugin:jsdoc/recommended-typescript"
+  - "plugin:jsdoc/recommended-typescript-error"
   - "plugin:n/recommended"
   - "plugin:promise/recommended"
   - "plugin:sonarjs/recommended"
@@ -27,49 +27,24 @@ env:
   es2021: true
   node: true
 
+rules:
+  unicorn/prefer-top-level-await: "off"
+  jsdoc/tag-lines: "off"
+
 overrides:
   - files: ["**/*.js"]
     rules:
       "@typescript-eslint/no-var-requires": "off"
-      "unicorn/prefer-module": "off"
+      unicorn/prefer-module: "off"
 
-  - files: ["test/**/*"]
-    env:
-      mocha: true
+  - files: ["**/__tests__/**/*"]
     rules:
+      "@typescript-eslint/no-non-null-assertion": "off"
+      "@typescript-eslint/no-unsafe-argument": "off"
+      "@typescript-eslint/no-unsafe-assignment": "off"
       sonarjs/no-duplicate-string: "off"
+      unicorn/no-useless-undefined: "off"
 
-rules:
-  n/no-unpublished-import: ["error", { "allowModules": ["@actions/core"] }]
-
-  # temporary rule overrides
-  # TODO(mc, 2023-04-12): fix problems, remove overrides, and remove  --quiet
-  prefer-const: "warn"
-  no-regex-spaces: "warn"
-  "@typescript-eslint/no-floating-promises": "warn"
-  "@typescript-eslint/no-unsafe-argument": "warn"
-  "@typescript-eslint/no-unsafe-assignment": "warn"
-  "@typescript-eslint/no-unsafe-call": "warn"
-  "@typescript-eslint/no-unsafe-member-access": "warn"
-  "@typescript-eslint/no-unsafe-return": "warn"
-  "@typescript-eslint/require-await": "warn"
-  "@typescript-eslint/restrict-plus-operands": "warn"
-  "@typescript-eslint/restrict-template-expressions": "warn"
-  "@typescript-eslint/unbound-method": "warn"
-  n/no-missing-import: "warn"
-  n/no-missing-require: "warn"
-  n/no-process-exit: "warn"
-  unicorn/better-regex: "warn"
-  unicorn/catch-error-name: "warn"
-  unicorn/import-style: "warn"
-  unicorn/no-array-push-push: "warn"
-  unicorn/no-process-exit: "warn"
-  unicorn/no-useless-undefined: "warn"
-  unicorn/prefer-export-from: "warn"
-  unicorn/prefer-module: "warn"
-  unicorn/prefer-node-protocol: "warn"
-  unicorn/prefer-optional-catch-binding: "warn"
-  unicorn/prefer-spread: "warn"
-  unicorn/prefer-top-level-await: "warn"
-  unicorn/prevent-abbreviations: "warn"
-  unicorn/text-encoding-identifier-case: "warn"
+  - files: ["src/action/**/*"]
+    rules:
+      n/no-unpublished-import: "off"

.github/workflows/ci-cd.yml

@@ -13,23 +13,8 @@ on:
     - cron: "25 12 * * 1"
 
 jobs:
-  lint:
-    name: Lint
-    runs-on: ubuntu-latest
-    timeout-minutes: 10
-
-    steps:
-      - name: Checkout source
-        uses: actions/checkout@v3
-
-      - name: Install Node.js and dependencies
-        uses: ./.github/actions/setup
-
-      - name: Run linters
-        run: npm run lint
-
   test:
-    name: Run tests using Node ${{ matrix.node }} on ${{ matrix.os }}
+    name: Unit tests / Node.js ${{ matrix.node-version }} / ${{ matrix.os }}
     runs-on: ${{ matrix.os }}
     timeout-minutes: 10
     strategy:
@@ -38,9 +23,8 @@ jobs:
         os:
           - ubuntu-latest
           - macos-latest
-          # TODO(mc, 2023-04-07): investigate and fix coverage-related failures on Windows
-          # - windows-latest
-        node:
+          - windows-latest
+        node-version:
           - 16
           - 18
 
@@ -51,10 +35,7 @@ jobs:
       - name: Install Node.js ${{ matrix.node }} and dependencies
         uses: ./.github/actions/setup
         with:
-          node-version: ${{ matrix.node }}
-
-      - name: Build the code
-        run: npm run build
+          node-version: ${{ matrix.node-version }}
 
       - name: Run tests
         run: npm run coverage
@@ -78,7 +59,7 @@ jobs:
           parallel-finished: true
 
   build:
-    name: Build
+    name: Build and lint
     runs-on: ubuntu-latest
     timeout-minutes: 10
 
@@ -92,18 +73,33 @@ jobs:
       - name: Build
         run: npm run build
 
+      - name: Verify no un-staged changes
+        run: |
+          git status --porcelain
+          git diff-files --quiet
+
+      - name: Run lints
+        run: npm run lint
+
       - name: Upload publish artifact
         uses: actions/upload-artifact@v3
         with:
           name: publish-artifact
           path: lib
 
   e2e:
-    name: Run end-to-end tests
+    name: E2E tests / Node.js ${{ matrix.node-version }}
     runs-on: ubuntu-latest
     timeout-minutes: 10
     needs: build
 
+    strategy:
+      fail-fast: false
+      matrix:
+        node-version:
+          - 16
+          - 18
+
     services:
       verdaccio:
         image: verdaccio/verdaccio:5
@@ -117,6 +113,7 @@ jobs:
       - name: Install Node.js and dependencies
         uses: ./.github/actions/setup
         with:
+          node-version: ${{ matrix.node-version }}
           install-command: npm install --production
 
       - name: Download publish artifact
@@ -130,20 +127,20 @@ jobs:
         shell: bash
         run: |
           echo "token=$(./e2e/00-login.sh)" >> "$GITHUB_OUTPUT"
-          echo "package=$(./e2e/01-setup-package.sh ./e2e/fixture 0.0.1)" >> "$GITHUB_OUTPUT"
+          echo "package=$(./e2e/01-setup-package.sh ./e2e/fixture/cool\ package 0.0.1)" >> "$GITHUB_OUTPUT"
 
       - name: Run CLI end-to-end test
         shell: bash
         env:
           TOKEN: ${{ steps.setup.outputs.token }}
           PACKAGE: ${{ steps.setup.outputs.package }}
         run: |
-          ./e2e/02-publish.sh ${PACKAGE} ${TOKEN}
-          ./e2e/03-verify.sh ${PACKAGE}
-          ./e2e/02-publish.sh ${PACKAGE} ${TOKEN}
-          ./e2e/01-setup-package.sh ${PACKAGE} 0.0.2
-          ./e2e/02-publish.sh ${PACKAGE} ${TOKEN}
-          ./e2e/03-verify.sh ${PACKAGE}
+          ./e2e/02-publish.sh "${PACKAGE}" ${TOKEN}
+          ./e2e/03-verify.sh "${PACKAGE}"
+          ./e2e/02-publish.sh "${PACKAGE}" ${TOKEN}
+          ./e2e/01-setup-package.sh "${PACKAGE}" 0.0.2
+          ./e2e/02-publish.sh "${PACKAGE}" ${TOKEN}
+          ./e2e/03-verify.sh "${PACKAGE}"
 
       - id: action-no-publish
         name: Publish with already published version
@@ -154,14 +151,14 @@ jobs:
           token: ${{ steps.setup.outputs.token }}
 
       - name: Check action output
-        if: ${{ steps.action-no-publish.outputs.type != 'none' }}
+        if: ${{ steps.action-no-publish.outputs.type }}
         run: |
-          echo "::error::Expected release type to be 'none', got '${{ steps.action-no-publish.outputs.type }}'"
+          echo "::error::Expected release type to be '', got '${{ steps.action-no-publish.outputs.type }}'"
           exit 1
 
       - name: Create new version for Action end-to-end test
         shell: bash
-        run: ./e2e/01-setup-package.sh ${{ steps.setup.outputs.package }} 0.0.3
+        run: ./e2e/01-setup-package.sh "${{ steps.setup.outputs.package }}" 0.0.3
 
       - id: action-publish
         name: Publish a new version
@@ -179,11 +176,10 @@ jobs:
 
   deploy:
     if: ${{ github.ref == 'refs/heads/main' }}
-    name: Publish to NPM
+    name: Publish
     runs-on: ubuntu-latest
     timeout-minutes: 10
     needs:
-      - lint
       - test
       - build
       - e2e