# Section 02 Code Review

## Critical

1. **`total_requested` reports post-truncation size**: Should capture original expanded size before truncation.

## Important

2. **Abandoned undo/redo action when all nodes fail**: `create_action()` called but no `commit_action()` when created==0. Same pattern as existing `_handle_node_ops`.
3. **Endpoint node missing `align_to_path` rotation**: Last line node drops rotation.
4. **First node in line missing `align_to_path` rotation**: First node never gets rotation.

## Suggestion

5. `type` propagation in grid/line is undocumented deviation (sensible enhancement).
6. Guard for `points.size() < 2` is defensive but undocumented.
7. Grid with neither `scene` nor `type` generates specs that all fail in `_create_build_node`.
