38/link command should not execute npm install if it was done before

[HaidarJbeily7]

Changes

• Added a check if the npm install was already executed before. I implemented it by checking where the package-lock.json file exists.

---

PR-Codex overview

This PR focuses on enhancing the link command by preventing unnecessary reinstallation of npm packages if the package-lock.json file already exists. It also includes a new test case to verify this behavior.

Detailed summary

• Added a check for the existence of package-lock.json before running npm install.
• Introduced a new test to confirm that package-lock.json is not modified on subsequent runs when it exists.
• Adjusted code structure to improve readability and maintainability.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

[maxonfjvipon]

@HaidarJbeily7 thanks for your contribution, but it would be nice to add a test that confirms that your changes work and don't break anything

[maxonfjvipon]

@HaidarJbeily7 looks good to me, just one simple comment from my side

[maxonfjvipon]

@HaidarJbeily7 LGTM!

[maxonfjvipon]

@rultor merge

[rultor]

@rultor merge

@maxonfjvipon OK, I'll try to merge now. You can check the progress of the merge here.

[rultor]

@rultor merge

@maxonfjvipon Done! FYI, the full log is here (took me 22min).

[maxonfjvipon]

@HaidarJbeily7 thanks!