Skip to content

feat: clarify Draco export ownership - #6

Open
alexchuber wants to merge 6 commits into
rebase-scene-stack-rootfrom
draco-stack-layer
Open

feat: clarify Draco export ownership#6
alexchuber wants to merge 6 commits into
rebase-scene-stack-rootfrom
draco-stack-layer

Conversation

@alexchuber

@alexchuber alexchuber commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • replace the misleading raw-encoder connection payload with typed per-export compression options
  • keep encoder preparation, extension registration, and Draco activation inside the opt-in block
  • keep glTF output neutral and preserve per-export compression isolation
  • split connection-point definitions by payload, colocate block infrastructure, and flatten the core node asset modules

Validation

  • 8/8 focused code-review agents completed on the Draco ownership changes
  • fixed the shared runtime guard ownership finding
  • pnpm test (59 tests)
  • pnpm format:check
  • pnpm lint
  • pnpm build
  • git diff --check

alexchuber and others added 2 commits September 10, 2026 01:28
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@alexchuber
alexchuber added this pull request to stack #7 September 10, 2026 05:32
alexchuber and others added 4 commits September 11, 2026 22:48
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>
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