chore: update comment

[MasterPtato]

Description

Please include a summary of the changes and the related issue. Please also include relevant motivation and context.

Type of change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• This change requires a documentation update

How Has This Been Tested?

Please describe the tests that you ran to verify your changes.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes

[railway-app]

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

Service	Status	Web	Updated (UTC)
kitchen-sink	❌ Build Failed (View Logs)	Web	May 5, 2026 at 6:18 pm
website	😴 Sleeping (View Logs)	Web	May 5, 2026 at 1:00 am
frontend-inspector	❌ Build Failed (View Logs)	Web	May 5, 2026 at 12:50 am
frontend-cloud	❌ Build Failed (View Logs)	Web	May 5, 2026 at 12:49 am
ladle	❌ Build Failed (View Logs)	Web	May 5, 2026 at 12:49 am
mcp-hub	✅ Success (View Logs)	Web	May 5, 2026 at 12:48 am

[MasterPtato]

• fix(rivetkit): fix empty state [slopfix] #4967
• chore: update comment #4966 👈 (View in Graphite)
• fix(pegboard): align drain grace defaults with stop threshold #4958 : 1 other dependent PR (#4960 )
• fix(rivetkit): preserve bridge error messages #4959
• fix(rivetkit): fire abort signal on shutdown finalize #4957
• fix(rivetkit): use startup kv preload #4956
• fix(client): flatten skipReadyWait option #4955
• ci(publish): add manual preview publish flow #4954
• fix(depot): add ltx decode diagnostics #4953
• fix(universaldb): preserve plain set values #4952
• fix(universaldb): align versionstamp operand encoding #4951
• fix(rivetkit): export database provider types #4950
• wip: kitchen sink agentic loop finish #4949
• chore(rivetkit): note duplicate admission logic #4948
• test(rivetkit): cover bypass fetch header forwarding #4947
• feat(kitchen-sink): add actor probe controls #4946
• fix(gateway): split actor response wait errors #4945
• fix(guard): handle bypass preflight before query parse #4944
• feat(kitchen-sink): add mock agent event log copy #4943
• fix(kitchen-sink): default mock agent to local endpoint #4942
• test(rivetkit): enable driver lifecycle regressions #4941
• test(rivetkit): verify cached websocket db refs exceed grace #4940
• fix(rivetkit-core): keep sleeping actors reachable during grace #4939
• test(rivetkit): cover async websocket close db access #4938
• fix(rivetkit): keep actors awake until keepAwake work finishes #4937
• fix(rivetkit): persist native onSleep state on errors #4936
• fix(rivetkit-core): allow dispatch during sleep grace #4935
• fix(rivetkit): make duplicate sleep requests idempotent #4934
• main

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

[claude]

Code Review: chore: update comment

Overview

This PR adds a 3-line comment block above the debug = "line-tables-only" setting in the workspace [profile.dev] section of Cargo.toml, explaining why that debug level was chosen.

---

Feedback

Positive

• The comment is genuinely useful. line-tables-only is a non-obvious value and the explanation (skips variable/type DWARF, meaningfully faster debug builds, no runtime cost) is exactly the kind of why that future maintainers need.
• The quantified claim (~2 min per engine cross-compile) gives concrete context for the trade-off.

Style note (minor)
Per CLAUDE.md conventions, comments should be written as normal, complete sentences and should avoid dashes/parenthetical structures:

Avoid fragmented structures with parentheticals and dashes. Instead, write complete sentences.

The current text uses a dash-separated structure:

# "line-tables-only" produces just the line-number DWARF needed for stack
# traces, skipping variable/type info. Meaningfully faster debug builds
# (~2 min per engine cross-compile) with no runtime cost.

A cleaner rewrite:

# "line-tables-only" produces only the line-number DWARF needed for stack
# traces and skips variable and type info. This makes debug builds
# meaningfully faster (roughly 2 min per engine cross-compile) with no
# runtime cost.

This is a minor style nit and not a blocker.

---

Summary

Tiny, helpful change. The only callout is the comment style not quite matching project conventions. Otherwise LGTM.

[github-actions]

Preview packages published to npm

Install with:

npm install rivetkit@pr-4966

All packages published as 0.0.0-pr.4966.4b8e773 with tag pr-4966.

Engine binary is shipped via @rivetkit/engine-cli on linux-x64-musl, linux-arm64-musl, darwin-x64, and darwin-arm64. Windows users should use the release installer or set RIVET_ENGINE_BINARY.

Docker images:

docker pull rivetdev/engine:slim-4b8e773
docker pull rivetdev/engine:full-4b8e773

Individual packages

npm install rivetkit@pr-4966
npm install @rivetkit/react@pr-4966
npm install @rivetkit/rivetkit-napi@pr-4966
npm install @rivetkit/workflow-engine@pr-4966

[MasterPtato]

Merge activity

• May 5, 6:19 PM UTC: @MasterPtato merged this pull request with Graphite.