CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

codebase-design

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

Quality

79%

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

67%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.

The SKILL.md body itself is well-crafted: a lean, internally consistent glossary of deliberately nonstandard vocabulary, concrete design heuristics, and small contrast code examples, all organized into scannable sections. Its one serious defect is that both "进一步深化" references point to files missing from the bundle, leaving the skill's escalation paths dead and the progressive-disclosure structure non-functional.

Suggestions

Add the missing DEEPENING.md and DESIGN-IT-TWICE.md to the bundle (e.g., under references/), or remove the "进一步深化" section and inline a brief summary of that guidance — both links currently point to files that do not exist.

If the reference files are added, update the link targets to match their actual bundle location (e.g., references/DEEPENING.md) so the paths and the file structure agree.

Trim the "## 关系" section, which restates relationships already implicit in the glossary definitions, to reduce body redundancy.

DimensionReasoningScore

Conciseness

The glossary defines only terms carrying project-specific, nonstandard meanings ("Depth - interface 上的 leverage", "Seam……_Avoid_: boundary") with explicit avoid-lists, and never re-explains basics Claude already knows. Minor trimmable redundancy: the "## 关系" section largely restates relationships implicit in the glossary definitions — fits anchor 4's "efficient; minor instances that could be trimmed".

4 / 5

Actionability

Concrete, immediately applicable heuristics: the "Deletion test" (想象删除这个 module……), three explicit interface-design questions (我能减少 methods 数量吗……), and three numbered testability rules with contrast TypeScript snippets (processOrder(order, paymentGateway) vs. internal new StripeGateway()). Per the instruction-only scoring note, absent full executables is not penalized, but the snippets are illustrative fragments rather than the complete copy-paste examples of anchor 5 — fits anchor 4's "mostly executable guidance; concrete code with minor gaps".

4 / 5

Workflow Clarity

This is a vocabulary/principles skill rather than a sequential procedure, and its application rule is unambiguous ("凡是在设计或重构代码时,都使用这套语言和原则" with explicit avoid-lists and design questions). Non-destructive, so no validation-cap applies; it sits above anchor 3's implicit checkpoints but lacks any sequenced design-session flow, matching anchor 4.

4 / 5

Progressive Disclosure

The "进一步深化" section clearly signals two one-level-deep references with descriptions of their content, but neither DEEPENING.md nor DESIGN-IT-TWICE.md exists in the bundle — no references/, scripts/, or assets/ directory is present, so the delegated material (dependency classification, seam discipline, the parallel sub-agent interface-design workflow) is inaccessible and every navigation pointer is broken. This matches anchor 2's broken/minimal effective organization despite a well-structured main body.

2 / 5

Total

14

/

20

Passed

Description

83%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 strong description: it states a clear (if slightly abstract) purpose and pairs it with an explicit, multi-scenario trigger clause covering design, deepening, seam placement, testability, and AI navigability. The vocabulary-oriented framing is distinctive; only synonym coverage and capability concreteness keep it from full marks.

DimensionReasoningScore

Specificity

Enumerates several concrete design actions ("设计或改进模块接口、寻找深化机会、决定 seam 放在哪里、让代码更容易测试或更适合 AI 导航"), but the capability statement itself ("用于设计深模块的共享词汇") remains somewhat abstract. Not anchor 5, which demands comprehensive concrete-capability coverage; clearly above anchor 3's 1-2 actions.

4 / 5

Completeness

Explicitly answers both: what ("用于设计深模块的共享词汇" — shared vocabulary for deep module design) and when ("适用于用户想……或其他技能需要深模块词汇时") with five concrete trigger scenarios. Matches anchor 5 exactly.

5 / 5

Trigger Term Quality

Includes natural phrases users would say, such as "设计或改进模块接口" and "让代码更容易测试", plus niche terms like "seam" and "深模块". Falls between anchor 4 and 5: good coverage but missing common variations like "重构" (refactor) or "模块化" (modularization).

4 / 5

Distinctiveness Conflict Risk

Occupies a distinct niche via signature vocabulary (深模块, seam, adapter, leverage/locality), making it unlikely to trigger for unrelated skills. Minor overlap risk with general refactoring or testing skills through "让代码更容易测试" and "让代码……更适合 AI 导航" — fits anchor 4's "mostly distinct; minor overlap risk".

4 / 5

Total

17

/

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

Path

skills/engineering/codebase-design/SKILL.md

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.