Skip to content

[codex] fix WorkOS list boundaries#809

Merged
RhysSullivan merged 1 commit into
mainfrom
codex/fix-workos-list-boundaries
May 13, 2026
Merged

[codex] fix WorkOS list boundaries#809
RhysSullivan merged 1 commit into
mainfrom
codex/fix-workos-list-boundaries

Conversation

@RhysSullivan
Copy link
Copy Markdown
Owner

Summary

  • collect all WorkOS pages before returning app-level list results
  • use a targeted membership lookup for admin authorization
  • paginate the raw user API-key list endpoint and add regression coverage

Root Cause

Several WorkOS list responses were consumed directly from the first page. That can make authorization, seat usage, and list views incomplete once a response spans multiple pages.

Validation

  • bun run test:node -- src/auth/workos.node.test.ts src/auth/api-keys.node.test.ts
  • node ../../node_modules/vitest/vitest.mjs run src/org/handlers.test.ts
  • bun run typecheck
  • targeted oxlint
  • targeted oxfmt --check

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
executor-marketing cbb18bd Commit Preview URL

Branch Preview URL
May 13 2026, 09:25 PM

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
executor-cloud cbb18bd May 13 2026, 09:26 PM

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented May 13, 2026

Open in StackBlitz

@executor-js/cli

npm i https://pkg.pr.new/@executor-js/cli@809

@executor-js/config

npm i https://pkg.pr.new/@executor-js/config@809

@executor-js/execution

npm i https://pkg.pr.new/@executor-js/execution@809

@executor-js/sdk

npm i https://pkg.pr.new/@executor-js/sdk@809

@executor-js/storage-core

npm i https://pkg.pr.new/@executor-js/storage-core@809

@executor-js/codemode-core

npm i https://pkg.pr.new/@executor-js/codemode-core@809

@executor-js/runtime-quickjs

npm i https://pkg.pr.new/@executor-js/runtime-quickjs@809

@executor-js/plugin-file-secrets

npm i https://pkg.pr.new/@executor-js/plugin-file-secrets@809

@executor-js/plugin-google-discovery

npm i https://pkg.pr.new/@executor-js/plugin-google-discovery@809

@executor-js/plugin-graphql

npm i https://pkg.pr.new/@executor-js/plugin-graphql@809

@executor-js/plugin-keychain

npm i https://pkg.pr.new/@executor-js/plugin-keychain@809

@executor-js/plugin-mcp

npm i https://pkg.pr.new/@executor-js/plugin-mcp@809

@executor-js/plugin-onepassword

npm i https://pkg.pr.new/@executor-js/plugin-onepassword@809

@executor-js/plugin-openapi

npm i https://pkg.pr.new/@executor-js/plugin-openapi@809

executor

npm i https://pkg.pr.new/executor@809

commit: cbb18bd

@RhysSullivan RhysSullivan marked this pull request as ready for review May 13, 2026 23:45
@RhysSullivan RhysSullivan merged commit 67b05a4 into main May 13, 2026
11 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.

1 participant