feat(flow editor): palette entries for the 9 input reporter nodes (#208)

[apotema]

Adds the input REPORTER nodes (flow-codegen #51 + #52) to the editor palette — an "Input" cluster. Mirrors the #207 string-reporter path.

Nodes (9, all rounded reporters with a value output, no exec pins)

• IsKeyDown/IsKeyPressed/IsKeyReleased — key field (bool out)
• IsMouseButtonDown/IsMouseButtonPressed/IsMouseButtonReleased — button field (bool out)
• GetMouseX/GetMouseY/GetMouseWheel — no field (f32 out)

What

• isReporterTypeName (flow_doc.zig): all 9 → rounded, no exec-in.
• .other pin-render arm: one shared arm emitting a value output (key/button is a FIELD, not a pin).
• other_field_specs (flow_io.zig): key (.text) for IsKey*, button (.text) for IsMouseButton*.
• Palette "Input" cluster (seed key="space" / button="left").
• Round-trip matches codegen: {"type":"IsKeyDown","key":"space"} etc.

Note (test-only)

The grown palette pushed the + Add raw call escape-hatch below the gui-test window's 720px fold (ImGui culls off-screen items, so the Test Engine couldn't click it). Bumped the hidden gui-test window to 1280x1080. Follow-up worth considering: the palette keeps growing (control/data/string/time/input clusters) — collapsible categories or test scroll-to-item would be more durable than raising the window.

zig build test 471/471; gui-test 26/26; zig build clean. Parent: labelle-gui#208.

[cursor]

PR Summary

Low Risk
Editor-only palette and classification changes with broad unit coverage; no auth, runtime, or data-path changes.

Overview
Adds nine input reporter nodes (flow-codegen #51/#52) to the flow editor so per-frame flows can read keyboard and mouse state from the palette, mirroring the earlier string-reporter work (#207).

Editor behavior: All nine are treated as rounded reporters with a single value output and no exec pins. Canvas rendering uses a shared isInputReporter path that exposes only value; IsKey* / IsMouseButton* bind key / button as on-node JSON fields (not data pins). other_field_specs wires those fields to text inspector widgets so saves match codegen (e.g. "key": "space").

Palette: A new Input cluster adds palette buttons for each node, seeding key="space" and button="left" where applicable.

Tests: Unit tests cover append, reporter classification, and parse/render round-trip. The hidden gui-test window height is raised to 1080 so the taller palette keeps + Add raw call… on-screen for ImGui Test Engine clicks.

^{Reviewed by Cursor Bugbot for commit 5ae0882. Bugbot is set up for automated code reviews on this repo. Configure here.}

[gemini-code-assist]

Code Review

This pull request adds support for nine input reporter nodes (key/mouse predicates and mouse getters) that poll input states and output a single value. It updates the node specifications, visual rendering, palette UI, and unit tests, as well as increasing the test window height to accommodate the expanded palette. The reviewer suggests simplifying the long chain of string comparisons in flow_doc.zig using an inline for loop to improve code readability and maintainability.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

[gemini-code-assist]

The long chain of std.mem.eql with or operators can be simplified and made more maintainable by using an inline for loop over an array of the input reporter names. This leverages Zig's compile-time unrolling to generate efficient code while keeping the list of names clean and easy to read/extend.

            } else if (inline for ([_][]const u8{
                "IsKeyDown",
                "IsKeyPressed",
                "IsKeyReleased",
                "IsMouseButtonDown",
                "IsMouseButtonPressed",
                "IsMouseButtonReleased",
                "GetMouseX",
                "GetMouseY",
                "GetMouseWheel",
            }) |name| {
                if (std.mem.eql(u8, n.type_name, name)) break true;
            } else false)

[apotema]

Done — extracted isInputReporter(type_name) (a small static-name set, mirroring the existing isReporterTypeName helper right below it), so the .other arm now reads as a call instead of the 9-way or chain.

[Copilot]

Pull request overview

Adds the nine input reporter nodes to the flow editor, ensuring they render as rounded/data-only nodes, can be created from the palette with appropriate seeded fields, and round-trip through parse/render consistent with codegen.

Changes:

• Extend reporter classification + .other pin rendering to cover the 9 input reporters, exposing a single value output and no exec pins.
• Add other_field_specs entries for IsKey* (key) and IsMouseButton* (button), and add an “Input” palette cluster that seeds those fields.
• Adjust the hidden gui-test window height to keep the “+ Add raw call...” button visible to the ImGui Test Engine.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
src/tests.zig	Adds unit tests covering seeding, field specs, reporter classification, and parse/render round-trips for input reporters.
src/modules/flow_doc.zig	Adds .other pin rendering for input reporters, includes them in isReporterTypeName, and adds an “Input” palette section.
src/gui_tests.zig	Increases hidden test window height to prevent TE interactions from failing due to culled off-screen items.
src/flow_io.zig	Adds other_field_specs entries for input reporter editable fields (key / button).

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.