flatlint 1.26.0 → 1.27.0

package/ChangeLog

@@ -1,3 +1,8 @@
+2025.01.08, v1.27.0
+
+feature:
+- a6b1991 flatlint: multiple fixes in one source
+
 2025.01.07, v1.26.0
 
 feature:

package/lib/compare/compare.js

@@ -15,7 +15,7 @@ import {
     equalStr,
 } from './equal.js';
 
-export const compare = (source, template) => {
+export const compare = (source, template, {index = 0} = {}) => {
     const templateTokens = prepare(template);
     const tokens = prepare(source);
     
@@ -27,7 +27,7 @@ export const compare = (source, template) => {
     let delta = 0;
     let skip = 0;
     
-    for (let index = 0; index < n; index++) {
+    for (; index < n; index++) {
         for (let templateIndex = 0; templateIndex < templateTokensLength; templateIndex++) {
             let currentTokenIndex = index + templateIndex - skip;
             

package/lib/flatlint.js

@@ -1,11 +1,7 @@
 import {loadPlugins} from '@putout/engine-loader';
 import {parse} from '#parser';
 import {run} from '#runner';
-
-const getValue = (token) => token.value;
-const print = (tokens) => tokens
-    .map(getValue)
-    .join('');
+import {print} from '#printer';
 
 export function lint(source, overrides = {}) {
     const {fix = true, plugins: pluginNames = []} = overrides;

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

@@ -20,6 +20,9 @@ function check(vars, path) {
     if (path.isEndsWithPunctuator(comma))
         return false;
     
+    if (path.isPrevIdentifier('function'))
+        return false;
+    
     for (const token of path.getAllNext()) {
         if (isPunctuator(token, semicolon))
             return false;

package/lib/printer/printer.js

@@ -0,0 +1,11 @@
+const isString = (a) => typeof a === 'string';
+const getValue = (token) => token.value;
+
+export const print = (tokens) => {
+    if (isString(tokens))
+        return tokens;
+    
+    return tokens
+        .map(getValue)
+        .join('');
+};

package/lib/runner/path.js

@@ -1,4 +1,8 @@
-import {isNewLine, isPunctuator} from '#types';
+import {
+    isIdentifier,
+    isNewLine,
+    isPunctuator,
+} from '#types';
 
 export const createPath = ({tokens, start, end}) => ({
     getAllPrev: createGetAllPrev({
@@ -13,6 +17,10 @@ export const createPath = ({tokens, start, end}) => ({
         tokens,
         end,
     }),
+    isPrevIdentifier: createIsPrevIdentifier({
+        tokens,
+        start,
+    }),
     isEndsWithPunctuator: createIsCurrentPunctuator({
         tokens,
         end,
@@ -38,6 +46,17 @@ const createIsNextPunctuator = ({tokens, end}) => (punctuator) => {
     return isPunctuator(current, punctuator);
 };
 
+const createIsPrevIdentifier = ({tokens, start}) => (value) => {
+    const SPACE = 1;
+    const FUNCTION = 1;
+    const current = tokens[start - (FUNCTION + SPACE)];
+    
+    if (!current)
+        return false;
+    
+    return isIdentifier(current, value);
+};
+
 const createGetAllPrev = ({tokens, start}) => function*() {
     for (let i = start; i >= 0; --i) {
         yield tokens[i];

package/lib/runner/replacer.js

@@ -22,57 +22,71 @@ export const replace = (tokens, {fix, rule, plugin}) => {
     let isFixed = false;
     const match = plugin.match?.() ?? returns({});
     
-    for (let [from, to] of entries(plugin.replace())) {
-        const [ok, start, end] = compare(tokens, from);
+    for (const [from, to] of entries(plugin.replace())) {
+        let index = 0;
         
-        if (!ok)
-            continue;
-        
-        const values = getCurrentValues({
-            from,
-            start,
-            end,
-            tokens,
-        });
-        
-        const matchFn = match[from];
-        
-        if (matchFn) {
-            const is = matchFn(values, createPath({
-                tokens,
+        while (index < tokens.length - 1) {
+            const [ok, start, end] = compare(tokens, from, {
+                index,
+            });
+            
+            if (!ok)
+                break;
+            
+            const values = getCurrentValues({
+                from,
                 start,
                 end,
-            }));
+                tokens,
+            });
+            
+            const matchFn = match[from];
+            
+            if (matchFn) {
+                const is = matchFn(values, createPath({
+                    tokens,
+                    start,
+                    end,
+                }));
+                
+                if (!is) {
+                    index = end;
+                    continue;
+                }
+            }
             
-            if (!is)
+            if (fix) {
+                log(`${rule}: ${from} -> ${to}`);
+                
+                const preparedTo = prepare(to);
+                const waysTo = findVarsWays(preparedTo);
+                
+                setValues({
+                    values,
+                    waysTo,
+                    to: preparedTo,
+                });
+                
+                tokens.splice(start, end - start, ...preparedTo);
+                isFixed = true;
+                index = end;
+                
                 continue;
-        }
-        
-        if (fix) {
-            log(`${rule}: ${from} -> ${to}`);
-            to = prepare(to);
-            const waysTo = findVarsWays(to);
+            }
             
-            setValues({
-                values,
-                waysTo,
-                to,
+            const {line, column} = tokens[start];
+            const message = plugin.report();
+            
+            places.push({
+                rule,
+                message,
+                line,
+                column,
             });
             
-            tokens.splice(start, end - start, ...to);
-            isFixed = true;
-            continue;
+            index = end;
+            isFixed = false;
         }
-        
-        const {line, column} = tokens[start];
-        const message = plugin.report();
-        
-        places.push({
-            rule,
-            message,
-            line,
-            column,
-        });
     }
     
     return [isFixed, places];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "flatlint",
-  "version": "1.26.0",
+  "version": "1.27.0",
   "description": "JavaScript tokens-based linter",
   "main": "lib/flatlint.js",
   "type": "module",
@@ -17,6 +17,9 @@
     "#parser": {
       "default": "./lib/parser/parser.js"
     },
+    "#printer": {
+      "default": "./lib/printer/printer.js"
+    },
     "#traverser": {
       "default": "./lib/traverser/traverser.js"
     },