Fix build for modern JS+github

  - update to latest dependencies
  - update github workflow with currently supported versions
  - update ESLint config (moves to new format)

Author: droberts-ctrlo

.github/workflows/ci.yaml

@@ -13,12 +13,12 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x, 14.x, 16.x, 17.x]
+        node-version: [20.x, 22.x, 24.x, 25.x]
 
     steps:
-    - uses: actions/checkout@v2
+    - uses: actions/checkout@v6
     - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v2
+      uses: actions/setup-node@v6
       with:
         node-version: ${{ matrix.node-version }}
         cache: 'yarn'

eslint.config.mjs

@@ -0,0 +1,40 @@
+import globals from "globals";
+import { defineConfig } from "eslint/config";
+
+export default defineConfig([
+    { files: ["**/*.{js,mjs,cjs}"], languageOptions: { globals: { ...globals.browser, ...globals.node, ...globals.worker } } },
+    { files: ["test/**/*.{js,mjs,cjs}"], languageOptions: { globals: globals.jest } },
+    {
+        "rules": {
+            "indent": [
+                "error",
+                4,
+                {
+                    "SwitchCase": 1
+                }
+            ],
+            "linebreak-style": [
+                "error",
+                "unix"
+            ],
+            "no-console": 0,
+            "no-empty": [
+                2,
+                {
+                    "allowEmptyCatch": true
+                }
+            ],
+            "no-unused-vars": [
+                2,
+                {
+                    "args": "none",
+                    "caughtErrors": "none"
+                }
+            ],
+            "semi": [
+                "error",
+                "always"
+            ]
+        }
+    }
+]);

package.json

@@ -30,62 +30,13 @@
   },
   "homepage": "https://github.com/fengari-lua/fengari#readme",
   "devDependencies": {
-    "eslint": "^5.15.1",
-    "jest": "^24.5.0"
+    "eslint": "^9.39.2",
+    "jest": "^30.2.0"
   },
   "dependencies": {
-    "readline-sync": "^1.4.9",
-    "sprintf-js": "^1.1.2",
-    "tmp": "^0.0.33"
+    "readline-sync": "^1.4.10",
+    "sprintf-js": "^1.1.3",
+    "tmp": "^0.2.5"
   },
-  "sideEffects": false,
-  "eslintConfig": {
-    "env": {
-      "browser": true,
-      "es6": true,
-      "node": true,
-      "worker": true
-    },
-    "extends": "eslint:recommended",
-    "rules": {
-      "indent": [
-        "error",
-        4,
-        {
-          "SwitchCase": 1
-        }
-      ],
-      "linebreak-style": [
-        "error",
-        "unix"
-      ],
-      "no-console": 0,
-      "no-empty": [
-        2,
-        {
-          "allowEmptyCatch": true
-        }
-      ],
-      "no-unused-vars": [
-        2,
-        {
-          "args": "none"
-        }
-      ],
-      "semi": [
-        "error",
-        "always"
-      ]
-    },
-    "overrides": [
-      {
-        "files": [
-          "test/**/*.test.js"
-        ],
-        "env": {
-          "jest": true
-        }
-      }
-    ]
-  }
+  "sideEffects": false
 }

src/lauxlib.js

@@ -968,7 +968,7 @@ if (typeof process === "undefined") {
             lf.buff[lf.n++] = com.c; /* 'c' is the first character of the stream */
         let status = lua_load(L, getF, lf, lua_tostring(L, -1), mode);
         let readstatus = lf.err;
-        if (filename) try { fs.closeSync(lf.f); } catch(e) {}  /* close file (even in case of errors) */
+        if (filename) try { fs.closeSync(lf.f); } catch (e) {}  /* close file (even in case of errors) */
         if (readstatus) {
             lua_settop(L, fnameindex);  /* ignore results from 'lua_load' */
             return errfile(L, "read", fnameindex, readstatus);

src/lbaselib.js

@@ -97,7 +97,7 @@ if (typeof process === "undefined") {
             try {
                 /* If the string is valid utf8, then we can use to_jsstring */
                 s = to_jsstring(s);
-            } catch(e) {
+            } catch (e) {
                 /* otherwise push copy of raw array */
                 let copy = new Uint8Array(s.length);
                 copy.set(s);

src/ldo.js

@@ -424,7 +424,7 @@ const luaD_rawrunprotected = function(L, f, ud) {
                     }
 
                     lj.status = LUA_ERRRUN;
-                } catch(e2) {
+                } catch (e2) {
                     if (lj.status === LUA_OK) {
                         /* also failed */
                         lj.status = -1;

test/defs.test.js

@@ -59,36 +59,36 @@ describe('to_jsstring', () => {
 describe('to_jsstring fails on invalid unicode', () => {
     test("non-utf8 char", () => {
         expect(() => defs.to_jsstring(defs.luastring_of(165)))
-            .toThrowError(RangeError);
+            .toThrow(RangeError);
     });
 
     test("invalid continuation byte", () => {
         expect(() => defs.to_jsstring(defs.luastring_of(208, 60)))
-            .toThrowError(RangeError);
+            .toThrow(RangeError);
     });
 
     test("invalid continuation byte", () => {
         expect(() => defs.to_jsstring(defs.luastring_of(225, 60, 145)))
-            .toThrowError(RangeError);
+            .toThrow(RangeError);
     });
 
     test("invalid continuation byte", () => {
         expect(() => defs.to_jsstring(defs.luastring_of(225, 145, 60)))
-            .toThrowError(RangeError);
+            .toThrow(RangeError);
     });
 
     test("invalid continuation byte", () => {
         expect(() => defs.to_jsstring(defs.luastring_of(242, 60, 145, 145)))
-            .toThrowError(RangeError);
+            .toThrow(RangeError);
     });
 
     test("invalid continuation byte", () => {
         expect(() => defs.to_jsstring(defs.luastring_of(242, 145, 60, 145)))
-            .toThrowError(RangeError);
+            .toThrow(RangeError);
     });
 
     test("invalid continuation byte", () => {
         expect(() => defs.to_jsstring(defs.luastring_of(242, 145, 145, 60)))
-            .toThrowError(RangeError);
+            .toThrow(RangeError);
     });
 });