Skip to content

docs(skill): emphasize --from flag for branch-based work#312

Merged
basnijholt merged 6 commits intomainfrom
clear-from
Jan 25, 2026
Merged

docs(skill): emphasize --from flag for branch-based work#312
basnijholt merged 6 commits intomainfrom
clear-from

Conversation

@basnijholt
Copy link
Owner

Summary

  • Added prominent guidance about using --from HEAD when spawning agents to review/test existing branches
  • Added "Scenario 0: Code review of current branch" as the first example in the skill
  • Includes comprehensive review criteria (DRY, KISS, YAGNI, no pointless wrappers, etc.)

Problem

When Claude uses the agent-cli-dev skill to spawn an agent for reviewing the current branch, it often forgets to pass --from HEAD. Without this flag, the new worktree starts from origin/main and doesn't contain the changes to review.

Changes

SKILL.md:

  • Added "IMPORTANT: Using --from for branch-based work" section
  • Clear examples showing --from HEAD for review tasks
  • Explains when to use vs. not use --from

examples.md:

  • Added "Scenario 0: Code review of current branch" as the first example
  • Includes comprehensive review focus criteria
  • Highlights the common mistake of forgetting --from

Test plan

  • Verify skill files are in sync (pre-commit hook validates this)
  • Test that Claude uses --from HEAD when asked to review current branch

…-dev skill

When spawning agents to review or test an existing branch, the --from flag
is required to base the worktree off that branch. Without it, the worktree
starts from origin/main and won't have the changes to review.

Added:
- "IMPORTANT: Using --from for branch-based work" section in SKILL.md
- "Scenario 0: Code review of current branch" example in examples.md
- Comprehensive review criteria (DRY, KISS, YAGNI, etc.) in the example
@basnijholt basnijholt merged commit 7a8c37e into main Jan 25, 2026
10 checks passed
@basnijholt basnijholt deleted the clear-from branch January 25, 2026 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant