libnpmfund 2.0.0 → 3.0.0

package/{index.js → lib/index.js}

@@ -15,11 +15,13 @@ function normalizeFunding (funding) {
 // Is the value of a `funding` property of a `package.json`
 // a valid type+url for `npm fund` to display?
 function isValidFunding (funding) {
-  if (!funding)
+  if (!funding) {
     return false
+  }
 
-  if (Array.isArray(funding))
+  if (Array.isArray(funding)) {
     return funding.every(f => !Array.isArray(f) && isValidFunding(f))
+  }
 
   try {
     var parsed = new URL(funding.url || funding)
@@ -30,8 +32,9 @@ function isValidFunding (funding) {
   if (
     parsed.protocol !== 'https:' &&
     parsed.protocol !== 'http:'
-  )
+  ) {
     return false
+  }
 
   return Boolean(parsed.host)
 }
@@ -53,8 +56,9 @@ function readTree (tree, opts) {
 
   function tracked (name, version) {
     const key = String(name) + String(version)
-    if (seen.has(key))
+    if (seen.has(key)) {
       return true
+    }
 
     seen.add(key)
   }
@@ -89,30 +93,36 @@ function readTree (tree, opts) {
 
   function getFundingDependencies (tree) {
     const edges = tree && tree.edgesOut && tree.edgesOut.values()
-    if (!edges)
+    if (!edges) {
       return empty()
+    }
 
     const directDepsWithFunding = Array.from(edges).map(edge => {
-      if (!edge || !edge.to)
+      if (!edge || !edge.to) {
         return empty()
+      }
 
       const node = edge.to.target || edge.to
-      if (!node.package)
+      if (!node.package) {
         return empty()
+      }
 
-      if (filterSet && filterSet.size > 0 && !filterSet.has(node))
+      if (filterSet && filterSet.size > 0 && !filterSet.has(node)) {
         return empty()
+      }
 
       const { name, funding, version } = node.package
 
       // avoids duplicated items within the funding tree
-      if (tracked(name, version))
+      if (tracked(name, version)) {
         return empty()
+      }
 
       const fundingItem = {}
 
-      if (version)
+      if (version) {
         fundingItem.version = version
+      }
 
       attachFundingInfo(fundingItem, funding)
 
@@ -126,8 +136,9 @@ function readTree (tree, opts) {
       (res, { node, fundingItem }, i) => {
         if (!fundingItem ||
           fundingItem.length === 0 ||
-          !node)
+          !node) {
           return res
+        }
 
         // recurse
         const transitiveDependencies = node.edgesOut &&
@@ -136,17 +147,18 @@ function readTree (tree, opts) {
 
         // if we're only counting items there's no need
         // to add all the data to the resulting object
-        if (countOnly)
+        if (countOnly) {
           return null
+        }
 
         if (hasDependencies(transitiveDependencies)) {
           fundingItem.dependencies =
             retrieveDependencies(transitiveDependencies)
         }
 
-        if (isValidFunding(fundingItem.funding))
+        if (isValidFunding(fundingItem.funding)) {
           res[node.package.name] = fundingItem
-        else if (hasDependencies(fundingItem.dependencies)) {
+        } else if (hasDependencies(fundingItem.dependencies)) {
           res[_trailingDependencies] =
             Object.assign(
               empty(),
@@ -170,11 +182,13 @@ function readTree (tree, opts) {
       (tree && tree.name)
     result.name = name || (tree && tree.path)
 
-    if (tree && tree.package && tree.package.version)
+    if (tree && tree.package && tree.package.version) {
       result.version = tree.package.version
+    }
 
-    if (tree && tree.package && tree.package.funding)
+    if (tree && tree.package && tree.package.funding) {
       result.funding = normalizeFunding(tree.package.funding)
+    }
 
     result.dependencies = retrieveDependencies(treeDependencies)
   }

package/package.json

@@ -1,8 +1,10 @@
 {
   "name": "libnpmfund",
-  "version": "2.0.0",
+  "version": "3.0.0",
+  "main": "lib/index.js",
   "files": [
-    "index.js"
+    "bin",
+    "lib"
   ],
   "description": "Programmatic API for npm fund",
   "repository": "https://github.com/npm/libnpmfund",
@@ -15,7 +17,7 @@
     "fund",
     "gitfund"
   ],
-  "author": "npm Inc. <support@npmjs.com>",
+  "author": "GitHub Inc.",
   "contributors": [
     {
       "name": "Ruy Adorno",
@@ -26,35 +28,30 @@
   "license": "ISC",
   "scripts": {
     "eslint": "eslint",
-    "lint": "npm run eslint -- index.js test.js",
+    "lint": "eslint '**/*.js'",
     "lintfix": "npm run lint -- --fix",
     "posttest": "npm run lint",
     "test": "tap",
     "snap": "tap",
     "preversion": "npm test",
     "postversion": "npm publish",
-    "prepublishOnly": "git push origin --follow-tags"
+    "prepublishOnly": "git push origin --follow-tags",
+    "postlint": "npm-template-check"
   },
   "tap": {
     "check-coverage": true
   },
-  "standard": {
-    "ignore": [
-      "/tap-snapshots/"
-    ]
-  },
   "devDependencies": {
-    "eslint": "^7.26.0",
-    "eslint-plugin-import": "^2.22.1",
-    "eslint-plugin-node": "^11.1.0",
-    "eslint-plugin-promise": "^5.1.0",
-    "eslint-plugin-standard": "^5.0.0",
+    "@npmcli/template-oss": "^2.4.2",
     "tap": "^15.0.9"
   },
   "dependencies": {
-    "@npmcli/arborist": "^3.0.0"
+    "@npmcli/arborist": "^5.0.0"
   },
   "engines": {
     "node": "^12.13.0 || ^14.15.0 || >=16"
+  },
+  "templateOSS": {
+    "version": "2.4.3"
   }
 }