Skip to content

Commit 89d85a3

Browse files
Copilothotlong
andcommitted
fix: replace replaceAll with split().join() for ES2020 compatibility
Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>
1 parent 8539b03 commit 89d85a3

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

packages/services/service-automation/src/engine.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -229,7 +229,7 @@ export class AutomationEngine implements IAutomationService {
229229
// TODO: Replace with safe expression evaluator (e.g., jexl) for production.
230230
let resolved = expression;
231231
for (const [key, value] of variables) {
232-
resolved = resolved.replaceAll(`{${key}}`, String(value));
232+
resolved = resolved.split(`{${key}}`).join(String(value));
233233
}
234234
try {
235235
return new Function(`return (${resolved})`)() as boolean;

packages/services/service-automation/src/plugins/logic-nodes-plugin.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export class LogicNodesPlugin implements Plugin {
3030
// TODO: Replace with safe expression evaluator (e.g., jexl) for production.
3131
let expr = cond.expression;
3232
for (const [k, v] of variables) {
33-
expr = expr.replaceAll(`{${k}}`, String(v));
33+
expr = expr.split(`{${k}}`).join(String(v));
3434
}
3535
try {
3636
if (new Function(`return (${expr})`)()) {

0 commit comments

Comments
 (0)