Compare commits

..

35 Commits

Author SHA1 Message Date
Rich Harris
2b799a3f1e print warnings in dev mode for unused data 2018-08-03 01:14:26 -04:00
Rich Harris
18d15c0120 emit events for unused data, and only in dev/export mode 2018-08-03 01:14:08 -04:00
Rich Harris
b20e15721c Merge branch 'master' into proxy-data 2018-08-03 00:15:26 -04:00
Rich Harris
de4f99807f Merge branch 'master' of github.com:sveltejs/sapper 2018-08-03 00:11:09 -04:00
Rich Harris
eae8351f77 Better/faster exporting
* add --build and --build-dir options to sapper export (#325)

* tweak export logging, update port-authority to prevent timeout bug

* better logging of export progress

* handle case where linked resource is already fetched

* default to .sapper/dev instead of .sapper

* handle query params and redirects

* dont write server_info.json either - second half of #318

* update changelog

* update lockfile

* try to track down ci test failures

* err wut

* curiouser and curiouser

* ok, seems to work now
2018-08-03 00:10:58 -04:00
Rich Harris
d386308301 update changelog 2018-08-02 13:23:12 -04:00
Rich Harris
13afbc84d7 dont write server_info.json either - second half of #318 2018-08-02 10:59:23 -04:00
Rich Harris
31327b3780 Merge pull request #333 from sveltejs/gh-332
only blur activeElement if there is one
2018-08-02 10:43:27 -04:00
Rich Harris
81f483d7b8 Merge pull request #334 from sveltejs/gh-318
dont emit client_info.json
2018-08-02 08:40:30 -04:00
Rich Harris
1bcf20511b dont emit client_info.json - fixes #318 2018-08-01 21:46:26 -04:00
Rich Harris
003fa8ab2c only blur activeElement if there is one - fixes #332 2018-08-01 21:34:30 -04:00
Rich Harris
d1fcd07c92 -> v0.15.4 2018-08-01 08:47:41 -04:00
Rich Harris
47a6d6f662 Merge pull request #326 from lukeed/feat/ignore
Add `ignore` option
2018-08-01 08:46:49 -04:00
Rich Harris
06cc22b10d detect unused data on initial render 2018-07-31 23:21:34 -04:00
Luke Edwards
4b2b6440d0 fix: use more specific ignore pattern;
~> leaked into another test’s route
2018-07-31 16:54:29 -07:00
Rich Harris
fc855f30f8 -> v0.15.3 2018-07-31 15:06:47 -04:00
Rich Harris
4a75fff4ec Merge pull request #329 from sveltejs/parallel-exports
parallelize exports
2018-07-31 15:05:38 -04:00
Rich Harris
7b7b695938 Merge pull request #320 from sveltejs/gh-319
Some dev documentation
2018-07-31 14:59:57 -04:00
Rich Harris
2fca2e295f Merge pull request #328 from sveltejs/export-no-minify-js
don't minify JS when minifying HTML
2018-07-31 14:59:27 -04:00
Rich Harris
eae991d369 parallelize exports 2018-07-31 14:56:29 -04:00
Rich Harris
c2b393d3fd dont minify JS when minifying HTML 2018-07-31 14:43:58 -04:00
Luke Edwards
566addd406 add tests for opts.ignore 2018-07-29 14:17:13 -07:00
Luke Edwards
3d77dacbd6 attach ignore options to test app, w/ matching routes 2018-07-29 14:17:04 -07:00
Luke Edwards
51b4f9cbbf add opts.ignore support 2018-07-29 14:01:44 -07:00
Robert Hall
1d611be83e Using npm 2018-07-25 15:50:56 -06:00
Robert Hall
1782904994 Some dev documentation 2018-07-25 10:10:22 -06:00
Rich Harris
e3ddbfc181 Merge pull request #314 from sveltejs/fix-child-segment
fix child.segment bug
2018-07-23 17:08:25 -04:00
Rich Harris
8e3830b646 fix child.segment bug 2018-07-23 17:02:35 -04:00
Rich Harris
b28cdff233 -> v0.15.2 2018-07-23 16:38:49 -04:00
Rich Harris
7f586ff1a3 Merge pull request #313 from sveltejs/gh-312
Skip layout components where none is provided
2018-07-23 16:37:30 -04:00
Rich Harris
731d4f535c skip layout components where none is provided - fixes #312 2018-07-23 16:31:00 -04:00
Rich Harris
f8c731ca21 failing tests for #312 2018-07-23 14:31:11 -04:00
Rich Harris
39eb3be01e -> v0.15.1 2018-07-22 21:25:33 -04:00
Rich Harris
d0bb728e25 -> v0.15.0 2018-07-22 21:04:03 -04:00
Rich Harris
58de0f9c99 Nested routes
Fixes #262
2018-07-22 21:00:37 -04:00
41 changed files with 8026 additions and 405 deletions

View File

@@ -1,5 +1,32 @@
# sapper changelog # sapper changelog
## 0.15.5
* Faster `export` with more explanatory output ([#335](https://github.com/sveltejs/sapper/pull/335))
* Only blur `activeElement` if it exists ([#332](https://github.com/sveltejs/sapper/issues/332))
* Don't emit `client_info.json` or `server_info.json` ([#318](https://github.com/sveltejs/sapper/issues/318))
## 0.15.4
* Add `ignore` option ([#326](https://github.com/sveltejs/sapper/pull/326))
## 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
## 0.15.0
* Nested routes (consult [migration guide](https://sapper.svelte.technology/guide#0-14-to-0-15) and docs on [layouts](https://sapper.svelte.technology/guide#layouts)) ([#262](https://github.com/sveltejs/sapper/issues/262))
## 0.14.2 ## 0.14.2
* Prevent unsafe replacements ([#307](https://github.com/sveltejs/sapper/pull/307)) * Prevent unsafe replacements ([#307](https://github.com/sveltejs/sapper/pull/307))

View File

@@ -31,6 +31,44 @@ npm run build
npm start 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 ## License

7172
package-lock.json generated Normal file

File diff suppressed because it is too large Load Diff

View File

@@ -1,6 +1,6 @@
{ {
"name": "sapper", "name": "sapper",
"version": "0.14.2", "version": "0.15.4",
"description": "Military-grade apps, engineered by Svelte", "description": "Military-grade apps, engineered by Svelte",
"main": "dist/middleware.ts.js", "main": "dist/middleware.ts.js",
"bin": { "bin": {
@@ -12,6 +12,7 @@
"runtime", "runtime",
"webpack", "webpack",
"sapper", "sapper",
"components",
"dist" "dist"
], ],
"directories": { "directories": {
@@ -22,12 +23,12 @@
"cheerio": "^1.0.0-rc.2", "cheerio": "^1.0.0-rc.2",
"chokidar": "^2.0.3", "chokidar": "^2.0.3",
"cookie": "^0.3.1", "cookie": "^0.3.1",
"devalue": "^1.0.1", "devalue": "^1.0.4",
"glob": "^7.1.2", "glob": "^7.1.2",
"html-minifier": "^3.5.16", "html-minifier": "^3.5.16",
"mkdirp": "^0.5.1", "mkdirp": "^0.5.1",
"node-fetch": "^2.1.1", "node-fetch": "^2.1.1",
"port-authority": "^1.0.2", "port-authority": "^1.0.5",
"pretty-bytes": "^5.0.0", "pretty-bytes": "^5.0.0",
"pretty-ms": "^3.1.0", "pretty-ms": "^3.1.0",
"require-relative": "^0.8.7", "require-relative": "^0.8.7",
@@ -67,7 +68,7 @@
"cy:open": "cypress open", "cy:open": "cypress open",
"test": "mocha --opts mocha.opts", "test": "mocha --opts mocha.opts",
"pretest": "npm run build", "pretest": "npm run build",
"build": "rollup -c", "build": "rm -rf dist && rollup -c",
"dev": "rollup -cw", "dev": "rollup -cw",
"prepublishOnly": "npm test", "prepublishOnly": "npm test",
"update_mime_types": "curl http://svn.apache.org/repos/asf/httpd/httpd/trunk/docs/conf/mime.types | grep -e \"^[^#]\" > src/middleware/mime-types.md" "update_mime_types": "curl http://svn.apache.org/repos/asf/httpd/httpd/trunk/docs/conf/mime.types | grep -e \"^[^#]\" > src/middleware/mime-types.md"

View File

@@ -19,7 +19,8 @@ export default [
}, },
plugins: [ plugins: [
typescript({ typescript({
typescript: require('typescript') typescript: require('typescript'),
target: "ES2017"
}) })
] ]
}, },

View File

@@ -4,8 +4,7 @@ import mkdirp from 'mkdirp';
import rimraf from 'rimraf'; import rimraf from 'rimraf';
import { EventEmitter } from 'events'; import { EventEmitter } from 'events';
import { minify_html } from './utils/minify_html'; import { minify_html } from './utils/minify_html';
import { create_compilers, create_main_manifests, create_routes, create_serviceworker_manifest } from '../core' import { create_compilers, create_main_manifests, create_routes, create_serviceworker_manifest } from '../core';
import { locations } from '../config';
import * as events from './interfaces'; import * as events from './interfaces';
export function build(opts: {}) { export function build(opts: {}) {
@@ -62,7 +61,6 @@ async function execute(emitter: EventEmitter, {
}); });
const client_info = client_stats.toJson(); const client_info = client_stats.toJson();
fs.writeFileSync(path.join(dest, 'client_info.json'), JSON.stringify(client_info));
fs.writeFileSync(path.join(dest, 'client_assets.json'), JSON.stringify(client_info.assetsByChunkName)); fs.writeFileSync(path.join(dest, 'client_assets.json'), JSON.stringify(client_info.assetsByChunkName));
const server_stats = await compile(server); const server_stats = await compile(server);

View File

@@ -9,6 +9,7 @@ import format_messages from 'webpack-format-messages';
import { locations } from '../config'; import { locations } from '../config';
import { EventEmitter } from 'events'; import { EventEmitter } from 'events';
import { create_routes, create_main_manifests, create_compilers, create_serviceworker_manifest } from '../core'; import { create_routes, create_main_manifests, create_compilers, create_serviceworker_manifest } from '../core';
import Deferred from './utils/Deferred';
import * as events from './interfaces'; import * as events from './interfaces';
export function dev(opts) { export function dev(opts) {
@@ -168,8 +169,6 @@ class Watcher extends EventEmitter {
}, },
result: info => { result: info => {
fs.writeFileSync(path.join(dest, 'server_info.json'), JSON.stringify(info, null, ' '));
this.deferreds.client.promise.then(() => { this.deferreds.client.promise.then(() => {
const restart = () => { const restart = () => {
log = ''; log = '';
@@ -225,11 +224,8 @@ class Watcher extends EventEmitter {
}); });
this.proc.on('message', message => { this.proc.on('message', message => {
if (message.__sapper__ && message.event === 'basepath') { if (!message.__sapper__) return;
this.emit('basepath', { this.emit(message.event, message);
basepath: message.basepath
});
}
}); });
this.proc.on('exit', emitFatal); this.proc.on('exit', emitFatal);
@@ -252,7 +248,6 @@ class Watcher extends EventEmitter {
}, },
result: info => { result: info => {
fs.writeFileSync(path.join(dest, 'client_info.json'), JSON.stringify(info));
fs.writeFileSync(path.join(dest, 'client_assets.json'), JSON.stringify(info.assetsByChunkName, null, ' ')); fs.writeFileSync(path.join(dest, 'client_assets.json'), JSON.stringify(info.assetsByChunkName, null, ' '));
this.deferreds.client.fulfil(); this.deferreds.client.fulfil();
@@ -401,19 +396,6 @@ function mungeWebpackError(message: string, duplicate: boolean) {
}; };
} }
class Deferred {
promise: Promise<any>;
fulfil: (value?: any) => void;
reject: (error: Error) => void;
constructor() {
this.promise = new Promise((fulfil, reject) => {
this.fulfil = fulfil;
this.reject = reject;
});
}
}
const INTERVAL = 10000; const INTERVAL = 10000;
class DevServer { class DevServer {

View File

@@ -7,7 +7,7 @@ import fetch from 'node-fetch';
import * as ports from 'port-authority'; import * as ports from 'port-authority';
import { EventEmitter } from 'events'; import { EventEmitter } from 'events';
import { minify_html } from './utils/minify_html'; import { minify_html } from './utils/minify_html';
import { locations } from '../config'; import Deferred from './utils/Deferred';
import * as events from './interfaces'; import * as events from './interfaces';
export function exporter(opts: {}) { export function exporter(opts: {}) {
@@ -52,6 +52,10 @@ async function execute(emitter: EventEmitter, {
const origin = `http://localhost:${port}`; const origin = `http://localhost:${port}`;
emitter.emit('info', {
message: `Crawling ${origin}`
});
const proc = child_process.fork(path.resolve(`${build}/server.js`), [], { const proc = child_process.fork(path.resolve(`${build}/server.js`), [], {
cwd: process.cwd(), cwd: process.cwd(),
env: Object.assign({ env: Object.assign({
@@ -64,11 +68,21 @@ async function execute(emitter: EventEmitter, {
const seen = new Set(); const seen = new Set();
const saved = new Set(); const saved = new Set();
const deferreds = new Map();
function get_deferred(pathname: string) {
if (!deferreds.has(pathname)) {
deferreds.set(pathname, new Deferred()) ;
}
return deferreds.get(pathname);
}
proc.on('message', message => { proc.on('message', message => {
if (!message.__sapper__ || message.event !== 'file') return; if (!message.__sapper__ || message.event !== 'file') return;
let file = new URL(message.url, origin).pathname.slice(1); const pathname = new URL(message.url, origin).pathname;
let file = pathname.slice(1);
let { body } = message; let { body } = message;
if (saved.has(file)) return; if (saved.has(file)) return;
@@ -83,24 +97,26 @@ async function execute(emitter: EventEmitter, {
emitter.emit('file', <events.FileEvent>{ emitter.emit('file', <events.FileEvent>{
file, file,
size: body.length size: body.length,
status: message.status
}); });
sander.writeFileSync(export_dir, file, body); sander.writeFileSync(export_dir, file, body);
get_deferred(pathname).fulfil();
}); });
async function handle(url: URL) { async function handle(url: URL) {
const pathname = url.pathname || '/';
if (seen.has(pathname)) return;
seen.add(pathname);
const deferred = get_deferred(pathname);
const r = await fetch(url.href); const r = await fetch(url.href);
const range = ~~(r.status / 100); const range = ~~(r.status / 100);
if (range >= 4) {
emitter.emit('failure', <events.FailureEvent>{
status: r.status,
pathname: url.pathname
});
return;
}
if (range === 2) { if (range === 2) {
if (r.headers.get('Content-Type') === 'text/html') { if (r.headers.get('Content-Type') === 'text/html') {
const body = await r.text(); const body = await r.text();
@@ -111,18 +127,14 @@ async function execute(emitter: EventEmitter, {
$('a[href]').each((i: number, $a) => { $('a[href]').each((i: number, $a) => {
const url = new URL($a.attribs.href, base.href); const url = new URL($a.attribs.href, base.href);
if (url.origin === origin) urls.push(url);
if (url.origin === origin && !seen.has(url.pathname)) {
seen.add(url.pathname);
urls.push(url);
}
}); });
for (const url of urls) { await Promise.all(urls.map(handle));
await handle(url);
}
} }
} }
await deferred.promise;
} }
return ports.wait(port) return ports.wait(port)

12
src/api/utils/Deferred.ts Normal file
View File

@@ -0,0 +1,12 @@
export default class Deferred {
promise: Promise<any>;
fulfil: (value?: any) => void;
reject: (error: Error) => void;
constructor() {
this.promise = new Promise((fulfil, reject) => {
this.fulfil = fulfil;
this.reject = reject;
});
}
}

View File

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

View File

@@ -1,11 +1,8 @@
import * as fs from 'fs'; import * as fs from 'fs';
import * as path from 'path'; import * as path from 'path';
import * as child_process from 'child_process';
import sade from 'sade'; import sade from 'sade';
import * as colors from 'ansi-colors'; import * as colors from 'ansi-colors';
import prettyMs from 'pretty-ms'; import prettyMs from 'pretty-ms';
// import upgrade from './cli/upgrade';
import * as ports from 'port-authority';
import * as pkg from '../package.json'; import * as pkg from '../package.json';
const prog = sade('sapper').version(pkg.version); const prog = sade('sapper').version(pkg.version);
@@ -65,19 +62,22 @@ prog.command('start [dir]')
prog.command('export [dest]') prog.command('export [dest]')
.describe('Export your app as static files (if possible)') .describe('Export your app as static files (if possible)')
.option('--build', '(Re)build app before exporting', true)
.option('--build-dir', 'Specify a custom temporary build directory', '.sapper/prod')
.option('--basepath', 'Specify a base path') .option('--basepath', 'Specify a base path')
.action(async (dest = 'export', opts: { basepath?: string }) => { .action(async (dest = 'export', opts: { build: boolean, 'build-dir': string, basepath?: string }) => {
console.log(`> Building...`);
process.env.NODE_ENV = 'production'; process.env.NODE_ENV = 'production';
process.env.SAPPER_DEST = '.sapper/.export'; process.env.SAPPER_DEST = opts['build-dir'];
const start = Date.now(); const start = Date.now();
try { try {
if (opts.build) {
console.log(`> Building...`);
const { build } = await import('./cli/build'); const { build } = await import('./cli/build');
await build(); await build();
console.error(`\n> Built in ${elapsed(start)}. Crawling site...`); console.error(`\n> Built in ${elapsed(start)}`);
}
const { exporter } = await import('./cli/export'); const { exporter } = await import('./cli/export');
await exporter(dest, opts); await exporter(dest, opts);

View File

@@ -2,6 +2,7 @@ import * as path from 'path';
import * as colors from 'ansi-colors'; import * as colors from 'ansi-colors';
import * as child_process from 'child_process'; import * as child_process from 'child_process';
import prettyMs from 'pretty-ms'; import prettyMs from 'pretty-ms';
import pb from 'pretty-bytes';
import { dev as _dev } from '../api/dev'; import { dev as _dev } from '../api/dev';
import * as events from '../api/interfaces'; import * as events from '../api/interfaces';
@@ -44,6 +45,32 @@ export function dev(opts: { port: number, open: boolean }) {
if (event.log) console.log(event.log); if (event.log) console.log(event.log);
}); });
watcher.on('preload', (event) => {
if (event.size > 25000) {
console.log(colors.bold.yellow(`${event.url} — large amount of preloaded data`));
console.log(`${colors.bold(pb(event.size))} of data was preloaded in total, above the recommended limit of ${pb(25000)}`);
}
});
watcher.on('unused_data', (event) => {
console.log(colors.bold.yellow(`${event.url} — unused preloaded data`));
console.log(`More data was returned from \`preload\` than was used during the initial render. Consider only returning essential data.`);
event.discrepancies.forEach(discrepancy => {
console.log(`${colors.bold(discrepancy.file)} loaded ${colors.bold(pb(discrepancy.preloaded))}, of which ${discrepancy.rendered > 2 ? `only ${colors.bold(pb(discrepancy.rendered))}` : 'none'} was used. The following properties were not referenced:`);
const slice = discrepancy.props.length > 12
? discrepancy.props.slice(0, 10)
: discrepancy.props;
console.log(slice.map((prop: string) => `${prop}`).join('\n'));
if (discrepancy.props.length > slice.length) {
console.log(`...and ${discrepancy.props.length - slice.length} more`);
}
});
});
watcher.on('build', (event: events.BuildEvent) => { watcher.on('build', (event: events.BuildEvent) => {
if (event.errors.length) { if (event.errors.length) {
console.log(`${colors.bold.red(`${event.type}`)}`); console.log(`${colors.bold.red(`${event.type}`)}`);

View File

@@ -3,6 +3,11 @@ import * as colors from 'ansi-colors';
import prettyBytes from 'pretty-bytes'; import prettyBytes from 'pretty-bytes';
import { locations } from '../config'; import { locations } from '../config';
function left_pad(str: string, len: number) {
while (str.length < len) str = ` ${str}`;
return str;
}
export function exporter(export_dir: string, { basepath = '' }) { export function exporter(export_dir: string, { basepath = '' }) {
return new Promise((fulfil, reject) => { return new Promise((fulfil, reject) => {
try { try {
@@ -13,11 +18,19 @@ export function exporter(export_dir: string, { basepath = '' }) {
}); });
emitter.on('file', event => { emitter.on('file', event => {
console.log(`${colors.bold.cyan(event.file)} ${colors.gray(`(${prettyBytes(event.size)})`)}`); const pb = prettyBytes(event.size);
const size_color = event.size > 150000 ? colors.bold.red : event.size > 50000 ? colors.bold.yellow : colors.bold.gray;
const size_label = size_color(left_pad(prettyBytes(event.size), 10));
const file_label = event.status === 200
? event.file
: colors.bold[event.status >= 400 ? 'red' : 'yellow'](`(${event.status}) ${event.file}`);
console.log(`${size_label} ${file_label}`);
}); });
emitter.on('failure', event => { emitter.on('info', event => {
console.log(`${colors.red(`> Received ${event.status} response when fetching ${event.pathname}`)}`); console.log(colors.bold.cyan(`> ${event.message}`));
}); });
emitter.on('error', event => { emitter.on('error', event => {

View File

@@ -6,5 +6,5 @@ export const locations = {
base: () => path.resolve(process.env.SAPPER_BASE || ''), base: () => path.resolve(process.env.SAPPER_BASE || ''),
app: () => path.resolve(process.env.SAPPER_BASE || '', process.env.SAPPER_APP || 'app'), app: () => path.resolve(process.env.SAPPER_BASE || '', process.env.SAPPER_APP || 'app'),
routes: () => path.resolve(process.env.SAPPER_BASE || '', process.env.SAPPER_ROUTES || 'routes'), routes: () => path.resolve(process.env.SAPPER_BASE || '', process.env.SAPPER_ROUTES || 'routes'),
dest: () => path.resolve(process.env.SAPPER_BASE || '', process.env.SAPPER_DEST || '.sapper') dest: () => path.resolve(process.env.SAPPER_BASE || '', process.env.SAPPER_DEST || `.sapper/${dev() ? 'dev' : 'prod'}`)
}; };

View File

@@ -14,6 +14,10 @@ export function create_main_manifests({ routes, dev_port }: {
const client_manifest = generate_client(routes, path_to_routes, dev_port); const client_manifest = generate_client(routes, path_to_routes, dev_port);
const server_manifest = generate_server(routes, path_to_routes); const server_manifest = generate_server(routes, path_to_routes);
write_if_changed(
`${locations.app()}/manifest/default-layout.html`,
`<svelte:component this={child.component} {...child.props}/>`
);
write_if_changed(`${locations.app()}/manifest/client.js`, client_manifest); write_if_changed(`${locations.app()}/manifest/client.js`, client_manifest);
write_if_changed(`${locations.app()}/manifest/server.js`, server_manifest); write_if_changed(`${locations.app()}/manifest/server.js`, server_manifest);
} }
@@ -44,7 +48,7 @@ function right_pad(str: string, len: number) {
} }
function generate_client( function generate_client(
routes: { components: PageComponent[], pages: Page[], server_routes: ServerRoute[] }, routes: { root: PageComponent, components: PageComponent[], pages: Page[], server_routes: ServerRoute[] },
path_to_routes: string, path_to_routes: string,
dev_port?: number dev_port?: number
) { ) {
@@ -58,15 +62,15 @@ function generate_client(
let code = ` let code = `
// This file is generated by Sapper — do not edit it! // This file is generated by Sapper — do not edit it!
import root from '${posixify(`${path_to_routes}/index.html`)}'; import root from '${get_file(path_to_routes, routes.root)}';
import error from '${posixify(`${path_to_routes}/_error.html`)}'; import error from '${posixify(`${path_to_routes}/_error.html`)}';
${routes.components.map(component => ${routes.components.map(component =>
`const ${component.name} = () => `const ${component.name} = () =>
import(/* webpackChunkName: "${component.name}" */ '${posixify(`${path_to_routes}/${component.file}`)}');`) import(/* webpackChunkName: "${component.name}" */ '${get_file(path_to_routes, component)}');`)
.join('\n')} .join('\n')}
export const routes = { export const manifest = {
ignore: [${server_routes_to_ignore.map(route => route.pattern).join(', ')}], ignore: [${server_routes_to_ignore.map(route => route.pattern).join(', ')}],
pages: [ pages: [
@@ -75,6 +79,8 @@ function generate_client(
pattern: ${page.pattern}, pattern: ${page.pattern},
parts: [ parts: [
${page.parts.map(part => { ${page.parts.map(part => {
if (part === null) return 'null';
if (part.params.length > 0) { if (part.params.length > 0) {
const props = part.params.map((param, i) => `${param}: match[${i + 1}]`); const props = part.params.map((param, i) => `${param}: match[${i + 1}]`);
return `{ component: ${part.component.name}, params: match => ({ ${props.join(', ')} }) }`; return `{ component: ${part.component.name}, params: match => ({ ${props.join(', ')} }) }`;
@@ -89,7 +95,10 @@ function generate_client(
root, root,
error error
};`.replace(/^\t\t/gm, '').trim(); };
// this is included for legacy reasons
export const routes = {};`.replace(/^\t\t/gm, '').trim();
if (dev()) { if (dev()) {
const sapper_dev_client = posixify( const sapper_dev_client = posixify(
@@ -109,15 +118,15 @@ function generate_client(
} }
function generate_server( function generate_server(
routes: { components: PageComponent[], pages: Page[], server_routes: ServerRoute[] }, routes: { root: PageComponent, components: PageComponent[], pages: Page[], server_routes: ServerRoute[] },
path_to_routes: string path_to_routes: string
) { ) {
const imports = [].concat( const imports = [].concat(
routes.server_routes.map(route => routes.server_routes.map(route =>
`import * as ${route.name} from '${posixify(`${path_to_routes}/${route.file}`)}';`), `import * as ${route.name} from '${posixify(`${path_to_routes}/${route.file}`)}';`),
routes.components.map(component => routes.components.map(component =>
`import ${component.name} from '${posixify(`${path_to_routes}/${component.file}`)}';`), `import ${component.name} from '${get_file(path_to_routes, component)}';`),
`import root from '${posixify(`${path_to_routes}/index.html`)}';`, `import root from '${get_file(path_to_routes, routes.root)}';`,
`import error from '${posixify(`${path_to_routes}/_error.html`)}';` `import error from '${posixify(`${path_to_routes}/_error.html`)}';`
); );
@@ -125,7 +134,7 @@ function generate_server(
// This file is generated by Sapper — do not edit it! // This file is generated by Sapper — do not edit it!
${imports.join('\n')} ${imports.join('\n')}
export const routes = { export const manifest = {
server_routes: [ server_routes: [
${routes.server_routes.map(route => `{ ${routes.server_routes.map(route => `{
// ${route.file} // ${route.file}
@@ -143,8 +152,11 @@ function generate_server(
pattern: ${page.pattern}, pattern: ${page.pattern},
parts: [ parts: [
${page.parts.map(part => { ${page.parts.map(part => {
if (part === null) return 'null';
const props = [ const props = [
`name: "${part.component.name}"`, `name: "${part.component.name}"`,
`file: "${part.component.file}"`,
`component: ${part.component.name}` `component: ${part.component.name}`
]; ];
@@ -162,7 +174,18 @@ function generate_server(
root, root,
error error
};`.replace(/^\t\t/gm, '').trim(); };
// this is included for legacy reasons
export const routes = {};`.replace(/^\t\t/gm, '').trim();
return code; return code;
} }
function get_file(path_to_routes: string, component: PageComponent) {
if (component.default) {
return `./default-layout.html`;
}
return posixify(`${path_to_routes}/${component.file}`);
}

View File

@@ -4,11 +4,22 @@ import { locations } from '../config';
import { Page, PageComponent, ServerRoute } from '../interfaces'; import { Page, PageComponent, ServerRoute } from '../interfaces';
import { posixify } from './utils'; import { posixify } from './utils';
const default_layout_file = posixify(path.resolve(
__dirname,
'../components/default-layout.html'
));
export default function create_routes(cwd = locations.routes()) { export default function create_routes(cwd = locations.routes()) {
const components: PageComponent[] = []; const components: PageComponent[] = [];
const pages: Page[] = []; const pages: Page[] = [];
const server_routes: ServerRoute[] = []; const server_routes: ServerRoute[] = [];
const default_layout: PageComponent = {
default: true,
name: '_default_layout',
file: null
};
function walk( function walk(
dir: string, dir: string,
parent_segments: Part[][], parent_segments: Part[][],
@@ -54,9 +65,7 @@ export default function create_routes(cwd = locations.routes()) {
.sort(comparator); .sort(comparator);
items.forEach(item => { items.forEach(item => {
if (item.basename[0] === '_') { if (item.basename[0] === '_') return;
if (item.basename !== (item.is_dir ? '_default' : '_default.html')) return;
}
if (item.basename[0] === '.') { if (item.basename[0] === '.') {
if (item.file !== '.well-known') return; if (item.file !== '.well-known') return;
@@ -91,58 +100,28 @@ export default function create_routes(cwd = locations.routes()) {
params.push(...item.parts.filter(p => p.dynamic).map(p => p.content)); params.push(...item.parts.filter(p => p.dynamic).map(p => p.content));
if (item.is_dir) { if (item.is_dir) {
const index = path.join(dir, item.basename, 'index.html'); const index = path.join(dir, item.basename, '_layout.html');
const component = fs.existsSync(index)
? {
name: `page_${get_slug(item.file)}`,
file: `${item.file}/index.html`
}
: null;
if (component) { const component = fs.existsSync(index) && {
components.push(component); name: `${get_slug(item.file)}__layout`,
} file: `${item.file}/_layout.html`
};
if (component) components.push(component);
walk( walk(
path.join(dir, item.basename), path.join(dir, item.basename),
segments, segments,
params, params,
stack.concat({ component
component: component || { ? stack.concat({ component, params })
missing: true, : stack.concat(null)
name: null,
file: path.join(item.file, 'index.html')
},
params
})
); );
} }
else if (item.basename === 'index.html') {
const is_branch = items.some(other_item => {
if (other_item === item) return false;
if (other_item.basename[0] === '_') {
return other_item.basename === (other_item.is_dir ? '_default' : '_default.html');
}
if (other_item.is_dir) {
return fs.existsSync(path.join(dir, other_item.basename, 'index.html'));
}
return other_item.is_page;
});
if (!is_branch) {
pages.push({
pattern: get_pattern(parent_segments),
parts: stack
});
}
}
else if (item.is_page) { else if (item.is_page) {
const component = { const component = {
name: `page_${get_slug(item.file)}`, name: get_slug(item.file),
file: item.file file: item.file
}; };
@@ -152,7 +131,7 @@ export default function create_routes(cwd = locations.routes()) {
}); });
components.push(component); components.push(component);
if (item.basename === '_default.html') { if (item.basename === 'index.html') {
pages.push({ pages.push({
pattern: get_pattern(parent_segments), pattern: get_pattern(parent_segments),
parts parts
@@ -176,21 +155,19 @@ export default function create_routes(cwd = locations.routes()) {
}); });
} }
const root_file = path.join(cwd, '_layout.html');
const root = fs.existsSync(root_file)
? {
name: 'main',
file: '_layout.html'
}
: default_layout;
walk(cwd, [], [], []); walk(cwd, [], [], []);
// check for clashes
const seen_pages: Map<string, Page> = new Map(); const seen_pages: Map<string, Page> = new Map();
pages.forEach(page => { pages.forEach(page => {
// check for missing intermediate index.html files
let i = page.parts.length;
const last_part = page.parts[i - 1];
while (i--) {
const part = page.parts[i];
if (part.component.missing) {
throw new Error(`Missing ${part.component.file}, which is required for ${last_part.component.file} to be valid`);
}
}
// check for clashes
const pattern = page.pattern.toString(); const pattern = page.pattern.toString();
if (seen_pages.has(pattern)) { if (seen_pages.has(pattern)) {
const file = page.parts.pop().component.file; const file = page.parts.pop().component.file;
@@ -215,6 +192,7 @@ export default function create_routes(cwd = locations.routes()) {
}); });
return { return {
root,
components, components,
pages, pages,
server_routes server_routes
@@ -228,9 +206,11 @@ type Part = {
}; };
function comparator( function comparator(
a: { basename: string, parts: Part[], file: string, is_dir: boolean }, a: { basename: string, parts: Part[], file: string, is_index: boolean },
b: { basename: string, parts: Part[], file: string, is_dir: boolean } b: { basename: string, parts: Part[], file: string, is_index: boolean }
) { ) {
if (a.is_index !== b.is_index) return a.is_index ? -1 : 1;
const max = Math.max(a.parts.length, b.parts.length); const max = Math.max(a.parts.length, b.parts.length);
for (let i = 0; i < max; i += 1) { for (let i = 0; i < max; i += 1) {

View File

@@ -21,7 +21,7 @@ export type Store = {
}; };
export type PageComponent = { export type PageComponent = {
missing?: boolean; default?: boolean;
name: string; name: string;
file: string; file: string;
}; };

View File

@@ -8,6 +8,9 @@ import fetch from 'node-fetch';
import { lookup } from './middleware/mime'; import { lookup } from './middleware/mime';
import { locations, dev } from './config'; import { locations, dev } from './config';
import sourceMapSupport from 'source-map-support'; import sourceMapSupport from 'source-map-support';
import prettyBytes from 'pretty-bytes';
import { wrap_data } from './middleware/wrap_data';
import { list_unused_properties } from './middleware/list_unused_properties';
sourceMapSupport.install(); sourceMapSupport.install();
@@ -26,7 +29,7 @@ type Page = {
}> }>
}; };
type RouteObject = { type Manifest = {
server_routes: ServerRoute[]; server_routes: ServerRoute[];
pages: Page[]; pages: Page[];
root: Component; root: Component;
@@ -39,6 +42,20 @@ type Store = {
get: () => any get: () => any
}; };
type Props = {
path: string;
query: Record<string, string>;
params: Record<string, string>;
error?: { message: string };
status?: number;
child: {
segment: string;
component: Component;
props: Props;
};
[key: string]: any;
};
interface Req extends ClientRequest { interface Req extends ClientRequest {
url: string; url: string;
baseUrl: string; baseUrl: string;
@@ -59,16 +76,39 @@ interface Component {
preload: (data: any) => any | Promise<any> preload: (data: any) => any | Promise<any>
} }
export default function middleware({ routes, store }: { const IGNORE = '__SAPPER__IGNORE__';
routes: RouteObject, function toIgnore(uri: string, val: any) {
store: (req: Req) => Store if (Array.isArray(val)) return val.some(x => toIgnore(uri, x));
if (val instanceof RegExp) return val.test(uri);
if (typeof val === 'function') return val(uri);
return uri.startsWith(val.charCodeAt(0) === 47 ? val : `/${val}`);
}
export default function middleware(opts: {
manifest: Manifest,
store: (req: Req) => Store,
ignore?: any,
routes?: any // legacy
}) { }) {
if (opts.routes) {
throw new Error(`As of Sapper 0.15, opts.routes should be opts.manifest`);
}
const output = locations.dest(); const output = locations.dest();
const { manifest, store, ignore } = opts;
let emitted_basepath = false; let emitted_basepath = false;
const middleware = compose_handlers([ const middleware = compose_handlers([
ignore && ((req: Req, res: ServerResponse, next: () => void) => {
req[IGNORE] = toIgnore(req.path, ignore);
next();
}),
(req: Req, res: ServerResponse, next: () => void) => { (req: Req, res: ServerResponse, next: () => void) => {
if (req[IGNORE]) return next();
if (req.baseUrl === undefined) { if (req.baseUrl === undefined) {
let { originalUrl } = req; let { originalUrl } = req;
if (req.url === '/' && originalUrl[originalUrl.length - 1] !== '/') { if (req.url === '/' && originalUrl[originalUrl.length - 1] !== '/') {
@@ -117,8 +157,8 @@ export default function middleware({ routes, store }: {
cache_control: 'max-age=31536000' cache_control: 'max-age=31536000'
}), }),
get_server_route_handler(routes.server_routes), get_server_route_handler(manifest.server_routes),
get_page_handler(routes, store) get_page_handler(manifest, store)
].filter(Boolean)); ].filter(Boolean));
return middleware; return middleware;
@@ -142,6 +182,8 @@ function serve({ prefix, pathname, cache_control }: {
: (file: string) => (cache.has(file) ? cache : cache.set(file, fs.readFileSync(path.resolve(output, file)))).get(file) : (file: string) => (cache.has(file) ? cache : cache.set(file, fs.readFileSync(path.resolve(output, file)))).get(file)
return (req: Req, res: ServerResponse, next: () => void) => { return (req: Req, res: ServerResponse, next: () => void) => {
if (req[IGNORE]) return next();
if (filter(req)) { if (filter(req)) {
const type = lookup(req.path); const type = lookup(req.path);
@@ -224,6 +266,8 @@ function get_server_route_handler(routes: ServerRoute[]) {
} }
return function find_route(req: Req, res: ServerResponse, next: () => void) { return function find_route(req: Req, res: ServerResponse, next: () => void) {
if (req[IGNORE]) return next();
for (const route of routes) { for (const route of routes) {
if (route.pattern.test(req.path)) { if (route.pattern.test(req.path)) {
handle_route(route, req, res, next); handle_route(route, req, res, next);
@@ -235,7 +279,7 @@ function get_server_route_handler(routes: ServerRoute[]) {
}; };
} }
function get_page_handler(routes: RouteObject, store_getter: (req: Req) => Store) { function get_page_handler(manifest: Manifest, store_getter: (req: Req) => Store) {
const output = locations.dest(); const output = locations.dest();
const get_chunks = dev() const get_chunks = dev()
@@ -246,17 +290,24 @@ function get_page_handler(routes: RouteObject, store_getter: (req: Req) => Store
? () => fs.readFileSync(`${locations.app()}/template.html`, 'utf-8') ? () => fs.readFileSync(`${locations.app()}/template.html`, 'utf-8')
: (str => () => str)(fs.readFileSync(`${locations.dest()}/template.html`, 'utf-8')); : (str => () => str)(fs.readFileSync(`${locations.dest()}/template.html`, 'utf-8'));
const { server_routes, pages } = routes; const { server_routes, pages } = manifest;
const error_route = routes.error; const error_route = manifest.error;
const should_wrap_data = dev() || process.env.SAPPER_EXPORT;
function handle_error(req: Req, res: ServerResponse, statusCode: number, error: Error | string) {
handle_page({
pattern: null,
parts: [
{ name: null, component: error_route }
]
}, req, res, statusCode, error);
}
function handle_page(page: Page, req: Req, res: ServerResponse, status = 200, error: Error | string = null) { function handle_page(page: Page, req: Req, res: ServerResponse, status = 200, error: Error | string = null) {
const get_params = page.parts[page.parts.length - 1].params || (() => ({})); const get_params = page.parts[page.parts.length - 1].params || (() => ({}));
const match = error ? null : page.pattern.exec(req.path); const match = error ? null : page.pattern.exec(req.path);
req.params = error
? {}
: get_params(match);
const chunks: Record<string, string | string[]> = get_chunks(); const chunks: Record<string, string | string[]> = get_chunks();
res.setHeader('Content-Type', 'text/html'); res.setHeader('Content-Type', 'text/html');
@@ -266,6 +317,8 @@ function get_page_handler(routes: RouteObject, store_getter: (req: Req) => Store
let preloaded_chunks = Array.isArray(chunks.main) ? chunks.main : [chunks.main]; let preloaded_chunks = Array.isArray(chunks.main) ? chunks.main : [chunks.main];
if (!error) { if (!error) {
page.parts.forEach(part => { page.parts.forEach(part => {
if (!part) return;
// using concat because it could be a string or an array. thanks webpack! // using concat because it could be a string or an array. thanks webpack!
preloaded_chunks = preloaded_chunks.concat(chunks[part.name]); preloaded_chunks = preloaded_chunks.concat(chunks[part.name]);
}); });
@@ -279,19 +332,11 @@ function get_page_handler(routes: RouteObject, store_getter: (req: Req) => Store
res.setHeader('Link', link); res.setHeader('Link', link);
const store = store_getter ? store_getter(req) : null; const store = store_getter ? store_getter(req) : null;
const props = { query: req.query, path: req.path };
if (error) {
props.error = error instanceof Error ? error : { message: error };
props.status = status;
}
let redirect: { statusCode: number, location: string }; let redirect: { statusCode: number, location: string };
let preload_error: { statusCode: number, message: Error | string }; let preload_error: { statusCode: number, message: Error | string };
Promise.all(page.parts.map(part => { const preload_context = {
return part.component.preload
? part.component.preload.call({
redirect: (statusCode: number, location: string) => { redirect: (statusCode: number, location: string) => {
if (redirect && (redirect.statusCode !== statusCode || redirect.location !== location)) { if (redirect && (redirect.statusCode !== statusCode || redirect.location !== location)) {
throw new Error(`Conflicting redirects`); throw new Error(`Conflicting redirects`);
@@ -337,62 +382,108 @@ function get_page_handler(routes: RouteObject, store_getter: (req: Req) => Store
return fetch(parsed.href, opts); return fetch(parsed.href, opts);
}, },
store store
}, req) };
const root_preloaded = manifest.root.preload
? manifest.root.preload.call(preload_context, {
path: req.path,
query: req.query,
params: {}
})
: {}; : {};
})).catch(err => {
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,
query: req.query,
params: part.params ? part.params(match) : {}
})
: {};
}))).catch(err => {
preload_error = { statusCode: 500, message: err }; preload_error = { statusCode: 500, message: err };
return []; // appease TypeScript return []; // appease TypeScript
}).then(preloaded => { }).then(preloaded => {
if (redirect) { if (redirect) {
const location = `${req.baseUrl}/${redirect.location}`;
res.statusCode = redirect.statusCode; res.statusCode = redirect.statusCode;
res.setHeader('Location', `${req.baseUrl}/${redirect.location}`); res.setHeader('Location', location);
res.end(); res.end();
if (process.send) {
process.send({
__sapper__: true,
event: 'file',
url: req.url,
method: req.method,
status: redirect.statusCode,
type: 'text/html',
body: `<script>window.location.href = "${location}"</script>`
});
}
return; return;
} }
if (preload_error) { if (preload_error) {
handle_page({ handle_error(req, res, preload_error.statusCode, preload_error.message);
pattern: null,
parts: [
{ name: null, component: error_route }
]
}, req, res, preload_error.statusCode, preload_error.message);
return; return;
} }
const serialized = {
preloaded: page.parts.map((part, i) => {
return part.component.preload ? try_serialize(preloaded[i]) : null;
}),
store: store && try_serialize(store.get())
};
const segments = req.path.split('/').filter(Boolean); const segments = req.path.split('/').filter(Boolean);
const data = Object.assign({}, props, { params: req.params }, { const props: Props = {
child: {} path: req.path,
query: req.query,
params: {},
child: null
};
if (error) {
props.error = error instanceof Error ? error : { message: error };
props.status = status;
}
const data = Object.assign({}, props, preloaded[0], {
params: {},
child: {
segment: segments[0]
}
}); });
// in dev and export modes, we wrap data in proxies to see
// how much of it is used in the initial render
const wrapped = should_wrap_data && wrap_data(preloaded);
// this is an easy way to 'reify' top-level values
const _preloaded = should_wrap_data
? wrapped.data.map((x: any) => x)
: preloaded;
let level = data.child; let level = data.child;
for (let i = 0; i < page.parts.length; i += 1) { for (let i = 0; i < page.parts.length; i += 1) {
const part = page.parts[i]; const part = page.parts[i];
if (!part) continue;
const get_params = part.params || (() => ({})); const get_params = part.params || (() => ({}));
Object.assign(level, { Object.assign(level, {
segment: segments[i],
component: part.component, component: part.component,
props: Object.assign({}, props, { props: Object.assign({}, props, {
params: get_params(match), params: get_params(match)
query: req.query }, _preloaded[i + 1])
}, preloaded[i])
}); });
level.props.child = {}; level.props.child = <Props["child"]>{
segment: segments[i + 1]
};
level = level.props.child; level = level.props.child;
} }
const { html, head, css } = routes.root.render(data, { const { html, head, css } = manifest.root.render(data, {
store store
}); });
@@ -402,9 +493,51 @@ function get_page_handler(routes: RouteObject, store_getter: (req: Req) => Store
.map(file => `<script src='${req.baseUrl}/client/${file}'></script>`) .map(file => `<script src='${req.baseUrl}/client/${file}'></script>`)
.join(''); .join('');
const unwrapped = should_wrap_data && wrapped.unwrap();
const preloaded_serialized = preloaded.map(try_serialize);
if (should_wrap_data && process.send) {
const discrepancies = [];
unwrapped.forEach((clone, i) => {
const loaded = preloaded_serialized[i];
if (!loaded) return;
const rendered = try_serialize(clone);
if (rendered !== loaded) {
const part = page.parts[i - 1];
const file = part ? part.file : '_layout.html';
discrepancies.push({
file,
preloaded: loaded.length,
rendered: rendered.length,
props: list_unused_properties(preloaded[i], clone)
});
}
});
if (discrepancies.length) {
process.send({
__sapper__: true,
event: 'unused_data',
url: req.url,
discrepancies
});
}
}
const serialized = {
preloaded: `[${preloaded_serialized.join(',')}]`,
store: store && try_serialize(store.get())
};
let inline_script = `__SAPPER__={${[ let inline_script = `__SAPPER__={${[
error && `error:1`,
`baseUrl:"${req.baseUrl}"`, `baseUrl:"${req.baseUrl}"`,
serialized.preloaded && `preloaded: [${serialized.preloaded}]`, serialized.preloaded && `preloaded:${serialized.preloaded}`,
serialized.store && `store:${serialized.store}` serialized.store && `store:${serialized.store}`
].filter(Boolean).join(',')}};`; ].filter(Boolean).join(',')}};`;
@@ -424,23 +557,37 @@ function get_page_handler(routes: RouteObject, store_getter: (req: Req) => Store
res.end(body); res.end(body);
if (process.send) { if (process.send) {
process.send({
__sapper__: true,
event: 'preload',
url: req.url,
size: serialized.preloaded.length
});
process.send({ process.send({
__sapper__: true, __sapper__: true,
event: 'file', event: 'file',
url: req.url, url: req.url,
method: req.method, method: req.method,
status: 200, status,
type: 'text/html', type: 'text/html',
body body
}); });
} }
}).catch(err => { }).catch(err => {
if (error) {
// we encountered an error while rendering the error page — oops
res.statusCode = 500; res.statusCode = 500;
res.end(err.message); res.end(`<pre>${escape_html(err.message)}</pre>`);
} else {
handle_error(req, res, 500, err);
}
}); });
} }
return function find_route(req: Req, res: ServerResponse) { return function find_route(req: Req, res: ServerResponse, next: () => void) {
if (req[IGNORE]) return next();
if (!server_routes.some(route => route.pattern.test(req.path))) { if (!server_routes.some(route => route.pattern.test(req.path))) {
for (const page of pages) { for (const page of pages) {
if (page.pattern.test(req.path)) { if (page.pattern.test(req.path)) {
@@ -450,12 +597,7 @@ function get_page_handler(routes: RouteObject, store_getter: (req: Req) => Store
} }
} }
handle_page({ handle_error(req, res, 404, 'Not found');
pattern: null,
parts: [
{ name: null, component: error_route }
]
}, req, res, 404, 'Not found');
}; };
} }
@@ -486,3 +628,15 @@ function try_serialize(data: any) {
return null; return null;
} }
} }
function escape_html(html: string) {
const chars: Record<string, string> = {
'"' : 'quot',
"'": '#39',
'&': 'amp',
'<' : 'lt',
'>' : 'gt'
};
return html.replace(/["'&<>]/g, c => `&${chars[c]};`);
}

View File

@@ -0,0 +1,34 @@
export function list_unused_properties(all: any, used: any) {
const props: string[] = [];
const seen = new Set();
function walk(keypath: string, a: any, b: any) {
if (seen.has(a)) return;
seen.add(a);
if (!a || typeof a !== 'object') return;
const is_array = Array.isArray(a);
for (const key in a) {
const child_keypath = keypath
? is_array ? `${keypath}[${key}]` : `${keypath}.${key}`
: key;
if (hasProp.call(b, key)) {
const a_child = a[key];
const b_child = b[key];
walk(child_keypath, a_child, b_child);
} else {
props.push(child_keypath);
}
}
}
walk(null, all, used);
return props;
}
const hasProp = Object.prototype.hasOwnProperty;

View File

@@ -0,0 +1,85 @@
type Obj = Record<string, any>;
export function wrap_data(data: any) {
const proxies = new Map();
const clones = new Map();
const handler = {
get(target: any, property: string): any {
const value = target[property];
const intercepted = intercept(value);
const target_clone = clones.get(target);
const child_clone = clones.get(value);
if (target_clone && target.hasOwnProperty(property)) {
target_clone[property] = child_clone || value;
}
return intercepted;
},
};
function get_or_create_proxy(obj: any) {
if (!proxies.has(obj)) {
proxies.set(obj, new Proxy(obj, handler));
}
return proxies.get(obj);
}
function intercept(obj: any) {
if (clones.has(obj)) return obj;
if (obj && typeof obj === 'object') {
if (Array.isArray(obj)) {
clones.set(obj, []);
return get_or_create_proxy(obj);
}
else if (isPlainObject(obj)) {
clones.set(obj, {});
return get_or_create_proxy(obj);
}
}
clones.set(obj, obj);
return obj;
}
return {
data: intercept(data),
unwrap: () => {
return clones.get(data);
}
};
}
const objectProtoOwnPropertyNames = Object.getOwnPropertyNames(Object.prototype).sort().join('\0')
function isPlainObject(obj: any) {
const proto = Object.getPrototypeOf(obj);
if (
proto !== Object.prototype &&
proto !== null &&
Object.getOwnPropertyNames(proto).sort().join('\0') !== objectProtoOwnPropertyNames
) {
return false;
}
if (Object.getOwnPropertySymbols(obj).length > 0) {
return false;
}
return true;
}
function pick(obj: Obj, props: string[]) {
const picked: Obj = {};
props.forEach(prop => {
picked[prop] = obj[prop];
});
return picked;
}

View File

@@ -1,12 +1,12 @@
import { detach, findAnchor, scroll_state, which } from './utils'; import { detach, findAnchor, scroll_state, which } from './utils';
import { Component, ComponentConstructor, Params, Query, Redirect, Routes, RouteData, ScrollPosition, Store, Target } from './interfaces'; import { Component, ComponentConstructor, Params, Query, Redirect, Manifest, RouteData, ScrollPosition, Store, Target } from './interfaces';
const initial_data = typeof window !== 'undefined' && window.__SAPPER__; const initial_data = typeof window !== 'undefined' && window.__SAPPER__;
export let root: Component; export let root: Component;
let target: Node; let target: Node;
let store: Store; let store: Store;
let routes: Routes; let manifest: Manifest;
let segments: string[] = []; let segments: string[] = [];
type RootProps = { type RootProps = {
@@ -56,10 +56,10 @@ function select_route(url: URL): Target {
const path = url.pathname.slice(initial_data.baseUrl.length); const path = url.pathname.slice(initial_data.baseUrl.length);
// avoid accidental clashes between server routes and pages // avoid accidental clashes between server routes and pages
if (routes.ignore.some(pattern => pattern.test(path))) return; if (manifest.ignore.some(pattern => pattern.test(path))) return;
for (let i = 0; i < routes.pages.length; i += 1) { for (let i = 0; i < manifest.pages.length; i += 1) {
const page = routes.pages[i]; const page = manifest.pages[i];
const match = page.pattern.exec(path); const match = page.pattern.exec(path);
if (match) { if (match) {
@@ -77,14 +77,14 @@ function select_route(url: URL): Target {
let current_token: {}; 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 (current_token !== token) return;
if (root) { if (root) {
// first, clear out highest-level root component // first, clear out highest-level root component
let level = data.child; let level = data.child;
for (let i = 0; i < changed_from; i += 1) { for (let i = 0; i < nullable_depth; i += 1) {
if (i === changed_from) break; if (i === nullable_depth) break;
level = level.props.child; level = level.props.child;
} }
@@ -106,7 +106,9 @@ function render(data: any, changed_from: number, scroll: ScrollPosition, token:
detach(end); detach(end);
} }
root = new routes.root({ Object.assign(data, root_data);
root = new manifest.root({
target, target,
data, data,
store, store,
@@ -126,10 +128,13 @@ function changed(a: Record<string, string | true>, b: Record<string, string | tr
return JSON.stringify(a) !== JSON.stringify(b); return JSON.stringify(a) !== JSON.stringify(b);
} }
let root_preload: Promise<any>;
let root_data: any;
function prepare_page(target: Target): Promise<{ function prepare_page(target: Target): Promise<{
redirect?: Redirect; redirect?: Redirect;
data?: any; data?: any;
changed_from?: number; nullable_depth?: number;
}> { }> {
if (root) { if (root) {
root.set({ preloading: true }); root.set({ preloading: true });
@@ -162,8 +167,19 @@ function prepare_page(target: Target): Promise<{
} }
}; };
if (!root_preload) {
root_preload = manifest.root.preload
? initial_data.preloaded[0] || manifest.root.preload.call(preload_context, {
path,
query,
params: {}
})
: {};
}
return Promise.all(page.parts.map(async (part, i) => { return Promise.all(page.parts.map(async (part, i) => {
if (i < changed_from) return null; if (i < changed_from) return null;
if (!part) return null;
const { default: Component } = await part.component(); const { default: Component } = await part.component();
const req = { const req = {
@@ -172,15 +188,17 @@ function prepare_page(target: Target): Promise<{
params: part.params ? part.params(target.match) : {} params: part.params ? part.params(target.match) : {}
}; };
const preloaded = ready || !initial_data.preloaded[i] const preloaded = ready || !initial_data.preloaded[i + 1]
? Component.preload ? await Component.preload.call(preload_context, req) : {} ? Component.preload ? await Component.preload.call(preload_context, req) : {}
: initial_data.preloaded[i]; : initial_data.preloaded[i + 1];
return { Component, preloaded }; return { Component, preloaded };
})).catch(err => { })).catch(err => {
error = { statusCode: 500, message: err }; error = { statusCode: 500, message: err };
return []; return [];
}).then(results => { }).then(async results => {
if (!root_data) root_data = await root_preload;
if (redirect) { if (redirect) {
return { redirect }; return { redirect };
} }
@@ -203,7 +221,7 @@ function prepare_page(target: Target): Promise<{
data: Object.assign({}, props, { data: Object.assign({}, props, {
preloading: false, preloading: false,
child: { child: {
component: routes.error, component: manifest.error,
props props
} }
}) })
@@ -214,33 +232,42 @@ function prepare_page(target: Target): Promise<{
const data = { const data = {
path, path,
preloading: false, 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(query, root_props.query)) data.query = query;
if (changed(params, root_props.params)) data.params = params; if (changed(params, root_props.params)) data.params = params;
let level = data.child; let level = data.child;
let nullable_depth = 0;
for (let i = 0; i < page.parts.length; i += 1) { for (let i = 0; i < page.parts.length; i += 1) {
const part = page.parts[i]; const part = page.parts[i];
if (!part) continue;
const get_params = part.params || (() => ({})); const get_params = part.params || (() => ({}));
if (i < changed_from) { if (i < changed_from) {
level.props.path = path; level.props.path = path;
level.props.query = query; level.props.query = query;
level.props.child = Object.assign({}, level.props.child); level.props.child = Object.assign({}, level.props.child);
nullable_depth += 1;
} else { } else {
level.segment = new_segments[i];
level.component = results[i].Component; level.component = results[i].Component;
level.props = Object.assign({}, level.props, props, { level.props = Object.assign({}, level.props, props, {
params: get_params(target.match), params: get_params(target.match),
}, results[i].preloaded); }, results[i].preloaded);
level.props.child = {}; level.props.child = {};
} }
level = level.props.child; level = level.props.child;
level.segment = segments[i + 1];
} }
return { data, changed_from }; return { data, nullable_depth };
}); });
} }
@@ -265,13 +292,13 @@ async function navigate(target: Target, id: number): Promise<any> {
prefetching = null; prefetching = null;
const token = current_token = {}; const token = current_token = {};
const { redirect, data, changed_from } = await loaded; const { redirect, data, nullable_depth } = await loaded;
if (redirect) { if (redirect) {
await goto(redirect.location, { replaceState: true }); await goto(redirect.location, { replaceState: true });
} else { } else {
render(data, changed_from, scroll_history[id], token); render(data, nullable_depth, scroll_history[id], token);
document.activeElement.blur(); if (document.activeElement) document.activeElement.blur();
} }
} }
@@ -336,7 +363,7 @@ function handle_popstate(event: PopStateEvent) {
let prefetching: { let prefetching: {
href: string; href: string;
promise: Promise<{ redirect?: Redirect, data?: any, changed_from?: number }>; promise: Promise<{ redirect?: Redirect, data?: any, nullable_depth?: number }>;
} = null; } = null;
export function prefetch(href: string) { export function prefetch(href: string) {
@@ -369,13 +396,23 @@ function trigger_prefetch(event: MouseEvent | TouchEvent) {
let inited: boolean; let inited: boolean;
let ready = false; let ready = false;
export function init(opts: { App: ComponentConstructor, target: Node, routes: Routes, store?: (data: any) => Store }) { export function init(opts: {
App: ComponentConstructor,
target: Node,
manifest: Manifest,
store?: (data: any) => Store,
routes?: any // legacy
}) {
if (opts instanceof HTMLElement) { if (opts instanceof HTMLElement) {
throw new Error(`The signature of init(...) has changed — see https://sapper.svelte.technology/guide#0-11-to-0-12 for more information`); throw new Error(`The signature of init(...) has changed — see https://sapper.svelte.technology/guide#0-11-to-0-12 for more information`);
} }
if (opts.routes) {
throw new Error(`As of Sapper 0.15, opts.routes should be opts.manifest`);
}
target = opts.target; target = opts.target;
routes = opts.routes; manifest = opts.manifest;
if (opts && opts.store) { if (opts && opts.store) {
store = opts.store(initial_data.store); store = opts.store(initial_data.store);
@@ -402,8 +439,10 @@ export function init(opts: { App: ComponentConstructor, target: Node, routes: Ro
history.replaceState({ id: uid }, '', href); history.replaceState({ id: uid }, '', href);
if (!initial_data.error) {
const target = select_route(new URL(window.location.href)); const target = select_route(new URL(window.location.href));
if (target) return navigate(target, uid); if (target) return navigate(target, uid);
}
}); });
} }
@@ -423,9 +462,9 @@ export function goto(href: string, opts = { replaceState: false }) {
} }
export function prefetchRoutes(pathnames: string[]) { export function prefetchRoutes(pathnames: string[]) {
if (!routes) throw new Error(`You must call init() first`); if (!manifest) throw new Error(`You must call init() first`);
return routes.pages return manifest.pages
.filter(route => { .filter(route => {
if (!pathnames) return true; if (!pathnames) return true;
return pathnames.some(pathname => route.pattern.test(pathname)); return pathnames.some(pathname => route.pattern.test(pathname));

View File

@@ -23,7 +23,7 @@ export type Page = {
}>; }>;
}; };
export type Routes = { export type Manifest = {
ignore: RegExp[]; ignore: RegExp[];
root: ComponentConstructor; root: ComponentConstructor;
error: () => Promise<{ default: ComponentConstructor }>; error: () => Promise<{ default: ComponentConstructor }>;

View File

@@ -1,11 +1,11 @@
import { init, prefetchRoutes } from '../../../runtime.js'; import { init, prefetchRoutes } from '../../../runtime.js';
import { Store } from 'svelte/store.js'; import { Store } from 'svelte/store.js';
import { routes } from './manifest/client.js'; import { manifest } from './manifest/client.js';
window.init = () => { window.init = () => {
return init({ return init({
target: document.querySelector('#sapper'), target: document.querySelector('#sapper'),
routes, manifest,
store: data => new Store(data) store: data => new Store(data)
}); });
}; };

View File

@@ -4,7 +4,7 @@ import express from 'express';
import serve from 'serve-static'; import serve from 'serve-static';
import sapper from '../../../dist/middleware.ts.js'; import sapper from '../../../dist/middleware.ts.js';
import { Store } from 'svelte/store.js'; import { Store } from 'svelte/store.js';
import { routes } from './manifest/server.js'; import { manifest } from './manifest/server.js';
let pending; let pending;
let ended; let ended;
@@ -86,13 +86,19 @@ const middlewares = [
}, },
sapper({ sapper({
routes, manifest,
store: () => { store: () => {
return new Store({ return new Store({
title: 'Stored title' title: 'Stored title'
}); });
} },
}) ignore: [
/foobar/i,
'/buzz',
'fizz',
x => x === '/hello'
]
}),
]; ];
if (BASEPATH) { if (BASEPATH) {
@@ -101,4 +107,8 @@ if (BASEPATH) {
app.use(...middlewares); app.use(...middlewares);
} }
['foobar', 'buzz', 'fizzer', 'hello'].forEach(uri => {
app.get('/'+uri, (req, res) => res.end(uri));
});
app.listen(PORT); app.listen(PORT);

View File

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

View File

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

View File

@@ -1,26 +0,0 @@
<svelte:head>
<title>Sapper project template</title>
</svelte:head>
<h1>Great success!</h1>
<a href='.'>home</a>
<a href='about'>about</a>
<a href='slow-preload'>slow preload</a>
<a href='redirect-from'>redirect</a>
<a href='blog/nope'>broken link</a>
<a href='blog/throw-an-error'>error link</a>
<a href='credentials?creds=include'>credentials</a>
<a rel=prefetch class='{page === "blog" ? "selected" : ""}' href='blog'>blog</a>
<div class='hydrate-test'></div>
<style>
h1 {
text-align: center;
font-size: 2.8em;
text-transform: uppercase;
font-weight: 700;
margin: 0 0 0.5em 0;
}
</style>

View File

@@ -0,0 +1,15 @@
{#if preloading}
<progress class='preloading-progress' value=0.5/>
{/if}
<svelte:component this={child.component} {rootPreloadFunctionRan} {...child.props}/>
<script>
export default {
preload() {
return {
rootPreloadFunctionRan: true
};
}
};
</script>

View File

@@ -1,25 +0,0 @@
<svelte:head>
<title>Blog</title>
</svelte:head>
<h1>Recent posts</h1>
<ul>
{#each posts as post}
<!-- we're using the non-standard `rel=prefetch` attribute to
tell Sapper to load the data for the page as soon as
the user hovers over the link or taps it, instead of
waiting for the 'click' event -->
<li><a rel='prefetch' href='blog/{post.slug}'>{post.title}</a></li>
{/each}
</ul>
<script>
export default {
preload({ params, query }) {
return fetch(`blog.json`).then(r => r.json()).then(posts => {
return { posts };
});
}
};
</script>

View File

@@ -1 +1,25 @@
<svelte:component this={child.component} {...child.props}/> <svelte:head>
<title>Blog</title>
</svelte:head>
<h1>Recent posts</h1>
<ul>
{#each posts as post}
<!-- we're using the non-standard `rel=prefetch` attribute to
tell Sapper to load the data for the page as soon as
the user hovers over the link or taps it, instead of
waiting for the 'click' event -->
<li><a rel='prefetch' href='blog/{post.slug}'>{post.title}</a></li>
{/each}
</ul>
<script>
export default {
preload({ params, query }) {
return fetch(`blog.json`).then(r => r.json()).then(posts => {
return { posts };
});
}
};
</script>

View File

@@ -1,6 +1,26 @@
{#if preloading} <svelte:head>
<progress class='preloading-progress' value=0.5/> <title>Sapper project template</title>
{/if} </svelte:head>
<svelte:component this={child.component} {...child.props}/> <h1>Great success!</h1>
<a href='.'>home</a>
<a href='about'>about</a>
<a href='slow-preload'>slow preload</a>
<a href='redirect-from'>redirect</a>
<a href='blog/nope'>broken link</a>
<a href='blog/throw-an-error'>error link</a>
<a href='credentials?creds=include'>credentials</a>
<a rel=prefetch class='{page === "blog" ? "selected" : ""}' href='blog'>blog</a>
<div class='hydrate-test'></div>
<style>
h1 {
text-align: center;
font-size: 2.8em;
text-transform: uppercase;
font-weight: 700;
margin: 0 0 0.5em 0;
}
</style>

View File

@@ -0,0 +1 @@
<h1>root preload function ran: {rootPreloadFunctionRan}</h1>

View File

@@ -1,9 +0,0 @@
<p>URL is {url}</p>
<script>
export default {
preload({ url }) {
if (url) return { url };
}
};
</script>

View File

@@ -100,12 +100,11 @@ describe('sapper', function() {
// Client scripts that should show up in the extraction directory. // Client scripts that should show up in the extraction directory.
const expectedClientRegexes = [ const expectedClientRegexes = [
/client\/[^/]+\/main(\.\d+)?\.js/, /client\/[^/]+\/main(\.\d+)?\.js/,
/client\/[^/]+\/page_index(\.\d+)?\.js/, /client\/[^/]+\/index(\.\d+)?\.js/,
/client\/[^/]+\/page_about(\.\d+)?\.js/, /client\/[^/]+\/about(\.\d+)?\.js/,
/client\/[^/]+\/page_blog_\$slug(\.\d+)?\.js/, /client\/[^/]+\/blog_\$slug(\.\d+)?\.js/,
/client\/[^/]+\/page_blog(\.\d+)?\.js/, /client\/[^/]+\/blog(\.\d+)?\.js/,
/client\/[^/]+\/page_show\$45url(\.\d+)?\.js/, /client\/[^/]+\/slow\$45preload(\.\d+)?\.js/,
/client\/[^/]+\/page_slow\$45preload(\.\d+)?\.js/,
]; ];
const allPages = walkSync(dest); const allPages = walkSync(dest);
@@ -370,16 +369,6 @@ function run({ mode, basepath = '' }) {
}); });
}); });
it('passes entire request object to preload', () => {
return nightmare
.goto(`${base}/show-url`)
.init()
.evaluate(() => document.querySelector('p').innerHTML)
.then(html => {
assert.equal(html, `URL is /show-url`);
});
});
it('calls a delete handler', () => { it('calls a delete handler', () => {
return nightmare return nightmare
.goto(`${base}/delete-test`) .goto(`${base}/delete-test`)
@@ -488,6 +477,42 @@ function run({ mode, basepath = '' }) {
}); });
}); });
// Ignores are meant for top-level escape.
// ~> Sapper **should** own the entire {basepath} when designated.
if (!basepath) {
it('respects `options.ignore` values (RegExp)', () => {
return nightmare.goto(`${base}/foobar`)
.evaluate(() => document.documentElement.textContent)
.then(text => {
assert.equal(text, 'foobar');
});
});
it('respects `options.ignore` values (String #1)', () => {
return nightmare.goto(`${base}/buzz`)
.evaluate(() => document.documentElement.textContent)
.then(text => {
assert.equal(text, 'buzz');
});
});
it('respects `options.ignore` values (String #2)', () => {
return nightmare.goto(`${base}/fizzer`)
.evaluate(() => document.documentElement.textContent)
.then(text => {
assert.equal(text, 'fizzer');
});
});
it('respects `options.ignore` values (Function)', () => {
return nightmare.goto(`${base}/hello`)
.evaluate(() => document.documentElement.textContent)
.then(text => {
assert.equal(text, 'hello');
});
});
}
it('does not attempt client-side navigation to server routes', () => { it('does not attempt client-side navigation to server routes', () => {
return nightmare.goto(`${base}/blog/how-is-sapper-different-from-next`) return nightmare.goto(`${base}/blog/how-is-sapper-different-from-next`)
.init() .init()
@@ -640,9 +665,9 @@ function run({ mode, basepath = '' }) {
.evaluate(() => document.querySelector('#sapper').textContent) .evaluate(() => document.querySelector('#sapper').textContent)
.then(text => { .then(text => {
assert.deepEqual(text.split('\n').filter(Boolean), [ assert.deepEqual(text.split('\n').filter(Boolean), [
'x: foo 1',
'y: bar 1', 'y: bar 1',
'z: baz 1' 'z: baz 1',
'child segment: baz'
]); ]);
return nightmare.click(`a`) return nightmare.click(`a`)
@@ -653,9 +678,9 @@ function run({ mode, basepath = '' }) {
}) })
.then(text => { .then(text => {
assert.deepEqual(text.split('\n').filter(Boolean), [ assert.deepEqual(text.split('\n').filter(Boolean), [
'x: foo 1',
'y: bar 1', 'y: bar 1',
'z: qux 2' 'z: qux 2',
'child segment: qux'
]); ]);
}); });
}); });
@@ -667,6 +692,14 @@ function run({ mode, basepath = '' }) {
assert.equal(title, 'it works'); assert.equal(title, 'it works');
}); });
}); });
it('runs preload in root component', () => {
return nightmare.goto(`${base}/preload-root`)
.page.title()
.then(title => {
assert.equal(title, 'root preload function ran: true');
});
});
}); });
describe('headers', () => { describe('headers', () => {

View File

@@ -6,48 +6,46 @@ describe('create_routes', () => {
it('creates routes', () => { it('creates routes', () => {
const { components, pages, server_routes } = create_routes(path.join(__dirname, 'samples/basic')); const { components, pages, server_routes } = create_routes(path.join(__dirname, 'samples/basic'));
const page_index = { name: 'page_index', file: '_default.html' }; const index = { name: 'index', file: 'index.html' };
const page_about = { name: 'page_about', file: 'about.html' }; const about = { name: 'about', file: 'about.html' };
const page_blog = { name: 'page_blog', file: 'blog/index.html' }; const blog = { name: 'blog', file: 'blog/index.html' };
const page_blog_index = { name: 'page_blog_index', file: 'blog/_default.html' }; const blog_$slug = { name: 'blog_$slug', file: 'blog/[slug].html' };
const page_blog_$slug = { name: 'page_blog_$slug', file: 'blog/[slug].html' };
assert.deepEqual(components, [ assert.deepEqual(components, [
page_index, index,
page_about, about,
page_blog, blog,
page_blog_index, blog_$slug
page_blog_$slug
]); ]);
assert.deepEqual(pages, [ assert.deepEqual(pages, [
{ {
pattern: /^\/?$/, pattern: /^\/?$/,
parts: [ parts: [
{ component: page_index, params: [] } { component: index, params: [] }
] ]
}, },
{ {
pattern: /^\/about\/?$/, pattern: /^\/about\/?$/,
parts: [ parts: [
{ component: page_about, params: [] } { component: about, params: [] }
] ]
}, },
{ {
pattern: /^\/blog\/?$/, pattern: /^\/blog\/?$/,
parts: [ parts: [
{ component: page_blog, params: [] }, null,
{ component: page_blog_index, params: [] } { component: blog, params: [] }
] ]
}, },
{ {
pattern: /^\/blog\/([^\/]+?)\/?$/, pattern: /^\/blog\/([^\/]+?)\/?$/,
parts: [ parts: [
{ component: page_blog, params: [] }, null,
{ component: page_blog_$slug, params: ['slug'] } { component: blog_$slug, params: ['slug'] }
] ]
} }
]); ]);
@@ -74,9 +72,9 @@ describe('create_routes', () => {
// had to remove ? and " because windows // had to remove ? and " because windows
// const quote = { name: 'page_$34', file: '".html' }; // const quote = { name: '$34', file: '".html' };
const hash = { name: 'page_$35', file: '#.html' }; const hash = { name: '$35', file: '#.html' };
// const question_mark = { name: 'page_$63', file: '?.html' }; // const question_mark = { name: '$63', file: '?.html' };
assert.deepEqual(components, [ assert.deepEqual(components, [
// quote, // quote,
@@ -104,15 +102,15 @@ describe('create_routes', () => {
it('sorts routes correctly', () => { it('sorts routes correctly', () => {
const { pages } = create_routes(path.join(__dirname, 'samples/sorting')); 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)), [
['_default.html'], ['index.html'],
['about.html'], ['about.html'],
['post/index.html', 'post/_default.html'], [null, 'post/index.html'],
['post/index.html', 'post/bar.html'], [null, 'post/bar.html'],
['post/index.html', 'post/foo.html'], [null, 'post/foo.html'],
['post/index.html', 'post/f[xx].html'], [null, 'post/f[xx].html'],
['post/index.html', 'post/[id([0-9-a-z]{3,})].html'], [null, 'post/[id([0-9-a-z]{3,})].html'],
['post/index.html', 'post/[id].html'], [null, 'post/[id].html'],
['[wildcard].html'] ['[wildcard].html']
]); ]);
}); });

View File

@@ -6,9 +6,9 @@
"noEmitOnError": true, "noEmitOnError": true,
"allowJs": true, "allowJs": true,
"lib": ["es5", "es6", "dom"], "lib": ["es5", "es6", "dom"],
"importHelpers": true "importHelpers": true,
"target": "ES5"
}, },
"target": "ES5",
"include": [ "include": [
"src" "src"
], ],