# Review Fish Wrapper Changes

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

- Skill: `scaryrawr/review-fish-wrapper-changes` (Agent Skill)
- Install (CLI): `npx skillmds@latest add scaryrawr/review-fish-wrapper-changes`
- Raw SKILL.md: https://api.skillmd.com/api/skills/scaryrawr/review-fish-wrapper-changes/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: scaryrawr (https://skillmd.com/u/scaryrawr)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/scaryrawr/review-fish-wrapper-changes

---


# 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.