fix: deleting member fails when removing segments (CM-1312)

[skwowet]

Summary

DELETE /member returns 500 when the request includes a segment that resolves to no active subprojects — e.g. an archived project group. Segment expansion returns an empty list, producing invalid SQL (segmentId in ()).

Changes

• Return early in excludeMembersFromSegments when there are no subproject IDs to delete
• Apply the same guard in excludeOrganizationsFromSegments

[Copilot]

Pull request overview

This PR fixes a 500 error on DELETE /member (and the analogous organization path) that occurred when a request included a segment resolving to no active subprojects (e.g., an archived project group). In that case getSegmentSubprojectIds returns an empty array, and building the delete query with segmentId in (:segmentIds) produced invalid SQL (segmentId in ()). The fix adds an early-return guard so the delete is skipped when there are no subproject IDs.

Changes:

• Add an early return in excludeMembersFromSegments when subprojectIds is empty.
• Apply the same guard in excludeOrganizationsFromSegments.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
backend/src/database/repositories/memberRepository.ts	Guards excludeMembersFromSegments against empty subproject IDs to avoid invalid SQL.
backend/src/database/repositories/organizationRepository.ts	Applies the same empty-guard to excludeOrganizationsFromSegments.

---

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