Azure · AlitzelMendez · Jul 17, 2025 · Jul 17, 2025 · Jul 17, 2025 · Jul 17, 2025
@@ -1,3 +1,3 @@
 name: Setup Node and install dependencies
 description: Uses specified Node version and installs dependencies (typically using "npm")

@@ -7,7 +7,7 @@
     default: 22.x
   install-command:
     description: "Command to install dependencies"
-    default: "npm ci"
+    default: "npm i --no-package-lock --force"
   working-directory:
     description: "Working directory"
     default: "."

@@ -1,3 +1,3 @@
 // @ts-check

 import { readdir } from "fs/promises";
@@ -5,6 +5,9 @@
 import { flatMapAsync, mapAsync } from "./array.js";
 import { Readme } from "./readme.js";
 
+/** @type {Map<string, SpecModel>} */
+const specModelCache = new Map();
+
 /**
  * @typedef {Object} ToJSONOptions
  * @prop {boolean} [includeRefs]
@@ -16,6 +19,7 @@
 
 export class SpecModel {
   /** @type {string} absolute path */
+  // @ts-ignore Ignore error that value may not be set in ctor (since we may returned cached value)
   #folder;
 
   /** @type {import('./logger.js').ILogger | undefined} */
@@ -30,8 +34,17 @@
    * @param {import('./logger.js').ILogger} [options.logger]
    */
   constructor(folder, options) {
-    this.#folder = resolve(folder);
+    const resolvedFolder = resolve(folder);
+
+    const cachedSpecModel = specModelCache.get(resolvedFolder);
+    if (cachedSpecModel !== undefined) {
+      return cachedSpecModel;
+    }
+
+    this.#folder = resolvedFolder;
     this.#logger = options?.logger;
+
+    specModelCache.set(resolvedFolder, this);
   }
 
   /**

@@ -1,5 +1,6 @@
 // @ts-check
 
+import { randomUUID } from "crypto";
 import { readdir } from "fs/promises";
 import { dirname, isAbsolute, join, resolve } from "path";
 import { describe, expect, it } from "vitest";
@@ -18,6 +19,15 @@
     await expect(specModel.getReadmes()).rejects.toThrowError(/no such file or directory/i);
   });
 
+  it("returns cached spec model", async () => {
+    const path = randomUUID();
+
+    const specModel1 = new SpecModel(path);
+    const specModel2 = new SpecModel(path);
+
+    expect(specModel1).toBe(specModel2);
+  });
+
   it("returns spec model", async () => {
     const folder = resolve(
       __dirname,

@@ -1,14 +1,14 @@
 // @ts-check
 
-import { dirname, resolve, join } from "path";
+import { dirname, join, resolve } from "path";
 import { describe, expect, it } from "vitest";
 import { Swagger } from "../src/swagger.js";
 
 import { fileURLToPath } from "url";
+import { ConsoleLogger } from "../src/logger.js";
 import { Readme } from "../src/readme.js";
-import { Tag } from "../src/tag.js";
 import { SpecModel } from "../src/spec-model.js";
-import { ConsoleLogger } from "../src/logger.js";
+import { Tag } from "../src/tag.js";
 
 const __dirname = dirname(fileURLToPath(import.meta.url));
 
@@ -51,6 +51,26 @@
     );
   });
 
+  it("returns examples", async () => {
+    const swagger = new Swagger(resolve(__dirname, "fixtures/swagger/ignoreExamples/swagger.json"));
+    const examples = await swagger.getExamples();
+
+    const expectedExamplePath = resolve(
+      __dirname,
+      "fixtures/swagger/ignoreExamples/examples/example.json",
+    );
+    expect(examples).toMatchObject(
+      new Map([
+        [
+          expectedExamplePath,
+          expect.objectContaining({
+            path: expect.stringContaining(expectedExamplePath),
+          }),
+        ],
+      ]),
+    );
+  });
+
   describe("getOperations", () => {
     it("should return normal operations", async () => {
       const testFixturePath = join(__dirname, "fixtures", "swagger", "specification");

@@ -1,3 +1,3 @@
 name: ARM Auto SignOff

 on:
@@ -16,25 +16,6 @@
   workflow_run:
     workflows: ["ARM Incremental TypeSpec"]
     types: [completed]
-  # For manual testing
-  workflow_dispatch:
-    inputs:
-      owner:
-        description: The account owner of the repository. The name is not case sensitive.
-        required: true
-        type: string
-      repo:
-        description: The name of the repository without the .git extension. The name is not case sensitive.
-        required: true
-        type: string
-      issue_number:
-        description: The number of the pull request.
-        required: true
-        type: string
-      head_sha:
-        description: The SHA of the commit.
-        required: true
-        type: string
 
 permissions:
   actions: read
@@ -51,7 +32,6 @@
     # issue_comment:edited - filter to only PR comments containing "next steps to merge",
     #   a signal that checks like "Swagger LintDiff" or "Swagger Avocado" status may have changed
     if: |
-      github.event_name == 'workflow_dispatch' ||
       github.event_name == 'workflow_run' ||
       (github.event_name == 'pull_request_target' &&
        (github.event.action == 'labeled' ||
@@ -96,11 +76,6 @@
             const { default: getLabelAction } =
               await import('${{ github.workspace }}/.github/workflows/src/arm-auto-signoff.js');
             return await getLabelAction({ github, context, core });
-        env:
-          OWNER: ${{ inputs.owner }}
-          REPO: ${{ inputs.repo }}
-          ISSUE_NUMBER: ${{ inputs.issue_number }}
-          HEAD_SHA: ${{ inputs.head_sha }}
 
       - if: |
           fromJson(steps.get-label-action.outputs.result).labelAction == 'add' ||

@@ -1,3 +1,3 @@
 // @ts-check

 import { setEquals } from "../../shared/src/equality.js";
@@ -12,18 +12,7 @@
  * @returns {Promise<{labelAction: LabelAction, issueNumber: number}>}
  */
 export default async function getLabelAction({ github, context, core }) {
-  let owner = process.env.OWNER || "";
-  let repo = process.env.REPO || "";
-  let issue_number = parseInt(process.env.ISSUE_NUMBER || "");
-  let head_sha = process.env.HEAD_SHA || "";
-
-  if (!owner || !repo || !issue_number || !head_sha) {
-    let inputs = await extractInputs(github, context, core);
-    owner = owner || inputs.owner;
-    repo = repo || inputs.repo;
-    issue_number = issue_number || inputs.issue_number;
-    head_sha = head_sha || inputs.head_sha;
-  }
+  const { owner, repo, issue_number, head_sha } = await extractInputs(github, context, core);
 
   return await getLabelActionImpl({
     owner,

@@ -1,3 +1,3 @@
 // @ts-check

 import { extractInputs } from "../src/context.js";
@@ -7,19 +7,7 @@
  * @param {import('@actions/github-script').AsyncFunctionArguments} AsyncFunctionArguments
  */
 export default async function updateLabels({ github, context, core }) {
-  let owner = process.env.OWNER;
-  let repo = process.env.REPO;
-  let issue_number = parseInt(process.env.ISSUE_NUMBER || "");
-  let run_id = parseInt(process.env.RUN_ID || "");
-
-  if (!owner || !repo || !(issue_number || run_id)) {
-    let inputs = await extractInputs(github, context, core);
-    owner = owner || inputs.owner;
-    repo = repo || inputs.repo;
-    issue_number = issue_number || inputs.issue_number;
-    run_id = run_id || inputs.run_id;
-  }
-
+  const { owner, repo, issue_number, run_id } = await extractInputs(github, context, core);
   await updateLabelsImpl({ owner, repo, issue_number, run_id, github, core });
 }
 

@@ -1,52 +1,9 @@
 import { describe, expect, it } from "vitest";
 import { PER_PAGE_MAX } from "../src/github.js";
 import updateLabels, { updateLabelsImpl } from "../src/update-labels.js";
 import { createMockCore, createMockGithub, createMockRequestError } from "./mocks.js";
 
 describe("updateLabels", () => {
-  it("loads inputs from env", async () => {
-    const github = createMockGithub();
-    github.rest.actions.listWorkflowRunArtifacts.mockResolvedValue({
-      data: {
-        artifacts: [{ name: "label-foo=true" }],
-      },
-    });
-
-    try {
-      process.env.OWNER = "TestRepoOwnerLoginEnv";
-      process.env.REPO = "TestRepoNameEnv";
-      process.env.ISSUE_NUMBER = "123";
-      process.env.RUN_ID = "456";
-
-      await expect(
-        updateLabels({
-          github: github,
-          context: null,
-          core: createMockCore(),
-        }),
-      ).resolves.toBeUndefined();
-    } finally {
-      delete process.env.OWNER;
-      delete process.env.REPO;
-      delete process.env.ISSUE_NUMBER;
-      delete process.env.RUN_ID;
-    }
-
-    expect(github.rest.actions.listWorkflowRunArtifacts).toBeCalledWith({
-      owner: "TestRepoOwnerLoginEnv",
-      repo: "TestRepoNameEnv",
-      run_id: 456,
-      per_page: PER_PAGE_MAX,
-    });
-    expect(github.rest.issues.addLabels).toBeCalledWith({
-      owner: "TestRepoOwnerLoginEnv",
-      repo: "TestRepoNameEnv",
-      issue_number: 123,
-      labels: ["foo"],
-    });
-    expect(github.rest.issues.removeLabel).toBeCalledTimes(0);
-  });
-
   it("loads inputs from context", async () => {
     const github = createMockGithub();
     github.rest.actions.listWorkflowRunArtifacts.mockResolvedValue({
@@ -96,64 +53,6 @@
     });
     expect(github.rest.issues.removeLabel).toBeCalledTimes(0);
   });
-
-  it("loads inputs from env and context", async () => {
-    const github = createMockGithub();
-    github.rest.actions.listWorkflowRunArtifacts.mockResolvedValue({
-      data: {
-        artifacts: [{ name: "label-foo=true" }],
-      },
-    });
-
-    const context = {
-      eventName: "workflow_run",
-      payload: {
-        action: "completed",
-        workflow_run: {
-          event: "pull_request",
-          head_sha: "abc123",
-          id: 456,
-          repository: {
-            name: "TestRepoName",
-            owner: {
-              login: "TestRepoOwnerLogin",
-            },
-          },
-          pull_requests: [{ number: 123 }],
-        },
-      },
-    };
-
-    try {
-      process.env.OWNER = "TestRepoOwnerLoginEnv";
-      process.env.REPO = "TestRepoNameEnv";
-
-      await expect(
-        updateLabels({
-          github: github,
-          context: context,
-          core: createMockCore(),
-        }),
-      ).resolves.toBeUndefined();
-    } finally {
-      delete process.env.OWNER;
-      delete process.env.REPO;
-    }
-
-    expect(github.rest.actions.listWorkflowRunArtifacts).toBeCalledWith({
-      owner: "TestRepoOwnerLoginEnv",
-      repo: "TestRepoNameEnv",
-      run_id: 456,
-      per_page: PER_PAGE_MAX,
-    });
-    expect(github.rest.issues.addLabels).toBeCalledWith({
-      owner: "TestRepoOwnerLoginEnv",
-      repo: "TestRepoNameEnv",
-      issue_number: 123,
-      labels: ["foo"],
-    });
-    expect(github.rest.issues.removeLabel).toBeCalledTimes(0);
-  });
 });
 
 describe("updateLabelsImpl", () => {

@@ -1,3 +1,3 @@
 name: Update Labels

 on:
@@ -9,26 +9,6 @@
     workflows:
       ["ARM Auto SignOff", "SDK Breaking Change Labels", "SDK Suppressions", "TypeSpec Requirement"]
     types: [completed]
-  workflow_dispatch:
-    inputs:
-      owner:
-        description: The account owner of the repository. The name is not case sensitive.
-        required: true
-        type: string
-      repo:
-        description: The name of the repository without the .git extension. The name is not case sensitive.
-        required: true
-        type: string
-      # simulate pull_request trigger
-      issue_number:
-        description: The number that identifies the issue.
-        required: false
-        type: number
-      # simulate workflow_run trigger
-      run_id:
-        description: The unique identifier of the workflow run.
-        required: false
-        type: number
 
 permissions:
   actions: read
@@ -52,11 +32,6 @@
 
       - name: Update Labels
         uses: actions/github-script@v7
-        env:
-          OWNER: ${{ inputs.owner }}
-          REPO: ${{ inputs.repo }}
-          ISSUE_NUMBER: ${{ inputs.issue_number }}
-          RUN_ID: ${{ inputs.run_id }}
         with:
           script: |
             const { default: updateLabels } =