fix: yarn argument value identified as a package (#166)

Fixes #156

In case of Number, handle it as Argument and skip this word.
After the fix:
![Screen Shot 2024-02-18 at 21 42
13](https://github.com/os-scar/overlay/assets/6542413/02c26ac7-62d4-4ba0-886b-3a3b1ec0df6b)

---------

Co-authored-by: Baruch Odem (Rothkoff) <baruchiro@gmail.com>

Author: yoavsbg

src/content/registry/npm.js

@@ -20,12 +20,24 @@ const parsePackageString = (str) => {
   };
 };
 
-const parseNpmCommand = createParseCommand(
-  'npm',
-  (line) => line.match(npmInstall),
-  (word) => word.length + 1,
-  parsePackageString
-);
+const npmOptionWithArgToIgnore = ['--network-timeout', '--network-concurrency'];
+
+//npm
+const handleArgument = (argument, restCommandWords) => {
+  let index = 0;
+  index += argument.length + 1; // +1 for the space removed by split
+
+  if (!npmOptionWithArgToIgnore.includes(argument)) {
+    return index;
+  }
+
+  if (argument.includes('=')) return index;
+
+  index += restCommandWords.shift().length + 1;
+  return index;
+};
+
+const parseNpmCommand = createParseCommand('npm', (line) => line.match(npmInstall), handleArgument, parsePackageString);
 
 // npx
 const parseOnlyFirstPackage = (str, argsAndPackagesWords) => {

src/content/registry/npm.test.js

@@ -20,6 +20,7 @@ describe('npm', () => {
       'npm install -g',
       '`npm install node-sass`', // this is not a valid command because of the `
       'npm create-react-app my-app',
+      'yarn add <yourPackage> --network-timeout 100000',
     ])('should return empty array if no packages found', (command) => {
       expect(parseCommand(command)).toStrictEqual([]);
     });

tests/real-examples/real-examples-results.yaml

@@ -27,6 +27,10 @@ links:
     comment: ignore 'npm install -d' & find '/usr/local/bin/npm install jade'
     post: answer
     registry: npm
+  https://stackoverflow.com/questions/51508364:
+    comment: ignore 'yarn add <yourPackage> --network-timeout 100000'
+    post: answer
+    registry: npm
   https://stackoverflow.com/questions/12008719:
     comment: npm install --save
     post: answer