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

Code Review General

skill-ucdavis-ai-skills-registry-code-review-general · by ucdavis

General code review process: priority ordering, what to block on, how to give actionable feedback

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

Install

$ agentstack add skill-ucdavis-ai-skills-registry-code-review-general

✓ 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-ucdavis-ai-skills-registry-code-review-general)

Reliability & compatibility

✓ Security review passed
0 installs to date
— no reviews yet
● 3mo 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 General? Claim this listing to set pricing, connect Stripe payouts, and keep 70% of every sale.
Sign up to claim

About

Code Review — General

Review Priority Order

  1. Correctness — does it do what it should? Are edge cases handled?
  2. Security — does it introduce vulnerabilities? (See security skill for checklist.)
  3. Performance — N+1 queries, unnecessary allocations in hot paths, missing indexes.
  4. Readability — will the next engineer understand this without context?
  5. Style — does it match project conventions? (Never block on style alone.)

Red Flags (always block)

  • Hardcoded secrets, API keys, or credentials.
  • Missing input validation at API/service boundaries.
  • Unhandled error paths that leave state corrupted.
  • Race conditions in concurrent code.
  • Silent exception swallowing (catch (e) {}).

Structural Issues (should fix)

  • Functions doing more than one thing.
  • Deep nesting — suggest early returns / guard clauses.
  • Duplicated logic that should be extracted.
  • Magic numbers or strings without named constants.

Suggestions (non-blocking)

  • Better variable/function names.
  • Simpler implementations.
  • Additional test cases for uncovered paths.

Tone

  • Quote the specific line. Explain the problem. Offer a concrete alternative.
  • Distinguish: "must fix", "should fix", "consider".
  • Approve with comments for minor issues — block only for correctness, security, or missing tests.
  • Separate your opinion from objective issues.

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