chore(build): misc updates

Author: hawkeye64

ui/.eslintignore

@@ -1 +1,3 @@
-docs
+/docs
+/dist
+

ui/.eslintrc.js

@@ -7,11 +7,15 @@ module.exports = {
   parserOptions: {
     parser: 'babel-eslint',
     ecmaVersion: 2018, // Allows for the parsing of modern ECMAScript features
-    sourceType: 'module' // Allows for the use of imports
+    sourceType: 'module', // Allows for the use of imports
+    ecmaFeatures: {
+      jsx: true
+    }
   },
 
   env: {
-    browser: true
+    browser: true,
+    es6: true
   },
 
   // Rules order is important, please avoid shuffling them
@@ -29,9 +33,9 @@ module.exports = {
     'plugin:quasar/standard',
 
     'standard'
-
   ],
 
+  // required to lint *.vue files
   plugins: [
     // https://eslint.vuejs.org/user-guide/#why-doesn-t-it-work-on-vue-file
     // required to lint *.vue files

ui/.gitignore

@@ -0,0 +1,12 @@
+# Testing
+/test/jest/coverage
+
+ui/yarn.lock
+ui/node_modules
+ui/dev/yarn.lock
+ui/dev/node_modules
+
+demo/yarn.lock
+demo/node_modules
+
+./dist

ui/build/build.api.js

@@ -2,4 +2,7 @@ const path = require('path')
 global.rootDir = path.resolve(__dirname, '..')
 global.distDir = path.resolve(__dirname, '../dist')
 
-require('quasar-json-api')()
+require('quasar-json-api')({
+  buildVetur: true,
+  buildTypes: true
+})

ui/build/index.js

@@ -1,9 +1,8 @@
 process.env.NODE_ENV = 'production'
 
 const parallel = require('os').cpus().length > 1
-const runJob = parallel ? require('child_process').fork : require
-// const runJob = require
-const { join } = require('path')
+// const runJob = parallel ? require('child_process').fork : require
+// const { join } = require('path')
 const { createFolder } = require('./utils')
 const { green, blue } = require('chalk')
 
@@ -15,10 +14,9 @@ require('./script.clean.js')
 console.log(` 📦 Building ${green('v' + require('../package.json').version)}...${parallel ? blue(' [multi-threaded]') : ''}\n`)
 
 createFolder('dist')
-createFolder('dist/icon-set')
-createFolder('dist/api')
-createFolder('dist/vetur')
 
-runJob(join(__dirname, './icons/build.all.js'))
-runJob(join(__dirname, './script.javascript.js'))
-runJob(join(__dirname, './script.css.js'))
+require('./build.api.js')
+require('./script.javascript')
+require('./script.css')
+// runJob(join(__dirname, './script.javascript.js'))
+// runJob(join(__dirname, './script.css.js'))

ui/build/script.app-ext.js

@@ -27,8 +27,8 @@ module.exports.syncAppExt = function (both = true) {
 
   // read app-ext package.json
   const appExtFile = resolvePath('app-extension/package.json')
-  let appExtJson = require(appExtFile),
-    finished = false
+  const appExtJson = require(appExtFile)
+  let finished = false
 
   // sync version numbers
   if (both === true) {
@@ -56,5 +56,5 @@ module.exports.syncAppExt = function (both = true) {
     return
   }
 
-  console.error(`   App Extension version and dependency NOT synced.\n`)
+  console.error('   App Extension version and dependency NOT synced.\n')
 }

ui/build/script.clean.js

@@ -3,4 +3,4 @@ var
   path = require('path')
 
 rimraf.sync(path.resolve(__dirname, '../dist/*'))
-console.log(` 💥 Cleaned build artifacts.\n`)
+console.log(' 💥 Cleaned build artifacts.\n')

ui/build/script.css.js

@@ -1,8 +1,9 @@
+/* eslint-disable array-bracket-spacing */
 const path = require('path')
-const sass = require('node-sass')
+const sass = require('sass')
 const postcss = require('postcss')
 const cssnano = require('cssnano')
-const rtl = require('postcss-rtl')
+const rtl = require('rtlcss')
 const autoprefixer = require('autoprefixer')
 
 const buildConf = require('./config')
@@ -24,7 +25,7 @@ const nano = postcss([
 
 Promise
   .all([
-    generate('src/index.sass', `dist/index`)
+    generate('src/index.sass', 'dist/index')
   ])
   .catch(e => {
     console.error(e)
@@ -53,19 +54,19 @@ function generate (src, dest) {
       resolve(result.css)
     })
   })
-  .then(code => buildConf.banner + code)
-  .then(code => postCssCompiler.process(code, { from: void 0 }))
-  .then(code => {
-    code.warnings().forEach(warn => {
-      console.warn(warn.toString())
+    .then(code => buildConf.banner + code)
+    .then(code => postCssCompiler.process(code, { from: void 0 }))
+    .then(code => {
+      code.warnings().forEach(warn => {
+        console.warn(warn.toString())
+      })
+      return code.css
     })
-    return code.css
-  })
-  .then(code => Promise.all([
-    generateUMD(dest, code),
-    postCssRtlCompiler.process(code, { from: void 0 })
-      .then(code => generateUMD(dest, code.css, '.rtl'))
-  ]))
+    .then(code => Promise.all([
+      generateUMD(dest, code),
+      postCssRtlCompiler.process(code, { from: void 0 })
+        .then(code => generateUMD(dest, code.css, '.rtl'))
+    ]))
 }
 
 function generateUMD (dest, code, ext = '') {

ui/build/script.javascript.js

@@ -5,7 +5,6 @@ const rollup = require('rollup')
 const uglify = require('uglify-es')
 const buble = require('@rollup/plugin-buble')
 const json = require('@rollup/plugin-json')
-const cjs = require('@rollup/plugin-commonjs')
 const { nodeResolve } = require('@rollup/plugin-node-resolve')
 
 const buildConf = require('./config')
@@ -20,27 +19,20 @@ const nodeResolveConfig = {
   preferBuiltins: false
 }
 
-const cjsConfig = {
-  include: [
-    /node_modules/
-  ]
-}
-
 const rollupPlugins = [
   nodeResolve(nodeResolveConfig),
   json(),
-  cjs(cjsConfig),
   buble(bubleConfig)
 ]
 
 const builds = [
   {
     rollup: {
       input: {
-        input: resolve('entry/index.esm.js')
+        input: pathResolve('entry/index.esm.js')
       },
       output: {
-        file: resolve('../dist/index.esm.js'),
+        file: pathResolve('../dist/index.esm.js'),
         format: 'es'
       }
     },
@@ -52,10 +44,10 @@ const builds = [
   {
     rollup: {
       input: {
-        input: resolve('entry/index.common.js')
+        input: pathResolve('entry/index.common.js')
       },
       output: {
-        file: resolve('../dist/index.common.js'),
+        file: pathResolve('../dist/index.common.js'),
         format: 'cjs',
         exports: 'auto'
       }
@@ -68,11 +60,11 @@ const builds = [
   {
     rollup: {
       input: {
-        input: resolve('entry/index.umd.js')
+        input: pathResolve('entry/index.umd.js')
       },
       output: {
         name: 'QIconPicker',
-        file: resolve('../dist/index.umd.js'),
+        file: pathResolve('../dist/index.umd.js'),
         format: 'umd'
       }
     },
@@ -88,24 +80,21 @@ const builds = [
 addAssets(builds, 'icon-set', 'iconSet')
 
 build(builds)
-  .then(() => {
-    require('./build.api')
-  })
 
 /**
  * Helpers
  */
 
-function resolve (_path) {
+function pathResolve (_path) {
   return path.resolve(__dirname, _path)
 }
 
 // eslint-disable-next-line no-unused-vars
 function addAssets (builds, type, injectName) {
   const
-    files = fs.readdirSync(resolve('../../ui/src/components/' + type)),
+    files = fs.readdirSync(pathResolve('../../ui/src/components/' + type)),
     plugins = [buble(bubleConfig)],
-    outputDir = resolve(`../dist/${type}`)
+    outputDir = pathResolve(`../dist/${type}`)
 
   fse.mkdirp(outputDir)
 
@@ -116,11 +105,11 @@ function addAssets (builds, type, injectName) {
       builds.push({
         rollup: {
           input: {
-            input: resolve(`../src/components/${type}/${file}`),
+            input: pathResolve(`../src/components/${type}/${file}`),
             plugins
           },
           output: {
-            file: addExtension(resolve(`../dist/${type}/${file}`), 'umd'),
+            file: addExtension(pathResolve(`../dist/${type}/${file}`), 'umd'),
             format: 'umd',
             name: `QIconPicker.${injectName}.${name}`
           }
@@ -200,7 +189,7 @@ function buildEntry (config) {
 }
 
 function injectVueRequirement (code) {
-  // eslint-disable-next-line
+  // eslint-disable-next-line quotes
   const index = code.indexOf(`Vue = Vue && Vue.hasOwnProperty('default') ? Vue['default'] : Vue`)
 
   if (index === -1) {