@jslint-org/jslint 2025.3.31 → 2025.12.28

package/CHANGELOG.md

@@ -2,11 +2,19 @@
 
 # Todo
 - doc - document supported/unsupported es6+ features
-- coverage - add macros `/*coverage-disable*/` and `/*coverage-enable*/`.
 - jslint - add html and css linting back into jslint.
 - jslint - add new warning requiring paren around plus-separated concatenations.
 - jslint - try to improve parser to be able to parse jquery.js without stopping.
 
+# v2025.12.28
+- ci - Update file publish.yml to use trusted-publishing for npm packages.
+- jslint - regression-fix - Warn about variable usage before initialization.
+- jslint - Allow parenthesis after ellipsis inside a function call.
+- ci - Upgrade nodejs used in ci to v24.
+
+# v2025.10.31
+- coverage - Add coverage-directives '/*coverage-disable*/', '/*coverage-enable*/'. '//coverage-ignore-line'.
+
 # v2025.3.31
 - ci - Upgrade nodejs used in ci to v22.
 - ubuntu-ci - bugfix - Fix out-of-date apt-list when installing graphicsmagick.

package/README.md

@@ -3,7 +3,7 @@ Douglas Crockford <douglas@crockford.com>
 
 
 # Status
-| Branch | [master<br>(v2025.3.31)](https://github.com/jslint-org/jslint/tree/master) | [beta<br>(Web Demo)](https://github.com/jslint-org/jslint/tree/beta) | [alpha<br>(Development)](https://github.com/jslint-org/jslint/tree/alpha) |
+| Branch | [master<br>(v2025.12.28)](https://github.com/jslint-org/jslint/tree/master) | [beta<br>(Web Demo)](https://github.com/jslint-org/jslint/tree/beta) | [alpha<br>(Development)](https://github.com/jslint-org/jslint/tree/alpha) |
 |--:|:--:|:--:|:--:|
 | CI | [![ci](https://github.com/jslint-org/jslint/actions/workflows/ci.yml/badge.svg?branch=master)](https://github.com/jslint-org/jslint/actions?query=branch%3Amaster) | [![ci](https://github.com/jslint-org/jslint/actions/workflows/ci.yml/badge.svg?branch=beta)](https://github.com/jslint-org/jslint/actions?query=branch%3Abeta) | [![ci](https://github.com/jslint-org/jslint/actions/workflows/ci.yml/badge.svg?branch=alpha)](https://github.com/jslint-org/jslint/actions?query=branch%3Aalpha) |
 | Coverage | [![coverage](https://jslint-org.github.io/jslint/branch-master/.artifact/coverage/coverage_badge.svg)](https://jslint-org.github.io/jslint/branch-master/.artifact/coverage/index.html) | [![coverage](https://jslint-org.github.io/jslint/branch-beta/.artifact/coverage/coverage_badge.svg)](https://jslint-org.github.io/jslint/branch-beta/.artifact/coverage/index.html) | [![coverage](https://jslint-org.github.io/jslint/branch-alpha/.artifact/coverage/coverage_badge.svg)](https://jslint-org.github.io/jslint/branch-alpha/.artifact/coverage/index.html) |
@@ -66,6 +66,8 @@ Douglas Crockford <douglas@crockford.com>
     - [Directive `/*property*/`](#directive-property)
     - [Directive `/*jslint-disable*/.../*jslint-enable*/`](#directive-jslint-disablejslint-enable)
     - [Directive `//jslint-ignore-line`](#directive-jslint-ignore-line)
+    - [Directive `/*coverage-disable*/.../*coverage-enable*/`](#directive-coverage-disablecoverage-enable)
+    - [Directive `//coverage-ignore-line`](#directive-coverage-ignore-line)
 
 10. [Package Listing](#package-listing)
 
@@ -891,6 +893,34 @@ eval("1"); //jslint-ignore-line
 ```
 
 
+<br><br>
+### Directive `/*coverage-disable*/.../*coverage-enable*/`
+
+```js
+/*coverage-disable*/
+
+// JSLint will ignore code-coverage in this region.
+
+if (false) {
+    console.log("hello world");
+}
+
+/*coverage-enable*/
+```
+
+
+<br><br>
+### Directive `//coverage-ignore-line`
+
+```js
+// JSLint will ignore code-coverage at given line.
+
+if (false) {
+    console.log("hello world"); //coverage-ignore-line
+}
+```
+
+
 <br><br>
 # Package Listing
 ![screenshot_package_listing.svg](https://jslint-org.github.io/jslint/branch-beta/.artifact/screenshot_package_listing.svg)
@@ -917,13 +947,14 @@ eval("1"); //jslint-ignore-line
 <br><br>
 ### pull-request merge
 - find highest issue-number at https://github.com/jslint-org/jslint/issues/, https://github.com/jslint-org/jslint/pulls/, and add +1 to it for PR-xxx
+- checkpoint local-branch-beta
 - `shGitPullrequest beta beta`
     - verify ci-success for origin-branch-alpha
     - https://github.com/kaizhu256/jslint/actions
 - `git push upstream alpha -f`
     - verify ci-success for upstream-branch-alpha
     - https://github.com/jslint-org/jslint/actions
-- goto https://github.com/jslint-org/jslint/compare/beta...kaizhu256:jslint:branch-p2024.11.24
+- goto https://github.com/jslint-org/jslint/compare/beta...kaizhu256:jslint:branch-p2025.12.14
 - click `Create pull request`
 - input `Add your description here...` with:
 ```
@@ -943,7 +974,7 @@ This PR will additionally:
     - verify ci-success for pull-request
     - https://github.com/jslint-org/jslint/actions/workflows/on_pull_request.yml
 - wait awhile before continuing ...
-- click `Rebase and merge`
+- click `Squash and merge`
     - verify ci-success for upstream-branch-beta
     - https://github.com/jslint-org/jslint/actions
 - `shGitPullrequestCleanup`
@@ -957,13 +988,15 @@ This PR will additionally:
 
 <br><br>
 ### branch-master commit
-- `shGitPullrequest master beta`
+- update ci.yml to latest nodejs-lts
+- checkpoint local-branch-beta
+- `shGitPullrequest master beta # re-run until version propagates`
     - verify ci-success for origin-branch-alpha
     - https://github.com/kaizhu256/jslint/actions
 - `git push upstream alpha -f`
     - verify ci-success for upstream-branch-alpha
     - https://github.com/jslint-org/jslint/actions
-- goto https://github.com/jslint-org/jslint/compare/beta...kaizhu256:jslint:branch-v2025.3.31
+- goto https://github.com/jslint-org/jslint/compare/beta...kaizhu256:jslint:branch-v2025.12.28
 - click `Create pull request`
 - input `Add a title` with: `# v20yy.mm.dd`
 - input `Add a description` with:
@@ -976,7 +1009,7 @@ This PR will additionally:
     - verify ci-success for pull-request
     - https://github.com/jslint-org/jslint/actions/workflows/on_pull_request.yml
 - wait awhile before continuing ...
-- click `Rebase and merge`
+- click `Squash and merge`
     - verify ci-success for upstream-branch-beta
     - https://github.com/jslint-org/jslint/actions
 - `shGitPullrequestCleanup`
@@ -996,6 +1029,12 @@ This PR will additionally:
 
 <br><br>
 ### branch-master publish
+- goto https://www.npmjs.com/package/@jslint-org/jslint/access <!--no-validate-->
+- click `Github Actions`
+- input `Organization or user*` with: `jslint-org`
+- input `Repository*` with: `jslint`
+- input `Workflow filename*` with: `publish.yml`
+- click `Set up connection` or `Update Package Settings`
 - `git push upstream beta:master`
     - verify ci-success for upstream-branch-master
     - https://github.com/jslint-org/jslint/actions

package/jslint.mjs

@@ -106,9 +106,9 @@
     formatted_message, free, freeze, from, froms, fsWriteFileWithParents,
     fud_stmt, functionName, function_list, function_stack, functions, get,
     getset, github_repo, globExclude, global, global_dict, global_list,
-    holeList, htmlEscape, id, identifier, import, import_list, import_meta_url,
-    inc, includeList, indent2, index, indexOf, init, initial, isArray,
-    isBlockCoverage, isHole, isNaN, is_equal, is_weird, join, jslint,
+    holeList, htmlEscape, id, identifier, ignoreLine, import, import_list,
+    import_meta_url, inc, includeList, indent2, index, indexOf, init, initial,
+    isArray, isBlockCoverage, isHole, isNaN, is_equal, is_weird, join, jslint,
     jslint_apidoc, jslint_assert, jslint_charset_ascii, jslint_cli,
     jslint_edition, jslint_phase1_split, jslint_phase2_lex, jslint_phase3_parse,
     jslint_phase4_walk, jslint_phase5_whitage, jslint_report, json,
@@ -163,7 +163,7 @@ let jslint_charset_ascii = (
     + "@ABCDEFGHIJKLMNOPQRSTUVWXYZ[\\]^_"
     + "`abcdefghijklmnopqrstuvwxyz{|}~\u007f"
 );
-let jslint_edition = "v2025.3.31";
+let jslint_edition = "v2025.12.28";
 let jslint_export;                      // The jslint object to be exported.
 let jslint_fudge = 1;                   // Fudge starting line and starting
                                         // ... column to 1.
@@ -4674,7 +4674,13 @@ function jslint_phase3_parse(state) {
 
             test_cause("free");
             the_paren.free = true;
-            if (the_argument.wrapped === true) {
+            if (
+                the_argument.wrapped === true
+
+// PR-483 - Allow parenthesis after ellipsis inside a function call.
+
+                && the_argument.ellipsis !== true
+            ) {
 
 // test_cause:
 // ["aa((0))", "infix_lparen", "unexpected_a", "(", 3]
@@ -5158,6 +5164,10 @@ function jslint_phase3_parse(state) {
                 the_label.dead = false;
                 the_label.init = true;
                 the_statement = parse_statement();
+
+// Issue #458 - Regression - Warn about variable usage before initialization.
+
+                the_label.dead = true;
                 functionage.statement_prv = the_statement;
                 the_statement.label = the_label;
                 the_statement.statement = true;
@@ -5204,9 +5214,13 @@ function jslint_phase3_parse(state) {
             }
             semicolon();
         }
-        if (the_label !== undefined) {
-            the_label.dead = true;
-        }
+
+// Issue #458 - Regression - Warn about variable usage before initialization.
+
+//        if (the_label !== undefined) {
+//            the_label.dead = true;
+//        }
+
         return the_statement;
     }
 
@@ -6987,8 +7001,16 @@ function jslint_phase3_parse(state) {
                         the_variable.names.push(name);
                         enroll(name, "variable", mode_const);
                     }
-                    name.dead = false;
+
+// Issue #458 - Regression - Warn about variable usage before initialization.
+
+//                    name.dead = false;
+
                     name.init = true;
+
+// test_cause:
+// ["const {aa}=bb;\nconst bb=0;", "lookup", "out_of_scope_a", "bb", 12]
+
                     if (token_nxt.id === "=") {
 
 // test_cause:
@@ -7039,8 +7061,16 @@ function jslint_phase3_parse(state) {
                     advance();
                     the_variable.names.push(name);
                     enroll(name, "variable", mode_const);
-                    name.dead = false;
+
+// Issue #458 - Regression - Warn about variable usage before initialization.
+
+//                    name.dead = false;
+
                     name.init = true;
+
+// test_cause:
+// ["const [aa]=bb;\nconst bb=0;", "lookup", "out_of_scope_a", "bb", 12]
+
                     if (ellipsis) {
                         name.ellipsis = true;
                         break;
@@ -7073,8 +7103,16 @@ function jslint_phase3_parse(state) {
                 enroll(name, "variable", mode_const);
                 if (token_nxt.id === "=" || mode_const) {
                     advance("=");
-                    name.dead = false;
+
+// Issue #458 - Regression - Warn about variable usage before initialization.
+
+//                    name.dead = false;
+
                     name.init = true;
+
+// test_cause:
+// ["const aa=bb;\nconst bb=0;", "lookup", "out_of_scope_a", "bb", 10]
+
                     name.expression = parse_expression(0);
                 }
                 the_variable.names.push(name);
@@ -10882,12 +10920,15 @@ body {
     background: #9d9;
 }
 .coverage .count {
-    color: #666;
+    color: #333;
 }
 .coverage .coverageIgnore {
     background: #ccc;
 }
 .coverage .coverageLow,
+.coverage .ignore {
+    background: #ccc;
+}
 .coverage .uncovered {
     background: #ebb;
 }
@@ -10911,6 +10952,10 @@ body {
 .coverage tr:hover td {
     background: #7d7;
 }
+.coverage pre:hover span.ignore,
+.coverage tr:hover td.coverageIgnore {
+    background: #ccc;
+}
 .coverage pre:hover span.uncovered,
 .coverage tr:hover td.coverageLow {
     background: #f99;
@@ -11112,6 +11157,7 @@ body {
             lineList.forEach(function ({
                 count,
                 holeList,
+                ignoreLine,
                 line,
                 startOffset
             }, ii) {
@@ -11159,7 +11205,11 @@ body {
 // true.
 
                             if (isHole) {
-                                lineHtml += " class=\"uncovered\"";
+                                lineHtml += (
+                                    ignoreLine
+                                    ? " class=\"ignore\""
+                                    : " class=\"uncovered\""
+                                );
                             }
                             lineHtml += ">";
                             chunk = "";
@@ -11179,7 +11229,9 @@ body {
 </span>
 <span class="count
                 ${(
-                    count <= 0
+                    (count <= 0 && ignoreLine)
+                    ? "ignore"
+                    : count <= 0
                     ? "uncovered"
                     : ""
                 )}"
@@ -11416,6 +11468,7 @@ function sentinel() {}
         functions,
         url: pathname
     }) {
+        let ignoreBlock = false;
         let lineList;
         let linesCovered;
         let linesTotal;
@@ -11425,14 +11478,23 @@ function sentinel() {}
         source.replace((
             /^.*$/gm
         ), function (line, startOffset) {
+            if (line === "/*coverage-disable*/") {
+                ignoreBlock = true;
+            }
             lineList[lineList.length - 1].endOffset = startOffset - 1;
             lineList.push({
                 count: -1,
                 endOffset: 0,
                 holeList: [],
+                ignoreLine: (
+                    ignoreBlock || line.endsWith("//coverage-ignore-line")
+                ),
                 line,
                 startOffset
             });
+            if (line === "/*coverage-enable*/") {
+                ignoreBlock = false;
+            }
             return "";
         });
         lineList.shift();
@@ -11485,11 +11547,13 @@ function sentinel() {}
             });
         });
         linesTotal = lineList.length;
-        linesCovered = lineList.filter(function ({
-            count
+        linesCovered = 0;
+        lineList.forEach(function ({
+            count,
+            ignoreLine
         }) {
-            return count > 0;
-        }).length;
+            linesCovered += count > 0 || ignoreLine;
+        });
         await moduleFs.promises.mkdir((
             modulePath.dirname(coverageDir + pathname)
         ), {

package/package.json

@@ -35,5 +35,5 @@
     "shCiArtifactUpload": 1,
     "shCiPublishNpm": 1,
     "type": "module",
-    "version": "2025.3.31"
+    "version": "2025.12.28"
 }