Skip to content

Commit 925d647

Browse files
Copilotpelikhan
andauthored
Align workflow designer skill mappings with current AW reference surface (#38033)
* Initial plan * Expand workflow designer drift mappings Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
1 parent acf92a0 commit 925d647

2 files changed

Lines changed: 78 additions & 16 deletions

File tree

.github/skills/agentic-workflow-designer/SKILL.md

Lines changed: 39 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,9 @@ Present a structured summary and ask for approval before generation.
117117
| "every Monday", "weekly" | fuzzy schedule shorthand `on: schedule: weekly` (compiler expands to cron) |
118118
| "when I say /review" | `on: slash_command:` with `name: review` (or requested command) |
119119
| "when an issue is labeled bug" | `on: issues:` with `types: [labeled]` and label filter guidance |
120+
| "run when label ai-review is added" | `on: label_command:` with `name`/`names`, optional event scoping, and label-as-command semantics |
121+
| "run on PRs from forks" | `on: pull_request:` plus explicit `forks:` allowlist and fork security guardrails |
122+
| "sometimes automatic, sometimes manual" | semi-active pattern: combine `schedule`/event triggers with `workflow_dispatch` |
120123
| "manually", "on demand" | `on: workflow_dispatch:` |
121124
| "when a deployment fails" | `on: deployment_status:` |
122125
| "when another workflow finishes" | `on: workflow_run:` |
@@ -127,11 +130,26 @@ Present a structured summary and ask for approval before generation.
127130
|---|---|
128131
| "post a comment" | `add-comment` |
129132
| "create an issue" | `create-issue` |
130-
| "open a PR", "submit changes" | `create-pull-request` |
131-
| "add labels" | `add-labels` |
132-
| "remove labels" | `remove-labels` |
133+
| "update issue title/body" | `update-issue` |
133134
| "close the issue" | `close-issue` |
134-
| "assign someone" | `assign-to-user` |
135+
| "assign someone", "remove assignment" | `assign-to-user`, `unassign-from-user` |
136+
| "set issue type/field/milestone" | `set-issue-type`, `set-issue-field`, `assign-milestone` |
137+
| "open a PR", "submit changes" | `create-pull-request` |
138+
| "update PR description/title" | `update-pull-request` |
139+
| "close the PR", "merge the PR" | `close-pull-request`, `merge-pull-request` |
140+
| "mark PR ready", "sync PR branch" | `mark-pull-request-as-ready-for-review`, `update-branch` |
141+
| "commit a fix to the PR branch" | `push-to-pull-request-branch` |
142+
| "approve / request changes" | `submit-pull-request-review` |
143+
| "inline review comment", "reply to review thread" | `create-pull-request-review-comment`, `reply-to-pull-request-review-comment`, `resolve-pull-request-review-thread` |
144+
| "start or edit discussion", "close discussion" | `create-discussion`, `update-discussion`, `close-discussion` |
145+
| "request reviewer", "hide comment" | `add-reviewer`, `hide-comment` |
146+
| "create/update project", "project status update" | `create-project`, `update-project`, `create-project-status-update` |
147+
| "update release", "upload release asset" | `update-release`, `upload-asset` |
148+
| "create/auto-fix code scan alert" | `create-code-scanning-alert`, `autofix-code-scanning-alert` |
149+
| "start an agent session", "assign to an agent" | `create-agent-session`, `assign-to-agent` |
150+
| "store persistent memory comment" | `comment-memory` |
151+
| "link a sub-issue" | `link-sub-issue` |
152+
| "add labels", "remove labels" | `add-labels`, `remove-labels` |
135153
| "nothing visible", "just analyze" | no safe outputs required |
136154

137155
### Network Mapping
@@ -149,11 +167,24 @@ Present a structured summary and ask for approval before generation.
149167
| User says... | Maps to |
150168
|---|---|
151169
| "read GitHub issues/PRs/workflows" | `tools.github` with `mode: gh-proxy` and minimal `toolsets` |
170+
| "use full MCP server/tool definitions" | `tools.github` with `mode: local` |
171+
| "use other MCP servers but keep token cost down" | `tools.cli-proxy: true` (hybrid CLI-proxy mode) |
152172
| "edit files" | `edit` tool (default unless restricted) |
153173
| "run commands/tests" | `bash` tool (default unless restricted) |
154174
| "browse web pages/docs" | `web-fetch` and/or `web-search` |
155175
| "test UI flows" | `playwright` |
156176

177+
### Pattern Heuristics
178+
179+
| User says... | Recommended named pattern |
180+
|---|---|
181+
| "triage issues automatically" | `IssueOps` |
182+
| "run on /commands with human approval loops" | `ChatOps` |
183+
| "run every weekday and keep improving" | `DailyOps` |
184+
| "monitor workflow failures and trends" | `MonitorOps` |
185+
| "process a big backlog in chunks" | `BatchOps` |
186+
| "run manually with input parameters" | `DispatchOps` |
187+
157188
### Integration Auth Mapping
158189

159190
When the user names a third-party service or MCP server:
@@ -289,8 +320,8 @@ Before final output, run this internal self-check:
289320
## References (load only when needed)
290321

291322
In-repo references:
292-
- `.github/aw/syntax.md`
293-
- `.github/aw/safe-outputs.md`
323+
- `.github/aw/syntax.md` (index → `.github/aw/syntax-core.md`, `.github/aw/syntax-agentic.md`, `.github/aw/syntax-tools-imports.md`)
324+
- `.github/aw/safe-outputs.md` (index → `.github/aw/safe-outputs-content.md`, `.github/aw/safe-outputs-management.md`, `.github/aw/safe-outputs-automation.md`, `.github/aw/safe-outputs-runtime.md`)
294325
- `.github/aw/network.md`
295326
- `.github/aw/patterns.md`
296327
- `.github/aw/subagents.md`
@@ -299,8 +330,8 @@ In-repo references:
299330
- `.github/aw/create-agentic-workflow.md`
300331

301332
Portable HTTPS references:
302-
- `https://github.com/github/gh-aw/blob/main/.github/aw/syntax.md`
303-
- `https://github.com/github/gh-aw/blob/main/.github/aw/safe-outputs.md`
333+
- `https://github.com/github/gh-aw/blob/main/.github/aw/syntax.md` (index → `.../syntax-core.md`, `.../syntax-agentic.md`, `.../syntax-tools-imports.md`)
334+
- `https://github.com/github/gh-aw/blob/main/.github/aw/safe-outputs.md` (index → `.../safe-outputs-content.md`, `.../safe-outputs-management.md`, `.../safe-outputs-automation.md`, `.../safe-outputs-runtime.md`)
304335
- `https://github.com/github/gh-aw/blob/main/.github/aw/network.md`
305336
- `https://github.com/github/gh-aw/blob/main/.github/aw/patterns.md`
306337
- `https://github.com/github/gh-aw/blob/main/.github/aw/triggers.md`

pkg/cli/data/agentic_workflow_designer_skill.md

Lines changed: 39 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,9 @@ Present a structured summary and ask for approval before generation.
117117
| "every Monday", "weekly" | fuzzy schedule shorthand `on: schedule: weekly` (compiler expands to cron) |
118118
| "when I say /review" | `on: slash_command:` with `name: review` (or requested command) |
119119
| "when an issue is labeled bug" | `on: issues:` with `types: [labeled]` and label filter guidance |
120+
| "run when label ai-review is added" | `on: label_command:` with `name`/`names`, optional event scoping, and label-as-command semantics |
121+
| "run on PRs from forks" | `on: pull_request:` plus explicit `forks:` allowlist and fork security guardrails |
122+
| "sometimes automatic, sometimes manual" | semi-active pattern: combine `schedule`/event triggers with `workflow_dispatch` |
120123
| "manually", "on demand" | `on: workflow_dispatch:` |
121124
| "when a deployment fails" | `on: deployment_status:` |
122125
| "when another workflow finishes" | `on: workflow_run:` |
@@ -127,11 +130,26 @@ Present a structured summary and ask for approval before generation.
127130
|---|---|
128131
| "post a comment" | `add-comment` |
129132
| "create an issue" | `create-issue` |
130-
| "open a PR", "submit changes" | `create-pull-request` |
131-
| "add labels" | `add-labels` |
132-
| "remove labels" | `remove-labels` |
133+
| "update issue title/body" | `update-issue` |
133134
| "close the issue" | `close-issue` |
134-
| "assign someone" | `assign-to-user` |
135+
| "assign someone", "remove assignment" | `assign-to-user`, `unassign-from-user` |
136+
| "set issue type/field/milestone" | `set-issue-type`, `set-issue-field`, `assign-milestone` |
137+
| "open a PR", "submit changes" | `create-pull-request` |
138+
| "update PR description/title" | `update-pull-request` |
139+
| "close the PR", "merge the PR" | `close-pull-request`, `merge-pull-request` |
140+
| "mark PR ready", "sync PR branch" | `mark-pull-request-as-ready-for-review`, `update-branch` |
141+
| "commit a fix to the PR branch" | `push-to-pull-request-branch` |
142+
| "approve / request changes" | `submit-pull-request-review` |
143+
| "inline review comment", "reply to review thread" | `create-pull-request-review-comment`, `reply-to-pull-request-review-comment`, `resolve-pull-request-review-thread` |
144+
| "start or edit discussion", "close discussion" | `create-discussion`, `update-discussion`, `close-discussion` |
145+
| "request reviewer", "hide comment" | `add-reviewer`, `hide-comment` |
146+
| "create/update project", "project status update" | `create-project`, `update-project`, `create-project-status-update` |
147+
| "update release", "upload release asset" | `update-release`, `upload-asset` |
148+
| "create/auto-fix code scan alert" | `create-code-scanning-alert`, `autofix-code-scanning-alert` |
149+
| "start an agent session", "assign to an agent" | `create-agent-session`, `assign-to-agent` |
150+
| "store persistent memory comment" | `comment-memory` |
151+
| "link a sub-issue" | `link-sub-issue` |
152+
| "add labels", "remove labels" | `add-labels`, `remove-labels` |
135153
| "nothing visible", "just analyze" | no safe outputs required |
136154

137155
### Network Mapping
@@ -149,11 +167,24 @@ Present a structured summary and ask for approval before generation.
149167
| User says... | Maps to |
150168
|---|---|
151169
| "read GitHub issues/PRs/workflows" | `tools.github` with `mode: gh-proxy` and minimal `toolsets` |
170+
| "use full MCP server/tool definitions" | `tools.github` with `mode: local` |
171+
| "use other MCP servers but keep token cost down" | `tools.cli-proxy: true` (hybrid CLI-proxy mode) |
152172
| "edit files" | `edit` tool (default unless restricted) |
153173
| "run commands/tests" | `bash` tool (default unless restricted) |
154174
| "browse web pages/docs" | `web-fetch` and/or `web-search` |
155175
| "test UI flows" | `playwright` |
156176

177+
### Pattern Heuristics
178+
179+
| User says... | Recommended named pattern |
180+
|---|---|
181+
| "triage issues automatically" | `IssueOps` |
182+
| "run on /commands with human approval loops" | `ChatOps` |
183+
| "run every weekday and keep improving" | `DailyOps` |
184+
| "monitor workflow failures and trends" | `MonitorOps` |
185+
| "process a big backlog in chunks" | `BatchOps` |
186+
| "run manually with input parameters" | `DispatchOps` |
187+
157188
### Integration Auth Mapping
158189

159190
When the user names a third-party service or MCP server:
@@ -289,8 +320,8 @@ Before final output, run this internal self-check:
289320
## References (load only when needed)
290321

291322
In-repo references:
292-
- `.github/aw/syntax.md`
293-
- `.github/aw/safe-outputs.md`
323+
- `.github/aw/syntax.md` (index → `.github/aw/syntax-core.md`, `.github/aw/syntax-agentic.md`, `.github/aw/syntax-tools-imports.md`)
324+
- `.github/aw/safe-outputs.md` (index → `.github/aw/safe-outputs-content.md`, `.github/aw/safe-outputs-management.md`, `.github/aw/safe-outputs-automation.md`, `.github/aw/safe-outputs-runtime.md`)
294325
- `.github/aw/network.md`
295326
- `.github/aw/patterns.md`
296327
- `.github/aw/subagents.md`
@@ -299,8 +330,8 @@ In-repo references:
299330
- `.github/aw/create-agentic-workflow.md`
300331

301332
Portable HTTPS references:
302-
- `https://github.com/github/gh-aw/blob/main/.github/aw/syntax.md`
303-
- `https://github.com/github/gh-aw/blob/main/.github/aw/safe-outputs.md`
333+
- `https://github.com/github/gh-aw/blob/main/.github/aw/syntax.md` (index → `.../syntax-core.md`, `.../syntax-agentic.md`, `.../syntax-tools-imports.md`)
334+
- `https://github.com/github/gh-aw/blob/main/.github/aw/safe-outputs.md` (index → `.../safe-outputs-content.md`, `.../safe-outputs-management.md`, `.../safe-outputs-automation.md`, `.../safe-outputs-runtime.md`)
304335
- `https://github.com/github/gh-aw/blob/main/.github/aw/network.md`
305336
- `https://github.com/github/gh-aw/blob/main/.github/aw/patterns.md`
306337
- `https://github.com/github/gh-aw/blob/main/.github/aw/triggers.md`

0 commit comments

Comments
 (0)