Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement DOMBuilder approach to XML generation #204

Draft
wants to merge 33 commits into
base: schema-8
Choose a base branch
from

Conversation

jthrilly
Copy link
Member

@jthrilly jthrilly commented Nov 11, 2024

The idea here is to avoid the risky and potentially error prone string manipulation technique we use to generate XML, and to instead use the dom builder directly to create and manipulate elements and attributes.

The downside of this approach is that it is incompatible with streaming the result to disk.

WIP. Functioning, but without thorough testing.

Copy link

vercel bot commented Nov 11, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
fresco-sandbox ❌ Failed (Inspect) Feb 7, 2025 11:18am
1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
barf ⬜️ Ignored (Inspect) Visit Preview Feb 7, 2025 11:18am

dependabot bot and others added 13 commits January 20, 2025 12:40
Bumps [motion](https://github.com/motiondivision/motion) from 11.17.1 to 12.0.0.
- [Changelog](https://github.com/motiondivision/motion/blob/main/CHANGELOG.md)
- [Commits](motiondivision/motion@v11.17.1...v12.0.0)

---
updated-dependencies:
- dependency-name: motion
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [eslint-config-next](https://github.com/vercel/next.js/tree/HEAD/packages/eslint-config-next) from 15.1.4 to 15.1.5.
- [Release notes](https://github.com/vercel/next.js/releases)
- [Changelog](https://github.com/vercel/next.js/blob/canary/release.js)
- [Commits](https://github.com/vercel/next.js/commits/v15.1.5/packages/eslint-config-next)

---
updated-dependencies:
- dependency-name: eslint-config-next
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
…_yarn/next/eslint-config-next-15.1.5

Bump eslint-config-next from 15.1.4 to 15.1.5
…_yarn/next/motion-12.0.0

Bump motion from 11.17.1 to 12.0.0
Bumps the npm_and_yarn group with 2 updates in the / directory: [nanoid](https://github.com/ai/nanoid) and [vite](https://github.com/vitejs/vite/tree/HEAD/packages/vite).


Updates `nanoid` from 3.3.7 to 3.3.8
- [Release notes](https://github.com/ai/nanoid/releases)
- [Changelog](https://github.com/ai/nanoid/blob/main/CHANGELOG.md)
- [Commits](ai/nanoid@3.3.7...3.3.8)

Updates `vite` from 5.4.11 to 5.4.14
- [Release notes](https://github.com/vitejs/vite/releases)
- [Changelog](https://github.com/vitejs/vite/blob/v5.4.14/packages/vite/CHANGELOG.md)
- [Commits](https://github.com/vitejs/vite/commits/v5.4.14/packages/vite)

---
updated-dependencies:
- dependency-name: nanoid
  dependency-type: indirect
  dependency-group: npm_and_yarn
- dependency-name: vite
  dependency-type: indirect
  dependency-group: npm_and_yarn
...

Signed-off-by: dependabot[bot] <[email protected]>
…_yarn/npm_and_yarn-1dd916d70e

Bump the npm_and_yarn group across 1 directory with 2 updates
Bumps [@types/uuid](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/uuid) from 9.0.8 to 10.0.0.
- [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases)
- [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/uuid)

---
updated-dependencies:
- dependency-name: "@types/uuid"
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [@radix-ui/react-dialog](https://github.com/radix-ui/primitives) from 1.1.4 to 1.1.5.
- [Changelog](https://github.com/radix-ui/primitives/blob/main/release-process.md)
- [Commits](https://github.com/radix-ui/primitives/commits)

---
updated-dependencies:
- dependency-name: "@radix-ui/react-dialog"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser) from 8.20.0 to 8.21.0.
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.21.0/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/parser"
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [@radix-ui/react-alert-dialog](https://github.com/radix-ui/primitives) from 1.1.4 to 1.1.5.
- [Changelog](https://github.com/radix-ui/primitives/blob/main/release-process.md)
- [Commits](https://github.com/radix-ui/primitives/commits)

---
updated-dependencies:
- dependency-name: "@radix-ui/react-alert-dialog"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps [@radix-ui/react-select](https://github.com/radix-ui/primitives) from 2.1.4 to 2.1.5.
- [Changelog](https://github.com/radix-ui/primitives/blob/main/release-process.md)
- [Commits](https://github.com/radix-ui/primitives/commits)

---
updated-dependencies:
- dependency-name: "@radix-ui/react-select"
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
…_yarn/next/types/uuid-10.0.0

Bump @types/uuid from 9.0.8 to 10.0.0
…_yarn/next/radix-ui/react-select-2.1.5

Bump @radix-ui/react-select from 2.1.4 to 2.1.5
jthrilly and others added 4 commits January 28, 2025 12:16
…_yarn/next/radix-ui/react-alert-dialog-1.1.5

Bump @radix-ui/react-alert-dialog from 1.1.4 to 1.1.5
…_yarn/next/typescript-eslint/parser-8.21.0

Bump @typescript-eslint/parser from 8.20.0 to 8.21.0
…_yarn/next/radix-ui/react-dialog-1.1.5

Bump @radix-ui/react-dialog from 1.1.4 to 1.1.5
jthrilly and others added 3 commits February 4, 2025 20:52
trying to fix cannot find matching keyid issues
@jthrilly jthrilly changed the title experiment with dom builder approach to XML generation Implement DOMBuilder approach to XML generation Feb 4, 2025
@jthrilly jthrilly mentioned this pull request Feb 5, 2025
@jthrilly jthrilly changed the base branch from next to schema-8 February 7, 2025 08:59
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.

2 participants