Skip to content

docs(skills): modernize Obsidian wrapper file workflows#19332

Closed
counterposition wants to merge 1 commit into
NousResearch:mainfrom
counterposition:docs/obsidian-file-tools
Closed

docs(skills): modernize Obsidian wrapper file workflows#19332
counterposition wants to merge 1 commit into
NousResearch:mainfrom
counterposition:docs/obsidian-file-tools

Conversation

@counterposition

Copy link
Copy Markdown
Contributor

Rationale

The bundled Obsidian skill still taught shell-first workflows for common vault operations. Hermes's file-tool schemas already recommend read_file instead of cat, write_file instead of heredocs, patch instead of shell text rewrites, and search_files instead of grep/find/ls.

This updates the skill so Obsidian vault work follows the same native file-tool guidance as the rest of Hermes.

Implementation

  • Rewrites the Obsidian wrapper guidance around resolved vault paths and Hermes-native file tools.
  • Preserves OBSIDIAN_VAULT_PATH, the fallback ~/Documents/Obsidian Vault, and the existing read/list/search/create/append/wikilink workflows.
  • Clarifies that file tools need concrete resolved paths, not paths containing $OBSIDIAN_VAULT_PATH.
  • Clarifies search_files usage for filename glob searches versus content regex searches with file_glob: "*.md".
  • Adds a short patch append idiom: replace the anchor with the anchor plus new content.
  • Regenerates the checked-in skill docs and catalog entry from website/scripts/generate-skill-docs.py.

User-visible benefit

Agents working in Obsidian vaults now get line-numbered and paginated reads, predictable JSON tool results, and fewer shell-quoting pitfalls for vault paths with spaces. The file tools also enforce write safety and stale-write checks at call time.

Risk

Low. This is a documentation/skill-instruction-only change. The main trade-off is that resolving the vault path can be a small extra step when the path is unknown.

Validation

  • Ran uv run python website/scripts/generate-skill-docs.py.
  • Ran git diff --check.

Note: running the skill-doc generator also currently surfaces unrelated optional-skill docs drift for kanban-video-orchestrator; that generated churn was intentionally left out of this PR.

@counterposition counterposition marked this pull request as ready for review May 3, 2026 19:24
@alt-glitch alt-glitch added type/docs Documentation improvements P3 Low — cosmetic, nice to have tool/skills Skills system (list, view, manage) labels May 3, 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/skills Skills system (list, view, manage) type/docs Documentation improvements

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants