sku 16.3.0-root-errorboundary-type-20260921013810 → 16.3.0

package/CHANGELOG.md

@@ -1,6 +1,6 @@
 # sku
 
-## 16.3.0-root-errorboundary-type-20260921013810
+## 16.3.0
 
 ### Minor Changes
 
@@ -8,9 +8,25 @@
 
 - Add `entrySideEffects` so apps can import isomorphic modules (such as Braid reset) before any consumer module. ([#1717](https://github.com/seek-oss/sku/pull/1717))
 
-### Patch Changes
+- Webpack SSR: Pass the listening server and port to `onStart` ([#1730](https://github.com/seek-oss/sku/pull/1730))
+
+  `onStart` still receives the Express app as its first argument. A new second argument carries `{ httpServer, port }`, so apps can set keep-alive timeouts and close the server on shutdown.
+
+  ```tsx
+  import type { Server } from 'sku';
+
+  export default (): Server => ({
+    renderCallback,
+    onStart: (app, { httpServer, port }) => {
+      httpServer.keepAliveTimeout = 20_000;
+      process.on('SIGTERM', () => {
+        httpServer.close(() => process.exit(0));
+      });
+    },
+  });
+  ```
 
-- `SkuRouteObject` now forbids `ErrorBoundary` and `errorElement` on the top-level html route, including via `lazy`. ([#1726](https://github.com/seek-oss/sku/pull/1726))
+  See the [Webpack SSR](https://seek-oss.github.io/sku/ssr/webpack-ssr) docs for more details.
 
 ## 16.2.0
 

package/dist/packages/sku/package.mjs

@@ -1,7 +1,7 @@
 //#region package.json
 var package_default = {
 	name: "sku",
-	version: "16.2.0",
+	version: "16.3.0",
 	description: "Front-end development toolkit, powered by Webpack, Babel, Vanilla Extract and Jest",
 	types: "./dist/index.d.mts",
 	bin: { "sku": "bin/bin.js" },

package/dist/services/vite/ssr/types.d.mts

@@ -35,48 +35,15 @@ type JsonValue = string | number | boolean | null | Array<JsonValue | undefined>
   [key: string]: JsonValue | undefined;
 };
 /**
- * Child route in a `routesEntry` tree.
- * This type lets you set `ErrorBoundary` and `errorElement`.
- * This type is not a public `sku/runtime` export.
+ * Route object for `routesEntry`: React Router `RouteObject` plus optional
+ * `sites` membership. Omit `sites` ⇒ route is on every config site; present ⇒
+ * only those names. Sku type helper only — not a wrapped RR re-export.
+ * `children` are also `SkuRouteObject` so nested routes may set `sites`
+ * (no parent→child inheritance — each route declares membership explicitly).
  */
-type SkuChildRouteObject<Site extends string = string> = Omit<RouteObject, 'children'> & {
+type SkuRouteObject<Site extends string = string> = Omit<RouteObject, 'children'> & {
   sites?: Site[];
-  children?: Array<SkuChildRouteObject<Site>>;
-};
-type RouteObjectLazy = NonNullable<RouteObject['lazy']>;
-type RouteObjectLazyFunction = Extract<RouteObjectLazy, (...args: never[]) => unknown>;
-type RouteObjectLazyObject = Exclude<RouteObjectLazy, RouteObjectLazyFunction>;
-type SkuRootLazyResult = Omit<Awaited<ReturnType<RouteObjectLazyFunction>>, 'ErrorBoundary' | 'errorElement'> & {
-  ErrorBoundary?: never;
-  errorElement?: never;
-};
-/**
- * Root `lazy` from `RouteObject['lazy']`.
- * Do not set `ErrorBoundary` or `errorElement` on the result.
- */
-type SkuRootLazy = ((...args: Parameters<RouteObjectLazyFunction>) => Promise<SkuRootLazyResult>) | (Omit<RouteObjectLazyObject, 'ErrorBoundary' | 'errorElement'> & {
-  ErrorBoundary?: never;
-  errorElement?: never;
-});
-/**
- * Route object for `routesEntry`. This type is a React Router `RouteObject`
- * with optional `sites`.
- * If you do not set `sites`, the route is on all config sites.
- * If you set `sites`, the route is only on those site names.
- * This is a sku type helper. It is not a React Router re-export.
- * Nested routes can also set `sites`. A child does not get `sites` from a parent.
- * Each route must set its site list.
- * Do not set `ErrorBoundary` or `errorElement` on the top-level html route.
- * Do not set `ErrorBoundary` or `errorElement` through `lazy`.
- * React Router replaces the route `Component` when that route fails.
- * This includes the `<html>` element.
- * Set the boundary on a child route.
- */
-type SkuRouteObject<Site extends string = string> = Omit<SkuChildRouteObject<Site>, 'ErrorBoundary' | 'errorElement' | 'children' | 'lazy'> & {
-  ErrorBoundary?: never;
-  errorElement?: never;
-  lazy?: SkuRootLazy;
-  children?: Array<SkuChildRouteObject<Site>>;
+  children?: Array<SkuRouteObject<Site>>;
 };
 /**
  * Args for optional `routesEntry` `mapRoutePath`.
@@ -236,4 +203,4 @@ declare module 'express-serve-static-core' {
   }
 }
 //#endregion
-export { CommitDocument, CommitDocumentOptions, DocumentAssets, DocumentDestination, JsonValue, MapRoutePath, MapRoutePathArgs, RenderAssets, RenderManifest, RenderOptions, RenderResult, RenderSuccess, type SiteOf, SkuChildRouteObject, SkuClientEntry, SkuClientGetReactContext, SkuClientGetRouterContext, SkuGetLanguage, SkuGetSite, SkuMiddleware, SkuOnHydrate, SkuOnListen, SkuRouteObject, SkuServerEntry, SkuServerGetReactContext, SkuServerGetRouterContext };
+export { CommitDocument, CommitDocumentOptions, DocumentAssets, DocumentDestination, JsonValue, MapRoutePath, MapRoutePathArgs, RenderAssets, RenderManifest, RenderOptions, RenderResult, RenderSuccess, type SiteOf, SkuClientEntry, SkuClientGetReactContext, SkuClientGetRouterContext, SkuGetLanguage, SkuGetSite, SkuMiddleware, SkuOnHydrate, SkuOnListen, SkuRouteObject, SkuServerEntry, SkuServerGetReactContext, SkuServerGetRouterContext };

package/dist/types/types.d.mts

@@ -1,6 +1,8 @@
 import { Plugin, PluginOption } from "vite";
 import { JSX, ReactNode } from "react";
 import { ChunkExtractor } from "@loadable/server";
+import { Server } from "node:http";
+import { Server as Server$1 } from "node:https";
 import { Express, RequestHandler } from "express";
 import { Linter } from "eslint";
 import { TestUserConfig } from "vitest/node";
@@ -27,9 +29,12 @@ interface RenderCallbackParams {
   extractor: ChunkExtractor;
   registerScript?: (script: string) => void;
 }
-interface Server {
+interface Server$2 {
   renderCallback: (params: RenderCallbackParams, ...requestHandlerParams: Parameters<RequestHandler>) => void;
-  onStart?: (app: Express) => void;
+  onStart?: (app: Express, server: {
+    httpServer: Server | Server$1;
+    port: number;
+  }) => void | Promise<void>;
   middleware?: RequestHandler | RequestHandler[];
 }
 interface RenderableRoute {
@@ -597,4 +602,4 @@ type SkuConfig = SkuConfigBase & (({
   bundler: 'vite';
 } & ViteSkuConfig));
 //#endregion
-export { Render, RenderAppProps, RenderCallbackParams, RenderableRoute, Server, SkuConfig, SkuConfigBase, SkuLanguage, SkuProvider, SkuRoute, SkuRouteObject, SkuSiteObject, TransformOutputPathFunctionParams, ViteRenderFunction, ViteSkuConfig, WebpackSkuConfig };
+export { Render, RenderAppProps, RenderCallbackParams, RenderableRoute, Server$2 as Server, SkuConfig, SkuConfigBase, SkuLanguage, SkuProvider, SkuRoute, SkuRouteObject, SkuSiteObject, TransformOutputPathFunctionParams, ViteRenderFunction, ViteSkuConfig, WebpackSkuConfig };

package/dist/webpack/server.mjs

@@ -15,11 +15,20 @@ const { values } = parseArgs({
 	strict: false
 });
 const { port } = values;
-const startCallback = () => {
+let server;
+const startCallback = async () => {
 	console.log("sku SSR server started on port", port);
-	if (typeof onStart === "function") onStart(app);
+	if (typeof onStart !== "function") return;
+	try {
+		await onStart(app, {
+			httpServer: server,
+			port: Number(port)
+		});
+	} catch (error) {
+		console.error(error);
+		process.exit(1);
+	}
 };
-let server;
 if (__SKU_DEV_HTTPS__) {
 	const pems = fs.readFileSync(".ssl/self-signed.pem");
 	server = https.createServer({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sku",
-  "version": "16.3.0-root-errorboundary-type-20260921013810",
+  "version": "16.3.0",
   "description": "Front-end development toolkit, powered by Webpack, Babel, Vanilla Extract and Jest",
   "types": "./dist/index.d.mts",
   "bin": {