Remove all default exports (major)

Mixing both default and named exports has no clearly defined behaviour
and can easily lead to unexpected results. Some tools use a combination of
"default" + "__esModule" property and others ignore named exports
completely if both are found. Those issues are difficult to debug when multiple tools are
used together.

This change makes some code redundant by not having to patch a hand crafted exports
object for commonjs environments.

Author: marvinhagemeister

Diff for: README.md

@@ -16,7 +16,7 @@ Works in Node & the browser, making it useful for universal/isomorphic rendering
 ### Render JSX/VDOM to HTML
 
 ```js
-import render from 'preact-render-to-string';
+import { render } from 'preact-render-to-string';
 import { h } from 'preact';
 /** @jsx h */
 
@@ -31,7 +31,7 @@ console.log(html);
 ### Render Preact Components to HTML
 
 ```js
-import render from 'preact-render-to-string';
+import { render } from 'preact-render-to-string';
 import { h, Component } from 'preact';
 /** @jsx h */
 
@@ -66,7 +66,7 @@ console.log(html);
 ```js
 import express from 'express';
 import { h } from 'preact';
-import render from 'preact-render-to-string';
+import { render } from 'preact-render-to-string';
 /** @jsx h */
 
 // silly example component:
@@ -92,8 +92,30 @@ app.get('/:fox', (req, res) => {
 
 ---
 
+## Migration guide
 
-### License
+### Migrating from 5.x to 6.x
+
+The only breaking change introduced with the `6.x` is that the `default` exports have been removed in favor of named exports. To update, replace the default import in your code with a named one.
+
+```diff
+- import render from 'preact-render-to-string';
++ import { render } from 'preact-render-to-string';
+```
+
+Similarily if you've been using the `jsx` renderer, the default import needs to be swapped with a named import:
+
+```diff
+- import render from 'preact-render-to-string/jsx';
++ import { render } from 'preact-render-to-string/jsx';
+```
+
+_Note: The named exports were already present in the `5.x` release line. So if you can't update today for any reason, you can apply the above changes safely to make a future update to `6.x` easier!_
+
+---
+
+
+## License
 
 [MIT]
 

Diff for: benchmarks/index.js

@@ -1,6 +1,6 @@
 import { h } from 'preact';
 import Suite from 'benchmarkjs-pretty';
-import renderToString from '../src/index';
+import { renderToString } from '../src/index';
 import TextApp from './text';
 // import StackApp from './stack';
 import { App as IsomorphicSearchResults } from './isomorphic-ui-search-results';

Diff for: config/node-commonjs.js

@@ -10,4 +10,3 @@ interface Options {
 }
 
 export function render(vnode: VNode, context?: any, options?: Options): string;
-export default render;

Diff for: jsx.d.ts

@@ -24,7 +24,7 @@
 	"scripts": {
 		"bench": "node -r @babel/register benchmarks index.js",
 		"build": "npm run -s transpile && npm run -s transpile:jsx && npm run -s copy-typescript-definition",
-		"postbuild": "node ./config/node-13-exports.js && node ./config/node-commonjs.js",
+		"postbuild": "node ./config/node-13-exports.js",
 		"transpile": "microbundle src/index.js -f es,umd --target web --external preact",
 		"transpile:jsx": "microbundle src/jsx.js -o dist/jsx.js --target web --external none && microbundle dist/jsx.js -o dist/jsx.js -f cjs",
 		"copy-typescript-definition": "copyfiles -f src/*.d.ts dist",

Diff for: package.json

@@ -6,11 +6,10 @@ interface Options {
 	pretty?: boolean | string;
 }
 
-export function render(vnode: VNode, context?: any, options?: Options): string;
 export function renderToString(
 	vnode: VNode,
 	context?: any,
 	options?: Options
 ): string;
+export const render: typeof renderToString;
 export function shallowRender(vnode: VNode, context?: any): string;
-export default render;

Diff for: src/index.d.ts

@@ -19,38 +19,42 @@ const UNSAFE_NAME = /[\s\n\\/='"\0<>]/;
 
 const noop = () => {};
 
-/** Render Preact JSX + Components to an HTML string.
- *	@name render
- *	@function
- *	@param {VNode} vnode	JSX VNode to render.
- *	@param {Object} [context={}]	Optionally pass an initial context object through the render path.
- *	@param {Object} [options={}]	Rendering options
- *	@param {Boolean} [options.shallow=false]	If `true`, renders nested Components as HTML elements (`<Foo a="b" />`).
- *	@param {Boolean} [options.xml=false]		If `true`, uses self-closing tags for elements without children.
- *	@param {Boolean} [options.pretty=false]		If `true`, adds whitespace for readability
- *	@param {RegEx|undefined} [options.voidElements]       RegeEx that matches elements that are considered void (self-closing)
- */
-renderToString.render = renderToString;
-
 /** Only render elements, leaving Components inline as `<ComponentName ... />`.
  *	This method is just a convenience alias for `render(vnode, context, { shallow:true })`
  *	@name shallow
  *	@function
  *	@param {VNode} vnode	JSX VNode to render.
  *	@param {Object} [context={}]	Optionally pass an initial context object through the render path.
  */
-let shallowRender = (vnode, context) => renderToString(vnode, context, SHALLOW);
+export let shallowRender = (vnode, context) =>
+	renderToString(vnode, context, SHALLOW);
 
 const EMPTY_ARR = [];
-function renderToString(vnode, context, opts) {
+
+/**
+ * Render Preact JSX + Components to an HTML string.
+ * @param {VNode} vnode	JSX VNode to render.
+ * @param {Object} [context={}]	Optionally pass an initial context object through the render path.
+ * @param {Object} [options={}]	Rendering options
+ * @param {Boolean} [options.shallow=false]	If `true`, renders nested Components as HTML elements (`<Foo a="b" />`).
+ * @param {Boolean} [options.xml=false]		If `true`, uses self-closing tags for elements without children.
+ * @param {Boolean} [options.pretty=false]		If `true`, adds whitespace for readability
+ * @param {RegEx|undefined} [options.voidElements]       RegeEx that matches elements that are considered void (self-closing)
+ */
+export function renderToString(vnode, context, opts) {
 	const res = _renderToString(vnode, context, opts);
 	// options._commit, we don't schedule any effects in this library right now,
 	// so we can pass an empty queue to this hook.
 	if (options.__c) options.__c(vnode, EMPTY_ARR);
 	return res;
 }
 
-/** The default export is an alias of `render()`. */
+export {
+	// Aliased export for React compat
+	renderToString as renderToStaticMarkup,
+	renderToString as render
+};
+
 function _renderToString(vnode, context, opts, inner, isSvgMode, selectValue) {
 	if (vnode == null || typeof vnode === 'boolean') {
 		return '';
@@ -407,13 +411,3 @@ function getFallbackComponentName(component) {
 	}
 	return name;
 }
-renderToString.shallowRender = shallowRender;
-
-export default renderToString;
-
-export {
-	renderToString as render,
-	renderToString as renderToStaticMarkup,
-	renderToString,
-	shallowRender
-};

Diff for: src/index.js

@@ -10,4 +10,3 @@ interface Options {
 }
 
 export function render(vnode: VNode, context?: any, options?: Options): string;
-export default render;

Diff for: src/jsx.d.ts

@@ -1,5 +1,5 @@
 import './polyfills';
-import renderToString from './index';
+import { renderToString } from './index';
 import { indent, encodeEntities, assign } from './util';
 import prettyFormat from 'pretty-format';
 
@@ -72,5 +72,4 @@ function renderToJsxString(vnode, context, opts, inner) {
 	return renderToString(vnode, context, opts, inner);
 }
 
-export default renderToJsxString;
 export { renderToJsxString as render };

Diff for: src/jsx.js

@@ -1,4 +1,4 @@
-import render from '../src/jsx';
+import { render } from '../src/jsx';
 import { h, createContext, Component } from 'preact';
 import chai, { expect } from 'chai';
 import sinonChai from 'sinon-chai';

Diff for: test/context.js

@@ -1,25 +1,21 @@
-import renderToString, {
+import {
 	render,
 	shallowRender,
 	renderToStaticMarkup,
-	renderToString as _renderToString
+	renderToString
 } from '../src';
 import { expect } from 'chai';
 
 describe('render-to-string', () => {
 	describe('exports', () => {
-		it('exposes renderToString as default', () => {
-			expect(renderToString).to.be.a('function');
-		});
-
 		it('exposes render as a named export', () => {
 			expect(render).to.be.a('function');
 			expect(render).to.equal(renderToString);
 		});
 
 		it('exposes renderToString as a named export', () => {
-			expect(_renderToString).to.be.a('function');
-			expect(_renderToString).to.equal(renderToString);
+			expect(renderToString).to.be.a('function');
+			expect(renderToString).to.equal(renderToString);
 		});
 
 		it('exposes renderToStaticMarkup as a named export', () => {

Diff for: test/index.js

@@ -1,4 +1,4 @@
-import render from '../src/jsx';
+import { render } from '../src/jsx';
 import { h } from 'preact';
 import chai, { expect } from 'chai';
 import sinonChai from 'sinon-chai';

Diff for: test/jsx.js

@@ -1,4 +1,4 @@
-import render from '../src';
+import { render } from '../src';
 import { h } from 'preact';
 
 let vdom = <div class="foo">content</div>;