Skip to content

Support CI.yml provisioning and updates with cli#14432

Open
skywing918 wants to merge 13 commits intomainfrom
updateCi
Open

Support CI.yml provisioning and updates with cli#14432
skywing918 wants to merge 13 commits intomainfrom
updateCi

Conversation

@skywing918
Copy link
Contributor

@skywing918 skywing918 commented Mar 10, 2026

#13810

  • add generate-cli command
  • implement ci.yaml(data plane) logic

generated ci.yml for ai agent.

# NOTE: Please refer to https://aka.ms/azsdk/engsys/ci-yaml before editing this file.

trigger:
  branches:
    include:
      - main
      - release/*
      - hotfix/*
    exclude:
      - feature/v4
  paths:
    include:
      - sdk/ai/
      - sdk/ai/ai-agents
      - sdk/ai/ci.yml
pr:
  branches:
    include:
      - main
      - feature/*
      - release/*
      - hotfix/*
    exclude:
      - feature/v4
  paths:
    include:
      - sdk/ai/
      - sdk/ai/ai-agents
      - sdk/ai/ci.yml
extends:
  template: ../../eng/pipelines/templates/stages/archetype-sdk-client.yml
  parameters:
    ServiceDirectory: ai
    Artifacts:
      - name: azure-rest-ai-inference
        safeName: azurerestaiinference
      - name: azure-ai-projects
        safeName: azureaaiprojects
      - name: azure-ai-agents
        safeName: azureaaiagents

Copilot AI review requested due to automatic review settings March 10, 2026 08:58
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new generate-ci-yaml CLI entry to js-sdk-release-tools to provision/update ci.yml (and related CI configuration) for JS packages, supporting the “brand-new SDK” workflow described in issue #13810.

Changes:

  • Add a new Node CLI (generate-ci-yaml) that infers version policy and invokes CI YAML creation/update.
  • Implement data-plane ci.yml create/update logic in ciYamlUtils, including syncing exclusions from ci.mgmt.yml when present.
  • Add a data-plane CI YAML template and expose the new CLI via package.json bin.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 8 comments.

File Description
tools/js-sdk-release-tools/src/generateCiYamlCli.ts New CLI wrapper to compute paths, detect version policy, and call CI YAML generator
tools/js-sdk-release-tools/src/common/ciYamlUtils.ts Implements create/update for data-plane ci.yml and refactors CI YAML writing
tools/js-sdk-release-tools/src/common/ciYamlTemplates/ci.template.yml New CI YAML template for data-plane ci.yml
tools/js-sdk-release-tools/package.json Adds generate-ci-yaml to published bin commands

@skywing918 skywing918 marked this pull request as draft March 10, 2026 09:10
@skywing918 skywing918 requested a review from Copilot March 10, 2026 09:34
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.

@skywing918 skywing918 marked this pull request as ready for review March 19, 2026 07:14
import path from "path";

const generateCiYamlCli = async (
sdkRepoPath: string | undefined,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will this CLI be used directly in inner loop? Need to confirm whether we had the agreement about the input/output of this too. Please check with Ray for more clarification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants