Skip to content

[docs] sync SKILL.md with workload scan and KWOK/EKS distributions#4585

Merged
ksail-bot[bot] merged 4 commits into
mainfrom
docs/sync-skill-workload-scan-7d94ecdc908f85ff
May 5, 2026
Merged

[docs] sync SKILL.md with workload scan and KWOK/EKS distributions#4585
ksail-bot[bot] merged 4 commits into
mainfrom
docs/sync-skill-workload-scan-7d94ecdc908f85ff

Conversation

@github-actions

@github-actions github-actions Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor

Summary

Two gaps in copilot-plugin/skills/ksail/SKILL.md found during a Doc Sync audit after the workload scan documentation was added in #4577.

Changes

  • Added scan to the ksail workload ... command list (was missing after the ksail workload scan command was introduced)
  • Added KWOK and EKS to the supported distributions list in the typical flows section (only Vanilla, K3s, Talos, VCluster were listed; the README and docs site already include KWOK and EKS)

Root file audit results

Check Result
README.md ≤ 250 lines ✅ (119 lines)
README scope (no duplicated content)
Distribution × Provider matrix parity (README vs index.mdx)
vsce/README.md concise marketplace description
SKILL.md workload command list ⚠️ Fixed: missing scan
SKILL.md distributions list ⚠️ Fixed: missing KWOK, EKS

Generated by Daily Docs · ● 1.2M ·

Add `scan` to the ksail workload command list and add KWOK and EKS
to the supported distributions list, reflecting the current CLI.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings May 4, 2026 18:52
@github-actions github-actions Bot removed the request for review from Copilot May 4, 2026 18:52
@github-project-automation github-project-automation Bot moved this to 🫴 Ready in 🌊 Project Board May 4, 2026
@devantler devantler marked this pull request as ready for review May 4, 2026 18:53
Copilot AI review requested due to automatic review settings May 4, 2026 18:53

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the KSail Copilot skill documentation to better match the current CLI/docs surface, specifically by syncing the workload command list and supported distributions mentioned in SKILL.md.

Changes:

  • Added scan to the documented ksail workload command list.
  • Added KWOK and EKS to the distributions listed in the typical usage section.
  • Adjusted the distribution/provider note in the skill docs to mention AWS for EKS.

Comment thread copilot-plugin/skills/ksail/SKILL.md Outdated
@ksail-bot ksail-bot Bot enabled auto-merge May 4, 2026 19:46
@github-actions

github-actions Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor Author

MegaLinter analysis: Success

✅ Linters with no issues

actionlint, git_diff, hadolint, jscpd, jsonlint, lychee, markdown-table-formatter, markdownlint, prettier, prettier, stylelint, syft, trivy-sbom, trufflehog, v8r, v8r, yamllint

See detailed reports in MegaLinter artifacts

MegaLinter is graciously provided by OX Security
Show us your support by starring ⭐ the repository

Copilot AI review requested due to automatic review settings May 4, 2026 22:41

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

Comment thread copilot-plugin/skills/ksail/SKILL.md Outdated
Comment thread copilot-plugin/skills/ksail/SKILL.md Outdated
The provider statement incorrectly implied Docker is the default for all
distributions. EKS only supports the AWS provider and ksail cluster create
is not yet functional for EKS. Update the sentence to:
- Say 'Provider defaults to Docker for local distributions' instead of 'by default'
- State that EKS requires --provider AWS
- Caveat that cluster create is not yet functional for EKS (init only)
- Link to EKS distribution docs for details

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@ksail-bot ksail-bot Bot added this pull request to the merge queue May 5, 2026
Merged via the queue into main with commit f9b35df May 5, 2026
57 checks passed
@ksail-bot ksail-bot Bot deleted the docs/sync-skill-workload-scan-7d94ecdc908f85ff branch May 5, 2026 00:17
@github-project-automation github-project-automation Bot moved this from 🫴 Ready to ✅ Done in 🌊 Project Board May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants