Simplified + unified debug naming conventions

- Reduced the number of levels in our debug naming in the frontend
- Unified components like "themes" and "routing" under one name
- Should help to make debug slightly more useful again

core/bridge.js

@@ -8,6 +8,8 @@
  *
  * This file is a great place for all the cross-component event handling in lieu of refactoring
  */
+
+const debug = require('@tryghost/debug')('bridge');
 const errors = require('@tryghost/errors');
 const config = require('./shared/config');
 const logging = require('@tryghost/logging');
@@ -73,6 +75,7 @@ class Bridge {
 
     reloadFrontend() {
         const apiVersion = this.getFrontendApiVersion();
+        debug('reload frontend', apiVersion);
         const siteApp = require('./server/web/site/app');
         siteApp.reload({apiVersion});
     }

core/frontend/services/routing/CollectionRouter.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('services:routing:collection-router');
+const debug = require('@tryghost/debug')('routing:collection-router');
 const events = require('../../../server/lib/common/events');
 const urlUtils = require('../../../shared/url-utils');
 const ParentRouter = require('./ParentRouter');

core/frontend/services/routing/ParentRouter.js

@@ -7,7 +7,7 @@
  * Only allows for .use and .get at the moment - we don't have clear use-cases for anything else yet.
  */
 
-const debug = require('@tryghost/debug')('services:routing:ParentRouter');
+const debug = require('@tryghost/debug')('routing:parent-router');
 
 const EventEmitter = require('events').EventEmitter;
 const express = require('../../../shared/express');

core/frontend/services/routing/StaticPagesRouter.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('services:routing:static-pages-router');
+const debug = require('@tryghost/debug')('routing:static-pages-router');
 const urlUtils = require('../../../shared/url-utils');
 const ParentRouter = require('./ParentRouter');
 const controllers = require('./controllers');

core/frontend/services/routing/StaticRoutesRouter.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('services:routing:static-routes-router');
+const debug = require('@tryghost/debug')('routing:static-routes-router');
 const events = require('../../../server/lib/common/events');
 const errors = require('@tryghost/errors');
 const urlUtils = require('../../../shared/url-utils');

core/frontend/services/routing/TaxonomyRouter.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('services:routing:taxonomy-router');
+const debug = require('@tryghost/debug')('routing:taxonomy-router');
 const config = require('../../../shared/config');
 const events = require('../../../server/lib/common/events');
 const ParentRouter = require('./ParentRouter');

core/frontend/services/routing/bootstrap.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('services:routing:bootstrap');
+const debug = require('@tryghost/debug')('routing');
 const _ = require('lodash');
 const events = require('../../../server/lib/common/events');
 const frontendSettings = require('../settings');
@@ -28,7 +28,7 @@ let siteRouter;
  * @returns {ExpressRouter}
  */
 module.exports.init = (options = {start: false}) => {
-    debug('bootstrap');
+    debug('bootstrap init', options);
 
     registry.resetAllRouters();
     registry.resetAllRoutes();
@@ -65,6 +65,7 @@ module.exports.init = (options = {start: false}) => {
  * @param {object} dynamicRoutes
  */
 module.exports.start = (apiVersion, dynamicRoutes) => {
+    debug('bootstrap start', apiVersion, dynamicRoutes);
     const RESOURCE_CONFIG = require(`./config/${apiVersion}`);
 
     const unsubscribeRouter = new UnsubscribeRouter();

core/server/services/themes/loader.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('themes:loader');
+const debug = require('@tryghost/debug')('themes');
 const config = require('../../../shared/config');
 const packageJSON = require('@tryghost/package-json');
 const themeList = require('./list');

core/server/services/themes/storage.js

@@ -1,3 +1,4 @@
+const debug = require('@tryghost/debug')('themes');
 const fs = require('fs-extra');
 
 const bridge = require('../../../bridge');
@@ -11,7 +12,7 @@ const settingsCache = require('../../../shared/settings-cache');
 const i18n = require('../../../shared/i18n');
 const logging = require('@tryghost/logging');
 const errors = require('@tryghost/errors');
-const debug = require('@tryghost/debug')('api:themes');
+
 const ObjectID = require('bson-objectid');
 
 let themeStorage;

core/server/services/themes/validate.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('theme:validate');
+const debug = require('@tryghost/debug')('themes');
 const _ = require('lodash');
 const Promise = require('bluebird');
 const fs = require('fs-extra');

core/server/web/members/app.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('web:members:app');
+const debug = require('@tryghost/debug')('members');
 const {URL} = require('url');
 const cors = require('cors');
 const bodyParser = require('body-parser');

core/server/web/parent/frontend.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('web:frontend');
+const debug = require('@tryghost/debug')('frontend');
 const express = require('../../../shared/express');
 
 const shared = require('../shared');
@@ -9,7 +9,7 @@ const shared = require('../shared');
  * @returns {import('express').RequestHandler}
  */
 module.exports = (options) => {
-    debug('FrontendApp setup start');
+    debug('FrontendApp setup start', options);
 
     // FRONTEND
     const frontendApp = express('frontend');

core/server/web/site/app.js

@@ -1,4 +1,4 @@
-const debug = require('@tryghost/debug')('web:site:app');
+const debug = require('@tryghost/debug')('frontend');
 const path = require('path');
 const express = require('../../../shared/express');
 const cors = require('cors');
@@ -73,7 +73,7 @@ function SiteRouter(req, res, next) {
 }
 
 module.exports = function setupSiteApp(options = {}) {
-    debug('Site setup start');
+    debug('Site setup start', options);
 
     const siteApp = express('site');
 

core/server/web/site/routes.js

@@ -1,5 +1,7 @@
+const debug = require('@tryghost/debug')('routing');
 const routing = require('../../../frontend/services/routing');
 
 module.exports = function siteRoutes(options = {}) {
+    debug('site Routes', options);
     return routing.bootstrap.init(options);
 };