Skip to content

Add ValidateBlock for glTF validation - #19

Merged
alexchuber merged 5 commits into
mainfrom
validate-block
Sep 18, 2026
Merged

alexchuber merged 5 commits into
mainfrom
validate-block

Conversation

@alexchuber

Copy link
Copy Markdown
Contributor

Summary

  • Add ValidateBlock to validate a Document with the Khronos glTF Validator and return the same document on success.
  • Fail on validation errors, log grouped diagnostics with locations, and ignore UNSUPPORTED_EXTENSION.
  • Add the CLI validate operation and load the validator only when used.
  • Cover real-document behavior, CLI failures, and compressed pipelines without mocking validator internals.

alexchuber and others added 5 commits September 17, 2026 17:31
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@alexchuber
alexchuber merged commit 4d487b5 into main Sep 18, 2026
2 checks passed
alexchuber added a commit to alexchuber/Node-Assets that referenced this pull request Sep 21, 2026
* feat: add document validation block

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

* fix: decouple validation tests and reduce copies

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

* docs: trim validation documentation

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

* docs: simplify CLI validation wording

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

* refactor: remove validation label option

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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