martino/uopi
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(logic): subscribe signals referenced by array action node params

Browse Source 
subscribeRefs() walked node kinds to build the signal subscription set but
omitted the new action.array.push/set/remove nodes. Signal or local-var
references in their expr/index params were never subscribed, so the live
cache held no value and they resolved to NaN at run time (action.accumulate
was handled, its modern equivalent action.array.push was not). Add the
missing cases, including the comma-split index list for array.set.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Martino Ferrari
2026-06-24 13:50:26 +02:00
co-authored by Claude Opus 4.6
parent 9d48292976
commit 9d9e538a90
1 changed files with 7 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
web/src/lib/logic.ts
+7
View File
@@ -399,7 +399,14 @@ export class LogicEngine {
case 'flow.loop': if ((node.params.mode ?? 'count') === 'while') collectRefs(node.params.cond ?? '').forEach(want); break;
case 'action.write':
case 'action.accumulate':
case 'action.array.push':
case 'action.log': collectRefs(node.params.expr ?? '').forEach(want); break;
case 'action.array.set':
collectRefs(node.params.expr ?? '').forEach(want);
String(node.params.index ?? '').split(',').forEach(s => collectRefs(s.trim()).forEach(want));
break;
case 'action.array.remove':
collectRefs(node.params.index ?? '').forEach(want); break;
case 'action.config.apply':
case 'action.config.read':
case 'action.config.write': {