Skip to content

Add record-name support to entity correlation#80

Merged
haasonsaas merged 1 commit intomainfrom
codex/entities-fuzzy-correlation
Apr 15, 2026
Merged

Add record-name support to entity correlation#80
haasonsaas merged 1 commit intomainfrom
codex/entities-fuzzy-correlation

Conversation

@haasonsaas
Copy link
Copy Markdown
Contributor

@haasonsaas haasonsaas commented Apr 15, 2026

Summary

  • add record_name to entities.v1.CorrelateRequest
  • regenerate Go, Python, and TypeScript bindings
  • keep fuzzy record-name correlation available through the public API for downstream services

Testing

  • buf lint
  • buf generate --path proto/entities/v1/entities.proto --timeout 10m
  • go test ./... -count=1
  • npm ci
  • npm run check:package

@cursor
Copy link
Copy Markdown

cursor bot commented Apr 15, 2026

PR Summary

Medium Risk
Adds a new field to the public Correlate RPC request, which is backward-compatible in protobuf but requires downstream clients/servers to handle the new input for consistent behavior.

Overview
Entity correlation API now accepts a record name. CorrelateRequest gains an optional record_name field in entities.proto, enabling callers to pass a record/display name alongside emails/domains/company name for correlation.

Go, Python, and TypeScript generated bindings are regenerated to expose this new request field (including new getters/types and updated serialized descriptors).

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

@haasonsaas haasonsaas merged commit 6e33c87 into main Apr 15, 2026
9 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