Skip to content

Feedback on your automating-notes skill #5

@RichardHightower

Description

@RichardHightower

I took a look at your automating-notes skill and wanted to share some thoughts.

Links:

The TL;DR

You're at 96/100, solid A grade. This is based on Anthropic's Claude Skills best practices framework. Your strongest area is Spec Compliance (14/15) — the YAML frontmatter is clean, naming conventions are spot-on, and your description nails the trigger phrases. The weakest area is Writing Style (8/10) — mostly minor tense inconsistencies that are easy wins.

What's Working Well

  • Progressive Disclosure nailed it (26/30). Your 5-file reference structure is chef's kiss — basics, recipes, advanced, dictionary, and PyXA API are perfectly organized with clear separation of concerns. Developers land in SKILL.md and find exactly what they need without confusion.

  • Utility is genuinely strong (17/20). You're solving a real problem. Notes automation in JXA isn't well-documented, and you cover three different approaches (JXA, PyXA, PyObjC). The input/output examples in the quickstart plus your template-driven folder paths for meetings/people show you've actually thought about how people use this.

  • Spec Compliance is basically perfect (14/15). Valid YAML, correct hyphen-case naming, solid third-person description. You're using allowed-tools too, which shows attention to detail.

The Big One: Redundant Code Between Files

Your quickstart in SKILL.md duplicates the full ensurePath function (~30 lines) that also appears in notes-recipes.md. This is burning tokens unnecessarily and creates a maintenance headache — if that function needs updating, you have to remember both places.

The fix: Keep a minimal working example in SKILL.md (just the function signature and one-liner for what it does), then say "See notes-recipes.md for the full implementation." This'll save ~20 lines and gets you +2 points on token economy. Easy win.

Other Things Worth Fixing

  1. Second-person tense in "When Not to Use" (lines 179-184). You've got "use Notion API..." but the rest of your skill stays in imperative third-person. Change it to: "Cross-platform note taking → use Notion API, Obsidian, or Markdown files." Keeps the voice consistent. +1 point.

  2. Missing TOC in notes-recipes.md. That file is 152 lines. Add a quick "Contents" section at the top linking to: Ensure folder path, Create note, Query, Move, Checklist, People dossiers. Helps developers scan faster. +1 point.

  3. Feedback loops could be more explicit. You have validation checklists and try/catch patterns, which is good. Consider adding a step-by-step "Run → Check → Fix" pattern in the advanced section so people know how to debug when something breaks. +2 points possible.

Quick Wins

  • Consolidate ensurePath function to SKILL.md only (reference recipes for full code) → +2 points
  • Fix tense in "When Not to Use" section → +1 point
  • Add table of contents to notes-recipes.md → +1 point
  • Tighten up feedback loop documentation → +2 points

That gets you to 99/100 with minimal effort. The redundancy fix is the highest bang for your buck.


Checkout your skill here: [SkillzWave.ai](https://skillzwave.ai) | [SpillWave](https://spillwave.com) We have an agentic skill installer that install skills in 14+ coding agent platforms. Check out this guide on how to improve your agentic skills.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions