docs(tdd): use backticked filename instead of @-prefix#1575
Open
gaurav0107 wants to merge 1 commit into
Open
Conversation
…icked filename The reference `@testing-anti-patterns.md` on line 359 of skills/test-driven-development/SKILL.md uses an `@` prefix that no markdown processor or skill loader resolves. The intended target file lives at skills/test-driven-development/testing-anti-patterns.md (the sibling that the surrounding section is pointing readers at). Replace the `@`-prefixed token with a plain backticked filename so the path is unambiguous and matches the citation style already used throughout this SKILL.md. Fixes obra#1529.
gaurav0107
changed the title
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Fixes #1529.
Line 359 of
skills/test-driven-development/SKILL.mdreferences@testing-anti-patterns.md. Nothing in this repo strips the@— there's no preamble convention for it, no skill loader that resolves it, and no other@-prefixed file reference in this SKILL.md. Every other internal file pointer here is either a bare filename or a backticked path, so the@reads as a mistake.The actual file lives next to the SKILL at
skills/test-driven-development/testing-anti-patterns.md.This PR replaces the
@-prefixed token with a backticked filename, matching the citation style used everywhere else in the same file:One line, one file. No behavior change — the surrounding section about mocks and test utilities is unchanged.
I checked the rest of the repo for the same pattern with
grep -nE '@[a-zA-Z0-9_-]+\.md'and found exactly one other instance:skills/writing-skills/SKILL.mdline 556 (See @testing-skills-with-subagents.md). I left that one alone — issue #1529 only names the TDD file, andwriting-skills/SKILL.mdis the kind of carefully-tuned content that probably wants its own discussion. Happy to file a follow-up if you want it fixed too.