Docs: fix stale CLI references and broken links across READMEs#6
Merged
Conversation
Root cause: @aauth/local-keys was split into a library (no bin) and @aauth/bootstrap (the CLI), but several READMEs and bootstrap skill files still instructed users and AI agents to run `npx @aauth/local-keys`, which fails with "could not determine executable to run". Changes: - Root README: add bootstrap, fetch, hardware-keys to packages table; replace broken `npx @aauth/local-keys` example with @aauth/bootstrap; normalize spec link casing. - local-keys/README.md: rewrite as library-only; remove obsolete CLI section; redirect CLI users to @aauth/bootstrap. - bootstrap/skills/setup.md + 4 platform skills: replace every `npx @aauth/local-keys ...` with `npx @aauth/bootstrap ...`. These skill files are loaded by AI agents during setup and would fail the same way. - mcp-agent, mcp-server, mcp-stdio, mcp-openclaw READMEs: fix "AAuth repo" link that pointed at packages-js as the spec source; now separately links packages-js and dickhardt/AAuth (the spec). - Add bootstrap/README.md, fetch/README.md, hardware-keys/README.md (previously missing). - CLAUDE.md: fix stale spec local path and casing. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Adds the missing resolved entries for the optional @aauth/hardware-keys-* platform sub-packages (darwin-arm64, darwin-x64, linux-x64-gnu, win32-x64-msvc). The optionalDependencies block in package-lock.json listed them at 0.8.1 but the deep resolved nodes were stale from a prior version, causing `npm ci` to fail in CI. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
rohanharikr
added a commit
that referenced
this pull request
May 11, 2026
The default person server URL in docs was inconsistent: bootstrap/skills/setup.md claimed `issuer.hello.coop`, fetch/skills/fetch.md used `person.hello-beta.net`, and the new READMEs I added in #6 mistakenly used `hello.coop` (copied from a stale example). Normalize everything to `person.hello-beta.net`. Also add a note wherever the default is introduced (root README, bootstrap README, setup.md skill) that the Hellō Beta PS resets data regularly and that users can point at their own PS via `--person-server` / `--ps`. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
4 tasks
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.
Summary
@aauth/local-keyswas split into a library (nobin) and@aauth/bootstrap(the CLI), but READMEs and bootstrap skill files still instructed users and AI agents to runnpx @aauth/local-keys, which fails with "could not determine executable to run".bootstrap,fetch,hardware-keys), and correct spec links that pointed at this repo instead ofdickhardt/AAuth.Changes
README.md— added@aauth/bootstrap,@aauth/fetch,@aauth/hardware-keysto the packages table (bootstrapwas missing entirely); replaced the brokennpx @aauth/local-keysquick-start example with@aauth/bootstrap.local-keys/README.md— rewritten as library-only. Obsolete CLI section removed; CLI users redirected to@aauth/bootstrap. API section retained.bootstrap/skills/setup.md+ 4 platform skills (github-pages,gitlab-pages,cloudflare-pages,netlify) — everynpx @aauth/local-keys …replaced withnpx @aauth/bootstrap …. These are loaded by AI agents during setup and were silently broken the same way.mcp-agent,mcp-server,mcp-stdio,mcp-openclawREADMEs — fixed the "AAuth repo" link that pointed at this repo as the source of the protocol spec; now linksaauth-dev/packages-jsanddickhardt/AAuthseparately.bootstrap/README.md,fetch/README.md,hardware-keys/README.md.CLAUDE.md— fixed stale spec local path and casing.Related spec-side cleanup: dickhardt/AAuth#27 updates the TypeScript implementation link to point at this repo.
Test plan
npx @aauth/bootstrap discoverworks (matches what the new docs claim)npx @aauth/bootstrap show,npx @aauth/bootstrap skill setupworkaauth-dev/packages-js,dickhardt/AAuth, in-repo paths)npx @aauth/local-keysreferences in markdown —grep -r "npx @aauth/local-keys" --include="*.md"returns nothing🤖 Generated with Claude Code