skills.rest
scomofo
scomofoCOMMUNITY
Explore all skills by this author →
GitHub

code-review

Review GDScript/Godot files for architecture, SOLID, and ADR compliance.

Updated Mar 28, 2026
Search Tags:#architecture#quality assurance#code review#testability#gdscript#adr compliance#godot 4.6
One-click install
npx skills add https://github.com/scomofo/dragon-forge --skill code-review-scomofo
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: code-review
Source: https://github.com/scomofo/dragon-forge/tree/main/.agents/skills/code-review
Command: npx skills add https://github.com/scomofo/dragon-forge --skill code-review-scomofo

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

It helps you quickly identify architectural and quality problems in code—such as ADR drift, SOLID violations, and testability gaps—before they become expensive bugs or integration failures.

Core Features & Use Cases

  • Targeted architectural review: Loads specified file(s) and checks coding standards, architecture direction/layering, and SOLID principles.
  • ADR compliance analysis: Optionally infers the governing ADR from a provided story, implementation header comments, or commit history, then flags violations (blocking) and drift (warning/info).
  • Testability and QA alignment: For logic/integration work, evaluates whether existing story QA test cases map to observable, testable code paths and highlights untestable acceptance criteria.

Quick Start

Run /code-review src/combat/attack.gd production/epics/combat/story-001.md to review the implementation against relevant ADR guidance and standards.

Frequently Asked Questions about code-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I check my GDScript files for SOLID violations and architecture drift?▼

You can review GDScript files for SOLID violations and architecture drift by loading them into an automated code review that checks layering alignment against project coding standards and produces a structured verdict with required changes and suggestions.

What is ADR compliance analysis and how does it work with Godot 4.6 projects?▼

ADR compliance analysis evaluates your Godot project changes by inferring the governing Architecture Decision Record from a provided story or commit history, then flags blocking violations and drift warnings against expected design consequences.

How do I evaluate testability gaps in GDScript gameplay and UI logic?▼

You evaluate testability gaps in GDScript gameplay and UI logic by mapping existing story QA test cases to observable, testable code paths and highlighting untestable acceptance criteria within the implementation.

Can I review engine and tooling code changes without a dedicated ADR document?▼

Yes, you can review engine and tooling code changes without a dedicated ADR document by optionally inferring the governing architecture direction from a provided story description, implementation header comments, or commit history.

What's the best way to automate quality assurance checks for GDScript across engine and gameplay layers?▼

The best way to automate quality assurance for GDScript across engine and gameplay layers is running an architectural review that determines engine specialists from configured technical preferences and assesses design alignment, SOLID adherence, and testability.

Related Skills

Explore More Software Engineering Skills→
XGanQAQXGanQAQ
1

code-review

Review Unity/C# codebases for coding standards, ADR compliance, and SOLID principles.

Community
Advanced
epieczkoepieczko

story.write

Convert decomposed epic items into formatted user story Markdown files.

Community
Intermediate
minami110minami110
21

GDScript Test

Execute GDUnit4 tests for Godot projects and write reports.

Community
Basic
umairinayatumairinayat
5

lint-and-validate

Lint, format, and statically analyze Node.js and Python code changes.

Community
Intermediate