Skip to content

fix: rename-completeness sweep + correct install hints + broken doc commands#224

Open
rylinjames wants to merge 1 commit into
mainfrom
fix/rename-sweep-and-install-hints
Open

fix: rename-completeness sweep + correct install hints + broken doc commands#224
rylinjames wants to merge 1 commit into
mainfrom
fix/rename-sweep-and-install-hints

Conversation

@rylinjames

Copy link
Copy Markdown
Collaborator

Closes a cluster of v0.12.0 rename casualties and onboarding breakage from the full-codebase audit (Part 1 #2 + #7).

What & why

Fix Was Now Impact
Install hints pip install 'tether[...]' (cli.py:39, 3027, 3617) 'fastcrest-tether[...]' bare tether is foreign-owned on PyPI — supply-chain + broken onboarding
Generated CI / report cmds tether validate <dir> (ci_template:53, verification_report:66,177) tether validate export <dir> validate is a Typer group; bare form removed → every adopter's CI failed
README SDK examples from tether.client import ReflexClient (188, 243) TetherClient ReflexClient raises ImportError — the only SDK snippets a new integrator copies
Grafana dashboards query reflex_* metrics tether_* (matches observability/prometheus.py) every panel was silently empty
Docs config file reflex_config.json tether_config.json (matches exporter) getting_started, verification, NOTES
pyproject classifier License :: OSI Approved :: Apache Software License removed misrepresented BUSL-1.1 + conflicts with PEP-639 SPDX license field
LICENSE Licensed Work Reflex Version 0.5.0... Tether (formerly Reflex) Version 0.5.0... names the current work as the legal scope, keeps 0.5–0.11 continuity

Verification

  • All changed Python byte-compiles (py_compile).
  • Both dashboard JSONs re-validated with json.load after the metric rename.
  • validate export / export --target/--output invocations confirmed against the actual Typer wiring in cli.py (not guessed).
  • CHANGELOG historical entries deliberately left unchanged (shipped-state record under the old name).

Intentional reflex-compat surfaces (env vars, license prefix, D1 names, fingerprint salt, cert field, import shim) were not touched — see the audit's Part 4 allowlist.

🤖 Generated with Claude Code

…ommands

Closes a cluster of v0.12.0 rename casualties and onboarding breakage found
in the full-codebase audit (reflex_context/02_research/2026-06-10-tether-oss-
improvement-audit.md, Part 1 #2 + #7):

Install hints (supply-chain): three hints still told users to
`pip install 'tether[...]'`, but the bare `tether` name is foreign-owned on
PyPI — the dist is `fastcrest-tether`. Fixed cli.py:39 (tui), :3027 (serve,gpu),
:3617 (serve extra).

Broken generated/doc commands: `validate` is a Typer group (`tether validate
export <dir>`); the bare `tether validate <dir>` form was removed (now
`validate-legacy`, hidden). Every adopter's generated CI and every
verification report reproducer emitted the dead form. Fixed ci_template.py:53,
verification_report.py:66,177.

README SDK examples imported `ReflexClient` (nonexistent → ImportError) in both
Python snippets — the only SDK examples a new integrator copies. Fixed to
`TetherClient`.

Grafana dashboards queried `reflex_*` metrics while the code emits `tether_*`
(observability/prometheus.py) — every panel was silently empty. Renamed all
metric queries + cosmetic titles/uids/tags in both dashboards; JSON revalidated.

Docs referenced `reflex_config.json`; the exporter writes `tether_config.json`
(verification_report.py:124, _pytorch_backend.py). Fixed docs/getting_started.md,
docs/verification.md, reference/NOTES.md. (CHANGELOG historical entries left as
shipped-state record.)

License accuracy: pyproject classifier claimed "OSI Approved :: Apache Software
License" on a BUSL-1.1 package (also conflicts with the PEP-639 SPDX `license`
field) — removed. LICENSE "Licensed Work: Reflex" → "Tether (formerly Reflex)"
so the legal scope names the current work while preserving 0.5.0–0.11.x
continuity.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant