CtrlK
BlogDocsLog inGet started
Tessl Logo

testing-and-ci

Testing conventions, CI pipeline rules, and smoke test coverage for SkillHub. Ensures agents write tests correctly and understand the CI gate requirements.

60

Quality

76%

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/testing-and-ci/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

82%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 strong, command-dense reference: exact paths, copy-paste make targets, naming conventions, and an explicit pre-PR validation checklist. Its main weaknesses are the absence of error-recovery guidance beyond the Maven trap note and roughly half of the CI workflow table being unrelated to testing.

DimensionReasoningScore

Conciseness

The body is dense with exact paths, make targets, and tables, and explains nothing Claude already knows. Minor content could be trimmed — the CI table's ~6 non-testing workflows (issue-triage, issue-backlog-rescore, release-notes, deepwiki, claim-issue-reward, statistic-member-reward) and the staging endpoint details — so it fits anchor 4 (efficient, minor instances that could be trimmed) rather than anchor 5's every-token-earns-its-place.

4 / 5

Actionability

Guidance is fully executable and copy-paste ready throughout: exact test directory paths, `make test-backend-app`, `make test-backend`, `make test-frontend`, `make test-e2e-frontend`, `make staging`/`staging-down`, explicit naming conventions (`{ControllerName}Test.java`, `SkillControllerTest.java`), and a concrete pre-PR checklist. Specific commands cover the common cases (backend, frontend, E2E, smoke, staging), matching anchor 5.

5 / 5

Workflow Clarity

The "Pre-PR Testing Checklist" provides a clear sequence with explicit validation checkpoints, and the Maven trap note is a genuine error-recovery loop (symptom "cannot find symbol" → cause: stale artifacts → fix: use -am or Makefile targets). However, there is no recovery guidance for staging or checklist failures (what to do when a step fails), so it sits at anchor 4 rather than anchor 5's full feedback loops.

4 / 5

Progressive Disclosure

The skill is a single file with no bundle directories; sections are well-organized (Trigger, Backend Testing, Frontend Testing, Smoke Tests, CI Pipeline, Staging, Pre-PR Checklist) and point to repo artifacts by path (scripts/*.sh, .github/workflows/*) rather than nesting references. At ~118 lines with the 11-row CI workflow table inlined — content that would belong in a reference file if a bundle existed — it fits anchor 4 (good structure, most content appropriately placed, minor organization gaps) rather than anchor 5's cleanly split content.

4 / 5

Total

17

/

20

Passed

Description

61%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 clearly identifies what the skill covers and is well-scoped to the SkillHub project, but it lacks an explicit 'Use when...' trigger clause and misses natural trigger variations like "unit tests" and "E2E". Actions are named as topic areas rather than concrete capabilities.

Suggestions

Add an explicit trigger clause, e.g. "Use when adding or modifying backend/frontend tests, changing CI/CD workflows, or adding smoke or E2E tests" — mirroring the body's existing Trigger section.

Include natural trigger variations users would say, such as "unit tests", "E2E tests", "PR checks", and "GitHub Actions".

State capabilities as concrete actions (e.g., "Run backend/frontend test suites, update smoke tests for changed workflows, validate PRs against the CI gate") rather than generic topic labels.

DimensionReasoningScore

Specificity

The description names the domain precisely ("Testing conventions, CI pipeline rules, and smoke test coverage for SkillHub") but the actions are topic labels rather than several specific concrete actions — "Ensures agents write tests correctly" is generic. It matches the anchor 'names domain and 1-2 concrete actions, but not comprehensive', falling short of anchor 4 which expects a list of several specific actions.

3 / 5

Completeness

The 'what' is clearly stated ("Testing conventions, CI pipeline rules, and smoke test coverage for SkillHub"), but there is no 'Use when...' clause or equivalent explicit trigger guidance — "Ensures agents write tests correctly and understand the CI gate requirements" is a purpose statement, not a trigger. Per the rubric guideline, a missing 'Use when' clause caps completeness at 3.

3 / 5

Trigger Term Quality

Terms like "tests", "CI", "CI pipeline", "smoke test", and "CI gate" are phrases users would naturally say. Coverage is good but misses common variations such as "unit tests", "E2E", "PR checks", or "GitHub Actions", so it fits anchor 4 (good keyword coverage, a few natural terms missing) rather than the comprehensive synonym/extension coverage of anchor 5.

4 / 5

Distinctiveness Conflict Risk

"for SkillHub" and "CI gate requirements" carve out a distinct project-specific niche with minimal conflict risk, but broad terms like "tests" and "CI" could still overlap with generic testing or code-review skills. This matches anchor 4 (mostly distinct, minor overlap risk) rather than anchor 5's fully distinct trigger set.

4 / 5

Total

14

/

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

referenced_paths_exist

Referenced path issues: 4 missing

Warning

Total

15

/

16

Passed

Repository
iflytek/skillhub
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.