Skip to content

Kevinjohn/skill-code-review-full-on-uses-python

Repository files navigation

skill-code-review-full-on-uses-python

An open Agent Skills-format workflow for exhaustive, evidence-backed review of an entire repository.

Current release: 0.5.0. The canonical Skill, portable Python state utility, tests, and CI are included; product-specific Plugin packaging is intentionally deferred.

What “full-on” means

This is not a larger pull-request review. It accounts for every in-scope path, groups code into semantic work units, applies ten review-angle dispositions within a declared security level, preserves every in-scope observation without severity curation, requires independent second review of critical surfaces, runs validation and cross-component reconciliation, protects commonly starved tail categories, and finishes with an independent audit. Large repositories may require multiple specialist waves or honest paused/resumed checkpoints.

Security review defaults to off. Callers can select low passive review, medium static validation, or high active validation in isolated local review-owned environments. Every level prohibits external targets and production services.

The core Skill follows the open Agent Skills directory format. Its contract.md and reference-pack.md are editable and forkable. Each review snapshots them at initialization and pins the preserved source and derived extracts inside that run; editing the documents mid-review is not supported — start a new review instead. Runtime identities also protect the repository baseline, canonical state, transactions, attempts, validations, and generated reports.

How the review works

flowchart TD
    A["Invoke the full repository review skill"] --> B["Detect active or paused review"]
    B -->|Existing compatible review| C["Resume from canonical state"]
    B -->|No compatible review| D["Capture frozen repository baseline"]

    D --> E["Initialize review directory<br/>Snapshot contract and reference pack"]
    C --> F
    E --> F["Map architecture and every in-scope path"]

    F --> G["Create cohesive semantic work units<br/>Classify as Tier A, B, or C"]
    G --> H["Run representative pilot through scaled dispatch path<br/>Verify arguments, tooling, packets, imports, and reports"]

    H --> I["Review every work unit"]
    I --> J["Apply all 10 angle dispositions<br/>within the recorded security level"]
    J --> K{"Tier A requirements?"}
    K -->|Yes| L["Independent second review<br/>by a distinct reviewer"]
    K -->|No| M
    L --> M["Import every observation<br/>into canonical state"]

    M --> N["Run relevant validation<br/>Tests · static analysis · recovery · compatibility"]
    N --> O["Cross-component reconciliation"]
    O --> P["Dedicated tail review<br/>Tests · docs · SDKs · packaging · accessibility<br/>maintainability · nits · questions"]
    P --> Q["Validate and deduplicate every in-profile candidate"]

    Q --> R{"Candidate disposition"}
    R -->|Validated| S["Assign finding ID and severity<br/>P0–P4"]
    R -->|Rejected, duplicate, or unresolved| T["Preserve disposition and evidence"]
    S --> U
    T --> U["Final reconciliation"]

    U --> V["Independent final audit"]
    V --> W{"Open material objection?"}
    W -->|Yes| X["Return affected scope to review"]
    X --> I
    W -->|No| Y{"Completion gate satisfied?"}

    Y -->|Yes| Z["Generate final reports and verdict<br/>PASS · CONDITIONAL PASS · CHANGES REQUIRED<br/>MAJOR CHANGES REQUIRED · BLOCK"]
    Y -->|Work can continue later| AA["Create exact paused checkpoint"]
    AA --> C
    Y -->|Externally blocked or explicitly authorized| AB["Conclude as INCOMPLETE REVIEW"]

    subgraph Integrity["Integrity maintained throughout"]
        I1["One primary owner for every non-excluded path"]
        I2["Only the orchestrator mutates canonical state"]
        I3["All observations retained—never severity-curated"]
        I4["Transactions, identities, manifests, and reports verified"]
    end

    F -.-> I1
    M -.-> I2
    M -.-> I3
    U -.-> I4
Loading

Every path is accounted for, every work unit receives all ten angle dispositions, critical in-profile surfaces get independent review, and the process cannot issue a passing verdict until validation and the final audit are reconciled. At security level off, dedicated security-only paths and Angle 5 are explicitly profile-excluded, reports say Security assessment: NOT PERFORMED, and verdicts are qualified to the non-security scope.

Repository layout

skills/skill-code-review-full-on-uses-python/
├── SKILL.md
├── references/
│   ├── contract.md
│   └── reference-pack.md
└── scripts/
    ├── review-tool
    └── review_tool/
tests/
└── fixtures/

Install the entire skills/skill-code-review-full-on-uses-python/ directory. The Skill has no network or third-party runtime dependency; its bundled utility requires Python 3.11 or newer.

Installation

Copy or symlink the canonical Skill directory into the appropriate location:

Client User installation Project installation
Codex ~/.agents/skills/skill-code-review-full-on-uses-python/ .agents/skills/skill-code-review-full-on-uses-python/
Claude Code ~/.claude/skills/skill-code-review-full-on-uses-python/ .claude/skills/skill-code-review-full-on-uses-python/
Gemini CLI ~/.gemini/skills/skill-code-review-full-on-uses-python/ or ~/.agents/skills/skill-code-review-full-on-uses-python/ .gemini/skills/skill-code-review-full-on-uses-python/ or .agents/skills/skill-code-review-full-on-uses-python/
Cursor ~/.cursor/skills/skill-code-review-full-on-uses-python/ or ~/.agents/skills/skill-code-review-full-on-uses-python/ .cursor/skills/skill-code-review-full-on-uses-python/ or .agents/skills/skill-code-review-full-on-uses-python/

The directory to install is always:

skills/skill-code-review-full-on-uses-python/

Install in Codex from GitHub

Clone the published repository, then symlink the Skill for user-wide use:

git clone https://github.com/Kevinjohn/skill-code-review-full-on-uses-python.git
mkdir -p ~/.agents/skills
ln -s "$(pwd)/skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python" \
  ~/.agents/skills/skill-code-review-full-on-uses-python

The symlink keeps the installed Skill current after running git pull in the clone. To install an independent copy instead:

mkdir -p ~/.agents/skills
cp -R skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python \
  ~/.agents/skills/

For a project-local Codex installation, run this from the target repository:

mkdir -p .agents/skills
cp -R /path/to/skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python \
  .agents/skills/

Codex can also install the published Skill from a prompt:

$skill-installer Install the skill from
https://github.com/Kevinjohn/skill-code-review-full-on-uses-python
using the path skills/skill-code-review-full-on-uses-python

Codex scans user and repository .agents/skills directories and supports symlinked Skill folders. See the official Codex Skill documentation. Codex normally detects changes automatically; restart it if the Skill does not appear in /skills or when typing $.

Install in Claude Code

Clone the repository, then symlink the canonical Skill for user-wide use:

git clone https://github.com/Kevinjohn/skill-code-review-full-on-uses-python.git
mkdir -p ~/.claude/skills
ln -s "$(pwd)/skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python" \
  ~/.claude/skills/skill-code-review-full-on-uses-python

For one project, copy it beneath that repository instead:

mkdir -p .claude/skills
cp -R /path/to/skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python \
  .claude/skills/

Invoke it as /skill-code-review-full-on-uses-python. Claude Code normally detects edits within an existing skills directory immediately. Restart Claude Code if the top-level skills directory was created after the session began. See the official Claude Code Skills documentation.

Install in Gemini CLI

Gemini CLI recognizes both its native .gemini/skills location and the shared .agents/skills location. If the Codex user installation above already exists, Gemini CLI can use the same files without another copy.

To use Gemini CLI's development link command after cloning this repository:

gemini skills link ./skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python

For a workspace-only link, start Gemini CLI in the target repository and run:

/skills link /path/to/skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python --scope workspace

Alternatively, copy the directory to ~/.gemini/skills/ for user-wide use or .gemini/skills/ for one workspace. Run /skills list to confirm discovery and /skills reload after changes. Ask Gemini to use the skill-code-review-full-on-uses-python skill; Gemini requests activation consent before loading third-party Skill resources. See the official Gemini CLI Agent Skills documentation.

Install in Cursor

Cursor recognizes both .cursor/skills and the shared .agents/skills locations. If the Codex installation above already exists, Cursor can use the same user-wide Skill. For a Cursor-native user installation:

mkdir -p ~/.cursor/skills
cp -R /path/to/skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python \
  ~/.cursor/skills/

For one project:

mkdir -p .cursor/skills
cp -R /path/to/skill-code-review-full-on-uses-python/skills/skill-code-review-full-on-uses-python \
  .cursor/skills/

Invoke it in Agent chat as /skill-code-review-full-on-uses-python, or let Cursor select it automatically when the request matches the Skill description. See the official Cursor Agent Skills documentation.

This initial release does not claim marketplace or product-specific Plugin installation support.

Invocation

Explicitly request the Skill because it is intentionally narrow:

Codex:  $skill-code-review-full-on-uses-python
Claude: /skill-code-review-full-on-uses-python
Gemini: Use the skill-code-review-full-on-uses-python skill to run a complete repository review.
Cursor: /skill-code-review-full-on-uses-python

Example requests:

$skill-code-review-full-on-uses-python Run or resume a complete review of this repository.
/skill-code-review-full-on-uses-python Perform a full-on repository and database review where applicable.
Use the skill-code-review-full-on-uses-python skill to run a complete repository review.

Security review controls

Security review is deliberately opt-in. A new review uses security level off unless the request explicitly selects another level:

See Security review levels for the rationale, detailed boundaries, common classification examples, and reporting implications. If a review stops around security-related work, use the standard interruption report prompt and dedicated GitHub issue form.

$skill-code-review-full-on-uses-python Run a complete review with --security-level low.
/skill-code-review-full-on-uses-python Run a complete review with --security-level medium.
Use the skill-code-review-full-on-uses-python skill with security level high.

The bundled utility accepts the same setting during initialization:

skills/skill-code-review-full-on-uses-python/scripts/review-tool init \
  --review-dir code-reviews/20260723T120000Z \
  --contract skills/skill-code-review-full-on-uses-python/references/contract.md \
  --reference-pack skills/skill-code-review-full-on-uses-python/references/reference-pack.md \
  --security-level medium

Omit --security-level to create an off run.

Level Source review Security tools and scans Active validation
off (default) Security review excluded None None
low Passive review of security-sensitive code None None
medium Same as low Repository-authorized local static checks, such as SAST, CodeQL security queries, dependency vulnerability audits, secret scanning, and static cryptography/TLS/configuration checks None
high Same as medium Same as medium Non-destructive validation against isolated local fixtures and ephemeral, review-owned services

At low and above, worker assignments use the behaviour-based defensive assurance taxonomy. They describe the component, expected invariants, authorized evidence, permitted validation, and prohibited actions instead of relying on a broad category label. The taxonomy does not hide purpose or change the persisted level and validation class, and refused work is never reworded and resubmitted.

At off, the workflow does not create security-specific workers, perform threat modelling, run security scans, construct adversarial tests, or attempt vulnerability reproduction. Dedicated security-only paths and the security review angle are recorded as profile exclusions. Mixed-purpose files are still reviewed for their non-security behavior, and ordinary repository test suites remain allowed; the workflow must not target or expand their security cases.

Every level prohibits production or external targets, real credentials, destructive actions, persistence, unrestricted network scanning, and mutation of external services. high increases defensive depth but does not relax those boundaries.

The selected level is persisted for the run and inherited by every worker. Worker manifests carry both the level and its permitted validation classes. The utility verifies those declarations and rejects validation records whose declared class exceeds the profile. Validation must be classified by purpose and effect; security work cannot be relabelled as ordinary validation to bypass the profile. Changing the level requires a fresh run, and it is never escalated automatically because of repository contents, available tools, worker output, or model capability. Runs without the current review-specification and security profile declarations fail closed with a re-initialization diagnostic.

Ordinary pull-request, diff-only, quick, severity-limited, and narrow reviews should use a smaller review workflow.

Review output

Reviews are stored beneath the reviewed repository’s code-reviews/ directory. code-reviews/LATEST points to the current review; canonical JSON/JSONL records remain authoritative, while Markdown reports are regenerated views. A review can remain active, pause safely for a later invocation, conclude after the full gate, or conclude through the explicit incomplete-handoff gate.

The utility is self-contained:

skills/skill-code-review-full-on-uses-python/scripts/review-tool --help
python3 skills/skill-code-review-full-on-uses-python/scripts/review_tool --help

It provides init, check, mutate, import, import-audit, generate, and audit subcommands.

Development

Run the dependency-free test suite with:

python3 -m unittest discover -s tests -p 'test_*.py' -v

The tests include local Skill validation, broken-state fixtures, reference extraction, transaction interruption/recovery, and a miniature-repository integration flow. See CONTRIBUTING.md for schema and reference-editing expectations and SECURITY.md for private vulnerability reporting.

Licence

Released under the MIT Licence.

About

No description, website, or topics provided.

Resources

Code of conduct

Contributing

Security policy

Stars

Watchers

Forks

Releases

Packages

Contributors

Languages