AgentStack
Browse Sign in
Browse Why AgentStack Sell Docs
Sign in
SKILL verified Unlicense Self-run

Code Review

skill-yale-som-hpc-claude-code-marketplace-code-review · by yale-som-hpc

Review code, diffs, or PRs for correctness, plus research/cluster pitfalls — hardcoded scratch paths, resource mismatch, GPFS tiny-file storms, package installs inside arrays. TRIGGER when reviewing code or a diff/PR, or before committing.

↗ Repository
— No reviews yet
0 installs
10 views
0.0% view→install

Install

$ agentstack add skill-yale-som-hpc-claude-code-marketplace-code-review

✓ scanned · ✓ verified, works with Claude Code, Cursor, and more.

Security review

✓ Passed

No issues found. Passed automated security review. · v0.1.0 How review works →

  • ✓ Prompt-injection patterns
  • ✓ Secret / credential exfiltration
  • ✓ Dangerous shell & filesystem operations
  • ✓ Untrusted network calls
  • ✓ Known-malicious package signatures

What it can access

  • ✓ Network access No
  • ✓ Filesystem access No
  • ✓ Shell / process execution No
  • ✓ Environment & secrets No
  • ✓ Dynamic code execution No

From automated source analysis of v0.1.0. “Used” means the capability is present in the source — more access means more to trust, not that it’s unsafe.

View the full security report →

Verified badge

Passed review? Show it. Paste this badge into your README, it links to the public security report.

AgentStack Verified badge Links to your public security report.
[![AgentStack Verified](https://agentstack.voostack.com/badges/verified.svg)](https://agentstack.voostack.com/security/report/skill-yale-som-hpc-claude-code-marketplace-code-review)

Reliability & compatibility

✓ Security review passed
0 installs to date
— no reviews yet
● 1mo ago

Declared compatibility

Claude CodeClaude Desktop

Compatibility is declared by the source manifest. End-to-end runtime verification is coming, see below.

Preview Execution monitoring

We're building live execution health for every listing: tool-call success rate, median latency, uptime, and last-checked timestamps, measured, not self-reported. It isn't live yet, so we don't show numbers we can't stand behind.

How agent discovery & health will work →
Are you the author of Code Review? Claim this listing to set pricing, connect Stripe payouts, and keep 70% of every sale.
Sign up to claim

About

Code Review

Find bugs that affect results, reproducibility, data safety, or cluster use. Skip style nits.

Findings must be

  • New in this change, unless asked for a broad audit.
  • Impactful: name the input/scenario.
  • Actionable: give the fix.

Severity

  • P0: blocks commit/run/merge. Wrong result, lost data, secret leak, cannot rerun, serious cluster misuse.
  • P1: fix before commit/merge. Realistic failure.
  • P2: fix later.
  • P3: nit. Do not report unless asked.

Verdict:

  • LGTM: no P0/P1.
  • FAIL: P0/P1 found.

Always check

  • Hardcoded personal paths: /Users/..., C:\Users\..., /home/netid/..., /gpfs/scratch60/netid/....
  • Raw data, secrets, env dirs, or large outputs staged.
  • Missing seeds in simulation, bootstrap, train/test split, UMAP, random forest, stochastic optimizer.
  • Silent data loss: dropna(), na.omit(), failed joins, duplicate drops, broad filters without counts.
  • Lockfile drift: deps changed but uv.lock / renv.lock not updated.
  • Raw data mutation: code writes into data/raw/ or overwrites inputs.
  • HPC resource mismatch: requested 1 CPU but uses all cores; requested GPU but code does not use it.
  • GPFS tiny-file storms.
  • Package installs inside arrays.
  • Scraping/API code without rate limit, cache, retry/backoff.

Process

  1. Read changed files fully.
  2. Check git status --short and staged files.
  3. Run existing checks if practical.
  4. Run a small realistic smoke test if practical.
  5. For Slurm changes: check resources, thread env vars, paths, logs, resumability.

Output

## Findings

### [P1] Missing seed makes bootstrap estimates non-reproducible
**File**: scripts/bootstrap.py:42
**Issue**: `np.random.default_rng()` is called without a fixed seed.
**Fix**: create `rng = np.random.default_rng(42)` at the entry point and pass it in.

## Verdict
FAIL

If clean:

## Findings
None.

## Verdict
LGTM

Checklist

  • [ ] Changed files read
  • [ ] Staged/untracked files checked
  • [ ] Seeds checked
  • [ ] Lockfiles checked
  • [ ] Smoke test/check run if practical
  • [ ] Slurm/resource behavior checked if relevant

Further reading

Source & license

This open-source skill is cataloged on AgentStack and links to its original source — we do not rehost the code.

Install and usage instructions live in the source repository linked above.

Reviews

No reviews yet, be the first.

Write a review

Versions

  • v0.1.0 Imported from the upstream source.

More in Agent Skills