CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

netalertx-code-standards

NetAlertX coding standards and conventions. Use this when writing code, reviewing code, or implementing features.

58

Quality

73%

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 ./.github/skills/code-standards/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

67%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 instruction-only standards skill: rules are specific, reference real files and functions, and include executable snippets, correct/wrong contrasts, and a per-step review workflow with a simulate-before-live-test validation loop. The main cost is token efficiency — the long opening bullet list duplicates the themed sections that follow, and the 'real case' narratives run long.

Suggestions

Delete or collapse the 23-bullet preamble that duplicates the themed sections (File Length, DRY, Database Access, Subprocess Safety, Time Utilities, Test Helpers); the sections already carry the rules with full detail.

Name the actual sanitizer functions in server/helper.py instead of 'use sanitizers from server/helper.py', and specify the minimum subprocess timeout policy (the snippet says 'Minimum 60s' while the rule only says 'explicit timeouts').

Trim each multi-paragraph 'A real case' narrative to a one-two line example, or move them into a references/ file so SKILL.md stays a lean rulebook.

DimensionReasoningScore

Conciseness

The body is mostly lean, imperative rules, but the 23-bullet preamble substantially duplicates the themed sections that follow (e.g., 'code files should be less than 500 LOC' vs the 'File Length' section, 'all subprocess calls must set explicit timeouts' vs 'Subprocess Safety', 'never access DB directly' vs 'Database Access'), and multi-paragraph 'A real case' narratives add padding. This fits anchor 3 — 'mostly efficient but includes some unnecessary explanation or could be tightened' — better than anchor 4, where only minor instances would be trimmable.

3 / 5

Actionability

Guidance is highly concrete: named functions and paths ('normalize_mac from plugin_helper.py', 'timeNowUTC from utils.datetime_utils', 'server/db/db_helper.py'), executable snippets for subprocess timeouts and test imports, exact grep commands for language-string reuse, and correct/wrong contrasts for docstrings and MAC literals. It stops short of anchor 5 due to minor gaps such as 'use sanitizers from server/helper.py' without naming the sanitizer functions, and vague directives like 'follow existing code style and structure'.

4 / 5

Workflow Clarity

A sequenced development workflow is stated with explicit ordering cues and review gates: 'before starting, prepare implementation plan', 'ask me to review before going to each next step (mention n step out of x)', 'add test creation as last step', plus a validation checkpoint — simulate with a disposable script before declaring a JS/PHP change ready for live testing. It is below anchor 5 because the workflow is scattered across the bullet preamble and several sections rather than consolidated, and some checkpoints are implicit; above anchor 3 because ordering and validation are explicitly present.

4 / 5

Progressive Disclosure

The body is organized into clearly headed, well-scoped sections (Database Access, Subprocess Safety, Docstrings, PHP/Python Boundary, Test Helpers, Path Hygiene) and a single-file layout is appropriate for a standards reference with no bundle files. This matches anchor 4 — 'good structure; most content is appropriately placed; minor organization gaps' — the gap being the redundant preamble bullet list that mirrors the section content and inline narrative case studies that could be tightened or split.

4 / 5

Total

15

/

20

Passed

Description

65%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 correctly uses third person and includes an explicit 'Use this when...' clause with natural trigger phrases, satisfying the completeness floor. Its main weakness is specificity: 'coding standards and conventions' names the domain without enumerating any concrete capabilities or coverage areas.

Suggestions

Replace the generic 'coding standards and conventions' with 2-3 concrete capability statements, e.g., 'Enforces NetAlertX conventions: DB access only via server/db/db_helper.py handlers, camelCase DB columns, timeNowUTC UTC timestamps, subprocess timeouts, lowercase MAC normalization, and shared test mocks from test/db_test_helpers.py.'

Broaden trigger coverage with natural variations users would say: 'contributing to NetAlertX, submitting PRs, refactoring server/ Python code, or adding frontend language strings'.

Keep the description in third person and under ~1024 characters while adding this specificity.

DimensionReasoningScore

Specificity

The description names the domain ('NetAlertX coding standards and conventions') but the actions are minimal and generic — 'standards and conventions' is a category, not a concrete capability, unlike anchor-3/5 examples that enumerate actions such as 'extracts text, fills forms, merges documents'. It is above anchor 1 because the domain is named; below anchor 3 because no concrete action or coverage area (DB access rules, timestamp handling, testing patterns) is stated.

2 / 5

Completeness

Both parts are present: the 'what' ('NetAlertX coding standards and conventions') and an explicit 'when' clause ('Use this when writing code, reviewing code, or implementing features') with concrete trigger phrases. It falls short of anchor 5 because the 'what' is broad — it does not state which areas of standards are covered (database access, testing, frontend/PHP boundary), so the answer to 'what does this do' could be more specific.

4 / 5

Trigger Term Quality

'writing code, reviewing code, or implementing features' are natural phrases a user would say, giving good keyword coverage. It sits at anchor 4 rather than 5 because common variations and synonyms are missing — e.g., 'refactoring', 'submitting PRs', 'fixing bugs', 'contributing to NetAlertX'.

4 / 5

Distinctiveness Conflict Risk

The project-scoped name 'NetAlertX' gives it a clear niche that would not trigger for generic skills, matching 'mostly distinct; minor overlap risk'. It is not a 5 because the trigger phrases ('writing code, reviewing code') are generic coding verbs — inside the NetAlertX repo they could overlap with closely related skills such as the referenced 'testing-workflow' skill.

4 / 5

Total

14

/

20

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.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

netalertx/NetAlertX

Path

.github/skills/code-standards/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.