Skip to content

Extract descriptor types into buffa-descriptor crate#8

Merged
iainmcgin merged 1 commit intomainfrom
buffa-descriptor-crate
Mar 26, 2026
Merged

Extract descriptor types into buffa-descriptor crate#8
iainmcgin merged 1 commit intomainfrom
buffa-descriptor-crate

Conversation

@iainmcgin
Copy link
Copy Markdown
Collaborator

Moves the generated FileDescriptorProto/DescriptorProto types and
vendored descriptor.proto sources from buffa-codegen into a new
buffa-descriptor crate that depends only on buffa. This makes the
descriptor types available without pulling in quote/syn/prettyplease,
laying the foundation for runtime reflection.

buffa-codegen re-exports buffa_descriptor::generated so all existing
crate::generated:: paths resolve unchanged. gen_descriptor_types now
takes an explicit output directory argument, eliminating the previous
cwd-sensitivity.

@iainmcgin iainmcgin requested a review from rpb-ant March 23, 2026 23:35
@iainmcgin iainmcgin marked this pull request as ready for review March 23, 2026 23:35
rpb-ant
rpb-ant previously approved these changes Mar 24, 2026
Moves the generated FileDescriptorProto/DescriptorProto types and
vendored descriptor.proto sources from buffa-codegen into a new
buffa-descriptor crate that depends only on buffa. This makes the
descriptor types available without pulling in quote/syn/prettyplease,
laying the foundation for runtime reflection.

buffa-codegen re-exports buffa_descriptor::generated so all existing
crate::generated:: paths resolve unchanged. gen_descriptor_types now
takes an explicit output directory argument, eliminating the previous
cwd-sensitivity.
@iainmcgin iainmcgin force-pushed the buffa-descriptor-crate branch from 93b88c2 to e1e0949 Compare March 24, 2026 20:54
@iainmcgin iainmcgin merged commit d2a5e42 into main Mar 26, 2026
7 checks passed
@iainmcgin iainmcgin deleted the buffa-descriptor-crate branch March 26, 2026 18:46
@github-actions github-actions bot locked and limited conversation to collaborators Mar 26, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants