Skip to content

docs: split README into topic docs, replace Homebrew with binary install - #4

Merged
wirwolf merged 4 commits into
masterfrom
claude/happy-gauss-yglyg3
Sep 15, 2026
Merged

wirwolf merged 4 commits into
masterfrom
claude/happy-gauss-yglyg3

Conversation

@wirwolf

@wirwolf wirwolf commented Sep 15, 2026

Copy link
Copy Markdown
  • Move full command reference into docs/commands.md, grouped by area, and document previously-undocumented commands (status, init, seal, unseal, rekey, local, logout, renew, uuid, dhparam, undelete, revert, ask, exists, versions, ls, option, vault, curl, x509 reissue/show, target delete).
  • Move sync (pull/plan/apply) and the local JSON file format into docs/sync.md as general functionality, independent of Docker.
  • Move Docker usage examples into docs/docker.md.
  • Replace the outdated Homebrew instructions in README with binary download examples (curl/wget/PowerShell) and a short Docker snippet, and drop the stale Slack link.

- Move full command reference into docs/commands.md, grouped by area,
  and document previously-undocumented commands (status, init, seal,
  unseal, rekey, local, logout, renew, uuid, dhparam, undelete, revert,
  ask, exists, versions, ls, option, vault, curl, x509 reissue/show,
  target delete).
- Move sync (pull/plan/apply) and the local JSON file format into
  docs/sync.md as general functionality, independent of Docker.
- Move Docker usage examples into docs/docker.md.
- Replace the outdated Homebrew instructions in README with binary
  download examples (curl/wget/PowerShell) and a short Docker snippet,
  and drop the stale Slack link.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 05783a2da0

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread docs/sync.md
directly or via Docker, see [docker.md](docker.md) for container examples).
It synchronizes secrets between Vault and a local directory of JSON files,
which is useful for reviewing changes before they're applied, or for
managing secrets under version control / GitOps-style workflows.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Remove the recommendation to version plaintext secrets

When users follow this GitOps recommendation, sync pull writes the actual Vault values as ordinary, unencrypted JSON files, including credentials such as the password shown later in this document. Committing that directory therefore exposes secrets permanently in repository history; recommend version control only with a documented encryption workflow, or explicitly warn users to exclude the generated files.

Useful? React with 👍 / 👎.

Comment thread README.md Outdated
Comment thread docs/commands.md Outdated
The combined "macOS (Intel) / Linux" wget snippet fetched
vault-manager-darwin-amd64.tar.gz, so on Linux the extracted Mach-O
binary would not run. Split into separate Linux (linux-amd64) and
macOS Intel (darwin-amd64) examples, each using its own release asset.
The uuid command takes a single PATH[:KEY] argument; the example
passed path and key as two separate positional args, which the
handler rejects with a usage error before ever reaching Vault.
@wirwolf
wirwolf merged commit 2d56b0b into master Sep 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants