fix(access): only complete once

wraithgar · Nathan Fritz · commit fc4648984c9f · 2023-03-02T00:20:22.000-08:00
diff --git a/lib/commands/access.js b/lib/commands/access.js
@@ -53,20 +53,22 @@ class Access extends BaseCommand {
       return commands
     }
 
-    switch (argv[2]) {
-      case 'grant':
-        return ['read-only', 'read-write']
-      case 'revoke':
-        return []
-      case 'list':
-      case 'ls':
-        return ['packages', 'collaborators']
-      case 'get':
-        return ['status']
-      case 'set':
-        return setCommands
-      default:
-        throw new Error(argv[2] + ' not recognized')
+    if (argv.length === 3) {
+      switch (argv[2]) {
+        case 'grant':
+          return ['read-only', 'read-write']
+        case 'revoke':
+          return []
+        case 'list':
+        case 'ls':
+          return ['packages', 'collaborators']
+        case 'get':
+          return ['status']
+        case 'set':
+          return setCommands
+        default:
+          throw new Error(argv[2] + ' not recognized')
+      }
     }
   }
 
diff --git a/test/lib/commands/access.js b/test/lib/commands/access.js
@@ -30,6 +30,7 @@ t.test('completion', async t => {
   ])
   testComp(['npm', 'access', 'grant'], ['read-only', 'read-write'])
   testComp(['npm', 'access', 'revoke'], [])
+  testComp(['npm', 'access', 'grant', ''], [])
 
   await t.rejects(
     access.completion({ conf: { argv: { remain: ['npm', 'access', 'foobar'] } } }),
