@round-core/lint 0.1.0 → 0.1.1

package/README.md

@@ -0,0 +1,14 @@
+# @round-core/lint
+
+ESLint plugin for RoundJS. Enables linting for `.round` files using standard JavaScript parsers (espree).
+
+## How it works
+
+1.  **Preprocessing**: Transforms the `.round` file into valid JSX.
+2.  **Global Injection**: Automatically detects loop variables (like `t` in `for(t in list)`) and injects `/* globals */` comments at the end of the file to prevent "undefined variable" errors.
+3.  **Component Tracking**: Identifies JSX-style component usage within RoundJS blocks and ensures ESLint doesn't mark them as unused variables.
+4.  **1:1 Mapping**: Using the `SourceMapper`, ESLint reported errors are translated back to the original `.round` file lines and columns, ensuring perfectly aligned highlights in VS Code.
+
+## Configuration
+
+Recommended usage is via the `eslint.config.js` generated by the RoundJS CLI.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@round-core/lint",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "description": "ESLint processor for RoundJS files",
   "type": "module",
   "main": "src/index.js",

package/src/processor.js

@@ -4,14 +4,8 @@ import { remapMessages } from './offset-map.js';
 
 export default {
   preprocess(code, filename) {
-    const mapper = new SourceMapper();
-    runPreprocess(code, mapper, 0);
-    
-    // MAGIC: Suppress false positives for RoundJS syntax without react-plugin
-    
-    // 1. Identify top-level names (imports and declarations)
     const topLevelNames = new Set();
-    const declMatches = mapper.code.matchAll(/^\s*(?:export\s+)?(?:function|const|let|var|class|import)\s+({[\s\S]*?}|[a-zA-Z0-9_$]+)/gm);
+    const declMatches = code.matchAll(/^\s*(?:export\s+)?(?:function|const|let|var|class|import)\s+({[\s\S]*?}|[a-zA-Z0-9_$]+)/gm);
     for (const m of declMatches) {
         const inner = m[1];
         if (inner.startsWith('{')) {
@@ -22,21 +16,28 @@ export default {
         }
     }
 
-    // 2. Mark components used in JSX as "used"
-    const cleanForComponents = mapper.code.replace(/`[\s\S]*?`/g, '');
-    const componentMatches = cleanForComponents.matchAll(/<([A-Z][a-zA-Z0-9_$]*)/g);
+    const componentMatches = code.matchAll(/<([A-Z][a-zA-Z0-9_$]*)/g);
     const components = new Set([...componentMatches].map(m => m[1]));
     
-    let magic = '\n\n/* eslint-disable */\n/* globals RoundControlFlow, e, item */\n;';
-    
-    // Usage for top-level components only
+    // Dynamic Global Injection for for/catch variables
+    const magicVars = new Set(['RoundControlFlow', 'Fragment']);
+    const forMatches = code.matchAll(/\{for\s*\(\s*([a-zA-Z0-9_$]+)/g);
+    for (const m of forMatches) magicVars.add(m[1]);
+    const catchMatches = code.matchAll(/catch\s*\(\s*([a-zA-Z0-9_$]+)/g);
+    for (const m of catchMatches) magicVars.add(m[1]);
+
+    const globalsList = Array.from(magicVars).join(', ');
+    let magic = `/* globals ${globalsList} */\n(void RoundControlFlow); (void Fragment);`;
     for (const comp of components) {
         if (topLevelNames.has(comp)) {
-            magic += `${comp};`; 
+            magic += ` (void ${comp});`; 
         }
     }
-    
-    mapper.add(magic, code.length);
+    magic += '\n';
+
+    const mapper = new SourceMapper();
+    runPreprocess(code, mapper, 0);
+    mapper.add('\n' + magic, code.length);
 
     fileState.set(filename, {
       source: code,
@@ -49,10 +50,6 @@ export default {
   postprocess(messages, filename) {
     const state = fileState.get(filename);
     
-    // Clean up state (optional, but good for memory if ESLint is long running)
-    // However, ESLint might run multiple fixes? 
-    // For now we keep it simple. If we delete here, we might lose it if 
-    // specific flows access it again. But usually postprocess is the end.
     fileState.delete(filename);
 
     if (!state) return messages[0];