CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

golang-expert

Senior Go engineer and technical advisor — use for any Go question, design decision, or implementation task. Covers idiomatic patterns, concurrency, error handling, project structure, performance optimisation, observability, and testing. Invoke whenever the user is writing, reviewing, debugging, or designing Go code, asking about goroutines, channels, interfaces, generics, modules, or benchmarks, or wants a second opinion on a Go architecture. Triggers on: Go, Golang, goroutine, channel, interface, gRPC, go.mod, go test, pprof, golangci-lint.

Quality

88%

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

SKILL.md
Quality
Evals
Security

Quality

Content

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

A well-structured skill body: house standards are stated as concrete, executable rules with a clear on-demand reference system verified against the real bundle files. The main deductions are a non-compiling error-handling snippet, no explicit fix-and-retry feedback loop in the implementation workflow, and a couple of boilerplate code examples Claude doesn't need.

Suggestions

Fix the error-handling snippet so it compiles: declare err via 'err := row.Scan(&u)' before the errors.Is(err, sql.ErrNoRows) check — it currently references an undefined err and would mislead as a copy-paste template.

Add an explicit feedback loop to the implementation workflow, e.g. step 5b: 'If validation fails, fix the reported issue and re-run before proceeding' — this would raise workflow clarity to the 5-anchor level.

Trim or relocate the constructor and graceful-shutdown boilerplate snippets (standard patterns Claude already knows) into the relevant reference files to tighten token efficiency.

DimensionReasoningScore

Conciseness

The body is largely policy and house standards rather than teaching basics, and the Go 1.22 loop-variable note is genuinely non-obvious knowledge. However, the constructor snippet ("func NewUserService(store Store, log *slog.Logger)...") and the graceful-shutdown snippet are standard patterns Claude already knows, so a few tokens could be trimmed — fitting 'efficient; minor instances of over-explanation' rather than the every-token-earns-its-place level 5 anchor.

4 / 5

Actionability

Most guidance is concrete and executable: specific validation commands ("go test -race ./...", "golangci-lint run", "go vet ./..."), an errgroup example, and exact error-wrapping formats. But the error-handling snippet will not compile — "errors.Is(err, sql.ErrNoRows)" references an `err` that is never declared because the "row.Scan(&u)" call is missing — a pseudocode-like gap that keeps it below the fully copy-paste-ready level 5 anchor.

4 / 5

Workflow Clarity

The implementation workflow has a clear sequence with an explicit validation checkpoint ("Validate — go test -race ./..., golangci-lint run, go vet ./..."), and the 'Non-negotiable standards' list functions as a checklist. What's missing is an explicit failure-recovery loop ('if validation fails, fix and re-run'), so it matches 'clear sequence with most checkpoints present; minor validation gaps' (4) rather than the feedback-loops-present level 5 anchor. This is not a destructive/batch skill, so the level-3 cap does not apply.

4 / 5

Progressive Disclosure

The body is a well-organized overview: a topic/file/when-to-read table instructs "Load these on demand — read only the file(s) relevant", all five referenced files exist and are substantial one-level-deep references, and sections explicitly defer detail ("Full wiring ... lives in references/concurrency.md"). This matches the clear-overview with well-signaled one-level-deep references anchor.

5 / 5

Total

17

/

20

Passed

Description

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

A strong description: it explicitly states what the skill does and when to invoke it, with a comprehensive, naturally-phrased trigger list including synonyms, file names, and tool commands. The only weakness is including the generic words 'channel' and 'interface' as standalone triggers, which creates minor conflict risk with non-Go skills.

DimensionReasoningScore

Specificity

The description lists multiple concrete capabilities — "writing, reviewing, debugging, or designing Go code" and "a second opinion on a Go architecture" — with comprehensive topic coverage ("idiomatic patterns, concurrency, error handling, project structure, performance optimisation, observability, and testing"). Coverage of the Go domain is thorough with no notable gaps, matching the comprehensive anchor rather than the 'minor gaps' level 4 anchor.

5 / 5

Completeness

Both questions are answered explicitly: 'what' via "Senior Go engineer and technical advisor ... Covers idiomatic patterns, concurrency, error handling ..." and 'when' via "Invoke whenever the user is writing, reviewing, debugging, or designing Go code, asking about goroutines, channels, interfaces ...". This mirrors the level-5 anchor's concrete what-and-when structure; the 'Use when...' clause is present, so the level-3 cap does not apply.

5 / 5

Trigger Term Quality

"Triggers on: Go, Golang, goroutine, channel, interface, gRPC, go.mod, go test, pprof, golangci-lint" covers synonyms (Go/Golang), file names (go.mod), commands (go test), and tool names (pprof, golangci-lint) — all phrases a user would naturally say when needing Go help. This matches the comprehensive synonym-and-extension anchor, not merely 'good keyword coverage' at level 4.

5 / 5

Distinctiveness Conflict Risk

The niche is clear (Go) and most triggers (Go, Golang, goroutine, go.mod, pprof) are highly distinct, but the standalone trigger words "channel" and "interface" are generic terms that could overlap with non-Go skills (UI interfaces, messaging channels). This lands on 'mostly distinct; minor overlap risk' (4) rather than the minimal-conflict level 5 anchor, whose example triggers are all unambiguously domain-specific.

4 / 5

Total

19

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

frontmatter_unknown_keys

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

Warning

Total

14

/

16

Passed

Repository

rikdc/ai-skills

Path

plugins/dev-skills/skills/golang-expert/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.