refactor(stovepipe): make IngestController operate on entities#361
Merged
Conversation
ubettigole
changed the base branch from
abettigole/protoless_cancel
to
abettigole/protoless_status
This was referenced Jul 14, 2026
ubettigole
force-pushed
the
abettigole/protoless_status
branch
from
2177a97 to
a3b7ac5
Compare
ubettigole
force-pushed
the
abettigole/protoless_ingest
branch
from
efa048e to
6383702
Compare
ubettigole
changed the title
ubettigole
changed the title
ubettigole
requested review from
a team,
behinddwalls and
sbalabanov
as code owners
behinddwalls
approved these changes
Jul 14, 2026
behinddwalls
force-pushed
the
abettigole/protoless_status
branch
from
a3b7ac5 to
0d3ad54
Compare
behinddwalls
force-pushed
the
abettigole/protoless_ingest
branch
from
6383702 to
1298dd5
Compare
behinddwalls
force-pushed
the
abettigole/protoless_status
branch
from
0d3ad54 to
07c6c36
Compare
behinddwalls
force-pushed
the
abettigole/protoless_ingest
branch
from
1298dd5 to
272e677
Compare
roychying
pushed a commit
to roychying/submitqueue
that referenced
this pull request
Jul 14, 2026
## Summary Move proto<->entity translation out of the gateway Land controller and into a dedicated mapper package. The controller now takes entity.LandRequest and returns entity.LandResult, keeping business logic proto-free; the GatewayServer adapter maps the wire request in and the response out. - add entity.LandResult; consolidate LandRequest/LandResult into entity/land.go - add service/.../server/mapper with ProtoToLandRequest + strategy resolution - keep field validation in the controller so it runs on controller-to-controller calls Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> ## Test Plan ## Issues ## Stack 1. @ uber#359 1. uber#360 1. uber#363 1. uber#361 Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
roychying
pushed a commit
to roychying/submitqueue
that referenced
this pull request
Jul 14, 2026
## Summary Remove proto coupling from CancelController.Cancel by changing its signature from (*pb.CancelRequest) (*pb.CancelResponse, error) to (entity.CancelRequest) error. The empty CancelResponse is now constructed at the handler layer. Add mapper.ProtoToCancelRequest in the mapper subpackage and wire it through GatewayServer.Cancel. Rename entity/cancel_request.go to entity/cancel.go for consistency with entity/land.go. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com> ## Test Plan ## Issues ## Stack 1. uber#359 1. @ uber#360 1. uber#363 1. uber#361 Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Remove proto coupling from IngestController.Ingest by changing its signature
to accept entity.IngestRequest and return entity.IngestResult.
Add entity.IngestRequest{Queue} and entity.IngestResult{ID} in stovepipe/entity/.
Create service/stovepipe/server/mapper/ package with ProtoToIngestRequest and
IngestResultToProto, and wire them through StovepipeServer.Ingest.
This is the first non-gateway domain to follow the protoless controller pattern,
standing up the mapper subpackage for stovepipe.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
behinddwalls
force-pushed
the
abettigole/protoless_ingest
branch
from
272e677 to
e09b121
Compare
roychying
pushed a commit
to roychying/submitqueue
that referenced
this pull request
Jul 14, 2026
## Summary
Remove proto coupling from StatusController.Status by changing its
signature
to accept entity.StatusRequest and return request.CurrentState directly.
Add mapper.ProtoToStatusRequest and mapper.CurrentStateToProto in the
mapper
subpackage. The Status mapper is the first with a non-trivial
entity-to-proto
direction (translating the CurrentState read model into
pb.StatusResponse).
Add entity.StatusRequest{ID} for input consistency with the other
controllers.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
## Test Plan
## Issues
## Stack
1. uber#359
1. uber#360
1. @ uber#363
1. uber#361
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Remove proto coupling from IngestController.Ingest by changing its signature
to accept entity.IngestRequest and return entity.IngestResult directly.
Add mapper.ProtoToIngestRequest and mapper.IngestResultToProto in the mapper
subpackage.
Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com
Test Plan
Unit tests
Issues
Stack