CtrlK
BlogDocsLog inGet started
Tessl Logo

project-conventions

Core conventions and patterns for ExcelMcp

36

Quality

32%

Does it follow best practices?

Impact

No eval scenarios have been run

SecuritybySnyk

Passed

No known issues

Optimize this skill with Tessl

npx tessl skill review --optimize ./.squad/skills/project-conventions/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

64%

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

This is a solid project conventions skill with strong actionability—concrete code examples, specific anti-patterns, and clear do/don't guidance make it immediately useful. The main weaknesses are moderate verbosity in the testing and error handling sections where similar points are restated, and the lack of explicit validation checkpoints in multi-step workflows. The content would benefit from splitting detailed testing and error handling guidance into separate referenced files.

Suggestions

Split the lengthy Testing and Bug Report Triage sections into a referenced file (e.g., TESTING.md) to improve progressive disclosure and reduce the main skill's token footprint.

Add explicit validation/verification steps to multi-step workflows—for example, after COM cleanup patterns, include a verification step or after the TDD cycle, specify how to confirm green state before proceeding.

DimensionReasoningScore

Conciseness

The content is mostly efficient and project-specific, but some sections are somewhat verbose—particularly the testing and bug triage sections which repeat similar ideas across multiple bullet points. Some guidance (e.g., explaining what PascalCase and camelCase are) is unnecessary for Claude.

2 / 3

Actionability

The skill provides concrete, executable code examples (exception propagation patterns with correct/incorrect examples), specific commands (dotnet test with filter syntax), exact file paths, and clear do/don't patterns. The guidance is specific enough to act on immediately.

3 / 3

Workflow Clarity

The TDD workflow (test FIRST → RED → implement → GREEN) and bug triage pattern provide clear sequences. However, there are no explicit validation checkpoints or feedback loops for the multi-step processes described (e.g., no 'verify COM cleanup succeeded' step, no explicit build-then-test sequence). The triage pattern is well-sequenced but lacks verification steps.

2 / 3

Progressive Disclosure

The file structure section provides good navigation to the codebase, and content is organized into logical sections (Patterns, Anti-Patterns, Triage). However, for a skill this long (~120 lines of dense content), some sections like Testing and Error Handling could benefit from being split into referenced files. No bundle files are provided despite the content's complexity warranting them.

2 / 3

Total

9

/

12

Passed

Description

0%

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

This description is critically underspecified. It provides no concrete actions, no trigger terms, no 'when to use' guidance, and no distinguishing characteristics. It reads more like an internal label than a functional description that Claude could use to select the right skill.

Suggestions

Replace the vague phrase 'Core conventions and patterns' with specific actions like 'Read, write, and manipulate Excel spreadsheets, create pivot tables, generate charts, and format cells'.

Add an explicit 'Use when...' clause with natural trigger terms such as 'Excel', 'spreadsheet', '.xlsx', 'tabular data', 'pivot table', 'workbook'.

Remove the internal tool name 'ExcelMcp' or supplement it with user-facing terminology so Claude can match this skill to natural user requests.

DimensionReasoningScore

Specificity

The description mentions no concrete actions whatsoever. 'Core conventions and patterns' is abstract language that doesn't describe what the skill actually does. 'ExcelMcp' hints at a domain but no capabilities are listed.

1 / 3

Completeness

The description fails to answer both 'what does this do' and 'when should Claude use it'. There is no 'Use when...' clause and no meaningful explanation of functionality.

1 / 3

Trigger Term Quality

'ExcelMcp' is a technical/internal term that users would not naturally say. There are no natural keywords like 'spreadsheet', 'Excel', '.xlsx', 'pivot table', or 'charts' that a user might use.

1 / 3

Distinctiveness Conflict Risk

'Core conventions and patterns' is extremely generic and could apply to virtually any skill. Without specific triggers or domain actions, it would be impossible to distinguish this from other skills.

1 / 3

Total

4

/

12

Passed

Validation

100%

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

Validation11 / 11 Passed

Validation for skill structure

No warnings or errors.

Repository
sbroenne/mcp-server-excel
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.