La compétence d'examen de code du dépôt agent-skills d'Addy Osmani : avant la fusion (ou lors de l'examen du code écrit par d'autres agents ou personnes), l'agent examine les modifications selon cinq dimensions : exactitude, lisibilité, architecture, sécurité et performance. Il examine d'abord les tests, puis l'implémentation, classe chaque remarque par niveau de gravité (Critique / Correction obligatoire / Nit / Optionnel / Pour information) et conclut par une approbation ou une demande de modifications.
Installations sur skills.sh 79294 (+975 / 24h)
Audit de sécurité : Non audité
- Pas encore scanné
Justification
- Le processus comprend 5 étapes : clarifier d'abord l'intention des modifications, examiner les tests avant l'implémentation, évaluer chaque fichier selon les cinq dimensions, classer les problèmes par gravité et vérifier les instructions de validation de l'auteur ; avant de générer le rapport, il faut éliminer les faux positifs en vérifiant les appelants, les garde-fous et les tests, et un résultat sans problème est une conclusion valide, sans obligation d'en inventer.
- La validité des tests est déterminée par expérimentation : inverser une condition dans les modifications et exécuter les tests, puis restaurer le fichier depuis une copie ; si les tests passent toujours, cela indique un cas de test manquant, qu'il faut identifier.
- Des références sur la taille des modifications sont fournies (environ 100 lignes conviennent à un examen unique, environ 1000 lignes nécessitent une division) ainsi que quatre méthodes de division ; pour les mises à jour de dépendances, il faut lire les journaux de modification, mettre à jour un seul paquet à la fois et examiner les différences dans les fichiers de verrouillage.
- En cas de code inutile détecté, dresser une liste avant de demander s'il doit être supprimé, sans suppression arbitraire ; lors de la signalement de problèmes structurels, proposer simultanément des solutions concrètes de refactorisation.
- Le dépôt est sous licence MIT, avec 24 autres compétences dans le même dépôt ; sur skills.sh, 975 installations en 24 heures, 79294 au total, avec une augmentation hebdomadaire des installations passant de 3503 à 5359 au cours des 8 dernières semaines (reflète uniquement l'attention portée).
Utilisation
- La commande d'installation s'affiche après validation de l'audit ; elle est déclenchée en demandant « Examiner cette PR / ce diff », lors de la préparation à la fusion après le développement d'une fonctionnalité, ou pour évaluer le code écrit par d'autres agents.
- Aucun compte ou service externe n'est requis ; l'étape « inverser une condition et exécuter les tests » nécessite que les tests du projet puissent être exécutés localement.
Risques
- Non audité pour cette période, classé C fixe, la commande d'installation n'est pas affichée ; SKILL.md est principalement une liste de contrôle d'examen, la seule action modifiant les fichiers étant « inverser temporairement une condition pour exécuter les tests, puis restaurer depuis une copie », ce qui est cohérent avec l'usage prévu, mais il faut confirmer que les fichiers ont été restaurés ; sur skills.sh, les audits Gen Agent Trust Hub et Socket sont « Pass », tandis que Snyk est « Warn ».
- Les sections sécurité et performance renvoient aux compétences security-and-hardening et performance-optimization du même dépôt, ainsi qu'aux fichiers de liste à la racine du dépôt ; ces liens peuvent être inaccessibles si seule cette compétence est installée.
- Les critères d'approbation sont larges : approbation accordée si la santé globale du code s'améliore ; les normes d'équipe telles que « répondre dans un délai d'un jour ouvrable » ont peu de sens pour les projets individuels.