Skip to content

Trace first-party OAuth usage - #1633

Merged
RhysSullivan merged 1 commit into
mainfrom
feat/oauth-first-party-observability
Aug 17, 2026
Merged

Trace first-party OAuth usage#1633
RhysSullivan merged 1 commit into
mainfrom
feat/oauth-first-party-observability

Conversation

@RhysSullivan

Copy link
Copy Markdown
Collaborator

What changed

  • add executor.oauth.client_first_party to OAuth start/request traces
  • add the same boolean to executor.oauth.complete
  • verify first-party flows emit true at both points

Why

OAuth completion traces already expose the client slug, but filtering for host-operated clients requires parsing the first-party: prefix. The boolean provides a stable, low-cardinality dimension for queries, dashboards, and alerts.

Validation

  • bun run --cwd packages/core/sdk test -- oauth-first-party.test.ts
  • bunx oxlint -c .oxlintrc.jsonc packages/core/sdk/src/oauth-service.ts packages/core/sdk/src/oauth-first-party.test.ts --deny-warnings
  • bun run typecheck

@github-actions

github-actions Bot commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Cloudflare preview

Torn down — the PR is closed.

@pkg-pr-new

pkg-pr-new Bot commented Aug 17, 2026

Copy link
Copy Markdown

Open in StackBlitz

@executor-js/cli

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

@executor-js/config

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

@executor-js/execution

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

@executor-js/sdk

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

@executor-js/codemode-core

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

@executor-js/runtime-quickjs

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

executor

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

@executor-js/plugin-file-secrets

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

@executor-js/plugin-graphql

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

@executor-js/plugin-keychain

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

@executor-js/plugin-mcp

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

@executor-js/plugin-onepassword

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

@executor-js/plugin-openapi

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

commit: 5bc1324

@RhysSullivan
RhysSullivan marked this pull request as ready for review August 17, 2026 22:27
@RhysSullivan
RhysSullivan merged commit deb39f7 into main Aug 17, 2026
21 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