Skip to content

Commit 45b72db

Browse files
Sync .github/skills directory with azure-sdk-tools for PR 15376 (#46991)
* fixed with upstream * remove old directories * update to match new skill names * 0.4.0 * fixes to pass pipeline * change priority and just use tags * time limits --------- Co-authored-by: Juan Ospina <70209456+jeo02@users.noreply.github.com>
1 parent 34a96ed commit 45b72db

60 files changed

Lines changed: 813 additions & 1042 deletions

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/skills/azsdk-common-apiview-feedback-resolution/SKILL.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,7 @@ metadata:
55
version: "1.0.0"
66
distribution: shared
77
description: "Analyze and resolve APIView review feedback on Azure SDK PRs. **UTILITY SKILL**. USE FOR: APIView comments, API review feedback, SDK API surface changes. DO NOT USE FOR: general code review, non-APIView feedback. INVOKES: azure-sdk-mcp:azsdk_apiview_get_comments, azure-sdk-mcp:azsdk_customized_code_update."
8-
compatibility:
9-
requires: "azure-sdk-mcp server, SDK pull request with APIView review link"
8+
compatibility: "azure-sdk-mcp server, SDK pull request with APIView review link"
109
---
1110

1211
# APIView Feedback Resolution

.github/skills/azsdk-common-apiview-feedback-resolution/eval.yaml

Lines changed: 0 additions & 29 deletions
This file was deleted.
Lines changed: 81 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,29 +1,82 @@
1-
name: azsdk-common-apiview-feedback-resolution-evals
2-
description: Evaluations for the APIView feedback resolution skill
3-
skill: azsdk-common-apiview-feedback-resolution
4-
version: "1.0"
1+
name: azsdk-common-apiview-feedback-resolution-eval
2+
description: Evaluation suite for the azsdk-common-apiview-feedback-resolution skill
3+
type: capability
4+
5+
environment: azsdk-mcp-mock
6+
7+
tags:
8+
area: azsdk-common-apiview-feedback-resolution
9+
type: ci-gate
10+
511
config:
6-
model: gpt-4o
7-
executor: mock
8-
trials_per_task: 1
9-
timeout_seconds: 300
10-
parallel: false
11-
metrics:
12-
- name: task_completion
13-
weight: 0.7
14-
threshold: 0.8
15-
description: Did the skill complete trigger and anti-trigger checks?
16-
- name: efficiency
17-
weight: 0.3
18-
threshold: 0.7
19-
description: Did the skill stay within behavior limits?
20-
graders:
21-
- type: text
22-
name: no_fatal_errors
23-
config:
24-
regex_not_match:
25-
- (?i)fatal error
26-
- (?i)unhandled exception
27-
- '(?i)panic:'
28-
tasks:
29-
- "tasks/*.yaml"
12+
runs: 1
13+
timeout: "90s"
14+
model: claude-opus-4.6
15+
executor: copilot-sdk
16+
17+
scoring:
18+
threshold: 0.8
19+
20+
stimuli:
21+
- name: apiview-basic-001
22+
prompt: "My Python SDK PR has APIView comments that I need to address. Can you help me retrieve and resolve them?"
23+
graders:
24+
- type: tool-calls
25+
config:
26+
required:
27+
- name: azure-sdk-mcp-azsdk_apiview_get_comments
28+
29+
- name: apiview-edge-001
30+
prompt: "An APIView reviewer says I need to rename 'ComputeProvisioningState' to 'ProvisioningState' in my SDK. Use the SDK customization tools to apply this rename."
31+
graders:
32+
- type: output-not-contains
33+
config:
34+
substring: "cannot help"
35+
36+
- name: apiview-no-feedback-001
37+
prompt: "Check if there are any APIView comments on my .NET SDK PR that I need to address before release."
38+
graders:
39+
- type: output-contains
40+
config:
41+
substring: "APIView"
42+
43+
- name: apiview-negative-001
44+
prompt: "I need to create a new TypeSpec project for my Azure Storage service."
45+
graders:
46+
- type: tool-calls
47+
config:
48+
disallowed:
49+
- name: azure-sdk-mcp-azsdk_apiview_get_comments
50+
- type: output-not-contains
51+
config:
52+
substring: "APIView"
53+
54+
- name: basic-trigger
55+
prompt: "I have APIView comments on my SDK PR that need to be resolved. Can you help?"
56+
graders:
57+
- type: output-contains
58+
config:
59+
substring: "APIView"
60+
61+
- name: anti-trigger
62+
prompt: "How do I set up a new Azure Storage account?"
63+
graders:
64+
- type: output-not-contains
65+
config:
66+
substring: "APIView"
67+
- type: output-not-contains
68+
config:
69+
substring: "API review"
70+
- type: output-not-contains
71+
config:
72+
substring: "apiview.dev"
73+
- type: output-not-contains
74+
config:
75+
substring: "apiviewstagingtest.com"
76+
77+
- name: url-trigger
78+
prompt: "I have review comments on my SDK PR from this page: https://spa.apiview.dev/review/abc123?activeApiRevisionId=def456 — can you help me resolve them?"
79+
graders:
80+
- type: output-contains
81+
config:
82+
substring: "apiview"

.github/skills/azsdk-common-apiview-feedback-resolution/evals/tasks/anti-trigger.yaml

Lines changed: 0 additions & 16 deletions
This file was deleted.

.github/skills/azsdk-common-apiview-feedback-resolution/evals/tasks/basic-trigger.yaml

Lines changed: 0 additions & 13 deletions
This file was deleted.

.github/skills/azsdk-common-apiview-feedback-resolution/evals/tasks/url-trigger.yaml

Lines changed: 0 additions & 13 deletions
This file was deleted.
Lines changed: 99 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,99 @@
1+
name: azsdk-common-apiview-feedback-resolution-trigger-eval
2+
description: Trigger and anti-trigger tests for the azsdk-common-apiview-feedback-resolution skill
3+
type: capability
4+
5+
environment: azsdk-mcp-mock
6+
7+
tags:
8+
area: azsdk-common-apiview-feedback-resolution
9+
type: ci-gate
10+
11+
config:
12+
runs: 1
13+
timeout: "90s"
14+
model: claude-opus-4.6
15+
executor: copilot-sdk
16+
17+
scoring:
18+
threshold: 0.8
19+
20+
stimuli:
21+
- name: trigger-resolve-apiview-feedback-on-my-pr
22+
prompt: "resolve APIView feedback on my PR"
23+
graders:
24+
- type: skill-invocation
25+
config:
26+
required: ["azsdk-common-apiview-feedback-resolution"]
27+
- name: trigger-fix-apiview-comments
28+
prompt: "fix APIView comments"
29+
graders:
30+
- type: skill-invocation
31+
config:
32+
required: ["azsdk-common-apiview-feedback-resolution"]
33+
- name: trigger-address-api-review-feedback
34+
prompt: "address API review feedback"
35+
graders:
36+
- type: skill-invocation
37+
config:
38+
required: ["azsdk-common-apiview-feedback-resolution"]
39+
- name: trigger-apiview-shows-issues-with-my-sdk
40+
prompt: "APIView shows issues with my SDK"
41+
graders:
42+
- type: skill-invocation
43+
config:
44+
required: ["azsdk-common-apiview-feedback-resolution"]
45+
- name: trigger-help-me-with-apiview-suggestions
46+
prompt: "help me with APIView suggestions"
47+
graders:
48+
- type: skill-invocation
49+
config:
50+
required: ["azsdk-common-apiview-feedback-resolution"]
51+
- name: trigger-fix-comments-apiview-url-1
52+
prompt: "Help me fix these comments: https://spa.apiview.dev/review/0qw98r9qwur9q823r?activeApiRevisionId=afiq948tquajf8jq9f"
53+
graders:
54+
- type: skill-invocation
55+
config:
56+
required: ["azsdk-common-apiview-feedback-resolution"]
57+
- name: trigger-address-feedback-apiview-url-2
58+
prompt: "Address this feedback: https://spa.apiview.dev/review/akldfja9qjta498tjaejg0?activeApiRevisionId=odiufa9sefighj98439"
59+
graders:
60+
- type: skill-invocation
61+
config:
62+
required: ["azsdk-common-apiview-feedback-resolution"]
63+
- name: trigger-resolve-comments-staging-url
64+
prompt: "Resolve these comments: https://apiviewstagingtest.com/review/akldfja9qjta498tjaejg0?activeApiRevisionId=odiufa9sefighj98439"
65+
graders:
66+
- type: skill-invocation
67+
config:
68+
required: ["azsdk-common-apiview-feedback-resolution"]
69+
70+
- name: anti-trigger-deploy-my-azure-function
71+
prompt: "deploy my Azure Function"
72+
graders:
73+
- type: skill-invocation
74+
config:
75+
disallowed: ["azsdk-common-apiview-feedback-resolution"]
76+
- name: anti-trigger-fix-my-pipeline-build-error
77+
prompt: "fix my pipeline build error"
78+
graders:
79+
- type: skill-invocation
80+
config:
81+
disallowed: ["azsdk-common-apiview-feedback-resolution"]
82+
- name: anti-trigger-create-a-new-typespec-definition
83+
prompt: "create a new TypeSpec definition"
84+
graders:
85+
- type: skill-invocation
86+
config:
87+
disallowed: ["azsdk-common-apiview-feedback-resolution"]
88+
- name: anti-trigger-how-do-i-release-an-sdk-package
89+
prompt: "how do I release an SDK package"
90+
graders:
91+
- type: skill-invocation
92+
config:
93+
disallowed: ["azsdk-common-apiview-feedback-resolution"]
94+
- name: anti-trigger-set-up-azure-storage-account
95+
prompt: "set up Azure Storage account"
96+
graders:
97+
- type: skill-invocation
98+
config:
99+
disallowed: ["azsdk-common-apiview-feedback-resolution"]

.github/skills/azsdk-common-apiview-feedback-resolution/evals/trigger_tests.yaml

Lines changed: 0 additions & 29 deletions
This file was deleted.

.github/skills/azsdk-common-apiview-feedback-resolution/tasks/basic-usage.yaml

Lines changed: 0 additions & 20 deletions
This file was deleted.

.github/skills/azsdk-common-apiview-feedback-resolution/tasks/edge-case.yaml

Lines changed: 0 additions & 19 deletions
This file was deleted.

0 commit comments

Comments
 (0)