Add validation for text outside of tags

[ntindle]

This pull request improves the robustness of the parser by adding explicit error handling for unsupported raw text outside of tags, and introduces new tests to verify this behavior.

Parser error handling improvements:

• Added an add_text method to the List class in gravitasml/parser.py that raises a ValueError when raw text is encountered outside of a tag, ensuring unsupported text is flagged early.
• Updated the parser logic to pass the whole Token object to add_text for better error reporting, instead of just the text value.

Test coverage enhancements:

• Added two new tests in tests/test_parser.py to confirm that a ValueError is raised when parsing documents with text outside of root tags, or with only raw text and no tags.

---

Note

Adds validation to raise a ValueError for raw text outside tags and updates tests to cover trailing/text-only cases.

• Parser (gravitasml/parser.py):
  • Add List.add_text(token: Token) to raise ValueError for raw text outside tags with line/column info.
  • Update Parser.parse to pass the full Token to add_text instead of just the string value.
• Tests (tests/test_parser.py):
  • Add tests asserting ValueError for trailing text after a root tag and for text-only documents.

^{Written by Cursor Bugbot for commit c5e0cac. This will update automatically on new commits. Configure here.}

[qodo-code-review]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪

🧪 PR contains tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Error Message Consistency Ensure the error message format in add_text (including line/column presence) consistently matches expectations across different tokenization scenarios so tests and downstream consumers receive predictable messages. def add_text(self, token: Token): """ Handles raw text when the parser is not currently inside a node. """ text = token.value if not text or text.isspace(): return raise ValueError( f"Text outside of a tag is unsupported at line {token.line_num}, column {token.column}" ) Parser State Coverage Verify that raw text encountered when current is neither List nor Node (e.g., at initialization or after closing the last tag) is always handled via List.add_text to avoid silent acceptance or inconsistent behavior. for t in self.tokens: if t.type == "TEXT": if isinstance(self.current, List): self.current.add_text(t) elif isinstance(self.current, Node): self.current.value += t.value # type: Node elif t.type == "TAG_OPEN":

[Copilot]

Pull request overview

This PR enhances the parser's robustness by adding explicit validation that rejects raw text outside of XML/HTML-like tags. Previously, such text would be silently ignored, but now it raises a descriptive ValueError with line and column information.

• Adds an add_text method to the List class that validates and rejects non-whitespace text outside of tags
• Updates the parser to pass full Token objects (instead of just values) for better error reporting
• Introduces comprehensive test coverage for the new validation behavior

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
gravitasml/parser.py	Adds add_text method to List class for validating text outside tags and updates the parse loop to pass Token objects
tests/test_parser.py	Adds two new test cases verifying ValueError is raised for text outside root tags and text-only documents