Thanks for the quick review,@cexbrayat, I added a new commit with the modifications, keeping the old commit just for reference. (If the PR gets accepted in the future, a squash might be necessary.)

Just some comments:

• I eliminated theaddNpmScript function, but without it, I could only use the same value for thedev andtest scripts as their referenced scripts, because I do not know in the template which package manager the developer uses. With the previous solution, if you used npm, it generatednpm run dev for instance, oryarn dev in case of yarn, etc.
• I also needed to modify the order of rendered templates. Now the E2E test related templates are rendered first so that they do not overwrite thetest script if unit tests are also generated.
• I tried to test the modified CLI locally and I think the generated scripts work correctly as you suggested in the issue, but maybe using real references (likenpm run dev,npm run test:unit) would be more elegant. However, I don't have an idea how to do so without using a custom function or implementing some kind of template parameters like using mustache or a similar lib in the render function.

I don't know if the current solution is OK with you, but if not, I'll try to modify it later as you suggest.

@szgabsz91 Can you rebase the PR on the latest main branch, please? I'll then take a look.

I did the rebase yesterday, if I see it correctly, the branch is up to date with the remote main. I also resolved the conflicts around the nightwatch package.json files.

Thanks!

FYI: I just rebased the branch, now it's up-to-date again. :)