Skip to content

[proto] add canonical organization membership type#2

Merged
haasonsaas merged 1 commit intomainfrom
jh/organization-membership
Apr 12, 2026
Merged

[proto] add canonical organization membership type#2
haasonsaas merged 1 commit intomainfrom
jh/organization-membership

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

@haasonsaas haasonsaas commented Apr 12, 2026

Summary

  • add identity.v1.OrganizationMembership as the canonical org-plus-role view
  • keep Organization entity-scoped and avoid pushing membership context onto it
  • regenerate Go and TypeScript outputs

Why

Chat's ListWorkspaces needs a shared type for "organization plus this user's role in it". That is a membership-scoped view, not an intrinsic property of Organization.

Validation

  • make lint generate
  • go build ./...

Follow-up

  • consumed by evalops/chat#885

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 12, 2026

PR Summary

Low Risk
Low risk schema addition: introduces a new protobuf message and regenerates Go/TS code, with no behavioral logic changes. Main risk is downstream compile/runtime incompatibilities for consumers that assume previous generated message indices/types.

Overview
Adds a new protobuf message OrganizationMembership to represent an Organization plus the current user’s role, keeping membership-scoped fields off Organization.

Regenerates Go and TypeScript outputs to include the new type and shifts generated message indices/descriptor metadata for existing Member and APIKey types accordingly.

Reviewed by Cursor Bugbot for commit 679cb8a. Bugbot is set up for automated code reviews on this repo. Configure here.

@haasonsaas haasonsaas merged commit 5567d86 into main Apr 12, 2026
6 of 7 checks passed
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