flatlint 1.28.0 → 1.29.0

package/ChangeLog

@@ -1,3 +1,9 @@
+2025.01.09, v1.29.0
+
+feature:
+- 8d308ac flatlint: remove-useless-comma: curly
+- f070e0c flatlint: convert-coma-to-semicolon: return
+
 2025.01.08, v1.28.0
 
 feature:

package/README.md

@@ -24,6 +24,11 @@ npm i flatlint
 ```diff
 -const a = 5,
 +const a = 5;
+
+function x() {
+-   return m,
++   return m;
+}
 ```
 
 </details>
@@ -106,6 +111,17 @@ const a = {
 
  </details>
 
+<details><summary>remove useless comma</summary>
+
+```diff
+function x() {
+    return m;
+-},
++}
+```
+
+ </details>
+
 <details><summary>add missing quote</summary>
 
 ```diff

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

@@ -15,4 +15,5 @@ export const match = () => ({
 export const replace = () => ({
     'const __a = __b,': 'const __a = __b;',
     '__a(__args),': '__a(__args);',
+    'return __expr,': 'return __expr;',
 });

package/lib/plugins/remove-useless-comma/index.js

@@ -1,4 +1,9 @@
-import {isIdentifier} from '#types';
+import {
+    isIdentifier,
+    isPunctuator,
+    openCurlyBrace,
+    openSquireBrace,
+} from '#types';
 
 export const report = () => 'Remove useless coma';
 
@@ -11,9 +16,21 @@ export const match = () => ({
         
         return false;
     },
+    '},': (vars, path) => {
+        for (const token of path.getAllPrev()) {
+            if (isPunctuator(token, openSquireBrace))
+                return false;
+        }
+        
+        if (path.isNextIdentifier('const'))
+            return true;
+        
+        return !path.isNextPunctuator(openCurlyBrace);
+    },
 });
 
 export const replace = () => ({
     '__a(__args) {},': '__a(__args) {}',
     '__a(),': '__a()',
+    '},': '}',
 });

package/lib/runner/path.js

@@ -2,12 +2,17 @@ import {
     isIdentifier,
     isNewLine,
     isPunctuator,
+    isWhiteSpace,
 } from '#types';
 
 export const createPath = ({tokens, start, end}) => ({
     tokens,
     start,
     end,
+    isNextIdentifier: createIsNextIdentifier({
+        tokens,
+        end,
+    }),
     getAllPrev: createGetAllPrev({
         tokens,
         start,
@@ -30,12 +35,24 @@ export const createPath = ({tokens, start, end}) => ({
     }),
 });
 
-const createIsNextPunctuator = ({tokens, end}) => (punctuator) => {
+const createIsNextIdentifier = ({tokens, end}) => (value) => {
     const current = tokens[end];
     
     if (!current)
         return false;
     
+    return isIdentifier(current, value);
+};
+
+const createIsNextPunctuator = ({tokens, end}) => (punctuator) => {
+    let current = tokens[end];
+    
+    if (!current)
+        return false;
+    
+    if (isWhiteSpace(current))
+        current = tokens[end + 1];
+    
     return isPunctuator(current, punctuator);
 };
 

package/lib/types/types.js

@@ -83,6 +83,10 @@ export const closeSquareBrace = Punctuator(']');
 export const semicolon = Punctuator(';');
 export const comma = Punctuator(',');
 export const colon = Punctuator(':');
+export const openCurlyBrace = Punctuator('{');
+export const closeCurlyBrace = Punctuator('}');
+export const openSquireBrace = Punctuator('[');
+export const closeSquireBrace = Punctuator(']');
 
 export const OK = true;
 export const NOT_OK = false;

package/package.json

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