Improve module location parsing (of stack info) when adding hook

[Steve-Mcl]

fixes #3401

• Bugfix (non-breaking change which fixes an issue)

Proposed changes

basic...

• handles brackets and no brackets in stack line
• handles short stack

details...

• Ensure "TypeError: Cannot read property 'slice' of undefined" if called from within callback does not occur
• if short stack, location is assigned "unknown:0:0" and node-red debug log will contain the hook name and stack
• if an error occurs parsing a stack, location is assigned "unknown:0:0" and node-red debug log will contain the hook name and stack

Local Tests...

I wrapped the new code in a temporary function and tested locally in a Node JS 16 env REPL...

Future:

refactor code to permit testing?

Checklist

• I have read the contribution guidelines
• For non-bugfix PRs, I have discussed this change on the forum/slack team.
• I have run grunt to verify the unit tests pass
• I have added suitable unit tests to cover the new/changed functionality

[coveralls]

Coverage increased (+0.03%) to 67.357% when pulling 2b958f5 on Steve-Mcl:fix-hook-module-location into 95f7177 on node-red:master.