Skip to content

chore(infra/skills-package-manager): update skills config#77

Merged
SoonIter merged 1 commit into
mainfrom
chore/gy-ss-jx
Jun 15, 2026
Merged

chore(infra/skills-package-manager): update skills config#77
SoonIter merged 1 commit into
mainfrom
chore/gy-ss-jx

Conversation

@SoonIter

Copy link
Copy Markdown
Member

This PR updates the skills package manager config to use the current skill-creator GitHub spec and enables selfSkill for this repository's local skills setup.

Copilot AI review requested due to automatic review settings June 15, 2026 06:12

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

Updates the repository’s skills.json configuration to align with the current skill-creator GitHub spec and enables selfSkill for local skills usage in this repo.

Changes:

  • Updated the skill-creator skill source to the newer GitHub spec/reference format and pinned commit.
  • Enabled "selfSkill": true in the skills package manager config.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@coderabbitai

coderabbitai Bot commented Jun 15, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Walkthrough

The skill-creator entry in the skills map within skills.json is updated to point to a new GitHub ref. The target string format is also adjusted by removing the .git suffix. All other entries (rstack-skill-evaluator, pr-creator) and the surrounding structure, including the selfSkill flag, remain unchanged.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description check ✅ Passed The description clearly relates to the changeset, explaining that it updates the skills package manager config and mentions the skill-creator GitHub spec and selfSkill changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title accurately describes the main change: updating skills configuration in the infra/skills-package-manager area, which aligns with the PR objectives of modifying skill-creator GitHub spec and enabling selfSkill.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/gy-ss-jx

Comment @coderabbitai help to get the list of available commands and usage tips.

@coderabbitai coderabbitai Bot 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.

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@skills.json`:
- Line 6: The skill-creator dependency path reference in skills.json is
incorrect. The path /skills/skill-creator does not exist in the
anthropics/skills repository at commit 57546260929473d4e0d1c1bb75297be2fdfa1949.
Verify the actual directory structure in the anthropics/skills repository at
that commit to determine the correct path where skill-creator is located, then
update the path value in the skill-creator entry to reference the correct
location.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: ed695230-3ba7-4854-89d9-ea8b953975ba

📥 Commits

Reviewing files that changed from the base of the PR and between 1d713ae and 668953e.

📒 Files selected for processing (1)
  • skills.json

Comment thread skills.json
@SoonIter SoonIter changed the title chore: update skills config chore(infra/skills-package-manager): update skills config Jun 15, 2026
@SoonIter SoonIter enabled auto-merge (squash) June 15, 2026 07:05
@SoonIter SoonIter disabled auto-merge June 15, 2026 07:05
@SoonIter SoonIter enabled auto-merge (squash) June 15, 2026 07:05
@SoonIter SoonIter requested a review from Timeless0911 June 15, 2026 07:05
@SoonIter SoonIter merged commit 4aaf098 into main Jun 15, 2026
5 checks passed
@SoonIter SoonIter deleted the chore/gy-ss-jx branch June 15, 2026 07:08
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.

3 participants