code-review

Estrellas0

Forks0

Actualizado10 de junio de 2026 a las 12:49

Review code and data pipeline changes for correctness, security, performance, maintainability, contract breaks, and missing verification. Use before merge, after implementation, for PRs, commits, pending diffs, or codebase risk scans.

Instalación

Instalar con Codex o Claude Copia este prompt, pégalo en Codex, Claude u otro asistente, y deja que revise la página de la skill y la instale por ti.

Fuente

datacoolie

datacoolie/dekit

Abrir repositorio de GitHub Ver repositorios del creador

Descarga

Ocupaciones relacionadasSOC

Basado en la clasificación ocupacional SOC

Analistas de garantía de calidad de software y probadoresOcupaciones informáticas y matemáticas·SOC 15-1253

SKILL.md

readonly

name	code-review
description	Review code and data pipeline changes for correctness, security, performance, maintainability, contract breaks, and missing verification. Use before merge, after implementation, for PRs, commits, pending diffs, or codebase risk scans.

Code Review

Review adversarially. Findings require evidence from code, diff, tests, or runtime behavior.

Scope Resolution

PR or URL: review PR diff.
Commit hash: review that commit.
--pending: review staged and unstaged changes.
codebase: scan broader architecture/risk surface.
No explicit target: review recent changes in context or ask for target.

Review Order

Establish scope and changed files.
Check requirement/spec compliance.
Scout affected dependents and edge cases.
Review correctness, contracts, security, performance, tests.
Separate blocking findings from non-blocking observations.

Checklist

Contract breaks: API, schema, grain, metric, config, behavior.
Error handling: no silent swallowing or hidden partial failure.
Validation: inputs checked at trust and data boundaries.
Security: no secrets, PII leaks, injection, auth/authz gaps.
Data correctness: idempotency, schema evolution, quality gates, reconciliation.
Performance: query fan-out, Spark collect/cartesian/skew, unbounded loops.
Maintainability: clear ownership, no duplicated business logic, no speculative abstraction.
Verification: tests or direct checks prove the changed behavior.

Severity

Critical: security exposure, data loss, breaking contract, irreversible damage.
High: likely production failure, wrong results, missing required verification.
Medium: maintainability or performance risk with plausible impact.
Low: style or cleanup that does not block.

Output

Lead with findings:

Findings:
- [Critical|High|Medium|Low] file:line - issue, impact, evidence, fix

Open questions:
- ...

Summary:
- Reviewed: <scope>
- Verification observed: <commands/results or missing>

Más de este repositorio

mismo repositorio

wiki

datacoolie/dekit

Build and maintain an internal technical LLM wiki when the user requests wiki work or a verified major or architectural change qualifies for automatic wiki review. Use for project memory, architecture notes, contracts, runbooks, decisions, glossary, incremental ingest, delta status, and wiki health. Do not use merely because a routine task or plan completed.

2026-07-180

brainstorm

datacoolie/dekit

Explore ambiguous ideas and solution directions before evidence-backed research or implementation planning. Use for ideation, framing, assumption checks, option generation, product or architecture trade-offs, and deciding whether research or a plan is needed.

2026-07-030

research

datacoolie/dekit

Research technical options with source-backed recommendations. Use for technology evaluation, architecture trade-offs, library/framework choices, best practices, scalability, security, maintainability, or when current external facts matter.

2026-07-030

plan

datacoolie/dekit

Create implementation plans, architecture decisions, and phased roadmaps. Use for Standard or Complex changes, new pipelines, schema changes, migrations, multi-file features, technology choices, or when acceptance criteria and verification strategy are unclear.

2026-06-170

data-ingestion

datacoolie/dekit

Design data ingestion into landing/bronze layers. Use for source onboarding, transfer pattern selection, full/incremental/CDC scope, change detection, landing zones, schema drift, idempotent file or batch processing, and ingestion quality gates.

2026-06-100

data-modeling

datacoolie/dekit

Design warehouse and semantic data models. Use for grain definition, fact/dimension design, star schema, Kimball, Data Vault, SCD, bridge tables, surrogate keys, semantic metrics, and model review.

2026-06-100

name	code-review
description	Review code and data pipeline changes for correctness, security, performance, maintainability, contract breaks, and missing verification. Use before merge, after implementation, for PRs, commits, pending diffs, or codebase risk scans.

Code Review

Review adversarially. Findings require evidence from code, diff, tests, or runtime behavior.

Scope Resolution

PR or URL: review PR diff.
Commit hash: review that commit.
--pending: review staged and unstaged changes.
codebase: scan broader architecture/risk surface.
No explicit target: review recent changes in context or ask for target.

Review Order

Establish scope and changed files.
Check requirement/spec compliance.
Scout affected dependents and edge cases.
Review correctness, contracts, security, performance, tests.
Separate blocking findings from non-blocking observations.

Checklist

Contract breaks: API, schema, grain, metric, config, behavior.
Error handling: no silent swallowing or hidden partial failure.
Validation: inputs checked at trust and data boundaries.
Security: no secrets, PII leaks, injection, auth/authz gaps.
Data correctness: idempotency, schema evolution, quality gates, reconciliation.
Performance: query fan-out, Spark collect/cartesian/skew, unbounded loops.
Maintainability: clear ownership, no duplicated business logic, no speculative abstraction.
Verification: tests or direct checks prove the changed behavior.

Severity

Critical: security exposure, data loss, breaking contract, irreversible damage.
High: likely production failure, wrong results, missing required verification.
Medium: maintainability or performance risk with plausible impact.
Low: style or cleanup that does not block.

Output

Lead with findings:

Findings:
- [Critical|High|Medium|Low] file:line - issue, impact, evidence, fix

Open questions:
- ...

Summary:
- Reviewed: <scope>
- Verification observed: <commands/results or missing>