Skip to content

Remove Repo-Relative Copilot Skill Link - #853

Merged
ptr727 merged 2 commits into
developfrom
fix/issue-852-copilot-skill-link
Aug 19, 2026
Merged

Remove Repo-Relative Copilot Skill Link#853
ptr727 merged 2 commits into
developfrom
fix/issue-852-copilot-skill-link

Conversation

@ptr727

Copy link
Copy Markdown
Owner

Summary

  • Name carried-instruction-file-guard without a repo-relative Markdown link.
  • Regenerate the GitHub Copilot and Claude skill distributions.

This addresses #852. The issue remains open until the change reaches main.

Validation

  • Complete local verification gate from OPERATIONS.md
  • python3 scripts/build_dist.py --check
  • 730 Python tests plus audit and safety self-tests
  • ruff, mypy, repository, prose, JSON, and spec validation
  • editorconfig-checker, ShellCheck, and PSScriptAnalyzer

CopilotAI lite review requested due to automatic review settings August 19, 2026 19:58

CopilotAI 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

Removes a repo-relative Markdown link to the sibling carried-instruction-file-guard Skill from the copilot-instructions-keeper Skill text, then regenerates the distributed Skill copies (Copilot + Claude) and updates the distribution source digest so installed Skills do not rely on in-repo relative paths.

Changes:

  • Replace the repo-relative Markdown link to carried-instruction-file-guard with a plain inline code reference.
  • Regenerate the Copilot and Claude Skill distributions to carry the updated text.
  • Update the Claude distribution .source-digest to reflect the new source state.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 3 comments.

FileDescription
.github/skills/copilot-instructions-keeper/SKILL.mdRemoves repo-relative sibling-skill link in the Copilot skill distribution copy.
.agents/skills/copilot-instructions-keeper/SKILL.mdRemoves repo-relative sibling-skill link in the canonical agent skill copy.
.claude-plugin/fleet-skills/skills/copilot-instructions-keeper/SKILL.mdRemoves repo-relative sibling-skill link in the Claude distribution copy.
.claude-plugin/fleet-skills/.source-digestUpdates the source digest after regenerating the distribution.

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread.github/skills/copilot-instructions-keeper/SKILL.md Outdated
Comment thread.agents/skills/copilot-instructions-keeper/SKILL.md Outdated
Comment thread.claude-plugin/fleet-skills/skills/copilot-instructions-keeper/SKILL.md Outdated
CopilotAI review requested due to automatic review settings August 19, 2026 20:02

CopilotAI 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 4 out of 4 changed files in this pull request and generated no new comments.

@ptr727
ptr727 merged commit 5c09d48 into developAug 19, 2026
8 checks passed
@ptr727
ptr727 deleted the fix/issue-852-copilot-skill-link branch August 19, 2026 20:05
@ptr727ptr727 mentioned this pull request Aug 19, 2026
ptr727 added a commit that referenced this pull request Aug 19, 2026
## Summary
- Promote the reviewed Copilot skill link fix from `develop` to `main`.
- Preserve the generated GitHub Copilot and Claude distributions.
Fixes#852.
## Included Pull Request
- #853
## Validation
- PR #853 completed 7/7 required checks.
- Copilot reviewed the full current-head diff with no open findings.
- The complete local verification gate passed on the final feature
commit.
Sign up for freeto 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

@ptr727