fix duplicate emits

juliangruber · Apr 9, 2020 · b5955ff · b5955ff
1 parent 51ccf4a
commit b5955ff
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/bin/ops.js b/bin/ops.js
@@ -10,7 +10,8 @@ const main = async () => {
       name: 'name',
       message: 'Name of the module to search',
       default: 'express'
-    }, {
+    },
+    {
       type: 'input',
       name: 'limit',
       message: 'How many modules to display at most',

diff --git a/index.js b/index.js
@@ -3,6 +3,7 @@ const cheerio = require('cheerio')
 
 module.exports = name => {
   const dependants = []
+  const visited = new Set()
   let offset = 0
 
   const next = async () => {
@@ -14,8 +15,9 @@ module.exports = name => {
       const dependant = $(el)
         .attr('href')
         .slice('/package/'.length)
-      if (dependant !== name && !dependants.includes(dependant)) {
+      if (dependant !== name && !visited.has(dependant)) {
         dependants.push(dependant)
+        visited.add(dependant)
       }
     })
     offset += 36