Generate comprehensive C4 architecture documentation for an existing repository/codebase using a bottom-up analysis approach.
50
Quality
37%
Does it follow best practices?
Impact
Pending
No eval scenarios have been run
Passed
No known issues
Optimize this skill with Tessl
npx tessl skill review --optimize ./.agent/skills/c4-architecture-c4-architecture/SKILL.mdQuality
Discovery
32%Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.
The description identifies a specific methodology (C4) and approach (bottom-up analysis) which provides some clarity, but it lacks explicit trigger guidance and comprehensive action verbs. The absence of a 'Use when...' clause significantly weakens its utility for skill selection among many options.
Suggestions
Add a 'Use when...' clause with trigger terms like 'C4 diagrams', 'architecture documentation', 'document codebase structure', 'system context diagram', 'container diagram'
List specific concrete actions such as 'analyze code structure, generate context diagrams, create container and component views, produce deployment diagrams'
Include file type triggers like '.md architecture files', 'PlantUML', or 'Structurizr DSL' if applicable to increase distinctiveness
| Dimension | Reasoning | Score |
|---|---|---|
Specificity | Names the domain (C4 architecture documentation) and mentions a specific approach (bottom-up analysis), but doesn't list concrete actions like 'create context diagrams, generate container views, map component relationships'. | 2 / 3 |
Completeness | Describes what it does (generate C4 documentation) but completely lacks a 'Use when...' clause or any explicit trigger guidance for when Claude should select this skill. | 1 / 3 |
Trigger Term Quality | Includes some relevant terms like 'C4', 'architecture documentation', 'codebase', and 'repository', but misses common variations users might say like 'system diagrams', 'architecture diagrams', 'code structure', or 'document my code'. | 2 / 3 |
Distinctiveness Conflict Risk | The C4 methodology reference provides some distinctiveness from generic documentation skills, but 'architecture documentation' could overlap with other documentation or diagramming skills without clearer boundaries. | 2 / 3 |
Total | 7 / 12 Passed |
Implementation
42%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill has good structural organization and progressive disclosure through sub-skill references, but lacks actionable content in the main document. The instructions section is generic boilerplate that doesn't provide C4-specific guidance, and there are no concrete steps, commands, or validation checkpoints that Claude could execute directly from this file.
Suggestions
Add concrete, executable steps in the main skill file showing how to initiate the workflow (e.g., specific commands to create the C4-Documentation directory, how to invoke the first sub-skill)
Replace the generic 'Instructions' section with C4-specific actionable guidance including validation criteria for each phase
Add explicit validation checkpoints between phases (e.g., 'Before proceeding to Phase 2, verify that all code-level docs exist in C4-Documentation/code/')
Remove or condense the extended thinking block - Claude doesn't need this explanation of what the workflow does
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The extended thinking block adds unnecessary verbosity explaining what the workflow does. The overview section is reasonably efficient but could be tighter. Some redundancy between the overview and the phase descriptions. | 2 / 3 |
Actionability | The skill provides no concrete, executable guidance - only links to sub-skills and vague instructions like 'Clarify goals, constraints, and required inputs.' There are no actual commands, code examples, or specific steps to follow in this document itself. | 1 / 3 |
Workflow Clarity | The 4-phase structure is clear and the numbered sub-skills suggest a sequence, but there are no validation checkpoints, no feedback loops for error recovery, and no explicit instructions on how to proceed between phases or handle failures. | 2 / 3 |
Progressive Disclosure | Excellent use of progressive disclosure with a clear overview and well-organized one-level-deep references to 9 sub-skills. Navigation is clear with numbered sections and descriptive links. | 3 / 3 |
Total | 8 / 12 Passed |
Validation
90%Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.
Validation — 10 / 11 Passed
Validation for skill structure
| Criteria | Description | Result |
|---|---|---|
frontmatter_unknown_keys | Unknown frontmatter key(s) found; consider removing or moving to metadata | Warning |
Total | 10 / 11 Passed | |
3395991
Table of Contents
If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.