Add ai-tools-setup, computer-health-check, knowledge-base, obsidian-v…#530
Conversation
…ault, and terminal skills Add new skills to root skills/ and both claude and cursor plugins. Bump claude plugin version to 0.1.0 and update plugin READMEs. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
📝 WalkthroughWalkthroughAdds five new agent skill specifications ( ChangesSkills and Plugin Documentation Expansion
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (7)
plugins/claude/skills/knowledge-base/SKILL.md (1)
45-53: 📐 Maintainability & Code Quality | 🔵 TrivialAdd language specifier to fenced code block.
The directory tree block lacks a language specifier (MD040). Add
text:+```text
knowledge-base/
INDEX.md # entry point: instructions + full note registry
topics/
/
_topic.md # topic map: what this topic covers + its notes
.md # one atomic note
assets/ # images / attachments referenced by notes🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@plugins/claude/skills/knowledge-base/SKILL.md` around lines 45 - 53, The directory tree example in the knowledge-base guide is a fenced code block without a language tag, which triggers MD040. Update the fenced block in SKILL.md to use the text language specifier on the opening fence for the tree example, keeping the existing content and structure unchanged.Source: Linters/SAST tools
skills/terminal/SKILL.md (1)
88-94: 📐 Maintainability & Code Quality | 🔵 TrivialAdd language specifier to fenced code block.
The code block at line 88 lacks a language specifier, which triggers markdownlint (MD040). Since this is pseudo-code/example sequence, add
textorshell:+```text
start_process("python3 -i") # or "node -i", "ssh user@host", "psql ..."
interact_with_process(pid, "import pandas as pd")
interact_with_process(pid, "df = pd.read_csv('/abs/path/data.csv')")<details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.In
@skills/terminal/SKILL.mdaround lines 88 - 94, The fenced example in the
SKILL.md terminal section is missing a language tag, triggering markdownlint
MD040. Update the code block around the interactive process example to use a
specifier such as text or shell so the snippet remains clearly labeled while
still presenting the pseudo-code sequence.</details> <!-- cr-comment:v1:9717e45c12f3b169eb63d5a1 --> _Source: Linters/SAST tools_ </blockquote></details> <details> <summary>plugins/claude/skills/obsidian-vault/SKILL.md (1)</summary><blockquote> `154-162`: _📐 Maintainability & Code Quality_ | _🔵 Trivial_ **Add language specifier to fenced code block.** Same issue as `skills/obsidian-vault/SKILL.md` — the directory tree block lacks a language specifier (MD040). Add `text`: ```diff
00-inbox/ # unsorted captures 10-notes/ # atomic notes 20-mocs/ # maps of content 30-projects/ 90-assets/ # images/attachments 99-archive/ templates/<details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.In
@plugins/claude/skills/obsidian-vault/SKILL.mdaround lines 154 - 162, The
directory tree fenced block in the Obsidian vault skill markdown is missing a
language specifier and should be updated to use a text fence for MD040
compliance. Locate the fenced block in SKILL.md and change the opening fence to
include the text language tag so the tree content is properly marked as plain
text.</details> <!-- cr-comment:v1:6498fd77ba1081c0cf1dc50b --> _Source: Linters/SAST tools_ </blockquote></details> <details> <summary>plugins/claude/skills/terminal/SKILL.md (1)</summary><blockquote> `88-94`: _📐 Maintainability & Code Quality_ | _🔵 Trivial_ **Add language specifier to fenced code block.** Same issue as `skills/terminal/SKILL.md` — the code block lacks a language specifier (MD040). Add `text`: ```diff+```text
start_process("python3 -i") # or "node -i", "ssh user@host", "psql ..."
interact_with_process(pid, "import pandas as pd")
interact_with_process(pid, "df = pd.read_csv('/abs/path/data.csv')")<details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.In
@plugins/claude/skills/terminal/SKILL.mdaround lines 88 - 94, The fenced
code block in the terminal skill docs is missing a language specifier, so update
the Markdown fence to use text for the example shown in SKILL.md. Locate the
code block containing start_process and interact_with_process, and change the
opening fence so it is explicitly annotated as text to satisfy MD040.</details> <!-- cr-comment:v1:b8615632594788f5b59421bb --> _Source: Linters/SAST tools_ </blockquote></details> <details> <summary>plugins/cursor/skills/knowledge-base/SKILL.md (1)</summary><blockquote> `45-53`: _📐 Maintainability & Code Quality_ | _🔵 Trivial_ **Add language specifier to fenced code block.** Same issue as `plugins/claude/skills/knowledge-base/SKILL.md` — the directory tree block lacks a language specifier (MD040). Add `text`: ```diff+```text
knowledge-base/
INDEX.md # entry point: instructions + full note registry
topics/
/
_topic.md # topic map: what this topic covers + its notes
.md # one atomic note
assets/ # images / attachments referenced by notes🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In `@plugins/cursor/skills/knowledge-base/SKILL.md` around lines 45 - 53, The directory tree example in the knowledge-base SKILL.md is missing a fenced code block language specifier, causing the MD040 lint issue. Update the fenced block in the knowledge-base section to use a plain text specifier on the opening fence so the tree remains readable; this should be done in the SKILL.md markdown block that describes the knowledge-base layout.Source: Linters/SAST tools
plugins/cursor/skills/terminal/SKILL.md (1)
88-94: 📐 Maintainability & Code Quality | 🔵 TrivialAdd language specifier to fenced code block.
Same issue as
skills/terminal/SKILL.md— the code block lacks a language specifier (MD040). Addtext:+```text
start_process("python3 -i") # or "node -i", "ssh user@host", "psql ..."
interact_with_process(pid, "import pandas as pd")
interact_with_process(pid, "df = pd.read_csv('/abs/path/data.csv')")<details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.In
@plugins/cursor/skills/terminal/SKILL.mdaround lines 88 - 94, The fenced
example in the terminal skill doc is missing a language tag, triggering MD040.
Update the fenced block in the relevant example section to use a language
specifier like text so the markdown lint passes, and keep the existing commands
and structure unchanged. Use the fenced code block in the terminal SKILL.md
content as the target for this fix.</details> <!-- cr-comment:v1:62f3a8630a005c029034622a --> _Source: Linters/SAST tools_ </blockquote></details> <details> <summary>skills/obsidian-vault/SKILL.md (1)</summary><blockquote> `154-162`: _📐 Maintainability & Code Quality_ | _🔵 Trivial_ **Add language specifier to fenced code block.** The directory tree block lacks a language specifier (MD040). Add `text`: ```diff
00-inbox/ # unsorted captures 10-notes/ # atomic notes 20-mocs/ # maps of content 30-projects/ 90-assets/ # images/attachments 99-archive/ templates/<details> <summary>🤖 Prompt for AI Agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.In
@skills/obsidian-vault/SKILL.mdaround lines 154 - 162, Add a language
specifier to the fenced directory-tree block in the SKILL.md example so it
satisfies MD040; update the fenced block that shows the 00-inbox through
templates structure to use a text code fence, keeping the existing content
unchanged.</details> <!-- cr-comment:v1:c6a45aa5ab7408713d917d2c --> _Source: Linters/SAST tools_ </blockquote></details> </blockquote></details> <details> <summary>🤖 Prompt for all review comments with AI agents</summary>Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.Nitpick comments:
In@plugins/claude/skills/knowledge-base/SKILL.md:
- Around line 45-53: The directory tree example in the knowledge-base guide is a
fenced code block without a language tag, which triggers MD040. Update the
fenced block in SKILL.md to use the text language specifier on the opening fence
for the tree example, keeping the existing content and structure unchanged.In
@plugins/claude/skills/obsidian-vault/SKILL.md:
- Around line 154-162: The directory tree fenced block in the Obsidian vault
skill markdown is missing a language specifier and should be updated to use a
text fence for MD040 compliance. Locate the fenced block in SKILL.md and change
the opening fence to include the text language tag so the tree content is
properly marked as plain text.In
@plugins/claude/skills/terminal/SKILL.md:
- Around line 88-94: The fenced code block in the terminal skill docs is missing
a language specifier, so update the Markdown fence to use text for the example
shown in SKILL.md. Locate the code block containing start_process and
interact_with_process, and change the opening fence so it is explicitly
annotated as text to satisfy MD040.In
@plugins/cursor/skills/knowledge-base/SKILL.md:
- Around line 45-53: The directory tree example in the knowledge-base SKILL.md
is missing a fenced code block language specifier, causing the MD040 lint issue.
Update the fenced block in the knowledge-base section to use a plain text
specifier on the opening fence so the tree remains readable; this should be done
in the SKILL.md markdown block that describes the knowledge-base layout.In
@plugins/cursor/skills/terminal/SKILL.md:
- Around line 88-94: The fenced example in the terminal skill doc is missing a
language tag, triggering MD040. Update the fenced block in the relevant example
section to use a language specifier like text so the markdown lint passes, and
keep the existing commands and structure unchanged. Use the fenced code block in
the terminal SKILL.md content as the target for this fix.In
@skills/obsidian-vault/SKILL.md:
- Around line 154-162: Add a language specifier to the fenced directory-tree
block in the SKILL.md example so it satisfies MD040; update the fenced block
that shows the 00-inbox through templates structure to use a text code fence,
keeping the existing content unchanged.In
@skills/terminal/SKILL.md:
- Around line 88-94: The fenced example in the SKILL.md terminal section is
missing a language tag, triggering markdownlint MD040. Update the code block
around the interactive process example to use a specifier such as text or shell
so the snippet remains clearly labeled while still presenting the pseudo-code
sequence.</details> --- <details> <summary>ℹ️ Review info</summary> <details> <summary>⚙️ Run configuration</summary> **Configuration used**: defaults **Review profile**: CHILL **Plan**: Pro **Run ID**: `7ec69d4d-6758-4eb7-8009-ce9cf94bdb6b` </details> <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between 9cd9cb4f0bcca09a2df91a369ce2f432dd85c0d3 and 5a7cf5eb3a7988865f6ac5628d2cf20e255b8a2e. </details> <details> <summary>📒 Files selected for processing (36)</summary> * `plugins/claude/.claude-plugin/plugin.json` * `plugins/claude/README.md` * `plugins/claude/skills/ai-tools-setup/SKILL.md` * `plugins/claude/skills/ai-tools-setup/references/claude-desktop-mcp.md` * `plugins/claude/skills/ai-tools-setup/references/hermes.md` * `plugins/claude/skills/ai-tools-setup/references/openclaw.md` * `plugins/claude/skills/computer-health-check/SKILL.md` * `plugins/claude/skills/computer-health-check/references/linux.md` * `plugins/claude/skills/computer-health-check/references/macos.md` * `plugins/claude/skills/computer-health-check/references/windows.md` * `plugins/claude/skills/knowledge-base/SKILL.md` * `plugins/claude/skills/obsidian-vault/SKILL.md` * `plugins/claude/skills/terminal/SKILL.md` * `plugins/cursor/README.md` * `plugins/cursor/skills/ai-tools-setup/SKILL.md` * `plugins/cursor/skills/ai-tools-setup/references/claude-desktop-mcp.md` * `plugins/cursor/skills/ai-tools-setup/references/hermes.md` * `plugins/cursor/skills/ai-tools-setup/references/openclaw.md` * `plugins/cursor/skills/computer-health-check/SKILL.md` * `plugins/cursor/skills/computer-health-check/references/linux.md` * `plugins/cursor/skills/computer-health-check/references/macos.md` * `plugins/cursor/skills/computer-health-check/references/windows.md` * `plugins/cursor/skills/knowledge-base/SKILL.md` * `plugins/cursor/skills/obsidian-vault/SKILL.md` * `plugins/cursor/skills/terminal/SKILL.md` * `skills/ai-tools-setup/SKILL.md` * `skills/ai-tools-setup/references/claude-desktop-mcp.md` * `skills/ai-tools-setup/references/hermes.md` * `skills/ai-tools-setup/references/openclaw.md` * `skills/computer-health-check/SKILL.md` * `skills/computer-health-check/references/linux.md` * `skills/computer-health-check/references/macos.md` * `skills/computer-health-check/references/windows.md` * `skills/knowledge-base/SKILL.md` * `skills/obsidian-vault/SKILL.md` * `skills/terminal/SKILL.md` </details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
2 participants
Summary by CodeRabbit