Skip to content

Add entity merge and split contract#78

Merged
haasonsaas merged 1 commit intomainfrom
codex/proto-entities-merge-split
Apr 15, 2026
Merged

Add entity merge and split contract#78
haasonsaas merged 1 commit intomainfrom
codex/proto-entities-merge-split

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

Summary\n- add MergeEntities and SplitEntity RPCs to entities.v1.EntityService\n- add ref-key, merge response, and split response messages for canonical lifecycle changes\n- regenerate Go, Python, and TypeScript bindings\n\n## Testing\n- buf generate --path proto/entities/v1/entities.proto --timeout 10m\n- buf lint --path proto/entities/v1/entities.proto\n- go test ./... -count=1\n- npm run check:package\n\nPartially addresses evalops/entities#11.

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 15, 2026

PR Summary

Medium Risk
Adds new RPCs and message types to a public API contract and regenerates multi-language bindings, which can impact client/server compatibility if versions drift or implementations lag.

Overview
Extends entities.v1.EntityService with two new unary RPCs: MergeEntities and SplitEntity, enabling canonical-entity lifecycle operations at the API level.

Introduces supporting proto messages (CrossSystemRefKey, MergeEntitiesRequest/Response, SplitEntityRequest/Response) and regenerates the Go, Connect-Go, Python, and TypeScript client/server stubs to include the new methods and types.

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

@haasonsaas haasonsaas merged commit cb4f038 into main Apr 15, 2026
9 checks passed
@haasonsaas haasonsaas deleted the codex/proto-entities-merge-split branch April 15, 2026 14:59
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