Skip to content

feat: add team member removal and a project filter for the member list - #69

Merged
bjarn merged 1 commit into
feat/team-rolesfrom
feat/team-member-removal
Oct 21, 2026
Merged

bjarn merged 1 commit into
feat/team-rolesfrom
feat/team-member-removal

Conversation

@bjarn

@bjarn bjarn commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

What changed

Non-breaking. Stacked on the team role management PR.

  • Adds team().members().remove(userId), which removes a member from the team. The Owner can't be removed; that request throws a ConflictException.
  • ListTeamMembersQuery.filterProjectId(...) is sent as filter[project_id] by team().members().list and iterate. It returns only members who can access that project: members with access to all projects plus members selected for it. An unknown project returns an empty list.
  • Removes the pending list from the operation coverage test, so it is strict again: every generated operation must have a method.

Usage

for (TeamMemberData member : lettermint.team().members().iterate(ListTeamMembersQuery.builder().filterProjectId(projectId).build())) {
    System.out.println(member.email());
}

lettermint.team().members().remove(userId);

Checks

  • ./gradlew checkGeneratedTypes with the latest spec: up to date (217 files)
  • ./gradlew clean build --no-daemon on JDK 17: passed (103 tests, 0 failures)
  • ./gradlew clean build --no-daemon on JDK 24: passed (103 tests, 0 failures)
  • CI build-output checks on the jar: Automatic-Module-Name: co.lettermint present, no okhttp bundled

Not run locally

  • JDK 21 and 25 and Windows (CI matrix).
  • ./gradlew checkstyleMain checkstyleTest: the build has no checkstyle tasks, and the workflow ignores the result (|| true).

🤖 Generated with Claude Code

Adds `team().members().remove(userId)`. `ListTeamMembersQuery` now has
`filterProjectId`, sent as `filter[project_id]`, which limits the member
list to members who can access that project. The operation coverage test
is strict again.

Co-Authored-By: Claude <noreply@anthropic.com>
@bjarn
bjarn requested a review from a team as a code owner October 20, 2026 20:46
@bjarn
bjarn added this pull request to stack #70 October 20, 2026 20:46
@bjarn
bjarn merged commit 1c773fe into main Oct 21, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants