Skip to content

Support adding/removing workspace secret keys - #266

Merged
schaloner merged 2 commits into
masterfrom
steve/seats-1178-update-java-sdk
Aug 21, 2026
Merged

Support adding/removing workspace secret keys#266
schaloner merged 2 commits into
masterfrom
steve/seats-1178-update-java-sdk

Conversation

@schaloner

Copy link
Copy Markdown
Contributor

No description provided.

@schaloner
schaloner requested a review from a team as a code owner August 21, 2026 13:28
@schaloner
schaloner requested review from bverbeken and mroloux and a lite review from Copilot and removed request for a team August 21, 2026 13:28

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds workspace secret-key management and rotation support to the Java SDK.

Changes:

  • Adds APIs to add and remove workspace secret keys.
  • Exposes workspace secret keys through Workspace.
  • Adds integration tests for key management and regeneration.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 1 comment.

Show a summary per file
File Summary
src/test/java/seatsio/workspaces/RemoveSecretKeyTest.java Tests removing a workspace secret key.
src/test/java/seatsio/workspaces/RegenerateWorkspaceSecretKeyTest.java Tests secret-key regeneration constraints.
src/test/java/seatsio/workspaces/AddSecretKeyTest.java Tests adding a workspace secret key.
src/main/java/seatsio/workspaces/Workspaces.java Implements secret-key management operations.
src/main/java/seatsio/workspaces/Workspace.java Adds secretKeys; requires preserving the existing seven-argument constructor for compatibility.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread src/main/java/seatsio/workspaces/Workspace.java
Comment thread src/test/java/seatsio/workspaces/RegenerateWorkspaceSecretKeyTest.java Outdated
Comment thread src/main/java/seatsio/workspaces/Workspaces.java
Copilot AI review requested due to automatic review settings August 21, 2026 14:21

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

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

@schaloner
schaloner merged commit 510be99 into master Aug 21, 2026
3 checks passed
@schaloner
schaloner deleted the steve/seats-1178-update-java-sdk branch August 21, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants