Add agentic workflows skill - #2723
Open
VeVarunSharma wants to merge 5 commits into
Open
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
🔒 PR Risk Scan ResultsScanned 3 changed file(s).
|
Contributor
🔍 Vally Lint Results✅ All checks passed
Summary
Full linter output |
Contributor
There was a problem hiding this comment.
Pull request overview
Adds an installable skill for creating and maintaining secure GitHub Agentic Workflows using current upstream guidance.
Changes:
- Adds workflow routing, security, validation, and issue-triage guidance.
- Registers the skill in generated documentation.
Show a summary per file
| File | Description |
|---|---|
skills/agentic-workflows/SKILL.md |
Defines the new skill. |
docs/README.skills.md |
Adds the skill catalog entry. |
Review details
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
- Files reviewed: 2/2 changed files
- Comments generated: 2
- Review effort level: Balanced
Comment on lines
+100
to
+101
| gh aw compile <workflow-id> | ||
| gh aw compile --validate |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8938494a-c76a-4a44-bfd2-a98edff33860
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8938494a-c76a-4a44-bfd2-a98edff33860
Comment on lines
+16
to
+21
| pre-agent-steps: | ||
| - name: Expose Copilot CLI to the sandbox | ||
| run: | | ||
| COPILOT_BIN="$(command -v copilot)" | ||
| test -x "$COPILOT_BIN" | ||
| sudo ln -sfn "$COPILOT_BIN" /usr/local/bin/copilot |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8938494a-c76a-4a44-bfd2-a98edff33860
Contributor
There was a problem hiding this comment.
Review details
Suppressed comments (1)
.github/workflows/pr-duplicate-check.md:12
- This changes the repository's live PR duplicate-check workflow even though the PR is described as adding only an installable skill. The model pin also propagates to the generated threat-detection job, while the adjacent pre-agent step changes runner setup; please either revert these source and lock-file changes or document and justify this separate operational change.
model: claude-sonnet-5
- Files reviewed: 4/4 changed files
- Comments generated: 0 new
- Review effort level: Balanced
Fork pull_request tokens cannot receive copilot-requests: write, so running the agent always fails authentication. Keep the check enabled for same-repository PRs and preserve the Copilot CLI sandbox path repair. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 8938494a-c76a-4a44-bfd2-a98edff33860
Contributor
There was a problem hiding this comment.
Review details
Suppressed comments (2)
Previously missed (1) — in code that hasn't changed since the last review.
.github/workflows/pr-duplicate-check.md:19
- This step fails on the installer's cache-miss path. In that path
install_copilot_cli.shinstalls the binary at/usr/local/bin/copilot, socommand -vreturns that same path andln -sfn /usr/local/bin/copilot /usr/local/bin/copilotexits nonzero under the step's fail-fast shell. Only create the symlink when the resolved binary is elsewhere, then recompile the lock file.
COPILOT_BIN="$(command -v copilot)"
test -x "$COPILOT_BIN"
sudo ln -sfn "$COPILOT_BIN" /usr/local/bin/copilot
.github/workflows/pr-duplicate-check.md:15
- This workaround is emitted only in the main agent job. The generated threat-detection job separately installs Copilot and invokes the same hard-coded
/usr/local/bin/copilotpath, so a tool-cache hit still leaves threat detection without that binary; itscontinue-on-errorthen masks the failure. Apply the shim through a runtime/compiler mechanism that covers every Copilot invocation, and regenerate the lock file.
pre-agent-steps:
- name: Expose Copilot CLI to the sandbox
- Files reviewed: 4/4 changed files
- Comments generated: 1
- Review effort level: Balanced
| @@ -3,7 +3,6 @@ description: 'Checks PRs for potential duplicate agents, instructions, skills, a | |||
| on: | |||
| pull_request: | |||
| types: [opened, synchronize, reopened] | |||
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.
Pull Request Checklist
npm startand verified thatREADME.mdis up to date.mainbranch for this pull request.Description
Adds an installable
agentic-workflowsskill for designing, creating, updating, debugging, and upgrading GitHub Agentic Workflows (gh-aw).The skill dynamically loads the current upstream
create.mddispatcher and task-specific guidance, applies repository-local overlays, enforces read-only agent permissions with safe outputs, and documents a concrete issue-triage pattern for labeling, duplicate detection, clarification, and assignment.Also repairs the PR Duplicate Check exposed by this fork contribution: fork
pull_requesttokens do not receivecopilot-requests: write, so fork runs now skip the advisory AI job instead of failing authentication. Same-repository PRs retain the check, with the cached Copilot CLI exposed at the sandbox path expected by the current lock file.Type of Contribution
Additional Notes
The skill intentionally keeps version-specific
gh-awsyntax dynamic by fetching official upstream guidance instead of bundling a stale local copy. It has no bundled assets.The workflow repair was compiled with the lock file's existing
gh-awv0.85.4 compiler. The original failure was caused by the fork token lackingCopilotRequests: write; a newer runner image also moved the cached Copilot executable out of/usr/local/bin, so a deterministic pre-agent link preserves same-repository execution.By submitting this pull request, I confirm that my contribution abides by the Code of Conduct and will be licensed under the MIT License.