Skip to content

docs(compute): remove beta label from rivet compute#5414

Merged
abcxff merged 3 commits into
mainfrom
stack/docs-compute-remove-beta-label-from-rivet-compute-qqkuorrw
Jul 15, 2026
Merged

docs(compute): remove beta label from rivet compute#5414
abcxff merged 3 commits into
mainfrom
stack/docs-compute-remove-beta-label-from-rivet-compute-qqkuorrw

Conversation

@abcxff

@abcxff abcxff commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@claude

claude Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Review: docs(compute): remove beta label from rivet compute

Overview

Removes the "Beta" designation from Rivet Compute across the dashboard deploy-option list, the pricing component, and the docs page, and refactors Provider/deployOptions in frontend/packages/shared-data/src/deploy.ts to use an explicit union type instead of one derived via satisfies + as const.

Correctness

  • Verified badge is optional on DeployOption, so dropping badge: "Beta" from the rivet entry is safe and the {option.badge ? <Badge>...} : null rendering in getting-started.tsx degrades cleanly (no leftover badge).
  • Checked for other lingering "beta" references tied to Rivet Compute (sitemap, docs landings) — the remaining badge: "Beta" hits in website/src/sitemap/mod.ts and docsLandings.ts are for the Rust/Effect.ts SDK quickstarts and agentOS, unrelated to this change, so nothing was missed.
  • compute-pricing.ts comment/doc updates ("pending-pricing notice" instead of "beta notice") accurately describe the still-existing finalized gating behavior in ComputePricing.tsx — no functional change there, just wording.

Code quality / style

  • The Provider type change (deriving it from an explicit union instead of (typeof deployOptions)[number]["name"]) is a minor design regression: previously the type was mechanically derived from the data (single source of truth), so adding/removing/renaming a deployOptions entry automatically kept Provider in sync. Now Provider and the array must be kept in sync by hand. This isn't required to remove the beta badge — worth double-checking this refactor was intentional/desired rather than an incidental side effect, since it's unrelated to the PR's stated purpose (removing the beta label).
  • That said, the new form is easy to read and each deployOptions entry is still checked against Provider via the name: Provider field, so a typo'd provider name will still fail to typecheck.

Test coverage

  • Pure content/label changes with a low-risk type refactor; no tests included, which is consistent with the scope of this PR (no behavioral logic beyond a string literal type).

Risk

Low risk — text/label removal plus a type-representation refactor that TypeScript will catch if deployOptions and Provider drift apart.

@abcxff
abcxff changed the base branch from stack/fix-container-runner-report-crashed-child-exits-as-errored-stops-pnvtxroq to main July 15, 2026 15:40
@abcxff
abcxff merged commit 83a9044 into main Jul 15, 2026
8 of 11 checks passed
@abcxff
abcxff deleted the stack/docs-compute-remove-beta-label-from-rivet-compute-qqkuorrw branch July 15, 2026 15:40
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.

1 participant