Skip to content

Add build validation pipeline status column to PR list - #28

Draft
tacoxse wants to merge 1 commit into
karlingen:mainfrom
tacoxse:add-build-validation-status
Draft

Add build validation pipeline status column to PR list#28
tacoxse wants to merge 1 commit into
karlingen:mainfrom
tacoxse:add-build-validation-status

Conversation

@tacoxse

@tacoxse tacoxse commented Jul 27, 2026

Copy link
Copy Markdown

Summary

Addresses the build status part of #1 — adds a Build column to the pull request list (between Merge Status and Comments) showing the status of the build validation pipelines for each PR.

  • Data source: the Policy Evaluations API (PolicyRestClient.getPolicyEvaluations), the same source the native PR page uses for build validation checks — filtered to enabled, non-deleted build policies, so both required and optional build validation are covered. No new extension scopes needed (vso.code suffices).
  • Rendering: one aggregated status icon per PR — failed if any pipeline is rejected/broken, else running, else queued, else succeeded. The tooltip breaks down each pipeline by name and status. Shows while loading, and an empty cell when no build validation policy applies.
  • Fetching: mirrors the existing comment-threads pattern — lazy per-visible-PR, 6-way concurrency cap, batched setState, module-level cache shared across component instances, graceful fallback to empty on API errors.

Per CONTRIBUTING.md: README updated with the interface change, manifest version bumped 1.0.43 → 1.0.44.

Testing

  • tsc --noEmit --skipLibCheck clean; production webpack build succeeds.
  • Verified visually against a live Azure DevOps organization (private packaged build): aggregated icon and per-pipeline tooltip render correctly for PRs with build validation policies; PRs without applicable policies show an empty cell.

Notes

🤖 Generated with Claude Code

Adds a "Build" column showing an aggregated status icon for the build
validation (branch policy) pipelines of each pull request, with a
tooltip breaking down each pipeline's status. Statuses are fetched
lazily per visible PR via the Policy Evaluations API, mirroring the
existing comment-threads loading pattern (cache + capped concurrency).

Addresses the build status portion of karlingen#1.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.

1 participant