Commit 014fdea
Sync .github/skills directory with azure-sdk-tools for PR 16380 (#48117)
* Add environment.skills to workflow-scenario eval specs
Wires environment.skills: [<skill dirs>] into the mock workflow eval YAML specs so Vally loads the relevant skill per-eval instead of relying on --skill-dir at the pipeline level.
* Fix flaky generate-sdk-locally eval stimuli
- Add missing environment.skills block to sdk-local-update-metadata-001
(all other positive stimuli in this file already had it)
- Replace ambiguous/fictional 'contoso' package paths with the real
fixture package (sdk/compute/azure-mgmt-compute, azure-resourcemanager-compute)
so the model doesn't need to ask for clarification on a made-up path
- Add explicit version/release-type/date to the version-update stimuli
since the tool genuinely requires that info
Verified locally: suite score went from ~42% to consistently 83-92%
across repeated runs (threshold 80%).
* Broaden sdk-release skill trigger phrases
The trigger.eval.yaml suite reproducibly failed at ~70%% because vague/
informational prompts (SDK release checklist, walk me through the
release process, how to publish a new SDK version, release a new
version of azure-sdk) weren't covered by the skill's narrow trigger
phrase list, so the model didn't reliably invoke the skill.
Added those phrasings to the description, Triggers section, and
Examples, and clarified that the skill should also handle how-to/
checklist style questions, not just direct action requests.
Verified locally: trigger.eval.yaml went from 70%% to 100%% across two
repeated runs (threshold 80%%).
* Add environment.skills to anti-trigger/negative eval stimuli
Per Copilot PR review feedback: anti-trigger and negative stimuli in
these eval specs didn't set environment.skills, so the skill was never
loaded/available for those prompts. That made the disallowed
skill-invocation / tool-calls grader checks trivially true (the skill
literally couldn't be invoked), providing no real signal.
Added environment.skills: [..] to all anti-trigger/negative stimuli in:
- azsdk-common-apiview-feedback-resolution/evals/trigger.eval.yaml
- azsdk-common-pipeline-fixer/evals/trigger.eval.yaml
- azsdk-common-pipeline-analysis/evals/trigger.eval.yaml
- azsdk-common-prepare-release-plan/evals/trigger.eval.yaml
- azsdk-common-sdk-release/evals/trigger.eval.yaml
- azsdk-common-generate-sdk-locally/evals/eval.yaml (sdk-local-negative-001)
Verified locally: all 6 suites still pass with the skill loaded and
available - the model correctly declines to invoke it on out-of-scope
prompts. No new failures introduced (5 trigger suites at 100%,
generate-sdk-locally-eval at 91.7%, same pre-existing unrelated failure
as before on sdk-local-analyzer-errors-001).
* Mirror .github/skills after patch conflict
---------
Co-authored-by: helen229 <gaoh@microsoft.com>
Co-authored-by: azure-sdk <azuresdk@microsoft.com>1 parent 7ef7c8c commit 014fdea
11 files changed
Lines changed: 279 additions & 13 deletions
File tree
- .github/skills
- azsdk-common-apiview-feedback-resolution/evals
- azsdk-common-generate-sdk-locally
- evals
- azsdk-common-generate-sdk-pipeline
- azsdk-common-pipeline-analysis/evals
- azsdk-common-pipeline-fixer/evals
- azsdk-common-prepare-release-plan
- evals
- azsdk-common-sdk-release
- evals
Lines changed: 4 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
23 | 27 | | |
24 | 28 | | |
25 | 29 | | |
| |||
Lines changed: 39 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
23 | 26 | | |
24 | 27 | | |
25 | 28 | | |
26 | 29 | | |
27 | 30 | | |
28 | 31 | | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
29 | 35 | | |
30 | 36 | | |
31 | 37 | | |
32 | 38 | | |
33 | 39 | | |
34 | 40 | | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
35 | 44 | | |
36 | 45 | | |
37 | 46 | | |
38 | 47 | | |
39 | 48 | | |
40 | 49 | | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
41 | 53 | | |
42 | 54 | | |
43 | 55 | | |
44 | 56 | | |
45 | 57 | | |
46 | 58 | | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
47 | 62 | | |
48 | 63 | | |
49 | 64 | | |
50 | 65 | | |
51 | 66 | | |
52 | 67 | | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
53 | 71 | | |
54 | 72 | | |
55 | 73 | | |
56 | 74 | | |
57 | 75 | | |
58 | 76 | | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
59 | 80 | | |
60 | 81 | | |
61 | 82 | | |
62 | 83 | | |
63 | 84 | | |
64 | 85 | | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
65 | 89 | | |
66 | 90 | | |
67 | 91 | | |
68 | 92 | | |
69 | 93 | | |
70 | 94 | | |
71 | 95 | | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
72 | 99 | | |
73 | 100 | | |
74 | 101 | | |
75 | 102 | | |
76 | 103 | | |
77 | 104 | | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
78 | 108 | | |
79 | 109 | | |
80 | 110 | | |
81 | 111 | | |
82 | 112 | | |
83 | 113 | | |
| 114 | + | |
| 115 | + | |
| 116 | + | |
84 | 117 | | |
85 | 118 | | |
86 | 119 | | |
87 | 120 | | |
88 | 121 | | |
89 | 122 | | |
| 123 | + | |
| 124 | + | |
| 125 | + | |
90 | 126 | | |
91 | 127 | | |
92 | 128 | | |
93 | 129 | | |
94 | 130 | | |
95 | 131 | | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
96 | 135 | | |
97 | 136 | | |
98 | 137 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
| 24 | + | |
23 | 25 | | |
24 | 26 | | |
25 | 27 | | |
| |||
83 | 85 | | |
84 | 86 | | |
85 | 87 | | |
| 88 | + | |
86 | 89 | | |
87 | 90 | | |
88 | 91 | | |
| |||
Lines changed: 43 additions & 7 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
23 | 26 | | |
24 | 27 | | |
25 | 28 | | |
| |||
29 | 32 | | |
30 | 33 | | |
31 | 34 | | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
32 | 38 | | |
33 | 39 | | |
34 | 40 | | |
| |||
37 | 43 | | |
38 | 44 | | |
39 | 45 | | |
40 | | - | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
41 | 50 | | |
42 | 51 | | |
43 | 52 | | |
| |||
53 | 62 | | |
54 | 63 | | |
55 | 64 | | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
56 | 68 | | |
57 | 69 | | |
58 | 70 | | |
| |||
63 | 75 | | |
64 | 76 | | |
65 | 77 | | |
66 | | - | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
67 | 82 | | |
68 | 83 | | |
69 | 84 | | |
| |||
74 | 89 | | |
75 | 90 | | |
76 | 91 | | |
77 | | - | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
78 | 96 | | |
79 | 97 | | |
80 | 98 | | |
| |||
85 | 103 | | |
86 | 104 | | |
87 | 105 | | |
88 | | - | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
89 | 110 | | |
90 | 111 | | |
91 | 112 | | |
| |||
96 | 117 | | |
97 | 118 | | |
98 | 119 | | |
99 | | - | |
| 120 | + | |
| 121 | + | |
| 122 | + | |
| 123 | + | |
100 | 124 | | |
101 | 125 | | |
102 | 126 | | |
| |||
108 | 132 | | |
109 | 133 | | |
110 | 134 | | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
111 | 138 | | |
112 | 139 | | |
113 | 140 | | |
| |||
118 | 145 | | |
119 | 146 | | |
120 | 147 | | |
121 | | - | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
122 | 152 | | |
123 | 153 | | |
124 | 154 | | |
| |||
129 | 159 | | |
130 | 160 | | |
131 | 161 | | |
132 | | - | |
| 162 | + | |
| 163 | + | |
| 164 | + | |
| 165 | + | |
133 | 166 | | |
134 | 167 | | |
135 | 168 | | |
| |||
141 | 174 | | |
142 | 175 | | |
143 | 176 | | |
| 177 | + | |
| 178 | + | |
| 179 | + | |
144 | 180 | | |
145 | 181 | | |
146 | 182 | | |
| |||
Lines changed: 55 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
0 commit comments