uw-code-review

A pre-commit review checklist checks Unity C# code against project rules, naming standards, and game-design test scenarios before the code is committed.

In plain words
What is it for?
It reviews target files, checks Unity-specific coding practices and naming, verifies that design scenarios have tests, and reports issues that must be fixed.
Why use it?
It helps catch quality, style, safety, and test-coverage problems after implementation and passing tests.

Skill for Claude CodeCodex

Install

Getting it into your agent

One page per mod, every tool's command on it. A separate URL per tool would split the same page into five that compete with each other.

agentmods
npx agentmods add skills/devdavv/unity-ai-workflow/uw-code-review
Any agent
npx skills add devdavv/unity-ai-workflow --skill uw-code-review
Clone the repo
git clone --depth 1 https://github.com/devdavv/unity-ai-workflow

Made for: Claude Code, Codex.

Per session 124 Skills are progressive disclosure: only the name and description are preloaded; the body loads when the skill is used.
When invoked 1,071 The whole file, excluding the scripts and references it only reads on demand.
Security scan A 0 findings. Scan, not verified.
Origin original No closer match found in the catalogue.
Token cost

What it costs to keep this loaded

Counted locally with the o200k_base tokenizer, which is exact for GPT models; Claude uses its own tokenizer and its counts differ. Treat this as one consistent yardstick across the catalogue rather than a bill. Prices are per million input tokens.

ModelPer sessionOnce invoked
Fable 5 $0.00124 $0.01071
Opus 5 $0.00062 $0.00535
Sonnet 5 $0.00025 $0.00214
Haiku 4.5 $0.00012 $0.00107

Measured 2d ago against content hash 69bd9ef4a1fc, method: parsed. Prices are Anthropic first-party input rates as of 2026-08-30, from the pricing page.

Security

Grade A, and why

uw-code-review scanned grade A with 0 findings against 26 rules in 11 categories — prompt injection, anti-refusal, data exfiltration, privilege escalation, supply chain, agent snooping, system-prompt leakage, SSRF and excessive agency — measured 2d ago.

A static scan of the body, not an audit. Every finding is printed with the line that produced it so you can judge whether it matters here. A mod is markdown that instructs an agent; that is exactly why what it instructs is worth reading.

Nothing flagged

None of the 26 patterns this scan looks for appear in this file: no shell pipes, no recursive deletes, no credential paths, no hidden text, no instruction-override or anti-refusal phrasing, no agent-config snooping. That is not a guarantee, it is the absence of the things that are checkable.

.claude/skills/uw-code-review/SKILL.md · 84 lines

How it starts

The opening of the file, as written. The whole thing — 84 lines — stays where its author put it; the contents beside it link to each section on GitHub.

Code Review

Pre-commit quality gate. Read the target files, then run this checklist. Output one structured report per review. Fix all issues before committing.

Before You Start

  1. Read docs/CODING_STANDARDS.md for formatting, class structure, serialization, null safety, and async rules.
  2. Read docs/NAMING_CONVENTIONS.md for naming rules (private fields, properties, files, asmdefs).
  3. Read docs/GDD.md for Gherkin scenarios — every scenario should have a corresponding test.
  4. Read the files being reviewed in full before assessing.

Checklist

1. Rules Compliance

  • All Inspector fields use [SerializeField] private — no public fields on MonoBehaviours
  • All component references cached in Awake() or Start() — never in Update/FixedUpdate/LateUpdate
  • GameDebug used instead of Debug.Log / Debug.LogError / Debug.LogWarning
  • New Input System only — no Input.GetKey* / Input.GetAxis*
  • TryGetComponent<T>() preferred over GetComponent<T>() where component may be absent
  • No Unity API calls from background threads
  • No .meta file modifications

2. Naming Conventions

  • Private fields: _camelCase
  • Public properties & methods: PascalCase
  • Parameters & local variables: camelCase
  • Booleans: is/has/can/should prefix (e.g. _isGrounded, CanJump)
  • Events: On prefix (e.g. OnDeath)
  • Static fields: s_camelCase
  • Class name matches file name

3. Code Quality

  • No magic numbers or hardcoded strings — use constants, ScriptableObjects, or serialized fields
  • No FindObjectOfType<T>() or GameObject.Find() (except in editor-only code)
  • No cross-assembly references that violate the TDD's asmdef dependency graph
  • K&R braces (opening brace on same line)
  • 4-space indentation (no tabs)

4. Performance

  • No new allocations inside Update/FixedUpdate/LateUpdate (no GC pressure in hot paths)
  • Particles are pooled or have Stop Action set to Destroy with pool backing
  • DOTween/PrimeTween tweens are killed in OnDestroy or OnDisable
  • No string concatenation in hot paths — use GameDebug with conditional compilation

Read the full file on GitHub · 84 lines

Changes

What this file has done since we first saw it

Hashed on every crawl. A supply-chain change to an agent config is a question of when, not whether, so the history is kept rather than the latest state alone.

  1. 2d ago First seen · 84 lines · 124 tokens per session scan A 69bd9ef4a1fc

Subscribe to this mod's changes

uw-code-review is a skill published in the GitHub repository devdavv/unity-ai-workflow (41 stars, last pushed 5mo ago), licensed MIT. It adds 124 tokens to every session and 1,071 once invoked, about $0.0006 per session on Opus 5. A static security scan graded it A with 0 findings. No closer match exists in the catalogue, so it is treated as the original; first seen 2026-08-30.

Related

Other skills, from other repositories

assets-shader-get-data

Get detailed data about a shader asset — properties, subshaders, passes, compilation messages, and supported status. Supports token-saving path-scoped reads via paths or viewQuery. Use 'assets-find' with t:Shader or 'assets-shader-list-all' to locate the shader first.

IvanMurzak/Unity-MCP · 70 tokens

gameobject-component-get

Get detailed information about a specific Component on a GameObject — type, enabled state, and (optionally) serialized fields and properties. Supports token-saving path-scoped reads via paths or viewQuery. Use 'gameobject-find' to list components first.

IvanMurzak/Unity-MCP · 59 tokens

gameobject-find

Find a specific GameObject in the opened Prefab (preferred when present) or the active Scene. Optionally include editable data, components preview, bounds, and limited hierarchy. Supports token-saving path-scoped reads via paths or viewQuery.

IvanMurzak/Unity-MCP · 55 tokens

gameobject-component-modify

Modify a specific Component on a GameObject in opened Prefab or in a Scene. Allows direct modification of component fields and properties without wrapping in GameObject structure. Use 'gameobject-component-get' first to inspect the component structure before modifying. Three modification surfaces are available…

IvanMurzak/Unity-MCP · 78 tokens

gameobject-modify

Modify GameObject fields and properties in opened Prefab or in a Scene. You can modify multiple GameObjects at once. Just provide the same number of GameObject references and SerializedMember objects. Three modification surfaces are available per GameObject (gameObjectDiffs, pathPatchesPerGameObject…

IvanMurzak/Unity-MCP · 79 tokens

object-modify

Modify a Unity UnityEngine.Object's serializable fields/properties. Three modification surfaces are available (objectDiff, pathPatches, jsonPatch) — see the skill body. Use 'object-get-data' first to inspect the object structure.

IvanMurzak/Unity-MCP · 57 tokens