flatlint 1.55.0 → 1.56.0

package/ChangeLog

@@ -1,3 +1,8 @@
+2025.01.17, v1.56.0
+
+feature:
+- 8683716 flatlint: convert-comma-to-semicolon: improve
+
 2025.01.17, v1.55.0
 
 feature:

package/lib/compare/collect-args.js

@@ -1,8 +1,11 @@
 import {
+    closeCurlyBrace,
     closeRoundBrace,
     isNewLine,
     NOT_OK,
     OK,
+    openCurlyBrace,
+    openRoundBrace,
 } from '#types';
 import {equal} from './equal.js';
 
@@ -13,16 +16,34 @@ export const collectArgs = ({currentTokenIndex, tokens}) => {
     if (equal(tokens[index], closeRoundBrace))
         return [NOT_OK];
     
+    let curlyBracesBalance = 0;
+    let roundBracesBalance = 0;
+    
     for (; index < n; index++) {
         const token = tokens[index];
         
+        if (equal(token, openRoundBrace))
+            ++roundBracesBalance;
+        
         if (equal(token, closeRoundBrace))
+            --roundBracesBalance;
+        
+        if (equal(token, openCurlyBrace))
+            ++curlyBracesBalance;
+        
+        if (equal(token, closeCurlyBrace))
+            --curlyBracesBalance;
+        
+        if (curlyBracesBalance < 0)
             break;
         
-        if (isNewLine(token))
+        if (roundBracesBalance < 0)
             break;
     }
     
+    if (isNewLine(tokens[index - 1]))
+        --index;
+    
     return [
         OK,
         --index,

package/lib/plugins/add-missing-round-brace/index.js

@@ -1,34 +1,27 @@
 import {
     arrow,
+    closeCurlyBrace,
     closeRoundBrace,
     isPunctuator,
+    openCurlyBrace,
     openRoundBrace,
 } from '#types';
 
 export const report = () => 'Add missing round brace';
 
 export const match = () => ({
-    '(__args) {': ({__args}) => {
-        if (!isPunctuator(openRoundBrace, __args))
-            return false;
-        
-        return !isPunctuator(closeRoundBrace, __args);
-    },
-    '__a(__args': (vars, path) => {
+    '__a(__args': ({__args}, path) => {
         if (path.isCurrentPunctuator(closeRoundBrace))
             return false;
         
-        for (const token of path.getAllNext()) {
-            if (isPunctuator(token, closeRoundBrace))
-                return false;
-        }
-        
-        return true;
+        return !path.isNextPunctuator(closeRoundBrace);
     },
 });
 
 export const replace = () => ({
     'if __a > __b': 'if (__a > __b)',
     '__a(__args': '__a(__args)',
-    '(__args) {': '(__args)) {',
+    'if (__a.__b(__args) {': 'if (__a.__b(__args)) {',
+    'if (__a(__args) {': 'if (__a(__args)) {',
 });
+

package/lib/plugins/convert-comma-to-semicolon/index.js

@@ -12,6 +12,9 @@ import {
 export const report = () => 'Use semicolon instead of trailing comma';
 export const match = () => ({
     '__a(__args),': (vars, path) => {
+        if (path.isNextKeyword())
+            return true;
+        
         for (const token of path.getAllPrev()) {
             if (isPunctuator(token, colon))
                 return false;

package/lib/runner/path.js

@@ -8,9 +8,6 @@ import {
     isTemplateMiddle,
 } from '#types';
 
-const {isArray} = Array;
-const maybeArray = (a) => isArray(a) ? a : [a];
-
 export const createPath = ({tokens, start, end}) => ({
     tokens,
     start,

package/lib/types/types.js

@@ -28,9 +28,10 @@ export const isKeyword = (token) => {
     const keywords = [
         'as',
         'await',
+        'const',
+        'continue',
         'var',
         'let',
-        'const',
         'export',
         'from',
         'import',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flatlint",
-  "version": "1.55.0",
+  "version": "1.56.0",
   "description": "JavaScript tokens-based linter",
   "main": "lib/flatlint.js",
   "type": "module",