Merge pull request #835 from reviewdog/introduce-bundle-script

introduce bundle script

Author: shogo82148

.github/workflows/check-dist.yml

@@ -0,0 +1,56 @@
+# In TypeScript actions, `dist/` is a special directory. When you reference
+# an action with the `uses:` property, `dist/index.js` is the code that will be
+# run. For this project, the `dist/index.js` file is transpiled from other
+# source files. This workflow ensures the `dist/` directory contains the
+# expected transpiled code.
+#
+# If this workflow is run from a feature branch, it will act as an additional CI
+# check and fail if the checked-in `dist/` directory does not match what is
+# expected from the build.
+name: Check dist/
+
+on:
+  push:
+    branches:
+      - main
+    paths-ignore:
+      - "**.md"
+  pull_request:
+    paths-ignore:
+      - "**.md"
+  workflow_dispatch:
+
+jobs:
+  check-dist:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
+
+      - uses: actions/setup-node@53b83947a5a98c8d113130e565377fae1a50d02f # v6.3.0
+        with:
+          node-version-file: ".node-version"
+          cache: "npm"
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Rebuild the dist/ directory
+        run: |
+          npm run bundle
+
+      - name: Compare the expected and actual dist/ directories
+        run: |
+          if [ "$(git diff --ignore-space-at-eol --text dist/ | wc -l)" -gt "0" ]; then
+            echo "Detected uncommitted changes after build.  See status below:"
+            git diff
+            exit 1
+          fi
+        id: diff
+
+      # If index.js was different than expected, upload the expected version as an artifact
+      - uses: actions/upload-artifact@bbbca2ddaa5d8feaa63e36b76fdaad77386f024f # v7.0.0
+        if: ${{ failure() && steps.diff.conclusion == 'failure' }}
+        with:
+          name: dist
+          path: dist/

.github/workflows/reviewdog.yml

@@ -163,7 +163,7 @@ jobs:
 
       - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
         with:
-          node-version-file: "package.json"
+          node-version-file: ".node-version"
           cache: "npm"
       - run: npm ci
       - uses: reviewdog/action-eslint@556a3fdaf8b4201d4d74d406013386aa4f7dab96 # v1.34.0

.github/workflows/test.yml

@@ -18,19 +18,9 @@ jobs:
       - name: Set up Node.js
         uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4.4.0
         with:
-          node-version-file: "package.json"
+          node-version-file: ".node-version"
           cache: npm
       - name: Install Dependencies
         run: npm ci
       - name: Run Test
         run: npm run test
-      - name: Ensure dist/ folder is up-to-date
-        if: ${{ runner.os == 'Linux' }}
-        shell: bash
-        run: |
-          npm run package
-          if [ "$(git diff --ignore-space-at-eol | wc -l)" -gt "0" ]; then
-            echo "Detected uncommitted changes after build.  See status below:"
-            git diff
-            exit 1
-          fi

.node-version

@@ -0,0 +1 @@
+24.13.0

.prettierrc.json

@@ -1,3 +1,3 @@
 {
-    "printWidth": 120
+  "printWidth": 120
 }

action.yml

@@ -34,7 +34,7 @@ inputs:
       Otherwise, exit code 1 for reviewdog if it finds at least 1 issue with severity greater than or equal to the given level.
       Possible values: [none,any,info,warning,error]
       Default is `none`.
-    default: 'none'
+    default: "none"
   fail_on_error:
     description: |
       Deprecated, use `fail_level` instead.
@@ -72,7 +72,7 @@ inputs:
 
 runs:
   using: "node24"
-  main: "dist/index.cjs"
+  main: "dist/index.js"
 branding:
   icon: "check-circle"
   color: "blue"