chore: update sdks

[jog1t]

No description provided.

[jog1t]

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

• feat(dashboard): provide ns when querying kv #3831 : 2 dependent PRs (#3561 , #3803 )
• chore: update sdks #3830 👈 (View in Graphite)
• refactor: performance improvements for inspector #3452
• fix(dashboard): use proper timestamp for actor status #3809
• fix(engine): require namespace for actor id-specific routes #3828
• main

---

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]

PR Review: SDK Updates

This PR updates the generated SDKs (Go, Rust, TypeScript) to make the namespace parameter required for two Actor APIs:

1. ActorsDelete - DELETE /actors/{actor_id}
2. ActorsKvGet - GET /actors/{actor_id}/kv/keys/{key}

Summary of Changes

TypeScript SDK:

• actorsDelete(): Changed namespace from optional (namespace?: string) to required (namespace: string)
• actorsKvGet(): Added required request parameter with namespace: string, previously missing query param

Go SDK:

• ActorsDelete: Changed Namespace from *string (optional pointer) to string (required)
• ActorsKvGet: Added new ActorsKvGetRequest struct with required Namespace field

Rust SDK:

• actors_delete: Changed namespace from Option<&str> to &str (required)
• actors_kv_get: Added required namespace: &str parameter

Review Feedback

✅ Positive Aspects

1. API Consistency: Making namespace required is good for API clarity and prevents errors from missing namespace context
2. Type Safety: All three SDK languages now enforce the requirement at compile time
3. Clean Generated Code: The changes appear to be properly generated with consistent patterns across all SDKs
4. Version Bump: Rust SDK properly incremented from 2.0.33 to 2.0.34-rc.1

⚠️ Breaking Changes

This is a BREAKING CHANGE that will affect existing users:

1. TypeScript:

   • Before: client.actorsDelete("actor_id") or client.actorsDelete("actor_id", {})
   • After: client.actorsDelete("actor_id", { namespace: "namespace" })
   • Before: client.actorsKvGet("actor_id", "key")
   • After: client.actorsKvGet("actor_id", "key", { namespace: "namespace" })

2. Go:

   • Before: ActorsDeleteRequest{ Namespace: nil } or with &someNamespace
   • After: ActorsDeleteRequest{ Namespace: "namespace" } (must provide value)

3. Rust:

   • Before: actors_delete(config, actor_id, None) or Some("ns")
   • After: actors_delete(config, actor_id, "ns") (must provide value)

Recommendation: Ensure this breaking change is:

• Documented in release notes/changelog
• Accompanied by migration guide for users
• Released as a major version (or clearly marked as breaking in minor release)

🔍 Questions & Considerations

1. Missing Context: What prompted this change? Was namespace incorrectly optional in the OpenAPI spec previously?

2. Go SDK Query Param Logic: In engine/sdks/go/api-full/client/client.go:268-271, there's now:

   queryParams.Add("namespace", fmt.Sprintf("%v", request.Namespace))
   if len(queryParams) > 0 {
       endpointURL += "?" + queryParams.Encode()
   }

   The if len(queryParams) > 0 check is now redundant since we always add namespace. Consider simplifying (though this is auto-generated, so may need to fix the generator).

3. Rust SDK Empty String Handling: With namespace: &str now required, there's no compile-time protection against empty strings. Consider if validation is needed at runtime (though this may be an API server concern).

4. Version Consistency: Only the Rust SDK shows a version bump in this PR. Do Go and TypeScript SDKs need version bumps as well?

📝 Minor Observations

1. All 96 files changed are version updates and generated code - appears automated and consistent
2. Documentation examples updated appropriately in all SDKs
3. No test files included - assuming tests are elsewhere or auto-generated separately

Verdict

APPROVED with recommendations

The code changes are technically sound and properly implement making namespace required across all SDKs. However, this is a significant breaking change that needs proper:

• Release planning (version numbering)
• User communication (migration guide, deprecation notice if applicable)
• Changelog documentation

The implementation itself looks correct and consistent across all three SDK languages.

[pkg-pr-new]

More templates

• actor-actions
• example-ai-agent
• example-ai-and-user-generated-actors-freestyle
• chat-room
• example-cloudflare-workers
• example-cloudflare-workers-hono
• example-cloudflare-workers-inline-client
• cross-actor-actions
• example-cursors
• example-cursors-raw-websocket
• custom-serverless
• example-sqlite
• example-elysia
• example-experimental-durable-streams-ai-agent
• hello-world
• example-hono
• hono-react
• example-kitchen-sink
• multi-region
• native-websockets
• next-js
• example-raw-fetch-handler
• example-raw-websocket-handler
• example-raw-websocket-handler-proxy
• example-react
• scheduling
• state
• example-stream
• example-trpc

@rivetkit/cloudflare-workers

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

@rivetkit/db

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

@rivetkit/framework-base

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

@rivetkit/next-js

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

@rivetkit/react

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

rivetkit

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

@rivetkit/sql-loader

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

@rivetkit/virtual-websocket

pnpm add https://pkg.pr.new/rivet-dev/rivet/@rivetkit/virtual-websocket@3830

@rivetkit/engine-runner

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

@rivetkit/engine-runner-protocol

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

commit: 4be0871

[graphite-app]

Graphite Automations

"Test" took an action on this PR • (01/11/26)

1 assignee was added to this PR based on Kacper Wojciechowski's automation.

[graphite-app]

Merge activity

• Jan 12, 9:55 PM UTC: NathanFlurry added this pull request to the Graphite merge queue.
• Jan 12, 9:56 PM UTC: CI is running for this pull request on a draft pull request (#3840) due to your merge queue CI optimization settings.
• Jan 12, 9:56 PM UTC: Merged by the Graphite merge queue via draft PR: #3840.