Skip to content

Update Notion example and streamed response ordering#4

Draft
shreyash-chonkie wants to merge 2 commits into
mainfrom
codex/update-notion-alpha15
Draft

Update Notion example and streamed response ordering#4
shreyash-chonkie wants to merge 2 commits into
mainfrom
codex/update-notion-alpha15

Conversation

@shreyash-chonkie

Copy link
Copy Markdown
Member

What changed

  • switch the Notion MCP example from AuthKit JWT validation to WorkOS-managed API-key validation
  • scope the upstream Notion token in WorkOS Vault using the validated API-key owner
  • update the Notion example and lockfile to Sidecar 0.1.0-alpha.15
  • preserve assistant text segments in their streamed order around widget tool calls in the Sidecar dev harness

Version audit

The Simple, Notion, and Stock Prices examples all resolve sidecar-ai, @sidecar-ai/openai, and @sidecar-ai/anthropic at 0.1.0-alpha.15; no alpha.14 pins remain in their manifests or lockfiles.

Validation

  • root typecheck, 128 tests, and build
  • Notion: public alpha.15 dependency resolution, sidecar check, standalone build test, and Vercel build
  • Stock Prices: public alpha.15 dependency resolution, typecheck, sidecar check, 11 tests, ChatGPT build, and Claude build

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sidecar-notion Ready Ready Preview, Comment Jul 13, 2026 12:44am

Request Review

@critic-by-feyn
critic-by-feyn Bot marked this pull request as ready for review July 15, 2026 06:05
@critic-by-feyn
critic-by-feyn Bot marked this pull request as draft July 15, 2026 06:05
@shreyash-chonkie

Copy link
Copy Markdown
Member Author

Hello vercel, I am Shreyash

@shreyash-chonkie

Copy link
Copy Markdown
Member Author

What does this change do?

@shreyash-chonkie

Copy link
Copy Markdown
Member Author
Hwllo world or something I guess.

@critic-by-feyn
critic-by-feyn Bot marked this pull request as ready for review July 15, 2026 06:54
@critic-by-feyn
critic-by-feyn Bot marked this pull request as draft July 15, 2026 06:54
@shreyash-chonkie
shreyash-chonkie marked this pull request as ready for review July 15, 2026 16:43
@shreyash-chonkie
shreyash-chonkie marked this pull request as draft July 15, 2026 16:43
Comment thread examples/notion/README.md
Comment on lines +105 to +111

When Vault has no stored Notion token for the configured owner id, ask the model
to run `authorize` first. That tool returns a Notion OAuth link. If the user
calls another Notion tool before linking, the tool result also includes the same
authorization link. The flow uses Notion's MCP OAuth discovery, dynamic client
registration, PKCE, and the local callback route at `/notion/oauth/callback`.
The callback stores the Notion access token and refresh token in WorkOS Vault.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

dwerwer

Comment thread examples/notion/README.md
Comment on lines +77 to +79
Hosted Notion MCP requires a Notion-audience OAuth token. This example uses a
WorkOS-managed API key as the MCP bearer credential. The server validates the
incoming bearer token with WorkOS API Keys and stores the separate upstream

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

uhuh

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