Skip to content

Conversation

@NicholasKissel
Copy link
Member

No description provided.

@vercel
Copy link

vercel bot commented Nov 7, 2025

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

Project Deployment Preview Comments Updated (UTC)
rivet-site Ready Ready Preview Comment Nov 7, 2025 11:23pm
3 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
rivet-cloud Ignored Ignored Nov 7, 2025 11:23pm
rivet-inspector Ignored Ignored Preview Nov 7, 2025 11:23pm
rivetkit-serverless Skipped Skipped Nov 7, 2025 11:23pm

@vercel vercel bot temporarily deployed to Preview – rivetkit-serverless November 7, 2025 23:20 Inactive
Copy link
Member Author


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 7, 2025

PR Review - Weekly Update Post

Overview

This PR adds a weekly changelog post for week 2025-45 (Nov 2, 2025). The changes are straightforward and follow the established patterns.

Positive Aspects ✅

  • Consistent Format: Follows the same structure as previous weekly update posts
  • Proper Metadata: Includes all required exports (author, published, category, keywords)
  • Week Number Accuracy: Uses ISO week 45 which correctly corresponds to November 2, 2025
  • Clean Binary: Image file is a valid PNG (2048x1024, RGBA)
  • Relevant Keywords: Keywords appropriately describe the changelog items

Issues Found 🔍

1. Minor: Truncated Commit Message (Low Priority)

File: Commit message
Issue: Commit message appears truncated: "chore(site): weekly updat" (missing 'e')
Fix: Should be "chore(site): weekly update"
Impact: Minor - doesn't affect functionality but looks incomplete in git history

2. Minor: Published Date vs Post Date Mismatch (Clarification Needed)

File: website/src/posts/2025-11-02-weekly-updates/page.mdx:2
Issue: Directory name is 2025-11-02 but the post is dated Nov 7, 2025 (today)
Context: Looking at historical posts, there's inconsistency:

  • 2025-10-05-weekly-updates has published = "2025-10-10"
  • 2025-09-28-weekly-updates has published = "2025-10-03"
  • This PR: 2025-11-02-weekly-updates has published = "2025-11-02"
    Question: Is the directory name meant to represent week start date vs publish date? May want to establish a clear convention.

3. Minor: Trailing Newlines (Style Consistency)

File: website/src/posts/2025-11-02-weekly-updates/page.mdx:11-12
Issue: File has two trailing newlines at the end
Context: Other weekly updates have 1 trailing newline (e.g., 2025-10-24, 2025-10-05)
Fix: Remove one of the trailing newlines for consistency

Recommendations 💡

  1. Fix truncated commit message if possible (might need to amend or create follow-up)
  2. Consider removing one trailing newline for consistency with other posts
  3. Document date convention for future weekly updates to avoid confusion

Content Quality

The changelog items are clear and concise:

  • WebSocket hibernation feature
  • Live WebSocket migration
  • Backoff state visibility

All items use good technical language and provide value to readers.

Security & Performance

No concerns. This is purely content addition with no code changes.

Test Coverage

N/A - This is documentation/content, no tests needed.

Final Verdict

LGTM with minor suggestions - The PR is good to merge as-is. The issues identified are cosmetic and don't block merging. Consider addressing the trailing newlines and establishing a date naming convention for future posts.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 7, 2025

More templates

@rivetkit/actor

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/actor@3414

@rivetkit/cloudflare-workers

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/cloudflare-workers@3414

@rivetkit/core

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/core@3414

@rivetkit/db

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/db@3414

@rivetkit/framework-base

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/framework-base@3414

@rivetkit/next-js

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/next-js@3414

@rivetkit/react

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/react@3414

rivetkit

pnpm add https://pkg.pr.new/rivet-dev/rivet/rivetkit@3414

@rivetkit/sql-loader

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/sql-loader@3414

@rivetkit/engine-runner

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner@3414

@rivetkit/engine-runner-protocol

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/engine-runner-protocol@3414

commit: ed7daba

@NicholasKissel NicholasKissel marked this pull request as ready for review November 8, 2025 00:09
@NicholasKissel NicholasKissel changed the title chore(site): weekly updat chore(site): weekly update Nov 8, 2025
@graphite-app
Copy link
Contributor

graphite-app bot commented Nov 8, 2025

Merge activity

  • Nov 8, 12:32 AM UTC: NicholasKissel added this pull request to the Graphite merge queue.
  • Nov 8, 12:33 AM UTC: CI is running for this pull request on a draft pull request (#3415) due to your merge queue CI optimization settings.
  • Nov 8, 12:34 AM UTC: Merged by the Graphite merge queue via draft PR: #3415.

graphite-app bot pushed a commit that referenced this pull request Nov 8, 2025
@claude
Copy link

claude bot commented Nov 8, 2025

PR Review: Weekly Update Post

Summary

This PR adds a weekly changelog post for week 45 covering WebSocket hibernation, live migration, and backoff state visibility. The changes are minimal and follow the established pattern for weekly updates.

Code Quality ✅

  • Formatting: Follows the established pattern from previous weekly updates
  • Consistency: Metadata structure matches other posts (author, published date, category, keywords)
  • Content: Clear, concise bullet points describing new features

Issues Found ⚠️

Date/Week Mismatch
The post is titled "2025-45" (week 45) but the published date is "2025-11-02" which falls in ISO week 44, not week 45. Current week (2025-11-08) is week 45.

Recommendation: Either:

  • Update the published date to a date in week 45 (e.g., "2025-11-08" or later in that week)
  • OR update the title to "Rivet Weekly Changelog 2025-44" to match November 2nd

Minor Observations

Image File
The PR includes image.png (299KB) which follows the pattern from other posts. Consider:

  • Ensuring the image is optimized for web (299KB seems reasonable but verify compression)
  • Verifying the image content is relevant to the changelog items

Content Suggestions (optional)
The bullet points are concise but might benefit from slightly more detail:

  • "WebSocket hibernation allows your actors to go to sleep while websockets stay open" - Clear ✅
  • "Live WebSocket migration for actor failover & upgrades" - Clear ✅
  • "Shows actors in backoff state if your backend is crashing" - Consider: "Dashboard now shows actors in backoff state when your backend is crashing" (adds context about where this is shown)

Security & Performance ✅

  • No security concerns
  • Static content, no performance impact
  • Binary image file is within reasonable size

Test Coverage

N/A - This is documentation/website content

Recommendation

Approve with minor fix: Resolve the date/week number mismatch before merging. Otherwise, the PR follows all conventions correctly.

@graphite-app graphite-app bot closed this Nov 8, 2025
@graphite-app graphite-app bot deleted the 11-07-chore_site_weekly_updat branch November 8, 2025 00:34
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