[dev] [claudfuen] feat/pentest-subscription-billing by github-actions[bot] · Pull Request #2220 · trycompai/comp

github-actions · 2026-03-04T19:38:50Z

Summary

Add Stripe subscription billing for penetration testing (subscribe, overage charges, usage tracking)
Gate the penetration tests page behind an active subscription (ModuleGate paywall)
Add billing page UI with subscription management, usage progress bar, and payment method section
Add Stripe webhook handler for subscription lifecycle events (renewals, cancellations, payment failures)
Preauthorize billing before creating a run (prevents orphaned runs on billing failure)
Overage confirmation dialog when all included runs are exhausted

Environment Variables

Three new env vars are required for pentest billing to function. All are optional — the app won't crash without them, but billing features will return descriptive errors.

Variable	Purpose	Example
`STRIPE_PENTEST_SUBSCRIPTION_PRICE_ID`	Stripe Price ID for the monthly pentest subscription plan	`price_1T6vZa...`
`STRIPE_PENTEST_OVERAGE_PRICE_ID`	Stripe Price ID for per-run overage charges	`price_1T6va1...`
`STRIPE_PENTEST_WEBHOOK_SECRET`	Webhook signing secret for the `/api/webhooks/stripe-pentest` endpoint	`whsec_...`

Stripe setup

Create a recurring price (e.g. $99/month) → set as STRIPE_PENTEST_SUBSCRIPTION_PRICE_ID
Create a one-time price for overage runs (e.g. $49) → set as STRIPE_PENTEST_OVERAGE_PRICE_ID
Create a webhook endpoint pointing to <app-url>/api/webhooks/stripe-pentest listening for:
- customer.subscription.updated
- customer.subscription.deleted
- invoice.payment_failed
Copy the webhook signing secret → set as STRIPE_PENTEST_WEBHOOK_SECRET

Test plan

🤖 Generated with Claude Code

Icons (AppShellWrapper — rail): - Compliance: CertificateCheck → Badge (certification badge) - Trust: CloudAuditing → Globe (public/global trust center) - Security: Security → ManageProtection (protection management) Icons (AppSidebar — compliance sidebar): - Auditor View: TaskComplete → DocumentSigned (auditors sign off) - Controls: Security → SettingsAdjust (adjustable params, no dupe) - Documents: Catalog → FolderDetails (folder of docs) - People: Group → UserMultiple (standard multi-user) - Risks: Warning → Scale (weighing risk) - Vendors: ShoppingBag → Partnership (business partners) - Questionnaire: Document → DocumentTasks (task-filled form) - Integrations: Integration → Connect (connecting systems) - Cloud Tests: Chemistry → TestTool (test tooling) Overview: - Replace <DraggableCards> with a plain 2-column grid - Delete DraggableCards.tsx, SortableCard.tsx, useCardOrder.ts Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

The id returned by the create API is the provider run ID stored in providerRunId, not the internal ptr... primary key. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

…ntest-subscription-billing

Show a locked marketing state when the org has no active pentest subscription. Includes a reusable ModuleGate component in @comp/ui with dark chrome-frame preview, and an animated pen-test demo that cycles through agents and reveals findings with severity badges. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

vercel · 2026-03-04T19:38:56Z

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

Project	Deployment	Actions	Updated (UTC)
app	Ready	Preview, Comment	Apr 3, 2026 9:18pm
comp-framework-editor	Ready	Preview, Comment	Apr 3, 2026 9:18pm
portal	Ready	Preview, Comment	Apr 3, 2026 9:18pm

cursor · 2026-03-04T19:38:57Z

PR Summary

High Risk
Touches Stripe subscription and off-session charging logic and tightens authorization to admin/owner only, so mistakes could block access or incorrectly charge customers. Also changes subscription run limits and billing/checkout UX, which impacts production revenue flows.

Overview
Penetration testing is now subscription-gated with clearer billing/usage UX. The pentest page fetches PentestSubscription + run usage and either shows a locked upsell (ModuleGate + new PentestPreviewAnimation) or the run table with a usage indicator; creating a run now warns/requires confirmation when it will incur an overage charge.

Billing flow is hardened and moved earlier in the run lifecycle. Server actions now require org admin/owner (not just membership), block duplicate active subscriptions, add preauthorizePentestRun() to validate subscription + (if over limit) charge Stripe before creating the run using a client-stable nonce/idempotency key and improved payment-method fallback, and add getPentestPricing() for dynamic price display.

Billing page and Stripe webhook coverage are expanded. The billing page is reworked into a subscriptions table (active/cancelled/past_due states, usage/renewal, manage/fix payment CTAs) with an expanded pentest usage section; Stripe pentest webhook now has comprehensive tests for checkout completion and subscription update/delete events.

Data/API contract tweaks. PentestSubscription.includedRunsPerPeriod default changes from 3 to 1, period counting uses < currentPeriodEnd, and openapi.json is updated (new internal bulk automation failure notification endpoint, evidence submission delete, GitHub repos listing, and minor pentest webhook/payload schema adjustments).

^{Written by Cursor Bugbot for commit 1b06c7a. This will update automatically on new commits. Configure here.}

...pp/src/app/(app)/[orgId]/security/penetration-tests/components/pentest-preview-animation.tsx

- Add preauthorize billing check before run creation (prevents unbilled runs) - Add dynamic pricing from Stripe (subscription + overage prices) - Add usage tracking in pentest page header and create dialog - Add overage confirmation dialog when included runs are exhausted - Add billing page with subscription table, usage details, and payment method - Add role-based auth (admin/owner only) for billing server actions - Add duplicate subscription guard in subscribeToPentestPlan - Add webhook unit tests (10 cases covering all event types) - Fix period boundary queries (lte → lt) to prevent double-counting - Fix payment method lookup to check subscription PM before customer PM - Fix PaymentIntent config (off_session instead of conflicting options) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

apps/app/src/app/(app)/[orgId]/security/penetration-tests/hooks/use-penetration-tests.ts

apps/app/src/app/(app)/[orgId]/security/penetration-tests/actions/billing.ts

- Fix animation interval spawning duplicate reset timers (added pausing flag) - Use stable nonce ref to prevent duplicate overage charges on retry - Remove dead getPentestUsage function and PentestUsage interface Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

cursor

Cursor Bugbot has reviewed your changes and found 2 potential issues.

^{Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

cursor · 2026-03-04T21:37:36Z

apps/app/src/app/(app)/[orgId]/security/penetration-tests/actions/billing.ts

+  });
+  if (existingSub?.status === 'active') {
+    throw new Error('Organization already has an active pentest subscription.');
+  }


Duplicate Stripe subscription possible for past-due status

High Severity

The duplicate-subscription guard in subscribeToPentestPlan only blocks active status, but past_due subscriptions are not blocked. The pentest page checks hasActiveSubscription (only true for active), so a past_due subscriber sees the "Get started" button which calls subscribeToPentestPlan. This creates a brand-new Stripe subscription while the old past_due one still exists. The DB record is overwritten via upsert, so the old subscription becomes untracked — the customer gets billed twice by Stripe with no way to cancel the orphaned subscription through the app.

Additional Locations (1)

apps/app/src/app/(app)/[orgId]/security/penetration-tests/penetration-tests-page-client.tsx#L187-L188

cursor · 2026-03-04T21:37:36Z

apps/app/src/app/(app)/[orgId]/security/penetration-tests/hooks/use-penetration-tests.ts

+        const authResult = await preauthorizePentestRun(organizationId, nonceRef.current);
+        if (!authResult.authorized) {
+          throw new Error(authResult.error ?? 'Billing authorization failed.');
+        }


Pre-creation billing check introduces overage race condition

Medium Severity

Moving billing authorization before run creation introduces a TOCTOU race. Two concurrent requests can both call preauthorizePentestRun, both see the same runsThisPeriod count below the included limit, and both get authorized: true without overage. Both then create runs, exceeding the included limit without any overage charge. The previous post-creation check didn't have this issue because the run already existed in the DB when the count was performed.

Additional Locations (1)

apps/app/src/app/(app)/[orgId]/security/penetration-tests/actions/billing.ts#L261-L273

Resolve conflicts by adopting main's API-based billing approach (pentest-billing controller/service) over branch's server actions. Removed billing.ts server action and updated page/test files accordingly. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Restructure to match current product messaging ("the agentic compliance platform"), remove duplicate sections, add table of contents, monorepo layout, and streamlined quick start guide. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

…, update theme colors - Remove react-wrap-balancer dependency, use Tailwind text-balance utility - Add dotenv import to packages/db prisma.config.ts so migrations work - Update primary color and add proper dark mode theme Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

claudfuen and others added 5 commits March 3, 2026 21:44

revert(ui): restore original compliance sidebar icons

9d9c947

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

fix(billing): look up run by providerRunId not internal id

3d282ab

The id returned by the create API is the provider run ID stored in providerRunId, not the internal ptr... primary key. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Merge branch 'main' of https://github.com/trycompai/comp into feat/pe…

16b6745

…ntest-subscription-billing

github-actions bot added the automated-pr label Mar 4, 2026

vercel bot deployed to Preview – app March 4, 2026 19:40 View deployment

vercel bot deployed to Preview – portal March 4, 2026 19:41 View deployment

cursor bot reviewed Mar 4, 2026

View reviewed changes

...pp/src/app/(app)/[orgId]/security/penetration-tests/components/pentest-preview-animation.tsx Show resolved Hide resolved

vercel bot deployed to Preview – app March 4, 2026 21:26 View deployment

vercel bot deployed to Preview – portal March 4, 2026 21:26 View deployment

cursor bot reviewed Mar 4, 2026

View reviewed changes

apps/app/src/app/(app)/[orgId]/security/penetration-tests/hooks/use-penetration-tests.ts Outdated Show resolved Hide resolved

apps/app/src/app/(app)/[orgId]/security/penetration-tests/actions/billing.ts Outdated Show resolved Hide resolved

vercel bot temporarily deployed to Preview – portal March 4, 2026 21:32 Inactive

vercel bot deployed to Preview – app March 4, 2026 21:34 View deployment

cursor bot reviewed Mar 4, 2026

View reviewed changes

Marfuen marked this pull request as draft March 17, 2026 19:02

vercel bot deployed to Preview – comp-framework-editor April 3, 2026 19:02 View deployment

vercel bot deployed to Preview – app April 3, 2026 19:03 View deployment

vercel bot deployed to Preview – portal April 3, 2026 19:03 View deployment

vercel bot deployed to Preview – comp-framework-editor April 3, 2026 19:33 View deployment

vercel bot deployed to Preview – app April 3, 2026 19:34 View deployment

vercel bot deployed to Preview – portal April 3, 2026 19:34 View deployment

vercel bot deployed to Preview – comp-framework-editor April 3, 2026 21:17 View deployment

vercel bot deployed to Preview – app April 3, 2026 21:18 View deployment

vercel bot deployed to Preview – portal April 3, 2026 21:18 View deployment

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[dev] [claudfuen] feat/pentest-subscription-billing#2220

[dev] [claudfuen] feat/pentest-subscription-billing#2220
github-actions[bot] wants to merge 10 commits intomainfrom
feat/pentest-subscription-billing

github-actions bot commented Mar 4, 2026 •

edited by claudfuen

Loading

Uh oh!

vercel bot commented Mar 4, 2026 •

edited

Loading

Uh oh!

cursor bot commented Mar 4, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor bot left a comment

Uh oh!

cursor bot Mar 4, 2026

Uh oh!

cursor bot Mar 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

github-actions bot commented Mar 4, 2026 • edited by claudfuen Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Environment Variables

Stripe setup

Test plan

Uh oh!

vercel bot commented Mar 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

cursor bot commented Mar 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PR Summary

Uh oh!

Uh oh!

Uh oh!

Uh oh!

cursor bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor bot Mar 4, 2026

Choose a reason for hiding this comment

Duplicate Stripe subscription possible for past-due status

Uh oh!

cursor bot Mar 4, 2026

Choose a reason for hiding this comment

Pre-creation billing check introduces overage race condition

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

github-actions bot commented Mar 4, 2026 •

edited by claudfuen

Loading

vercel bot commented Mar 4, 2026 •

edited

Loading

cursor bot commented Mar 4, 2026 •

edited

Loading