rip-lang 2.9.2 → 3.0.0

package/src/repl.js

@@ -30,7 +30,6 @@ const colors = {
   red: '\x1b[31m',
   green: '\x1b[32m',
   yellow: '\x1b[33m',
-  blue: '\x1b[34m',
   magenta: '\x1b[35m',
   cyan: '\x1b[36m',
   gray: '\x1b[90m'
@@ -152,7 +151,7 @@ export class RipREPL {
       const c = {
         dependencies: new Set(),
         markDirty() {
-          if (dead || locked || !dirty) { if (!dead && !locked && !dirty) { dirty = true; for (const s of subs) if (s.markDirty) s.markDirty(); for (const s of subs) if (!s.markDirty) ctx.__pendingEffects.add(s); } }
+          if (!dead && !locked && !dirty) { dirty = true; for (const s of subs) if (s.markDirty) s.markDirty(); for (const s of subs) if (!s.markDirty) ctx.__pendingEffects.add(s); }
         },
         get value() {
           if (dead) return v;
@@ -268,7 +267,6 @@ export class RipREPL {
 
     if (trimmed.endsWith('+') || trimmed.endsWith('-')) return false;
     if (trimmed.endsWith('*') && !trimmed.endsWith('**')) return false;
-    if (trimmed.endsWith('/') && !trimmed.endsWith('//') && !/\/[gimsuvy]*$/.test(trimmed)) return false;
 
     return true;
   }
@@ -369,8 +367,8 @@ export class RipREPL {
   async moduleEval(js) {
     // Extract declared variables (both let and const)
     const declaredVars = new Set();
-    for (const match of js.matchAll(/^let\s+(\w+)/gm)) {
-      declaredVars.add(match[1]);
+    for (const match of js.matchAll(/^let\s+([^=]+);$/gm)) {
+      for (const name of match[1].split(/\s*,\s*/)) declaredVars.add(name.trim());
     }
     for (const match of js.matchAll(/^const\s+(\w+)\s*=/gm)) {
       declaredVars.add(match[1]);
@@ -405,8 +403,16 @@ export class RipREPL {
     const existingVars = Object.keys(this.vars);
     const existingNonReactive = existingVars.filter(v => !this.reactiveVars.has(v));
 
+    // Remove existing vars from hoisted let declarations (handles comma-separated)
+    const existingSet = new Set(existingNonReactive);
+    js = js.replace(/^let\s+([^=]+);$/gm, (match, varList) => {
+      const vars = varList.split(/\s*,\s*/);
+      const newVars = vars.filter(v => !existingSet.has(v));
+      if (newVars.length === 0) return '';
+      return `let ${newVars.join(', ')};`;
+    });
+    // Remove let from initialized existing vars: let x = → x =
     for (const v of existingNonReactive) {
-      js = js.replace(new RegExp(`^let ${v};\\n?`, 'm'), '');
       js = js.replace(new RegExp(`^let ${v}(\\s*=)`, 'm'), `${v}$1`);
     }
 
@@ -609,9 +615,10 @@ ${colors.cyan}Tips:${colors.reset}
     }
 
     console.log(`${colors.bright}Command history:${colors.reset}`);
-    this.history.slice(-20).forEach((cmd, i) => {
-      const num = this.history.length - 20 + i + 1;
-      console.log(`  ${colors.gray}${num}:${colors.reset} ${cmd.replace(/\n/g, '⏎ ')}`);
+    const recent = this.history.slice(-20);
+    const startIdx = this.history.length - recent.length;
+    recent.forEach((cmd, i) => {
+      console.log(`  ${colors.gray}${startIdx + i + 1}:${colors.reset} ${cmd.replace(/\n/g, '⏎ ')}`);
     });
   }