raystack / frontier / 25898107018

42%

main: 42%

Build Type

Pull #1615

github

Committed by whoAbhishekSah

Commit Message

fix(group): also clean up org<->group hierarchy on delete

DeleteModel only clears Object-side relations on the group, which misses
the inverse hierarchy tuple created in OnGroupCreated:
  organization#member@group#member
(Object is the org, Subject is the group — pre-existing leak in the old
group.Service.Delete; surfaced during this refactor's audit.)

Adds membership.OnGroupDeleted as the symmetric counterpart to
OnGroupCreated: bundles RemoveAllGroupMembers + unlinkGroupFromOrg so
both hierarchy directions plus member policies/relations are cleaned up
in one call. deleter.DeleteGroup now invokes it before DeleteModel.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Pull Request Pull Request #1615: refactor(group): move group-delete cleanup into membership package

Coverage Stats

36 of 51 new or added lines in 5 files covered. (70.59%)

34 existing lines in 2 files now uncovered.

15988 of 37680 relevant lines covered (42.43%)

11.89 hits per line