Repository navigation
Add docs site, standalone root README and mod guide - #2
Merged
Merged
Conversation
- docs/: index, getting started, a page per plugin (credentials, variables, commands, safety, troubleshooting), a shared safety model and a guide to building a mod; _config.yml so GitHub Pages can serve it from /docs - docs/examples/homelab-guard: the guide's worked example, a tool.check mod that asks before confirmed destructive homelab commands, with tests; CI validates and tests it - README: quick start, credentials at a glance, example prompts, safety summary, update/uninstall and a docs index - mods/README.md points to the guide and keeps the conventions - proxmox README: token creation is no longer listed as a free action Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CE5mahfuiAeTFLfTQP2dbR
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
The guard misses multiline confirmed commands, and several published Pages links are broken.
Review effort: Balanced
Findings: 2
Open (8)
Match multiline destructive commands · New Update example patterns for multiline commands · New Document transport-specific API requirements · New Use repository URL for CONTRIBUTING.md · New Use repository URL for full reference · New Link CONTRIBUTING.md to repository · New Link to repository copy outside docs · New Use repository URL for full reference · New
What changed in this PR
Adds a GitHub Pages-ready documentation site, expands standalone onboarding, and introduces a tested function-hook mod guide.
Changes:
- Adds installation, plugin, safety, and troubleshooting documentation.
- Adds the tested
homelab-guardworked example and CI validation. - Corrects Proxmox token-creation guidance.
| File | Description |
|---|---|
README.md |
Expands root onboarding and documentation links. |
plugins/proxmox/README.md |
Corrects token-creation safety guidance. |
plugins/proxmox/CHANGELOG.md |
Records the documentation correction. |
mods/README.md |
Redirects readers to the expanded mod guide. |
docs/index.md |
Adds the documentation landing page. |
docs/getting-started.md |
Documents installation and configuration. |
docs/truenas.md |
Adds TrueNAS setup and safety guidance. |
docs/proxmox.md |
Adds Proxmox setup and command guidance. |
docs/unifi.md |
Adds UniFi setup and safety guidance. |
docs/safety.md |
Compares plugin safety models. |
docs/mods.md |
Adds the function-hook development guide. |
docs/_config.yml |
Configures GitHub Pages. |
docs/examples/homelab-guard/.claude-plugin/plugin.json |
Defines the example plugin. |
docs/examples/homelab-guard/hooks/hooks.json |
Registers its hook module. |
docs/examples/homelab-guard/hooks/register.ts |
Implements command permission checks. |
docs/examples/homelab-guard/hooks/guard.test.ts |
Tests guard behavior. |
.github/workflows/ci.yml |
Validates and tests the example. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- homelab-guard example matches commands split across lines (s flag), with a regression test; the guide's copy is regenerated from the source - links from docs/ to files outside it use repository URLs, so they work on GitHub Pages - TrueNAS: variable requirements are per transport (API or SSH) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01CE5mahfuiAeTFLfTQP2dbR
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
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
This PR adds three things: a
docs/site, a root README that works on its own, and a step-by-step guide to building a mod with a tested example.Changes
docs/(plain Markdown that renders on GitHub; GitHub Pages can serve it from/docsvia_config.yml):index.mdgetting-started.md: install, a sourced env file, first checks, update and uninstall, and moving from the old repos.truenas.md,proxmox.md,unifi.md: per plugin, where to create credentials (read-only first), the variables, commands and example prompts, the safety rules, and troubleshooting.safety.md: the shared model, with a per-plugin comparison table covering gates, previews, what runs without asking, redaction, TLS defaults and read-only credentials.mods.md: the hook model, layout, a worked example, develop/check/publish steps, and limits.docs/examples/homelab-guard: the guide's worked example.tool.checkhook onBash. It turns anallowinto anaskfortn.py … --confirm,unifi.py … --yesandpve-api.sh delete, never loosens adeny, and fails safe toask.docs/mods.mdis generated from these files.claude plugin validateandclaude plugin teston it.README.md: quick start, credentials at a glance, example prompts per plugin, a safety summary, update/uninstall, and a docs index.mods/README.md: now points to the guide and keeps the conventions.plugins/proxmox/README.md: token creation is no longer listed as a free action, finishing the review fix from Consolidate truenas, proxmox and unifi plugins into one marketplace #1. A CHANGELOG line records it.Testing
*.md: 0 broken.claude plugin validate .passes.claude plugin validatepasses,claude plugin testgives 4 pass, andtsc(strict) is clean.After merge
To publish the site, enable Pages under Settings → Pages: branch
main, folder/docs.🤖 Generated with Claude Code
https://claude.ai/code/session_01CE5mahfuiAeTFLfTQP2dbR
Generated by Claude Code