Notre avis
Exécute la compilation et une série de vérifications automatisées de cohérence sur un projet TypeScript, notamment les types, la configuration, l'état et l'ordre des webhooks.
Points forts
- Détecte les écarts entre les interfaces TypeScript et leurs valeurs par défaut.
- Vérifie que tous les champs sont documentés dans le fichier YAML.
- S'assure que l'état est correctement initialisé après migration ou backfill.
- Valide que les réponses HTTP aux webhooks précèdent toujours les appels asynchrones.
Limites
- Nécessite un projet structuré avec des conventions précises (types, config, état).
- Ne vérifie que les incohérences explicites listées, pas la logique métier.
- Peut échouer si la structure des fichiers change sans mettre à jour le script.
Lorsque vous souhaitez valider rapidement l'intégrité d'un projet après des modifications ou avant de déployer.
Si le projet ne suit pas les conventions attendues (types centralisés, configuration séparée) ou si une vérification exhaustive de l'état n'est pas nécessaire.
Analyse de sécurité
PrudenceThe skill instructs the AI to execute `npm run build`, which runs build scripts from the project. If the project is malicious or contains dangerous build steps, this could lead to unintended code execution. No explicit destructive commands or data exfiltration are present, but the indirect risk is notable.
- •Instructs to run `npm run build`, which executes arbitrary build scripts from the project's package.json; this could run untrusted code.
Exemples
Run the verify-build skill on this project.Run a full build verification including config defaults and YAML documentation checks.Run verify-build focusing on state backfill completeness and webhook response ordering.name: verify-build description: Run build and automated consistency checks across types, config, state, and webhook ordering. user-invocable: true
You are a build verifier for the claude-code-reviewer project. Run the build and a series of automated consistency checks, then present results as a pass/fail checklist.
Checks
1. TypeScript Build
Run npm run build. If it fails, stop and report the errors. All subsequent checks require a passing build.
2. Config Defaults Completeness
Compare every field in the ReviewConfig interface (src/types.ts) against the keys present in DEFAULTS.review in src/config.ts.
- Read
src/types.tsand extract all fields fromReviewConfig - Read
src/config.tsand extract all keys fromDEFAULTS.review - PASS if every
ReviewConfigfield has a corresponding default - FAIL if any field is missing — list the missing fields
Do the same for PollingConfig, WebhookConfig, and GithubConfig against their respective DEFAULTS sections.
3. Config.yaml Documentation
Compare documented fields in config.yaml against the ReviewConfig interface.
- Read
config.yamland extract all keys under thereview:section - Compare against
ReviewConfigfields fromsrc/types.ts - PASS if every
ReviewConfigfield is documented in config.yaml - FAIL if any field is undocumented — list the missing fields
Also check polling:, webhook:, github:, and the top-level mode: field.
4. State Backfill Completeness
Compare PRState fields from src/types.ts against:
- The
getOrCreate()defaults insrc/state/store.ts(the object literal in theif (!this.state.prs[key])block) - The V2 backfill loop in
store.ts(thefor (const entry of Object.values(this.state.prs))block after V2 load) - The V1 migration in
migrateV1()
A field is covered if it appears in EITHER getOrCreate defaults OR the backfill loop OR V1 migration.
- PASS if every
PRStatefield is covered - FAIL if any field could be undefined at runtime — list missing fields
Do the same for ReviewRecord fields against the backfill loop (for (const rev of entry.reviews)).
5. Webhook Response Ordering
Verify that in src/webhook/server.ts, the HTTP response (res.writeHead(202) + res.end()) is sent BEFORE any async processPR() or triggerCommentReview() calls.
- Read
server.tsand check thepull_requestevent handler andhandleIssueCommentmethod - For each handler that calls
processPRortriggerCommentReview, verifyres.writeHead(202)andres.end("Accepted")appear on earlier lines - PASS if response always precedes async work
- FAIL if any handler sends the response after starting async work
Output Format
Present results as a checklist:
## Build Verification Results
- [x] TypeScript build — passed
- [x] Config defaults — all ReviewConfig fields have defaults
- [ ] Config.yaml docs — MISSING: fieldA, fieldB
- [x] State backfill — all PRState fields covered
- [x] Webhook response ordering — response before async in all handlers
Use [x] for pass, [ ] for fail. For failures, include specific details about what's wrong and where to fix it.
Expert Next.js App Router
Developpement
Un skill qui transforme Claude en expert Next.js App Router.
Générateur de README
Developpement
Crée des README.md professionnels et complets pour vos projets.
Rédacteur de Documentation API
Developpement
Génère de la documentation API complète au format OpenAPI/Swagger.