CtrlK
BlogDocsLog inGet started
Tessl Logo

add-tools

Create tool configurations for a Sim integration by reading API docs

53

Quality

61%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Low

Low-risk findings worth noting

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.

The body delivers exceptionally concrete, project-specific guidance with a complete multi-step workflow and strong validation checkpoints, and it largely avoids explaining things Claude already knows. Its weaknesses are structural: a ~630-line monolith whose provenance, file-output, and block-wiring sections belong in one-level-deep reference files, plus redundancy between inline rules and checklists that inflates the token budget.

Suggestions

Split the 'Resolved Secrets and Provenance Boundaries' section into a dedicated reference file (e.g. references/provenance.md) and keep a short decision summary plus a pointer in SKILL.md.

Move the 7-step 'Wiring Tools into the Block' detail into a reference file (e.g. references/block-wiring.md), keeping only the requirement statement and the checklist inline.

De-duplicate the 'Checklist Before Finishing' against the rules sections it restates, and add one executable modelInput/secretProvenance example to make the provenance rules copy-paste ready.

DimensionReasoningScore

Conciseness

The body is mostly project-specific rules Claude would not know (boundary rules, provenance mechanics, registry/CI commands) rather than explanations of known concepts, so it avoids the worst padding. However, at ~630 lines it has real tightening opportunities: the dense provenance section reads as near-legal prose, and the checklists restate rules already spelled out above. 'Mostly efficient but includes some unnecessary explanation or could be tightened' is the best fit — not 4 because the redundancy between the inline rules and the two checklists is noticeable.

3 / 5

Actionability

Guidance is highly concrete: two complete TypeScript configuration templates, exact file paths ('apps/sim/tools/{service}/types.ts', 'apps/sim/lib/internal/tool-operations/registry.server.ts'), exact commands ('bun run tool-metadata:generate', 'bun run check:tool-request-boundary'), and a fully specified 7-step block-wiring procedure. It is not 5 because the central templates contain unresolved placeholders ('// Define output structure here', '// Map resolved tool params') and the provenance section gives rules without a single executable modelInput example to copy.

4 / 5

Workflow Clarity

The sequence is explicit and complete — read API docs, scaffold the directory, pick exactly one execution boundary, generate config, register in registry.ts, regenerate metadata, wire the block (7 numbered sub-steps with its own checklist), then a mandatory 'Final Validation' section that re-reads each tool file and cross-references the API docs with an explicit failure path ('If any response schema is still unknown, explicitly tell the user instead of guessing'). This matches the anchor with explicit validation steps, feedback loops, and checklists for a complex process.

5 / 5

Progressive Disclosure

Section structure is good (clear headers, consistent formatting) and the couple of cross-references ('.agents/skills/tool-registry-boundary/SKILL.md', the 'migrate-application-operation' skill) are one level deep. But the skill is a ~630-line monolith with no bundle files: the provenance rules, file-output rules, and block-wiring details are exactly the kind of bulk material the anchors say belongs in a separate reference file. That fits 'Some structure but could be better organized; content that should be separate is inline' — not 4, which requires most content appropriately split across files.

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 is concise and third-person with a clear 'what', but it lacks any 'when to use' trigger guidance and only names 1-2 of the skill's actual actions. Adding a 'Use when...' clause with natural trigger phrases would substantially improve completeness and trigger quality.

Suggestions

Add an explicit 'Use when...' clause, e.g. 'Use when the user asks to add tools, create a tool configuration, or integrate a new service into Sim.'

Include 1-2 more concrete actions to reflect the full scope, such as registering tools, regenerating metadata, and wiring them into the service block.

Add natural trigger synonyms users would say ('add tools', 'new integration', 'wire up a service') to improve keyword coverage.

DimensionReasoningScore

Specificity

The description names a concrete action ("Create tool configurations for a Sim integration") and a concrete method ("by reading API docs"), but stops at 1-2 actions without covering what the skill body actually does (registering tools, block wiring, metadata generation). It matches the anchor 'Names domain and 1-2 concrete actions, but not comprehensive' and not the level above, which requires several listed specific actions.

3 / 5

Completeness

The 'what' is clear (create tool configurations by reading API docs), but there is no 'Use when...' clause or equivalent trigger guidance, which per the rubric caps completeness at 3. It is not a 2 because the 'what' is concrete rather than vague, and not a 4 because 'when' is entirely absent rather than weakly implied.

3 / 5

Trigger Term Quality

Relevant keywords like "tool configurations", "Sim integration", and "API docs" are present, but common natural phrasings users would actually say ("add tools", "add an integration", "wire up a service") are missing. It sits between the generic 'Works with files' level and the good-coverage level 4.

3 / 5

Distinctiveness Conflict Risk

The phrase "for a Sim integration" carves out a specific niche, making confusion with generic tooling skills unlikely, though terms like "tool configurations" and "API docs" alone are broad if the Sim context is absent. Mostly distinct with minor overlap risk, matching the level-4 anchor rather than the level-5 anchor whose niche and triggers are both fully distinct.

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 (636 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
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.