remove files to support ts debug; add force curly after if; minor adjustments to to test case

Author: meowbmw

.vscode/launch.json

@@ -4,10 +4,13 @@ import { baseConfig } from "@schleifner/eslint-config-base/config.mjs";
 
 export default tseslint.config(
   {
-    ignores: [
-      "dist/**",
-      "**/*.mjs",
-    ],
+    ignores: ["dist/**", "**/*.mjs"],
   },
-  ...baseConfig
+  ...baseConfig,
+  {
+    files: ["**/*.ts"],
+    rules: {
+      curly: ["error", "all"]
+    },
+  }
 );

.vscode/settings.json

@@ -140,10 +140,14 @@ const noRepeatedMemberAccess = createRule({
       // Skip nodes that are part of larger member expressions
       // Example: In a.b.c, we process the top-level MemberExpression only,
       // not the sub-expressions a.b or a
-      if (node.parent?.type === AST_NODE_TYPES.MemberExpression) return;
+      if (node.parent?.type === AST_NODE_TYPES.MemberExpression) {
+        return;
+      }
 
       const chainInfo = analyzeChain(node);
-      if (!chainInfo) return;
+      if (!chainInfo) {
+        return;
+      }
 
       const scope = sourceCode.getScope(node);
       const chainMap = getChainMap(scope);
@@ -154,13 +158,17 @@ const noRepeatedMemberAccess = createRule({
       // Update chain statistics for each part of the hierarchy
       for (const chain of chainInfo) {
         // Skip single-level chains
-        if (!chain.includes(".")) continue;
+        if (!chain.includes(".")) {
+          continue;
+        }
 
         const chainInfo = chainMap.get(chain) || {
           count: 0,
           modified: false,
         };
-        if (chainInfo.modified) break;
+        if (chainInfo.modified) {
+          break;
+        }
 
         chainInfo.count++;
         chainMap.set(chain, chainInfo);

eslint.config.mjs

@@ -101,7 +101,9 @@ export default createRule({
       // Check for string concatenation with + operator
       BinaryExpression(node) {
         // Only check inside loops
-        if (loopDepth === 0) return;
+        if (loopDepth === 0) {
+          return;
+        }
 
         const leftType: ts.Type = parserServices.getTypeAtLocation(node.left);
         const rightType: ts.Type = parserServices.getTypeAtLocation(node.right);

loader.mjs

@@ -26,7 +26,7 @@ describe("Rule: no-spread", () => {
         // ignore array access
         `
               const x = data[0].value;
-              data[0].count++;
+              data[0].count=data[0].count+1;
               send(data[0].id);
             `,
         // Dynamic property access (should be ignored)
@@ -100,7 +100,7 @@ describe("Rule: no-spread", () => {
         `,
         `
         const v1 = a.b.c;
-        a.b++;
+        a.b = a.b + 1;
         const v2 = a.b.c; 
         const v3 = a.b.c; 
         `,
@@ -149,7 +149,6 @@ describe("Rule: no-spread", () => {
         {
           code: `
           const a = data.x + data.y;
-          // data.update();
           const b = data.x * 2;
           notify(data.x);
         `,