Skip to content

feat(skills): separate public and contributor workflows - #2899

Open
johnnygreco wants to merge 4 commits into
mainfrom
feat/2736-publish-user-skills/johnnygreco
Open

feat(skills): separate public and contributor workflows#2899
johnnygreco wants to merge 4 commits into
mainfrom
feat/2736-publish-user-skills/johnnygreco

Conversation

@johnnygreco

Copy link
Copy Markdown
Collaborator

🏗️ build-from-issue-agent

Summary

Publish the four user-facing OpenShell skills as a portable top-level collection while keeping repository contributor workflows internal. Public skills now use installed CLI help and published documentation as their sources of truth, and repository guidance clearly distinguishes using OpenShell from developing it.

Related Issue

Closes #2736

Changes

  • Moved openshell-cli, generate-sandbox-policy, debug-inference, and debug-openshell-cluster to skills/.
  • Marked all 18 contributor workflows under .agents/skills/ with internal discovery metadata.
  • Removed the duplicated CLI reference and replaced source-checkout dependencies with CLI self-discovery and published documentation links.
  • Extended sync-agent-infra to validate audience placement, metadata, unique names, references, portability, and public discovery.
  • Updated README, contributor guidance, published quickstart and docs guidance, issue contact links, and cross-skill canonical paths.

Deviations from Plan

None — implemented as planned.

Testing

  • npx -y skills add <disposable-local-copy> --list finds exactly the four public skills.
  • Inventory validation confirms 4 public skills, 18 internal contributor skills, and no duplicate names or source-checkout dependencies in public skills.
  • mise run pre-commit passes.
  • mise run docs passes with zero errors.
  • mise run ci passes. An initial Go SDK test returned a transient EOF; mise run go:ci and the full CI rerun both passed.
  • Unit tests added/updated — N/A; this change contains Markdown, skill frontmatter, and path updates only.
  • E2E tests added/updated — N/A; gateway and sandbox runtime behavior are unchanged.

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Documentation updated:

  • README.md: public versus contributor skills and the npx skills add NVIDIA/OpenShell installation command.
  • AGENTS.md and CONTRIBUTING.md: canonical locations, inventories, and maintenance guidance.
  • docs/CONTRIBUTING.mdx and docs/get-started/quickstart.mdx: contributor workflow distinction and public skill installation.

@copy-pr-bot

copy-pr-bot Bot commented Aug 23, 2026

Copy link
Copy Markdown

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@github-actions

Copy link
Copy Markdown

johntmyers
johntmyers previously approved these changes Aug 26, 2026
@johntmyers

Copy link
Copy Markdown
Collaborator

lgtm @johnnygreco but you'll need to rebase and resolve conflicts then can re-stamp

Closes #2736

Publish the four user-facing OpenShell skills from the top-level skills directory, mark contributor workflows internal, and update portability guidance, validation, and documentation.

Signed-off-by: Johnny Greco <jogreco@nvidia.com>
Signed-off-by: Johnny Greco <jogreco@nvidia.com>
Signed-off-by: Johnny Greco <jogreco@nvidia.com>
Signed-off-by: Johnny Greco <jogreco@nvidia.com>
@johnnygreco
johnnygreco force-pushed the feat/2736-publish-user-skills/johnnygreco branch from 44b4c7a to 3c8523c Compare August 27, 2026 18:46
@johnnygreco

Copy link
Copy Markdown
Collaborator Author

/ok to test 3c8523c

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.

feat: publish user-facing OpenShell skills separately

2 participants