CtrlK
BlogDocsLog inGet started
Tessl Logo

code-review

Use when reviewing Basic Machines code for house style, architecture risk, pre-merge hardening, or whether a change fits basic-memory/basic-memory-cloud conventions.

80

Quality

100%

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

SKILL.md
Quality
Evals
Security

Quality

Content

100%Weight 40%Scale 1-3

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

A tight, well-structured review skill: concrete anti-patterns to check, an explicit reporting template with severity levels, and clean section organization. It assumes Claude's competence and avoids verbosity while remaining fully actionable for a report-only workflow.

DimensionReasoningScore

Conciseness

Lean throughout — no explaining of what code review or a diff is; content is delivered as compact bulleted rubric items, a house-rules checklist, and a small severity table, so every token earns its place rather than padding concepts Claude already knows.

3 / 3

Actionability

Concrete, falsifiable patterns to flag ('No speculative getattr(obj, "attr", default)', 'No broad exception swallowing', 'Dataclasses for internal value/result objects; Pydantic at validation/serialization boundaries', 'Narrow Protocol's') plus a copy-ready severity table and reporting template; under the instruction-skill note, the absence of executable code is not penalized because the guidance is specific and actionable.

3 / 3

Workflow Clarity

Sections imply a clear sequence (Scope → Review Rubric → House Rules → Reporting Format) and provide explicit checklists for what to examine; as a report-only, non-destructive skill the validation-checkpoint requirement does not cap it, and the simple-skill note allows 3 for well-organized single-purpose skills.

3 / 3

Progressive Disclosure

No bundle files exist and the body is a single self-contained file organized into well-signaled sections with no nested references, matching the score-3 carve-out for skills that need no external references and are well organized.

3 / 3

Total

12

/

12

Passed

Description

100%Weight 40%Scale 1-3

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, third-person description with an explicit 'Use when' trigger, several concrete review actions, and a clearly scoped domain that limits conflict with other skills. It earns the top anchor on every dimension.

DimensionReasoningScore

Specificity

Lists multiple concrete review actions — 'house style, architecture risk, pre-merge hardening, or whether a change fits basic-memory/basic-memory-cloud conventions' — matching the score-3 anchor of several specific concrete actions rather than the vague single-action anchor at 2.

3 / 3

Completeness

Explicitly answers both what (reviewing for house style, architecture risk, pre-merge hardening, convention fit) and when via the leading 'Use when reviewing ...' clause, satisfying the score-3 anchor for both what AND when.

3 / 3

Trigger Term Quality

Natural terms a user would say when requesting review are present ('reviewing ... code', 'house style', 'architecture risk', 'basic-memory/basic-memory-cloud'); coverage is broad rather than the partial set at level 2.

3 / 3

Distinctiveness Conflict Risk

Scoped to 'Basic Machines code' and the named basic-memory/basic-memory-cloud repos, giving it a clear niche unlikely to trigger for unrelated review skills.

3 / 3

Total

12

/

12

Passed

Validation

100%

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

Validation16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository
basicmachines-co/basic-memory
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.