Add Copilot CLI skill definitions for azure-ai packages#48270
Open
jpalvarezl wants to merge 5 commits intomainfrom
Open
Add Copilot CLI skill definitions for azure-ai packages#48270jpalvarezl wants to merge 5 commits intomainfrom
jpalvarezl wants to merge 5 commits intomainfrom
Conversation
Add reusable skill files under sdk/ai/.skills/ to provide Copilot CLI agents with domain-specific guidance for common Azure SDK for Java workflows including TypeSpec codegen, test execution, release notes, duplicate class verification, and secret management. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a set of reusable Copilot CLI skill definition markdown files under sdk/ai/.skills/ to provide domain-specific guidance for common Azure SDK for Java workflows (TypeSpec codegen/customization, testing, release notes, GitHub CLI usage, etc.).
Changes:
- Introduces multiple
SKILL.mddefinitions for AI-related developer workflows (TypeSpec generation/customization, test execution, release note drafting). - Adds skills for operational tooling (GitHub
gh, test-proxy publishing, Maven local repo inspection, Key Vault work-resources secret management). - Adds a skill for comparing generated models vs
openai-javamodel shapes.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 6 comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/ai/.skills/codegen/SKILL.md | Skill instructions for tsp-client update/sync/generate workflows. |
| sdk/ai/.skills/dup-classes/SKILL.md | Skill instructions for field-by-field model shape comparison vs openai-java. |
| sdk/ai/.skills/github/SKILL.md | Skill instructions for GitHub interactions via gh CLI. |
| sdk/ai/.skills/release-notes/SKILL.md | Skill instructions for drafting/updating CHANGELOG/README content from a PR. |
| sdk/ai/.skills/run-tests/SKILL.md | Skill instructions for running Maven tests (modes, secrets, troubleshooting). |
| sdk/ai/.skills/search-m2/SKILL.md | Skill instructions for locating classes/JARs in ~/.m2/repository. |
| sdk/ai/.skills/test-proxy/SKILL.md | Skill instructions for pushing test-proxy recordings/assets. |
| sdk/ai/.skills/tsp-naming-collision/SKILL.md | Skill instructions for resolving TypeSpec → Java codegen naming collisions (suffix 1). |
| sdk/ai/.skills/tsp-type-override/SKILL.md | Skill instructions for @@alternateType overrides and validation/testing guidance. |
| sdk/ai/.skills/wr-load/SKILL.md | Skill instructions for managing secrets via work-resources (wr-*) CLI. |
| - **JPMS/module-path errors** (e.g., `okio` module issues): add `-Dsurefire.useModulePath=false` | ||
| - **Build plugins block the run**: add skip flags as needed, e.g. `-Denforcer.skip=true -Dcodesnippet.skip=true -Dcheckstyle.skip=true` | ||
| - **Reactor blocking errors in async tests** (Netty thread): add `$env:AZURE_TEST_HTTP_CLIENTS = "okhttp"` | ||
| - **SSL handshake / PKIX path building failed** (`SSLHandshakeException`, `unable to find valid certification path to requested target`): the JVM's trust store is missing the corporate root CA. Add `-Djavax.net.ssl.trustStoreType=WINDOWS-ROOT` to use the Windows certificate store instead. This is common on corporate networks with proxy/firewall TLS interception. |
There was a problem hiding this comment.
-Djavax.net.ssl.trustStoreType=WINDOWS-ROOT is Windows-specific. This troubleshooting bullet should explicitly call out that it only applies on Windows (and suggest an alternative or omit it for non-Windows shells) to avoid sending Linux/macOS users down a dead end.
Suggested change
| - **SSL handshake / PKIX path building failed** (`SSLHandshakeException`, `unable to find valid certification path to requested target`): the JVM's trust store is missing the corporate root CA. Add `-Djavax.net.ssl.trustStoreType=WINDOWS-ROOT` to use the Windows certificate store instead. This is common on corporate networks with proxy/firewall TLS interception. | |
| - **SSL handshake / PKIX path building failed** (`SSLHandshakeException`, `unable to find valid certification path to requested target`): the JVM's trust | |
| store is missing the corporate root CA. On **Windows**, you can add `-Djavax.net.ssl.trustStoreType=WINDOWS-ROOT` to use the Windows certificate store | |
| instead. On **Linux/macOS**, work with your IT team to import the corporate root CA into the JVM trust store (or configure a custom trust store via | |
| `-Djavax.net.ssl.trustStore` / `-Djavax.net.ssl.trustStorePassword`). This is common on corporate networks with proxy/firewall TLS interception. |
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.
Summary
Adds reusable Copilot CLI skill definitions under \sdk/ai/.skills/\ to provide domain-specific guidance for common Azure SDK for Java workflows.
Skills included
tsp-clientghCLINotes