[SLOP(claude-opus-4-8-medium)] fix(rivetkit-wasm): add missing inspector_tabs field to ActorConfigInput conversion

[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 5245
Push local edits: forklift submit
Merge when ready: forklift merge 5245

[railway-app]

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

Service	Status	Web	Updated (UTC)
website	😴 Sleeping (View Logs)	Web	Jun 14, 2026 at 2:23 am
frontend-cloud	😴 Sleeping (View Logs)	Web	Jun 14, 2026 at 2:08 am
kitchen-sink	❌ Build Failed (View Logs)	Web	Jun 13, 2026 at 12:18 am
ladle	✅ Success (View Logs)	Web	Jun 13, 2026 at 12:14 am
frontend-inspector	❌ Build Failed (View Logs)	Web	Jun 13, 2026 at 12:12 am
mcp-hub	✅ Success (View Logs)	Web	Jun 13, 2026 at 12:11 am

[claude]

Code Review

Overview

This PR adds the missing inspector_tabs: None field to the From<WasmActorConfig> for ActorConfigInput conversion in rivetkit-wasm. This is a compile-time fix: inspector_tabs was recently added to ActorConfigInput in rivetkit-core, and the struct literal in the wasm conversion needed to be updated. The fix correctly hard-codes None since custom inspector tabs depend on a PathBuf filesystem root that has no meaning in a browser wasm host.

---

Correctness

The fix is correct. InspectorTabEntry::Custom carries a root: PathBuf for serving assets from the filesystem — a native/server-only capability. Hard-coding None in the wasm path is the right call. WasmActorConfig intentionally omits an inspector_tabs field, so there is no user-facing regression.

Minor comment precision nit: The comment says the feature "has no meaning in a browser wasm host", which is fully true for Custom tabs. The HideBuiltin variant does not require a filesystem path and could theoretically apply to a wasm host too — but since WasmActorConfig has no field for users to express either variant, None is still the only sensible value. The comment is a small simplification, not a bug.

---

Pre-existing drift in WasmActorConfig

Unrelated to this PR, WasmActorConfig has several fields that are not forwarded in the From impl:

• create_state_timeout_ms
• on_create_timeout_ms
• on_wake_timeout_ms
• on_before_actor_start_timeout_ms
• on_request_timeout_ms

These do not exist on ActorConfigInput, suggesting they were once valid and since removed (or were never in core). Worth tracking as follow-up cleanup to avoid confusion — silently dropping struct fields can mask user config mistakes. Not a blocker for this PR.

---

Summary

The change is minimal, correct, and follows project conventions. The comment is accurate and readable. No test changes are needed since this is a compile-level fix and the wasm integration suite covers the conversion path.

Verdict: approve.