Skip to content

fix: resolve Azure CLI on Windows by using az.cmd - #9215

Merged
DOsinga merged 1 commit into
mainfrom
fix/6738-az-cmd-windows
May 14, 2026
Merged

fix: resolve Azure CLI on Windows by using az.cmd#9215
DOsinga merged 1 commit into
mainfrom
fix/6738-az-cmd-windows

Conversation

@DOsinga

@DOsinga DOsinga commented May 14, 2026

Copy link
Copy Markdown
Collaborator

On Windows, Azure CLI is installed as az.cmd rather than az.exe. Rust's Command::new("az") cannot resolve .cmd extensions, unlike cmd.exe and PowerShell which do this transparently.

Uses cfg!(windows) to select az.cmd on Windows and az elsewhere.

Fixes #6738

On Windows, Azure CLI is installed as az.cmd rather than az.exe.
Rust's Command::new("az") cannot resolve .cmd extensions, unlike
cmd.exe and PowerShell which do this transparently.

Fixes #6738

Signed-off-by: Douwe Osinga <douwe@squareup.com>
@DOsinga
DOsinga force-pushed the fix/6738-az-cmd-windows branch from a0dd409 to bfc0f00 Compare May 14, 2026 17:03
@DOsinga
DOsinga added this pull request to the merge queue May 14, 2026
Merged via the queue into main with commit 6b47e01 May 14, 2026
23 checks passed
@DOsinga
DOsinga deleted the fix/6738-az-cmd-windows branch May 14, 2026 17:46
lifeizhou-ap added a commit that referenced this pull request May 15, 2026
* main: (102 commits)
  Dynamically refresh skill instructions each turn (#9217)
  Build non-vulkan linux variants using ubuntu 22.04 (#9211)
  fix(ui): show tool name in approval prompt (#9216)
  feat: add Atomic Chat as declarative OpenAI-compatible provider (#9210)
  chore: bump package.json versions from 0.19.1 to 0.20.0 (#9218)
  feat: support GOOSE_OAUTH_CALLBACK_PORT for stable OAuth redirect_uri (#9209)
  [RFC] feat(oauth): proactive token refresh to avoid re-auth on every session (#8386)
  fix: resolve Azure CLI on Windows by using az.cmd (#9215)
  fix: handle non-interactive terminal in goose configure on Windows (#9214)
  Better parsing of pasted html as markdown so agents understand (#9190)
  fix: persist accumulated cost in session DB to survive reload (#9191)
  fix(publish-npm): build binary from current SHA + add compat check (#9212)
  feat(desktop): add goose://new-session deep link to open fresh chat (#9196)
  Add PR previews using cloudflare pages (#9208)
  fix: prevent tool-use marker leakage in toolshim output (#8310)
  Prompt injection mitigation: update pattern-based detection (#9198)
  remove goose2 related skills (#9189)
  Switch GH pages deploy to actions/artifact workflow (#9025)
  fix(summon): re-apply canonical limits when delegate overrides model (#9183)
  Split code signing from build (#8587)
  ...
shafqatevo pushed a commit to shafqatevo/goose that referenced this pull request Aug 7, 2026
Signed-off-by: Douwe Osinga <douwe@squareup.com>
Co-authored-by: Douwe Osinga <douwe@squareup.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.

Azure Entra ID authentication do not works

2 participants