Skip to content

fix(data-warehouse-source): name Stripe's restricted-key scopes and link the docs - #392

Open
kyleswank wants to merge 1 commit into
mainfrom
kyle/stripe-restricted-key-scopes
Open

kyleswank wants to merge 1 commit into
mainfrom
kyle/stripe-restricted-key-scopes

Conversation

@kyleswank

Copy link
Copy Markdown

What

The Stripe pre-flight gotcha in data-warehouse-source's skill was one clause buried in a generic bullet — "wants a restricted key, not sk_live_…" — with no scopes and no docs link. The agent's wizard_ask prompt to the user for the key ended up bare, e.g. "restricted API key beginning rk_live_ for this account, with no extra text or whitespace and the required read scopes" — no indication of which scopes, no link.

Change

Gives Stripe its own pre-flight bullet (matching the Supabase one's depth) that:

  • Lists the exact scopes: Read on Balance transactions/Charges/Customers/Disputes/Payment methods/Payouts/Products (Core), Read on Coupons/Credit notes/Invoices/Prices/Subscriptions (Billing), Read on Connect, Write on Webhooks
  • Links https://posthog.com/docs/cdp/sources/stripe#option-1-restricted-api-key-recommended for the exact dashboard steps
  • Explicitly tells the agent Stripe has no URL parameter to pre-select those scopes, so it doesn't invent a deep link claiming one exists (checked Stripe's own docs and how other integrations — Fivetran, Airbyte — document this; no such feature exists)

Testing

  • npm run build — skill builds cleanly, verified rendered SKILL.md contains the new bullet correctly stitched
  • npm test — 206/206 passing

🤖 Generated with Claude Code

https://claude.ai/code/session_01PcwfZr4Ag73MP2xwFFUF33

…ink the docs

The Stripe pre-flight gotcha only said "wants a restricted key, not
sk_live_..." with no scopes and no docs link, so the wizard's wizard_ask
prompt to the user was bare. Give Stripe its own bullet listing the exact
scopes (Core/Billing/Connect reads, Webhooks write) and linking the
restricted-key docs section. Also tell the agent explicitly that Stripe
has no URL parameter to pre-select those scopes, so it doesn't invent one.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PcwfZr4Ag73MP2xwFFUF33
@kyleswank
kyleswank requested a review from a team as a code owner September 11, 2026 18:48
@github-actions

Copy link
Copy Markdown

🧙 Wizard CI

Run the Wizard CI and test your changes against wizard-workbench example apps by replying with a GitHub comment using one of the following commands:

Test all apps:

  • /wizard-ci all

Test all apps in a directory:

  • /wizard-ci ai-observability
  • /wizard-ci basic-integration
  • /wizard-ci mcp-analytics
  • /wizard-ci replay-vision
  • /wizard-ci revenue
  • /wizard-ci self-driving
  • /wizard-ci warehouse
  • /wizard-ci warehouse-seeded

Test an individual app:

  • /wizard-ci ai-observability/anthropic
  • /wizard-ci ai-observability/google-adk
  • /wizard-ci ai-observability/groq
Show more apps
  • /wizard-ci ai-observability/manual-capture
  • /wizard-ci ai-observability/openai
  • /wizard-ci ai-observability/openai-agents
  • /wizard-ci ai-observability/opentelemetry
  • /wizard-ci ai-observability/vercel-ai
  • /wizard-ci basic-integration/android
  • /wizard-ci basic-integration/angular
  • /wizard-ci basic-integration/astro
  • /wizard-ci basic-integration/django
  • /wizard-ci basic-integration/fastapi
  • /wizard-ci basic-integration/flask
  • /wizard-ci basic-integration/flutter
  • /wizard-ci basic-integration/javascript-node
  • /wizard-ci basic-integration/javascript-web
  • /wizard-ci basic-integration/laravel
  • /wizard-ci basic-integration/next-js
  • /wizard-ci basic-integration/nuxt
  • /wizard-ci basic-integration/python
  • /wizard-ci basic-integration/rails
  • /wizard-ci basic-integration/react-native
  • /wizard-ci basic-integration/react-router
  • /wizard-ci basic-integration/sveltekit
  • /wizard-ci basic-integration/swift
  • /wizard-ci basic-integration/tanstack-router
  • /wizard-ci basic-integration/tanstack-start
  • /wizard-ci basic-integration/vue
  • /wizard-ci mcp-analytics/custom-dispatcher
  • /wizard-ci mcp-analytics/typescript-sdk
  • /wizard-ci replay-vision/javascript-node
  • /wizard-ci replay-vision/next-js
  • /wizard-ci replay-vision/react-vite
  • /wizard-ci revenue/stripe
  • /wizard-ci self-driving/astro
  • /wizard-ci self-driving/fastapi
  • /wizard-ci self-driving/nuxt
  • /wizard-ci self-driving/react-router
  • /wizard-ci self-driving/sveltekit
  • /wizard-ci warehouse/monorepo-env
  • /wizard-ci warehouse/multi-source-next
  • /wizard-ci warehouse/stripe-node
  • /wizard-ci warehouse/zero-source
  • /wizard-ci warehouse-seeded/next-stripe
  • /wizard-ci warehouse-seeded/next-stripe-declined

Test against a wizard branch:

  • /wizard-ci all wizard:my-branch

Add wizard:<branch> to any command above to pin the wizard branch. It defaults to main.

Results will be posted here when complete.

@kyleswank

Copy link
Copy Markdown
Author

/wizard-ci warehouse

@wizard-ci-bot

wizard-ci-bot Bot commented Sep 11, 2026

Copy link
Copy Markdown

🧙 Wizard CI Results

Trigger ID: e9117a9
Workflow: View run

App Confidence PR YARA
warehouse/monorepo-env FAIL 9/11 Failed (logs)
warehouse/multi-source-next FAIL 7/11 Failed (logs)
warehouse/stripe-node FAIL 9/11 Failed (logs)
warehouse/zero-source FAIL 10/11 Failed (logs)

Configuration

Setting Value
Wizard ref main
Context Mill ref kyle/stripe-restricted-key-scopes
PostHog ref master

Search for trigger ID e9117a9 in wizard-workbench PRs.

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