add-lua-binding-complex

Add comprehensive Cataclysm-BN Lua bindings for complex C++ classes with methods, inheritance, constructors, properties, operators, and ownership rules.

Quality

72%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

Securityby

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./.agents/skills/add-lua-binding-complex/SKILL.md

Quality

Content

77%

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

The body is highly actionable with executable code, clear sequencing, validation, and feedback loops, but it is monolithic — all detail lives inline with no bundle files — and it includes some generic C++ convention explanation Claude already knows.

Suggestions

Trim or remove the 'Coding Conventions' examples that restate generic C++23 knowledge (auto, trailing returns, designated initializers, std::ranges), keeping only project-specific conventions.

Move the 'Advanced Patterns' and 'Common Patterns Reference' sections into a bundled reference file (e.g., references/advanced-patterns.md) and link to it from SKILL.md to improve progressive disclosure.

Dimension	Reasoning	Score
Conciseness	Mostly efficient project-specific macro usage (SET_MEMB, SET_FX, LUNA_DOC) that earns its place, but the 'Coding Conventions' section demonstrates generic C++23 concepts Claude already knows (auto, trailing return types, designated initializers, std::ranges).	2 / 3
Actionability	Fully executable code using real project macros, exact file paths, and runnable build/test commands ('cmake --build --preset linux-full …', 'cata_test-tiles "[lua]"'), making it copy-paste ready.	3 / 3
Workflow Clarity	A clear numbered Step 1–5 sequence is paired with an explicit 'Build and Verify' validation section, Troubleshooting issue→fix feedback loops, and a 'Files to Modify' checklist.	3 / 3
Progressive Disclosure	Headers and a dedicated 'Reference Files' list are well organized, but no bundle files exist and ~370 lines are entirely inline, with substantial Advanced and Common Patterns that could be split into separate reference files.	2 / 3
	Total	10 / 12 Passed

Description

67%

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 specific and distinctive, naming six concrete binding features and a clear niche, but lacks an explicit 'Use when…' trigger clause and natural term variations, which caps completeness and trigger-term quality.

Suggestions

Add an explicit 'Use when…' clause stating when Claude should invoke this skill (e.g., 'Use when adding or extending Lua bindings for complex Cataclysm-BN C++ classes').

Broaden trigger terms with natural phrasings a user would say, such as 'expose a C++ class to Lua', 'Lua API', or 'Lua modding'.

Dimension	Reasoning	Score
Specificity	Lists multiple concrete capabilities — 'methods, inheritance, constructors, properties, operators, and ownership rules' — tied to the concrete action of adding Lua bindings for complex C++ classes.	3 / 3
Completeness	Clearly states what the skill does, but provides no 'Use when…' clause, so the trigger guidance is only implied; per the guideline this caps completeness at 2.	2 / 3
Trigger Term Quality	'Cataclysm-BN Lua bindings' and 'C++ classes' are relevant domain terms, but common natural variations such as 'expose to Lua', 'Lua API', or 'Lua modding' are missing.	2 / 3
Distinctiveness Conflict Risk	Narrowly scoped to Cataclysm-BN Lua bindings for complex C++ classes, with the 'complex' qualifier distinguishing it from a likely sibling simple-binding skill, making conflicts unlikely.	3 / 3
	Total	10 / 12 Passed

Validation

87%

Warnings & errors only

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

Validation — 14 / 16 Passed

Validation for skill structure

Criteria	Description	Result
metadata_version	'metadata.version' is missing	Warning
relative_links	Relative link issues: 8 missing	Warning

	Total	14 / 16 Passed

Repository: cataclysmbn/Cataclysm-BN
Path: .agents/skills/add-lua-binding-complex/SKILL.md
Commit: 584939c

Reviewed: 1 day ago

Table of Contents

Discovery Implementation Validation

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.