Loading skill
Install any skill in seconds. Free to start, no credit card required.
Get Started Free →jame581/godot-code-review
Use when reviewing GDScript or C# Godot code — checklist of best practices, common anti-patterns, and Godot-specific pitfalls
Passed 21 of 22 test cases with the skill, and 18 of 22 without it.
and 6 more in the full report
4 fixed17 passed either way0 still failing1 broke
Tokens, turns and time describe resource use, not a dollar price.
One of 4 test cases the skill fixed. Same model, same prompt, with and without the skill.
Audit this GDScript snippet `inventory_item.gd`: ```gdscript extends Node var item_id = 0 func _ready(): print(get_parent().get_parent().name) ``` Please format your response using the standard Godot review summary format.
| Checked | Without skill | With skill |
|---|---|---|
| Uses the markdown checkbox bullet format `- [ ]` for each identified issue in critical or improvement sections. | Missed | Passed |
| Concludes the review summary with `Reviewed against: Godot 4.3+ best practices`. | Missed | Passed |
The instructions your model reads when the skill loads.
A structured review guide for Godot 4.3+ projects covering GDScript and C#. Work through each checklist section, then produce a review summary using the output template at the end.
Related skills: godot-testing for TDD and test coverage, scene-organization for scene tree best practices, godot-optimization for performance review.
extends hierarchiesget_parent() chains@onready (GDScript) or GetNode<T>() (C#) targets direct children or named paths within the same sceneget_parent() chain# BAD: tight coupling, breaks if the tree changes
func take_damage(amount: int) -> void:
get_parent().get_parent().get_node("HUD").update_health(health)
// BAD: tight coupling, breaks if the tree changes
public void TakeDamage(int amount)
{
GetParent().GetParent().GetNode("HUD").Call("UpdateHealth", _health);
}
# GOOD: parent/ancestor listens; child stays decoupled
signal health_changed(new_health: int)
func take_damage(amount: int) -> void:
health -= amount
test cases passed, vs 18 / 22 without
More skills from the same source and nearby in the registry, with their headline benchmark lift.