Skip to content

docs(skills): modernize Obsidian skill to use file tools (salvage #19332)#20413

Merged
teknium1 merged 2 commits into
mainfrom
salvage/pr-19332
May 5, 2026
Merged

docs(skills): modernize Obsidian skill to use file tools (salvage #19332)#20413
teknium1 merged 2 commits into
mainfrom
salvage/pr-19332

Conversation

@teknium1
Copy link
Copy Markdown
Contributor

@teknium1 teknium1 commented May 5, 2026

Rewrites the Obsidian skill to prefer the agent's native file tools (read_file, write_file, search_files, patch) over shell commands (cat, find, grep, heredocs). Aligns with AGENTS.md guidance ("Do NOT use cat/head/tail to read files", "Do NOT use grep/rg/find to search") and avoids space/quoting pitfalls on vault paths like ~/Documents/Obsidian Vault.

Changes:

  • skills/note-taking/obsidian/SKILL.md (+32/-38)
  • website/docs/reference/skills-catalog.md (description bump, +1/-1)
  • website/docs/user-guide/skills/bundled/note-taking/note-taking-obsidian.md (auto-generated equivalent, +32/-38)
  • scripts/release.py — AUTHOR_MAP entry for counterposition

Closes #19332 via salvage.

Original PR by @counterposition — authorship preserved.

@teknium1 teknium1 merged commit 79902a0 into main May 5, 2026
8 of 11 checks passed
@teknium1 teknium1 deleted the salvage/pr-19332 branch May 5, 2026 20:51
@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) tool/file File tools (read, write, patch, search) labels May 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

P3 Low — cosmetic, nice to have tool/file File tools (read, write, patch, search) tool/skills Skills system (list, view, manage) type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants