feat(iam): implement deleteTeam method - #257
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 8797091. Configure here.
Greptile SummaryAdds the public
Confidence Score: 3/5The PR is not safe to merge until deleteTeam uses the management endpoint and propagates application-level deletion errors. The new operation currently targets the default IAM backend rather than the team management backend, and it can return success for response bodies that explicitly report an error. Files Needing Attention: packages/iam/src/lib/team/delete.ts Important Files Changed
Reviews (1): Last reviewed commit: "feat(iam): implement deleteTeam method" | Re-trigger Greptile |
8797091 to
af43f96
Compare

Note
Medium Risk
Destructive IAM team deletion is a sensitive operation; the change is small and follows existing team API patterns but callers can permanently remove teams if misused.
Overview
Adds
deleteTeamto@tigrisdata/iam, completing team lifecycle support alongside existing create/edit/list helpers.The new API issues a DELETE to
teams/{teamId}, mirrors other team methods for client setup andstatus: 'error'handling, and returns{ data: { teamId } }on success. Types and the function are re-exported from the package entrypoint. A minor changeset documents the release; Biome’s JSON schema URL is bumped to 2.5.8.Reviewed by Cursor Bugbot for commit af43f96. Bugbot is set up for automated code reviews on this repo. Configure here.