rip-lang 2.8.2 → 2.8.4

package/README.md

@@ -9,7 +9,7 @@
 </p>
 
 <p align="center">
-  <a href="CHANGELOG.md"><img src="https://img.shields.io/badge/version-2.8.2-blue.svg" alt="Version"></a>
+  <a href="CHANGELOG.md"><img src="https://img.shields.io/badge/version-2.8.4-blue.svg" alt="Version"></a>
   <a href="#zero-dependencies"><img src="https://img.shields.io/badge/dependencies-ZERO-brightgreen.svg" alt="Dependencies"></a>
   <a href="#"><img src="https://img.shields.io/badge/tests-1021%2F1021-brightgreen.svg" alt="Tests"></a>
   <a href="LICENSE"><img src="https://img.shields.io/badge/license-MIT-green.svg" alt="License"></a>

package/bin/rip

@@ -1,6 +1,6 @@
 #!/usr/bin/env bun
 
-import { readFileSync, writeFileSync, existsSync } from 'fs';
+import { readFileSync, writeFileSync, existsSync, statSync } from 'fs';
 import { execSync, spawn } from 'child_process';
 import { fileURLToPath } from 'url';
 import { dirname, join } from 'path';
@@ -183,9 +183,13 @@ async function main() {
     }
   }
 
+  // Helper to check if path is a regular file (not a directory)
+  const isFile = (path) => existsSync(path) && statSync(path).isFile();
+
   // Fallback: Check for bin/ script in git repo root
   // Allows `rip migrate --status` to find and run {repo}/bin/migrate
-  if (inputFile && !inputFile.startsWith('-') && !existsSync(inputFile)) {
+  // Also triggers if inputFile is a directory (not a compilable file)
+  if (inputFile && !inputFile.startsWith('-') && !isFile(inputFile)) {
     try {
       // Check if we're in a git repo
       const repoRoot = execSync('git rev-parse --show-toplevel', {
@@ -220,8 +224,8 @@ async function main() {
       // Read from stdin if no file specified (file descriptor 0)
       source = readFileSync(0, 'utf-8');
     } else {
-      // Check if file exists first
-      if (!existsSync(inputFile)) {
+      // Check if file exists and is a regular file
+      if (!isFile(inputFile)) {
         console.error(`Error: File not found: ${inputFile}`);
         process.exit(1);
       }

package/docs/dist/rip.browser.js

@@ -1946,16 +1946,16 @@ Rewriter = function() {
           val.generated = true;
           return tokens.splice(idx, 0, generate("{", val, token, prevToken));
         };
-        endImplicitObject = function(j) {
-          j = j != null ? j : i;
+        endImplicitObject = function(j2) {
+          j2 = j2 != null ? j2 : i;
           stack.pop();
-          tokens.splice(j, 0, generate("}", "}", token, prevToken));
+          tokens.splice(j2, 0, generate("}", "}", token, prevToken));
           return i += 1;
         };
-        implicitObjectContinues = (j) => {
+        implicitObjectContinues = (j2) => {
           var nextTerminatorIdx;
           nextTerminatorIdx = null;
-          this.detectEnd(j, function(token2) {
+          this.detectEnd(j2, function(token2) {
             return token2[0] === "TERMINATOR";
           }, function(token2, i2) {
             return nextTerminatorIdx = i2;
@@ -2100,12 +2100,35 @@ Rewriter = function() {
           }
         }
         newLine = prevTag === "OUTDENT" || prevToken.newLine;
-        if (indexOf.call(IMPLICIT_END, tag) >= 0 || indexOf.call(CALL_CLOSERS, tag) >= 0 && newLine || (tag === ".." || tag === "...") && this.findTagsBackwards(i, ["INDEX_START"])) {
+        var isLogicalOp = tag === "||" || tag === "&&";
+        var logicalOpHasMoreArgs = false;
+        if (isLogicalOp && i + 1 < tokens.length) {
+          var nextTok = tokens[i + 1][0];
+          var j = i + 1;
+          if (nextTok === "(" || nextTok === "[" || nextTok === "{") {
+            var depth = 1;
+            j++;
+            while (j < tokens.length && depth > 0) {
+              var lt = tokens[j][0];
+              if (lt === "(" || lt === "[" || lt === "{")
+                depth++;
+              else if (lt === ")" || lt === "]" || lt === "}")
+                depth--;
+              j++;
+            }
+          } else if (nextTok !== "TERMINATOR" && nextTok !== "OUTDENT" && nextTok !== ",") {
+            j++;
+          }
+          if (j < tokens.length && tokens[j][0] === ",") {
+            logicalOpHasMoreArgs = true;
+          }
+        }
+        if (indexOf.call(IMPLICIT_END, tag) >= 0 && !(isLogicalOp && logicalOpHasMoreArgs) || indexOf.call(CALL_CLOSERS, tag) >= 0 && newLine || (tag === ".." || tag === "...") && this.findTagsBackwards(i, ["INDEX_START"])) {
           while (inImplicit()) {
             [stackTag, stackIdx, { sameLine, startsLine }] = stackTop();
             if (inImplicitCall() && prevTag !== "," || prevTag === "," && tag === "TERMINATOR" && nextTag == null) {
               endImplicitCall();
-            } else if (inImplicitObject() && sameLine && tag !== "TERMINATOR" && prevTag !== ":" && !((tag === "POST_IF" || tag === "FOR" || tag === "WHILE" || tag === "UNTIL") && startsLine && implicitObjectContinues(i + 1))) {
+            } else if (inImplicitObject() && !isLogicalOp && sameLine && tag !== "TERMINATOR" && prevTag !== ":" && !((tag === "POST_IF" || tag === "FOR" || tag === "WHILE" || tag === "UNTIL") && startsLine && implicitObjectContinues(i + 1))) {
               endImplicitObject();
             } else if (inImplicitObject() && tag === "TERMINATOR" && prevTag !== "," && !(startsLine && this.looksObjectish(i + 1))) {
               endImplicitObject();
@@ -2670,6 +2693,7 @@ var parserInstance = {
       case 267:
       case 281:
       case 285:
+      case 289:
       case 344:
       case 350:
         return [$[$0]];
@@ -2679,6 +2703,7 @@ var parserInstance = {
       case 213:
       case 236:
       case 268:
+      case 290:
         return [...$[$0 - 2], $[$0]];
       case 5:
       case 63:
@@ -2766,7 +2791,6 @@ var parserInstance = {
       case 273:
       case 274:
       case 276:
-      case 289:
       case 309:
       case 342:
       case 358:
@@ -3093,8 +3117,6 @@ var parserInstance = {
         return [...$[$0 - 5], ...$[$0 - 4], ...$[$0 - 2], ...$[$0 - 1]];
       case 284:
         return [...$[$0]];
-      case 290:
-        return Array.isArray($[$0 - 2]) ? [...$[$0 - 2], $[$0]] : [$[$0 - 2], $[$0]];
       case 291:
         return ["try", $[$0]];
       case 292:
@@ -3116,9 +3138,9 @@ var parserInstance = {
       case 299:
         return ["throw", $[$0 - 1]];
       case 300:
-        return $[$0 - 1].length === 1 ? $[$0 - 1][0] : $[$0 - 1];
+        return $[$0 - 1].length === 1 ? $[$0 - 1][0] : ["block", ...$[$0 - 1]];
       case 301:
-        return $[$0 - 2].length === 1 ? $[$0 - 2][0] : $[$0 - 2];
+        return $[$0 - 2].length === 1 ? $[$0 - 2][0] : ["block", ...$[$0 - 2]];
       case 302:
         return ["while", $[$0]];
       case 303:
@@ -5182,7 +5204,21 @@ ${stmts.join(`
 `)}
 ${this.indent()}}`;
     }
-    return this.formatStatements(statements, context);
+    if (statements.length === 0)
+      return "undefined";
+    if (statements.length === 1)
+      return this.generate(statements[0], context);
+    const last = statements[statements.length - 1];
+    const lastIsControl = Array.isArray(last) && ["break", "continue", "return", "throw"].includes(last[0]);
+    if (lastIsControl) {
+      const bodyParts = statements.map((s) => this.addSemicolon(s, this.generate(s, "statement")));
+      return `{
+${this.withIndent(() => bodyParts.map((p) => this.indent() + p).join(`
+`))}
+${this.indent()}}`;
+    }
+    const parts = statements.map((s) => this.generate(s, "value"));
+    return `(${parts.join(", ")})`;
   }
   generateTry(head, rest, context, sexpr) {
     const needsReturns = context === "value";
@@ -5281,9 +5317,7 @@ ${this.indent()}}`;
     const normalize = (v) => v instanceof String ? v.valueOf() : v;
     for (const whenClause of whens) {
       const [, test, body] = whenClause;
-      const firstTest = normalize(Array.isArray(test) && test.length > 0 ? test[0] : null);
-      const isTestList = Array.isArray(test) && test.length > 0 && typeof firstTest === "string" && !firstTest.match(/^[-+*\/%<>=!&|^~]$|^(typeof|delete|new|not|await|yield)$/);
-      const tests = isTestList ? test : [test];
+      const tests = test;
       for (const t of tests) {
         const tValue = normalize(t);
         let caseValue;
@@ -6887,18 +6921,31 @@ ${this.indent()}}`;
     if (typeof code !== "string")
       return code;
     while (code.startsWith("(") && code.endsWith(")")) {
-      let depth = 0;
+      let parenDepth = 0;
+      let bracketDepth = 0;
       let canUnwrap = true;
+      let hasCommaAtDepth1 = false;
       for (let i = 0;i < code.length; i++) {
-        if (code[i] === "(")
-          depth++;
-        if (code[i] === ")")
-          depth--;
-        if (depth === 0 && i < code.length - 1) {
+        const ch = code[i];
+        if (ch === "(")
+          parenDepth++;
+        if (ch === ")")
+          parenDepth--;
+        if (ch === "[" || ch === "{")
+          bracketDepth++;
+        if (ch === "]" || ch === "}")
+          bracketDepth--;
+        if (ch === "," && parenDepth === 1 && bracketDepth === 0) {
+          hasCommaAtDepth1 = true;
+        }
+        if (parenDepth === 0 && i < code.length - 1) {
           canUnwrap = false;
           break;
         }
       }
+      if (hasCommaAtDepth1) {
+        canUnwrap = false;
+      }
       if (canUnwrap) {
         code = code.slice(1, -1);
       } else {
@@ -7124,11 +7171,12 @@ ${this.indent()}}`;
     for (let i = 0;i < whens.length; i++) {
       const whenClause = whens[i];
       const [, test, body] = whenClause;
+      const condition = Array.isArray(test) ? test[0] : test;
       if (i === 0) {
-        code += `if (${this.generate(test, "value")}) {
+        code += `if (${this.generate(condition, "value")}) {
 `;
       } else {
-        code += ` else if (${this.generate(test, "value")}) {
+        code += ` else if (${this.generate(condition, "value")}) {
 `;
       }
       this.indentLevel++;
@@ -7511,8 +7559,8 @@ function compileToJS(source, options = {}) {
   return new Compiler(options).compileToJS(source);
 }
 // src/browser.js
-var VERSION = "2.8.2";
-var BUILD_DATE = "2026-02-04@07:57:52GMT";
+var VERSION = "2.8.4";
+var BUILD_DATE = "2026-02-04@10:18:01GMT";
 var dedent = (s) => {
   const m = s.match(/^[ \t]*(?=\S)/gm);
   const i = Math.min(...(m || []).map((x) => x.length));