overpy 9.6.4 → 9.6.6

package/README.md

@@ -602,6 +602,10 @@ If specified, allows redefining a `macro`, `#!define` or `enum` member. Can be u
 
 Will generate a summary of the used elements per rule in the output, and add a comment with the number of elements used for each rule/condition/action.
 
+## #!writeToOutputFile
+
+If specified, the compiled code will not be copied to the clipboard but instead written to a file with the same name as the main file, but with a `.ws.txt` extension (eg `myGamemode.opy` will produce `myGamemode.ws.txt`).
+
 ## Optimizations
 
 By default, OverPy automatically optimizes gamemodes for speed. This means useless code is removed, calculations are done when possible, and function patterns are replaced with builtin functions.

package/overpy.d.ts

@@ -18,6 +18,7 @@ declare namespace overpy {
         availableExtensionPoints: number;
         translationLanguages: string[];
         translatedStrings: unknown[];
+        writeToOutputFile: boolean;
     }
 
     function compile(

package/overpy.js

@@ -38582,6 +38582,10 @@ ${scriptText}`, {
       setDisableInspector(true);
       return;
     }
+    if (content2.startsWith("#!writeToOutputFile")) {
+      setWriteToOutputFile(true);
+      return;
+    }
     if (content2.startsWith("#!disableTranslationSourceLines")) {
       setDisableTranslationSourceLines(true);
       return;
@@ -38605,19 +38609,19 @@ ${scriptText}`, {
       return;
     }
     if (content2.startsWith("#!optimizeForSize")) {
-      addToken("__enableOptimizeForSize__");
+      currentLine.tokens.push(new Token("__enableOptimizeForSize__", getFileStackCopy()));
       return;
     }
     if (content2.startsWith("#!disableOptimizeForSize")) {
-      addToken("__disableOptimizeForSize__");
+      currentLine.tokens.push(new Token("__disableOptimizeForSize__", getFileStackCopy()));
       return;
     }
     if (content2.startsWith("#!optimizeStrict")) {
-      addToken("__enableOptimizeStrict__");
+      currentLine.tokens.push(new Token("__enableOptimizeStrict__", getFileStackCopy()));
       return;
     }
     if (content2.startsWith("#!disableOptimizeStrict")) {
-      addToken("__disableOptimizeStrict__");
+      currentLine.tokens.push(new Token("__disableOptimizeStrict__", getFileStackCopy()));
       return;
     }
     if (content2.startsWith("#!setupTx") || content2.startsWith("#!setupTags")) {
@@ -38682,10 +38686,10 @@ ${scriptText}`, {
     }
     if (content2.startsWith("#!rulePrefix ")) {
       let prefix = content2.substring("#!rulePrefix ".length).trim();
-      addToken("__rulePrefix__");
-      addToken("(");
-      addToken(prefix);
-      addToken(")");
+      currentLine.tokens.push(new Token("__rulePrefix__", getFileStackCopy()));
+      currentLine.tokens.push(new Token("(", getFileStackCopy()));
+      currentLine.tokens.push(new Token(prefix, getFileStackCopy()));
+      currentLine.tokens.push(new Token(")", getFileStackCopy()));
       return;
     }
     if (content2.startsWith("#!rulePrefixTemplate")) {
@@ -39686,6 +39690,10 @@ var Ast2 = class _Ast {
   //Used for translated strings, if set to true then player var won't be used
   forceNotResolvingTranslation;
   //Used for translated strings, if true then the resulting string array won't be indexed and will need to be indexed later on with the _() function
+  isGotoInSameScope;
+  //Used for gotos, if true then the goto doesn't jump to another scope and can be optimized more easily. For now, only set on automatically generated gotos
+  isSwitchIf;
+  //true if it is the "if true" from a switch
   argIndex = 0;
   childIndex = 0;
   wasParsed = false;
@@ -39761,6 +39769,8 @@ var Ast2 = class _Ast {
     clone.numValue = this.numValue;
     clone.stringTokens = structuredClone(this.stringTokens);
     clone.isSpectatorTranslation = this.isSpectatorTranslation;
+    clone.isGotoInSameScope = this.isGotoInSameScope;
+    clone.isSwitchIf = this.isSwitchIf;
     return clone;
   }
 };
@@ -39880,6 +39890,15 @@ function astContainsFunctions(ast, functionNames, errorOnTrue = false) {
   }
   return false;
 }
+function astIsInLambdaFunction(ast) {
+  if (!ast.parent) {
+    return false;
+  }
+  if (["__filteredArray__", "__mappedArray__"].includes(ast.parent.name) && ast.parent.argIndex === 1) {
+    return true;
+  }
+  return astIsInLambdaFunction(ast.parent);
+}
 function astIsLiteral(ast) {
   if (typeof ast.type === "string" && !["Hero", "Map", "Gamemode", "Team", "Button", "Color"].includes(ast.type)) {
     if (["IntLiteral", "UnsignedIntLiteral", "SignedIntLiteral", "FloatLiteral", "UnsignedFloatLiteral", "SignedFloatLiteral", "GlobalVariable", "PlayerVariable", "Subroutine", "HeroLiteral", "MapLiteral", "GamemodeLiteral", "TeamLiteral", "ButtonLiteral", "StringLiteral", "LocalizedStringLiteral", "CustomStringLiteral"].concat(Object.keys(constantValues)).includes(ast.type)) {
@@ -44948,6 +44967,9 @@ astParsingFunctions.__equals__ = function(content) {
 
 // src/compiler/functions/__filteredArray__.ts
 astParsingFunctions.__filteredArray__ = function(content) {
+  if (astIsInLambdaFunction(content)) {
+    error("Cannot nest .filter() or .map()");
+  }
   if (enableOptimization) {
     if (!optimizeForSize2) {
       if (!astContainsFunctions(content.args[1], ["__currentArrayElement__", "__currentArrayIndex__"])) {
@@ -45199,7 +45221,9 @@ astParsingFunctions.__if__ = function(content) {
       } else {
         content.args[0] = astParsingFunctions.__not__(new Ast2("__not__", [content.args[0]]));
       }
-      return new Ast2("__skipIf__", [content.args[0], new Ast2("__distanceTo__", [new Ast2(label, [], [], "Label")])]);
+      let skip = new Ast2("__skipIf__", [content.args[0], new Ast2("__distanceTo__", [new Ast2(label, [], [], "Label")])]);
+      skip.isGotoInSameScope = true;
+      return skip;
     }
     if (includeEnd) {
       content.parent.children.splice(content.parent.childIndex + 1, 0, getAstForEnd());
@@ -45311,6 +45335,9 @@ astParsingFunctions.__map__ = function(content) {
 
 // src/compiler/functions/__mappedArray__.ts
 astParsingFunctions.__mappedArray__ = function(content) {
+  if (astIsInLambdaFunction(content)) {
+    error("Cannot nest .filter() or .map()");
+  }
   if (enableOptimization) {
     if (content.args[1].name === "__currentArrayElement__") {
       return content.args[0];
@@ -45536,10 +45563,10 @@ astParsingFunctions.__rule__ = function(content) {
   function iterateOnRuleActions(children) {
     for (let i2 = 0; i2 < children.length; i2++) {
       setFileStack(content.fileStack);
-      if (children[i2].name === "__skip__" && children[i2].args[0].name !== "__distanceTo__" || children[i2].name === "__skipIf__" && children[i2].args[1].name !== "__distanceTo__") {
+      if ((children[i2].name === "__skip__" && children[i2].args[0].name !== "__distanceTo__" || children[i2].name === "__skipIf__" && children[i2].args[1].name !== "__distanceTo__") && !children[i2].isGotoInSameScope) {
         isRelativeGotoEncountered = true;
       }
-      if (children[i2].name === "__skip__" || children[i2].name === "__skipIf__") {
+      if ((children[i2].name === "__skip__" || children[i2].name === "__skipIf__") && !children[i2].isGotoInSameScope) {
         isGotoEncountered = true;
       }
       if (content.type === "Label") {
@@ -45549,7 +45576,7 @@ astParsingFunctions.__rule__ = function(content) {
         declaredLabels.push(content.name);
       }
       iterateOnRuleActions(children[i2].children);
-      if (!isRelativeGotoEncountered) {
+      if (!isRelativeGotoEncountered && !children[i2].isSwitchIf) {
         if (children[i2].name === "pass") {
           children.splice(i2, 1);
           i2--;
@@ -45819,7 +45846,10 @@ astParsingFunctions.__switch__ = function(content) {
     error("Parent is undefined in __switch__");
   }
   content.parent.children.splice(content.parent.childIndex + 1, 0, ...casesChildren);
-  var result = new Ast2("__if__", [getAstForTrue()], [new Ast2("__skip__", [new Ast2("__valueInArray__", [new Ast2("__array__", caseOffsets), new Ast2("__add__", [getAstFor1(), new Ast2(".index", [new Ast2("__array__", switchCaseArgs), content.args[0]])])])])]);
+  let skip = new Ast2("__skip__", [new Ast2("__valueInArray__", [new Ast2("__array__", caseOffsets), new Ast2("__add__", [getAstFor1(), new Ast2(".index", [new Ast2("__array__", switchCaseArgs), content.args[0]])])])]);
+  skip.isGotoInSameScope = true;
+  var result = new Ast2("__if__", [getAstForTrue()], [skip]);
+  result.isSwitchIf = true;
   result.doNotReparse = true;
   return result;
 };
@@ -65001,7 +65031,7 @@ function astToWs(content) {
         }
         str = str.replaceAll("\uEC51", separator);
         return astToWs(new Ast2(".split", [getAstForCustomString(str), separatorAst]));
-      } else if (content.args.length >= 3 && content.args.every((x) => areAstsAlwaysEqual(x, content.args[0]))) {
+      } else if (!astIsInLambdaFunction(content) && content.args.length >= 3 && content.args.every((x) => areAstsAlwaysEqual(x, content.args[0]))) {
         let estimatedElementCount = 1 + (content.args[0].name === "__customString__" ? 4 : content.args[0].args.length);
         if (estimatedElementCount === 1 && content.args.length >= 11 || estimatedElementCount === 2 && content.args.length >= 5 || estimatedElementCount === 3 && content.args.length >= 4 || estimatedElementCount >= 4) {
           return astToWs(new Ast2("__mappedArray__", [
@@ -65068,13 +65098,17 @@ function astToWs(content) {
     content.name = newName;
   } else if (content.name === "__globalVar__") {
     incrementNbElements();
+    content.argIndex = 0;
     return tows("__global__", valueKw) + "." + astToWs(content.args[0]);
   } else if (content.name === "__number__") {
     incrementNbElements(2);
     return trimNb(content.args[0].name);
   } else if (content.name === "__playerVar__") {
     incrementNbElements();
-    return "(" + astToWs(content.args[0]) + ")." + astToWs(content.args[1]);
+    content.argIndex = 0;
+    let result2 = "(" + astToWs(content.args[0]) + ").";
+    content.argIndex = 1;
+    return result2 + astToWs(content.args[1]);
   } else if (content.name === "ceil") {
     content.name = "__round__";
     content.args = [content.args[0], new Ast2("__roundUp__", [], [], "__Rounding__")];
@@ -65204,6 +65238,7 @@ function astToWs(content) {
       if (content.args[i].type === "void") {
         error("Expected a value, but got " + functionNameToString(content.args[i]) + " which is an action", content.args[i].fileStack);
       }
+      content.argIndex = i;
       result += astToWs(content.args[i]);
       if (content.type === "void") {
         incrementNbElements(Math.floor(nbHeroesInValue / 2));
@@ -65824,7 +65859,8 @@ rule "Disable inspector":
     spentExtensionPoints,
     availableExtensionPoints,
     translationLanguages: translationLanguages2,
-    translatedStrings
+    translatedStrings,
+    writeToOutputFile
   };
 }
 function compileRules(astRules) {
@@ -69776,6 +69812,8 @@ var rulePrefixTemplateFilestack = [];
 var setRulePrefixTemplateFilestack = (filestack) => rulePrefixTemplateFilestack = filestack;
 var useVariableForCompressionAlphabet = false;
 var setUseVariableForCompressionAlphabet = (use) => useVariableForCompressionAlphabet = use;
+var writeToOutputFile = false;
+var setWriteToOutputFile = (write) => writeToOutputFile = write;
 var decompilerGotos;
 var resetDecompilerGotos = () => decompilerGotos = [];
 var nbTabs;
@@ -69857,6 +69895,7 @@ function resetGlobalVariables(language) {
   rulePrefixTemplate = "";
   rulePrefixTemplateFilestack = [];
   useVariableForCompressionAlphabet = false;
+  writeToOutputFile = false;
 }
 var operatorPrecedence = {
   "=": 1,
@@ -72341,6 +72380,9 @@ You can also specify \`noTlErr\` to have spectators view the default language wh
   "disableTranslationSourceLines": {
     "description": "If set, the source lines of the translations will not be included in the generated .po files. Use this if you are not actively translating your gamemode, to prevent cluttering git diffs."
   },
+  "writeToOutputFile": {
+    "description": "If specified, the compiled code will not be copied to the clipboard but instead written to a file with the same name as the main file, but with a `.ws.txt` extension (eg `myGamemode.opy` will produce `myGamemode.ws.txt`)."
+  },
   "postCompileHook": {
     "description": `
 Specifies a JavaScript file to be executed after compilation, with the compiled code as a \`content\` variable. The script must return the modified code.

package/package.json

@@ -7,7 +7,7 @@
         "url": "https://github.com/Zezombye/overpy"
     },
     "description": "High-level language for the Overwatch Workshop, with decompilation and compilation.",
-    "version": "9.6.4",
+    "version": "9.6.6",
     "readme": "README.md",
     "keywords": [
         "overpy",