Skip to content

Feedback on your working-with-documents skill #33

@RichardHightower

Description

@RichardHightower

I noticed your skill tackles document handling in a pretty comprehensive way—definitely a practical tool for developers who work across different file formats. That said, with a C-grade score of 74/100, there's some solid room to strengthen the architecture and make the guidance even tighter.

Links:

The TL;DR

You're sitting at 74/100, solidly in C territory. This is based on Anthropic's best practices for skill design. Your strongest area is Ease of Use (21/25)—the metadata and triggers are clear, and the workflow structure works well. The weakness? Progressive Disclosure Architecture (18/30) is dragging you down. Your 272-line file needs better organization and layered structure.

What's Working Well

  • Solid trigger coverage: Your description hits specific use cases (DOCX, PDF, PPTX, tracked changes, format conversion) that'll activate appropriately for document tasks.
  • Clear workflows: The unpack → edit → pack pattern for DOCX is straightforward, and your quick reference table at the top is genuinely helpful.
  • Real code examples: Each operation has concrete Python examples (pdfplumber, python-docx, pandoc) that developers can actually use, not vague pseudo-code.
  • Validation for PPTX: You're already thinking about verification with validate.py and thumbnail.py checks—that's the right mindset.

The Big One: Missing Architecture for 272 Lines

Here's what's holding you back: all your content is crammed into one SKILL.md file without layering. You hit 272 lines, which means you should have a table of contents and offloaded detailed patterns to reference files.

Why it matters: Developers scanning your skill should see the essentials first, then drill into complexity. Right now they're scrolling past design guidelines and XML patterns to find what they need.

The fix:

  1. Add a table of contents at the top (sections: Word, PDF, PowerPoint, Format Conversion, Dependencies)
  2. Move detailed content to references/:
    • references/xml-patterns.md — OOXML structure details
    • references/design-guidelines.md — color palettes and layout rules (honestly, consider if these belong at all—they're design advice, not doc manipulation)
    • references/ocr-setup.md — detailed OCR configuration
  3. Keep SKILL.md to ~150 lines with just essential workflows

Impact: +6 points easy.

Other Things Worth Fixing

  1. No validation for DOCX/PDF workflows — You validate PPTX (good!), but DOCX editing and PDF merging lack verification steps. Add: "Verify DOCX opens without corruption" and "Verify PDF page count matches expected." (+1 point)

  2. Redundant tool explanations without decision criteria — Lines 82-99 show pdfplumber, then lines 138-150 show pdftotext doing similar work. Add context: "Use pdfplumber for programmatic extraction with table support; use pdftotext CLI for quick one-off extractions." (+1 point)

  3. Design guidelines feel out of scope — Color palettes and layout rules in lines 228-250 are UX advice, not document manipulation. Either move to references or remove entirely. (+1 point)

Quick Wins

  • High impact: Restructure into SKILL.md + references/ (fixes the architecture issue)
  • Medium impact: Add TOC, validation patterns, tool differentiation
  • Low lift: Remove or relocate design guidelines
  • Combined improvement: ~9 points, could push you to a solid B-

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

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions