Skip to content

Conversation

@stanleyphu
Copy link
Contributor

@stanleyphu stanleyphu requested a review from a team as a code owner January 6, 2026 22:41
@stanleyphu stanleyphu requested a review from hadihallak January 6, 2026 22:41
@greptile-apps
Copy link
Contributor

greptile-apps bot commented Jan 6, 2026

Greptile Summary

Version bump release (7.79.3 → 7.80.0) that includes two feature additions merged in previous commits:

API Key Management (#1434)

  • Added organizations.listOrganizationApiKeys() to list API keys for an organization with pagination support
  • Added organizations.createOrganizationApiKey() to create new organization API keys with idempotency support
  • Added apiKeys.deleteApiKey() to delete API keys by ID
  • Properly separates ApiKey interface (without value field) from CreatedApiKey interface (with value field) for security - the full API key value is only exposed during creation
  • All new interfaces, serializers, and type definitions follow existing codebase patterns

Invitation Resent Event (#1433)

  • Added InvitationResentEvent and InvitationResentEventResponse interfaces for the invitation.resent webhook event
  • Updated Event and EventResponse union types to include the new event interfaces
  • Added deserialization support in the deserializeEvent() switch statement

All changes follow TypeScript best practices, include comprehensive test coverage, and properly handle sensitive data (no logging of API key values).

Confidence Score: 5/5

  • This PR is safe to merge with no concerns
  • This is a straightforward version bump release that includes well-tested features. The changes follow established patterns in the codebase, include comprehensive test coverage, properly handle sensitive data (API key values), use correct TypeScript typing, and maintain backwards compatibility. No custom security rules were violated.
  • No files require special attention

Important Files Changed

Filename Overview
package.json Version bumped from 7.79.3 to 7.80.0
src/workos.ts VERSION constant updated to 7.80.0 for User-Agent header
src/sso/snapshots/sso.spec.ts.snap Test snapshots updated to reflect new version in User-Agent header

Sequence Diagram

sequenceDiagram
    participant Client as Client Application
    participant SDK as WorkOS SDK
    participant API as WorkOS API

    Note over Client,API: API Key Management Flow

    rect rgb(240, 248, 255)
        Note right of Client: List Organization API Keys
        Client->>SDK: organizations.listOrganizationApiKeys({organizationId, ...pagination})
        SDK->>API: GET /organizations/:id/api_keys
        API-->>SDK: List of ApiKey objects (without value field)
        SDK-->>Client: AutoPaginatable<ApiKey>
    end

    rect rgb(255, 250, 240)
        Note right of Client: Create Organization API Key
        Client->>SDK: organizations.createOrganizationApiKey({organizationId, name, permissions})
        SDK->>API: POST /organizations/:id/api_keys
        API-->>SDK: CreatedApiKey object (includes value field)
        SDK-->>Client: CreatedApiKey with full API key value
        Note over Client: Full API key value only available on creation
    end

    rect rgb(255, 240, 245)
        Note right of Client: Delete API Key
        Client->>SDK: apiKeys.deleteApiKey(id)
        SDK->>API: DELETE /api_keys/:id
        API-->>SDK: 204 No Content
        SDK-->>Client: void
    end

    rect rgb(245, 255, 245)
        Note right of Client: Webhook Event Processing
        API->>Client: Webhook: invitation.resent event
        Client->>SDK: deserializeEvent(eventResponse)
        SDK-->>Client: InvitationResentEvent with typed data
    end
Loading

Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@stanleyphu stanleyphu merged commit 18f60f5 into main Jan 7, 2026
7 checks passed
@stanleyphu stanleyphu deleted the v7.80.0 branch January 7, 2026 19:01
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.

4 participants