chore(kitchen-sink): configurable endpoint

[NathanFlurry]

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

[NathanFlurry]

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

• chore(claude): add pkg-pr-new preview deploy lookup instructions #4767
• refactor(pegboard-gateway2): consolidate hibernation in-flight and tunnel flush #4754
• chore(kitchen-sink): configurable endpoint #4766 👈 (View in Graphite)
• refactor(rivetkit-core): rename StopReason to ShutdownKind and kill enum fallthroughs #4765
• chore(envoy-client): log command and event payloads #4751
• fix(rivetkit): minor sleep-cleanup follow-ups #4761
• docs(rivetkit): clean up stale setPreventSleep references in docs and JSDoc #4760
• docs(rivetkit): document keepAwake/waitUntil counter-arm race and surface bridge errors #4759
• test(engine): force overdue branch in alarm-during-sleep test with negative offset #4758
• fix(rivetkit-core): return stopping not starting when sleep/destroy called mid-shutdown #4757
• test(rivetkit): remove sleepWithPreventSleep fixture and tests #4756
• test(engine): add regression test for alarm-during-sleep-transition race #4755
• test(rivetkit-core): cover sleep/destroy guards and prevent_sleep no-op #4750
• docs: add keepAwake vs waitUntil table and internal sleep-sequence invariants #4749
• feat(rivetkit): expose c.keepAwake(promise) through native path #4748
• refactor(rivetkit): deprecate setPreventSleep to no-op in TypeScript #4747
• fix(rivetkit-core): enumerate non-drained counters in grace deadline warn log #4746
• fix(rivetkit-core): abort graceful cleanup task when grace deadline elapses #4745
• fix(rivetkit-core): error on sleep/destroy before startup or already-requested #4743
• fix(rivetkit-core): cap SerializeState shutdown timeout at 15s #4742
• refactor(rivetkit-core): remove prevent_sleep field and deprecate set_prevent_sleep to no-op #4741
• fix(api-public): remove orphan import_export actors module #4764
• main

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

[claude]

PR Review: chore(kitchen-sink): configurable endpoint

Summary: This PR makes the Rivet endpoint configurable via VITE_RIVET_ENDPOINT in the kitchen-sink example and adds an onSleep lifecycle handler to counterWithLifecycle. The majority of the diff is automated reformatting.

---

Bug: Production Endpoint Regression

Severity: Medium

The old endpoint logic:

const rivetEndpoint = import.meta.env.DEV
    ? "http://localhost:6420"
    : `${window.location.origin}/api/rivet`;

The new logic:

const rivetEndpoint =
    import.meta.env.VITE_RIVET_ENDPOINT ?? "http://localhost:6420";

The original code routed production traffic through the Hono server at /api/rivet (served by src/server.ts), which is the canonical server-side registry handler. The new code removes that production path and falls back to http://localhost:6420 (the raw envoy port) unless VITE_RIVET_ENDPOINT is explicitly set at build time.

Since VITE_RIVET_ENDPOINT is a Vite build-time variable, it must be set before vite build runs. The Dockerfile does not set it, and there is no .env.example documenting it. Any deployment built without VITE_RIVET_ENDPOINT will silently use http://localhost:6420 in production — bypassing the Hono server and connecting to a port that is not publicly exposed.

Recommendation: Restore the production fallback and use VITE_RIVET_ENDPOINT only as an override:

const rivetEndpoint =
    import.meta.env.VITE_RIVET_ENDPOINT ??
    (import.meta.env.DEV
        ? "http://localhost:6420"
        : `${window.location.origin}/api/rivet`);

Or, if the intent is to fully replace the old approach, update the Dockerfile with ARG/ENV VITE_RIVET_ENDPOINT and document the variable in the README.

---

Minor: onSleep Demo Could Use a Clarifying Comment

File: examples/kitchen-sink/src/actors/counter/lifecycle.ts

onSleep: async (c) => {
    c.state.events.push("onSleep:start");
    await new Promise<void>((resolve) => setTimeout(resolve, 1000));
    c.state.events.push("onSleep:end");
},

The 1-second delay is fine since the default sleepGracePeriod is 15,000 ms. A short inline comment noting that the delay is intentional and demonstrates an async sleep hook would help users learning from this example understand the grace period relationship.

---

Formatting Churn

Roughly 80% of the diff is formatting-only changes: line-wrapping function signatures, arrow function parentheses, JSX indentation, and promise chain alignment. These are all minor cosmetic improvements with no behavioral impact.

---

Summary

Category	Finding	Severity
Bug	Production endpoint regression — no fallback to /api/rivet for deployed builds without VITE_RIVET_ENDPOINT	Medium
Documentation	VITE_RIVET_ENDPOINT not documented in README or Dockerfile	Low
Code quality	onSleep async demo would benefit from a comment explaining the grace period relationship	Low
Formatting	Large automated reformatting — no issues found	None

The production endpoint regression is the main concern and should be addressed before merging.

[github-actions]

Preview packages published to npm

Install with:

npm install rivetkit@pr-4766

All packages published as 0.0.0-pr.4766.3ff566a with tag pr-4766.

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-3ff566a
docker pull rivetdev/engine:full-3ff566a

Individual packages

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