Skip to content

Add shared report-issue skill for all plugins#63

Open
priyanshu92 wants to merge 4 commits intomainfrom
users/priyanshuag/add-report-issue-shared-skill
Open

Add shared report-issue skill for all plugins#63
priyanshu92 wants to merge 4 commits intomainfrom
users/priyanshuag/add-report-issue-shared-skill

Conversation

@priyanshu92
Copy link
Copy Markdown
Collaborator

@priyanshu92 priyanshu92 commented Mar 26, 2026

  • Add report-issue shared skill with workflow, template, and config
  • Add thin SKILL.md wrappers to all 4 plugins (canvas-apps, code-apps, model-apps, power-pages) with plugin-specific descriptions
  • Add sync-shared-skills.js script to auto-generate wrappers for new plugins
  • Add CI workflow using GitHub App to auto-sync on PRs
  • Support targeted plugin config (wildcard or explicit plugin list)
  • Template supports {{PLUGIN_NAME}} placeholder from plugin.json
  • Update AGENTS.md with shared skills pattern documentation

- Add report-issue shared skill with workflow, template, and config
- Add thin SKILL.md wrappers to all 3 plugins with plugin-specific descriptions
- Add sync-shared-skills.js script to auto-generate wrappers for new plugins
- Add CI workflow using GitHub App to auto-sync on PRs
- Support targeted plugin config (wildcard or explicit plugin list)
- Template supports {{PLUGIN_NAME}} placeholder from plugin.json
- Update AGENTS.md with shared skills pattern documentation

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@priyanshu92 priyanshu92 requested a review from a team as a code owner March 26, 2026 07:39
Copilot AI review requested due to automatic review settings March 26, 2026 07:39
@priyanshu92 priyanshu92 enabled auto-merge (squash) March 26, 2026 07:39
Copy link
Copy Markdown
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

Introduces a cross-plugin “/report-issue” shared skill so users can file GitHub issues with a consistent, centrally maintained workflow, while keeping each plugin’s skill catalog discoverable via thin per-plugin wrappers.

Changes:

  • Add a shared report-issue skill (workflow + template + config) under shared/skills/, and add per-plugin wrapper SKILL.md files.
  • Add scripts/sync-shared-skills.js to generate per-plugin wrappers from shared templates.
  • Add a CI workflow to auto-run the sync and push generated wrappers back to PR branches; document the pattern in AGENTS.md.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
shared/skills/report-issue/report-issue-workflow.md Defines the shared multi-phase workflow for collecting details, previewing, and creating GitHub issues.
shared/skills/report-issue/config.json Declares plugin targeting for the shared skill (currently all plugins).
shared/skills/report-issue/SKILL.template.md Template wrapper to be instantiated per plugin with {{PLUGIN_NAME}}.
scripts/sync-shared-skills.js Repo utility to generate per-plugin SKILL.md wrappers from shared templates/config.
plugins/power-pages/skills/report-issue/SKILL.md Generated wrapper exposing the shared workflow in the power-pages plugin.
plugins/model-apps/skills/report-issue/SKILL.md Generated wrapper exposing the shared workflow in the model-apps plugin.
plugins/code-apps/skills/report-issue/SKILL.md Generated wrapper exposing the shared workflow in the code-apps plugin.
AGENTS.md Documents the new shared-skills pattern and sync mechanism.
.github/workflows/sync-shared-skills.yml CI job to run the sync script and push generated wrappers back to PR branches.

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

Comment thread .github/workflows/sync-shared-skills.yml
Comment thread shared/skills/report-issue/report-issue-workflow.md Outdated
Comment thread shared/skills/report-issue/report-issue-workflow.md
Comment thread shared/skills/report-issue/report-issue-workflow.md
Comment thread shared/skills/report-issue/report-issue-workflow.md
Comment thread shared/skills/report-issue/report-issue-workflow.md
Comment thread scripts/sync-shared-skills.js
- Plugin name is now read as-is from plugin.json
- Works for any new plugin without edits

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@priyanshu92 priyanshu92 disabled auto-merge March 26, 2026 14:24
- Add report-issue skill wrapper for canvas-apps plugin
- Add canvas-apps to bug report issue template dropdown
- Add canvas-apps to feature request issue template dropdown

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings April 1, 2026 05:33
Copy link
Copy Markdown
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

Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.


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

Comment thread shared/skills/report-issue/report-issue-workflow.md
Comment thread scripts/sync-shared-skills.js
Comment thread .github/workflows/sync-shared-skills.yml
Comment thread .github/workflows/sync-shared-skills.yml
Comment thread AGENTS.md
Comment thread plugins/canvas-apps/skills/report-issue/SKILL.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants