fix(skill/sandboxes): use canonical aka.ms install URLs and aca auth login#1732
Open
paulyuk wants to merge 1 commit into
Open
fix(skill/sandboxes): use canonical aka.ms install URLs and aca auth login#1732paulyuk wants to merge 1 commit into
paulyuk wants to merge 1 commit into
Conversation
…login Switches plugin/skills/sandboxes/references/install.md to: - https://aka.ms/aca-cli-install (Linux/macOS) and https://aka.ms/aca-cli-install-ps (Windows) instead of the long raw.githubusercontent.com URLs. Both aka.ms shortcuts already redirect to the same install scripts (verified 200/301), so behaviour is identical but the user-facing URL is what the public ACA Sandboxes docs and ground-truth references use. - aca auth login as the canonical CLI entry point, with an explicit note that it delegates to az login under the hood. The skill previously told users to run az login directly, which works but bypasses the CLI's own auth verb and is inconsistent with the docs. Verified locally: aca auth login is a real subcommand on aca 1.0.0-beta.1 (Commands: login | status | help, 'Log in to Azure (delegates to az login)'). Why: surfaced by the ACA Sandboxes Vally eval suite (Run 3) where pos-install-aca-{linux,macos,windows} all failed output-contains for 'https://aka.ms/aca-cli-install' and 'aca auth login' against the docs ground truth. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Switches
plugin/skills/sandboxes/references/install.mdto the canonical install surface documented in the ACA Sandboxes ground-truth docs.Changes
curl ... raw.githubusercontent.com/.../install.sh | shcurl -fsSL https://aka.ms/aca-cli-install | shirm raw.githubusercontent.com/.../install.ps1 | iexirm https://aka.ms/aca-cli-install-ps | iexaz loginaca auth login(delegates toaz loginunder the hood)Both aka.ms shortcuts already redirect to the same install scripts (verified
301 \u2192 raw.githubusercontent.com/.../install.{sh,ps1}), so behavior is identical \u2014 only the user-facing URL changes.aca auth loginis a real subcommand onaca 1.0.0-beta.1(Commands: login | status | help, Log in to Azure (delegates to az login)).Why
Surfaced by the ACA Sandboxes Vally eval suite (Run 3):
pos-install-aca-linux,pos-install-aca-macos,pos-install-aca-windowsall failedoutput-containsgraders looking forhttps://aka.ms/aca-cli-installandaca auth loginagainst the docs ground truth. Eval suite \u2192coreai-microsoft/adc-devx#214(EMU).Risk
Zero. Behavior identical; only the URL surface and auth verb change.
az loginstill works for users who use it directly.Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com