Skip to content

Conversation

@NathanFlurry
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Nov 13, 2025

Deployment failed with the following error:

Too many requests - try again in 3 minutes (more than 120, code: "api-deployments-flood-pro").

Learn More: https://vercel.com/docs/limits#rate-limits

Copy link
Member Author

NathanFlurry commented Nov 13, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add 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.

@claude
Copy link

claude bot commented Nov 13, 2025

PR Review: Website Wording Tweaks

This PR makes small but effective improvements to website copy across the marketing pages. Overall, the changes are positive and improve clarity.

What Works Well

  1. Consistency in terminology: The change from real-time to realtime (one word) throughout the site creates better consistency and follows common industry convention for this compound adjective.

  2. Improved link targets: Changing CTAs from /docs/quickstart/ to /docs is likely better for user experience, as it gives users the docs landing page rather than jumping directly to quickstart.

  3. Better copy precision:

    • server-authoritative state is more precise than authoritative state (website/src/data/use-cases.ts:102)
    • flexible state management is more accurate than automatic state management for workflows (website/src/data/use-cases.ts:42)
    • Adding Open-source and self-hostable to hero description provides important context upfront (HeroSection.tsx:23)
  4. Reordering use cases: Moving Multiplayer Apps ahead of Workflows in the display order (UseCases.tsx:145) may better highlight a key use case.

Minor Observations

  1. Checkmark wording change (UseCases.tsx:77): The new wording is more concise. Cloud or on-prem is clearer, Supports your agent SDK is more user-focused, but Realtime might be too terse.

  2. Consistency check: The term realtime is now used consistently, which is good. Verify this aligns with documentation and technical materials.

  3. MCP description change (use-cases.ts:95): The new version is slightly more technical (stateful MCP) - ensure this terminology is accurate to how MCP is used in the product.

Suggestions for Future Consideration

  1. Hero copy length: The hero description is now quite long with the addition of Open-source and self-hostable. Consider if this might be better as a separate line or callout.

  2. Link consistency: Ensure the docs landing page provides clear next steps for new users who expect a quickstart path.

  3. A/B testing opportunity: The use case reordering might be worth tracking.

Security and Performance

  • No security concerns (text-only changes)
  • No performance impact
  • No broken links introduced (all changed paths are internal routes)

Code Quality

  • Changes follow existing code patterns
  • Proper TypeScript/TSX syntax maintained
  • No formatting issues
  • Consistent with Next.js/React conventions

Final Verdict

Approved - This is a solid set of website copy improvements. The changes enhance clarity, consistency, and positioning. The modifications are low-risk and align with marketing best practices.

Recommendation: Merge after confirming that:

  1. The /docs landing page is well-suited for users clicking Get Started CTAs
  2. The term realtime (one word) is consistently used in docs and technical materials
  3. The stateful Model Context Protocol terminology accurately reflects the product capabilities

@NathanFlurry NathanFlurry force-pushed the 11-09-chore_site_fix_generating_examples branch from 0031d2e to 9ea29d6 Compare November 13, 2025 02:47
@NathanFlurry NathanFlurry force-pushed the 11-09-chore_site_tweak_wording branch from 0ff831e to 60aeb32 Compare November 13, 2025 02:47
@vercel
Copy link

vercel bot commented Nov 13, 2025

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

Project Deployment Preview Comments Updated (UTC)
rivet-site Error Error Nov 13, 2025 9:10pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
rivet-cloud Ignored Ignored Nov 13, 2025 9:10pm
rivet-inspector Ignored Ignored Preview Nov 13, 2025 9:10pm
rivetkit-serverless Skipped Skipped Nov 13, 2025 9:10pm

@claude
Copy link

claude bot commented Nov 13, 2025

Pull Request Review

Thank you for this PR! I've reviewed the changes and they look good overall. This is a straightforward copy/UX improvement for the marketing website.

Positive Points

  1. Consistent Terminology: The change from "Real-Time" to "Realtime" (single word) is consistent and aligns with modern tech industry convention.

  2. Improved Marketing Copy: Adding "Open-source and self-hostable" to the hero description highlights key differentiators. "Cloud or on-prem" is clearer than "Cloud & on-prem". "Supports your agent SDK" is more inclusive than "Works with AI SDK".

  3. Better Use Case Ordering: Promoting "Agent Orchestration & MCP" to the top position makes sense given current market trends toward AI applications.

  4. More Accurate Descriptions: "flexible state management" is more accurate than "automatic state management" for workflows. "server-authoritative state" is more technically precise.

Observations

  1. Link Changes: All "Get Started" buttons now point to /docs instead of /docs/quickstart/. Verify that the /docs landing page guides users appropriately - if users land there without clear next steps, it could impact conversion.

  2. Consistency Check: Recommend doing a full audit to ensure "realtime" terminology is consistent across all marketing pages, documentation, and code comments.

  3. Use Case Reordering: Swapping "Workflows" and "Multiplayer Apps" positions changes the visual hierarchy. Ensure this aligns with your product strategy.

Overall Assessment

Status: Approved with minor recommendations

This is a clean, well-scoped PR that improves marketing copy clarity. No code quality, security, or performance concerns since these are static content changes.

  • Test Coverage: N/A (marketing content only)
  • Performance Impact: None
  • Security Impact: None

@claude
Copy link

claude bot commented Nov 13, 2025

PR Review: chore(site): tweak wording

Summary

This PR updates marketing copy across the Rivet website homepage, focusing on terminology consistency and improving link destinations. The changes are primarily cosmetic and don't affect functionality.

Code Quality ✅

Positive aspects:

  • Clean, focused changes with clear intent
  • Consistent terminology improvements throughout
  • No functional code changes, reducing risk
  • Follows the repository's conventional commit format

Changes Analysis

1. Terminology Standardization

  • "Real-Time" → "Realtime": Good consistency improvement. Modern convention favors "realtime" as a single word in technical contexts
  • "Open-source and self-hostable": Important addition to the hero section that clearly communicates key value propositions

2. Link Improvements

  • /docs/quickstart//docs: Simplifies navigation entry point (3 occurrences)
  • ⚠️ Recommendation: Verify that /docs provides a good landing experience for new users

3. Content Refinements

  • "automatic state management" → "flexible state management": More accurate description
  • "authoritative state" → "server-authoritative state": Clearer technical terminology
  • "Cloud & on-prem" → "Cloud or on-prem": Better grammar
  • "Works with AI SDK" → "Supports your agent SDK": More inclusive and accurate

4. Use Case Reordering

  • Agent Orchestration moved to top: Smart prioritization that aligns with current market trends

Potential Issues 🔍

No critical issues identified, but some minor considerations:

  1. Link Change: Ensure /docs route provides appropriate landing content
  2. Copy Length: Verify hero description doesn't cause layout issues on mobile
  3. Consistency: Check if other pages still use "real-time" (hyphenated)

Testing Recommendations 📋

  • Visual regression testing on multiple viewport sizes
  • Verify all updated links work correctly
  • Check text doesn't overflow containers on mobile

Final Verdict: APPROVED

This is a clean, low-risk PR that improves marketing copy consistency and accuracy.

Minor suggestions for follow-up:

  1. Document the "realtime" style choice
  2. Verify the /docs landing experience is optimal
  3. Check for consistency across all site pages

@NathanFlurry NathanFlurry force-pushed the 11-09-chore_site_fix_generating_examples branch from ba3e30a to 0911e7e Compare November 13, 2025 21:09
@NathanFlurry NathanFlurry force-pushed the 11-09-chore_site_tweak_wording branch from 2ae9f22 to ad24dd6 Compare November 13, 2025 21:09
@vercel vercel bot temporarily deployed to Preview – rivetkit-serverless November 13, 2025 21:09 Inactive
@claude
Copy link

claude bot commented Nov 13, 2025

PR Review: chore(site): tweak wording

Summary

This PR makes minor wording improvements to the marketing website, focusing on consistency and clarity. The changes are straightforward and improve the messaging.

✅ Positive Aspects

  1. Consistency Improvements

    • Standardizing "Real-Time" → "Realtime" (single word) throughout the site improves brand consistency
    • This aligns with common tech industry usage (e.g., Firebase Realtime Database)
  2. Better Navigation

    • Changing quickstart links from /docs/quickstart/ to /docs is more appropriate if the docs landing page is the better entry point
    • Consistent across all CTAs (HeroSection.tsx, CTASection.tsx, DeploymentOptionsSection.tsx)
  3. Enhanced Messaging

    • Adding "Open-source and self-hostable" to the hero description (HeroSection.tsx:23) is a strong value proposition
    • The reordering of use cases to put "Agent Orchestration & MCP" and "Multiplayer Apps" first makes sense given current market trends
  4. More Precise Language

    • "automatic state management" → "flexible state management" is more accurate (UseCases.tsx:177, use-cases.ts:42)
    • "authoritative state" → "server-authoritative state" is more technically precise (UseCases.tsx:182, use-cases.ts:102)
    • "Model Context Protocol and persistent state" → "persistence and stateful Model Context Protocol" better emphasizes the stateful nature (UseCases.tsx:172, use-cases.ts:95)
  5. Improved Checkmark Items

    • "Cloud & on-prem" → "Cloud or on-prem" is clearer (UseCases.tsx:77)
    • "Supports realtime" → "Realtime" is more concise
    • "Works with AI SDK" → "Supports your agent SDK" is more flexible and inclusive

🔍 Observations & Suggestions

No Issues Found - This is a clean wording update with no code quality, performance, security, or test coverage concerns.

Minor Considerations:

  1. Link Change Validation (Low Priority)

    • The change from /docs/quickstart/ to /docs should be verified to ensure:
      • The /docs page provides a good entry experience
      • There are no broken links elsewhere pointing to /docs/quickstart/
    • This is likely already handled, but worth a quick check
  2. Use Case Reordering (Informational)

    • Moving "Multiplayer Apps" before "Workflows" (UseCases.tsx:143-146) changes the visual hierarchy
    • This appears intentional to prioritize AI agents and multiplayer use cases
    • Consider: Does this match your current positioning strategy?

✅ Code Quality

  • TypeScript/TSX: Clean, no issues
  • Formatting: Consistent with existing code
  • Component Structure: No changes to logic, only content
  • Conventions: Follows Next.js and React best practices

✅ Performance

No performance impact - these are static string changes.

✅ Security

No security concerns - content-only changes.

✅ Test Coverage

No tests needed for marketing copy changes.

Recommendation

Approve and merge

This is a solid improvement to the marketing messaging. The changes enhance clarity, consistency, and better communicate Rivet's value proposition.

@claude
Copy link

claude bot commented Nov 13, 2025

PR Review: Wording Tweaks for Marketing Site

Thanks for the PR! This is a nice set of marketing copy refinements. Here's my review:

✅ Positive Observations

  1. Consistency Improvements: The change from "Real-Time" to "Realtime" (single word) creates better consistency across the marketing site.

  2. Better Information Density: Adding "Open-source and self-hostable" to the hero description (HeroSection.tsx:23) provides important value propositions upfront.

  3. More Accurate Terminology:

    • "server-authoritative state" is more technically precise than "authoritative state"
    • "flexible state management" is better than "automatic state management" for workflows
    • "Supports your agent SDK" is more inclusive than "Works with AI SDK"
  4. Better URL: Changing from /docs/quickstart/ to /docs is cleaner and likely more stable as documentation structure evolves.

  5. Improved Use Case Ordering: Moving "Multiplayer Apps" before "Workflows" (UseCases.tsx:145) better aligns with the realtime-first messaging.

📝 Minor Suggestions

  1. Capitalization Consistency: In UseCases.tsx:77, the checkmark text uses "Cloud or on-prem" with "on-prem" lowercase, while "Realtime" is capitalized. Consider either:

    • "Cloud or on-prem" / "Realtime" / "Supports your agent SDK" (current)
    • "Cloud or On-Prem" / "Realtime" / "Supports Your Agent SDK" (title case)
  2. Technical Accuracy: The phrase "stateful Model Context Protocol" (use-cases.ts:95) might be slightly confusing. MCP itself is a protocol - perhaps "stateful MCP servers" or "persistent MCP integration" would be clearer? This is a minor point though.

Code Quality

  • ✅ All changes are TypeScript/TSX - no type issues introduced
  • ✅ No security concerns (pure copy changes)
  • ✅ No performance impact
  • ✅ Follows existing code patterns and formatting
  • ✅ Clean, focused changes (16 additions, 16 deletions)

Testing Considerations

Since this is a Next.js marketing site with copy changes:

  • Verify the site builds successfully
  • Check that all updated links (/docs) resolve correctly
  • Test responsive layouts to ensure the slightly longer hero description doesn't cause wrapping issues on mobile

Summary

This is a clean, well-scoped PR that improves the marketing copy. The changes are low-risk and enhance clarity and consistency. Approved

The only item I'd suggest considering is the capitalization consistency in the checkmark list, but that's a very minor point and doesn't block approval.

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.

2 participants