fix(infra): deploy workers as tether-* + accept tether_version field#236
Open
rylinjames wants to merge 1 commit into
Open
fix(infra): deploy workers as tether-* + accept tether_version field#236rylinjames wants to merge 1 commit into
rylinjames wants to merge 1 commit into
Conversation
Audit §3.11 / Part 1 #18 — every networked feature was silently dead. All 4 Cloudflare workers deployed under reflex-* names while every v0.12.0 client defaults to tether-*.workers.dev, so activation, license heartbeat, telemetry, curate uploads, and episode uploads all hit dead hostnames (every failure swallowed at debug). Renamed the worker `name` in all 4 wrangler.toml to tether-* so they deploy at the URLs clients already call. The D1 `database_name` fields stay reflex-* for data continuity (intentional). Field drift (also §3.11): the telemetry worker REQUIRED `reflex_version` and the license worker READ `body.reflex_version`, but the renamed client sends `tether_version` — so telemetry got a 400 on every heartbeat (100% loss) and the license worker recorded version as all-"unknown". Both now accept `tether_version || reflex_version` (D1 columns keep the reflex_version name). Also: license worker /admin/init next-step pointed at the dead src/reflex/pro/_public_key.py shim path (→ src/tether); deploy READMEs updated to the tether-* hostnames. Both worker.js pass `node --check`. Operator action: redeploy the 4 workers (`wrangler deploy`) for the new names to take effect. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Audit §3.11 / Part 1 #18 — every networked feature was silently dead by default.
Problem
All 4 Cloudflare workers deployed under
reflex-*names while every v0.12.0 client defaults totether-*.workers.dev— so activation, license heartbeat, telemetry, curate uploads, and episode uploads all hit dead hostnames (failures swallowed at debug).Plus field drift: the telemetry worker required
reflex_versionand the license worker readbody.reflex_version, but the renamed client sendstether_version→ telemetry 400'd on every heartbeat (100% loss); license version recorded as all-"unknown".Fix
namein all 4wrangler.tomltotether-*. D1database_namestaysreflex-*(data continuity — intentional).tether_version || reflex_version(D1 columns keep thereflex_versionname)./admin/initpointer (src/reflex/→src/tether/) and updated deploy READMEs to thetether-*hostnames.Both
worker.jspassnode --check.Operator action
Redeploy the 4 workers (
wrangler deploy) for the new names to take effect.🤖 Generated with Claude Code