Skills Guide

Revue de code Go

VérifiéSûr

Analyse les packages Go pour la sécurité de concurrence, la gestion d'erreurs, la conception idiomatique et la performance.

Télécharger et utiliser
Spar Skills Guide Bot
DeveloppementAvancé
23/07/2026
Claude Code
#go#code-review#concurrency#error-handling#performance

Recommandé pour

Développeur backendClaude CodeRevue de codeDéveloppeur frontendDéveloppeur fullstackRefactoring

Synthèse automatique

Passe en revue des paquets Go en se concentrant sur la sécurité de la concurrence, la gestion explicite des erreurs, la conception idiomatique et les chemins d'allocation critiques.

Points forts

  • Couvre cinq axes précis : performance/mémoire, architecture des paquets, idiomatisme Go, concurrence et robustesse/sécurité
  • Détecte les problèmes classiques de Go comme les fuites de goroutines, les defer dans les boucles et les erreurs silencieusement ignorées
  • S'appuie sur go vet, go test -race et staticcheck pour appuyer ses constats
  • Ne modifie pas le code source et produit un rapport structuré hiérarchisé

Limites

  • Doit être invoqué manuellement, il ne se déclenche pas automatiquement sur les fichiers .go
  • Dépend de fichiers partagés (_shared/review-workflow.md et review-output-format.md) non inclus dans ce SKILL.md
  • Ne fournit pas de corrections appliquées ni de patch, seulement des constats
Quand l'utiliser

Utilisez-le lorsqu'un paquet Go doit être audité avant fusion, surtout pour du code concurrent ou sensible aux performances.

Quand l'éviter

Évitez-le pour des scripts Go triviaux ou lorsque vous attendez des corrections de code automatiques plutôt qu'une revue.

Analyse de sécurité

Sûr
Score qualité82/100

This is a pure instruction file for reviewing Go code. It declares no allowed-tools, disables model invocation (manual-only), and only mentions optional, non-destructive developer commands (`go vet`, `go test -race`, `staticcheck`). There is no network exfiltration, no file deletion, no obfuscated payloads, no privilege or safety-bypass instructions. The instructions explicitly say not to modify source code, so there is no meaningful execution risk.

Aucun point d'attention détecté

Comment cette analyse est faite

Exemples

Review a Go HTTP handler
Review the Go package in ./internal/handler for concurrency safety and error handling issues.
Audit goroutine lifecycle
Check ./pkg/worker for goroutine leaks, channel deadlocks, and missing context.Context propagation.
Performance pass
Review ./service for unnecessary heap allocations and garbage collection pressure, and suggest zero-allocation or sync.Pool improvements.

name: review-go description: >- Reviews Go packages for concurrency safety, error handling, idiomatic design, and performance. Invoke manually for .go files or Go review requests. disable-model-invocation: true

Go Review

Role

Senior Go Engineer and Backend Systems Architect. Evaluate concurrency safety, explicit error handling, package boundaries, and allocation hot paths. Flag race conditions, goroutine leaks, and silent error swallowing.

Workflow

Follow _shared/review-workflow.md. Optional tools: go vet ./..., go test -race ./..., staticcheck ./....

Instructions & Review Criteria

1. Efficiency, Performance & Memory

  • Identify CPU or memory allocation bottlenecks, unnecessary heap allocations, or sub-optimal data structure usage.
  • Spot issues related to garbage collection pressure. Recommend zero-allocation patterns or sync.Pool where appropriate.

2. Code Design, Architecture & Interfaces

  • Assess package structure and decoupling boundaries. Keep interfaces small and implicit where idiomatic.
  • Verify clean separation of concerns without over-engineered abstractions.

3. Best Practices & Idiomatic Go

  • Enforce idiomatic Go patterns per Effective Go and gofmt / go vet rules.
  • Evaluate error handling: explicit errors, %w wrapping, reliable defer cleanup (watch defer in loops).

4. Concurrency Safety & Goroutines

  • Audit race conditions, thread-safety, sync.Mutex / sync.RWMutex usage.
  • Verify goroutine and channel lifecycle — no leaks, deadlocks, or panic on closed channels. Check context.Context propagation.

5. Robustness, Security & Edge Cases

  • Scan for nil pointers, unsafe conversions, integer overflows.
  • Identify unvalidated input, insecure crypto, or path traversal risks.

Response Structure

Use _shared/review-output-format.md. Do not modify source code.

Example finding

Input: go func() without WaitGroup or shutdown sync

### Critical Fixes (High Priority)
- `handler.go:88` — goroutine leak; no shutdown sync

Additional resources

Du fichier au premier essai

Le téléchargement contient exactement le contenu affiché sur cette fiche. Cette copie peut différer de la dernière version du dépôt source.

Consulter la source et ses fichiers annexes

Ce téléchargement contient uniquement SKILL.md. Les scripts, références, modèles et dépendances éventuels ne sont pas inclus. Récupérez les fichiers nécessaires depuis la source en conservant leur arborescence avant de poursuivre.

1. Télécharger le fichier

Télécharger SKILL.md

Depuis la racine d’un projet de test, enregistrez le fichier sous .skill-downloads/go-code-review-mrx3xz19/SKILL.md, ou utilisez la commande suivante. Ce dossier sert à examiner la skill avant son installation.

Commande pour macOS/Linux ou WSL : un shell POSIX et curl sont nécessaires. Sous Windows sans WSL, utilisez le bouton de téléchargement et créez le dossier manuellement.

mkdir -p '.skill-downloads/go-code-review-mrx3xz19' && curl --fail --show-error --location 'https://skills-guides.com/api/skills/go-code-review-mrx3xz19/download' --output '.skill-downloads/go-code-review-mrx3xz19/SKILL.md.part' && mv '.skill-downloads/go-code-review-mrx3xz19/SKILL.md.part' '.skill-downloads/go-code-review-mrx3xz19/SKILL.md'

2. Vérifier les prérequis

Lisez SKILL.md avant de l’utiliser. Si le fichier cite des scripts ou services externes, examinez-les et préparez les accès nécessaires dans votre environnement de test.

Aucune section de prérequis reconnue dans ce fichier. Cela ne signifie pas qu’il est autonome : les bibliothèques, outils et accès peuvent être cités ailleurs dans le contenu ou dans le dépôt source.

Les plateformes de la fiche sont des indications du catalogue. Les chemins ci-dessous suivent la documentation des outils ; ils ne prouvent pas que cette skill y a été exécutée. Aucune version minimale de ces outils n’est attestée pour cette fiche.

3. Charger la skill

Après vérification, copiez le dossier complet dans le projet avec l’une des commandes ci-dessous, puis ouvrez une nouvelle session de l’outil. La commande s’arrête si un dossier du même nom existe déjà : comparez les fichiers avant de le remplacer.

Claude Code
test -f '.skill-downloads/go-code-review-mrx3xz19/SKILL.md' && test -s '.skill-downloads/go-code-review-mrx3xz19/SKILL.md' && mkdir -p '.claude/skills' && test ! -e '.claude/skills/review-go' && cp -R '.skill-downloads/go-code-review-mrx3xz19' '.claude/skills/review-go'

OpenSkills

OpenSkills doit déjà être installé. La procédure locale install/read est contrôlée avec OpenSkills 1.5.0, qui exige Node.js ≥ 20.6.0. Aucun slug du catalogue n’est résolu automatiquement.

Installation officielle d’OpenSkills
openskills install './.skill-downloads/go-code-review-mrx3xz19'

openskills read 'go-code-review-mrx3xz19'

La lecture doit afficher le contenu du fichier installé et son dossier de base. Elle ne lance ni la skill ni ses dépendances.

Pour un agent qui lit AGENTS.md, synchronisez ensuite l’index. Cette commande modifie AGENTS.md : relisez le diff avant de l’utiliser.

openskills sync

Mode manuel ou autre outil : joignez SKILL.md à une conversation, ou ouvrez-le dans un agent capable de lire les fichiers. Ajoutez les ressources annexes nécessaires. Il n’existe pas de dossier universel reconnu par tous les outils.

4. Faire un premier essai

Utilisez une entrée courte et fictive : un endpoint d’API, un petit document ou un brief d’interface. Dans une nouvelle conversation, joignez le fichier ou donnez son chemin à l’agent, puis adaptez cette demande :

Lis .skill-downloads/go-code-review-mrx3xz19/SKILL.md et applique ses instructions à cet exemple : [décrire une entrée de test]. Indique les fichiers, dépendances ou accès qui manquent. Produis le résultat demandé par la skill sans inventer de données ni exécuter de script non examiné.

Résultat attendu : une sortie correspondant aux instructions de la skill, ou une liste précise des prérequis manquants. Une simple confirmation de lecture ne valide pas le premier usage. Si une ressource externe est indisponible, ce parcours reste à tester.

Skills similaires

Bonnes pratiques React Hook Form

Developpement

Maîtrisez React Hook Form : évitez les anti-patterns, gérez les abonnements, et traitez correctement reset, dirty state et entrées contrôlées.

Claude CodeCursorWindsurf+2intermediateSûr 108,623
Skills Guide Bot

Migration Emotion vers Mantine

Developpement

Migrer les styled-components Emotion vers des composants Mantine avec style props et modules CSS. Pour convertir .styled.tsx ou supprimer les imports @emotion.

Claude CodeCursorWindsurf+1intermediateSûr 48,981
Skills Guide Bot

Détecteur de dérive de périmètre

Developpement

Analyse les diffs git par rapport à une intention pour détecter la dérive de périmètre, fichiers non liés et PR trop larges, puis suggère de garder, diviser ou justifier.

Claude CodeCursorCodexintermediateSûr 135,015
Skills Guide Bot