Merge pull request #16 from AurorNZ/esbuild_esm

Generate build with esm

.gitattributes

@@ -1 +1,3 @@
-* text=auto eol=lf
+* text=auto eol=lf
+
+dist/** -diff linguist-generated=true

.github/workflows/pull-request-verification.yml

@@ -13,11 +13,21 @@ jobs:
   build:
     runs-on: ubuntu-latest
     steps:
-    - uses: actions/checkout@v3
+    - uses: actions/checkout@v4
+      with:
+        fetch-depth: 0 # Need history for changelog generation
     - uses: volta-cli/action@v4
     - run: |
-        pnpm install
+        pnpm i
         pnpm run all
+    # We need to make sure the checked-in `index.mjs` actually matches what we expect it to be.
+    - name: Compare the expected and actual dist/ directories
+      run: |
+        if [ "$(git diff --ignore-space-at-eol dist/ | wc -l)" -gt "0" ]; then
+          echo "Detected uncommitted changes after build.  See status below:"
+          git diff
+          exit 1
+        fi
 
   test-inline:
     runs-on: ubuntu-latest

__tests__/csv-escape.test.ts

@@ -1,6 +1,6 @@
 import {describe, expect, test} from 'vitest'
 
-import {csvEscape} from '../src/list-format/csv-escape'
+import {csvEscape} from '../src/list-format/csv-escape.ts'
 
 describe('csvEscape() backslash escapes every character except subset of definitely safe characters', () => {
   test('simple filename should not be modified', () => {

__tests__/filter.test.ts

@@ -1,7 +1,7 @@
 import {describe, expect, test} from 'vitest'
 
-import {File, ChangeStatus} from '../src/file'
-import {Filter} from '../src/filter'
+import {File, ChangeStatus} from '../src/file.ts'
+import {Filter} from '../src/filter.ts'
 
 describe('yaml filter parsing tests', () => {
   test('throws if yaml is not a dictionary', () => {

__tests__/git.test.ts

@@ -1,7 +1,7 @@
 import {describe, expect, test} from 'vitest'
 
-import {ChangeStatus} from '../src/file'
-import * as git from '../src/git'
+import {ChangeStatus} from '../src/file.ts'
+import * as git from '../src/git.ts'
 
 describe('parsing output of the git diff command', () => {
   test('parseGitDiffOutput returns files with correct change status', () => {

__tests__/shell-escape.test.ts

@@ -1,6 +1,6 @@
 import {describe, expect, test} from 'vitest'
 
-import {backslashEscape, shellEscape} from '../src/list-format/shell-escape'
+import {backslashEscape, shellEscape} from '../src/list-format/shell-escape.ts'
 
 describe('escape() backslash escapes every character except subset of definitely safe characters', () => {
   test('simple filename should not be modified', () => {

action.yml

@@ -49,7 +49,7 @@ outputs:
     description: JSON array with names of all filters matching any of changed files
 runs:
   using: 'node20'
-  main: 'dist/index.js'
+  main: 'dist/index.mjs'
 branding:
   color: blue
   icon: filter