@maizzle/framework 4.4.0-beta.8 → 4.4.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@maizzle/framework",
-  "version": "4.4.0-beta.8",
+  "version": "4.4.0",
   "description": "Maizzle is a framework that helps you quickly build HTML emails with Tailwind CSS.",
   "license": "MIT",
   "main": "src/index.js",
@@ -41,8 +41,8 @@
   ],
   "dependencies": {
     "@maizzle/cli": "^1.5.1",
-    "autoprefixer": "^10.4.13",
-    "browser-sync": "^2.27.11",
+    "autoprefixer": "^10.4.14",
+    "browser-sync": "^2.28.3",
     "color-shorthand-hex-to-six-digit": "^3.0.2",
     "email-comb": "^5.2.0",
     "front-matter": "^4.0.0",
@@ -50,7 +50,7 @@
     "glob-promise": "^4.1.0",
     "html-crush": "^4.0.0",
     "is-url-superb": "^5.0.0",
-    "juice": "^8.0.0",
+    "juice": "^9.0.0",
     "lodash": "^4.17.20",
     "ora": "^5.1.0",
     "postcss": "^8.4.21",
@@ -58,27 +58,27 @@
     "postcss-merge-longhand": "^5.1.7",
     "posthtml": "^0.16.6",
     "posthtml-attrs-parser": "^0.1.1",
-    "posthtml-base-url": "^1.0.1",
-    "posthtml-component": "^1.0.0-beta.16",
+    "posthtml-base-url": "^2.0.0",
+    "posthtml-component": "^1.1.0",
     "posthtml-content": "^0.1.0",
     "posthtml-extend": "^0.6.0",
-    "posthtml-extra-attributes": "^1.0.0",
-    "posthtml-fetch": "^2.2.0",
+    "posthtml-extra-attributes": "^2.0.0",
+    "posthtml-fetch": "^3.0.0",
     "posthtml-markdownit": "^1.3.1",
     "posthtml-match-helper": "^1.0.3",
-    "posthtml-mso": "^1.0.4",
-    "posthtml-postcss-merge-longhand": "^1.0.2",
-    "posthtml-safe-class-names": "^2.0.0",
-    "posthtml-url-parameters": "^1.0.4",
+    "posthtml-mso": "^2.0.0",
+    "posthtml-postcss-merge-longhand": "^2.0.1",
+    "posthtml-safe-class-names": "^3.0.0",
+    "posthtml-url-parameters": "^2.0.0",
     "pretty": "^2.0.0",
     "query-string": "^7.1.3",
     "string-remove-widows": "^2.1.0",
     "string-strip-html": "^8.2.0",
-    "tailwindcss": "^3.2.4"
+    "tailwindcss": "^3.2.7"
   },
   "devDependencies": {
-    "ava": "^5.1.1",
-    "c8": "^7.11.0",
+    "ava": "^5.2.0",
+    "c8": "^7.13.0",
     "np": "*",
     "xo": "0.39.1"
   },

package/src/commands/serve.js

@@ -36,132 +36,154 @@ const serve = async (env = 'local', config = {}) => {
 
   const spinner = ora()
 
-  try {
-    await buildToFile(env, config)
-
-    let templates = get(config, 'build.templates')
-    templates = Array.isArray(templates) ? templates : [templates]
-
-    const templatePaths = [...new Set(templates.map(config => `${get(config, 'source', 'src')}/**`))]
-    const tailwindConfig = get(config, 'build.tailwind.config', 'tailwind.config.js')
-    const globalPaths = [
-      'src/**',
-      ...new Set(get(config, 'build.browsersync.watch', []))
-    ]
-
-    if (typeof tailwindConfig === 'string') {
-      globalPaths.push(tailwindConfig)
-    }
-
-    // Watch for Template file changes
-    browsersync()
-      .watch(templatePaths)
-      .on('change', async file => {
-        config = await getConfig(env, config)
-
-        if (config.events && typeof config.events.beforeCreate === 'function') {
-          await config.events.beforeCreate(config)
-        }
-
-        // Don't render if file type is not configured
-        // eslint-disable-next-line
-        const filetypes = templates.reduce((acc, template) => {
-          return [...acc, ...get(template, 'filetypes', ['html'])]
-        }, [])
-
-        if (!filetypes.includes(path.extname(file).slice(1))) {
-          return
-        }
-
-        if (get(config, 'build.console.clear')) {
-          clearConsole()
-        }
-
-        const start = new Date()
-
-        spinner.start('Building email...')
-
-        file = file.replace(/\\/g, '/')
+  // Build all emails first
+  await buildToFile(env, config)
+
+  // Set some paths to watch
+  let templates = get(config, 'build.templates')
+  templates = Array.isArray(templates) ? templates : [templates]
+
+  const templatePaths = [...new Set(templates.map(config => `${get(config, 'source', 'src')}/**`))]
+  const tailwindConfig = get(config, 'build.tailwind.config', 'tailwind.config.js')
+  const globalPaths = [
+    'src/**',
+    ...new Set(get(config, 'build.browsersync.watch', []))
+  ]
+
+  if (typeof tailwindConfig === 'string') {
+    globalPaths.push(tailwindConfig)
+  }
 
-        const renderOptions = {
-          maizzle: config,
+  // Watch for Template file changes
+  browsersync()
+    .watch(templatePaths)
+    .on('change', async file => {
+      config = await getConfig(env, config)
+
+      if (config.events && typeof config.events.beforeCreate === 'function') {
+        await config.events.beforeCreate(config)
+      }
+
+      // Don't render if file type is not configured
+      // eslint-disable-next-line
+      const filetypes = templates.reduce((acc, template) => {
+        return [...acc, ...get(template, 'filetypes', ['html'])]
+      }, [])
+
+      if (!filetypes.includes(path.extname(file).slice(1))) {
+        return
+      }
+
+      // Clear console if enabled
+      if (get(config, 'build.console.clear')) {
+        clearConsole()
+      }
+
+      // Start the spinner
+      const start = new Date()
+      spinner.start('Building email...')
+
+      // Render the template
+      renderToString(
+        await fs.readFile(file.replace(/\\/g, '/'), 'utf8'),
+        {
+          maizzle: merge(
+            config,
+            {
+              build: {
+                current: {
+                  path: path.parse(file)
+                }
+              }
+            }
+          ),
           ...config.events
         }
-
-        renderToString(
-          await fs.readFile(file, 'utf8'),
-          renderOptions
-        )
-          .then(async ({html, config}) => {
-            let source = ''
-            let dest = ''
-            let ext = ''
-
-            if (Array.isArray(config.build.templates)) {
-              const match = config.build.templates.find(template => template.source === path.parse(file).dir)
-              source = get(match, 'source')
-              dest = get(match, 'destination.path', 'build_local')
-              ext = get(match, 'destination.ext', 'html')
-            } else if (isObject(config.build.templates)) {
-              source = get(config, 'build.templates.source')
-              dest = get(config, 'build.templates.destination.path', 'build_local')
-              ext = get(config, 'build.templates.destination.ext', 'html')
-            }
-
-            const fileDir = path.parse(file).dir.replace(source, '')
-            const finalDestination = path.join(dest, fileDir, `${path.parse(file).name}.${ext}`)
-
-            await fs.outputFile(config.permalink || finalDestination, html)
-          })
-          .then(() => {
-            browsersync().reload()
-            spinner.succeed(`Compiled in ${(Date.now() - start) / 1000}s [${file}]`)
-          })
-          .catch(() => spinner.warn(`Received empty HTML, please save your file again [${file}]`))
+      )
+        .then(async ({html, config}) => {
+          // Write the file to disk
+          let source = ''
+          let dest = ''
+          let ext = ''
+
+          if (Array.isArray(config.build.templates)) {
+            const match = config.build.templates.find(template => template.source === path.parse(file).dir)
+            source = path.normalize(get(match, 'source'))
+            dest = path.normalize(get(match, 'destination.path', 'build_local'))
+            ext = get(match, 'destination.ext', 'html')
+          } else if (isObject(config.build.templates)) {
+            source = path.normalize(get(config, 'build.templates.source'))
+            dest = path.normalize(get(config, 'build.templates.destination.path', 'build_local'))
+            ext = get(config, 'build.templates.destination.ext', 'html')
+          }
+
+          const fileDir = path.parse(file).dir.replace(source, '')
+          const finalDestination = path.join(dest, fileDir, `${path.parse(file).name}.${ext}`)
+
+          await fs.outputFile(config.permalink || finalDestination, html)
+        })
+        .then(() => {
+          browsersync().reload()
+          spinner.succeed(`Compiled in ${(Date.now() - start) / 1000}s [${file}]`)
+        })
+        .catch(error => {
+          throw error
+        })
+    })
+
+  // Watch for changes in all other files
+  browsersync()
+    .watch(globalPaths, {ignored: templatePaths})
+    .on('change', () => buildToFile(env, config)
+      .then(() => browsersync().reload())
+      .catch(error => {
+        throw error
       })
-
-    // Watch for changes in all other files
-    browsersync()
-      .watch(globalPaths, {ignored: templatePaths})
-      .on('change', () => buildToFile(env, config).then(() => browsersync().reload()))
-      .on('unlink', () => buildToFile(env, config).then(() => browsersync().reload()))
-
-    // Watch for changes in config files
-    browsersync()
-      .watch('config*.js')
-      .on('change', async file => {
-        const parsedEnv = path.parse(file).name.split('.')[1] || 'local'
-
-        Config
-          .getMerged(parsedEnv)
-          .then(config => buildToFile(parsedEnv, config).then(() => browsersync().reload()))
+    )
+    .on('unlink', () => buildToFile(env, config)
+      .then(() => browsersync().reload())
+      .catch(error => {
+        throw error
       })
-
-    // Browsersync options
-    const baseDir = templates.map(t => t.destination.path)
-
-    // Initialize Browsersync
-    browsersync()
-      .init(
-        merge(
-          {
-            notify: false,
-            open: false,
-            port: 3000,
-            server: {
-              baseDir,
-              directory: true
-            },
-            tunnel: false,
-            ui: {port: 3001},
-            logFileChanges: false
+    )
+
+  // Watch for changes in config files
+  browsersync()
+    .watch('config*.js')
+    .on('change', async file => {
+      const parsedEnv = path.parse(file).name.split('.')[1] || 'local'
+
+      Config
+        .getMerged(parsedEnv)
+        .then(config => buildToFile(parsedEnv, config)
+          .then(() => browsersync().reload())
+          .catch(error => {
+            throw error
+          })
+        )
+    })
+
+  // Browsersync options
+  const baseDir = templates.map(t => t.destination.path)
+
+  // Initialize Browsersync
+  browsersync()
+    .init(
+      merge(
+        {
+          notify: false,
+          open: false,
+          port: 3000,
+          server: {
+            baseDir,
+            directory: true
           },
-          get(config, 'build.browsersync', {})
-        ), () => {})
-  } catch (error) {
-    spinner.fail(error)
-    throw error
-  }
+          tunnel: false,
+          ui: {port: 3001},
+          logFileChanges: false
+        },
+        get(config, 'build.browsersync', {})
+      ), () => {})
 }
 
 module.exports = serve

package/src/generators/output/to-disk.js

@@ -2,7 +2,6 @@ const path = require('path')
 const fs = require('fs-extra')
 const glob = require('glob-promise')
 const {get, isEmpty, merge} = require('lodash')
-const {asyncForEach} = require('../../utils/helpers')
 
 const Config = require('../config')
 const Tailwind = require('../tailwindcss')
@@ -31,7 +30,7 @@ module.exports = async (env, spinner, config) => {
     : await Tailwind.compile({config})
 
   // Parse each template config object
-  await asyncForEach(templatesConfig, async templateConfig => {
+  for await (const templateConfig of templatesConfig) {
     if (!templateConfig) {
       const configFileName = env === 'local' ? 'config.js' : `config.${env}.js`
       throw new Error(`No template sources defined in \`build.templates\`, check your ${configFileName} file`)
@@ -79,18 +78,20 @@ module.exports = async (env, spinner, config) => {
       : templateConfig.filetypes || get(templateConfig, 'filetypes', 'html')
 
     // List of files that won't be copied to the output directory
-    const omitted = Array.isArray(templateConfig.omit) ?
-      templateConfig.omit :
-      [get(templateConfig, 'omit', '')]
+    const omitted = Array.isArray(templateConfig.omit)
+      ? templateConfig.omit
+      : [get(templateConfig, 'omit', '')]
 
     // Parse each template source
-    await asyncForEach(templateSource, async source => {
+    for await (const source of templateSource) {
       /**
        * Copy single-file sources correctly
        * If `src` is a file, `dest` cannot be a directory
        * https://github.com/jprichardson/node-fs-extra/issues/323
        */
-      const out = fs.lstatSync(source).isFile() ? `${outputDir}/${path.basename(source)}` : outputDir
+      const out = fs.lstatSync(source).isFile()
+        ? `${outputDir}/${path.basename(source)}`
+        : outputDir
 
       await fs
         .copy(source, out, {filter: file => {
@@ -119,7 +120,7 @@ module.exports = async (env, spinner, config) => {
             await config.events.beforeCreate(config)
           }
 
-          await asyncForEach(templates, async file => {
+          for await (const file of templates) {
             config.build.current = {
               path: path.parse(file)
             }
@@ -202,18 +203,18 @@ module.exports = async (env, spinner, config) => {
                   throw error
               }
             }
-          })
+          }
 
           const assets = {source: '', destination: 'assets', ...get(templateConfig, 'assets')}
 
           if (Array.isArray(assets.source)) {
-            await asyncForEach(assets.source, async source => {
+            for await (const source of assets.source) {
               if (fs.existsSync(source)) {
                 await fs
                   .copy(source, path.join(templateConfig.destination.path, assets.destination))
                   .catch(error => spinner.warn(error.message))
               }
-            })
+            }
           } else {
             if (fs.existsSync(assets.source)) {
               await fs
@@ -228,8 +229,8 @@ module.exports = async (env, spinner, config) => {
             })
         })
         .catch(error => spinner.warn(error.message))
-    })
-  })
+    }
+  }
 
   if (config.events && typeof config.events.afterBuild === 'function') {
     await config.events.afterBuild(files)

package/src/generators/output/to-string.js

@@ -37,7 +37,7 @@ module.exports = async (html, options) => {
       config: merge(config, {
         build: {
           tailwind: {
-            config: get(options, 'tailwind.config', {})
+            config: get(options, 'tailwind.config')
           }
         }
       })

package/src/generators/posthtml/defaultConfig.js

@@ -1,3 +1,4 @@
 module.exports = {
-  recognizeNoValueAttribute: true
+  recognizeNoValueAttribute: true,
+  recognizeSelfClosing: true
 }

package/src/generators/posthtml/index.js

@@ -9,7 +9,13 @@ const defaultConfig = require('./defaultConfig')
 module.exports = async (html, config) => {
   const layoutsOptions = get(config, 'build.layouts', {})
   const componentsOptions = get(config, 'build.components', {})
-  const expressionsOptions = merge({strictMode: false}, get(config, 'build.posthtml.expressions', {}))
+  const expressionsOptions = merge(
+    {
+      loopTags: ['each', 'for'],
+      strictMode: false
+    },
+    get(config, 'build.posthtml.expressions', {})
+  )
 
   const posthtmlOptions = merge(defaultConfig, get(config, 'build.posthtml.options', {}))
   const posthtmlPlugins = get(config, 'build.posthtml.plugins', [])

package/src/generators/tailwindcss.js

@@ -8,6 +8,22 @@ const {requireUncached} = require('../utils/helpers')
 const mergeLonghand = require('postcss-merge-longhand')
 const {get, isObject, isEmpty, merge} = require('lodash')
 
+const addImportantPlugin = () => {
+  return {
+    postcssPlugin: 'add-important',
+    Rule(rule) {
+      const shouldAddImportant = get(rule, 'raws.tailwind.layer') === 'variants'
+      || get(rule, 'parent.type') === 'atrule'
+
+      if (shouldAddImportant) {
+        rule.walkDecls(decl => {
+          decl.important = true
+        })
+      }
+    }
+  }
+}
+
 module.exports = {
   compile: async ({css = '', html = '', config = {}}) => {
     // Compute the Tailwind config to use
@@ -34,25 +50,30 @@ module.exports = {
     const layoutsRoot = get(config, 'build.layouts.root')
     const componentsRoot = get(config, 'build.components.root')
 
+    const layoutsPath = typeof layoutsRoot === 'string' && layoutsRoot ?
+      `${layoutsRoot}/**/*.html`.replace(/\/\//g, '/') :
+      './src/layouts/**/*.html'
+
+    const componentsPath = typeof componentsRoot === 'string' && componentsRoot ?
+      `${componentsRoot}/**/*.html`.replace(/\/\//g, '/') :
+      './src/components/**/*.html'
+
     const tailwindConfig = merge({
-      important: true,
       content: {
         files: [
-          typeof layoutsRoot === 'string' && layoutsRoot ?
-            `${layoutsRoot}/**/*.html`.replace(/\/\//g, '/') :
-            './src/layouts/**/*.html',
-          typeof componentsRoot === 'string' && componentsRoot ?
-            `${componentsRoot}/**/*.html`.replace(/\/\//g, '/') :
-            './src/components/**/*.html',
+          layoutsPath,
+          componentsPath,
           {raw: html, extension: 'html'}
         ]
       }
     }, userConfig(config))
 
-    // Add back the `{raw: html}` option if user provided own config
+    // If `content` is an array, add it to `content.files`
     if (Array.isArray(tailwindConfig.content)) {
       tailwindConfig.content = {
         files: [
+          layoutsPath,
+          componentsPath,
           ...tailwindConfig.content,
           {raw: html, extension: 'html'}
         ]
@@ -99,6 +120,7 @@ module.exports = {
     const toProcess = [
       postcssNested(),
       tailwindcss(tailwindConfig),
+      get(tailwindConfig, 'important') === false ? () => {} : addImportantPlugin(),
       get(config, 'shorthandCSS', get(config, 'shorthandInlineCSS')) === true ?
         mergeLonghand() :
         () => {},

package/src/transformers/index.js

@@ -53,7 +53,7 @@ exports.addURLParams = (html, config) => addURLParams(html, config, true)
 exports.preventWidows = (html, config) => preventWidows(html, config)
 exports.replaceStrings = (html, config) => replaceStrings(html, config, true)
 exports.safeClassNames = (html, config) => safeClassNames(html, config, true)
-exports.removeUnusedCSS = (html, config) => removeUnusedCSS(html, config)
+exports.removeUnusedCSS = (html, config) => removeUnusedCSS(html, config, true)
 exports.removeAttributes = (html, config) => removeAttributes(html, config, true)
 exports.attributeToStyle = (html, config) => attributeToStyle(html, config, true)
 exports.removeInlineSizes = (html, config) => removeInlineSizes(html, config, true)

package/src/transformers/inlineCss.js

@@ -14,7 +14,7 @@ module.exports = async (html, config = {}, direct = false) => {
 
   if (get(config, 'inlineCSS') === true || !isEmpty(options)) {
     options.applyAttributesTableElements = true
-    juice.styleToAttribute = get(options, 'styleToAttribute', {'vertical-align': 'valign'})
+    juice.styleToAttribute = get(options, 'styleToAttribute', {})
 
     juice.widthElements = get(options, 'applyWidthAttributes', []).map(i => i.toUpperCase())
     juice.heightElements = get(options, 'applyHeightAttributes', []).map(i => i.toUpperCase())

package/src/transformers/removeUnusedCss.js

@@ -1,10 +1,12 @@
 const {comb} = require('email-comb')
-const {get, merge} = require('lodash')
+const {get, merge, isEmpty, isObject} = require('lodash')
 const removeInlinedClasses = require('./removeInlinedSelectors')
 
-module.exports = async (html, config = {}) => {
-  // If it's explicitly disabled, return the HTML
-  if (get(config, 'removeUnusedCSS') === false) {
+module.exports = async (html, config = {}, direct = false) => {
+  config = direct ? config : get(config, 'removeUnusedCSS')
+
+  // Don't purge CSS if `removeUnusedCSS` is not set
+  if (!config || (isObject(config) && isEmpty(config))) {
     return html
   }
 
@@ -34,9 +36,13 @@ module.exports = async (html, config = {}) => {
     whitelist: [...get(config, 'whitelist', []), ...safelist]
   }
 
-  const options = merge(defaultOptions, get(config, 'removeUnusedCSS', config))
+  const options = merge(defaultOptions, get(config, 'removeUnusedCSS', {}))
 
-  html = await removeInlinedClasses(html, options)
+  /**
+   * Remove possibly inlined selectors, as long as we're not calling
+   * this function directly, i.e. Maizzle.removeUnusedCSS()
+   *  */
+  html = direct ? html : await removeInlinedClasses(html, options)
 
   return comb(html, options).result
 }

package/src/utils/helpers.js

@@ -1,9 +1,4 @@
 module.exports = {
-  asyncForEach: async (array, callback) => {
-    for (let index = 0; index < array.length; index++) {
-      await callback(array[index], index, array) // eslint-disable-line
-    }
-  },
   requireUncached: module => {
     try {
       delete require.cache[require.resolve(module)]