This repository was archived by the owner on Oct 28, 2026. It is now read-only.
Repository navigation
Remove dead SiteSitter link + add README badges - #2
Merged
Merged
Conversation
added 3 commits
June 12, 2026 00:39
- Remove the SiteSitter entry from the Authensor Safety Stack list; the AUTHENSOR/SiteSitter repo is now private and the link 404s. - Add a small badge row (npm version, node engine, MIT license). All three resolve against live data: npm reads the published version from the registry, node reads the engines field, license links to the LICENSE file.
Make the no-control-plane first run legible for newcomers: - Add a 'First run (no control plane yet)' section: `safeclaw run --dry-run` works standalone and exits 0 once a provider key is set; write/exec/network actions fail closed by design; `doctor` connectivity is an expected [WARN]; `health` reporting unreachable and exiting non-zero is the expected fail-closed probe result. - Remove only the npm version badge, which rendered a stale orange beta.1 (repo is beta.2, only beta.1 is published) and signaled 'unmaintained'. Node + license badges kept (both accurate). Docs-only; no code or CLI behavior changes. Behavior verified: dry-run exits 0, doctor connectivity WARN (exit 0 with key set), health exits non-zero when control plane unreachable.
…ead npx/--demo paths Replace every 'npx authensor up' (the package never existed — E404) with the real, public setup: clone AUTHENSOR/AUTHENSOR and run 'docker compose up -d' (verified on main; serves http://localhost:3000, SafeClaw's default port). Sites: README (x2), docs/install.md (x2), src/cli.js (x2), src/doctor.js, src/server.js, ui/dashboard/app.js. Stop advertising 'safeclaw init --demo' in user docs (demo auto-provisioning has no working backend; it degrades to a Google Form). The --demo flag and provisionDemo code path are kept (they fail gracefully) and tests are unchanged; only the doc/UX promises are removed: README commands list, docs/quickstart.md, docs/cli.md. Tests: 446/446 pass. Zero 'npx authensor up' remains repo-wide.
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Light presentation touch-up for the README. No functional/code changes.
Changes
Original commit (dead link + badges)
github.com/AUTHENSOR/SiteSitter, which is now private and returns a 404. Removed that one entry. Confirmed no other references to now-private repos remain.>=20) and license (MIT) badges. (The npm version badge originally added here has since been removed — see below.)Follow-up commit (first-run legibility + badge fix)
1.0.0-beta.1(the repo/README reportbeta.2, but onlybeta.1is published to npm), which reads as "old/unmaintained" — the opposite of intent. Removed only that line; node + license badges kept (both verified accurate). Reversible once abeta.2publish reconciles the versions.safeclaw run --dry-run "Summarize README.md"works standalone and exits 0 once the wizard/inithas saved a provider API key.safeclaw doctorconnectivity is an expected[WARN];safeclaw healthreporting "unreachable" and exiting non-zero is the expected fail-closed probe result.Verification (run from a fresh clone of this branch)
safeclaw run --dry-run "..."(provider key set) → prints provider/policy simulation, exit 0.safeclaw doctor(provider key set, no control plane) → connectivity[WARN], overall exit 0.safeclaw health(no control plane) → "Control plane unreachable", exit 1 (fail-closed by design).npm test→ 446 tests. In a normal local environment all pass (control plane absent ⇒ connectivity check returns instantly). Note: in a network-restricted sandbox the 14tests/doctor.test.jscases that callrunDiagnostics()can exceed vitest's default 5000ms timeout because the connectivity probe tolocalhost:3000hangs+retries instead of refusing instantly — that is an environment timing artifact, not a code change in this PR.Notes for maintainer (out of scope for this presentation PR — raise separately)
npx authensor upis referenced but not published to npm (README, docs/install.md, src/doctor.js connectivity hint, etc.). It currently 404s, so the advertised path to stand up a control plane is a dead end. Fix needs a maintainer decision (publishauthensor, or replace every reference with the real setup). Not guessed here.safeclaw init --demo(docs/quickstart.md step 3) claims it provisions a demo Authensor token, but empirically prints "Auto-provisioning failed: fetch failed" and drops to a Google Form. Needs the demo provisioning made real, or the step rewritten. Left untouched.@authensor/safeclaw@1.0.0-beta.1; repo/README are1.0.0-beta.2. Publishingbeta.2would let the npm version badge be restored accurately.Follow-up commit
4eb40c1— control-plane setup is now real (no dead commands)npx authensor upremoved everywhere (it 404'd — the package never existed). Replaced with the real, public setup that serves the control plane on SafeClaw's default port (http://localhost:3000):Verified against the flagship's
mainbranch (hasdocker-compose.yml+ a Quickstart mapping the control plane to:3000). Fixed in: README (×2), docs/install.md (×2), src/cli.js (×2), src/doctor.js, src/server.js, ui/dashboard/app.js.safeclaw init --demode-advertised — demo token auto-provisioning has no working backend (it degrades to a Google Form), so it's removed from the README command list, docs/quickstart.md, and docs/cli.md. The--demoflag andprovisionDemocode path are kept (they fail gracefully); all tests unchanged.Result: 446/446 tests pass, zero
npx authensor upremains repo-wide, and a newcomer following any doc reaches a real, working control-plane setup.