[SLOP(claude-opus-4-8-medium)] docs(pool-configuration): fix runner-config SDK snippet to use RivetClient from engine-api-full

[NathanFlurry]

No description provided.

[NathanFlurry]

Stack for rivet-dev/rivet

• [SLOP(gpt-5)] feat(cli): add rivetkit cli #5255 kprpzusk · 2026-06-13 23:00:59
• [SLOP(claude-opus-4-8[1m]-medium)] docs(website): consolidate landing pages, deploy quickstarts, and docs restructuring #5254 psyxxqon · 2026-06-13 23:00:58
• [SLOP(claude-opus-4-8)] refactor(rivetkit): deprecate actor kv api across core and typescript #5253 polvovvk · 2026-06-13 23:00:57
• [SLOP(claude-opus-4-8)] docs(examples): split effect example into hello-world-effect and chat-room-effect #5252 wolqzmyo · 2026-06-13 23:00:56
• [SLOP(gpt-5)] fix(rivetkit): normalize protocol args source-side #5251 lvpxytwl · 2026-06-13 00:10:46
• [SLOP(claude-opus-4-8)] docs(actors): apply docs audit fixes #5250 rtqklwqu · 2026-06-13 00:10:45
• [SLOP(claude-opus-4-8)] docs(actors): rewrite in-memory state page, remove sandbox page, fix stateSaveInterval default, forbid em dashes #5249 pqwwounv · 2026-06-13 00:10:44
• [SLOP(claude-opus-4-8-medium)] docs(pool-configuration): fix runner-config SDK snippet to use RivetClient from engine-api-full #5248 tmoxrznq · 2026-06-13 00:10:42 👈
• [SLOP(claude-opus-4-8-medium)] docs(inspector-tabs): mark illustrative type-shape snippets as @nocheck #5247 yosxkyox · 2026-06-13 00:10:41
• [SLOP(claude-opus-4-8-medium)] docs(quickstart): clean up rust quickstart to import shared actor and auto-download engine #5246 lrzpsprl · 2026-06-13 00:10:40
• [SLOP(claude-opus-4-8-medium)] fix(rivetkit-wasm): add missing inspector_tabs field to ActorConfigInput conversion #5245 ookuputp · 2026-06-13 00:10:39
• main

Get stack: forklift get 5248
Push local edits: forklift submit
Merge when ready: forklift merge 5248

[claude]

Code Review

Overview

This PR updates two documentation snippets that show how to configure runner pools via the SDK. The changes fix the import path, constructor option name, method call style, and config schema nesting to match the current @rivetkit/engine-api-full generated client.

Changes Summary

	Before	After
Import	rivetkit/client	@rivetkit/engine-api-full
Constructor option	endpoint	environment
Method	rivet.runnerConfigs.upsert()	rivet.runnerConfigsUpsert()
Config shape	flat serverless at top level	nested under datacenters.default
@nocheck on pool-configuration.mdx	absent	added

Looks Good

• Import correction — @rivetkit/engine-api-full is the correct generated-client package. The old rivetkit/client path does not exist for this use case.
• environment vs endpoint — matches the actual RivetClient constructor option; the old name would cause a silent no-op at runtime.
• Method name — runnerConfigsUpsert is the flat generated-SDK style vs the old chained runnerConfigs.upsert; correct for the target package.
• datacenters nesting — aligns with the current API schema that scopes serverless config per datacenter.
• @nocheck on pool-configuration.mdx — consistent with freestyle.mdx, appropriate for illustrative SDK snippets that are not meant to be compiled directly.

Minor Observation

Both snippets use process.env.* and hardcoded placeholder strings that would not type-check cleanly, so the @nocheck flag is warranted on both. No action needed.

Verdict

Documentation-only change. Corrections are accurate and consistent across both files. No bugs, no security concerns, no performance impact. LGTM.