Compétences when-reviewing-code-comprehensively-use-code-review-assistant

📦

when-reviewing-code-comprehensively-use-code-review-assistant

Name: when-reviewing-code-comprehensively-use-code-review-assistant
Author: DNYoussef

v1.0.0 Révision du contenu r1 Risque élevé ⚙️ Commandes externes📁 Accès au système de fichiers🔑 Variables d’environnement

Effectuer des revues de code complètes

Les grandes pull requests sont difficiles à examiner de manière cohérente sur les plans de la sécurité, des performances, des tests et de la documentation. Cette compétence coordonne des agents de revue spécialisés et produit des retours sur la préparation à la fusion.

Prend en charge: Claude Codex Code(CC)

⚠️ 38 Médiocre

Maintain this Skill? Claim it ⚠️ 38 Médiocre

Arrêtez-vous et demandez confirmation avant l’installation.

Examinez le plan et obtenez le consentement explicite de l’utilisateur avant de modifier des fichiers.

Installer avec mon Agent

Copiez cette demande dans votre Agent. Elle contient la page Skill de référence et le manifeste.

Demande d’agent

Review the Skillstore skill "when-reviewing-code-comprehensively-use-code-review-assistant" from https://skillstore.io/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant.md and its manifest at https://skillstore.io/api/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/manifest. Verify the artifact. Stop and obtain explicit user consent before installing or changing files.

Votre Agent doit toujours présenter son plan et demander toute confirmation exigée par la politique de sécurité.

Ressources lisibles par les agents

Utilisez ces liens lorsqu’un Agent IA, un crawler ou un script a besoin d’un contexte propre au lieu de lire toute la page.

Détail Markdown GET /skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant.md manifest signé GET /api/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/manifest lockfile signé GET /api/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/lockfile

Tester

Utilisation de « when-reviewing-code-comprehensively-use-code-review-assistant ». Examine la pull request 123 pour toutes les dimensions de qualité.

Résultat attendu:

Un résumé des scores, les problèmes bloquants, les corrections recommandées et une décision claire de préparation à la fusion.

Utilisation de « when-reviewing-code-comprehensively-use-code-review-assistant ». Concentre-toi sur la sécurité et les risques liés aux dépendances.

Résultat attendu:

Une revue de sécurité priorisée couvrant les vulnérabilités critiques, les constats liés aux dépendances, les risques de secrets et les préoccupations d’authentification.

Utilisation de « when-reviewing-code-comprehensively-use-code-review-assistant ». Évalue si la branche de release peut être fusionnée.

Résultat attendu:

Une revue orientée release avec un statut de réussite ou d’échec, les blocages non résolus et le travail nécessaire avant approbation.

Audit de sécurité

Risque élevé

v10 • 09/07/2026 Ouvrir le rapport versionné

Most static findings were false positives caused by Markdown fences, example tables, and review criteria. I confirmed the generated auto-fix shell script risk and added semantic findings for npx-based package execution and repository-modifying auto-fixes. No prompt injection text was found.

Fichiers analysés

1,943

Lignes analysées

Éléments à examiner

Faux positifs ignorés

Préoccupations de sécurité confirmées (2)

Élevé

SKILL.md:36-56 PROCESS.md:13-31 README.md:8-18

Remote Package Execution Through npx

The skill directs agents to run npx commands, including claude-flow@alpha and other tooling, which can download and execute package code. This requires explicit confirmation, version pinning, and a trusted environment.

The cited sections explicitly instruct use of npx and npm commands during review setup and execution. The risk is contextual package execution, not a scanner-only pattern.

Moyen

SKILL.md:740-762 PROCESS.md:363-379

Auto-Fix Script Modifies Repository Files

The generated auto-fix script runs formatters, uses sed to edit configuration, and creates test files from a generated list. These operations can change source files and should require dry-run review and path validation.

The cited script block directly shows repository-modifying commands and file creation. The risk depends on user approval and path hygiene, so medium severity is appropriate.

Éléments d’évaluation des capacités (4)

Il s’agit de capacités locales réelles qui peuvent être attendues pour cette compétence ; elles nécessitent donc un examen, mais ne sont pas comptabilisées comme comportement malveillant confirmé.

Moyen

PROCESS.md:374

Shell command substitution

for file in $(cat missing-tests.txt); do

The auto-fix script iterates over missing-tests.txt and writes derived test file paths. A malformed or attacker-controlled list could cause unsafe word splitting or overwrite unintended repository paths.

Moyen

PROCESS.md:363

Unix shell invocation

#!/bin/bash

This begins an auto-fix shell script that runs npm and npx tools and modifies repository files. It is legitimate review automation, but risky without explicit user confirmation and path controls.

Moyen

SKILL.md:754

Shell command substitution

for file in $(cat missing-tests.txt); do

Moyen

SKILL.md:741

Unix shell invocation

#!/bin/bash

This begins an auto-fix shell script that runs npm and npx tools and modifies repository files. It is legitimate review automation, but risky without explicit user confirmation and path controls.

Facteurs de risque

⚙️ Commandes externes (50)

PROCESS.md:644-651 PROCESS.md:374 PROCESS.md:631 PROCESS.md:363 README.md:49 SKILL.md:37-41 SKILL.md:41-46 SKILL.md:46-57 SKILL.md:57-60 SKILL.md:60-69 SKILL.md:69-72 SKILL.md:72-90 SKILL.md:90-93 SKILL.md:93-99 SKILL.md:99-102 SKILL.md:102-118 SKILL.md:118-121 SKILL.md:121-126 SKILL.md:126-135 SKILL.md:135-136 SKILL.md:136-137 SKILL.md:137-147 SKILL.md:147-151 SKILL.md:151-156 SKILL.md:156-161 SKILL.md:161-164 SKILL.md:164-181 SKILL.md:181-184 SKILL.md:184-193 SKILL.md:193-196 SKILL.md:196-216 SKILL.md:216-219 SKILL.md:219-225 SKILL.md:225-240 SKILL.md:240-243 SKILL.md:243-248 SKILL.md:248-257 SKILL.md:257-258 SKILL.md:258-259 SKILL.md:259-269 SKILL.md:269-273 SKILL.md:273-278 SKILL.md:278-283 SKILL.md:283-286 SKILL.md:286-295 SKILL.md:295-298 SKILL.md:298-310 SKILL.md:310-313 SKILL.md:313-322 SKILL.md:322-325

📁 Accès au système de fichiers (4)

PROCESS.md:643 PROCESS.md:643 SKILL.md:168 SKILL.md:213

🔑 Variables d’environnement (8)

PROCESS.md:371 PROCESS.md:371 PROCESS.md:371 SKILL.md:751 SKILL.md:751 SKILL.md:645 SKILL.md:647 SKILL.md:751

Audité par: codex Voir l’historique des audits →

Partager et citer ce rapport

Partagez le rapport d’évaluation versionné, le badge neutre, la carte intégrable et les citations. Skillstore présente les preuves sans décider si ce Skill est sûr.

Ouvrir le rapport versionné

Copier le lien du rapport

https://skillstore.io/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/audits/10?utm_source=security_passport&utm_medium=share&utm_campaign=versioned_report

Badge Markdown

[![Skillstore security assessment](https://skillstore.io/badges/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/security.svg)](https://skillstore.io/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant?utm_source=security_passport_badge)

Badge HTML

<a href="https://skillstore.io/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant?utm_source=security_passport_badge"><img src="https://skillstore.io/badges/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/security.svg" alt="Skillstore security assessment" loading="lazy"></a>

Carte intégrable

<iframe src="https://skillstore.io/embed/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant.html" title="Skillstore Security Assessment" sandbox="allow-popups allow-popups-to-escape-sandbox" loading="lazy" referrerpolicy="no-referrer" width="420" height="180"></iframe>

Citations académiques (APA · BibTeX · CFF)

Citation APA

DNYoussef. (2026). when-reviewing-code-comprehensively-use-code-review-assistant security audit report (audit version 10) [Author version 1.0.0]. Skillstore. https://skillstore.io/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/audits/10

Citation BibTeX

@techreport{dnyoussef-dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant-2026,
  author = {DNYoussef},
  title = {when-reviewing-code-comprehensively-use-code-review-assistant security audit report (audit version 10)},
  institution = {Skillstore},
  year = {2026},
  number = {10},
  url = {https://skillstore.io/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/audits/10},
  note = {Author version 1.0.0}
}

CITATION.cff

cff-version: 1.2.0
message: "If you use this Skill, cite its author and this versioned security audit report."
title: "when-reviewing-code-comprehensively-use-code-review-assistant security audit report (audit version 10)"
version: "1.0.0"
type: report
authors:
  - name: "DNYoussef"
date-released: "2026-07-09"
url: "https://skillstore.io/skills/dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant/audits/10"
identifiers:
  - type: other
    value: "skillstore:dnyoussef-when-reviewing-code-comprehensively-use-code-review-assistant:audit:10"
    description: "Skillstore immutable audit report identifier"

Score Skillstore

Pourquoi ce score Niveau de confiance des preuves: Moyen

Architecture

100

Maintenabilité

Contenu

Communauté

Conformité aux spécifications

Ce que vous pouvez créer

Examiner une pull request de fonctionnalité

Effectuer une revue large avant de demander l’approbation des mainteneurs.

Vérifier les risques de sécurité et de dépendances

Concentrer l’effort de revue sur les problèmes OWASP, les secrets, l’authentification et les dépendances vulnérables.

Définir des seuils de préparation à la fusion

Utiliser des seuils de notation pour décider si une pull request doit avancer.

Essayez ces prompts

Revue basique de pull request

Examine la pull request 123 sur les plans de la sécurité, des performances, du style, des tests et de la documentation. Résume les blocages et les prochaines actions recommandées.

Revue ciblée sécurité et tests

Examine cette pull request pour l’authentification, l’autorisation, les secrets, les risques liés aux dépendances et les tests manquants. Signale uniquement les problèmes qui devraient bloquer la fusion.

Évaluation de la préparation à la fusion

Évalue la préparation à la fusion pour cette pull request. Note chaque dimension de revue et explique les changements requis pour atteindre l’approbation.

Renforcement du workflow de revue

Audite le workflow de revue et le processus de correction automatique. Identifie les commandes dangereuses, les validations manquantes, les seuils faibles et les améliorations CI.

Bonnes pratiques

Effectuer la revue dans une branche propre avec le contrôle de version activé.
Examiner l’exécution des commandes et les scripts générés avant d’autoriser les modifications de fichiers.
Ajuster les seuils au risque du projet, aux besoins de conformité et aux exigences CI.

Éviter

Exécuter des scripts de correction automatique sur des branches de production sans revue.
Considérer le score comme une approbation finale sans jugement humain.
Utiliser la compétence sans les outils de sécurité, de test et de lint requis installés.

Foire aux questions

Cette compétence remplace-t-elle la revue de code humaine ?

Non. Elle organise les contrôles et les constats, mais les mainteneurs doivent vérifier les résultats avant d’approuver une pull request.

Quels outils attend-elle ?

Les exemples incluent ESLint, npm audit, gitleaks, les outils de couverture, l’analyse de bundle et les scripts de revue du projet.

Peut-elle modifier des fichiers ?

Oui, les exemples de correction automatique peuvent modifier des fichiers. Examine d’abord les commandes et utilise le contrôle de version.

Est-elle adaptée à la CI ?

Oui, elle inclut des contrôles de préparation à la fusion de type CI. Épingle les dépendances et exige une approbation pour les commandes risquées.

Quelles dimensions de revue sont couvertes ?

Elle couvre la sécurité, les performances, le style de code, la couverture de tests, la qualité des tests et l’exhaustivité de la documentation.

Nécessite-t-elle une configuration de projet ?

Oui. Elle suppose que le dépôt dispose de scripts correspondants, de commandes de test, de règles de lint et d’une configuration d’outils de sécurité.