Review Fish Wrapper Changes

Review fish wrapper changes for token injection consistency and safe argument passthrough.

scaryrawr 638dbaa 724 B Updated

File contents

Review fish wrapper changes

Use this skill for PR/code review requests touching functions/ or completions/.

Checklist

  • Wrapper functions still call the underlying binary via command <tool> $argv.
  • ARTIFACTS_ACCESSTOKEN is sourced from Azure CLI access token command.
  • Azure Artifacts resource ID remains unchanged unless requested.
  • No accidental behavior changes in write-npm.fish defaults or usage flow.
  • completions/pnpm.fish keeps placeholder token fallback and completion formatting behavior.

Focus on functional risks; skip style-only comments.

scaryrawr/az-auth.fish/tree/main/.github/skills/review-fish-wrapper-changes commit 638dbaa2c8

Frequently asked questions

npx skillmds@latest add scaryrawr/review-fish-wrapper-changes