diff --git a/changelog/+query-groups-typed-members-accessor.housekeeping.md b/changelog/+query-groups-typed-members-accessor.housekeeping.md new file mode 100644 index 00000000..0e6eeeca --- /dev/null +++ b/changelog/+query-groups-typed-members-accessor.housekeeping.md @@ -0,0 +1 @@ +Routed the two remaining `existing_group.members.peer_ids` accesses in `infrahub_sdk/query_groups.py` through the typed `_get_relationship_many(name="members")` accessor, removing the last two `# type: ignore[union-attr]` suppressions in that module. Internal typing hygiene with no behavioural change. diff --git a/infrahub_sdk/query_groups.py b/infrahub_sdk/query_groups.py index cc7faebd..3fcde346 100644 --- a/infrahub_sdk/query_groups.py +++ b/infrahub_sdk/query_groups.py @@ -175,7 +175,9 @@ async def update_group(self) -> None: return # Calculate how many nodes should be deleted - self.unused_member_ids = list(set(existing_group.members.peer_ids) - set(members)) # type: ignore[union-attr] + self.unused_member_ids = list( + set(existing_group._get_relationship_many(name="members").peer_ids) - set(members) + ) if not self.delete_unused_nodes: return @@ -267,7 +269,9 @@ def update_group(self) -> None: return # Calculate how many nodes should be deleted - self.unused_member_ids = list(set(existing_group.members.peer_ids) - set(members)) # type: ignore[union-attr] + self.unused_member_ids = list( + set(existing_group._get_relationship_many(name="members").peer_ids) - set(members) + ) if not self.delete_unused_nodes: return