chore: revert file changes not required for this round of updates

Author: sghoweri

package.json

@@ -6,8 +6,7 @@
     "nohoist": [
       "@pattern-lab/engine-*",
       "**/@pattern-lab/engine-*",
-      "**/@pattern-lab/uikit-workshop",
-      "**/@pattern-lab/starterkit-handlebars-vanilla"
+      "**/@pattern-lab/uikit-workshop"
     ]
   },
   "dependencies": {

packages/core/package.json

@@ -4,8 +4,6 @@
   "version": "5.3.0",
   "main": "./src/index.js",
   "dependencies": {
-    "better-opn": "^1.0.0",
-    "browser-sync": "^2.26.7",
     "@pattern-lab/engine-mustache": "^5.0.0",
     "@pattern-lab/live-server": "^5.0.0",
     "chalk": "1.1.3",

packages/core/src/index.js

@@ -83,7 +83,7 @@ const patternlab_module = function(config) {
      * @memberof patternlab
      * @name getConfig
      * @instance
-     * @returns {string} the current patternlab-node config (defaults + customizations)
+     * @returns {object} the current patternlab-node config (defaults + customizations)
      */
     getConfig() {
       return config;

packages/core/src/lib/server.js

@@ -1,8 +1,7 @@
 'use strict';
 
 const path = require('path');
-const browserSync = require('browser-sync').create();
-const opn = require('better-opn');
+const liveServer = require('@pattern-lab/live-server');
 
 const events = require('./events');
 const logger = require('./log');
@@ -49,7 +48,10 @@ const server = patternlab => {
           //watch for builds to complete
           patternlab.events.on(events.PATTERNLAB_BUILD_END, () => {
             if (serverReady) {
-              browserSync.reload();
+              _module.reload({
+                file: '',
+                action: 'reload',
+              });
             }
           });
         };
@@ -60,42 +62,9 @@ const server = patternlab => {
           let resolveMsg = '';
           setTimeout(() => {
             try {
-              browserSync.init(
-                {
-                  logLevel: 'info',
-                  ui: false,
-                  notify: false,
-                  open: false,
-                  logFileChanges: false,
-                  reloadOnRestart: true,
-                  port: liveServerConfig.port, // try to use this port but choose another if unavailable
-                  server: liveServerConfig.root,
-                  files: [
-                    `${liveServerConfig.root}/**/*.css`,
-                    `${liveServerConfig.root}/**/*.js`,
-                  ],
-                  watchOptions: {
-                    ignoreInitial: false,
-                  },
-                },
-                function(err, bs) {
-                  const port = bs.options.get('port');
-                  serverReady = true; // so we only spin this up once Webpack has finished up initially
-
-                  resolveMsg = `Pattern Lab is being served from http://127.0.0.1:${port}`;
-                  opn(`http://127.0.0.1:${port}`);
-                  logger.info(resolveMsg);
-                }
-              );
-
-              browserSync.watch(
-                `${liveServerConfig.root}/**.html`,
-                (event, file) => {
-                  if (!patternlab.isBusy) {
-                    browserSync.reload('**/*.html');
-                  }
-                }
-              );
+              liveServer.start(liveServerConfig);
+              resolveMsg = `Pattern Lab is being served from http://127.0.0.1:${liveServerConfig.port}`;
+              logger.info(resolveMsg);
             } catch (e) {
               const err = `Pattern Lab serve failed to start: ${e}`;
               logger.error(`Pattern Lab serve failed to start: ${e}`);
@@ -120,15 +89,13 @@ const server = patternlab => {
         let action;
         try {
           if (!patternlab.isBusy) {
-            // @todo: re-evaluate to see if this specific logic is still necessary
-            // if (_data.file.indexOf('css') > -1 || _data.action === 'refresh') {
-            //   action = 'refreshed CSS';
-            //   // browserSync.refreshCSS();
-            // } else {
-            //   action = 'reloaded';
-            //   // browserSync.reload();
-            // }
-            browserSync.reload();
+            if (_data.file.indexOf('css') > -1 || _data.action === 'refresh') {
+              action = 'refreshed CSS';
+              liveServer.refreshCSS();
+            } else {
+              action = 'reloaded';
+              liveServer.reload();
+            }
             resolve(`Server ${action} successfully`);
           }
         } catch (e) {

packages/development-edition-engine-handlebars/build-tools.js

@@ -8,11 +8,7 @@
     "pl:help": "patternlab --help",
     "pl:install": "patternlab install --config ./patternlab-config.json",
     "pl:serve": "patternlab serve --config ./patternlab-config.json",
-    "pl:version": "patternlab --version",
-    "setup": "yarn --force && npx patternlab add --starterkits '@pattern-lab/starterkit-handlebars-vanilla' && npm run pl:build && npm run uikit:build",
-    "start": "npm-run-all -p  uikit:watch pl:serve",
-    "uikit:watch": "cd ../uikit-workshop && npm run watch",
-    "uikit:build": "cd ../uikit-workshop && npm run build"
+    "pl:version": "patternlab --version"
   },
   "keywords": [
     "Pattern Lab",
@@ -37,7 +33,6 @@
     "@pattern-lab/engine-mustache": "^5.0.0",
     "@pattern-lab/plugin-tab": "^5.3.0",
     "@pattern-lab/starterkit-mustache-demo": "^5.0.0",
-    "@pattern-lab/uikit-workshop": "^5.3.2",
-    "npm-run-all": "^4.1.5"
+    "@pattern-lab/uikit-workshop": "^5.3.2"
   }
 }