Refactor

Author: Kikobeats

packages/metascraper-amazon/index.js

@@ -5,16 +5,11 @@ const { titleize, isUrl } = require('@metascraper/helpers')
 const REGEX_AMAZON_URL = /https?:\/\/(.*amazon\..*\/.*|.*amzn\..*\/.*|.*a\.co\/.*)/i
 const isAmazonUrl = url => REGEX_AMAZON_URL.test(url)
 
-const wrap = rule => ({ htmlDom, url }) => {
-  if (!isAmazonUrl(url)) return
-  const value = rule(htmlDom)
-  return value
-}
+const wrap = rule => ({ htmlDom, url }) => isAmazonUrl(url) && rule(htmlDom)
 
 const wrapUrl = rule => ({ htmlDom, url }) => {
   const value = wrap(rule)({htmlDom, url})
-  if (!isUrl(value)) return
-  return value
+  return isUrl(value) && value
 }
 
 module.exports = () => ({

packages/metascraper-author/index.js

@@ -29,8 +29,7 @@ const wrap = rule => ({ htmlDom }) => {
 
 const strict = rule => $ => {
   const value = rule($)
-  if (!REGEX_STRICT.test(value)) return
-  return value
+  return REGEX_STRICT.test(value) && value
 }
 
 const getFirst = ($, collection) =>

packages/metascraper-description/index.js

@@ -16,9 +16,7 @@ const removeLocation = value => value.replace(REGEX_LOCATION, '')
 
 const wrap = rule => ({ htmlDom }) => {
   const value = rule(htmlDom)
-
-  if (!isString(value)) return
-  return titleize(removeLocation(value), { capitalize: false })
+  return isString(value) && titleize(removeLocation(value), { capitalize: false })
 }
 
 /**

packages/metascraper-publisher/index.js

@@ -15,9 +15,7 @@ const REGEX_TITLE = /^.*?\|\s+(.*)$/
 
 const wrap = rule => ({ htmlDom }) => {
   const value = rule(htmlDom)
-
-  if (!isString(value)) return
-  return condenseWhitespace(value)
+  return isString(value) && condenseWhitespace(value)
 }
 
 /**