Skip to content

XML: Node DOM + response caps for xml_query/xml_structure#44

Merged
mateicanavra merged 4 commits into
mainfrom
fix/xml-dom-parsing
Aug 8, 2025
Merged

XML: Node DOM + response caps for xml_query/xml_structure#44
mateicanavra merged 4 commits into
mainfrom
fix/xml-dom-parsing

Conversation

@mateicanavra

Copy link
Copy Markdown
Collaborator
  • Replace global DOMParser with @xmldom/xmldom for Node compatibility.
  • Parse full files; cap only response size (new optional maxResponseBytes; legacy maxBytes as response cap).
  • Implement truncation for xml_query and xml_structure with metadata.
  • Keep fast-xml-parser for xml_to_json.

Build/tests green.

…nt variables in expandHome function, and implement corresponding unit tests for validation.
…tes/legacy maxBytes) for xml_to_json and xml_to_json_string; summarize when exceeding cap
@mateicanavra

Copy link
Copy Markdown
Collaborator Author

Follow-up: extended response-size caps to xml_to_json and xml_to_json_string. These tools now parse full files and only cap/summary the JSON output using optional maxResponseBytes (legacy maxBytes respected). Build/tests remain green.

@mateicanavra

Copy link
Copy Markdown
Collaborator Author

Added xml_tools integration tests covering:

  • xml_structure structure + namespace extraction
  • xml_query local-name() and response truncation
  • xml_to_json_string response caps
  • xml_to_json write with response caps

Suite passes locally (39 pass).

@mateicanavra mateicanavra merged commit af942bd into main Aug 8, 2025
2 checks passed
@mateicanavra mateicanavra deleted the fix/xml-dom-parsing branch August 8, 2025 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant