Ably Branch Review

This skill reviews only the changes on the current branch (compared to main) against the conventions defined in the ably-new-command skill. It uses a combination of LSP for structural analysis (type hierarchies, call chains) and grep/read for text-pattern checks, choosing the right tool for each check.

Unlike ably-codebase-review which sweeps the entire codebase, this skill is scoped to what has changed on the current branch, making it fast and focused for pre-PR review.

Tool selection guide

Use LSP when you need type-system awareness:

• goToDefinition — resolve what base class a command actually extends
• findReferences — trace all callers of a method through inheritance
• hover — reveal resolved types, inherited members, full type hierarchy
• documentSymbol — list all methods/properties in a class structurally