CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

add-tools

Create tool configurations for a Sim integration by reading API docs

53

Quality

67%

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 ./.agents/skills/add-tools/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

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

This is a strong, unusually rigorous operational skill: executable templates, exact commands, named audit gates, and explicit validation/feedback loops. Its weaknesses are a 640-line monolithic structure with no reference files to split major policy areas into, and scattered redundancy (tautological sections, a closing checklist duplicating inline rules).

Suggestions

Split the 'Resolved Secrets and Provenance Boundaries' policy and the 'Wiring Tools into the Block' guide into references/ files (e.g. references/provenance.md, references/block-wiring.md), keeping SKILL.md as a lean overview with one-level-deep, clearly signaled pointers.

Delete the tautological 'Required vs Optional' subsection and inline '// Use false for optional' comments; the rule 'always explicitly set required' alone is sufficient.

Add a code template for the in-process handler (apps/sim/lib/internal/{service}/execute-tool.ts) mirroring the external ToolConfig template, so both execution paths are copy-paste ready.

DimensionReasoningScore

Conciseness

The body is dense with project-specific constraints Claude cannot know (boundary rules, provenance policy, audit commands) and avoids explaining general concepts, but it has several padded spots: the tautological 'Required vs Optional' section ('Always explicitly set required: true or required: false / Optional params should have required: false'), a '// Use false for optional' comment, and a final checklist repeating ~10 rules stated verbatim earlier. This is 'could be tightened' territory (anchor 3) rather than minor trimming (anchor 4).

3 / 5

Actionability

Guidance is highly concrete: complete TypeScript config templates for both external and internal boundaries, exact file paths, and exact commands ('bun run generate:tool-metadata', 'bun run check:tool-request-boundary', 'bun run apps/sim/scripts/check-block-registry.ts origin/staging'). The gap that keeps it from a 5: the in-process handler implementation (execute-tool.ts) — half of one execution path — is described only in prose, never shown as code.

4 / 5

Workflow Clarity

Sections are ordered in execution order (structure → config → params → outputs → types → barrel → registering → block wiring → checklists → final validation), with explicit validation steps (named audit scripts plus a required 'Final Validation' cross-reference pass), a genuine error-recovery branch ('If the docs do not clearly show the response JSON... stop short of guessing' with four concrete fallbacks), and checklists for both block wiring and completion. This matches the anchor-5 example's structure of sequence + validation + feedback loops + checklists.

5 / 5

Progressive Disclosure

The single SKILL.md is ~640 lines with well-organized headers but no bundle references at all (no references/, scripts/, or assets/ exist); large distinct topics — the ~60-line provenance policy and ~150-line block-wiring guide — are inlined where separate reference files clearly belong. Structure is good (above anchor 2's 'minimal structure'), but 'most content is appropriately placed' (anchor 4) does not hold for a monolith of this size.

3 / 5

Total

15

/

20

Passed

Description

53%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 states a clear, third-person 'what' anchored to a specific niche (Sim tool configurations from API docs), but completely lacks 'when to use' trigger guidance and natural user phrasings. It sits at the midpoint of the rubric: unambiguous but not comprehensive.

Suggestions

Add an explicit trigger clause, e.g. 'Use when adding or updating tools for a Sim integration, or when the user asks to connect a new service/API to Sim.'

Broaden trigger vocabulary with natural synonyms users would say: 'add tools', 'new integration', 'connect a service', 'API integration'.

Mention the fuller scope of the skill (registering tools, wiring blocks, validating against API docs) so the description isn't understated relative to the body.

DimensionReasoningScore

Specificity

The description names the domain ('Sim integration') and 1-2 concrete actions ('Create tool configurations', 'by reading API docs'), but omits registering tools, block wiring, and validation that the body covers. It matches anchor 3 rather than 4 because it does not list several specific actions.

3 / 5

Completeness

A clear 'what' is present ('Create tool configurations for a Sim integration by reading API docs'), but there is no 'Use when...' clause or equivalent trigger guidance, which caps completeness at 3 per the judging guidelines. The 'when' is entirely absent, not merely weakly implied.

3 / 5

Trigger Term Quality

'tool configurations', 'Sim integration', and 'API docs' are relevant keywords, but common natural variations like 'add a tool', 'integrate a service', or 'new integration' are missing. Not a 2 because the terms are domain-specific rather than generic; not a 4 because coverage of natural phrasings is thin.

3 / 5

Distinctiveness Conflict Risk

'Sim integration' pins a clear niche so the description is mostly distinct with only minor overlap risk against adjacent tooling/registry skills. Not 5 because no explicit trigger phrases establish the niche, and not 3 because the Sim-specific framing keeps it distinguishable from generic tool-creation skills.

4 / 5

Total

13

/

20

Passed

Validation

81%

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

Validation — 13 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

skill_md_line_count

SKILL.md is long (646 lines); consider splitting into references/ and linking

Warning

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

referenced_paths_exist

Referenced path issues: 1 missing

Warning

Total

13

/

16

Passed

Repository

simstudioai/sim

Path

.agents/skills/add-tools/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.