Skip to content

Commit 48d9d48

Browse files
committed
fix: narrow markdownlint ignore and correct skill references in ship command
- .markdownlint-cli2.yaml: Replace overly broad .claude/commands/** ignore with specific list of 6 new lifecycle command files. This preserves full linting for pre-existing command files (push-pr.md, pr-review.md, validate-pr-description.md, etc.) - .claude/commands/ship.md: Correct references to validate-pr-description and push-pr. These are slash commands (.claude/commands/*.md), not skills (.claude/skills/*/SKILL.md). Update instructions to use command invocation syntax instead of Skill tool references.
1 parent 043db08 commit 48d9d48

2 files changed

Lines changed: 11 additions & 5 deletions

File tree

.claude/commands/ship.md

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ argument-hint: [target-branch]
66

77
@CLAUDE.md
88

9-
Invoke the pipeline-validator and validate-pr-description skills.
9+
Invoke the pipeline-validator skill.
1010

1111
Ship the current branch: $ARGUMENTS
1212

@@ -30,8 +30,8 @@ Pre-flight checks (all must pass):
3030

3131
1. Run pre-flight checks (all 5 above)
3232
2. If any check fails: report what failed, why, and how to fix. Stop.
33-
3. If all pass: validate PR description (invoke validate-pr-description skill)
34-
4. Create PR via /push-pr delegation (invoke Skill tool with push-pr)
33+
3. If all pass: validate PR description (run /validate-pr-description command)
34+
4. Create PR via /push-pr command
3535
5. Report: what shipped, PR link, any warnings
3636

3737
## Output

.markdownlint-cli2.yaml

Lines changed: 8 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -138,5 +138,11 @@ ignores:
138138
# HTML tags, violating MD033 and MD041. These are tool-managed metadata, not authored markdown.
139139
- "**/CLAUDE.md"
140140

141-
# Slash command files have YAML frontmatter, not headings. MD041 not applicable.
142-
- ".claude/commands/**"
141+
# New lifecycle command files have only YAML frontmatter, no H1 heading. MD041 not applicable.
142+
# Only the 6 new lifecycle commands are excluded; other command files retain full linting.
143+
- ".claude/commands/spec.md"
144+
- ".claude/commands/plan.md"
145+
- ".claude/commands/build.md"
146+
- ".claude/commands/test.md"
147+
- ".claude/commands/review.md"
148+
- ".claude/commands/ship.md"

0 commit comments

Comments
 (0)