feat/add az support by mikeyroush · Pull Request #1256 · max-sixty/worktrunk

mikeyroush · 2026-03-04T07:03:59Z

Summary

Add Azure DevOps support for pr:<N> switching and CI status integration.
Fix worktree name resolution by stripping remote prefixes in resolve_worktree_name.
Update docs to mention Azure DevOps behavior in pr:<N> help text and CI status output.

Context

This change set introduces Azure DevOps parity for PR-based workflows and CI visibility, plus a small correctness fix in worktree name normalization.

Commits included:

a8052d03 — feat: add Azure DevOps support for pr: switching and CI status
f201bd5c — fix: strip remote prefix in resolve_worktree_name
239fa127 — docs: mention Azure DevOps in pr:<N> help text and CI status

Fixes #1144

Test plan

pre-commit run --all-files passes
cargo test --test integration test_skill_files_are_in_sync_with_docs passes
Manual validation:
- wt switch pr:<N> works against Azure DevOps remotes
- CI status displays correctly for Azure-backed branches
- Existing GitHub/GitLab pr: flows remain unaffected

`wt switch origin/branch-name` now resolves to `branch-name` instead of treating the full string as a branch name. This matches `git switch` behavior where `origin/foo` creates a local tracking branch named `foo`. Previously, `wt switch -c origin/branch-name` would create a new local branch literally named `origin/branch-name` based on the default branch. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Add Azure DevOps as a first-class provider alongside GitHub and GitLab: - Detect Azure DevOps repos from remote URLs (dev.azure.com, ssh.dev.azure.com, *.visualstudio.com) - `wt switch pr:<N>` resolves Azure DevOps PRs via `az repos pr show` with auto-detection of organization from git remotes - CI status detection via `az pipelines runs list` and `az repos pr list` - Auto-detect platform from remote URLs — no configuration needed - `wt config show --full` reports `az` tool and auth status Closes max-sixty#1144 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Update switch command help to document Azure DevOps PR support, update CI status references, and sync generated docs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

worktrunk-bot

Well-structured addition that closely follows the existing GitHub/GitLab patterns. The detect_pr_provider refactor to auto-detect from remote URLs is a nice improvement over hardcoding GitHub.

A few observations:

AzPipelineRun.url — API URL vs web URL. The Azure DevOps REST API's url field on build resources is typically the API endpoint (e.g., https://dev.azure.com/{org}/{project}/_apis/build/Builds/{id}), not the web URL. The GitHub and GitLab CI modules use html_url/web_url respectively. Worth verifying that az pipelines runs list --output json gives a clickable browser link here — if not, you'd need to either extract _links.web.href from the response or construct the web URL manually (like url_from_web does for PRs).

pr:N now works on GitLab repos too — detect_pr_provider includes GitLab, so pr:42 on a GitLab-origin repo will resolve MR #42. This is a reasonable UX improvement, but the help text only says "GitHub/Azure DevOps PR". Either mention GitLab or document this as intentional.

PlatformData::AzureDevOps::host is hardcoded to "dev.azure.com" even when the repo uses a *.visualstudio.com remote. The field isn't read downstream currently, so this isn't a bug today — but it's a trap for future code. Consider either populating it from the actual remote host or removing it if it's not needed.

src/commands/list/ci_status/azure.rs

src/git/remote_ref/azure.rs

worktrunk-bot

CI is failing with two clippy::collapsible_if errors and a cargo fmt issue. Both are in PR code — see inline suggestions.

src/git/remote_ref/azure.rs

src/git/repository/worktrees.rs

- Fix PR URLs missing _git segment in Azure DevOps links - Fix doubled project name in CI status PR URL - Fix early return in detect_azure_org on blank remote lines - Collapse nested if-let per clippy collapsible_if lint - Use actual host from API response instead of hardcoded dev.azure.com - Construct web URL for pipeline runs instead of using REST API URL

worktrunk-bot

Previous feedback was well addressed — collapsible-if lints, comment wording, host detection from web_url, and the web URL construction for pipeline runs. One remaining issue with the pipeline URL.

src/commands/list/ci_status/azure.rs

- Add missing project segment to Azure pipeline web URL - Check --base conflict before detect_pr_provider() so the error fires even without a forge remote configured - Add GitHub/GitLab remote URLs to PR/MR error-path tests so detect_pr_provider() succeeds and mocks are exercised - Update snapshots for Azure DevOps additions to help text, config show, and CI platform validation messages - Collapse nested if-lets per clippy Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

worktrunk-bot

Previous feedback was well addressed — pipeline URL now includes the project segment, comment wording is fixed, host detection uses web_url, and the --base conflict check was moved before network calls (good improvement).

One remaining doc gap: the shortcuts table in after_long_help (line 298 of src/cli/mod.rs) still describes pr:{N} as "GitHub PR #N's branch", but pr: now works for Azure DevOps too. The arg help on line 397 already correctly says "GitHub/Azure DevOps PR" — the shortcuts table should match:

| `pr:{N}` | GitHub/Azure DevOps PR #N's branch |

worktrunk-bot

CI is failing on both Linux and Windows — cargo fmt reports formatting issues. This needs a cargo fmt pass on the changed files before the PR can merge.

- Run cargo fmt on all changed files - Update shortcuts table to say "GitHub/Azure DevOps PR" (was "GitHub PR") - Sync docs and update help snapshot Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

docs/content/switch.md

src/commands/worktree/switch.rs

max-sixty · 2026-03-04T21:09:31Z

src/git/repository/worktrees.rs

+    /// If `name` starts with `{remote}/` where `{remote}` is a configured remote,
+    /// returns the branch name without the prefix. This matches `git switch` behavior
+    /// where `origin/foo` creates a local tracking branch named `foo`.
+    fn strip_remote_prefix(&self, name: &str) -> Option<String> {


@worktrunk-bot can you explain the change here? does it have a wider blast radius than just Azure? (also make transparent any other changes with a blast radius bigger than Azure)

This was the issue I was trying to fix:

❯ wt switch origin/feat ✗ No branch named origin/feat ↳ To create a new branch, run wt switch --create origin/feat; to list branches, run wt list --branches --remotes

The change does affect all platforms, not just Azure — it strips origin/ prefixes from branch names when resolving worktree names. It's not related to Azure support, but seemed like low hanging fruit.

great find! this indeed was a bug. I think it should be generally fixed by #1266 ...

Sounds good. I can work on reverting this section.

- De-emphasize Azure DevOps in docs: replace full section with single line at bottom of GitHub section, revert shortcuts table - Use repo.primary_remote() instead of hardcoded "origin" in detect_pr_provider() to respect checkout.defaultRemote Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

max-sixty · 2026-03-27T00:06:10Z

@mikeyroush Did I drop this?? I'm very sorry if so...

@worktrunk-bot could you update for the latest changes, resolve merge conflicts, etc?

max-sixty · 2026-03-30T06:21:37Z

@worktrunk-bot please re the above

mroush and others added 3 commits March 4, 2026 01:02

docs: mention Azure DevOps in pr:<N> help text and CI status

1b16b48

Update switch command help to document Azure DevOps PR support, update CI status references, and sync generated docs. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

worktrunk-bot reviewed Mar 4, 2026

View reviewed changes

src/commands/list/ci_status/azure.rs Show resolved Hide resolved

src/commands/list/ci_status/azure.rs Outdated Show resolved Hide resolved

src/git/remote_ref/azure.rs Show resolved Hide resolved

worktrunk-bot reviewed Mar 4, 2026

View reviewed changes

src/git/remote_ref/azure.rs Outdated Show resolved Hide resolved

src/git/repository/worktrees.rs Outdated Show resolved Hide resolved

worktrunk-bot reviewed Mar 4, 2026

View reviewed changes

src/commands/list/ci_status/azure.rs Show resolved Hide resolved

worktrunk-bot reviewed Mar 4, 2026

View reviewed changes

style: apply cargo fmt and fix pr: shortcut table docs

6955182

- Run cargo fmt on all changed files - Update shortcuts table to say "GitHub/Azure DevOps PR" (was "GitHub PR") - Sync docs and update help snapshot Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

worktrunk-bot approved these changes Mar 4, 2026

View reviewed changes

max-sixty reviewed Mar 4, 2026

View reviewed changes

docs/content/switch.md Outdated Show resolved Hide resolved

max-sixty reviewed Mar 4, 2026

View reviewed changes

src/commands/worktree/switch.rs Outdated Show resolved Hide resolved

max-sixty reviewed Mar 4, 2026

View reviewed changes

worktrunk-bot approved these changes Mar 4, 2026

View reviewed changes

max-sixty mentioned this pull request Mar 5, 2026

[ci] duplication checks #1267

Closed

Conversation

mikeyroush commented Mar 4, 2026

Summary

Context

Test plan

Uh oh!

worktrunk-bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

worktrunk-bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

worktrunk-bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

worktrunk-bot left a comment

Choose a reason for hiding this comment

Uh oh!

worktrunk-bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

max-sixty Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

MikeyFS Mar 4, 2026

Choose a reason for hiding this comment

Uh oh!

max-sixty Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

mikeyroush Mar 5, 2026

Choose a reason for hiding this comment

Uh oh!

max-sixty commented Mar 27, 2026

Uh oh!

max-sixty commented Mar 30, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants