Refactor: Deduplicate BinaryExpression and LogicalExpression transformation logic

[LalitNarayanYadav]

…Addresses #8694

Changes:

• Deduplicated transformation logic shared by BinaryExpression and LogicalExpression
• Introduced a helper transformBinaryOrLogical to centralize the operator-to-method-call transformation
• Replaced both AST callback handlers with references to the shared helper

This reduces duplication and improves maintainability without changing behavior.

---

Screenshots of the change:

N/A (non-visual refactor)

---

PR Checklist

• npm run lint passes
• Inline reference is included / updated
• Unit tests are included / updated

[p5-bot]

Continuous Release

CDN link

• https://raw.esm.sh/pr/p5@9298693/lib/p5.min.js

Published Packages

• https://pkg.pr.new/p5@9298693

Commit hash: 48b9509

Previous deployments

324c592

• https://raw.esm.sh/pr/p5@ba24879/lib/p5.min.js
• https://pkg.pr.new/p5@ba24879

---

This is an automated message.

[davepagurek]

Thanks @LalitNarayanYadav, this is a nice well-scoped change! Are you interested in continuing to chip away at this file in some more PRs?

[LalitNarayanYadav]

Thanks @davepagurek ! Yes, absolutely, I'd love to keep going.

I'm already working on the next cleanup: introducing a makeGuardedCallbacks wrapper to eliminate the repeated uniform context guard that's currently copy-pasted across ~12 handlers in ASTCallbacks:

if (ancestors.some(a => nodeIsUniform(a) || nodeIsUniformCallbackFn(a, state.uniformCallbackNames))) {
  return;
}

The idea is to wrap each handler automatically at the point where ASTCallbacks is used, so the guard logic lives in one place rather than being repeated in every handler body. I'll open that as a follow-up PR shortly.