Standardize .claude/commands to the canonical lab dev-command set

[eberrigan]

Standardize .claude/commands to the canonical lab dev-command set

Two .claude/commands lineages have diverged across lab repos:

• descriptive (sleap-roots-analyze): pre-merge-check, update-changelog, prepare-release, cleanup-merged
• short (sleap-roots, this repo / salk-bloom / bloom-desktop): pre-merge, changelog, release, cleanup-branch

Decision: standardize on descriptive (verb-first) names with best-of content (adopt the richer command body where the lineages differ). Several of this repo's commands (fix-formatting, validate-env, new-feature, copilot-review, and the 8-phase pre-merge) are the canonical source bodies — they're richer than analyze's and should be promoted (under descriptive names).

Canonical set (repo-agnostic)

Group	Commands
Core loop	tdd · lint · fix-formatting · coverage · run-ci-locally · validate-env
Feature/PR	new-feature · review-pr · copilot-review · pr-description · review-openspec · pre-merge-check
Release / housekeeping	update-changelog · prepare-release · cleanup-merged
OpenSpec	proposal · apply · archive

Drop: bare black; generate-pr-review.
Keep per-repo: domain commands (benchmark-pr, new-pipeline, debug-test, test, docs-update, investigate) stay — repo-specific, not part of the canonical generic set.
Source of truth for new repos: the scaffolding-lab-python-repo skill.

Deltas for this repo (sleap-roots)

Already aligned/canonical-source: fix-formatting, validate-env, new-feature, copilot-review, cleanup-merged, lint, coverage, run-ci-locally, review-pr, pr-description.

• Rename pre-merge → pre-merge-check (keep its richer 8-phase body — this is the canonical content).
• Rename changelog → update-changelog.
• Add prepare-release — source: sleap-roots-analyze.
• Add review-openspec — source: sleap-roots-analyze.
• Add tdd (the red→green→refactor workflow) — source: sleap-roots-analyze. (test / debug-test are complementary run/debug commands; keep them as repo-specific.)
• Leave a thin alias or note for the renamed commands if muscle memory matters.

[eberrigan]

Clarification — canonical review-pr content (found while implementing this standard).

The canonical review-pr is the subagent-team reviewer (the sleap-roots version: launches parallel adversarial review lenses, synthesizes, posts a verdict) — not the thin comment-fetcher in sleap-roots-analyze / sleap-roots-contracts. The comment-fetch role is already covered by copilot-review, so the fetcher review-pr is redundant.

Make the canonical review-pr repo-agnostic with two modes:

• PR number → review the PR diff, post a review (own-PR → fall back to a comment);
• no PR / branch name → review git diff $(git merge-base origin/main HEAD)..HEAD and report only (post nothing to GitHub).

This also fixes an inaccuracy in pre-merge-check Phase 3.5: it described a pre-PR local-diff review that the fetcher review-pr could not do. With the two-mode command, Phase 3.5 uses the report-only local mode pre-PR, and Phase 6 uses PR mode after the PR exists. Source domain-specific review concerns from openspec/project.md rather than hardcoding them.

Reference implementation: talmolab/sleap-roots-contracts → .claude/commands/review-pr.md + pre-merge-check.md (Phase 3.5).