Skip to content

Add SKILL.md and copilot-instructions.md for core CAPs#1895

Merged
anupsdf merged 2 commits intostellar:masterfrom
anupsdf:ai
Mar 11, 2026
Merged

Add SKILL.md and copilot-instructions.md for core CAPs#1895
anupsdf merged 2 commits intostellar:masterfrom
anupsdf:ai

Conversation

@anupsdf
Copy link
Contributor

@anupsdf anupsdf commented Mar 10, 2026

Add SKILL.md and copilot-instructions.md for core CAPs.

Copilot AI review requested due to automatic review settings March 10, 2026 05:06
@anupsdf anupsdf enabled auto-merge (squash) March 10, 2026 05:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds contributor/assistant guidance files to help keep Core Advancement Proposal (CAP) edits consistent and reviewable across the repository.

Changes:

  • Add .github/copilot-instructions.md with repository-wide editing guidance focused on CAP work under core/.
  • Add .claude/skills/core/SKILL.md defining a “core” skill/workflow for drafting/editing CAPs and synchronizing CAP status with core/README.md.

Reviewed changes

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

File Description
.github/copilot-instructions.md Establishes CAP-focused editing and scope-discipline rules plus formatting guidance (incl. ./bin/prettier.sh).
.claude/skills/core/SKILL.md Introduces a structured workflow for CAP authoring and CAP status synchronization against core/README.md.

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

Copy link
Contributor

@dmkozh dmkozh left a comment

Choose a reason for hiding this comment

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

Do we have any evidence/motivation to consider this skill useful? It seems quite trivial to me, I'd expect any agent to be able to easily do mass edits on CAPs without messing up.

@anupsdf
Copy link
Contributor Author

anupsdf commented Mar 11, 2026

Do we have any evidence/motivation to consider this skill useful? It seems quite trivial to me, I'd expect any agent to be able to easily do mass edits on CAPs without messing up.

Most likely any agent would be able to do edits to move the CAPs to different section when status changes. However, I added this to make the instructions be more specific to minimize AI messing things up (like in case there is a regression in a new AI model).

Copy link
Contributor

@dmkozh dmkozh left a comment

Choose a reason for hiding this comment

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

Ok, I guess we could try this out, but I'm generally really skeptical of pre-emptively adding skills or copilot instructions without solving a specific issue.

@anupsdf anupsdf disabled auto-merge March 11, 2026 19:15
@anupsdf
Copy link
Contributor Author

anupsdf commented Mar 11, 2026

Ok, I guess we could try this out, but I'm generally really skeptical of pre-emptively adding skills or copilot instructions without solving a specific issue.

Yeah, I think this is not that important but something we could try out. I have seen formatting issues before and even I have forgotten to keep the README and individual CAPs in sync, so I thought we could leverage AI with specific instructions.

@anupsdf anupsdf merged commit c994634 into stellar:master Mar 11, 2026
5 checks passed
@leighmcculloch
Copy link
Member

leighmcculloch commented Mar 12, 2026

I guess we could try this out, but I'm generally really skeptical of pre-emptively adding skills or copilot instructions without solving a specific issue.

+1 plus, we should be experimenting with skills locally with ai tooling, because iterating is important, and then pushing to repos when we find something that works.

Is this a skill we have used or is this written but never used?

@anupsdf
Copy link
Contributor Author

anupsdf commented Mar 12, 2026

I guess we could try this out, but I'm generally really skeptical of pre-emptively adding skills or copilot instructions without solving a specific issue.

+1 plus, we should be experimenting with skills locally with ai tooling, because iterating is important, and then pushing to repos when we find something that works.

Is this a skill we have used or is this written but never used?

Oh. I have actually tested the SKILL before pushing. I used it for generating the changes in #1894 and then separated the SKILL.md and copilot-instructions.md into a separate GH branch so I could have a separate PR. So, its not untested. Sorry if I didn't clarify that earlier. cc. @dmkozh , @leighmcculloch

@leighmcculloch
Copy link
Member

Thanks for clarifying! Apologies for incorrectly reading between the lines.

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.

4 participants