Rewrite in ReScript

.eslintrc.json

@@ -1,11 +1,15 @@
 {
   "root": true,
-  "parser": "@typescript-eslint/parser",
-  "plugins": ["jest", "@typescript-eslint"],
-  "extends": [
-    "eslint:recommended",
-    "plugin:@typescript-eslint/recommended",
-    "prettier"
-  ],
-  "env": { "node": true, "jest/globals": true }
+  "parserOptions": {
+    "sourceType": "module",
+    "ecmaVersion": 2023, // https://node.green/#ES2023
+    "ecmaFeatures": {
+      "impliedStrict": true
+    }
+  },
+  "extends": ["eslint:recommended", "prettier"],
+  "env": {
+    "node": true,
+    "es2023": true // https://node.green/#ES2023
+  }
 }

.github/workflows/pull-request.yml

@@ -10,7 +10,7 @@ name: PR status checks
 
 jobs:
   lint:
-    name: Lint
+    name: Formatting
     runs-on: ubuntu-latest
     steps:
       - name: Checkout
@@ -19,25 +19,32 @@ jobs:
         uses: actions/setup-node@v4
         with:
           node-version: "20"
-      - run: npm ci
+      - run: npm ci --include dev
+      - run: npm run format
+      - name: Check if any files changed
+        run: if [ -n "$(git status --untracked-files --porcelain)" ]; then (exit 1); fi
+        shell: bash
+
+  test:
+    name: Tests
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Install node
+        uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+      - run: npm ci --include dev
       - run: npm run build
-      - run: npm run lint
-      - run: npm run format-check
+      - run: npm run test
 
   dogfood:
-    name: Dogfood the action
+    name: Dogfooding
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v4 # To run the version of the action from the PR.
 
-      - name: package*.json updated
-        uses: ./
-        with:
-          file-pattern: |
-            package.json
-            package-lock.json
-          skip-label: "skip package*.json"
-
       - name: package-lock.json updated
         uses: ./
         with:
@@ -48,6 +55,13 @@ jobs:
       - name: dist/index.js updated
         uses: ./
         with:
-          prereq-pattern: "src/*.ts"
+          prereq-pattern: "src/*.res"
           file-pattern: "dist/index.js"
           skip-label: "skip packaging"
+
+      - name: README.md updated
+        uses: ./
+        with:
+          prereq-pattern: "action.yml"
+          file-pattern: "README.md"
+          skip-label: "skip README"

.gitignore

@@ -104,3 +104,7 @@ typings/
 
 # Files built by tsc
 out/
+
+/lib/
+.bsb.lock
+**/*.res.mjs

.vscode/settings.json

@@ -1,22 +1,3 @@
 {
-  "[javascript]": {
-    "editor.formatOnSave": true,
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
-  },
-  "[json]": {
-    "editor.formatOnSave": true,
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
-  },
-  "[markdown]": {
-    "editor.formatOnSave": true,
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
-  },
-  "[typescript]": {
-    "editor.formatOnSave": true,
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
-  },
-  "[yaml]": {
-    "editor.formatOnSave": true,
-    "editor.defaultFormatter": "esbenp.prettier-vscode"
-  }
+  "rescript.settings.askToStartBuild": false
 }

.vscode/tasks.json

@@ -1,16 +1,28 @@
 {
-  "version": "2.0.0",
-  "tasks": [
-    {
-      "type": "typescript",
-      "tsconfig": "tsconfig.json",
-      "option": "watch",
-      "problemMatcher": ["$tsc-watch"],
-      "group": {
-        "kind": "build",
-        "isDefault": true
-      },
-      "label": "tsc: watch - tsconfig.json"
-    }
-  ]
+	"version": "2.0.0",
+	"tasks": [
+		{
+			"type": "npm",
+			"script": "build",
+			"problemMatcher": [],
+			"label": "npm: build",
+			"detail": "rescript build",
+			"group": {
+				"kind": "build",
+				"isDefault": true
+			}
+		},
+		{
+			"type": "npm",
+			"script": "test",
+			"dependsOn": "npm: build",
+			"group": {
+				"kind": "test",
+				"isDefault": true
+			},
+			"problemMatcher": [],
+			"label": "npm: test",
+			"detail": "pta tests/*.mjs"
+		}
+	]
 }

README.md

@@ -22,7 +22,7 @@ file-pattern: |
 acts as two patterns: `package.json` and `package-lock.json`. Success occurs if
 _any_ of the patterns match.
 
-### `prereq-pattern`
+### `prereq-pattern` (optional)
 
 A pre-requisite glob pattern that, if specified, will cause the action to proceed
 only if the pattern matches. If a match isn't found then the action is considered
@@ -31,17 +31,17 @@ successful.
 See the notes about patterns for `file-pattern` for details on how matching
 occurs and the flexibility in specifying the pattern.
 
-### `skip-label`
+### `skip-label` (optional)
 
 The name of a label to forcibly skip the changed file check.
 
-### `failure-message`
+### `failure-message` (optional)
 
 The message to emit when the check fails. All other inputs can be specified in
 the message using `${}` syntax, e.g. `${file-pattern}` for the `file-pattern`
 input. All values will be quoted for easy identification of any whitespace.
 
-### `token`
+### `token` (optional)
 
 A GitHub auth token to use for private repositories. Falls back to anonymous access if
 not provided. Usually you want to use `${{ secrets.GITHUB_TOKEN }}` for this.

__tests__/gh.test.ts → __tests__/gh.test.mjs

@@ -1,6 +1,14 @@
-import { ImportMock } from "ts-mock-imports";
 import * as github from "@actions/github";
-import * as gh from "../src/gh";
+import {
+  afterAll,
+  afterEach,
+  beforeEach,
+  describe,
+  expect,
+  test,
+  vi,
+} from "vitest";
+import * as gh from "../src/GH.res.mjs";
 
 test("gh.pullRequestEvent() return undefined by default", () => {
   expect(gh.pullRequestPayload()).toBeUndefined();
@@ -15,12 +23,10 @@ describe("Stub github.context w/ a pull_request payload", () => {
     },
   };
 
-  beforeEach(() => {
-    ImportMock.mockOther(github, "context", context);
-  });
-  afterEach(() => ImportMock.restore());
+  afterEach(() => vi.restoreAllMocks());
 
   test("pull_request context is returned by gh.pullRequestEvent()", () => {
+    vi.spyOn(github, "context", "get").mockReturnValue(context);
     expect(gh.pullRequestPayload()).toEqual(context.payload);
   });
 });
@@ -33,12 +39,11 @@ describe("Stub github.context w/ a 'push' payload", () => {
       repository: { name: "test", owner: { login: "anonymous" } },
     },
   };
-  beforeEach(() => {
-    ImportMock.mockOther(github, "context", context);
-  });
-  afterEach(() => ImportMock.restore());
+
+  afterEach(() => vi.restoreAllMocks());
 
   test("'undefined' is returned by gh.pullRequestEvent()", () => {
+    vi.spyOn(github, "context", "get").mockReturnValue(context);
     expect(gh.pullRequestPayload()).toBeUndefined();
   });
 });

__tests__/main.test.ts → __tests__/main.test.mjs

@@ -1,4 +1,5 @@
-import * as main from "../src/main";
+import { describe, expect, test } from "vitest";
+import * as main from "../src/Main.res.mjs";
 
 describe("repr()", () => {
   test("output quotes strings", () => {

action.yml

@@ -12,9 +12,11 @@ inputs:
   prereq-pattern:
     description: "Glob pattern that guards whether the action will run"
     required: false
+    default: "**"
   skip-label:
     description: "Label to trigger skipping the check"
     required: false
+    default: ""
   failure-message:
     description: "String template used when the check fails"
     required: false