Skip to content

Use protocol-grpc 0.13 APIs - #7

Merged
samuel-williams-shopify merged 1 commit into
mainfrom
use-call-routing
Aug 17, 2026
Merged

Use protocol-grpc 0.13 APIs#7
samuel-williams-shopify merged 1 commit into
mainfrom
use-call-routing

Conversation

@samuel-williams-shopify

@samuel-williams-shopify samuel-williams-shopify commented Aug 13, 2026

Copy link
Copy Markdown
Contributor

Adopt the routing and metadata APIs released in protocol-grpc 0.13.0.

  • Parse incoming request paths with Route.parse and build test paths with Route.build.
  • Build request headers with Metadata.build and extract response metadata with Metadata.extract.
  • Construct Call.for(request, response) only after the response exists, keeping the call fully initialized and its response read-only.
  • Require protocol-grpc ~> 0.13.0 and remove the temporary Git commit pin.
  • Remove all remaining uses of the deprecated Protocol::GRPC::Methods namespace.

Validation:

  • bundle exec sus test/async/grpc/client.rb test/async/grpc/dispatcher.rb
  • bundle exec bake test
  • bundle exec rubocop (35 files, no offenses)
  • bundle exec bake decode:index:coverage lib (26/26 public definitions documented)

@samuel-williams-shopify samuel-williams-shopify changed the title Use call routing information Use gRPC route helpers Aug 16, 2026
@samuel-williams-shopify
samuel-williams-shopify force-pushed the use-call-routing branch 2 times, most recently from 24d22fa to 1edb11e Compare August 16, 2026 23:23
@samuel-williams-shopify samuel-williams-shopify changed the title Use gRPC route helpers Use protocol-grpc 0.13 APIs Aug 16, 2026
@samuel-williams-shopify
samuel-williams-shopify marked this pull request as ready for review August 17, 2026 00:01
@samuel-williams-shopify
samuel-williams-shopify merged commit 2a5d91a into main Aug 17, 2026
34 of 40 checks passed
@samuel-williams-shopify
samuel-williams-shopify deleted the use-call-routing branch August 17, 2026 00:02
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.

1 participant