Skip to content

feat(connectors): export OAuth helpers as a narrow entrypoint - #336

Merged
drewstone merged 2 commits into
mainfrom
feat/connectors-oauth-subpath-20260928
Sep 28, 2026
Merged

drewstone merged 2 commits into
mainfrom
feat/connectors-oauth-subpath-20260928

Conversation

@drewstone

Copy link
Copy Markdown
Contributor

The broad connectors entrypoint loads the concrete catalog, including Node-only dependencies that break Cloudflare Worker startup for OAuth-only consumers. This adds a dedicated public OAuth subpath backed directly by the provider-agnostic OAuth helper and verifies it from the packed artifact.\n\nValidation: pnpm run release passed on Node 24.20, including build, tarball export-map checks, packed self-reference import, a real S256 authorization flow with persisted verifier, worker entry import, and an assertion that the OAuth bundle excludes the connector catalog, graceful-fs, Excel, and unzipper dependencies. Existing OAuth consumer tests passed 77/77. No version bump or publication is included.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

tangletools
tangletools previously approved these changes Sep 28, 2026

@tangletools tangletools left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

✅ Auto-approved PR — 51bc662d

Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-09-28T19:36:22Z

@tangletools tangletools left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

✅ Auto-approved PR — c85fc0ee

Blanket team auto-approval is intentional. The merge gates are CI and codex-p1.
No automated review runs on this PR. This approval rests on the rule above alone.

tangletools · auto-approval · reason: blanket_auto_approve · 2026-09-28T19:59:21Z

@drewstone
drewstone merged commit dfd0d3c into main Sep 28, 2026
1 check 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.

2 participants