fix(search): don't pass unused args

wraithgar · ruyadorno · commit 2d682e4cab0c · 2021-03-04T16:13:27.000-05:00
The searchopts get parsed and added to the query elsewhere, they're not part of the `include` array they are an extra querystring that is added to the search request. PR-URL: #2803 Credit: @wraithgar Close: #2803 Reviewed-by: @ruyadorno
diff --git a/lib/search.js b/lib/search.js
@@ -8,7 +8,7 @@ const packageFilter = require('./search/package-filter.js')
 const output = require('./utils/output.js')
 const usageUtil = require('./utils/usage.js')
 
-function prepareIncludes (args, searchopts) {
+function prepareIncludes (args) {
   return args
     .map(s => s.toLowerCase())
     .filter(s => s)
@@ -47,7 +47,7 @@ class Search {
     const opts = {
       ...this.npm.flatOptions,
       ...this.npm.flatOptions.search,
-      include: prepareIncludes(args, this.npm.flatOptions.search.opts),
+      include: prepareIncludes(args),
       exclude: prepareExcludes(this.npm.flatOptions.search.exclude),
     }
 
