Skip to content

fix(writing-skills): replace @ reference with markdown link#631

Open
stablegenius49 wants to merge 1 commit intoobra:mainfrom
stablegenius49:pr-factory/issue-280-writing-skills-link
Open

fix(writing-skills): replace @ reference with markdown link#631
stablegenius49 wants to merge 1 commit intoobra:mainfrom
stablegenius49:pr-factory/issue-280-writing-skills-link

Conversation

@stablegenius49
Copy link

Summary

  • replace @testing-skills-with-subagents.md with a standard markdown link in skills/writing-skills/SKILL.md
  • keep the guidance consistent with earlier advice in the same skill to avoid force-loading via @ references

Fixes #280

Validation

  • ran a local markdown link check for skills/writing-skills/SKILL.md to confirm the linked file resolves (links_checked=1 missing=0)

@coderabbitai
Copy link

coderabbitai bot commented Mar 6, 2026

📝 Walkthrough

Walkthrough

The change replaces an @ syntax file reference with a standard Markdown hyperlink in the Testing methodology section, aligning the file reference style with official Anthropic documentation conventions.

Changes

Cohort / File(s) Summary
Documentation link formatting
skills/writing-skills/SKILL.md
Updated file reference from @ syntax to Markdown link format in Testing methodology section for consistency and adherence to official documentation standards.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A link was dressed in fancy @ attire,
But Markdown brackets raised it higher,
Now testing skills shine bright,
Consistent style, perfectly right! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: replacing an @ reference with a markdown link in the writing-skills file.
Description check ✅ Passed The description is directly related to the changeset, explaining the replacement of the @ reference with a markdown link and its purpose.
Linked Issues check ✅ Passed The PR fully addresses issue #280 by replacing the @ reference with a standard markdown link as specified in the linked issue.
Out of Scope Changes check ✅ Passed The changes are scoped to the specific requirement: only the @ reference syntax is replaced with markdown link syntax in the writing-skills file.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
skills/writing-skills/SKILL.md (1)

316-316: ⚠️ Potential issue | 🟡 Minor

Missed @ reference: Apply the same fix here.

Line 316 still uses @graphviz-conventions.dot, which is the same @ syntax issue that was fixed at line 556. For consistency with the file's own guidance (lines 286-288) and the fix applied elsewhere, this should also be changed to a markdown link.

📝 Suggested fix
-See `@graphviz-conventions.dot` for graphviz style rules.
+See [graphviz-conventions.dot](graphviz-conventions.dot) for graphviz style rules.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@skills/writing-skills/SKILL.md` at line 316, Replace the literal @ reference
"@graphviz-conventions.dot" with a proper markdown link consistent with the
other fixes (e.g., [graphviz-conventions.dot](graphviz-conventions.dot)) so the
reference follows the file's guidance and matches the change already made
elsewhere; locate the occurrence of "@graphviz-conventions.dot" in SKILL.md and
update it to a markdown link.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@skills/writing-skills/SKILL.md`:
- Line 316: Replace the literal @ reference "@graphviz-conventions.dot" with a
proper markdown link consistent with the other fixes (e.g.,
[graphviz-conventions.dot](graphviz-conventions.dot)) so the reference follows
the file's guidance and matches the change already made elsewhere; locate the
occurrence of "@graphviz-conventions.dot" in SKILL.md and update it to a
markdown link.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: c8dff73b-7116-4f1b-a0c4-0b85dbf86879

📥 Commits

Reviewing files that changed from the base of the PR and between e4a2375 and 2611d44.

📒 Files selected for processing (1)
  • skills/writing-skills/SKILL.md

IgorTavcar added a commit to IgorTavcar/superpowers that referenced this pull request Mar 9, 2026
Upstream PR: obra#631

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
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.

writing-skills: Inconsistent file reference syntax (@ vs markdown links)

1 participant