CtrlK
BlogDocsLog inGet started
Tessl Logo

hns-lsel-applier

Local Self-Evolution Loop (LSEL) APPLY engine — the playback-only consumer of approved decision.json records that drives `.moai/hooks/lsel-apply.sh` for the GOOS-local PROPOSE→APPLY seam closure (SPEC-LSEL-LOCAL-EVOLUTION-001 M3). Reads an approved decision.json, validates the target against the frozen allowlist (.claude/lsel/frozen-allowlist.json), mechanically refuses execution-meta targets lacking a synchronous-approval marker, applies the referenced diff.patch, appends an apply-ledger.jsonl row, and commits one lsel-<proposal-id>-tagged Conventional Commit on the feature branch. M3 scope: APPLY bypass closure only (the frozen Go applier stays frozen — its write-flag at internal/harness/applier.go:22 stays false; REQ-LSEL-003).

62

Quality

74%

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 ./.claude/skills/hns-lsel-applier/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

81%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 body is a well-structured, action-oriented operations runbook with a clearly sequenced pipeline and strong validation/feedback loops for the destructive apply/commit/revert operations. It is mostly token-efficient and well-organized, with only minor repetition of the frozen-flag invariant and some procedural detail that could live in references.

DimensionReasoningScore

Conciseness

The body is dense and assumes domain competence (no explaining of git/PDF/library basics), but the frozen-write-flag invariant is restated across 'Token discipline', 'What this skill does', the model-mediated layer, 'What this skill does NOT do', and the post-apply verification — minor over-explanation that could be trimmed.

4 / 5

Actionability

It gives concrete, executable guidance: the five-step pipeline with explicit exit codes/log categories and a copy-paste verification block (`sed -n '22p' internal/harness/applier.go`, `grep`, `tail -1`, named test scripts), with only minor gaps since the apply itself is delegated to the hook rather than shown as code.

4 / 5

Workflow Clarity

The APPLY pipeline is a clearly sequenced five-step process with explicit validation checkpoints (frozen-allowlist hard-reject, execution-meta forced-gate) and strong feedback loops (verify.sh 2-attempt retry + auto git revert, mandatory /moai gate superset with revert-on-fail, blocker-report escalation).

5 / 5

Progressive Disclosure

Content is well-organized into labeled sections with one-level-deep, clearly signaled cross-references (SPEC files, design-report HTML, frozen applier source, curator skill, named test scripts); minor gaps remain as substantial procedural detail is inlined rather than split into reference files.

4 / 5

Total

17

/

20

Passed

Description

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

The description is highly specific about the APPLY engine's concrete actions and occupies a distinct niche, but it lacks an explicit 'Use when...' trigger clause and leans on internal jargon rather than natural user keywords. Third-person voice is used correctly throughout.

Suggestions

Add an explicit 'Use when ...' trigger clause (e.g., when an approved LSEL decision.json is queued for playback) so the 'when' is stated, not implied.

Surface one or two natural trigger phrases a caller would actually say, in addition to the internal spec/hook identifiers.

DimensionReasoningScore

Specificity

The description enumerates multiple concrete actions ('Reads an approved decision.json', 'validates the target against the frozen allowlist', 'mechanically refuses execution-meta targets', 'applies the referenced diff.patch', 'appends an apply-ledger.jsonl row', 'commits one lsel-<proposal-id>-tagged Conventional Commit'), giving comprehensive coverage of the skill's behavior.

5 / 5

Completeness

The 'what' is explicit and detailed (five concrete APPLY steps), but the 'when' is only weakly implied via scope language and there is no explicit 'Use when...' trigger clause, which per the guidelines caps completeness at 3.

3 / 5

Trigger Term Quality

It surfaces relevant domain terms ('LSEL APPLY', 'decision.json', 'PROPOSE→APPLY', 'Conventional Commit', 'frozen allowlist') but these are internal-harness jargon rather than natural user phrases, and no 'Use when...' trigger phrasing is present.

3 / 5

Distinctiveness Conflict Risk

It carves a clear niche (the LSEL APPLY playback engine for a specific spec/hook path) with distinct, specific triggers and minimal overlap risk with other skills.

5 / 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

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

Total

15

/

16

Passed

Repository
modu-ai/moai-adk
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.