@malloy-publisher/server 0.0.175 → 0.0.177

package/src/service/project.ts

@@ -121,7 +121,7 @@ export class Project {
       projectPath: string,
       connections: ApiConnection[],
    ): Promise<Project> {
-      if (!(await fs.promises.stat(projectPath)).isDirectory()) {
+      if (!(await fs.promises.stat(projectPath))?.isDirectory()) {
          throw new ProjectNotFoundError(
             `Project path ${projectPath} not found`,
          );
@@ -383,7 +383,7 @@ export class Project {
             .access(packagePath)
             .then(() => true)
             .catch(() => false)) ||
-         !(await fs.promises.stat(packagePath)).isDirectory()
+         !(await fs.promises.stat(packagePath))?.isDirectory()
       ) {
          throw new PackageNotFoundError(`Package ${packageName} not found`);
       }

package/src/service/project_store.ts

@@ -569,22 +569,10 @@ export class ProjectStore {
    private async cleanupAndCreatePublisherPath() {
       const reInit = process.env.INITIALIZE_STORAGE === "true";
 
-      // Ensure serverRootPath exists and is a directory
-      try {
-         const stats = await fs.promises.stat(this.serverRootPath);
-         if (!stats.isDirectory()) {
-            throw new Error(
-               `Server root path ${this.serverRootPath} exists but is not a directory`,
-            );
-         }
-      } catch (error) {
-         if ((error as NodeJS.ErrnoException).code === "ENOENT") {
-            // Directory doesn't exist, create it
-            await fs.promises.mkdir(this.serverRootPath, { recursive: true });
-         } else {
-            throw error;
-         }
-      }
+      // Ensure serverRootPath exists as a directory (mkdir recursive is a
+      // no-op when the directory already exists and avoids a Bun-on-Windows
+      // bug where fs.promises.stat resolves to undefined instead of throwing)
+      await fs.promises.mkdir(this.serverRootPath, { recursive: true });
 
       if (reInit) {
          const uploadDocsPath = path.join(
@@ -1233,9 +1221,8 @@ export class ProjectStore {
       if (projectPath.endsWith(".zip")) {
          projectPath = await this.unzipProject(projectPath);
       }
-      const projectDirExists = (
-         await fs.promises.stat(projectPath)
-      ).isDirectory();
+      const projectDirExists =
+         (await fs.promises.stat(projectPath))?.isDirectory() ?? false;
       if (projectDirExists) {
          await fs.promises.rm(absoluteTargetPath, {
             recursive: true,

package/tests/unit/ducklake/ducklake.test.ts

@@ -10,7 +10,7 @@ import {
 } from "../../../src/service/connection";
 import {
    getSchemasForConnection,
-   getTablesForSchema,
+   listTablesForSchema,
 } from "../../../src/service/db_utils";
 
 type ApiConnection = components["schemas"]["Connection"];
@@ -421,8 +421,8 @@ describe("DuckLake Connection Tests", () => {
             }
 
             // Test table listing for first schema
-            const schemaName = schemas[0].name;
-            const tables = await getTablesForSchema(
+            const schemaName = schemas[0].name!;
+            const tables = await listTablesForSchema(
                ducklakeConnection,
                schemaName,
                connection,