refactor(shell): reduce layout helper indirection #139

Merged

barrettruth merged 1 commit from refactor/ops-dispatch-cleanup into main

2026-04-28 23:36:56 +00:00

barrettruth commented

2026-04-28 23:33:43 +00:00

Problem

The remaining shell cleanup work still had repeated optional function checks, indirect variable-name expansion in the Fibonacci builder, and duplicated master-stack orientation mapping.

Solution

Add a shared function-defined helper, make the Fibonacci layout builder thread leaf ids explicitly instead of using indirect expansion, and centralize master-stack axis and mirroring helpers so the same behavior is expressed with less branching and indirection.

## Problem The remaining shell cleanup work still had repeated optional function checks, indirect variable-name expansion in the Fibonacci builder, and duplicated master-stack orientation mapping. ## Solution Add a shared function-defined helper, make the Fibonacci layout builder thread leaf ids explicitly instead of using indirect expansion, and centralize master-stack axis and mirroring helpers so the same behavior is expressed with less branching and indirection.