Versioned security assessment

Report ID: SA-88A205C7

7/8/2026, 3:47:23 PM

requesting-code-review security assessment v4

Skill Security Certification Report

Audit History
Audit model: codex Historical report
Skill name
requesting-code-review
Version
v4
Maintainer
ZhanlinCui
Source
github.com/ZhanlinCui/Ultimate-Agent-Skills-Collection/tree/main/requesting-code-review
Coverage
2 Files scanned · 253 Lines analyzed
Policy version
Unavailable

Highest confirmed finding severity

Medium

1 confirmed security finding requires attention.

Installation context

Historical evidence

This report may not describe the currently installable artifact. Open the current Skill page for install guidance.

Open current Skill page

This report does not block or authorize the manifest or ZIP.

Most static warnings are false positives caused by Markdown fences, placeholder names, and example review text. I found no prompt injection, credential access, network exfiltration, or malicious intent in the reviewed files. One medium concern remains because the review template embeds revision placeholders in executable git commands.

Report position

Historical report

Open audit history before using this report to install.

Audit attestation

Not attestable

The required immutable binding is incomplete.

Human verification

Not verified

No human verification is recorded for this report.

Coverage

2 Files scanned · 253 Lines analyzed

1 item shown for review

Limitations

This report does not claim runtime or sandbox execution and does not prove the absence of side effects.

Evidence chain

Follow the evidence from source binding to the install contract. Available evidence supports verification; it is not a safety guarantee.

  1. Source

    Binding unavailable

  2. Artifact

    Identity incomplete

  3. Audit

    Complete

  4. Install contract

    Open manifest to verify

    Open manifest

Capabilities observed

Observed means this report recorded supporting evidence. Not recorded does not prove that a capability is absent.

Contains scripts

May execute code included with the Skill.

Not recorded by this audit

Network access

May connect to external services.

Not recorded by this audit

Filesystem access

May read or write local files.

Not recorded by this audit

Env variables

May read values from the process environment.

Not recorded by this audit

External commands

May invoke commands or programs outside the Skill.

Observed in 13 evidence locations

Risk findings

Confirmed security concerns are separated from items that still need review.

Confirmed security concerns (1)

RISK-001 Medium
code-reviewer.md:25-28
Unvalidated Git Revision Placeholders in Shell Commands
The review template places {BASE_SHA} and {HEAD_SHA} directly inside git diff shell commands. If those values come from untrusted text instead of validated git revisions, a user who executes the block in a shell could run unintended commands.
The commands are legitimate git review commands, but the placeholders are executable shell arguments. The risk depends on whether callers validate the revision values before use.

Remediation

Suggested fixes recorded by this audit. Applying them is the maintainer’s responsibility.

  1. FIX-001
    Medium
    Git diff commands use revision placeholders inside a shell block.
    State that BASE_SHA and HEAD_SHA must be validated git revisions before execution, or show commands using already verified commit hashes only.

Expert evidence

Immutable subject identity, scanner metadata, dismissed matches, and source-level evidence.

Artifact subject

Marketplace commit
Unavailable
Content hash
Unavailable
Tree hash
Unavailable
Skill path
Unavailable
Audit payload hash
Unavailable

Analysis metadata

Audit model: codex

Analysis state: Complete

Scope is limited to the recorded files, lines, methods, and evidence. No runtime or sandbox execution is claimed.

Verify and export

JSON report CITATION.cff SARIF 2.1.0 Signed installation manifest

The manifest and lockfile bind install artifacts to cryptographic hashes. This integrity claim is separate from the security assessment.

Audit attestation: not_attestable