Update files as per review

Update ci workflow with currently supported NodeJS version
Add variable to catch statements
Add appropriate eslint-disable directives

Author: droberts-ctrlo

.github/workflows/ci.yaml

@@ -13,7 +13,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [12.x, 14.x, 16.x, 17.x]
+        node-version: [22.x, 24.x, 25.x]
 
     steps:
     - uses: actions/checkout@v2

eslint.config.mjs

@@ -2,7 +2,8 @@ import globals from "globals";
 import { defineConfig } from "eslint/config";
 
 export default defineConfig([
-    { files: ["**/*.{js,mjs,cjs}"], languageOptions: { globals: { ...globals.browser, ...globals.node, ...globals.worker, ...globals.jest } } },
+    { files: ["**/*.{js,mjs,cjs}"], languageOptions: { globals: { ...globals.browser, ...globals.node, ...globals.worker } } },
+    { files: ["test/**/*.{js,mjs,cjs}"], languageOptions: { globals: globals.jest } },
     {
         "rules": {
             "indent": [

src/lauxlib.js

@@ -968,7 +968,8 @@ 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 {}  /* close file (even in case of errors) */
+        // eslint-disable-next-line no-unused-vars
+        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 {
+            } catch (e) { // eslint-disable-line no-unused-vars
                 /* otherwise push copy of raw array */
                 let copy = new Uint8Array(s.length);
                 copy.set(s);
@@ -269,7 +269,7 @@ const luaB_ipairs = function(L) {
 const b_str2int = function(s, base) {
     try {
         s = to_jsstring(s);
-    } catch {
+    } catch (e) { // eslint-disable-line no-unused-vars
         return null;
     }
     let r = /^[\t\v\f \n\r]*([+-]?)0*([0-9A-Za-z]+)[\t\v\f \n\r]*$/.exec(s);

src/ldo.js

@@ -424,7 +424,7 @@ const luaD_rawrunprotected = function(L, f, ud) {
                     }
 
                     lj.status = LUA_ERRRUN;
-                } catch {
+                } catch (e) { //eslint-disable-line no-unused-vars
                     if (lj.status === LUA_OK) {
                         /* also failed */
                         lj.status = -1;

src/loadlib.js

@@ -202,7 +202,7 @@ if (typeof process !== "undefined") { // Only with Node
         try {
             let fd = fs.openSync(filename, 'r');
             fs.closeSync(fd);
-        } catch {
+        } catch (e) { // eslint-disable-line no-unused-vars
             return false;
         }
         return true;

src/lobject.js

@@ -478,7 +478,7 @@ const lua_strx2number = function(s) {
 const lua_str2number = function(s) {
     try {
         s = to_jsstring(s);
-    } catch {
+    } catch (e) { //eslint-disable-line no-unused-vars
         return null;
     }
     /* use a regex to validate number and also to get length

test/test-suite/ltests.js

@@ -691,7 +691,8 @@ const checkpanic = function(L) {
     try {    /* set jump buffer */
         runJS(L, L1, { script: code, offset: 0 });    /* run code unprotected */
         lua.lua_pushliteral(L, "no errors");
-    } catch {    /* error handling */
+        // eslint-disable-next-line no-unused-vars
+    } catch (e) {    /* error handling */
         /* move error message to original state */
         lua.lua_pushstring(L, lua.lua_tostring(L1, -1));
     }