Skip to content

build: accept in-memory descriptor set bytes - #488

Open
Yong-yuan-X wants to merge 1 commit into
anthropics:mainfrom
Yong-yuan-X:fix/407-descriptor-set-bytes
Open

Yong-yuan-X wants to merge 1 commit into
anthropics:mainfrom
Yong-yuan-X:fix/407-descriptor-set-bytes

Conversation

@Yong-yuan-X

Copy link
Copy Markdown
Contributor

Summary

Build scripts that prepare descriptor bytes in memory currently need to write a temporary file before passing them to buffa-build.

Add Config::descriptor_set_bytes accepting Vec<u8> or &[u8], while preserving the existing path-based descriptor_set API. Owned vectors are accepted without copying, and callers manage Cargo rebuild directives for the sources of the bytes.

Add regression coverage for output parity with file input, file selection, imported descriptors, reflection sidecars, source replacement, and malformed input, along with an unreleased changelog fragment.

Closes #407.

Validation

  • All 44 buffa-build unit tests pass.
  • The new documentation example compiles.
  • Formatting, workspace Clippy, and strict buffa-build Rustdoc checks pass.

@github-actions

Copy link
Copy Markdown

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

This branch has not been deployed

No deployments
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.

Allow passing descriptor_set as bytes instead of a file path

1 participant