Repository navigation
feat(integrate-revenuecat): settle how the paywall is built before coding it - #62
Merged
Merged
Conversation
…ding it Given "hook up RevenueCat", agents split: one hand-coded a paywall wired to offerings, another asked whether to use the paywall builder. The skills listed both paths as equal follow-ups with no recommendation. integrate-revenuecat now has an explicit paywall choice step: RevenueCat Paywall is the default for apps with no paywall screen, an existing screen is kept and wired to offerings, agents ask when they can and otherwise take the default and say so. purchase-flow and paywall point back at that choice. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…at Paywall Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
trinya
marked this pull request as ready for review
September 25, 2026 14:31
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, have a team admin enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit aa6f5be. Configure here.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This was referenced Sep 25, 2026
…l-design Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
jefago
approved these changes
Sep 25, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Why
Codex and Claude Code, both with the toolkit installed, got the same prompt ("Now hook up RevenueCat") and did different things:
The skills never made the call.
integrate-revenuecatended by listingrevenuecat-paywallandrevenuecat-purchase-flowas equal follow-ups.revenuecat-purchase-flow's description matched "build paywall purchase logic", which is close to what "hook up" means. Nothing recommended the builder, so each agent went with its own default. Codex doesn't usually stop to ask, so "ask the user" on its own won't fix this. The skill needs a stated default.What changed
integrate-revenuecat: new 5c. Choose how the paywall is built, and the overview points to it.create-paywall-ai/rc paywalls generate(both onmainsince 2.3.0).revenuecat-purchase-flow: the description is scoped to custom (hand-coded) paywalls, and the intro points back at 5c.revenuecat-paywall: removes a stale "(when available)" and points at 5c.For reviewers
Testing
Only skill text changed. I haven't done a behavioral run yet. The useful check is a sample app with no paywall, run through Codex and Claude Code several times each with "hook up RevenueCat", counting builder vs. hand-coded vs. asked, before and after this change.
🤖 Generated with Claude Code
Note
Low Risk
Documentation-only skill guidance changes with no runtime, auth, or data-handling impact.
Overview
Adds an explicit paywall strategy step so agents don’t treat dashboard RevenueCatUI paywalls and hand-coded purchase flows as interchangeable follow-ups after SDK integration.
integrate-revenuecatnow requires settling Section 5c before paywall code: RevenueCat Paywall is the default when there’s no existing paywall (create viarevenuecat-paywall-design, present viarevenuecat-paywall); custom UI stays the path when the app already has a subscription screen, wired throughgetOfferings()andrevenuecat-purchase-flow. The section documents trade-offs (OTA design vs app releases, paywall A/B testing), when to ask vs default non-interactively, and that unpublished drafts show the SDK fallback until published on the offering. The skill description also triggers on “hook up RevenueCat” and adding subscriptions/IAP.Follow-up skills are re-scoped: former “5c. Subsequent skills” becomes 5d and lists
revenuecat-paywall-design;revenuecat-purchase-flowis described as custom-paywall-only and defers to 5c;revenuecat-paywalldrops stale “when available” and points at 5c for users who haven’t chosen a path.Reviewed by Cursor Bugbot for commit 968b719. Bugbot is set up for automated code reviews on this repo. Configure here.