fix: correct workflow paths and directory references#1891
Closed
Sertxito wants to merge 1 commit into
Closed
Conversation
- learning-hub-updater.md: Fix path from website/learning-hub to website/src/content/docs/learning-hub, wrap URLs in <>, remove trailing colons from section headings - resource-staleness-report.md: Remove non-existent prompts/ directory, add workflows/ and plugins/, update guidelines and examples - duplicate-resource-detector.md: Add missing plugins/ directory and reading instructions, update frontmatter description - pr-duplicate-check.md: Add hooks and plugins to all directory lists, update frontmatter description and noop messages, expand template examples All workflows now consistently scan: agents/, instructions/, skills/, hooks/, workflows/, and plugins/
Contributor
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
Updates the repository’s workflow instruction documents to expand coverage beyond agents/instructions/skills, adding hooks, workflows, and plugins (and aligning the Learning Hub updater with the new website content path).
Changes:
- Expanded “resource” definitions in staleness/duplicate workflows to include hooks, workflows, and plugins.
- Updated PR duplicate-check guidance to detect duplicates across the expanded resource set.
- Corrected Learning Hub content path references and normalized link formatting.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.
| File | Description |
|---|---|
| .github/workflows/resource-staleness-report.md | Adds hooks/workflows/plugins to the staleness audit scope and guidance |
| .github/workflows/pr-duplicate-check.md | Extends PR duplicate detection instructions to include hooks and plugins |
| .github/workflows/learning-hub-updater.md | Updates Learning Hub directory path and link formatting |
| .github/workflows/duplicate-resource-detector.md | Expands weekly duplicate scan scope to include hooks/workflows/plugins |
Comment on lines
34
to
+42
| - `agents/` (`.agent.md` files) | ||
| - `instructions/` (`.instructions.md` files) | ||
| - `skills/` (folders — the SKILL.md inside each folder is the resource) | ||
| - `hooks/` (folders — the README.md inside each folder is the resource) | ||
| - `workflows/` (`.md` files) | ||
| - `plugins/` (folders — the `.github/plugin/plugin.json` inside each folder is the resource) | ||
|
|
||
| If **no files** from these directories were modified, call `noop` with the message: | ||
| "No agent, instruction, skill, or workflow files were changed in this PR — no duplicate check needed." | ||
| "No agent, instruction, skill, hook, workflow, or plugin files were changed in this PR — no duplicate check needed." |
Comment on lines
25
to
+30
| - `agents/` (`.agent.md` files) | ||
| - `prompts/` (`.prompt.md` files) | ||
| - `instructions/` (`.instructions.md` files) | ||
| - `hooks/` (folders — check the folder's files) | ||
| - `skills/` (folders — check the folder's files) | ||
| - `hooks/` (folders — check the folder's files) | ||
| - `workflows/` (`.md` files) | ||
| - `plugins/` (folders — check `.github/plugin/plugin.json` inside each) |
| ``` | ||
|
|
||
| For hook and skill folders, check all files within the folder and use the **most recent** major change date across any file in that folder. | ||
| For hook, skill, and plugin folders, check all files within the folder and use the **most recent** major change date across any file in that folder. |
| @@ -1,5 +1,5 @@ | |||
| --- | |||
| description: Weekly scan of agents, instructions, and skills to identify potential duplicate resources and report them for review | |||
| description: Weekly scan of agents, instructions, skills, hooks, and workflows to identify potential duplicate resources and report them for review | |||
Comment on lines
28
to
+33
| - `agents/` (`.agent.md` files) | ||
| - `instructions/` (`.instructions.md` files) | ||
| - `skills/` (folders — check `SKILL.md` inside each) | ||
| - `hooks/` (folders — check `README.md` inside each) | ||
| - `workflows/` (`.md` files) | ||
| - `plugins/` (folders — check `.github/plugin/plugin.json` inside each) |
| - `skills/` (folders — the SKILL.md inside each folder is the resource) | ||
| - `hooks/` (folders — the README.md inside each folder is the resource) | ||
| - `workflows/` (`.md` files) | ||
| - `plugins/` (folders — the `.github/plugin/plugin.json` inside each folder is the resource) |
| - `skills/` (folders — read `SKILL.md` inside each) | ||
| - `hooks/` (folders — read `README.md` inside each) | ||
| - `workflows/` (`.md` files) | ||
| - `plugins/` (folders — read `.github/plugin/plugin.json` inside each) |
| - Limit the report to the top **5** most likely duplicate groups to keep feedback actionable. | ||
| - For skills, report by folder name (e.g., `skills/my-skill/`) using the description from `SKILL.md`. | ||
| - For hooks, report by folder name (e.g., `hooks/my-hook/`) using the description from `README.md`. | ||
| - For plugins, report by folder name (e.g., `plugins/my-plugin/`) using the description from `.github/plugin/plugin.json`. |
Contributor
Author
|
Context for reviewers: this PR updates repository workflow instructions only (.github/workflows/*.md) and does not add or onboard an external plugin. Plugin onboarding/intake follows the external-plugin issue workflow; this change is a normal repo contribution targeting staged. |
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.
Changes
Scope
Notes