Skip to content
Draft
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
6 changes: 3 additions & 3 deletions .github/CODEOWNERS
Original file line number Diff line number Diff line change
Expand Up @@ -147,13 +147,13 @@
/specification/iothub/ @marcodalessandro @riteshrao

# PRLabel: %KeyVault
/specification/keyvault/resource-manager/ @vickm @chen-karen @cheathamb36 @Azure/azure-sdk-write-keyvault
/specification/keyvault/resource-manager/ @chen-karen @Azure/azure-sdk-write-keyvault

# PRLabel: %KeyVault
/specification/keyvault/Security.*/ @vickm @chen-karen @cheathamb36 @Azure/azure-sdk-write-keyvault @heaths @chlowell
/specification/keyvault/Security.*/ @chen-karen @Azure/azure-sdk-write-keyvault @heaths

# PRLabel: %KeyVault
/specification/keyvault/data-plane/ @vickm @chen-karen @cheathamb36 @Azure/azure-sdk-write-keyvault @heaths
/specification/keyvault/data-plane/ @chen-karen @Azure/azure-sdk-write-keyvault @heaths

# PRLabel: %Logic App
/specification/logic/ @pankajsn @tonytang-microsoft-com
Expand Down
4 changes: 4 additions & 0 deletions .github/copilot-instructions.md
Original file line number Diff line number Diff line change
Expand Up @@ -359,6 +359,10 @@ Follow the steps in [typespec to sdk](..\eng\common\instructions\azsdk-tools\typ

Run [check package readiness](..\eng\common\instructions\azsdk-tools\check-package-readiness.instructions.md) to check the release readiness of an SDK package. This prompt will collect the required information from the user, execute the readiness check, and present the results.

## SDK customizations in TypeSpec projects

TypeSpec supports making client-specific customizations to a TypeSpec project to change how an SDK is generated. When making client-specific changes, read the [typespec client customizations reference](../eng/common/knowledge/customizing-client-tsp.md) to understand the types of customizations supported and how to apply them.

# Instructions for GitHub coding agent to generate SDKs using GitHub.com

Run only SDK generation step and Update SDK details back in issue description and release plan. Do not run TypeSpec validation, or service label check. This is applicable only when GitHub copilot coding agent is used to generate SDK and if a Release Plan is available in the context. Run SDK generation tool even if release plan already has SDK generation pipeline URL to support regenerating SDK.
Expand Down
585 changes: 286 additions & 299 deletions .github/package-lock.json

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion .github/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@
"@apidevtools/json-schema-ref-parser": "^14.2.1",
"debug": "^4.4.3",
"js-yaml": "^4.1.0",
"marked": "^16.3.0",
"marked": "^17.0.0",
"markdown-table": "^3.0.4",
"simple-git": "^3.27.0",
"zod": "^4.1.11"
Expand Down
16 changes: 13 additions & 3 deletions .github/shared/cmd/spec-model.js
Original file line number Diff line number Diff line change
Expand Up @@ -4,15 +4,20 @@ import { debugLogger } from "../src/logger.js";
import { SpecModel } from "../src/spec-model.js";

const USAGE =
"Usage: npx spec-model path/to/spec [--debug] [--include-refs] [--relative-paths] [--no-embed-errors]\n" +
"Example: npx spec-model specification/widget";
"Usage: npx spec-model path/to/spec [--debug] [--include-operations] [--include-refs] [--relative-paths] [--no-embed-errors]\n" +
"Examples:\n" +
" npx spec-model specification/widget\n" +
" npx spec-model specification --include-operations --include-refs > out.txt; grep out.txt '\"error\":'";

// Exclude first two args (node, script file)
let args = process.argv.slice(2);

const debug = args.includes("--debug");
args = args.filter((a) => a != "--debug");

const includeOperations = args.includes("--include-operations");
args = args.filter((a) => a != "--include-operations");

const includeRefs = args.includes("--include-refs");
args = args.filter((a) => a != "--include-refs");

Expand Down Expand Up @@ -42,7 +47,12 @@ const specModel = new SpecModel(specPath, { logger });

console.log(
JSON.stringify(
await specModel.toJSONAsync({ embedErrors: !noEmbedErrors, includeRefs, relativePaths }),
await specModel.toJSONAsync({
embedErrors: !noEmbedErrors,
includeOperations,
includeRefs,
relativePaths,
}),
null,
2,
),
Expand Down
9 changes: 7 additions & 2 deletions .github/shared/eslint.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,11 @@ import globals from "globals";
import tseslint from "typescript-eslint";

/** @type {import('eslint').Linter.Config[]} */
export default defineConfig(eslint.configs.recommended, tseslint.configs.recommended, {
languageOptions: { globals: globals.node },
export default defineConfig(eslint.configs.recommended, tseslint.configs.recommendedTypeChecked, {
languageOptions: {
// we only run in node, not browser
globals: globals.node,
// required to use tseslint.configs.recommendedTypeChecked
parserOptions: { projectService: true },
},
});
Loading