fix: issue while merging member organizations (CM-1109)

[themarolt]

Note

Medium Risk
Touches member-organization merge/delete logic used to reconcile work experiences; incorrect date normalization could change which rows are matched for removal or merging.

Overview
Fixes member-organization merging/removal when dateStart/dateEnd values are sometimes strings instead of Date objects.

Adds a small toIsoString helper and uses it when building SQL predicates and merged role payloads, avoiding invalid casts/toISOString() calls during merge operations.

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

[Copilot]

Pull request overview

This PR fixes a runtime issue during member-organization merge flows by normalizing dateStart/dateEnd values to ISO strings regardless of whether they come through as Date objects or strings, aligning with IMemberOrganization’s Date | string | null typing.

Changes:

• Added a local toIsoString(Date | string) helper to consistently convert date values to ISO strings.
• Updated removeMemberRole() to use toIsoString() for dateStart/dateEnd SQL replacements (avoids calling .toISOString() on string values).
• Updated mergeRoles() to use toIsoString() when constructing an updated “current role” payload.

---

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