Name: system-net-review
Rating: 76.8 (1 reviews)
Author: dotnet

system-net-review

Guidance for writing and modifying System.Net networking code in dotnet/runtime. Covers resource lifecycle, connection pooling, cross-platform interop, protocol compliance, async patterns, and security posture. For full code review, delegates to the @system-net-review agent. Trigger words: system.net, networking, http client, sockets, ssl, tls, quic, http2, http3, socketshttphandler, sslstream, connection pool.

Quality

96%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

Securityby

Passed

No findings from the security scan

Quality

Content

92%Weight 40%Scale 1-3

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

A dense, highly actionable reference: concrete code patterns, decision trees, and a clear delegation workflow, all written concisely. The main gap is progressive disclosure — everything lives in one long file rather than being split into reference materials.

Suggestions

Move the seven 'Code Patterns' blocks (and possibly the per-framework 'Key rules') into a references/ file (e.g. PATTERNS.md), keeping SKILL.md as an overview with one-level-deep links, to improve progressive disclosure.

Add a short 'Quick reference' or table-of-contents block at the top so readers can jump to the relevant decision framework without scanning the whole file.

If splitting files, ensure each reference link is clearly signaled (e.g. '**Buffer handling**: See [PATTERNS.md#buffers]') so navigation stays one level deep.

Dimension	Reasoning	Score
Conciseness	Lean decision trees, terse DO/DON'T code blocks, and 'Key rules' bullets assume Claude's competence; no basic-concept padding (e.g., it never explains what HTTP or TLS is). Every section earns its tokens.	3 / 3
Actionability	Provides concrete, copy-paste-ready C# for SocketAsyncEventArgs reuse, ArrayPool rent/return, SslClientAuthenticationOptions, linked CancellationTokenSource, and HTTP/2 flow control, plus specific decision-tree routing — fully executable guidance.	3 / 3
Workflow Clarity	The Review Delegation section gives a clear sequenced process (invoke @system-net-review → collect feedback → parent dedups/posts), and the six decision frameworks act as routing checklists; no destructive/batch operation requires a validation loop that is absent.	3 / 3
Progressive Disclosure	Well-organized with clear section navigation and explicit delegation of the detailed checklist to the agent ('Do not duplicate the agent's CHECK items here'), but it is a single ~275-line monolithic file with no bundle reference files, so content is not split across files as the top anchor expects.	2 / 3
	Total	11 / 12 Passed

Description

100%Weight 40%Scale 1-3

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, third-person description that states concrete capabilities, provides explicit trigger terms, and carves out a clearly distinct niche. It satisfies both the 'what' and 'when' requirements without verbosity.

Dimension	Reasoning	Score
Specificity	Lists multiple concrete actions and domains — 'writing and modifying System.Net networking code', 'Covers resource lifecycle, connection pooling, cross-platform interop, protocol compliance, async patterns, and security posture', and 'delegates to the @system-net-review agent' — matching the 'lists multiple specific concrete actions' anchor.	3 / 3
Completeness	Clearly answers 'what' (writing/modifying System.Net code across six named areas, delegating full review to an agent) and 'when' via an explicit trigger-words clause, satisfying the 'explicit trigger guidance' requirement that otherwise caps at 2.	3 / 3
Trigger Term Quality	Explicit 'Trigger words:' list covers natural terms a dotnet contributor would say — 'system.net, networking, http client, sockets, ssl, tls, quic, http2, http3, socketshttphandler, sslstream, connection pool' — giving good coverage of common variations.	3 / 3
Distinctiveness Conflict Risk	Narrow niche (System.Net in dotnet/runtime) with distinctive triggers like 'socketshttphandler', 'sslstream', and 'quic' make it unlikely to fire for unrelated skills.	3 / 3
	Total	12 / 12 Passed

Validation

93%

Warnings & errors only

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

Criteria	Description	Result
relative_links	Relative link issues: 7 suspicious	Warning

	Total	15 / 16 Passed

Repository: dotnet/runtime
Path: .github/skills/system-net-review/SKILL.md
Commit: ba10a6e

Reviewed: 8 days ago

Table of Contents

Discovery Implementation Validation

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.