remove the DOCSIFY global made by Rollup, and move Docsify into a separate file where we can import it from tests while leaving the entry point for the bundle without any exports so that Rollup will not create a global variable from it

Author: trusktr

build/build.js

@@ -50,14 +50,12 @@ async function buildCore() {
   promises.push(build({
     input: 'src/core/index.js',
     output: 'docsify.js',
-    globalName: 'DOCSIFY'
   }))
 
   if (isProd) {
     promises.push(build({
       input: 'src/core/index.js',
       output: 'docsify.min.js',
-      globalName: 'DOCSIFY',
       plugins: [uglify()]
     }))
   }

src/core/Docsify.js

@@ -0,0 +1,23 @@
+import { initMixin } from './init';
+import { routerMixin } from './router';
+import { renderMixin } from './render';
+import { fetchMixin } from './fetch';
+import { eventMixin } from './event';
+import initGlobalAPI from './global-api';
+
+export function Docsify() {
+  this._init();
+}
+
+const proto = Docsify.prototype;
+
+initMixin(proto);
+routerMixin(proto);
+renderMixin(proto);
+fetchMixin(proto);
+eventMixin(proto);
+
+/**
+ * Global API
+ */
+initGlobalAPI();

src/core/index.js

@@ -1,37 +1,5 @@
-import Prism from 'prismjs';
-import marked from 'marked';
-import { initMixin } from './init';
-import { routerMixin } from './router';
-import { renderMixin } from './render';
-import { fetchMixin } from './fetch';
-import { eventMixin } from './event';
-import initGlobalAPI from './global-api';
-import { Compiler as DocsifyCompiler } from './render/compiler';
-import * as util from './util';
 import * as dom from './util/dom';
-import { slugify } from './render/slugify';
-import { get } from './fetch/ajax';
-
-export function Docsify() {
-  this._init();
-}
-
-const proto = Docsify.prototype;
-
-initMixin(proto);
-routerMixin(proto);
-renderMixin(proto);
-fetchMixin(proto);
-eventMixin(proto);
-
-/**
- * Global API
- */
-initGlobalAPI(); // deprecated
-
-// Rollup assigns all exports from this file onto a single DOCSIFY global.
-export { util, dom, get, slugify, DocsifyCompiler, marked, Prism };
-export const version = '__VERSION__';
+import { Docsify } from './Docsify';
 
 /**
  * Run Docsify

test/unit/docsify.test.js

@@ -68,18 +68,6 @@ describe('Docsify public API', () => {
     expect(typeof DOM.window.DocsifyCompiler).to.equal('function');
     expect(typeof DOM.window.marked).to.equal('function');
     expect(typeof DOM.window.Prism).to.equal('object');
-
-    // new global API, everything under one namespace (DOCSIFY)
-    expect(typeof DOM.window.DOCSIFY).to.equal('object');
-    expect(typeof DOM.window.DOCSIFY.Docsify).to.equal('function');
-    expect(typeof DOM.window.DOCSIFY.util).to.equal('object');
-    expect(typeof DOM.window.DOCSIFY.dom).to.equal('object');
-    expect(typeof DOM.window.DOCSIFY.get).to.equal('function');
-    expect(typeof DOM.window.DOCSIFY.slugify).to.equal('function');
-    expect(typeof DOM.window.DOCSIFY.version).to.equal('string');
-    expect(typeof DOM.window.DOCSIFY.DocsifyCompiler).to.equal('function');
-    expect(typeof DOM.window.DOCSIFY.marked).to.equal('function');
-    expect(typeof DOM.window.DOCSIFY.Prism).to.equal('object');
   });
 
   describe('Docsify config function', function() {
@@ -102,7 +90,7 @@ describe('Docsify public API', () => {
       };
 
       const { documentReady } = require('../../src/core/util/dom');
-      const { Docsify } = require('../../src/core/index');
+      const { Docsify } = require('../../src/core/Docsify');
       await new Promise(resolve => documentReady(resolve));
 
       new Docsify(); // eslint-disable-line
@@ -136,7 +124,7 @@ describe('Docsify public API', () => {
       };
 
       const { documentReady } = require('../../src/core/util/dom');
-      const { Docsify } = require('../../src/core/index');
+      const { Docsify } = require('../../src/core/Docsify');
       await new Promise(resolve => documentReady(resolve));
 
       new Docsify(); // eslint-disable-line