Fix CI workflow step names and add event listener cleanup

[Copilot]

Addresses CodeRabbit review comments on PR #8: mismatched Node.js version in step names and missing event listener cleanup causing potential memory leaks.

Changes

• .github/workflows/ci.yml: Update all step names from "Setup Node.js 22.x" to "Setup Node.js 24.x" to match actual node-version: 24.x

• blend-trees-2d-cartesian.controls.mjs: Add componentWillUnmount to clean up canvas event listeners

componentWillUnmount() {
    const { canvas } = this;
    canvas.removeEventListener('mousemove', this.boundMouseEvent);
    canvas.removeEventListener('mousedown', this.boundMouseEvent);
    canvas.removeEventListener('touchmove', this.boundMouseEvent);
    canvas.removeEventListener('touchstart', this.boundMouseEvent);
}

• blend-trees-2d-directional.controls.mjs: Add cleanup return function to useEffect for proper listener removal on unmount

Original prompt

gh pr checkout 8

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

---

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[Copilot]

Pull request overview

This PR addresses code quality issues identified in a previous code review by fixing documentation inconsistencies and preventing potential memory leaks.

• Updates CI workflow step names to accurately reflect Node.js 24.x usage
• Implements proper event listener cleanup in animation control components to prevent memory leaks

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
.github/workflows/ci.yml	Updates all 6 step names from "Setup Node.js 22.x" to "Setup Node.js 24.x" to match actual configuration
blend-trees-2d-directional.controls.mjs	Adds cleanup return function in useEffect to remove mousemove and mousedown event listeners on unmount
blend-trees-2d-cartesian.controls.mjs	Refactors to use bound event handler and adds componentWillUnmount to clean up all 4 event listeners (mouse and touch events)

---

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

[JaclynCodes]

@copilot componentWillUnmount() {
const { canvas } = this;
canvas.removeEventListener('mousemove', this.boundMouseEvent);
canvas.removeEventListener('mousedown', this.boundMouseEvent);
canvas.removeEventListener('touchmove', this.boundMouseEvent);
canvas.removeEventListener('touchstart', this.boundMouseEvent);
}