By no means a complete solution, but I run this command to create a human readable version of the flow file as the file flows.json.formatted and commit that to github too. In settings.js, flowFilePretty must be set to true. Then the formatted file may be git diffed to see the changes.
# generates formatted versions of node red flow files matching flow*.json
# in files flow*.json.formatted
# NOTE must have flowFilePretty: true in settings.js
for f in flow*.json; do
sed -r 's/\\n/\n/g' "$f" > "$f.formatted"
# remove any cred files converted
Yes that is great for expanding the json. I do wonder if that is really a great way to do a code review of a visual programming paradigm. I would have thought a walkthrough of the actual flow would be (or should be) a better way to try to understand what the flow is trying to do.
I guess in a professional production environment it is needed to have a review process. Currently I do not really see how should I review flow in the graphic surface. @dceejay it is a possibility to do this in pair to get directly an explanation from the author, that is fine. But can you then check if there is a change (diff) which could be relevant or create a risk / bug? From my point of view it isn't possible to see differences in the graphic surface and I am on the same side like @Colin to use this tooling to find differences.
@Colin thanks for this information I will try to enable the pretty formatting and check if this will help out more.
I agree - for diff then yes traditional diff/meld tools will highlight the differences more easily. As long as that is what you mean by review then fine. Normally I mean it to be your first example - of a walkthrough with the author - where I think visual flow would be better... - or indeed do both
For reviewing the flow itself it might be worth looking at the techniques used for reviewing integrated circuit design, which is basically similar in concept. Functional blocks with inputs and outputs wired together.
Edit: or any other sort of digital circuit design for that matter.