feat: add :type(registry) to query selector syntax (#9017)

Author: wraithgar

docs/lib/content/using-npm/dependency-selectors.md

@@ -62,7 +62,7 @@ The [`npm query`](/commands/npm-query) command exposes a new dependency selector
 - `:missing` when a dependency is not found on disk
 - `:semver(<spec>, [selector], [function])` match a valid [`node-semver`](https://github.com/npm/node-semver) version or range to a selector
 - `:path(<path>)` [glob](https://www.npmjs.com/package/glob) matching based on dependencies path relative to the project
-- `:type(<type>)` [based on currently recognized types](https://github.com/npm/npm-package-arg#result-object)
+- `:type(<type>)` [based on currently recognized types](https://github.com/npm/npm-package-arg#result-object).  You can also use the aggregate type of `registry` for any registry dependency (e.g. tag, version, range, alias)
 - `:outdated(<type>)` when a dependency is outdated
 - `:vuln(<selector>)` when a dependency has a known vulnerability
 

workspaces/arborist/lib/query-selector-all.js

@@ -427,11 +427,19 @@ class Results {
     if (!this.currentAstNode.typeValue) {
       return this.initialItems
     }
+    // TODO this differs subtly with `:type()` because it now iterates on edgesIn, which means extraneous deps won't show up
+    // note how "@npmcli/[email protected]" is in the `:type()` results in the test but not in any of the other results.
     return this.initialItems
       .flatMap(node => {
         const found = []
+        const { typeValue } = this.currentAstNode
         for (const edge of node.edgesIn) {
-          if (npa(`${edge.name}@${edge.spec}`).type === this.currentAstNode.typeValue) {
+          const parsedArg = npa(`${edge.name}@${edge.spec}`)
+          if (typeValue === 'registry') {
+            if (parsedArg.registry) {
+              found.push(edge.to)
+            }
+          } else if (parsedArg.type === typeValue) {
             found.push(edge.to)
           }
         }

workspaces/arborist/test/query-selector-all.js

@@ -569,6 +569,7 @@ t.test('query-selector-all', async t => {
       '[email protected]',
       '[email protected]',
       '[email protected]',
+      // TODO this should show up in the other :type tests, see lib/query-selector-all.js
       '@npmcli/[email protected]',
       '[email protected]',
       '[email protected]',
@@ -596,6 +597,22 @@ t.test('query-selector-all', async t => {
       '[email protected]',
       '[email protected]',
     ]],
+    [':type(registry)', [
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      'ipsum@npm:[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+      '[email protected]',
+    ]],
     [':type(git)', []],
 
     // path pseudo