-> v0.15.3

parallelize exports

CHANGELOG.md

@@ -1,5 +1,14 @@
 # sapper changelog
 
+## 0.15.3
+
+* Crawl pages in parallel when exporting ([#329](https://github.com/sveltejs/sapper/pull/329))
+* Don't minify inline JS when exporting ([#328](https://github.com/sveltejs/sapper/pull/328))
+
+## 0.15.2
+
+* Collapse component chains where no intermediate layout component is specified ([#312](https://github.com/sveltejs/sapper/issues/312))
+
 ## 0.15.1
 
 * Prevent confusing error when no root layout is specified

README.md

@@ -31,6 +31,44 @@ npm run build
 npm start
 ```
 
+## Development
+
+Pull requests are encouraged and always welcome. [Pick an issue](https://github.com/sveltejs/sapper/issues?q=is%3Aissue+is%3Aopen+sort%3Aupdated-desc) and help us out!
+
+To install and work on Sapper locally:
+
+```bash
+git clone git@github.com:sveltejs/sapper.git
+cd sapper
+npm install
+npm run dev
+```
+
+### Linking to a Live Project
+
+You can make changes locally to Sapper and test it against a local Sapper project. For a quick project that takes almost no setup, use the default [sapper-template](https://github.com/sveltejs/sapper-template) project. Instruction on setup are found in that project repository.
+
+To link Sapper to your project, from the root of your local Sapper git checkout:
+
+```bash
+cd sapper
+npm link
+```
+
+Then, to link from `sapper-template` (or any other given project):
+
+```bash
+cd sapper-template
+npm link sapper
+```
+
+You should be good to test changes locally.
+
+### Running Tests
+
+```bash
+npm run test
+```
 
 ## License
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "sapper",
-  "version": "0.15.1",
+  "version": "0.15.3",
   "description": "Military-grade apps, engineered by Svelte",
   "main": "dist/middleware.ts.js",
   "bin": {
@@ -23,7 +23,7 @@
     "cheerio": "^1.0.0-rc.2",
     "chokidar": "^2.0.3",
     "cookie": "^0.3.1",
-    "devalue": "^1.0.1",
+    "devalue": "^1.0.4",
     "glob": "^7.1.2",
     "html-minifier": "^3.5.16",
     "mkdirp": "^0.5.1",

src/api/export.ts

@@ -118,9 +118,7 @@ async function execute(emitter: EventEmitter, {
 					}
 				});
 
-				for (const url of urls) {
-					await handle(url);
-				}
+				await Promise.all(urls.map(handle));
 			}
 		}
 	}

src/api/utils/minify_html.ts

@@ -8,7 +8,7 @@ export function minify_html(html: string) {
 		decodeEntities: true,
 		html5: true,
 		minifyCSS: true,
-		minifyJS: true,
+		minifyJS: false,
 		removeAttributeQuotes: true,
 		removeComments: true,
 		removeOptionalTags: true,

src/core/create_manifests.ts

@@ -79,6 +79,8 @@ function generate_client(
 					pattern: ${page.pattern},
 					parts: [
 						${page.parts.map(part => {
+							if (part === null) return 'null';
+
 							if (part.params.length > 0) {
 								const props = part.params.map((param, i) => `${param}: match[${i + 1}]`);
 								return `{ component: ${part.component.name}, params: match => ({ ${props.join(', ')} }) }`;
@@ -150,6 +152,8 @@ function generate_server(
 					pattern: ${page.pattern},
 					parts: [
 						${page.parts.map(part => {
+							if (part === null) return 'null';
+
 							const props = [
 								`name: "${part.component.name}"`,
 								`component: ${part.component.name}`

src/core/create_routes.ts

@@ -101,27 +101,21 @@ export default function create_routes(cwd = locations.routes()) {
 
 			if (item.is_dir) {
 				const index = path.join(dir, item.basename, '_layout.html');
-				const layout = fs.existsSync(index)
-					? {
-						name: `${get_slug(item.file)}__layout`,
-						file: `${item.file}/_layout.html`
-					}
-					: null;
 
-				if (layout) {
-					components.push(layout);
-				} else if (components.indexOf(default_layout) === -1) {
-					components.push(default_layout);
-				}
+				const component = fs.existsSync(index) && {
+					name: `${get_slug(item.file)}__layout`,
+					file: `${item.file}/_layout.html`
+				};
+
+				if (component) components.push(component);
 
 				walk(
 					path.join(dir, item.basename),
 					segments,
 					params,
-					stack.concat({
-						component: layout || default_layout,
-						params
-					})
+					component
+						? stack.concat({ component, params })
+						: stack.concat(null)
 				);
 			}
 

src/middleware.ts

@@ -292,6 +292,8 @@ function get_page_handler(manifest: Manifest, store_getter: (req: Req) => Store)
 		let preloaded_chunks = Array.isArray(chunks.main) ? chunks.main : [chunks.main];
 		if (!error) {
 			page.parts.forEach(part => {
+				if (!part) return;
+
 				// using concat because it could be a string or an array. thanks webpack!
 				preloaded_chunks = preloaded_chunks.concat(chunks[part.name]);
 			});
@@ -366,6 +368,8 @@ function get_page_handler(manifest: Manifest, store_getter: (req: Req) => Store)
 			: {};
 
 		Promise.all([root_preloaded].concat(page.parts.map(part => {
+			if (!part) return null;
+
 			return part.component.preload
 				? part.component.preload.call(preload_context, {
 					path: req.path,
@@ -411,23 +415,28 @@ function get_page_handler(manifest: Manifest, store_getter: (req: Req) => Store)
 
 			const data = Object.assign({}, props, preloaded[0], {
 				params: {},
-				child: {}
+				child: {
+					segment: segments[0]
+				}
 			});
 
 			let level = data.child;
 			for (let i = 0; i < page.parts.length; i += 1) {
 				const part = page.parts[i];
+				if (!part) continue;
+
 				const get_params = part.params || (() => ({}));
 
 				Object.assign(level, {
-					segment: segments[i],
 					component: part.component,
 					props: Object.assign({}, props, {
 						params: get_params(match)
 					}, preloaded[i + 1])
 				});
 
-				level.props.child = <Props["child"]>{};
+				level.props.child = <Props["child"]>{
+					segment: segments[i + 1]
+				};
 				level = level.props.child;
 			}
 
@@ -443,9 +452,9 @@ function get_page_handler(manifest: Manifest, store_getter: (req: Req) => Store)
 
 			let inline_script = `__SAPPER__={${[
 				error && `error:1`,
-				`baseUrl: "${req.baseUrl}"`,
-				serialized.preloaded && `preloaded: ${serialized.preloaded}`,
-				serialized.store && `store: ${serialized.store}`
+				`baseUrl:"${req.baseUrl}"`,
+				serialized.preloaded && `preloaded:${serialized.preloaded}`,
+				serialized.store && `store:${serialized.store}`
 			].filter(Boolean).join(',')}};`;
 
 			const has_service_worker = fs.existsSync(path.join(locations.dest(), 'service-worker.js'));

src/runtime/index.ts

@@ -77,14 +77,14 @@ function select_route(url: URL): Target {
 
 let current_token: {};
 
-function render(data: any, changed_from: number, scroll: ScrollPosition, token: {}) {
+function render(data: any, nullable_depth: number, scroll: ScrollPosition, token: {}) {
 	if (current_token !== token) return;
 
 	if (root) {
 		// first, clear out highest-level root component
 		let level = data.child;
-		for (let i = 0; i < changed_from; i += 1) {
-			if (i === changed_from) break;
+		for (let i = 0; i < nullable_depth; i += 1) {
+			if (i === nullable_depth) break;
 			level = level.props.child;
 		}
 
@@ -134,7 +134,7 @@ let root_data: any;
 function prepare_page(target: Target): Promise<{
 	redirect?: Redirect;
 	data?: any;
-	changed_from?: number;
+	nullable_depth?: number;
 }> {
 	if (root) {
 		root.set({ preloading: true });
@@ -179,6 +179,7 @@ function prepare_page(target: Target): Promise<{
 
 	return Promise.all(page.parts.map(async (part, i) => {
 		if (i < changed_from) return null;
+		if (!part) return null;
 
 		const { default: Component } = await part.component();
 		const req = {
@@ -231,33 +232,42 @@ function prepare_page(target: Target): Promise<{
 		const data = {
 			path,
 			preloading: false,
-			child: Object.assign({}, root_props.child)
+			child: Object.assign({}, root_props.child, {
+				segment: segments[0]
+			})
 		};
 		if (changed(query, root_props.query)) data.query = query;
 		if (changed(params, root_props.params)) data.params = params;
 
 		let level = data.child;
+		let nullable_depth = 0;
+
 		for (let i = 0; i < page.parts.length; i += 1) {
 			const part = page.parts[i];
+			if (!part) continue;
+
 			const get_params = part.params || (() => ({}));
 
 			if (i < changed_from) {
 				level.props.path = path;
 				level.props.query = query;
 				level.props.child = Object.assign({}, level.props.child);
+
+				nullable_depth += 1;
 			} else {
-				level.segment = new_segments[i];
 				level.component = results[i].Component;
 				level.props = Object.assign({}, level.props, props, {
 					params: get_params(target.match),
 				}, results[i].preloaded);
+
 				level.props.child = {};
 			}
 
 			level = level.props.child;
+			level.segment = segments[i + 1];
 		}
 
-		return { data, changed_from };
+		return { data, nullable_depth };
 	});
 }
 
@@ -282,12 +292,12 @@ async function navigate(target: Target, id: number): Promise<any> {
 	prefetching = null;
 
 	const token = current_token = {};
-	const { redirect, data, changed_from } = await loaded;
+	const { redirect, data, nullable_depth } = await loaded;
 
 	if (redirect) {
 		await goto(redirect.location, { replaceState: true });
 	} else {
-		render(data, changed_from, scroll_history[id], token);
+		render(data, nullable_depth, scroll_history[id], token);
 		document.activeElement.blur();
 	}
 }
@@ -353,7 +363,7 @@ function handle_popstate(event: PopStateEvent) {
 
 let prefetching: {
 	href: string;
-	promise: Promise<{ redirect?: Redirect, data?: any, changed_from?: number }>;
+	promise: Promise<{ redirect?: Redirect, data?: any, nullable_depth?: number }>;
 } = null;
 
 export function prefetch(href: string) {

test/app/routes/[x]/[y]/_layout.html

@@ -1,6 +1,8 @@
 <span>y: {segment} {count}</span>
 <svelte:component this={child.component} {...child.props}/>
 
+<span>child segment: {child.segment}</span>
+
 <script>
 	import counts from '../_counts.js';
 

test/app/routes/[x]/_layout.html

@@ -1,20 +0,0 @@
-<span>x: {segment} {count}</span>
-<svelte:component this={child.component} {...child.props}/>
-
-<script>
-	import counts from './_counts.js';
-
-	export default {
-		preload() {
-			return {
-				count: counts.x += 1
-			};
-		},
-
-		oncreate() {
-			this.set({
-				segment: this.get().params.x
-			});
-		}
-	};
-</script>

test/common/test.js

@@ -629,9 +629,9 @@ function run({ mode, basepath = '' }) {
 					.evaluate(() => document.querySelector('#sapper').textContent)
 					.then(text => {
 						assert.deepEqual(text.split('\n').filter(Boolean), [
-							'x: foo 1',
 							'y: bar 1',
-							'z: baz 1'
+							'z: baz 1',
+							'child segment: baz'
 						]);
 
 						return nightmare.click(`a`)
@@ -642,9 +642,9 @@ function run({ mode, basepath = '' }) {
 					})
 					.then(text => {
 						assert.deepEqual(text.split('\n').filter(Boolean), [
-							'x: foo 1',
 							'y: bar 1',
-							'z: qux 2'
+							'z: qux 2',
+							'child segment: qux'
 						]);
 					});
 			});

test/unit/create_routes/index.js

@@ -2,13 +2,6 @@ const path = require('path');
 const assert = require('assert');
 const { create_routes } = require('../../../dist/core.ts.js');
 
-
-const _default_layout = {
-	default: true,
-	name: '_default_layout',
-	file: null
-};
-
 describe('create_routes', () => {
 	it('creates routes', () => {
 		const { components, pages, server_routes } = create_routes(path.join(__dirname, 'samples/basic'));
@@ -21,7 +14,6 @@ describe('create_routes', () => {
 		assert.deepEqual(components, [
 			index,
 			about,
-			_default_layout,
 			blog,
 			blog_$slug
 		]);
@@ -44,7 +36,7 @@ describe('create_routes', () => {
 			{
 				pattern: /^\/blog\/?$/,
 				parts: [
-					{ component: _default_layout, params: [] },
+					null,
 					{ component: blog, params: [] }
 				]
 			},
@@ -52,7 +44,7 @@ describe('create_routes', () => {
 			{
 				pattern: /^\/blog\/([^\/]+?)\/?$/,
 				parts: [
-					{ component: _default_layout, params: [] },
+					null,
 					{ component: blog_$slug, params: ['slug'] }
 				]
 			}
@@ -110,15 +102,15 @@ describe('create_routes', () => {
 	it('sorts routes correctly', () => {
 		const { pages } = create_routes(path.join(__dirname, 'samples/sorting'));
 
-		assert.deepEqual(pages.map(p => p.parts.map(part => part.component.file)), [
+		assert.deepEqual(pages.map(p => p.parts.map(part => part && part.component.file)), [
 			['index.html'],
 			['about.html'],
-			[_default_layout.file, 'post/index.html'],
-			[_default_layout.file, 'post/bar.html'],
-			[_default_layout.file, 'post/foo.html'],
-			[_default_layout.file, 'post/f[xx].html'],
-			[_default_layout.file, 'post/[id([0-9-a-z]{3,})].html'],
-			[_default_layout.file, 'post/[id].html'],
+			[null, 'post/index.html'],
+			[null, 'post/bar.html'],
+			[null, 'post/foo.html'],
+			[null, 'post/f[xx].html'],
+			[null, 'post/[id([0-9-a-z]{3,})].html'],
+			[null, 'post/[id].html'],
 			['[wildcard].html']
 		]);
 	});