-
Notifications
You must be signed in to change notification settings - Fork 129
chore(site): update pricing page with Actions and remove compute billing #3266
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?
Conversation
NicholasKissel
commented
Oct 26, 2025
- Updated pricing structure to include Actions with 1M limit on free tier and 1M included for paid plans
- Removed compute billing from pricing model
- Updated pricing calculator to include Actions with allowance
- Updated documentation to reflect new Actions pricing structure
- Updated all pricing displays and calculations
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
How to use the Graphite Merge QueueAdd the label merge-queue to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
PR Review: Pricing Page UpdatesSummaryThis PR updates the pricing structure to include Actions billing and removes compute billing. The changes are spread across documentation and website pricing components. Positive Aspects
Potential Issues1. Inconsistent pricing information (Critical)There is a discrepancy in the Save State Reads pricing between files:
Calculation: $0.20 per million = $200 per billion, NOT $1.00 per billion This is a significant pricing inconsistency that needs to be resolved. Either:
Recommendation: Use consistent units across all files. I suggest using "per million" for consistency with other metrics. 2. Missing bandwidth allowance in calculator (Minor)The pricing calculator notes "10 GB bandwidth included" (line 96 in PricingCalculator.tsx), but the documentation doesn't clearly state this allowance in the pricing tables. The calculator hardcodes this on line 23: const bandwidthCost = Math.max(0, bandwidthGB - 10) * COST_PER_GB_BW;However, this 10GB allowance isn't documented in pricing.md or clearly shown in the tier comparison tables. 3. Documentation formatting issue (Minor)In pricing.md line 42, the table formatting is broken in the "Usage" column. The calculation is all on one line making it hard to read: Consider adding line breaks or restructuring the table for better readability. Suggestions1. Add validation/testsConsider adding unit tests for the 2. Clarify Actions applicabilityThe docs state "Applies to Actor actions and workflow executions" but it would be helpful to clarify if Functions and Containers are billed for actions too, or if this is Actor-specific. 3. Consider adding storage allowance to pricing tablesCurrently, the tiers show "5GB Storage included/limit" but the usage pricing table doesn't mention any storage allowance. Are the first 5GB free? This should be clarified. Code Quality
Testing Recommendations
Action ItemsMust fix before merge:
Should consider:
Overall, this is a clean pricing update, but the critical pricing discrepancy needs to be resolved before merging to avoid customer confusion and potential revenue loss. |
More templates
@rivetkit/actor
@rivetkit/cloudflare-workers
@rivetkit/core
@rivetkit/db
@rivetkit/framework-base
@rivetkit/next-js
@rivetkit/react
rivetkit
@rivetkit/sql-loader
@rivetkit/engine-runner
@rivetkit/engine-runner-protocol
commit: |
Merge activity
|
…ing (#3266) - Updated pricing structure to include Actions with 1M limit on free tier and 1M included for paid plans - Removed compute billing from pricing model - Updated pricing calculator to include Actions with allowance - Updated documentation to reflect new Actions pricing structure - Updated all pricing displays and calculations
- Updated pricing structure to include Actions with 1M limit on free tier and 1M included for paid plans - Removed compute billing from pricing model - Updated pricing calculator to include Actions with allowance - Updated documentation to reflect new Actions pricing structure - Updated all pricing displays and calculations
ebfbb55 to
cf2413f
Compare
Code Review - PR #3266SummaryThis PR updates the pricing structure to include Actions billing and removes compute-based billing. Overall, the changes are well-structured and consistent across documentation and UI components. ✅ Strengths
🐛 Issues Found1. Inconsistent Pricing Values (High Priority)Location: The pricing table shows: Save State Reads: $1.00 per billion readsBut the pricing.md documentation states: Save State Reads: 1M included, then $0.20 per millionIssue: These values don't align. If $0.20 per million = $200 per billion, not $1.00 per billion. Recommendation: Verify the correct pricing and ensure consistency. The 2. Missing Newline at End of FileLocation: The file ends without a newline character. While not critical, this is a common convention in version control. 3. Indentation InconsistencyLocation: The indentation changed from tabs to mixed spacing. The diff shows removal of tabs followed by re-addition with different indentation. According to CLAUDE.md, Rust uses hard tabs, but this is TypeScript/TSX. However, consistency within the file should be maintained. 💡 Suggestions1. Pricing Constant DiscrepancyLocation: const COST_PER_GB_BW = 0.15;
const COST_PER_MILLION_READS = 0.2;
const COST_PER_MILLION_WRITES = 1.0;
const COST_PER_GB_MONTH = 0.40;The bandwidth cost changed from $0.05 to $0.15, but the comment on line 92 still says: Note: 10 GB bandwidth included, $0.15/GB for additionalThis is now correct, but verify that the 10GB included allowance is accurate for all tiers. 2. Bandwidth Allowance ClarificationLocation: const bandwidthCost = Math.max(0, bandwidthGB - 10) * COST_PER_GB_BW;The code assumes 10GB is always included, but the pricing tiers show different egress limits:
Recommendation: Either document that the calculator is simplified or adjust the logic to match tier-specific allowances. 3. Storage Cost ChangedThe storage cost changed from $0.20 to $0.40 per GB-month (doubled). Ensure this is intentional and documented in release notes if applicable. 🔒 Security ConsiderationsNo security concerns identified. This PR only updates pricing display logic and documentation. 🧪 Test CoverageRecommendation: Consider adding/updating tests for:
📋 Action ItemsMust Fix:
Nice to Have:
Overall, this is a solid PR with good consistency across files. The main blocker is the pricing value discrepancy that needs resolution before merging. |
