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

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]
 

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';

config/node-commonjs.js

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

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",

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;

src/index.d.ts

@@ -39,18 +39,24 @@ renderToString.render = renderToString;
  *	@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) {
+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 +413,3 @@ function getFallbackComponentName(component) {
 	}
 	return name;
 }
-renderToString.shallowRender = shallowRender;
-
-export default renderToString;
-
-export {
-	renderToString as render,
-	renderToString as renderToStaticMarkup,
-	renderToString,
-	shallowRender
-};

src/index.js

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

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 };

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';

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', () => {

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';

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>;