chore: Clean up

[MasterPtato]

No description provided.

[MasterPtato]

Stack for rivet-dev/rivet

• #5552 [slopfix] fix(universaldb): match fdb conflict semantics in …
• #5551 [slopfix] fix(universaldb,util,pegboard-envoy): index dedup …
• #5398 feat(kitchen-sink): add churnDb actor and seed driver for co …
• #5397 chore: Clean up 👈
• #5382 [slopfix] docs(self-hosting): correct postgres config format …
• #5356 [slopfix] fix(pegboard): decrement kv_storage_used metric on …
• #5354 [slopfix] refactor(universaldb,ups): single/multi-node udb o …
• #5353 [slopfix] perf(universaldb): fold postgres drain_batch into …
• #5342 [SLOP(claude-opus-4-8-high)] perf(universaldb): batch leader …
• #5341 [slopfix] docs(self-hosting): promote postgres from experime …
• #5340 [slopfix] fix(self-host): repair dev compose generation, kit …
• #5337 [SLOP(claude-opus-4-8-high)] feat(universaldb): graceful pos …
• #5336 [slopfix] test(universaldb): postgres leader failover + abor …
• #5335 [SLOP(claude-opus-4-8-high)] feat(ups): table-backed postgre …
• #5329 [SLOP(claude-opus-4-8-high)] feat(universaldb): postgres lea …
• #5327 [SLOP(claude-opus-4-8)] feat(pegboard-envoy): rate-limit env …
• #5326 [SLOP(claude-opus-4-8)] feat(util): add rate limiter primiti …
• #5322 [SLOP(claude-opus-4-8)] feat(rivetkit-core): add serde durat …
• #5321 Add observation for serde_bare
• #5320 Add observation and ser/de metrics for serde calls
• #5314 [SLOP(claude-opus-4-8-medium)] chore: add profiling cargo pr …
• #5306 [SLOP(claude-opus-4-8)] fix(kitchen-sink): serve built front …
• main

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

_{change mnloxsnm}

[claude]

Review

Small, low-risk cleanup PR reverting what looks like temporary debug instrumentation. One issue to fix before merge.

Issues

engine/packages/gasoline/src/db/kv/mod.rs — leftover dead variable

async fn publish_signal(...) -> WorkflowResult<()> {
    let started_at = Instant::now();
    self.pools
        ...

started_at (line 2607) was only ever read by the tracing::info! block that this PR removes. It's now an unused local — cargo check/clippy will flag it as an unused-variable warning, and depending on CI's warning policy this could fail the build. Please remove the let started_at = Instant::now(); line (and the now-unnecessary Instant import if it becomes unused elsewhere in the file — a quick grep shows Instant is still used by db/kv/system.rs but worth double-checking mod.rs itself doesn't lose its only remaining use).

Other changes

self-host/compose/template/src/docker-compose.ts — drops RIVET_OTEL_SAMPLER_RATIO=1

Without this override, engine/packages/metrics-server/src/providers.rs falls back to its default of 0.001 (0.1% trace sampling) for the local dev compose stack. That's presumably intended here (reverting a temporary 100%-sampling debug aid alongside the log line above), but since the PR has no description, it's worth confirming this is deliberate and not an accidental side effect — dev-environment tracing will become much sparser by default after this change. Per CLAUDE.md, this edit correctly targets the template source (self-host/compose/template/) rather than a generated self-host/compose/dev* file, so no regen step is missing there.

Test coverage

No tests included, but neither change has behavior worth covering with a new test — this is pure debug-logging/config removal with no new logic.

Summary

Straightforward cleanup. Fix the unused started_at variable before merging; the otel sampler ratio removal is likely fine but worth a one-line confirmation in the PR description.