Fix report-issue workflow bundling - #185
Merged
Priyanshu Agrawal (priyanshu92) merged 1 commit intoJun 10, 2026
Merged
Conversation
Point report-issue wrappers at plugin-local workflow paths and add marketplace-safe symlinks back to the shared workflow so installed plugins receive the workflow content without duplicating it in source. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot started reviewing on behalf of
Priyanshu Agrawal (priyanshu92)
June 10, 2026 09:53
View session
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes the /report-issue skill when plugins are installed from the marketplace by removing references to the repo-root shared/ directory (which is not present in an installed plugin) and instead referencing a workflow path that lives inside the installed plugin root.
Changes:
- Update the shared
report-issueSKILL template to reference${CLAUDE_PLUGIN_ROOT}/skills/report-issue/report-issue-workflow.mdinstead of a repo-relative../../shared/...path. - Update each plugin’s
skills/report-issue/SKILL.mdwrapper to reference the plugin-local bundled workflow path. - Update repository guidance (
AGENTS.md) to document an install-safe cross-plugin shared-skill pattern (wrapper points to plugin-local workflow path; plugin contains a workflow file/symlink).
Show a summary per file
| File | Description |
|---|---|
| shared/skills/report-issue/SKILL.template.md | Updates the template to reference a plugin-local workflow path that works after marketplace install. |
| plugins/power-pages/skills/report-issue/SKILL.md | Switches wrapper to plugin-local workflow path (install-safe). |
| plugins/model-apps/skills/report-issue/SKILL.md | Switches wrapper to plugin-local workflow path (install-safe). |
| plugins/mcp-apps/skills/report-issue/SKILL.md | Switches wrapper to plugin-local workflow path (install-safe). |
| plugins/code-apps/skills/report-issue/SKILL.md | Switches wrapper to plugin-local workflow path (install-safe). |
| plugins/canvas-apps/skills/report-issue/SKILL.md | Switches wrapper to plugin-local workflow path (install-safe), addressing the reported breakage. |
| AGENTS.md | Documents the updated cross-plugin shared-skill bundling approach to prevent future install-path regressions. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 12/12 changed files
- Comments generated: 0
Priyanshu Agrawal (priyanshu92)
enabled auto-merge (squash)
June 10, 2026 09:56
amitjoshi438
approved these changes
Jun 10, 2026
Priyanshu Agrawal (priyanshu92)
deleted the
priyanshu92/fix-report-issue-bundling
branch
June 10, 2026 10:41
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The report-issue skill was broken after marketplace installation because each plugin wrapper pointed at a repo-root
shared/path that is not available from the installed plugin root.This updates the wrappers to load the workflow from the plugin-local
${CLAUDE_PLUGIN_ROOT}/skills/report-issue/report-issue-workflow.mdpath and adds same-marketplace symlinks from each plugin back to the shared workflow source. Marketplace installers dereference these symlinks into the plugin cache, keeping the installed layout portable without duplicating the workflow content in source.Also updates the shared skill template and repository guidance so future cross-plugin shared skills use the same install-safe pattern.
Validation:
node scripts/validate-skill-descriptions.js.node scripts/validate-plugin-names.js.Tested with both Claude Code and GitHub Copilot CLI.
Fixes #180