@malloy-publisher/server 0.0.198-dev6 → 0.0.199

package/dist/server.mjs

@@ -237482,6 +237482,17 @@ function initializeMcpServer(environmentStore) {
 init_errors();
 init_logger();
 var import_body_parser = __toESM(require_body_parser(), 1);
+
+// src/query_param_utils.ts
+function normalizeQueryArray(value) {
+  if (value === undefined || value === null)
+    return;
+  if (Array.isArray(value))
+    return value.map(String);
+  return [String(value)];
+}
+
+// src/server-old.ts
 var LEGACY_API_PREFIX = "/api/v0";
 function remapMaterializationResponse(mat) {
   if (!mat || typeof mat !== "object")
@@ -238802,13 +238813,6 @@ class PackageMemoryGovernor {
 }
 
 // src/server.ts
-function normalizeQueryArray(value) {
-  if (value === undefined || value === null)
-    return;
-  if (Array.isArray(value))
-    return value.map(String);
-  return [String(value)];
-}
 function parseArgs() {
   const args = process.argv.slice(2);
   let sawServerRoot = false;

package/package.json

@@ -1,7 +1,7 @@
 {
    "name": "@malloy-publisher/server",
    "description": "Malloy Publisher Server",
-   "version": "0.0.198-dev6",
+   "version": "0.0.199",
    "main": "dist/server.mjs",
    "bin": {
       "malloy-publisher": "dist/server.mjs"

package/src/query_param_utils.ts

@@ -0,0 +1,18 @@
+/**
+ * Express query-param normalization helpers.
+ *
+ * Kept in a standalone file (no transitive imports) so unit specs can
+ * exercise them without dragging in `server.ts` — which transitively
+ * constructs `EnvironmentStore` and kicks off an async storage init
+ * (clone of `malloy-samples`, package downloads, ...). When that init
+ * runs in a `bun test` process it races the test runner's exit and
+ * leaves a partially-populated `publisher_data/` on disk, which the
+ * next process (integration tests) then trips over.
+ */
+
+/** Normalize an Express query param into a string[] or undefined. */
+export function normalizeQueryArray(value: unknown): string[] | undefined {
+   if (value === undefined || value === null) return undefined;
+   if (Array.isArray(value)) return value.map(String);
+   return [String(value)];
+}

package/src/server-old.ts

@@ -41,7 +41,7 @@ import {
    NotImplementedError,
 } from "./errors";
 import { logger } from "./logger";
-import { normalizeQueryArray } from "./server";
+import { normalizeQueryArray } from "./query_param_utils";
 import { EnvironmentStore } from "./service/environment_store";
 
 const LEGACY_API_PREFIX = "/api/v0";

package/src/server.ts

@@ -42,14 +42,10 @@ import { registerLegacyRoutes } from "./server-old";
 import { EnvironmentStore } from "./service/environment_store";
 import { ManifestService } from "./service/manifest_service";
 import { MaterializationService } from "./service/materialization_service";
+import { normalizeQueryArray } from "./query_param_utils";
 import { PackageMemoryGovernor } from "./service/package_memory_governor";
 
-/** Normalize an Express query param into a string[] or undefined. */
-export function normalizeQueryArray(value: unknown): string[] | undefined {
-   if (value === undefined || value === null) return undefined;
-   if (Array.isArray(value)) return value.map(String);
-   return [String(value)];
-}
+export { normalizeQueryArray } from "./query_param_utils";
 
 // Parse command line arguments
 function parseArgs() {
@@ -123,10 +119,12 @@ function parseArgs() {
    // Zero-config invocation (`npx @malloy-publisher/server`) opts in to
    // the bundled DuckDB-only sample config so the Quick Start works
    // without any flags. Any explicit --server_root or --config disables
-   // this — the user told us where to look. Skip in NODE_ENV=test so
-   // specs that import this module for utility helpers (e.g.
-   // db_utils.spec.ts -> normalizeQueryArray) don't get the bundled
-   // default leaked into their EnvironmentStore construction.
+   // this — the user told us where to look. Skip in NODE_ENV=test as a
+   // belt-and-suspenders so any spec that ends up evaluating this
+   // module doesn't accidentally pin the EnvironmentStore to the
+   // bundled malloy-samples config; query-param helpers have been
+   // moved to `./query_param_utils` precisely so unit specs no longer
+   // need to import this module at all.
    if (!sawServerRoot && !sawConfig && process.env.NODE_ENV !== "test") {
       process.env.PUBLISHER_USE_BUNDLED_DEFAULT = "true";
    }

package/src/service/db_utils.spec.ts

@@ -12,7 +12,7 @@ mock.module("@google-cloud/bigquery", () => ({
 }));
 
 import { Connection } from "@malloydata/malloy";
-import { normalizeQueryArray } from "../server";
+import { normalizeQueryArray } from "../query_param_utils";
 import {
    extractErrorDataFromError,
    getSchemasForConnection,