Update code as per review

Fix ESLint config
Add Node 20.x to CI script
Remove ESLint comments

Author: droberts-ctrlo

.github/workflows/ci.yaml

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

eslint.config.mjs

@@ -27,7 +27,8 @@ export default defineConfig([
             "no-unused-vars": [
                 2,
                 {
-                    "args": "none"
+                    "args": "none",
+                    "caughtErrors": "none"
                 }
             ],
             "semi": [

src/lauxlib.js

@@ -968,7 +968,6 @@ 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;
-        // 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' */

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) { // eslint-disable-line no-unused-vars
+            } catch (e) {
                 /* 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 (e) { // eslint-disable-line no-unused-vars
+    } catch (e) {
         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 (e) { //eslint-disable-line no-unused-vars
+                } catch (e) {
                     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 (e) { // eslint-disable-line no-unused-vars
+        } catch (e) {
             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 (e) { //eslint-disable-line no-unused-vars
+    } catch (e) {
         return null;
     }
     /* use a regex to validate number and also to get length

test/test-suite/ltests.js

@@ -691,7 +691,6 @@ const checkpanic = function(L) {
     try {    /* set jump buffer */
         runJS(L, L1, { script: code, offset: 0 });    /* run code unprotected */
         lua.lua_pushliteral(L, "no errors");
-        // 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));