code-review

name	code-review
description	BoxLang code review guidelines. Load this skill when asked to review, audit, or critique code for correctness, style, or best practices.

Code Review Guidelines

When reviewing BoxLang code, check for the following in order of priority:

1. Correctness

• Variables at script scope must NOT use var; only use var inside functions.
• Verify null-safe navigation (?.) is used when properties may be absent.
• Confirm closures capture the right variables (closures capture by reference).

2. Style & Readability

• Operators and commas must have surrounding spaces: a + b, fn( a, b ).
• Use named arguments for clarity when calling BIFs: aiChat( prompt: msg, params: { model: "..." } ).
• Prefer lambdas (->) for simple one-liners and closures (=>) when outer scope is needed.

3. Performance

• Avoid repeated aiChat() calls inside loops — batch or pipeline where possible.
• Prefer runAsync() for work that can be non-blocking.

4. Response Format

Structure every code review as:

**Summary**: One-sentence verdict.

**Issues** (numbered list, most critical first):
1. ...

**Suggestions** (optional improvements):
- ...