chore: phase 0 prereqs (rivetkit workspace member, serde_bare align, wasm skip.agentOs default)

[abcxff]

No description provided.

[abcxff]

Stack for rivet-dev/rivet

• feat(rivetkit-agent-os): phase 3 process — 11 arms (exec/spawn/wait/kill/stop/list/all/tree/get/writeStdin/closeStdin) + ExecResultDto #5197 unkrsszt · 2026-06-08 18:29:29
• test(rivetkit-agent-os): partial-failure driver test for readFiles batch DTO (Option<content>, Option<error>) #5196 pzzzpmtr · 2026-06-08 18:29:28
• feat(rivetkit-agent-os): phase 3 filesystem — mkdir/readdir/exists/move/deleteFile/readFiles/writeFiles/readdirRecursive arms + batch DTOs #5195 onyykwlk · 2026-06-08 18:29:27
• test(rivetkit-agent-os): config parsing tests + documented JS↔Rust software shape mismatch (phase 3 prep) #5194 ouyvrlxt · 2026-06-08 18:29:26
• feat(rivetkit-agent-os): phase 2 — stat action arm + structured-payload wrapper tests for cross-encoding parity #5193 slkvonmt · 2026-06-08 18:29:25
• feat(rivetkit): phase 1c — agentOs() native ladder + engine driver branch + driver-suite e2e green #5192 nxzrussy · 2026-06-08 18:29:24
• feat(rivetkit-napi): phase 1b — NapiActorFactory.fromAgentOs static constructor + writeFile arm + e2e factory round-trip #5191 mtlnunov · 2026-06-08 18:29:22
• feat(rivetkit-agent-os): phase 1a — rust crate skeleton + readFile arm + dispatcher e2e against real sidecar #5190 wrxkrssv · 2026-06-08 18:29:21
• feat(rivetkit): JSON_COMPAT_UINT8_ARRAY byte-payload encoding parity with TS (encode + decode + cross-language tests) #5189 ukssumln · 2026-06-08 18:29:20
• chore: phase 0 prereqs (rivetkit workspace member, serde_bare align, wasm skip.agentOs default) #5188 qvnkvkxw · 2026-06-08 18:29:19 👈
• docs: PLAN2 + TODOLIST + RIVETKIT_RUST_FIX (clean slate, post-wipe) #5187 yqrwozvn · 2026-06-08 18:29:18
• main

Check out this stack: forklift get https://github.com/rivet-dev/rivet/pull/5188
Pull/update this stack: forklift sync
Publish local edits: forklift submit
Merge when ready: forklift merge

[claude]

PR Review: chore: phase 0 prereqs

Overview

This small prereqs PR does three things:

1. Adds rivetkit-rust/packages/rivetkit to the Cargo workspace
2. Aligns serde_bare in the client crate to use the workspace version instead of a pinned one
3. Defaults agentOs: true in the wasm driver test skip config to prevent failures from native-only agent-os dependencies

---

Issues

serde_cbor is unmaintained and conflicts with the rest of the codebase

rivetkit-client/Cargo.toml depends on serde_cbor = "0.11.2", and pulling the package into the workspace now makes this visible in the lockfile. Two concerns:

• serde_cbor is officially unmaintained. The ecosystem successor is ciborium, which the rivetkit crate itself already depends on (visible in the new lockfile entry). Using both in the same workspace is inconsistent.
• serde_cbor 0.11.2 transitively pulls in half 1.8.3, creating a second version of half in the build graph alongside the existing half 2.7.1. Duplicate major versions add build time and binary size for no benefit.

The rivetkit-client crate should be migrated to ciborium (or use the workspace ciborium dep) before or alongside bringing it into the workspace.

---

Nits

Comment style (shared-harness.ts)

The added comment is accurate, but per CLAUDE.md conventions it should use complete, flowing sentences rather than a semicolon-separated two-liner and a parenthetical. Suggested rewrite:

// The agent-os suite requires the NAPI runtime because rivetkit-agent-os
// depends on agent-os-client which uses tokio::process, a native-only API.
// Callers can re-enable the suite by passing `skip: { agentOs: false }`.

Empty PR description — even for a prereq/chore PR, a one-sentence summary of what downstream PRs this unblocks would help reviewers triage the stack quickly.

---

What looks good

• serde_bare.workspace = true alignment is the right move and keeps version management centralized.
• Defaulting agentOs: true in the wasm harness is correct — native-only tests silently failing on wasm is a real footgun.
• The new rivetkit crate lockfile entry uses ciborium and vbare correctly per the architecture constraints.