-> v0.13.4

blur active element on navigation - fixes #287
wait until server restarts before emitting hot reload update
2026-01-15 12:24:47 +00:00 · 2018-06-18 22:45:51 -04:00 · 2018-06-18 22:44:55 -04:00 · 2018-06-18 22:31:10 -04:00 · 2018-06-18 12:22:42 -04:00 · 2018-06-18 12:22:37 -04:00
8 changed files with 113 additions and 28 deletions
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,5 +1,26 @@
 # sapper changelog
 ## 0.13.4
 * Focus `<body>` after navigation ([#287](https://github.com/sveltejs/sapper/issues/287))
 * Fix timing of hot reload updates
 * Emit `fatal` event if server crashes ([#285](https://github.com/sveltejs/sapper/pull/285))
 * Emit `stdout` and `stderr` events on dev watcher ([#285](https://github.com/sveltejs/sapper/pull/285))
 * Always refresh client assets in dev ([#286](https://github.com/sveltejs/sapper/pull/286))
 * Correctly initialise rebuild stats
 ## 0.13.3
 * Make `fatal` events clonable for IPC purposes
 ## 0.13.2
 * Emit a `basepath` event ([#284](https://github.com/sveltejs/sapper/pull/284))
 ## 0.13.1
 * Reinstate ten-second interval between dev server heartbeats ([#276](https://github.com/sveltejs/sapper/issues/276))
 ## 0.13.0
 * Expose `dev`, `build`, `export` and `find_page` APIs ([#272](https://github.com/sveltejs/sapper/issues/272))
--- a/package.json
+++ b/package.json
@@ -1,6 +1,6 @@
 {
  "name": "sapper",
-  "version": "0.13.0",
+  "version": "0.13.4",
  "description": "Military-grade apps, engineered by Svelte",
  "main": "dist/middleware.ts.js",
  "bin": {
--- a/src/api/dev.ts
+++ b/src/api/dev.ts
@@ -6,7 +6,6 @@ import * as ports from 'port-authority';
 import mkdirp from 'mkdirp';
 import rimraf from 'rimraf';
 import format_messages from 'webpack-format-messages';
 import prettyMs from 'pretty-ms';
 import { locations } from '../config';
 import { EventEmitter } from 'events';
 import { create_routes, create_main_manifests, create_compilers, create_serviceworker_manifest } from '../core';
@@ -91,7 +90,7 @@ class Watcher extends EventEmitter {
 		if (this.port) {
 			if (!await ports.check(this.port)) {
 				this.emit('fatal', <events.FatalEvent>{
-					error: new Error(`Port ${this.port} is unavailable`)
+					message: `Port ${this.port} is unavailable`
 				});
 				return;
 			}
@@ -131,6 +130,12 @@ class Watcher extends EventEmitter {
 		// TODO watch the configs themselves?
 		const compilers = create_compilers({ webpack: this.dirs.webpack });
 		const emitFatal = () => {
 			this.emit('fatal', <events.FatalEvent>{
 				message: `Server crashed`
 			});
 		};
 		this.watch(compilers.server, {
 			name: 'server',
@@ -143,10 +148,6 @@ class Watcher extends EventEmitter {
 				fs.writeFileSync(path.join(dest, 'server_info.json'), JSON.stringify(info, null, '  '));
 				this.deferreds.client.promise.then(() => {
 					this.dev_server.send({
 						status: 'completed'
 					});
 					const restart = () => {
 						ports.wait(this.port).then((() => {
 							this.emit('ready', <events.ReadyEvent>{
@@ -155,10 +156,15 @@ class Watcher extends EventEmitter {
 							});
 							this.deferreds.server.fulfil();
 							this.dev_server.send({
 								status: 'completed'
 							});
 						}));
 					};
 					if (this.proc) {
 						this.proc.removeListener('exit', emitFatal);
 						this.proc.kill();
 						this.proc.on('exit', restart);
 					} else {
@@ -172,6 +178,24 @@ class Watcher extends EventEmitter {
 						}, process.env),
 						stdio: ['ipc']
 					});
 					this.proc.stdout.on('data', chunk => {
 						this.emit('stdout', chunk);
 					});
 					this.proc.stderr.on('data', chunk => {
 						this.emit('stderr', chunk);
 					});
 					this.proc.on('message', message => {
 						if (message.__sapper__ && message.event === 'basepath') {
 							this.emit('basepath', {
 								basepath: message.basepath
 							});
 						}
 					});
 					this.proc.on('exit', emitFatal);
 				});
 			}
 		});
@@ -243,8 +267,8 @@ class Watcher extends EventEmitter {
 			this.restarting = true;
 			this.current_build = {
-				changed: new Set(),
+				changed: new Set([filename]),
-				rebuilding: new Set(),
+				rebuilding: new Set([type]),
 				unique_warnings: new Set(),
 				unique_errors: new Set()
 			};
@@ -353,8 +377,11 @@ class Deferred {
 	}
 }
 const INTERVAL = 10000;
 class DevServer {
 	clients: Set<http.ServerResponse>;
 	interval: NodeJS.Timer;
 	_: http.Server;
 	constructor(port: number, interval = 10000) {
@@ -385,13 +412,14 @@ class DevServer {
 		this._.listen(port);
-		setInterval(() => {
+		this.interval = setInterval(() => {
 			this.send(null);
-		});
+		}, INTERVAL);
 	}
 	close() {
 		this._.close();
 		clearInterval(this.interval);
 	}
 	send(data: any) {
--- a/src/api/export.ts
+++ b/src/api/export.ts
@@ -66,7 +66,7 @@ async function execute(emitter: EventEmitter, {
 	const saved = new Set();
 	proc.on('message', message => {
-		if (!message.__sapper__) return;
+		if (!message.__sapper__ || message.event !== 'file') return;
 		let file = new URL(message.url, origin).pathname.slice(1);
 		let { body } = message;
--- a/src/api/interfaces.ts
+++ b/src/api/interfaces.ts
@@ -7,11 +7,11 @@ export type ReadyEvent = {
 export type ErrorEvent = {
 	type: string;
-	error: Error;
+	message: string;
 };
 export type FatalEvent = {
-	error: Error;
+	message: string;
 };
 export type InvalidEvent = {
--- a/src/middleware.ts
+++ b/src/middleware.ts
@@ -1,15 +1,12 @@
 import * as fs from 'fs';
 import * as path from 'path';
-import { resolve, URL } from 'url';
+import { URL } from 'url';
 import { ClientRequest, ServerResponse } from 'http';
 import cookie from 'cookie';
 import mkdirp from 'mkdirp';
 import rimraf from 'rimraf';
 import devalue from 'devalue';
 import fetch from 'node-fetch';
 import { lookup } from './middleware/mime';
 import { locations, dev } from './config';
 import { Route, Template } from './interfaces';
 import sourceMapSupport from 'source-map-support';
 sourceMapSupport.install();
@@ -59,7 +56,7 @@ export default function middleware({ App, routes, store }: {
 	const output = locations.dest();
-	const client_assets = JSON.parse(fs.readFileSync(path.join(output, 'client_assets.json'), 'utf-8'));
+	let emitted_basepath = false;
 	const middleware = compose_handlers([
 		(req: Req, res: ServerResponse, next: () => void) => {
@@ -69,6 +66,16 @@ export default function middleware({ App, routes, store }: {
 					: '';
 			}
 			if (!emitted_basepath && process.send) {
 				process.send({
 					__sapper__: true,
 					event: 'basepath',
 					basepath: req.baseUrl
 				});
 				emitted_basepath = true;
 			}
 			if (req.path === undefined) {
 				req.path = req.url.replace(/\?.*/, '');
 			}
@@ -96,7 +103,7 @@ export default function middleware({ App, routes, store }: {
 			cache_control: 'max-age=31536000'
 		}),
-		get_route_handler(client_assets, App, routes, store)
+		get_route_handler(App, routes, store)
 	].filter(Boolean));
 	return middleware;
@@ -139,9 +146,13 @@ function serve({ prefix, pathname, cache_control }: {
 	};
 }
-const resolved = Promise.resolve();
+function get_route_handler(App: Component, routes: RouteObject[], store_getter: (req: Req) => Store) {
 	const output = locations.dest();
 	const get_chunks = dev()
 		? () => JSON.parse(fs.readFileSync(path.join(output, 'client_assets.json'), 'utf-8'))
 		: (assets => () => assets)(JSON.parse(fs.readFileSync(path.join(output, 'client_assets.json'), 'utf-8')));
 function get_route_handler(chunks: Record<string, string>, App: Component, routes: RouteObject[], store_getter: (req: Req) => Store) {
 	const template = dev()
 		? () => fs.readFileSync(`${locations.app()}/template.html`, 'utf-8')
 		: (str => () => str)(fs.readFileSync(`${locations.dest()}/template.html`, 'utf-8'));
@@ -152,6 +163,8 @@ function get_route_handler(chunks: Record<string, string>, App: Component, route
 		const handlers = route.handlers[Symbol.iterator]();
 		function next() {
 			const chunks: Record<string, string> = get_chunks();
 			try {
 				const { value: handler, done } = handlers.next();
@@ -281,6 +294,7 @@ function get_route_handler(chunks: Record<string, string>, App: Component, route
 						if (process.send) {
 							process.send({
 								__sapper__: true,
 								event: 'file',
 								url: req.url,
 								method: req.method,
 								status: 200,
@@ -320,6 +334,7 @@ function get_route_handler(chunks: Record<string, string>, App: Component, route
 								process.send({
 									__sapper__: true,
 									event: 'file',
 									url: req.url,
 									method: req.method,
 									status: res.statusCode,
@@ -375,7 +390,7 @@ function get_route_handler(chunks: Record<string, string>, App: Component, route
 		function render_page({ head, css, html }) {
 			const page = template()
 				.replace('%sapper.base%', `<base href="${req.baseUrl}/">`)
-				.replace('%sapper.scripts%', `<script>__SAPPER__={baseUrl: "${req.baseUrl}"}</script><script src='${req.baseUrl}/client/${chunks.main}'></script>`)
+				.replace('%sapper.scripts%', `<script>__SAPPER__={baseUrl: "${req.baseUrl}"}</script><script src='${req.baseUrl}/client/${get_chunks().main}'></script>`)
 				.replace('%sapper.html%', html)
 				.replace('%sapper.head%', `<noscript id='sapper-head-start'></noscript>${head}<noscript id='sapper-head-end'></noscript>`)
 				.replace('%sapper.styles%', (css && css.code ? `<style>${css.code}</style>` : ''));
@@ -451,10 +466,6 @@ function compose_handlers(handlers: Handler[]) {
 	};
 }
 function read_json(file: string) {
 	return JSON.parse(fs.readFileSync(file, 'utf-8'));
 }
 function try_serialize(data: any) {
 	try {
 		return devalue(data);
--- a/src/runtime/index.ts
+++ b/src/runtime/index.ts
@@ -161,6 +161,7 @@ function navigate(target: Target, id: number) {
 		}
 		render(Page, props, scroll_history[id], token);
 		document.activeElement.blur();
 	});
 }
--- a/test/common/test.js
+++ b/test/common/test.js
@@ -133,6 +133,7 @@ function run({ mode, basepath = '' }) {
 		let capture;
 		let base;
 		let captured_basepath;
 		const nightmare = new Nightmare();
@@ -179,7 +180,13 @@ function run({ mode, basepath = '' }) {
 				let handler;
 				proc.on('message', message => {
-					if (message.__sapper__) return;
+					if (message.__sapper__) {
 						if (message.event === 'basepath') {
 							captured_basepath = basepath;
 						}
 						return;
 					}
 					if (handler) handler(message);
 				});
@@ -597,6 +604,23 @@ function run({ mode, basepath = '' }) {
 						assert.ok(!hasProgressIndicator);
 					});
 			});
 			it('emits a basepath', () => {
 				assert.equal(captured_basepath, basepath);
 			});
 			// skipped because Nightmare doesn't seem to focus the <a> correctly
 			it.skip('resets the active element after navigation', () => {
 				return nightmare
 					.goto(base)
 					.init()
 					.click('a[href="about"]')
 					.wait(100)
 					.evaluate(() => document.activeElement.nodeName)
 					.then(name => {
 						assert.equal(name, 'BODY');
 					});
 			});
 		});
 		describe('headers', () => {
Author	SHA1	Message	Date
Rich Harris	611dc4f6be	-> v0.13.4	2018-06-18 22:45:51 -04:00
Rich Harris	0b43eaa992	blur active element on navigation - fixes #287	2018-06-18 22:44:55 -04:00
Rich Harris	47cdc1c4c8	wait until server restarts before emitting hot reload update	2018-06-18 22:31:10 -04:00
Rich Harris	31c071ad72	Merge branch 'master' of github.com:sveltejs/sapper	2018-06-18 12:22:42 -04:00
Rich Harris	e91edaee12	Merge pull request #286 from sveltejs/invalidate-client-assets always refresh client_assets in dev	2018-06-18 12:22:37 -04:00
Rich Harris	34c1fee5db	Merge branch 'master' of github.com:sveltejs/sapper	2018-06-17 13:02:13 -04:00
Rich Harris	5375422633	Merge pull request #285 from sveltejs/event-on-exit emit a fatal event if server crashes	2018-06-17 13:02:03 -04:00
Rich Harris	1dafe934b0	initialise rebuild stats	2018-06-17 13:01:51 -04:00
Rich Harris	e1a33c6a9b	always refresh client_assets in dev	2018-06-17 13:00:27 -04:00
Rich Harris	0800fa016b	emit a fatal event if server crashes	2018-06-16 20:18:32 -04:00
Rich Harris	8f3454c3b1	-> v0.13.3	2018-06-16 13:49:44 -04:00
Rich Harris	f0d7a1aaab	change fatal events to be clonable, for IPC purposes	2018-06-16 13:49:19 -04:00
Rich Harris	8240595d70	-> v0.13.2	2018-06-15 10:58:55 -04:00
Rich Harris	658d8dd50c	Merge pull request #284 from sveltejs/emit-basepath Emit basepath event	2018-06-15 10:57:04 -04:00
Rich Harris	9eeeaa24c1	emit a basepath event on first run	2018-06-14 17:20:46 -04:00
Rich Harris	9c4a3592ff	remove some unused code	2018-06-14 16:34:16 -04:00
Rich Harris	be7cff4818	-> v0.13.1	2018-06-05 14:41:12 -04:00
Rich Harris	d6632cf312	Merge pull request #281 from sveltejs/gh-276 reinstate ten second interval between heartbeats	2018-06-05 20:40:17 +02:00
Rich Harris	f6e012ec73	reinstate ten second interval between heartbeats - fixes #276	2018-06-05 14:35:37 -04:00