Skip to content

chore: link Sliver docs to specific pages and correct skill guidance - #190

Merged
briangreunke merged 1 commit into
mainfrom
brian/sliver-c2-doc-link-fixes
Oct 20, 2026
Merged

briangreunke merged 1 commit into
mainfrom
brian/sliver-c2-doc-link-fixes

Conversation

@briangreunke

Copy link
Copy Markdown
Contributor

Follow-up to #189, addressing review feedback on sliver-c2/SKILL.md.

Changes

  • Specific links. Each upstream reference now links to its matching page on sliver.sh/docs (via ?name=) instead of the docs root.
  • C2 transport selection. Clarified that the callback endpoint (host, port, domain) is embedded at implant generation; start_*_listener only runs the server-side receiver, it does not set the implant's endpoint.
  • Native MCP. Noted that upstream marks it experimental — not all commands are exposed yet and HTTP/SSE auth is unsupported — and dropped the "all commands" claim. Recommend it only where it exposes a command the wrapper doesn't cover.
  • Bumped sliver-c2 1.0.1 → 1.0.2.

Checks

  • just validate strict=true passes (sliver-c2@1.0.2)
  • All sliver.sh/docs references are specific pages (no bare docs-root links)

🤖 Generated with Claude Code

Follow-up to the Sliver docs change. Addresses review feedback on SKILL.md:
- Each upstream reference now links to its specific sliver.sh page instead
  of the docs root.
- C2 Transport Selection: clarify that the callback endpoint is embedded at
  implant generation; starting a listener only runs the server-side receiver.
- Native MCP: note upstream marks it experimental, not all commands are
  exposed, and HTTP/SSE auth is unsupported; drop the "all commands" claim.

Bumps sliver-c2 to 1.0.2.

Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>

Copilot AI 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.

🔵 Needs a closer look

No concrete defects were identified, but upstream link destinations and native MCP claims remain unverified.

No open issues 🎉

What changed in this PR

Follows up on #189 by refining Sliver’s upstream documentation links and correcting operational guidance.

Changes:

  • Replaces docs-root links with topic-specific pages.
  • Clarifies implant endpoint configuration and native MCP limitations.
  • Bumps sliver-c2 to version 1.0.2.
File Description
capabilities/​sliver-c2/​skills/​sliver-c2/​SKILL.md Updates links and corrects transport and native MCP guidance.
capabilities/​sliver-c2/​capability.yaml Bumps the patch version to 1.0.2.

Review effort: Balanced

@briangreunke
briangreunke merged commit a797047 into main Oct 20, 2026
6 checks passed
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.

2 participants