flatlint 1.90.1 → 1.91.1

package/ChangeLog

@@ -1,3 +1,13 @@
+2025.01.31, v1.91.1
+
+feature:
+- c79d232 flatlint: remove-useless-round-brace: improve
+
+2025.01.31, v1.91.0
+
+feature:
+- 9485514 flatlint: add-missing-assign: isDeclarationKeyword
+
 2025.01.31, v1.90.1
 
 feature:

package/lib/plugins/add-missing-assign/index.js

@@ -1,7 +1,7 @@
 import {
     assign,
+    isDeclarationKeyword,
     isKeyword,
-    isOneOfKeywords,
     isOneOfPunctuators,
     semicolon,
 } from '#types';
@@ -9,13 +9,13 @@ import {
 export const report = () => 'Add missing assign';
 
 export const match = () => ({
-    '__x __a __expr': ({__expr, __x}, path) => {
+    '__x __a __expr': ({__expr, __x}) => {
         const [first] = __expr;
         
         if (isKeyword(first))
             return false;
         
-        if (!isOneOfKeywords(__x, ['const', 'let', 'var']))
+        if (!isDeclarationKeyword(__x))
             return false;
         
         return !isOneOfPunctuators(assign, __expr);

package/lib/plugins/remove-useless-round-brace/index.js

@@ -1,5 +1,4 @@
 import {
-    assign,
     hasRoundBraces,
     isBalancedRoundBraces,
     isPunctuator,
@@ -17,21 +16,12 @@ export const match = () => ({
     },
     
     '__a);': (vars, path) => {
-        let result = false;
-        
         for (const current of path.getAllPrev()) {
-            if (isPunctuator(current, openRoundBrace)) {
-                result = false;
-                break;
-            }
-            
-            if (isPunctuator(current, assign)) {
-                result = true;
-                break;
-            }
+            if (isPunctuator(current, openRoundBrace))
+                return false;
         }
         
-        return result;
+        return true;
     },
 });
 
@@ -40,4 +30,3 @@ export const replace = () => ({
     'from "__b")': 'from "__b"',
     '__a);': '__a;',
 });
-

package/lib/runner/replacer.js

@@ -10,7 +10,8 @@ import {createPath} from './path.js';
 
 const returns = (a) => () => a;
 const {entries} = Object;
-const log = debug('flatlint');
+const logCompare = debug('flatlint:compare');
+const logFix = debug('flatlint:fix');
 
 export const replace = (tokens, {fix, rule, plugin}) => {
     const places = [];
@@ -22,7 +23,7 @@ export const replace = (tokens, {fix, rule, plugin}) => {
         let index = 0;
         
         while (index < tokens.length - 1) {
-            log(`compare: ${rule}: ${from} -> ${to}`);
+            logCompare(`${rule}: ${from} -> ${to}`);
             const [ok, start, end] = compare(tokens, from, {
                 index,
             });
@@ -53,7 +54,7 @@ export const replace = (tokens, {fix, rule, plugin}) => {
             }
             
             if (fix) {
-                log(`fix: ${rule}: ${from} -> ${to}`);
+                logFix(`${rule}: ${from} -> ${to}`);
                 
                 const preparedTo = prepare(to);
                 const waysTo = findVarsWays(preparedTo);

package/lib/types/types.js

@@ -39,6 +39,15 @@ export const isKeyword = (token, name) => {
     return isIdentifier(token, name);
 };
 
+export const isDeclarationKeyword = (token, name) => {
+    const {value} = token;
+    
+    if (!keyword.isDeclarationKeyword(value))
+        return false;
+    
+    return isIdentifier(token, name);
+};
+
 export const isOneOfPunctuators = (token, punctuators) => {
     for (const punctuator of punctuators) {
         if (isPunctuator(token, punctuator))

package/package.json

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