- name
- verification
- description
- Verification steps for code changes. Use after ANY code change to ensure quality, or when creating plans because those should include verification steps.
- review
- never
# Verification
The stop hook runs compile, lint, effect LS, test, vscode:bundle, and knip automatically when the agent stops — no need to run these manually.
Run this **only if the stop hook doesn't catch it**:
1. **Dupes** — `npm run check:dupes`, check `jscpd-report` for flagged changes
2. **GitHub Actions** (only if changing `.github/workflows/*.{yml,yaml}` or `.github/actions/*/action.{yml,yaml}`) — `npm run check:actions`
3. **Playwright** (only if working in these packages: `salesforcedx-vscode-core`, `salesforcedx-vscode-org`, `salesforcedx-vscode-services`, `salesforcedx-vscode-org-browser`, `salesforcedx-vscode-metadata`, `salesforcedx-vscode-apex-testing`, `salesforcedx-vscode-apex-log`, `playwright-vscode-ext`)
- Run from root: `npm run test:web -w <package-name> -- --retries 0` / `npm run test:desktop -w <package-name> -- --retries 0`
## Rules
- Don't change /src AND /test together (except imports/renames)
- All commands run from repo root; use `-w` for single package, never `cd` into a package
- Don't call a failure "pre-existing" without verifying on a prior commit
## Plans
Include verification steps after the "actual" todos. Follow this checklist.
## References
- `references/unit-tests.md` — unit tests
- `references/compile.md` — compile; TS4023 / TS1261 skills
- `@.claude/skills/ts4023-effect-errors/` — TS4023
- `@.claude/skills/ts1261-filename-casing/` — TS1261
- `@.claude/skills/playwright-e2e/` — Playwright E2E
- `effect-advocate` subagent — invoke after Effect-related changes to flag missed Effect idioms (schemas, Option, retry/timeout/cache, services reuse).
عرض على GitHub