refactor(skills): reinstall from upstreams, drop skills-lock.json#1402
Merged
botantler[bot] merged 2 commits intomainfrom Apr 19, 2026
Merged
refactor(skills): reinstall from upstreams, drop skills-lock.json#1402botantler[bot] merged 2 commits intomainfrom
botantler[bot] merged 2 commits intomainfrom
Conversation
Re-install each agent skill from its original upstream repo via `gh skill install`, so every installed SKILL.md now carries `metadata.github-*` frontmatter pointing at the true origin instead of `devantler-tech/skills`. `gh skill update --all` reads that metadata directly, making the sidecar `skills-lock.json` redundant. - Delete `skills-lock.json`. - Reinstall 11 of 12 skills from upstream via `gh skill install <owner/repo> <path> --agent github-copilot --scope project --dir .agents/skills`. - `siderolabs/docs` is not re-installed: the upstream ships `public/skill.md` (lowercase) which `gh skill install` rejects (it only recognises `SKILL.md`). Tracking upstream for a fix — until then, manage that skill out-of-band or drop it. - Bump `update-skills.yaml` to pin the refreshed reusable workflow and swap `skills-lock` for the new `dir` input. BREAKING CHANGE: `skills-lock.json` is removed. The `siderolabs` skill is temporarily unavailable pending an upstream rename to `SKILL.md`. Refs: devantler-tech/skills#16, devantler-tech/actions#95, devantler-tech/reusable-workflows#207 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refactors how Copilot agent skills are managed in-repo by removing the skills-lock.json sidecar and switching to upstream-installed skills whose SKILL.md frontmatter points at their real source, aligning with the newer “update from metadata + directory” workflow.
Changes:
- Deleted
skills-lock.jsonand updated the scheduled “update skills” workflow to target.agents/skillsdirectly. - Reinstalled/updated multiple skills so their
metadata.github-*frontmatter references upstream repos/refs instead ofdevantler-tech/skills. - Expanded/updated the
gitops-knowledgeskill content with new references (e.g., Terraform bootstrap, gitless image automation) and refreshed guidance.
Reviewed changes
Copilot reviewed 24 out of 24 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
skills-lock.json |
Removed the lockfile previously used to pin skills to a single upstream source. |
.github/workflows/update-skills.yaml |
Updates the reusable workflow pin and switches inputs to use .agents/skills as the managed directory. |
.agents/skills/siderolabs/SKILL.md |
Removes the SideroLabs skill from the repo. |
.agents/skills/refactor/SKILL.md |
Updates skill provenance metadata to point at upstream. |
.agents/skills/gitops-repo-audit/SKILL.md |
Updates skill provenance metadata to point at upstream. |
.agents/skills/gitops-knowledge/SKILL.md |
Updates provenance metadata and refreshes top-level skill guidance. |
.agents/skills/gitops-knowledge/evals/evals.json |
Adds/updates eval coverage for Terraform-based Flux Operator bootstrap guidance. |
.agents/skills/gitops-knowledge/references/terraform-bootstrap.md |
Adds a Terraform bootstrap reference for Flux Operator. |
.agents/skills/gitops-knowledge/references/sources.md |
Updates GitRepository guidance (e.g., sparseCheckout) and expands secret examples. |
.agents/skills/gitops-knowledge/references/resourcesets.md |
Refreshes ResourceSet patterns and expands Permute guidance. |
.agents/skills/gitops-knowledge/references/repo-patterns.md |
Generalizes repo naming in documented patterns. |
.agents/skills/gitops-knowledge/references/kustomization.md |
Updates health-check/wait guidance for Kustomizations. |
.agents/skills/gitops-knowledge/references/helmrelease.md |
Refreshes HelmRelease snippet(s) to match newer guidance. |
.agents/skills/gitops-knowledge/references/gitless-image-automation.md |
Adds a new reference describing “gitless” image automation with ResourceSets. |
.agents/skills/gitops-knowledge/references/flux-operator.md |
Updates Terraform guidance and adds OCI layerSelector details. |
.agents/skills/gitops-knowledge/references/best-practices.md |
Reorders/adjusts best-practice bullets for Flux setups. |
.agents/skills/gitops-cluster-debug/SKILL.md |
Updates skill provenance metadata to point at upstream. |
.agents/skills/github-issues/SKILL.md |
Updates skill provenance metadata to point at upstream. |
.agents/skills/github-actions-docs/SKILL.md |
Updates skill provenance metadata and refreshes routing guidance content. |
.agents/skills/git-commit/SKILL.md |
Updates skill provenance metadata to point at upstream. |
.agents/skills/gh-stack/SKILL.md |
Updates skill provenance metadata to point at upstream. |
.agents/skills/gh-cli/SKILL.md |
Updates skill provenance metadata to point at upstream. |
.agents/skills/find-skills/SKILL.md |
Updates provenance metadata and refreshes “skills discovery/install” guidance. |
.agents/skills/copilot-instructions-blueprint-generator/SKILL.md |
Updates skill provenance metadata to point at upstream. |
The previous PR-head SHA was unreachable from the default branch after reusable-workflows#207 squash-merged, triggering zizmor's "commit with no history in referenced repository" alert. Repoint at the merge commit on `main`. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
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.
Re-install each agent skill from its original upstream repo via
gh skill install, so every installedSKILL.mdnow carriesmetadata.github-*frontmatter pointing at the true origin instead ofdevantler-tech/skills.gh skill update --allreads that metadata directly, making the sidecarskills-lock.jsonredundant.Type of change
What changed
skills-lock.json.gh skill install <owner/repo> <path> --agent github-copilot --scope project --dir .agents/skills..github/workflows/update-skills.yamlto the refreshed reusable workflow and swap theskills-lockinput for the newdirinput.siderolabs/docsships its skill aspublic/skill.md(lowercase), whichgh skill installrejects — it only recognisesSKILL.md. Until the upstream renames the file (orgh skillgains a case-insensitive fallback), thesiderolabsskill has been removed from this repo. Manage it out-of-band if you need it.Breaking change
skills-lock.jsonis removed. Thesiderolabsskill is temporarily unavailable pending an upstream rename toSKILL.md.Related