microsoft / microsoft/typespec

[protobuf] Refactor protobuf transform module

Open
#1,855 0 comments 0 reactions 1 assignee Claimed by @witemple-msft View on GitHub
emitter:protobuf feature triaged:core
Dominant language
Java
Stars
5.9k
Forks
394
Avg merge
1d 23h
Merged PRs (30d)
104

Description

The transformation module (packages/protobuf/src/transform/index.ts) grew to an unwieldy state as I was developing the protobuf emitter, and this issue tracks some refactoring work to be completed after merging:

- [ ] Extract common emitter state into a context object.
- [ ] Split transform module into smaller modules with related functionality.
- [ ] Add state (stateMap/stateSet) accessors that provide a strongly-typed wrapper API over the compiler API.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.