Remove semicolons, run formatting. Remove devDependencies

Author: webketje

.eslintrc.yml

@@ -3,6 +3,17 @@ env:
   es6: true
 extends:
   - 'eslint:recommended'
+  - 'plugin:node/recommended'
+  - 'plugin:import/recommended'
   - 'prettier'
 parserOptions:
   ecmaVersion: 2020
+rules:
+  no-console: error
+  prefer-const: error
+  no-var: error
+  node/exports-style: [0, error]
+  import/first: error
+  import/no-anonymous-default-export: error
+  import/no-unassigned-import: error
+  import/no-internal-modules: error

.prettierrc.yml

@@ -1,6 +1,6 @@
 trailingComma: none
 tabWidth: 2
-semi: true
+semi: false
 singleQuote: true
 bracketSpacing: true
 arrowParens: always

lib/get-transformer.js

@@ -1,21 +1,21 @@
-const jstransformer = require('jstransformer');
-const toTransformer = require('inputformat-to-jstransformer');
+const jstransformer = require('jstransformer')
+const toTransformer = require('inputformat-to-jstransformer')
 
 /**
  * Gets jstransformer for an extension, and caches them
  */
 
-const cache = {};
+const cache = {}
 
 function getTransformer(ext) {
   if (ext in cache) {
-    return cache[ext];
+    return cache[ext]
   }
 
-  const transformer = toTransformer(ext);
-  cache[ext] = transformer ? jstransformer(transformer) : false;
+  const transformer = toTransformer(ext)
+  cache[ext] = transformer ? jstransformer(transformer) : false
 
-  return cache[ext];
+  return cache[ext]
 }
 
-module.exports = getTransformer;
+module.exports = getTransformer

lib/index.js

@@ -1,8 +1,8 @@
-const debug = require('debug')('@metalsmith/layouts');
-const match = require('multimatch');
-const path = require('path');
-const isUtf8 = require('is-utf8');
-const getTransformer = require('./get-transformer');
+const debug = require('debug')('@metalsmith/layouts')
+const match = require('multimatch')
+const path = require('path')
+const isUtf8 = require('is-utf8')
+const getTransformer = require('./get-transformer')
 
 /**
  * Resolves layouts, in the following order:
@@ -13,132 +13,132 @@ const getTransformer = require('./get-transformer');
 
 function getLayout({ file, settings }) {
   if (file.layout || file.layout === false) {
-    return file.layout;
+    return file.layout
   }
 
-  return settings.default;
+  return settings.default
 }
 
 /**
  * Engine, renders file with the appropriate layout
  */
 
 function render({ filename, files, metadata, settings, metalsmith }) {
-  const file = files[filename];
-  const layout = getLayout({ file, settings });
-  const extension = layout.split('.').pop();
+  const file = files[filename]
+  const layout = getLayout({ file, settings })
+  const extension = layout.split('.').pop()
 
-  debug(`rendering ${filename} with layout ${layout}`);
+  debug(`rendering ${filename} with layout ${layout}`)
 
   // Stringify file contents
-  const contents = file.contents.toString();
+  const contents = file.contents.toString()
 
-  const transform = getTransformer(extension);
-  const locals = { ...metadata, ...file, contents };
-  const layoutPath = path.join(metalsmith.path(settings.directory), layout);
+  const transform = getTransformer(extension)
+  const locals = { ...metadata, ...file, contents }
+  const layoutPath = path.join(metalsmith.path(settings.directory), layout)
 
   // Transform the contents
   return transform
     .renderFileAsync(layoutPath, settings.engineOptions, locals)
     .then((rendered) => {
       // Update file with results
       // eslint-disable-next-line no-param-reassign
-      file.contents = Buffer.from(rendered.body);
-      debug(`done rendering ${filename}`);
+      file.contents = Buffer.from(rendered.body)
+      debug(`done rendering ${filename}`)
     })
     .catch((err) => {
       // Prepend error message with file path
       // eslint-disable-next-line no-param-reassign
-      err.message = `${filename}: ${err.message}`;
-      throw err;
-    });
+      err.message = `${filename}: ${err.message}`
+      throw err
+    })
 }
 
 /**
  * Validate, checks whether a file should be processed
  */
 
 function validate({ filename, files, settings }) {
-  const file = files[filename];
-  const layout = getLayout({ file, settings });
+  const file = files[filename]
+  const layout = getLayout({ file, settings })
 
-  debug(`validating ${filename}`);
+  debug(`validating ${filename}`)
 
   // Files without a layout cannot be processed
   if (!layout) {
-    debug(`validation failed, ${filename} does not have a layout set`);
-    return false;
+    debug(`validation failed, ${filename} does not have a layout set`)
+    return false
   }
 
   // Layouts without an extension cannot be processed
   if (!layout.includes('.')) {
-    debug(`validation failed, layout for ${filename} does not have an extension`);
-    return false;
+    debug(`validation failed, layout for ${filename} does not have an extension`)
+    return false
   }
 
   // Files that are not utf8 are ignored
   if (!isUtf8(file.contents)) {
-    debug(`validation failed, ${filename} is not utf-8`);
-    return false;
+    debug(`validation failed, ${filename} is not utf-8`)
+    return false
   }
 
   // Files without an applicable jstransformer are ignored
-  const extension = layout.split('.').pop();
-  const transformer = getTransformer(extension);
+  const extension = layout.split('.').pop()
+  const transformer = getTransformer(extension)
 
   if (!transformer) {
-    debug(`validation failed, no jstransformer found for layout for ${filename}`);
+    debug(`validation failed, no jstransformer found for layout for ${filename}`)
   }
 
-  return transformer;
+  return transformer
 }
 
 /**
  * Plugin, the main plugin used by metalsmith
  */
 
 module.exports = (options) => (files, metalsmith, done) => {
-  const metadata = metalsmith.metadata();
+  const metadata = metalsmith.metadata()
   const defaults = {
     pattern: '**',
     directory: 'layouts',
     engineOptions: {},
     suppressNoFilesError: false
-  };
-  const settings = { ...defaults, ...options };
+  }
+  const settings = { ...defaults, ...options }
 
   // Check whether the pattern option is valid
   if (!(typeof settings.pattern === 'string' || Array.isArray(settings.pattern))) {
     return done(
       new Error(
         'invalid pattern, the pattern option should be a string or array of strings. See https://www.npmjs.com/package/@metalsmith/layouts#pattern'
       )
-    );
+    )
   }
 
   // Filter files by the pattern
-  const matchedFiles = match(Object.keys(files), settings.pattern);
+  const matchedFiles = match(Object.keys(files), settings.pattern)
 
   // Filter files by validity
-  const validFiles = matchedFiles.filter((filename) => validate({ filename, files, settings }));
+  const validFiles = matchedFiles.filter((filename) => validate({ filename, files, settings }))
 
   // Let the user know when there are no files to process, unless the check is suppressed
   if (validFiles.length === 0) {
     const message =
-      'no files to process. See https://www.npmjs.com/package/@metalsmith/layouts#suppressnofileserror';
+      'no files to process. See https://www.npmjs.com/package/@metalsmith/layouts#suppressnofileserror'
 
     if (settings.suppressNoFilesError) {
-      debug(message);
-      return done();
+      debug(message)
+      return done()
     }
 
-    return done(new Error(message));
+    return done(new Error(message))
   }
 
   // Map all files that should be processed to an array of promises and call done when finished
   return Promise.all(
     validFiles.map((filename) => render({ filename, files, metadata, settings, metalsmith }))
   )
     .then(() => done())
-    .catch(/* istanbul ignore next */ (error) => done(error));
-};
+    .catch(/* istanbul ignore next */ (error) => done(error))
+}