flatlint 1.38.0 → 1.39.1

package/ChangeLog

@@ -1,3 +1,13 @@
+2025.01.12, v1.39.1
+
+feature:
+- 5497596 add-missing-semicolon: improve
+
+2025.01.12, v1.39.0
+
+feature:
+- 8fad253 flatlint: add-missing-comma: add
+
 2025.01.11, v1.38.0
 
 feature:

package/README.md

@@ -64,6 +64,18 @@ const m = {
 
  </details>
 
+<details><summary>add missing comma</summary>
+
+```diff
+import {
+-    a
++    a,
+    b,
+} from 'c';
+```
+
+ </details>
+
 <details><summary>add <code>const</code> to <code>export</code></summary>
 
 ```diff

package/lib/compare/values.js

@@ -10,6 +10,9 @@ import {collectArray} from './collect-array.js';
 import {collectExpression} from './collect-expression.js';
 import {collectArgs} from './collect-args.js';
 
+const {isArray} = Array;
+const maybeArray = (a) => isArray(a) ? a : [a];
+
 const {entries} = Object;
 
 export function findVarsWays(tokens) {
@@ -36,21 +39,25 @@ export function getValues(tokens, waysFrom) {
         let end = index;
         let ok = true;
         
-        if (isTemplateArray(name))
+        if (isTemplateArray(name)) {
             [ok, end] = collectArray({
                 currentTokenIndex: index,
                 tokens,
             });
-        else if (isTemplateExpression(name))
+        } else if (isTemplateExpression(name)) {
             end = collectExpression({
                 currentTokenIndex: index,
                 tokens,
             });
-        else if (isTemplateArgs(name))
+        } else if (isTemplateArgs(name)) {
             [ok, end] = collectArgs({
                 currentTokenIndex: index,
                 tokens,
             });
+        } else {
+            values[name] = tokens[index];
+            continue;
+        }
         
         if (!ok) {
             values[name] = [];
@@ -65,7 +72,8 @@ export function getValues(tokens, waysFrom) {
 
 export function setValues({to, waysTo, values}) {
     for (const [name, index] of entries(waysTo)) {
-        to.splice(index, 1, ...values[name]);
+        const current = maybeArray(values[name]);
+        to.splice(index, 1, ...current);
     }
 }
 

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

@@ -0,0 +1,43 @@
+import {
+    assign,
+    closeCurlyBrace,
+    closeRoundBrace,
+    colon,
+    comma,
+    dot,
+    isIdentifier,
+    isOperator,
+    more,
+    openRoundBrace,
+} from '#types';
+
+export const report = () => 'Add missing comma';
+
+export const match = () => ({
+    __a: ({__a}, path) => {
+        const punctuators = [
+            assign,
+            comma,
+            openRoundBrace,
+            closeRoundBrace,
+            closeCurlyBrace,
+            colon,
+            dot,
+            more,
+        ];
+        
+        if (!isIdentifier(__a))
+            return false;
+        
+        if (isOperator(__a))
+            return false;
+        
+        return !path.isNextPunctuator(punctuators);
+    },
+});
+
+export const replace = () => ({
+    __a: '__a,',
+});
+
+

package/lib/plugins.js

@@ -3,18 +3,20 @@ import * as addMissingRoundBraces from './plugins/add-missing-round-braces/index
 import * as addMissingSquireBrace from './plugins/add-missing-square-brace/index.js';
 import * as addMissingQuote from './plugins/add-missing-quote/index.js';
 import * as addMissingSemicolon from './plugins/add-missing-semicolon/index.js';
+import * as addMissingComma from './plugins/add-missing-comma/index.js';
+import * as addConstToExport from './plugins/add-const-to-export/index.js';
 import * as convertCommaToSemicolon from './plugins/convert-comma-to-semicolon/index.js';
 import * as convertFromToRequire from './plugins/convert-from-to-require/index.js';
 import * as removeUselessRoundBrace from './plugins/remove-useless-round-brace/index.js';
 import * as removeUselessComma from './plugins/remove-useless-comma/index.js';
 import * as removeInvalidCharacter from './plugins/remove-invalid-character/index.js';
-import * as addConstToExport from './plugins/add-const-to-export/index.js';
 
 export const plugins = [
     ['wrap-assignment-in-parens', wrapAssignmentInParens],
     ['add-missing-round-braces', addMissingRoundBraces],
     ['add-missing-squire-brace', addMissingSquireBrace],
     ['add-missing-semicolon', addMissingSemicolon],
+    ['add-missing-comma', addMissingComma],
     ['add-missing-quote', addMissingQuote],
     ['add-const-to-export', addConstToExport],
     ['convert-comma-to-semicolon', convertCommaToSemicolon],

package/lib/types/types.js

@@ -22,6 +22,9 @@ export const isOperator = (token) => {
         'const',
         'export',
         'from',
+        'import',
+        'return',
+        'function',
     ];
     
     for (const operator of operators) {
@@ -99,6 +102,8 @@ export const closeSquareBrace = Punctuator(']');
 export const colon = Punctuator(':');
 export const comma = Punctuator(',');
 export const dot = Punctuator('.');
+export const more = Punctuator('>');
+export const assign = Punctuator('=');
 export const openRoundBrace = Punctuator('(');
 export const semicolon = Punctuator(';');
 export const openCurlyBrace = Punctuator('{');

package/package.json

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