fix(env): simplify self-host runtime defaults - #814
izadoesdev wants to merge 6 commits into
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
2 Skipped Deployments
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Advanced Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
The latest updates on your projects. Learn more about Unkey Deploy
|
fb3a142 to
d3ca51e
Compare
|
3d87ada to
ebe714f
Compare
|
@greptileai review this. Final head ebe714f is rebased onto staging containing merged #816 and #806. Browser mode forwarding is now present and the existing Next regression verifies self-host and hosted production URL defaults plus signup tracking. SDK mocks use exported SDK types. The external Docker commit is preserved. All 30 focused tests, lint, 33 type tasks, and 29 pre-push tasks pass. |
|
@greptileai review this. Final head 218cb66 adds only --isolate to the existing Links test command, fixing the Linux CI ordering conflict where a route test replaced the shared environment module before producer tests imported readBooleanEnv. All 27 Links tests and all 29 repository test tasks pass. The previously reviewed runtime changes and merged dependency base are unchanged. |
SELFHOST=truekeeps omitted service URLs local, lets queues reuseREDIS_URL, and prevents server product telemetry from starting, even with copied Databuddy keys. Explicit URL overrides still win; hosted defaults remain unchanged.Adds explicit workspace dependencies for the existing environment helper. No new runtime settings or schema changes.
Validation: 30 focused environment, browser-config, Redis, and telemetry tests pass. The browser regression verifies both mode values, local versus hosted production URLs, and preserved pixel behavior. Root lint (22 policy checks), all 33 typecheck tasks, frozen install, and all 29 pre-push tasks pass.
Depends on #816 for exposing self-host mode to the dashboard browser bundle and #806 for
SELFHOSTforwarding through Turbo; both are merged. Rebased directly onto staging containing #816. Preserves the existing Docker build argument and browser-mode forwarding; complements the short environment template and Compose setup in #804.