Skip to content

Fixup claude skill#1

Merged
yarikoptic merged 3 commits intoyarikoptic:claude-skillfrom
asmacdo:fixup-claude-skill
Nov 11, 2025
Merged

Fixup claude skill#1
yarikoptic merged 3 commits intoyarikoptic:claude-skillfrom
asmacdo:fixup-claude-skill

Conversation

@asmacdo
Copy link
Copy Markdown

@asmacdo asmacdo commented Nov 11, 2025

When I attempted to use this skill with claude-code it did not show up due to not following the spec https://code.claude.com/docs/en/skills (Note: that spec is specifically for claude-code, hopefully this format works with both claude-code and the browser)

After conforming to the spec, I used this skill to proof (see 2nd commit message detail for prompt), and it operated correctly, and found a broken file path (broken by my change). The output format was nice though I did make a small change to that (commit 3)

asmacdo and others added 3 commits November 11, 2025 12:55
The file path reference was incorrect - it pointed to
`/.claude/skills/proof-spec.md` but the actual file is located at
`.claude/skills/proof-spec/SKILL.md`.

Original prompt: use proof-spec skill to review all changes since 373da35

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
Updated the output format template to include filename before line
numbers (e.g., `filename:line X` instead of just `line X`) to make
issue locations more immediately identifiable.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@yarikoptic
Copy link
Copy Markdown
Owner

Thank you @asmacdo !

@yarikoptic yarikoptic merged commit cc51733 into yarikoptic:claude-skill Nov 11, 2025
10 checks passed
yarikoptic pushed a commit that referenced this pull request Feb 24, 2026
[ENH] Update to BIDS-Prov specification pull request
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.

2 participants