Conversation
There was a problem hiding this comment.
Pull request overview
Adds a new official Rulesync skill (explain-issue) under .rulesync/skills/ to guide agents in summarizing a GitHub issue’s background and proposed solution, with explicit handling for missing issue input and unspecified solutions.
Changes:
- Introduce
.rulesync/skills/explain-issue/SKILL.mddefining the newexplain-issueskill. - Instruct agents to gather issue body + comments and produce a concise two-part explanation (background + proposed solution).
- Add explicit guidance for cases where the solution is not decided/recorded in the issue discussion.
| Run the following in parallel: | ||
|
|
||
| - Get the issue description and metadata: `gh issue view <issue_number>` | ||
| - Get the issue comments: `gh issue view <issue_number> --comments` |
There was a problem hiding this comment.
gh issue view <issue_number> --comments already outputs the issue body/metadata along with comments, so running both gh issue view <issue_number> and gh issue view <issue_number> --comments is redundant and doubles the GitHub API/CLI work. Consider collapsing this into a single gh issue view invocation (or using --json/--jq if you need specific fields) to keep the skill faster and simpler.
| Run the following in parallel: | |
| - Get the issue description and metadata: `gh issue view <issue_number>` | |
| - Get the issue comments: `gh issue view <issue_number> --comments` | |
| Run the following command: | |
| - Get the issue description, metadata, and comments: `gh issue view <issue_number> --comments` |
| --- | ||
| name: explain-issue | ||
| description: "Explain a GitHub issue: the background problem and the proposed solution" | ||
| targets: | ||
| - "*" | ||
| --- |
There was a problem hiding this comment.
PR description mentions documenting a maintainer preference for Japanese responses in .github/copilot-instructions.md, but that change doesn’t appear to be included in this PR. Either add the corresponding update to .github/copilot-instructions.md or adjust the PR description to match the actual changes.
|
@dyoshikawa Thank you! |
Summary
explain-issueofficial skill under.rulesync/skills/Test plan
pnpm cicheck