@n6k.io/build 0.0.1 → 0.0.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@n6k.io/build",
-  "version": "0.0.1",
+  "version": "0.0.3",
   "private": false,
   "description": "Static build pipeline (.mdx + ts/tsx/js/jsx → esbuild bundle → HTML) for n6k apps",
   "homepage": "https://github.com/n6k-io/build#readme",
@@ -81,7 +81,7 @@
   },
   "devDependencies": {
     "@eslint/js": "^10.0.1",
-    "@n6k.io/db": "link:@n6k.io/db",
+    "@n6k.io/db": "^0.5.0",
     "@storybook/react": "^10.4.4",
     "@tanstack/react-query": "^5.101.0",
     "@types/bun": "latest",

package/src/util/logger.ts

@@ -11,31 +11,14 @@ export function initLogger(debug: boolean) {
   }
 }
 
-// pino does not expose its underlying stream in its public types, so narrow to
-// the one method we need (`end`) via a runtime type guard rather than a cast.
-function hasFlushableStream(
-  l: unknown,
-): l is { stream: { end: (cb: () => void) => void } } {
-  return (
-    typeof l === "object" &&
-    l !== null &&
-    "stream" in l &&
-    typeof l.stream === "object" &&
-    l.stream !== null &&
-    "end" in l.stream &&
-    typeof l.stream.end === "function"
-  );
-}
-
+// Drain pino's buffered logs before the process exits. pino keeps its output
+// stream on an internal Symbol (not a public `.stream` property), so there is
+// nothing to `.end()` directly — `logger.flush(cb)` is the supported drain: it
+// invokes the callback once the underlying stream (including the pino-pretty
+// worker thread) has flushed.
 export function flushLogger(): Promise<void> {
   return new Promise((resolve) => {
-    logger.flush(() => {
-      if (hasFlushableStream(logger)) {
-        logger.stream.end(() => resolve());
-      } else {
-        resolve();
-      }
-    });
+    logger.flush(() => resolve());
   });
 }