Skillquality 0.45

self-review

Run a 4-pillar, 6-dimension alignment audit on the current project. Checks design currency, executes artifact verification, detects skill deposits. Use when the user says "self-review", "审视一下", or "audit". Report-only — never auto-fix.

Price
free
Protocol
skill
Verified
no

What it does

Self-Review

Audit process: discover anchors in 4 pillars (Design, Artifact, Skill, Progress), lock scope from Progress, then check alignment across all 6 pillar-pair dimensions. Platform-agnostic, language-agnostic.

Rules

  • Report only, never auto-fix — flag issues, do not modify files. Exception: Dimension 2 MUST execute verification (build, run, CLI) — read-only execution.
  • Scope first — lock scope from Progress before any dimension. Never flag deferred/out-of-scope items.
  • Progress first — start from "what are we doing now?" and radiate outward.
  • Scan by content, not by file type — a commit message can carry Design intent, a code comment can carry Progress status. Never skip content because "it's not in the right file type for this pillar."
  • Infer before skipping — when a pillar has no explicit files, check implicit sources before declaring it empty.
  • Built-in principles + user standards — the 6 principles are the baseline; project rules and domain skills add to it, never replace it.
  • Be specific — cite file paths and line numbers.
  • No false positives — only flag real drift. "Not yet built" is not drift if deferred.
  • Skill deposits need criteria — evaluate against deposit criteria in dimensions.md (Dimension 3).
  • Always add current year to search queries — for Currency and Assumptions checks.
  • Platform-agnostic, language-agnostic.

Execution Procedure

def self_review(project_path):
    # STEP 1: Discover Anchors
    anchors = scan_anchors(project_path)               # see The 4 Pillars + Step 1 tables
    standards = scan_standards(project_path)            # see Project-Specific Standards
    domain_skills = scan_domain_skills(project_path)    # see Domain Skills
    infer_missing(anchors)                              # see Implicit Anchor Inference

    if anchors.design:
        introspection = evaluate_design(anchors.design) # see Design Introspection
        currency = check_currency(anchors.progress)     # see Currency & Assumptions Check

    # STEP 2: Lock Scope
    scope = lock_scope(anchors.progress)               # see Step 2: Lock Scope
    assert scope.current_phase                         # GATE — cannot audit without knowing phase

    # STEP 3: Run 6 Dimensions                         # references/dimensions.md
    # Priority — always run
    d1 = check_progress_design(anchors, scope, standards)
    d2 = check_progress_artifact(anchors, scope, standards)
    assert d2.verification_executed                    # GATE — must execute, not just report
    assert d2.verification_level >= min_level(artifact_type)  # GATE — see Verification Depth by Artifact Type in dimensions.md
    d3 = check_progress_skill(anchors, scope, standards)

    # Cross-pillar — always run
    d4 = check_design_artifact(anchors, scope, standards)
    d5 = check_design_skill(anchors, scope, standards)
    d6 = check_artifact_skill(anchors, scope, standards)

    # STEP 4: Report
    report(introspection, scope, [d1, d2, d3, d4, d5, d6])  # see Step 4: Report

Audit Principles

Six principles guide all dimension checks. They are embedded as check questions in each dimension — you don't need to apply them separately.

#PrincipleOne-line definition
T1Verifiable & VerifiedClaims must be testable, and evidence of passing must exist
T2No Internal ContradictionNo conflicting statements across pillars
T3Simplest Sufficient SolutionDon't introduce what isn't needed
E1FeasibleAchievable with current resources
E2Boundary-CompleteNo fatal gaps within current scope
E3MaintainableOthers can understand and continue the work

The 4 Pillars

PillarCore questionWhat it coversCommon carriers (examples, not definitions)
DesignWhy are we doing this? What should it become?Intent, decisions, constraints, specsCLAUDE.md, design docs, requirements, even commit messages or verbal agreements
ArtifactWhat was actually produced?Any deliverable — code, documents, designs, videos, skills, configssrc/, articles, exported videos, design files, SKILL.md itself
SkillHow do we do things? What did we learn?Reusable methods, standards, accumulated know-howStyle guides, workflows, conventions, templates, coding patterns
ProgressWhere are we? What's next?Plans, status, milestones, trackingprogress.md, changelogs, git log, TODO comments, roadmaps

Process

Step 1: Discover Anchors

Scan the project for anchors in each pillar (all optional — skip pillars with no anchors after attempting inference):

PillarTypical anchors
DesignCLAUDE.md, **/design/*.md, **/docs/*.md, README.md
Artifactsrc/, lib/, app/, dist/, output/, **/types.ts
Skill.claude/skills/, skill/, **/references/*.md, **/templates/
Progress**/progress.md, **/plan/*.md, CHANGELOG.md, **/roadmap.md

Report which anchors you found and which pillars have none.

Project-Specific Standards

Beyond the standard anchors, scan for project-specific quality standards that should inform the audit. These files vary by platform — check whichever exist:

PlatformProject instructionsLocal/private overridesRules/standards
Claude CodeCLAUDE.mdCLAUDE.local.md.claude/rules/*.md
Cursor.cursor/rules/*.mdc.cursor/rules/personal.mdc (gitignored).cursorrules (legacy)
CodexAGENTS.mdAGENTS.override.md
OpenClawAGENTS.md, SOUL.mdvia openclaw.json
GenericREADME.md, CONTRIBUTING.md.editorconfig, linter configs

If found, these standards become additional checks in cross-pillar dimensions. For example:

  • CLAUDE.md says "capture-pane is debug only" → Dimension 4 checks code doesn't use capture-pane for core logic
  • AGENTS.override.md says "run tests before committing" → Dimension 2 checks if test evidence exists
  • CONTRIBUTING.md says "no direct pushes to main" → Dimension 6 checks branch workflow

Domain Skills

Scan .claude/skills/ (or platform equivalent) for two types of skills:

Domain skills — skills whose description matches the current project's domain. Treat the standards and rules defined in those skills as additional check criteria — layer them on top of the built-in principles when running dimensions. Only include domain skills that are clearly relevant. Do not force-apply unrelated skills.

Example: A video-editing skill defines "jump cuts must not exceed 3 frames" → Dimension 4 and 6 check this rule against the artifact.

Rule-skills — skills whose primary value is enforceable MUST/NEVER constraints (not workflows). These follow the rules-as-skills pattern. In-repo rule-skills are always relevant to the project they live in, regardless of domain match. Extract each MUST/NEVER constraint and elevate it to an audit standard with the same authority as CLAUDE.md rules.

Detection signals:

  • Description or body contains MUST/NEVER/ALWAYS constraint patterns
  • Has a Constraints section with enumerated rules
  • Name contains "rules", "constraints", or "standards"
  • Follows rules-as-skills three-layer format (trigger → constraints → verification)

Example: A maintenance-rules skill defines "MUST NOT push SKILL.md body over 500 lines" → Dimension 2 verifies line count, Dimension 6 checks compliance.

Report detected rule-skills and their extracted constraints in the Anchors section under Skill.

Implicit Anchor Inference

When a pillar has no dedicated files, attempt to infer its content from implicit sources before skipping:

PillarImplicit sources to check
DesignCommit messages, PR descriptions, README intro, inline comments stating intent
ArtifactAny output file (this pillar is almost never empty)
SkillRecurring code patterns, consistent conventions across files
ProgressGit log timeline, branch names, TODO/FIXME comments

Mark inferred anchors as [inferred] in the report. If inference produces nothing useful, then skip the pillar with a note.

Design Introspection

When Design anchors are found, evaluate the design itself — not just its alignment with other pillars. Apply the following checks:

  • Clarity: Is the stated goal well-defined? Could someone new understand the intent?
  • Value: Is this a well-formulated problem? Is it worth solving in this form?
  • Scope: Is the scope appropriate — not too broad, not too narrow?
  • Assumptions: Are there stale assumptions, blind spots, or unconsidered alternatives?
  • Currency: Is the project solving the right problem with current tools? (See detailed check below.)
  • Simplicity: Is this the simplest sufficient solution? Can anything be removed without losing capability? Are there simpler alternatives with equal explanatory/functional power?

Currency & Assumptions Check

Scope rule: Only check currency for in-scope items that are pending, in-progress, or blocked. Completed, working items are NOT checked — don't fix what isn't broken. Don't search dependency versions (that's a package manager's job).

Check process:

  1. Extract unsolved problems from Progress (pending/blocked items only)
  2. For each unsolved problem, search: "<problem description> solution <current year>"
  3. Evaluate search results:
FindingLabelAction
No relevant new solutionCurrentNo action
Community solution exists but immature (<6 months, few adopters, no major platform support)EmergingNote in report, don't recommend switching
Community solution with significant adoption (major platform support, established project)Problem DissolvedFlag: "This problem may already be solved by X. Verify before continuing custom work." Include source URL
A key design assumption has been invalidated (API added, limitation removed, standard changed)Assumption InvalidatedFlag: "Design assumes X, but Y is now supported as of [date]." Include source URL
  1. For Assumptions specifically: extract hard assumptions from Design anchors (patterns like "because X doesn't support Y", "since there's no way to Z", "limited by W"). Search each to verify it still holds.

Always add the current year to search queries (e.g., "React server components 2026").

For each major design decision identified above, ask these three questions:

  1. Can this actually be built/delivered with the resources available? (What would block it?)
  2. What scenario within the current scope would break this? (What's the weakest point?)
  3. Is there a way to achieve the same goal with fewer moving parts? (What can be removed?)

If any question reveals a concern, report it. These questions only apply to decisions that significantly affect the project direction — skip for minor choices.

Report design introspection findings as a preamble before dimensional checks.

Step 2: Lock Scope

Before running any dimension checks, establish the audit boundary from Progress:

  1. Read current phase from Progress anchors. Identify what the project claims to be working on right now.

  2. Classify all items into three categories:

CategoryDefinitionAudit treatment
In-scopeItems belonging to the current phase/milestoneFull dimensional check against all 6 principles
DeferredItems explicitly postponed to a future phaseOnly check: is it tracked somewhere? Not lost silently?
Out-of-scopeItems not in any planDo not check. Do not flag absence as a defect
  1. State the locked scope explicitly in the report so the reader knows what was and wasn't audited.

The core rule: do not judge current work by future plans. A feature designed for Phase 3 that doesn't exist yet is not a defect in Phase 1. An architecture that will need refactoring later is acceptable if the current phase is independently testable and shippable.

Flag Phase 1 artifacts that depend on unbuilt Phase 2 — those are real defects. But "Phase 2 hasn't started yet" is never a finding.

Step 3: Run 6 Dimensions

Each dimension checks drift between two pillars. This is the complete C(4,2) combination — no exceptions, no artificial extras.

Priority (Progress-centric) — always run:

  1. Progress <> Design — Are we aligned with design intent?
  2. Progress <> Artifact — Does claimed status match actual deliverables? Must execute verification.
  3. Progress <> Skill — Any lessons to capture? Existing skills need updating?

Cross-pillar — always run: 4. Design <> Artifact — Does the output match the design? 5. Design <> Skill — Do our methods support our design goals? 6. Artifact <> Skill — Does the output follow established methods?

See references/dimensions.md for detailed checks per dimension.

Step 4: Report

Output a structured report:

## Self-Review Report

### Design Introspection
- [findings on clarity, value, scope, assumptions, currency, simplicity]

### Scope Lock
- **Current phase**: [phase name/description]
- **In-scope**: [items being audited]
- **Deferred**: [items tracked but not yet due]
- **Out-of-scope**: [not applicable to this audit]

### Anchors Found
- **Design**: [paths]
- **Artifact**: [paths]
- **Skill**: [paths]
- **Progress**: [paths]

### Priority Dimensions (Progress-centric)

#### 1. Progress <> Design [Aligned / Drift / Broken]
- [specific findings with file paths, scoped to current phase]

#### 2. Progress <> Artifact [Aligned / Drift / Broken]
- [specific findings, with verification evidence]

#### 3. Progress <> Skill [Aligned / Drift / Broken]
- [specific findings]

### Cross-pillar Dimensions

#### 4. Design <> Artifact [Aligned / Drift / Broken]
- [specific findings, scoped to current phase]

#### 5. Design <> Skill [Aligned / Drift / Broken]
- [specific findings]

#### 6. Artifact <> Skill [Aligned / Drift / Broken]
- [specific findings]

### Summary
- X/6 aligned, Y/6 drifted, Z/6 broken
- [top priority fixes, if any]
- [skill deposit candidates with recommendation]

Standard Recommendations

If the audit finds recurring quality issues with no corresponding standard, include a Standard Recommendation in the report:

  1. Specify the exact file path where the standard should be persisted (use the platform table in Step 1)
  2. Draft the standard text ready to be added
  3. Explain why this standard is worth adding (what recurring issue it prevents)

Persistence routing:

  • Project-local standards (single-project, simple) → platform-native file: .claude/rules/<topic>.md (CC), AGENTS.override.md (Codex), .cursor/rules/<topic>.mdc (Cursor)
  • Cross-project reusable standards → suggest creating a rule-skill via Skill-Forge. Rule-skills are portable across platforms and publishable — see the rules-as-skills methodology
  • Platform lacks native rules (e.g., OpenClaw) → rule-skill is the only persistent option

Default to platform-native files (lower friction). Suggest rule-skills when the standard could benefit other projects. Always present both options — the user decides.

Capabilities

skillsource-motifulskill-self-reviewtopic-agent-skillstopic-ai-agentstopic-audittopic-claude-codetopic-code-qualitytopic-codextopic-cursortopic-developer-toolstopic-llmtopic-openclawtopic-quality-assurancetopic-self-review

Install

Installnpx skills add motiful/self-review
Transportskills-sh
Protocolskill

Quality

0.45/ 1.00

deterministic score 0.45 from registry signals: · indexed on github topic:agent-skills · 6 github stars · SKILL.md body (15,574 chars)

Provenance

Indexed fromgithub
Enriched2026-05-18 19:14:32Z · deterministic:skill-github:v1 · v1
First seen2026-05-18
Last seen2026-05-18

Agent access