fix: handle empty organization data correctly (CM-1113)

[themarolt]

Note

Medium Risk
Changes organization upsert behavior to return undefined (skip) instead of throwing when verified identities are lost during domain normalization, which can affect whether organizations are created/updated in ingestion flows.

Overview
Improves findOrCreateOrganization to better handle empty/invalid identity data by defaulting data.identities to an empty array and re-computing verifiedIdentities after domain normalization/filtering.

If normalization removes all verified identities and there is no displayName, the function now logs and returns undefined (skips processing) instead of erroring, preventing failures on malformed organization inputs.

^{Reviewed by Cursor Bugbot for commit ae719a8. Bugbot is set up for automated code reviews on this repo. Configure here.}

[Copilot]

Pull request overview

This PR adjusts organization upsert behavior to correctly handle cases where domain normalization invalidates previously verified organization identities, avoiding incorrect lookups/creation attempts with now-empty identity data.

Changes:

• Make verifiedIdentities re-assignable and re-derive it after domain normalization + identity cleanup.
• Skip further processing (return undefined) when normalization removes all verified identities and there is no displayName.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.