Skip to content

Refactor: Deduplicate BinaryExpression and LogicalExpression transformation logic#8741

Open
LalitNarayanYadav wants to merge 1 commit intoprocessing:dev-2.0from
LalitNarayanYadav:refactor/strands-binary-logical-dedup
Open

Refactor: Deduplicate BinaryExpression and LogicalExpression transformation logic#8741
LalitNarayanYadav wants to merge 1 commit intoprocessing:dev-2.0from
LalitNarayanYadav:refactor/strands-binary-logical-dedup

Conversation

@LalitNarayanYadav
Copy link
Copy Markdown
Contributor

…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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant