Skip to content

feat: add API key notification rotatable field from api-specifications#2389 - #230

Merged
jedpattersonpaddle merged 1 commit into
mainfrom
cursor/dx-1011-sdk-spec-sync-6a86
Aug 6, 2026
Merged

feat: add API key notification rotatable field from api-specifications#2389#230
jedpattersonpaddle merged 1 commit into
mainfrom
cursor/dx-1011-sdk-spec-sync-6a86

Conversation

@cursor

@cursor cursor Bot commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

Adds the nullable rotatable field to the shared API-key webhook payload and runtime entity for all five released API-key notification events.

Source: api-specifications#2389
Linear: DX-1011

Spec-to-SDK mapping

Spec change SDK files changed
Add ApiKeyPayload.rotatable: boolean | null to the public webhook response src/notifications/types/api-key/api-key-notification-response.ts
Expose camel-case rotatable on the runtime notification entity src/notifications/entities/api-key/api-key-notification.ts
Cover created, updated, and expiring events with rotatable: true src/__tests__/mocks/notifications/api-key-created.mock.ts, api-key-updated.mock.ts, api-key-expiring.mock.ts
Cover expired and revoked events with rotatable: false src/__tests__/mocks/notifications/api-key-expired.mock.ts, api-key-revoked.mock.ts
Document and version the additive public API CHANGELOG.md, package.json (3.9.0)

Intentionally excluded

  • Internal apps/paddle-events/** contracts are excluded because they are marked @internalUseOnly.
  • Example notification payload changes are represented through SDK mocks, not as a separate public surface.
  • api_key.exposure_created uses a separate payload and is unaffected.
  • The generic /events resource has untyped data, so no resource model changes are needed.
  • The pre-existing exposed_at drift is outside api-specifications#2389 and remains unchanged.

Validation

  • pnpm install --frozen-lockfile — passed
  • pnpm build — passed
  • pnpm typecheck:tests — passed
  • pnpm test --runInBand — 34 suites, 275 tests passed
  • pnpm test --runInBand src/__tests__/notifications/notifications-parser.test.ts — 1 suite, 57 tests passed, including all five API-key events
  • pnpm prettier — passed
  • pnpm lint — passed after fixing the documented, git-ignored generated src/version.ts final newline locally

Created by an automated spec-sync agent.

Open in Web View Automation 

Co-authored-by: Jed Patterson <jed.patterson@paddle.com>
@cursor
cursor Bot marked this pull request as ready for review July 17, 2026 18:47
@cursor
cursor Bot requested a review from a team as a code owner July 17, 2026 18:47
@jedpattersonpaddle
jedpattersonpaddle merged commit 38b2533 into main Aug 6, 2026
8 of 10 checks passed
@jedpattersonpaddle
jedpattersonpaddle deleted the cursor/dx-1011-sdk-spec-sync-6a86 branch August 6, 2026 09:14
@jedpattersonpaddle jedpattersonpaddle mentioned this pull request Aug 7, 2026
3 tasks
jedpattersonpaddle added a commit that referenced this pull request Aug 7, 2026
3.9.0 is already published on npm from #230; ship the remaining
changelog entry as 3.10.0 and record 3.9.0 retrospectively.
jedpattersonpaddle added a commit that referenced this pull request Aug 7, 2026
* Release v3.9.0

* Bump release to v3.10.0

3.9.0 is already published on npm from #230; ship the remaining
changelog entry as 3.10.0 and record 3.9.0 retrospectively.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants