Skip to content

fix: get rid of old hub.rivet.dev references#5123

Merged
abcxff merged 1 commit into
mainfrom
06-02-fix_get_rid_of_old_hub.rivet.dev_references
Jun 4, 2026
Merged

fix: get rid of old hub.rivet.dev references#5123
abcxff merged 1 commit into
mainfrom
06-02-fix_get_rid_of_old_hub.rivet.dev_references

Conversation

@abcxff
Copy link
Copy Markdown
Contributor

@abcxff abcxff commented Jun 2, 2026

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

Copy link
Copy Markdown
Contributor Author

abcxff commented Jun 2, 2026

@abcxff abcxff force-pushed the 06-02-fix_frontend_onboarding_prompts branch from 310086b to a3735c0 Compare June 2, 2026 19:39
@abcxff abcxff force-pushed the 06-02-fix_get_rid_of_old_hub.rivet.dev_references branch from 970b31e to 36601dd Compare June 2, 2026 19:39
@railway-app railway-app Bot temporarily deployed to rivet-frontend / rivet-pr-5123 June 2, 2026 19:40 Destroyed
@railway-app
Copy link
Copy Markdown

railway-app Bot commented Jun 2, 2026

🚅 Deployed to the rivet-pr-5123 environment in rivet-frontend

Service Status Web Updated (UTC)
kitchen-sink 🕒 Building (View Logs) Web Jun 4, 2026 at 2:37 pm
frontend-cloud ✅ Success (View Logs) Web Jun 4, 2026 at 2:35 pm
frontend-inspector ❌ Build Failed (View Logs) Web Jun 4, 2026 at 2:33 pm
website ✅ Success (View Logs) Web Jun 4, 2026 at 2:33 pm
mcp-hub ✅ Success (View Logs) Web Jun 4, 2026 at 2:32 pm
ladle ✅ Success (View Logs) Web Jun 2, 2026 at 7:44 pm

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jun 2, 2026

Code Review

Summary: This PR replaces 16 occurrences of hub.rivet.dev with dashboard.rivet.dev across documentation, examples, and one commented-out line in utils.ts. No production logic is changed.

Findings

Incomplete coverage — stale references in .agents/skills/

Four files in the checked-in skill reference cache still point to the old URL:

.agents/skills/multiplayer-game/reference/connect/cloudflare.md
.agents/skills/multiplayer-game/reference/connect/supabase.md
.agents/skills/rivetkit/reference/connect/cloudflare.md
.agents/skills/rivetkit/reference/connect/supabase.md

These are cached copies from the external rivet-dev/skills repo (skills-lock.json lists them). They are loaded into AI agent context and would point users to the old URL. The fix likely needs to land in rivet-dev/skills first and then be re-locked here, but it is worth tracking.

Dead commented-out code in utils.ts:138

The updated line is inside a block that has been commented out since it was introduced. Keeping the URL current in dead code is unnecessary maintenance. If the URL was never meant to ship, the comment block should be deleted outright.


No other hub.rivet.dev references found in *.md, *.mdx, *.ts, *.tsx, *.js, *.json, *.yaml, *.toml, or *.rs files outside the .agents/ cache. No behavioral or security changes.

@claude
Copy link
Copy Markdown
Contributor

claude Bot commented Jun 2, 2026

Code Review: fix: get rid of old hub.rivet.dev references

Overview

This PR replaces all hub.rivet.dev URLs with dashboard.rivet.dev across 11 files — docs, example READMEs, a blog post, and one TypeScript source file. A clean, focused change.


Issue: Incomplete Coverage

The PR updated the examples/*-render/ directories but missed the parallel examples/*-render-vercel/ directories. All four are tracked in git and still contain stale URLs:

  • examples/react-render-vercel/README.md
  • examples/stream-render-vercel/README.md
  • examples/state-render-vercel/README.md
  • examples/chat-room-render-vercel/README.md

Each has two occurrences of hub.rivet.dev in the env-var table (same pattern as the files that were updated).


Minor Notes

Dead commented-out code in utils.ts (pre-existing, not introduced here): The URL inside the block comment at line ~137 has been correctly updated, but the entire object it belongs to is commented out. If it is not going to be uncommented, removing it would reduce noise. Either way, not a blocker for this PR.

PR description: The template checkboxes are unfilled and the testing section is empty. For a pure find-and-replace change this is low risk, but marking the type as "Bug fix" would keep the PR history clean.


Summary

The change is correct and necessary where it applies. The only actionable ask is to extend the same substitution to the four missed *-render-vercel READMEs before merging.

@abcxff abcxff marked this pull request as ready for review June 2, 2026 19:42
@abcxff abcxff force-pushed the 06-02-fix_get_rid_of_old_hub.rivet.dev_references branch from 36601dd to fd26f7e Compare June 4, 2026 14:30
@abcxff abcxff force-pushed the 06-02-fix_frontend_onboarding_prompts branch from a3735c0 to 4de0ea1 Compare June 4, 2026 14:30
@railway-app railway-app Bot temporarily deployed to rivet-frontend / rivet-pr-5123 June 4, 2026 14:30 Destroyed
Copy link
Copy Markdown
Contributor Author

abcxff commented Jun 4, 2026

Merge activity

  • Jun 4, 2:31 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Jun 4, 2:38 PM UTC: Graphite rebased this pull request as part of a merge.
  • Jun 4, 2:39 PM UTC: @abcxff merged this pull request with Graphite.

@abcxff abcxff changed the base branch from 06-02-fix_frontend_onboarding_prompts to graphite-base/5123 June 4, 2026 14:35
@abcxff abcxff changed the base branch from graphite-base/5123 to main June 4, 2026 14:36
@abcxff abcxff force-pushed the 06-02-fix_get_rid_of_old_hub.rivet.dev_references branch from fd26f7e to 0e2c3b7 Compare June 4, 2026 14:37
@railway-app railway-app Bot temporarily deployed to rivet-frontend / rivet-pr-5123 June 4, 2026 14:37 Destroyed
@abcxff abcxff merged commit a5c32c6 into main Jun 4, 2026
14 of 19 checks passed
@abcxff abcxff deleted the 06-02-fix_get_rid_of_old_hub.rivet.dev_references branch June 4, 2026 14:39
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