fix(epoxy): use vbare

[MasterPtato]

Fixes RVT-6176

[MasterPtato]

• fix(epoxy): use vbare #4616 👈 (View in Graphite)
• fix: dont upsert for ensure if missing #4615
• fix(envoy): require runner config for ws conn #4612
• fix(pb): backport serverless fix #4611
• fix(pb): optimize actor key allocation #4610
• fix: move runner configs to epoxy #4605
• chore(epoxy): clean up kv get #4604
• fix(pb-envoy): reduce round trips for ws connection #4603
• fix(envoy-client): fix handling actor event ack #4599
• fix(envoy-client): add tls #4595 : 1 other dependent PR (#4597 )
• chore: fix driver test suite #4591
• chore: tunnel auth #4588
• chore: remove udb as a dependency of envoy-client #4587
• chore: revert actor v2 hack #4586
• fix: fix pool metadata protocol version refresh, rivetkit native ws #4585
• feat: US-001 - Define SqliteKv trait in rivetkit-sqlite-native #4584
• feat(rust/engine-runner): minimal rust engine runner #4289
• chore: delete kv channel #4583
• feat(engine): rust runner #4582
• chore: remove non-engine drivers #4574 : 1 other dependent PR (#4581 )
• feat: dynamic actors #4397
• fix: driver test suite & sqlite reliability #4559
• fix: remove serverless token #4572
• fix: misc token fixes #4569
• fix(pb): clean up actor stop decision handling #4566
• fix(envoy): use global instance, add signal handlers #4565
• chore: misc fixes, add pb snapshot test #4561
• fix(cache): make in memory cache global #4560
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[claude]

PR Review: fix(epoxy): use vbare

Bug: Double-deserialize in get_local.rs

In get_local.rs, after let cache_value = cache_key.deserialize(&value)? returns a CachedValue, the code then calls cache_key.deserialize(&value) a second time on the inner cache_value.value bytes (raw KV payload). Those bytes are not versioned-CachedValue-encoded, so this will fail at runtime.

Fix: return the already-deserialized value directly:

cache_value: cache_value.value.is_some().then_some(cache_value)

Issue: Modifying envoy-protocol/v1.bare

Per CLAUDE.md: Never modify an existing published .bare protocol version. Removing serverlessDrainGracePeriod from ProtocolMetadata shifts the field layout (BARE is position-dependent), breaking any older envoy build. Move to v2.bare or document v1 was never deployed.

Positives

1. CachedValue with value=None replaces the empty-Vec sentinel hack correctly
2. vbare versioned headers are the right architectural direction
3. CommittedValue in epoxy-protocol is a clean type consolidation
4. matches!() for is_serverless is cleaner
5. Test imports updated correctly

The double-deserialize bug and v1.bare modification are the two items to address before merging.

[linear]

RVT-6176 Update epoxy v2 to be vbare

[MasterPtato]

Merge activity

• Apr 13, 12:21 AM UTC: A user started a stack merge that includes this pull request via Graphite.
• Apr 13, 12:44 AM UTC: Graphite rebased this pull request as part of a merge.
• Apr 13, 12:44 AM UTC: @MasterPtato merged this pull request with Graphite.