Skip to content

NEW @W-21741623@ Flow engine not detecting leftValueReference usage in decision conditions#452

Merged
nikhil-mittal-165 merged 2 commits intodevfrom
trust/flow-engine-unusedresource-fix
Apr 20, 2026
Merged

NEW @W-21741623@ Flow engine not detecting leftValueReference usage in decision conditions#452
nikhil-mittal-165 merged 2 commits intodevfrom
trust/flow-engine-unusedresource-fix

Conversation

@nikhil-mittal-165
Copy link
Copy Markdown
Contributor

@nikhil-mittal-165 nikhil-mittal-165 commented Apr 17, 2026

This issue addresses forcedotcom/code-analyzer#2026

where due to our flow engine not detecting leftValueReference usage in decision conditions
there were false positives for unusedResource violations have added evidence of testing,

@git2gus
Copy link
Copy Markdown

git2gus Bot commented Apr 17, 2026

Git2Gus App is installed but the .git2gus/config.json doesn't have right values. You should add the required configuration.

@nikhil-mittal-165
Copy link
Copy Markdown
Contributor Author

With Fix:

image

Without fix :

image

The unused resource is getting flagged because of a missing comma,

Copy link
Copy Markdown
Contributor

@aruntyagiTutu aruntyagiTutu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please post on flow collab channel as well.

@aruntyagiTutu
Copy link
Copy Markdown
Contributor

aruntyagiTutu commented Apr 17, 2026

please change title to explain the issue, do not mention Trust etc, just tech details.

feel free to add details in PR description.

@nikhil-mittal-165 nikhil-mittal-165 changed the title NEW @W-21741623@ Trust Issue for unusedResource fix in flow engine NEW @W-21741623@ Flow engine not detecting leftValueReference usage in decision conditions Apr 17, 2026
@nikhil-mittal-165 nikhil-mittal-165 merged commit 69d6d57 into dev Apr 20, 2026
7 checks passed
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.

3 participants