@kaliber/build 0.0.151 → 0.0.152-beta.2

package/lib/serve.js

@@ -69,12 +69,7 @@ app.use((err, req, res, next) => {
   console.error(err)
   if (reportError) reportError(err, req)
 
-  const response = res.status(500)
-  if (isProduction) {
-    findFile(req.path, internalServerError)
-      .then(file => file ? response.sendFile(file) : next())
-      .catch(next)
-  } else response.send(`<pre><title style='display: block;'>${err.stack || err.toString()}</title><pre>`)
+  serveInternalServerError(err, { req, res, next })
 })
 
 app.listen(port, () => console.log(`Server listening at port ${port}`))
@@ -84,7 +79,7 @@ async function resolveFile(req, res, next) {
     const { path } = req
     /** @type {Array<[string, (file:any) => any]>} */
     const combinations = [
-      [indexWithRouting, file => serveIndexWithRouting(req, res, file)],
+      [indexWithRouting, file => serveIndexWithRouting(file, { req, res, next })],
       [notFound, file => res.status(404).sendFile(file)],
       [index, file => res.status(200).sendFile(file)],
     ]
@@ -112,7 +107,7 @@ async function findFile(path, file) {
   return null
 }
 
-async function fileExists(path) {
+function fileExists(path) {
   return isProduction
     ? (!fileExists.cache || fileExists.cache[path] === undefined)
       ? accessFile(path).then(exists => (addPathToCache(path, exists), exists))
@@ -124,7 +119,7 @@ async function fileExists(path) {
   }
 }
 
-async function accessFile(path) {
+function accessFile(path) {
   return new Promise(resolve => access(path, err => resolve(!err)))
 }
 
@@ -138,26 +133,60 @@ function possibleDirectories(path) {
   return possibleDirectories
 }
 
-function serveIndexWithRouting(req, res, file) {
+function serveIndexWithRouting(file, { req, res, next }) {
   const envRequire = isProduction ? require : require('import-fresh')
+
   const routeTemplate = envRequire(file)
 
-  const routes = routeTemplate.routes
   const location = parsePath(req.url)
 
-  return Promise.resolve(routes)
-    .then(routes => (routes && routes.match(location, req)) || { status: 200, data: null })
-    .then(data => {
-      if (!routes || !routes.resolveIndex) return [data, routeTemplate]
-
-      const indexLocation = routes.resolveIndex(location, req)
-      if (!indexLocation) return [data, routeTemplate]
-
-      const indexPath = resolve(target, publicPathDir, indexLocation, indexWithRouting)
-      return [data, envRequire(indexPath)]
-    })
-    .then(([{ status, headers, data }, template]) =>
-      Promise.resolve(template({ location, data })).then(html => [status, headers, html])
-    )
-    .then(([ status, headers, html ]) => res.status(status).set(headers).send(html))
+  const [dataOrPromise, template] = getDataAndRouteTemplate(routeTemplate, { location, req })
+
+  if (dataOrPromise.then)
+    dataOrPromise
+      .then(({ status, headers, data }) => {
+        const html = renderTemplate(template, location, { data })
+        res.status(status).set(headers).send(html)
+      })
+      .catch(error => serveInternalServerError(error, { req, res, next }))
+  else {
+    try {
+      const { data, status, headers } = dataOrPromise
+      const html = renderTemplate(template, location, { data })
+      res.status(status).set(headers).send(html)
+    } catch (error) {
+      serveInternalServerError(error, { req, res, next })
+    }
+  }
+}
+
+function getDataAndRouteTemplate(routeTemplate, { location, req }) {
+  const envRequire = isProduction ? require : require('import-fresh')
+
+  const routes = routeTemplate.routes
+  const dataOrPromise = (routes && routes.match(location, req)) || { status: 200, data: null }
+
+  if (!routes || !routes.resolveIndex)
+    return [dataOrPromise, routeTemplate]
+
+  const indexLocation = routes.resolveIndex(location, req)
+  if (!indexLocation)
+    return [dataOrPromise, routeTemplate]
+
+  const indexPath = resolve(target, publicPathDir, indexLocation, indexWithRouting)
+  return [dataOrPromise, envRequire(indexPath)]
+}
+
+function renderTemplate(template, location, { data }) {
+  const html = template({ location, data })
+  return html
+}
+
+function serveInternalServerError(error, { res, req, next }) {
+  const response = res.status(500)
+  if (isProduction) {
+    findFile(req.path, internalServerError)
+      .then(file => file ? response.sendFile(file) : next())
+      .catch(next)
+  } else response.send(`<pre><title style='display: block;'>${error.stack || error.toString()}</title><pre>`)
 }

package/package.json

@@ -1,5 +1,5 @@
 {
-  "version": "0.0.151",
+  "version": "0.0.152-beta.2",
   "name": "@kaliber/build",
   "description": "Zero configuration, opinionated webpack / react build setup",
   "scripts": {