-
Notifications
You must be signed in to change notification settings - Fork 20
Add Stripe Projects CLI provisioning to scaffolding and SDK skills #41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -10,7 +10,10 @@ Scaffolds a headless agent in TypeScript targeting OpenRouter. The generated pro | |
| ## Prerequisites | ||
|
|
||
| - Bun 1.1+ | ||
| - `OPENROUTER_API_KEY` from [openrouter.ai/settings/keys](https://openrouter.ai/settings/keys) | ||
| - `OPENROUTER_API_KEY` — get one from the [dashboard](https://openrouter.ai/settings/keys), or provision via [Stripe Projects](https://openrouter.ai/announcements/openrouter-on-stripe-projects) CLI: | ||
| ```bash | ||
| stripe projects add openrouter/api # creates account + key, writes to .env | ||
| ``` | ||
| - For full SDK reference, see the `openrouter-typescript-sdk` skill | ||
|
|
||
| --- | ||
|
|
@@ -114,6 +117,7 @@ After getting the name and checklist selections, follow this workflow: | |
| - [ ] If HTTP server selected: generate src/server.ts (spec in references/entry-points.md) | ||
| - [ ] If MCP server selected: generate src/mcp-server.ts (spec in references/entry-points.md) | ||
| - [ ] Generate .env.example | ||
| - [ ] If Stripe CLI is available (`command -v stripe`): offer to run `stripe projects add openrouter/api` to auto-provision the key into .env | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @devin [suggestion] DetailsWhy: Same issue as in Fix: Extend the check: Reviewed at |
||
| - [ ] Generate test/agent.test.ts | ||
| - [ ] Run `bun install` to fetch dependencies | ||
| - [ ] Verify: run `bunx tsc --noEmit` | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@devin [suggestion]
command -v stripecheck omits Projects plugin —stripe projects addwill fail if the plugin isn't installedDetails
Why: The Stripe Projects CLI exists as a separate plugin that must be installed with
stripe plugin install projectsafter the Stripe CLI itself.command -v stripeverifies only the CLI binary is in PATH — if the plugin isn't installed,stripe projects add openrouter/apierrors withstripe: 'projects' is not a stripe command. Theopenrouter-typescript-sdkskill already notes this requirement (line 63); this step is inconsistent with it.Fix: Extend the availability check to also verify the plugin:
Ref: OpenRouter — Stripe Projects prerequisites — "Install the Projects plugin:
stripe plugin install projects"Reviewed at
4023997