Skip to content

Comments

feat: add stream prefix options.proto#32

Merged
yordis merged 1 commit intomainfrom
stream-prefix
Feb 12, 2026
Merged

feat: add stream prefix options.proto#32
yordis merged 1 commit intomainfrom
stream-prefix

Conversation

@yordis
Copy link
Member

@yordis yordis commented Feb 12, 2026

Summary

  • Add trogon.stream.v1alpha1 options for stream identity prefixes (extension number 870011)
  • Supports prefix and optional separator (defaults to ":") for aggregate stream naming
  • Update field number registry and examples in extension naming docs

Test plan

  • buf lint passes
  • buf build passes

@cursor
Copy link

cursor bot commented Feb 12, 2026

PR Summary

Low Risk
Adds a new protobuf options definition and documentation updates only; no runtime behavior changes unless downstream codegen/consumers start using the new extension.

Overview
Introduces proto/trogon/stream/v1alpha1/options.proto, adding an EnumValueOptions wrapper and a google.protobuf.EnumValueOptions extension (enum_value = 870011) to attach a required prefix and optional separator for stream identity naming.

Updates protobuf-extension-naming.md to reserve extension number 870011 for trogon.stream.v1alpha1, shift the available range to 870012–870999, and include the new module in the examples list.

Written by Cursor Bugbot for commit e85af42. This will update automatically on new commits. Configure here.

@github-actions
Copy link

github-actions bot commented Feb 12, 2026

The latest Buf updates on your PR. Results from workflow Buf CI / buf (pull_request).

BuildFormatLintBreakingUpdated (UTC)
✅ passed✅ passed✅ passed✅ passedFeb 12, 2026, 10:05 PM

@coderabbitai
Copy link

coderabbitai bot commented Feb 12, 2026

Warning

Rate limit exceeded

@yordis has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 25 minutes and 7 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch stream-prefix

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Signed-off-by: Yordis Prieto <yordis.prieto@gmail.com>
@yordis yordis merged commit 49483b9 into main Feb 12, 2026
5 checks passed
@yordis yordis deleted the stream-prefix branch February 12, 2026 22:07
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