Skip to content

feat: add Azure Pipelines integration for setup-vp#108

Open
tonoizer wants to merge 7 commits into
voidzero-dev:mainfrom
tonoizer:feat/azure-pipelines-integration
Open

feat: add Azure Pipelines integration for setup-vp#108
tonoizer wants to merge 7 commits into
voidzero-dev:mainfrom
tonoizer:feat/azure-pipelines-integration

Conversation

@tonoizer

@tonoizer tonoizer commented Jul 10, 2026

Copy link
Copy Markdown

Summary

  • Add a reusable Azure Pipelines step template (azure/setup-vp.yml) with Bash and PowerShell bootstraps for Linux/macOS and Windows agents.
  • Extract shared dependency-light CI primitives into src/ci/* and keep the GitLab runtime as thin adapters over that layer.
  • Add an Azure runtime (src/azure/*, dist/azure/index.mjs) with prepare / finalize phases around Azure Cache@2, including cross-platform SFW support and logging-command adapters.
  • Document the integration in rfcs/azure-pipelines-integration.md and extend README/AGENTS/CI smoke coverage.

Test plan

  • vp run typecheck
  • vp run test (306 tests)
  • vp run check
  • vp run build
  • bash -n azure/bootstrap.sh gitlab/bootstrap.sh
  • PowerShell syntax check for azure/bootstrap.ps1
  • Portable bundle smoke test (dist/azure/index.mjs invalid-phase failure)
  • GitHub Action bundle unchanged (dist/index.mjs)
  • Real Azure Pipelines validation on ubuntu-latest, macOS-latest, and windows-latest (release-blocking follow-up)

Notes

  • plans/ was intentionally excluded from this PR.
  • Azure end-to-end cache and cross-OS validation still needs an Azure DevOps project with a GitHub service connection before release.

tonoizer and others added 4 commits July 10, 2026 22:15
Introduce a reusable Azure step template with prepare/cache/finalize lifecycle,
shared portable CI primitives extracted from the GitLab runtime, and a compiled
dist/azure bundle for Linux, macOS, and Windows agents.

Co-authored-by: Cursor <cursoragent@cursor.com>
@fengmk2 fengmk2 requested a review from naokihaba July 11, 2026 03:20
@naokihaba

Copy link
Copy Markdown
Collaborator

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: f84c28dbd4

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread dist/gitlab/index.mjs Outdated
Comment thread src/azure/commands.ts Outdated
Comment thread .github/workflows/test.yml Outdated
@naokihaba naokihaba self-assigned this Jul 11, 2026
Comment thread dist/gitlab/index.mjs Outdated
@naokihaba naokihaba self-requested a review July 11, 2026 16:06
@naokihaba

Copy link
Copy Markdown
Collaborator

@codex review

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9f5b303fd1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread azure/setup-vp.yml
@tonoizer

Copy link
Copy Markdown
Author

@naokihaba what else do we need to get this PR merged/CI run for full validation?

@naokihaba naokihaba requested a review from fengmk2 July 13, 2026 04:27
@naokihaba

Copy link
Copy Markdown
Collaborator

@fengmk2

I have finished reviewing your code.
Before we can merge it, we just need to make sure it passes two checks.

First, we need to run it on an actual Azure DevOps project.
Second, we need to run it in the GitLab CI environment to confirm the refactoring hasn't broken anything or caused any regressions.

Do we happen to have an Azure DevOps project set up in our company that we could use?

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