CtrlK
BlogDocsLog inGet started
Tessl Logo

codebase-design

用于设计深模块的共享词汇。适用于用户想设计或改进模块接口、寻找深化机会、决定 seam 放在哪里、让代码更容易测试或更适合 AI 导航,或其他技能需要深模块词汇时。

60

Quality

70%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./skills/engineering/codebase-design/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

61%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A lean, well-organized conceptual skill body with a disciplined glossary, concrete heuristics, and apt code contrasts — it respects the reader's intelligence and never pads. Its two real weaknesses are the absence of any sequenced application workflow and, most seriously, a 'Going deeper' section whose two referenced files do not exist in the bundle, breaking progressive disclosure.

Suggestions

Create the referenced DEEPENING.md and DESIGN-IT-TWICE.md files (or remove the 'Going deeper' section) — every link in the body must resolve to a real bundle file.

Add a short numbered procedure for applying the vocabulary when evaluating a module (e.g., 1. write down the interface as callers see it, 2. run the deletion test, 3. count real adapters at the seam, 4. check testability rules) so usage has an explicit sequence.

Trim the overlap between the Glossary and Relationships sections (leverage/locality are defined twice) to tighten token efficiency.

DimensionReasoningScore

Conciseness

The body is dense and assumes competence — glossary entries are one to three sentences, the deep/shallow ASCII diagrams are compact, and it defines terms by contrast ('_Avoid_: boundary') rather than padding. It does not fully reach anchor 5 because the Relationships section partially restates the glossary ('Depth 为 callers 产生 Leverage…' repeats definitions of leverage/locality) and the three interface-design questions repeat the deep/shallow discussion.

4 / 5

Actionability

Concrete, usable guidance is present: the deletion test ('想象删除这个 module…'), three specific interface-design questions, the two-adapters rule, and contrasting TypeScript snippets ('function processOrder(order, paymentGateway) {}' vs constructing 'new StripeGateway()'). It stops short of anchor 5 because the code snippets are illustrative fragments rather than complete executable examples, and the application guidance is heuristic rather than copy-paste ready.

4 / 5

Workflow Clarity

There is no multi-step process in the body — it is a vocabulary/reference skill whose core instruction ('凡是在设计或重构代码时,都使用这套语言和原则') is unambiguous, which under the simple-skill note could support a higher score. However, applying the material has no explicit sequence (no 'evaluate a module: identify its interface, apply the deletion test, check adapter count' style checklist), and the promised deepening workflows (DEEPENING.md, DESIGN-IT-TWICE.md) are dangling references, leaving checkpoints implicit. This lands on anchor 3 rather than 4.

3 / 5

Progressive Disclosure

The in-body structure is good (clear section headers, tight glossary), but the rubric directs scoring against the actual bundle: the 'Going deeper' section points to [DEEPENING.md](DEEPENING.md) and [DESIGN-IT-TWICE.md](DESIGN-IT-TWICE.md), and no such files exist anywhere in the skill directory — there is no references/, scripts/, or assets/ at all. Navigation to the deeper material fails entirely, which is worse than the 'references present but not clearly signaled' state of anchor 3 and lands noticeably below the midpoint between anchors 2 and 3.

2 / 5

Total

13

/

20

Passed

Description

78%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A well-constructed description that clearly states what the skill provides (a shared deep-module design vocabulary) and gives an explicit, multi-trigger 'use when' clause in impersonal voice. Trigger terms are natural and domain-specific. The only real limitation is that the capability side is thin — it names contexts of use rather than several concrete actions.

Suggestions

Strengthen the 'what' clause by naming the concrete deliverables of the vocabulary (e.g., a glossary of module/interface/seam/depth terms plus design principles and testability heuristics) rather than only 'shared vocabulary'.

Add common synonym triggers such as 重构/refactoring, 封装/encapsulation, or 'API design' so users phrasing the need differently still match.

DimensionReasoningScore

Specificity

The description names its domain clearly ("用于设计深模块的共享词汇" — a shared vocabulary for designing deep modules) and enumerates concrete contexts ("设计或改进模块接口", "决定 seam 放在哪里", "让代码更容易测试"), but the skill's actual action is singular — providing vocabulary — so capability coverage reads as domain-plus-uses rather than a list of several concrete actions. It sits above anchor 2 (generic actions) because the use contexts are specific, but below anchor 4 because it does not list multiple distinct operations the skill performs.

3 / 5

Completeness

It explicitly answers both questions: the 'what' is stated up front ("用于设计深模块的共享词汇") and the 'when' is an explicit 适用于…时 clause listing five concrete triggers including a cross-skill trigger ("其他技能需要深模块词汇时"). This matches anchor 5 (clear what AND when with concrete trigger phrases) and exceeds anchor 4, whose 'when' is merely adequate.

5 / 5

Trigger Term Quality

Trigger phrases are natural and varied: "模块接口" (module interface), "seam", "更容易测试" (easier to test), "AI 导航" (AI navigation), and "深化机会" (deepening opportunities) — phrases a user designing code would plausibly say. It falls short of anchor 5 because common synonyms like 重构 (refactor), 封装 (encapsulation), or API 设计 are absent, but it clearly exceeds anchor 3's 'some relevant keywords, missing variations'.

4 / 5

Distinctiveness Conflict Risk

The niche is distinct — 'deep modules', 'seam placement', and 'deepening' are specialized Ousterhout/Feathers-style design vocabulary unlikely to collide with generic skills, and the explicit cross-skill trigger clarifies its role. It does not reach anchor 5 because "设计或改进模块接口" overlaps moderately with general API-design or refactoring skills.

4 / 5

Total

16

/

20

Passed

Validation

93%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

relative_links

Relative link issues: 2 missing

Warning

Total

15

/

16

Passed

Repository
vinvcn/mattpocock-skills-zh-CN
Reviewed

Table of Contents

Is this your skill?

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.