Skip to content

Conversation

@NathanFlurry
Copy link
Member

  • Created StateManager class for connection state management
  • Moved state proxying and change tracking to dedicated manager
  • Changed conn methods to use symbols for internal access:
    • actor, stateEnabled, persistRaw (getters)
    • hasChanges(), markSaved(), sendMessage() (methods)
  • Updated all references across codebase to use new symbols
  • Maintains backward compatibility for public API (send, disconnect, etc)
  • Consistent naming pattern with instance/mod.ts StateManager

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

- Created StateManager class for connection state management
- Moved state proxying and change tracking to dedicated manager
- Changed conn methods to use symbols for internal access:
  - actor, stateEnabled, persistRaw (getters)
  - hasChanges(), markSaved(), sendMessage() (methods)
- Updated all references across codebase to use new symbols
- Maintains backward compatibility for public API (send, disconnect, etc)
- Consistent naming pattern with instance/mod.ts StateManager

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vercel
Copy link

vercel bot commented Nov 9, 2025

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

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

Copy link
Member Author

NathanFlurry commented Nov 9, 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.

@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 9, 2025

More templates

@rivetkit/actor

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

@rivetkit/cloudflare-workers

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

@rivetkit/core

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

@rivetkit/db

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

@rivetkit/framework-base

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

@rivetkit/next-js

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

@rivetkit/react

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

rivetkit

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

@rivetkit/sql-loader

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

@rivetkit/engine-runner

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

@rivetkit/engine-runner-protocol

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

commit: eb167b2

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