minecraft-datapack-language 15.4.23__py3-none-any.whl → 15.4.26__py3-none-any.whl

minecraft_datapack_language/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '15.4.23'
-__version_tuple__ = version_tuple = (15, 4, 23)
+__version__ = version = '15.4.26'
+__version_tuple__ = version_tuple = (15, 4, 26)
 
 __commit_id__ = commit_id = None

minecraft_datapack_language/ast_nodes.py

@@ -40,6 +40,7 @@ class VariableDeclaration(ASTNode):
 @dataclass
 class VariableAssignment(ASTNode):
     name: str
+    scope: Optional[str]
     value: Any
 
 

minecraft_datapack_language/cli_build.py

@@ -176,7 +176,7 @@ def _generate_scoreboard_objectives(ast: Dict[str, Any], output_dir: Path) -> Li
             if 'body' in func:
                 for statement in func['body']:
                     # Look for variable assignments and usage
-                    if statement['type'] in ['variable_assignment', 'explicit_scope_assignment']:
+                    if statement['type'] == 'variable_assignment':
                         if statement['name'] not in seen_variables:
                             variables.append(statement['name'])
                             seen_variables.add(statement['name'])
@@ -280,7 +280,8 @@ def _process_say_command_with_variables(content: str, selector: str, variable_sc
                 var_selector = var_parts[1][:-1]  # Remove trailing >
                 components.append(f'{{"score":{{"name":"{var_selector}","objective":"{base_var}"}}}}')
             else:
-                # Simple variable: $variable$ - use declared scope if available, otherwise default to current selector
+                # Simple variable: $variable$ - determine selector based on declared scope
+                var_selector = "@e[type=armor_stand,tag=mdl_server,limit=1]"  # Default to global
                 if variable_scopes and var_name in variable_scopes:
                     declared_scope = variable_scopes[var_name]
                     if declared_scope == 'global':
@@ -289,8 +290,7 @@ def _process_say_command_with_variables(content: str, selector: str, variable_sc
                         var_selector = declared_scope
                     print(f"DEBUG: Variable {var_name} using declared scope {declared_scope} -> selector {var_selector}")
                 else:
-                    var_selector = selector  # Use current selector (@s)
-                    print(f"DEBUG: Variable {var_name} has no declared scope, using default selector {var_selector}")
+                    print(f"DEBUG: Variable {var_name} has no declared scope, using default global selector")
                 
                 components.append(f'{{"score":{{"name":"{var_selector}","objective":"{var_name}"}}}}')
     
@@ -336,25 +336,16 @@ def _process_statement(statement: Any, namespace: str, function_name: str, state
         var_name = statement['name']
         value = statement['value']
         
-        # For implicit assignments, always default to @s (current entity)
-        # Declared scopes only affect variable substitutions, not assignments
-        var_selector = selector  # Default to current selector (@s)
-        print(f"DEBUG: Variable {var_name} assignment using selector: {var_selector} (implicit scope, ignoring declared scope: {variable_scopes.get(var_name, 'none')})")
-    
-    elif statement['type'] == 'explicit_scope_assignment':
-        var_name = statement['name']
-        value = statement['value']
-        explicit_scope = statement['scope']
+        # Determine the correct selector for this variable based on its declared scope
+        var_selector = selector  # Default to current selector
+        if variable_scopes and var_name in variable_scopes:
+            declared_scope = variable_scopes[var_name]
+            if declared_scope == 'global':
+                var_selector = "@e[type=armor_stand,tag=mdl_server,limit=1]"
+            else:
+                var_selector = declared_scope
+        print(f"DEBUG: Variable {var_name} assignment using selector: {var_selector} (declared scope: {variable_scopes.get(var_name, 'none')})")
         
-        # For explicit scope assignments, use the specified scope
-        if explicit_scope == 'global':
-            var_selector = "@e[type=armor_stand,tag=mdl_server,limit=1]"
-        else:
-            var_selector = explicit_scope
-        print(f"DEBUG: Variable {var_name} assignment using explicit selector: {var_selector}")
-    
-    # Handle assignment value processing for both types
-    if statement['type'] in ['variable_assignment', 'explicit_scope_assignment']:
         # Handle different value types
         if isinstance(value, int):
             commands.append(f"scoreboard players set {var_selector} {var_name} {value}")
@@ -850,22 +841,22 @@ def _ast_to_pack(ast: Dict[str, Any], mdl_files: List[Path]) -> Pack:
                             else:
                                 # Simple function call without scope
                                 function.commands.append(f"function {func_namespace}:{func_name}")
-                        elif statement.get('type') in ['variable_assignment', 'explicit_scope_assignment']:
+                        elif statement.get('type') == 'variable_assignment':
                             # Handle variable assignments
                             var_name = statement['name']
                             value = statement['value']
                             
-                            # Determine selector based on assignment type
-                            if statement.get('type') == 'explicit_scope_assignment':
-                                # Explicit scope assignment - use specified scope
-                                explicit_scope = statement['scope']
-                                if explicit_scope == 'global':
-                                    var_selector = "@e[type=armor_stand,tag=mdl_server,limit=1]"
-                                else:
-                                    var_selector = explicit_scope
-                            else:
-                                # Implicit assignment - always default to @s
-                                var_selector = "@s"
+                            # Determine selector based on variable scope
+                            var_selector = "@s"  # Default
+                            if 'variables' in ast:
+                                for var_decl in ast['variables']:
+                                    if var_decl.get('name') == var_name:
+                                        var_scope = var_decl.get('scope')
+                                        if var_scope == 'global':
+                                            var_selector = "@e[type=armor_stand,tag=mdl_server,limit=1]"
+                                        elif var_scope:
+                                            var_selector = var_scope
+                                        break
                             
                             if hasattr(value, 'value'):
                                 # Simple literal value

minecraft_datapack_language/mdl_parser_js.py

@@ -316,15 +316,10 @@ class MDLParser:
                     suggestion="Replace 'for' with 'while' and adjust the loop structure"
                 )
             
-            # Check if this is a variable assignment
-            # Pattern 1: identifier = (simple assignment)
+            # Check if this is a variable assignment (identifier followed by =)
             if (self.current + 1 < len(self.tokens) and 
                 self.tokens[self.current + 1].type == TokenType.ASSIGN):
                 return self._parse_variable_assignment()
-            
-            # Pattern 2: identifier<scope> = (explicit scope assignment)
-            elif self._is_explicit_scope_assignment():
-                return self._parse_explicit_scope_assignment()
             else:
                 # Assume it's a command
                 return self._parse_command()
@@ -342,31 +337,29 @@ class MDLParser:
         
         # Check for scope selector after variable name
         scope = None
-        if not self._is_at_end() and self._peek().type == TokenType.SCOPE:
-            self._match(TokenType.SCOPE)  # consume 'scope'
+        if not self._is_at_end() and self._peek().type == TokenType.LANGLE:
+            self._match(TokenType.LANGLE)  # consume '<'
             
-            # Parse scope selector in angle brackets
-            if not self._is_at_end() and self._peek().type == TokenType.LANGLE:
-                self._match(TokenType.LANGLE)  # consume '<'
-                
-                # Parse scope selector content
-                scope_parts = []
-                while not self._is_at_end() and self._peek().type != TokenType.RANGLE:
-                    scope_parts.append(self._peek().value)
-                    self._advance()
-                
-                if self._is_at_end():
-                    raise create_parser_error(
-                        message="Unterminated scope selector",
-                        file_path=self.source_file,
-                        line=self._peek().line,
-                        column=self._peek().column,
-                        line_content=self._peek().value,
-                        suggestion="Add a closing '>' to terminate the scope selector"
-                    )
-                
-                self._match(TokenType.RANGLE)  # consume '>'
-                scope = ''.join(scope_parts)
+            # Parse scope selector content
+            scope_parts = []
+            while not self._is_at_end() and self._peek().type != TokenType.RANGLE:
+                scope_parts.append(self._peek().value)
+                self._advance()
+            
+            if self._is_at_end():
+                raise create_parser_error(
+                    message="Unterminated scope selector",
+                    file_path=self.source_file,
+                    line=self._peek().line,
+                    column=self._peek().column,
+                    line_content=self._peek().value,
+                    suggestion="Add a closing '>' to terminate the scope selector"
+                )
+            
+            self._match(TokenType.RANGLE)  # consume '>'
+            scope = ''.join(scope_parts)
+            # Update the name to include the scope selector
+            name = f"{name}<{scope}>"
         
         self._match(TokenType.ASSIGN)
         
@@ -382,80 +375,40 @@ class MDLParser:
         name_token = self._match(TokenType.IDENTIFIER)
         name = name_token.value
         
-        self._match(TokenType.ASSIGN)
-        
-        # Parse the value (could be a number or expression)
-        value = self._parse_expression()
-        
-        self._match(TokenType.SEMICOLON)
-        
-        return {"type": "variable_assignment", "name": name, "value": value}
-    
-    def _is_explicit_scope_assignment(self) -> bool:
-        """Check if current position is an explicit scope assignment pattern: var<scope> = value"""
-        if self.current >= len(self.tokens):
-            return False
+        # Check for scope selector after variable name
+        scope = None
+        if not self._is_at_end() and self._peek().type == TokenType.LANGLE:
+            self._match(TokenType.LANGLE)  # consume '<'
             
-        # Look for pattern: IDENTIFIER LANGLE ... RANGLE ASSIGN
-        idx = self.current + 1
-        
-        # Must have LANGLE after identifier
-        if idx >= len(self.tokens) or self.tokens[idx].type != TokenType.LANGLE:
-            return False
-        idx += 1
-        
-        # Skip tokens until we find RANGLE
-        while idx < len(self.tokens) and self.tokens[idx].type != TokenType.RANGLE:
-            idx += 1
-        
-        # Must find RANGLE
-        if idx >= len(self.tokens) or self.tokens[idx].type != TokenType.RANGLE:
-            return False
-        idx += 1
-        
-        # Must have ASSIGN after RANGLE
-        if idx >= len(self.tokens) or self.tokens[idx].type != TokenType.ASSIGN:
-            return False
+            # Parse scope selector content
+            scope_parts = []
+            while not self._is_at_end() and self._peek().type != TokenType.RANGLE:
+                scope_parts.append(self._peek().value)
+                self._advance()
             
-        return True
-    
-    def _parse_explicit_scope_assignment(self) -> dict:
-        """Parse variable assignment with explicit scope: var<scope> = value"""
-        # Parse variable name
-        name_token = self._match(TokenType.IDENTIFIER)
-        name = name_token.value
-        
-        # Parse scope: <scope>
-        self._match(TokenType.LANGLE)
-        
-        # Parse scope selector content
-        scope_parts = []
-        while not self._is_at_end() and self._peek().type != TokenType.RANGLE:
-            scope_parts.append(self._peek().value)
-            self._advance()
-        
-        if self._is_at_end():
-            raise create_parser_error(
-                message="Unterminated scope selector in assignment",
-                file_path=self.source_file,
-                line=self._peek().line,
-                column=self._peek().column,
-                line_content=self._peek().value,
-                suggestion="Add a closing '>' to terminate the scope selector"
-            )
-        
-        self._match(TokenType.RANGLE)
-        scope = ''.join(scope_parts)
+            if self._is_at_end():
+                raise create_parser_error(
+                    message="Unterminated scope selector",
+                    file_path=self.source_file,
+                    line=self._peek().line,
+                    column=self._peek().column,
+                    line_content=self._peek().value,
+                    suggestion="Add a closing '>' to terminate the scope selector"
+                )
+            
+            self._match(TokenType.RANGLE)  # consume '>'
+            scope = ''.join(scope_parts)
+            # Update the name to include the scope selector
+            name = f"{name}<{scope}>"
         
-        # Parse assignment
         self._match(TokenType.ASSIGN)
         
-        # Parse the value
+        # Parse the value (could be a number or expression)
         value = self._parse_expression()
         
         self._match(TokenType.SEMICOLON)
         
-        return {"type": "explicit_scope_assignment", "name": name, "scope": scope, "value": value}
+        return {"type": "variable_assignment", "name": name, "scope": scope, "value": value}
     
     def _parse_if_statement(self) -> IfStatement:
         """Parse if statement."""
@@ -756,8 +709,47 @@ class MDLParser:
         """Parse a command."""
         command_parts = []
         while not self._is_at_end() and self._peek().type != TokenType.SEMICOLON:
-            command_parts.append(self._peek().value)
-            self._advance()
+            current_token = self._peek()
+            
+            # Check if this is an identifier that might be followed by a scope selector
+            if current_token.type == TokenType.IDENTIFIER:
+                identifier_name = current_token.value
+                command_parts.append(identifier_name)
+                self._advance()  # consume the identifier
+                
+                # Look ahead to see if there's a scope selector
+                if not self._is_at_end() and self._peek().type == TokenType.LANGLE:
+                    # This is a scoped variable - parse the scope selector
+                    self._match(TokenType.LANGLE)  # consume '<'
+                    
+                    # Parse scope selector content
+                    scope_parts = []
+                    while not self._is_at_end() and self._peek().type != TokenType.RANGLE:
+                        scope_parts.append(self._peek().value)
+                        self._advance()
+                    
+                    if self._is_at_end():
+                        raise create_parser_error(
+                            message="Unterminated scope selector in command",
+                            file_path=self.source_file,
+                            line=self._peek().line,
+                            column=self._peek().column,
+                            line_content=self._peek().value,
+                            suggestion="Add a closing '>' to terminate the scope selector"
+                        )
+                    
+                    self._match(TokenType.RANGLE)  # consume '>'
+                    scope_selector = ''.join(scope_parts)
+                    
+                    # Add the scope selector to the command parts
+                    command_parts.append(f"<{scope_selector}>")
+                else:
+                    # No scope selector, continue with next token
+                    continue
+            else:
+                # Regular token, just add it
+                command_parts.append(current_token.value)
+                self._advance()
         
         if self._is_at_end():
             raise create_parser_error(
@@ -907,9 +899,10 @@ class MDLParser:
             identifier_name = token.value
             self._advance()  # consume the identifier
             
-            # Check for explicit scope syntax: identifier<scope>
+            # Check if this identifier is followed by a scope selector
             if not self._is_at_end() and self._peek().type == TokenType.LANGLE:
-                self._advance()  # consume <
+                # This is a scoped variable - parse the scope selector
+                self._match(TokenType.LANGLE)  # consume '<'
                 
                 # Parse scope selector content
                 scope_parts = []
@@ -927,21 +920,12 @@ class MDLParser:
                         suggestion="Add a closing '>' to terminate the scope selector"
                     )
                 
-                self._advance()  # consume >
+                self._match(TokenType.RANGLE)  # consume '>'
                 scope_selector = ''.join(scope_parts)
                 
-                # Return variable expression with explicit scope encoded in name
-                return VariableExpression(f"{identifier_name}<{scope_selector}>")
-            
-            # Check if the identifier contains a scope selector (legacy single-token format)
-            elif '<' in identifier_name and identifier_name.endswith('>'):
-                # Extract variable name and scope selector
-                parts = identifier_name.split('<', 1)
-                if len(parts) == 2:
-                    var_name = parts[0]
-                    scope_selector = parts[1][:-1]  # Remove the closing >
-                    # For variable expressions in assignments, keep the full scoped name
-                    return VariableExpression(identifier_name)
+                # Create a scoped variable expression
+                full_name = f"{identifier_name}<{scope_selector}>"
+                return VariableExpression(full_name)
             
             # Regular variable expression without scope
             return VariableExpression(identifier_name)

{minecraft_datapack_language-15.4.23.dist-info → minecraft_datapack_language-15.4.26.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: minecraft-datapack-language
-Version: 15.4.23
+Version: 15.4.26
 Summary: Compile JavaScript-style MDL language or Python API 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

{minecraft_datapack_language-15.4.23.dist-info → minecraft_datapack_language-15.4.26.dist-info}/RECORD

@@ -1,8 +1,8 @@
 minecraft_datapack_language/__init__.py,sha256=i-qCchbe5b2Fshgc6yCU9mddOLs2UBt9SAcLqfUIrT0,606
-minecraft_datapack_language/_version.py,sha256=2M9DVfO2B0gW3xckA3jnSFVOsLpE1n8q1owBMbxNoIU,708
-minecraft_datapack_language/ast_nodes.py,sha256=pgjI2Nlap3ixFPgWqGSkqncG9zB91h5BKgRjtcJqMew,2118
+minecraft_datapack_language/_version.py,sha256=ZFtqs_SG6x6U-SOyZin31273WN7DoOUGVgDsZjWRysA,708
+minecraft_datapack_language/ast_nodes.py,sha256=U-CB3R7p7OjaljI47BWsRrQtliw0yAWUrJVoxzOLmQw,2143
 minecraft_datapack_language/cli.py,sha256=p5A_tEEXugN2NhQFbbgfwi4FxbWYD91RWeKR_A3Vuec,6263
-minecraft_datapack_language/cli_build.py,sha256=hid5hdIwd3NVppBoatkij_OeH9HsBjKTT1WseRdSUs0,48510
+minecraft_datapack_language/cli_build.py,sha256=u0XIOH_zTARPC6dvWf-411OqrF7RjT7Z9Hkp6hTeZsI,47990
 minecraft_datapack_language/cli_check.py,sha256=bPq9gHsxQ1CIiftkrAtRCifWkVAyjp5c8Oay2NNQ1qs,6277
 minecraft_datapack_language/cli_colors.py,sha256=Hr8awY966bGSnVdXL3WnmRhSP1wH56vTQKGt5z-kIQM,7878
 minecraft_datapack_language/cli_help.py,sha256=Rc-v9E2kctsdN_lMunqdKuZ8EZ8rcZIjBCOPrXLBQeE,21363
@@ -14,12 +14,12 @@ minecraft_datapack_language/linter.py,sha256=7UqbygC5JPCGg-BSOq65NB2xEJBu_OUOYII
 minecraft_datapack_language/mdl_errors.py,sha256=mz6uyPkeBpbMHj4PiAyVecEVJ9_hdSfR45QAjG6oYf0,15690
 minecraft_datapack_language/mdl_lexer_js.py,sha256=VvbhKm727khdSAABxa03hoIIA7H3hWi3RLp9BSXbhY0,28277
 minecraft_datapack_language/mdl_linter.py,sha256=z85xoAglENurCh30bR7kEHZ_JeMxcYaLDcGNRAl-RAI,17253
-minecraft_datapack_language/mdl_parser_js.py,sha256=ttnxvZCHKwRu51IL9U0w7WIX1q19qXNJJYt0D4f4a8M,44149
+minecraft_datapack_language/mdl_parser_js.py,sha256=kaeTxzQDRDUj4YdpeR9Yrp8d8JCuOwmd36HZSeSrCgY,43992
 minecraft_datapack_language/pack.py,sha256=nYiXQ3jgJlDfc4m-65f7C2LFhDRioaUU_XVy6Na4SJI,34625
 minecraft_datapack_language/utils.py,sha256=Aq0HAGlXqj9BUTEjaEilpvzEW0EtZYYMMwOqG9db6dE,684
-minecraft_datapack_language-15.4.23.dist-info/licenses/LICENSE,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
-minecraft_datapack_language-15.4.23.dist-info/METADATA,sha256=dnH_xWsKU808MSG335P3esNgF5C7H8ujM-O-7hs6nb4,37917
-minecraft_datapack_language-15.4.23.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-minecraft_datapack_language-15.4.23.dist-info/entry_points.txt,sha256=c6vjBeCiyQflvPHBRyBk2nJCSfYt3Oc7Sc9V87ySi_U,108
-minecraft_datapack_language-15.4.23.dist-info/top_level.txt,sha256=ADtFI476tbKLLxEAA-aJQAfg53MA3k_DOb0KTFiggfw,28
-minecraft_datapack_language-15.4.23.dist-info/RECORD,,
+minecraft_datapack_language-15.4.26.dist-info/licenses/LICENSE,sha256=OXLcl0T2SZ8Pmy2_dmlvKuetivmyPd5m1q-Gyd-zaYY,35149
+minecraft_datapack_language-15.4.26.dist-info/METADATA,sha256=CP-fUjVPMIttFRw5N2w_ElF9baJ9wBCYjSbOjmSD-4A,37917
+minecraft_datapack_language-15.4.26.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+minecraft_datapack_language-15.4.26.dist-info/entry_points.txt,sha256=c6vjBeCiyQflvPHBRyBk2nJCSfYt3Oc7Sc9V87ySi_U,108
+minecraft_datapack_language-15.4.26.dist-info/top_level.txt,sha256=ADtFI476tbKLLxEAA-aJQAfg53MA3k_DOb0KTFiggfw,28
+minecraft_datapack_language-15.4.26.dist-info/RECORD,,