Content
55%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill provides highly actionable, executable guidance for SurrealDB development with strong security focus and clear TDD workflow. However, it is severely bloated with redundant content repeated across multiple sections and explains many concepts Claude already knows. The lack of progressive disclosure makes it a monolithic document that would benefit greatly from splitting into focused reference files.
Suggestions
Extract the OWASP mapping table, vulnerability details, and security patterns into a separate SECURITY.md reference file
Remove redundant examples - parameterized queries and password hashing are shown 4+ times; consolidate into single authoritative examples
Delete explanatory text Claude already knows (e.g., 'PDF files are...', what connection pooling does, basic async patterns)
Move the full test suite examples to a separate TESTING.md file, keeping only 1-2 representative tests inline
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Extremely verbose at ~800+ lines with significant redundancy. Patterns are repeated multiple times (e.g., parameterized queries shown in sections 3, 4, 5, 8, and 13). Explains concepts Claude already knows like what connection pooling is, basic Python async patterns, and OWASP categories. | 1 / 3 |
Actionability | Provides fully executable code examples throughout - Python tests, SurrealQL queries, JavaScript SDK usage, and bash commands are all copy-paste ready. Concrete examples with specific syntax for every pattern. | 3 / 3 |
Workflow Clarity | Clear TDD workflow with explicit steps (write failing test → implement → refactor → verify). Pre-implementation checklist with phases provides validation checkpoints. Testing section includes specific pytest commands for verification. | 3 / 3 |
Progressive Disclosure | Monolithic wall of text with no references to external files. All content is inline despite being 800+ lines. Content that should be in separate reference files (OWASP mapping table, full test suites, vulnerability details) is embedded directly. | 1 / 3 |
Total | 8 / 12 Passed |