PyPI - minecraft-datapack-language - Versions diffs - 17.0.19__tar.gz → 17.0.21__tar.gz - Mend

Files changed (439) hide show

{minecraft_datapack_language-17.0.19 → minecraft_datapack_language-17.0.21}/minecraft_datapack_language/mdl_compiler.py RENAMED Viewed

@@ -1251,18 +1251,18 @@ class MDLCompiler:
             if isinstance(expression.right, BinaryExpression):
                 self._store_temp_command(f"scoreboard players operation @s {temp_var} *= @s {right_temp}")
             else:
-                # For literal values, keep explicit multiply command for compatibility
+                # Normalize and use operation with a temp constant to maximize compatibility
                 if isinstance(expression.right, LiteralExpression):
-                    # Normalize number formatting (e.g., 2.0 -> 2)
                     literal_str = self._normalize_integer_literal_string(self._expression_to_value(expression.right), ctx="multiply literal")
                     if literal_str == "0":
                         self._store_temp_command(f"scoreboard players set @s {temp_var} 0")
                     elif literal_str == "1":
                         pass
                     else:
-                        self._store_temp_command(f"scoreboard players multiply @s {temp_var} {literal_str}")
+                        const_var = self._generate_temp_variable_name()
+                        self._store_temp_command(f"scoreboard players set @s {const_var} {literal_str}")
+                        self._store_temp_command(f"scoreboard players operation @s {temp_var} *= @s {const_var}")
                 else:
-                    # If right_value is a score reference string, strip the leading 'score '
                     if isinstance(right_value, str) and right_value.startswith("score "):
                         parts = right_value.split()
                         if len(parts) >= 3:
@@ -1272,7 +1272,6 @@ class MDLCompiler:
                         else:
                             self._store_temp_command(f"scoreboard players operation @s {temp_var} *= {right_value}")
                     else:
-                        # If RHS is a numeric literal string (incl. negatives), use multiply
                         if self._is_numeric_literal_string(right_value):
                             lit = self._normalize_integer_literal_string(str(right_value), ctx="multiply literal string")
                             if lit == "0":
@@ -1280,7 +1279,9 @@ class MDLCompiler:
                             elif lit == "1":
                                 pass
                             else:
-                                self._store_temp_command(f"scoreboard players multiply @s {temp_var} {lit}")
+                                const_var = self._generate_temp_variable_name()
+                                self._store_temp_command(f"scoreboard players set @s {const_var} {lit}")
+                                self._store_temp_command(f"scoreboard players operation @s {temp_var} *= @s {const_var}")
                         else:
                             self._store_temp_command(f"scoreboard players operation @s {temp_var} *= {right_value}")
@@ -1299,9 +1300,8 @@ class MDLCompiler:
             if isinstance(expression.right, BinaryExpression):
                 self._store_temp_command(f"scoreboard players operation @s {temp_var} /= @s {right_temp}")
             else:
-                # For literal values, keep explicit divide command for compatibility
+                # Normalize and use operation with a temp constant to maximize compatibility
                 if isinstance(expression.right, LiteralExpression):
-                    # Normalize number formatting (e.g., 2.0 -> 2)
                     lit = self._normalize_integer_literal_string(self._expression_to_value(expression.right), ctx="divide literal")
                     if lit == "0":
                         raise MDLCompilerError("Division by zero literal is not allowed", "Use a non-zero integer literal")
@@ -1309,12 +1309,17 @@ class MDLCompiler:
                         pass
                     elif lit.startswith("-"):
                         abs_lit = lit[1:]
-                        self._store_temp_command(f"scoreboard players divide @s {temp_var} {abs_lit}")
-                        self._store_temp_command(f"scoreboard players multiply @s {temp_var} -1")
+                        const_div = self._generate_temp_variable_name()
+                        self._store_temp_command(f"scoreboard players set @s {const_div} {abs_lit}")
+                        self._store_temp_command(f"scoreboard players operation @s {temp_var} /= @s {const_div}")
+                        const_neg = self._generate_temp_variable_name()
+                        self._store_temp_command(f"scoreboard players set @s {const_neg} -1")
+                        self._store_temp_command(f"scoreboard players operation @s {temp_var} *= @s {const_neg}")
                     else:
-                        self._store_temp_command(f"scoreboard players divide @s {temp_var} {lit}")
+                        const_div = self._generate_temp_variable_name()
+                        self._store_temp_command(f"scoreboard players set @s {const_div} {lit}")
+                        self._store_temp_command(f"scoreboard players operation @s {temp_var} /= @s {const_div}")
                 else:
-                    # If right_value is a score reference string, strip the leading 'score '
                     if isinstance(right_value, str) and right_value.startswith("score "):
                         parts = right_value.split()
                         if len(parts) >= 3:
@@ -1324,7 +1329,6 @@ class MDLCompiler:
                         else:
                             self._store_temp_command(f"scoreboard players operation @s {temp_var} /= {right_value}")
                     else:
-                        # If RHS is a numeric literal string (incl. negatives), use divide
                         if self._is_numeric_literal_string(right_value):
                             lit = self._normalize_integer_literal_string(str(right_value), ctx="divide literal string")
                             if lit == "0":
@@ -1333,10 +1337,16 @@ class MDLCompiler:
                                 pass
                             elif lit.startswith("-"):
                                 abs_lit = lit[1:]
-                                self._store_temp_command(f"scoreboard players divide @s {temp_var} {abs_lit}")
-                                self._store_temp_command(f"scoreboard players multiply @s {temp_var} -1")
+                                const_div = self._generate_temp_variable_name()
+                                self._store_temp_command(f"scoreboard players set @s {const_div} {abs_lit}")
+                                self._store_temp_command(f"scoreboard players operation @s {temp_var} /= @s {const_div}")
+                                const_neg = self._generate_temp_variable_name()
+                                self._store_temp_command(f"scoreboard players set @s {const_neg} -1")
+                                self._store_temp_command(f"scoreboard players operation @s {temp_var} *= @s {const_neg}")
                             else:
-                                self._store_temp_command(f"scoreboard players divide @s {temp_var} {lit}")
+                                const_div = self._generate_temp_variable_name()
+                                self._store_temp_command(f"scoreboard players set @s {const_div} {lit}")
+                                self._store_temp_command(f"scoreboard players operation @s {temp_var} /= @s {const_div}")
                         else:
                             self._store_temp_command(f"scoreboard players operation @s {temp_var} /= {right_value}")
         else:

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: minecraft-datapack-language
-Version: 17.0.19
+Version: 17.0.21
 Summary: Compile MDL language with explicit scoping into a Minecraft datapack (1.21+ ready). Features variables, control flow, error handling, and VS Code extension.
 Project-URL: Homepage, https://www.mcmdl.com
 Project-URL: Documentation, https://www.mcmdl.com/docs

@@ -0,0 +1,3 @@
+current: "17.0.20"
+tag: "v17.0.20"
+updated_at: "2025-09-11T03:00:28Z"

@@ -1307,7 +1307,7 @@ def _parse_comparison(self) -> Any:
 - Scoreboard arithmetic is integer-only. MDL normalizes integer-like literals: `2.0` -> `2`. Non-integer literals (e.g., `2.5`) cause a compile-time error when used in scoreboard math.
 - Literal addition/subtraction uses `scoreboard players add/remove`; elides `+0`/`-0`.
-- Literal multiplication/division uses `scoreboard players multiply/divide`; elides `*1`/`/1`. `*0` sets the temp to zero. Division by zero is a compile-time error.
+- Literal multiplication/division uses temporary constants with `scoreboard players operation` (e.g., set a temp score to the constant, then `*=`/`/=`). `*1`/`/1` are elided; `*0` sets the temp to zero. Division by zero is a compile-time error.
 - Mixed expressions are lowered via temporary scores to preserve precedence. Score-to-score operations use `scoreboard players operation`.
 ### AST Traversal and Code Generation

@@ -0,0 +1,3 @@
+current: "17.0.19"
+tag: "v17.0.19"
+updated_at: "2025-09-11T00:53:51Z"

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
-__version__ = version = '17.0.19'
-__version_tuple__ = version_tuple = (17, 0, 19)
+__version__ = version = '17.0.21'
+__version_tuple__ = version_tuple = (17, 0, 21)
-__commit_id__ = commit_id = 'g8f2457ec2'
+__commit_id__ = commit_id = 'g1acb1fc92'

@@ -1,3 +0,0 @@
-current: "17.0.18"
-tag: "v17.0.18"
-updated_at: "2025-09-11T00:49:05Z"

@@ -1,3 +0,0 @@
-current: "17.0.17"
-tag: "v17.0.17"
-updated_at: "2025-09-11T00:46:17Z"

minecraft-datapack-language 17.0.19__tar.gz → 17.0.21__tar.gz

minecraft-datapack-language 17.0.19tar.gz → 17.0.21tar.gz