@malloy-publisher/server 0.0.196-dev → 0.0.197-dev

package/tests/integration/mcp/mcp_resource.integration.spec.ts

@@ -37,29 +37,29 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
    });
 
    // --- Test Constants ---
-   const homeProjectUri = "malloy://project/malloy-samples";
-   const faaPackageUri = "malloy://project/malloy-samples/package/faa";
+   const homeProjectUri = "malloy://environment/malloy-samples";
+   const faaPackageUri = "malloy://environment/malloy-samples/package/faa";
    const flightsModelUri =
-      "malloy://project/malloy-samples/package/faa/models/flights.malloy";
+      "malloy://environment/malloy-samples/package/faa/models/flights.malloy";
    const FLIGHTS_SOURCE = "flights";
    const FLIGHTS_CARRIER_QUERY = "flights_by_carrier";
    const FLIGHTS_MONTH_VIEW = "flights_by_month";
    const OVERVIEW_NOTEBOOK = "overview.malloynb";
    const nonExistentPackageUri =
-      "malloy://project/malloy-samples/package/nonexistent";
+      "malloy://environment/malloy-samples/package/nonexistent";
    const nonExistentModelUri =
-      "malloy://project/malloy-samples/package/faa/models/nonexistent.malloy";
-   const nonExistentProjectUri = "malloy://project/invalid_project";
+      "malloy://environment/malloy-samples/package/faa/models/nonexistent.malloy";
+   const nonExistentProjectUri = "malloy://environment/invalid_project";
    const invalidUri = "invalid://format";
 
-   const validSourceUri = `malloy://project/malloy-samples/package/faa/models/flights.malloy/sources/${FLIGHTS_SOURCE}`;
-   const validQueryUri = `malloy://project/malloy-samples/package/faa/models/flights.malloy/queries/${FLIGHTS_CARRIER_QUERY}`;
-   const validViewUri = `malloy://project/malloy-samples/package/faa/models/flights.malloy/sources/${FLIGHTS_SOURCE}/views/${FLIGHTS_MONTH_VIEW}`;
-   const validNotebookUri = `malloy://project/malloy-samples/package/faa/notebooks/${OVERVIEW_NOTEBOOK}`;
-   const nonExistentSourceUri = `malloy://project/malloy-samples/package/faa/models/flights.malloy/sources/non_existent_source`;
-   const nonExistentQueryUri = `malloy://project/malloy-samples/package/faa/models/flights.malloy/queries/non_existent_query`;
-   const nonExistentViewUri = `malloy://project/malloy-samples/package/faa/models/flights.malloy/sources/${FLIGHTS_SOURCE}/views/non_existent_view`;
-   const nonExistentNotebookUri = `malloy://project/malloy-samples/package/faa/notebooks/non_existent.malloynb`;
+   const validSourceUri = `malloy://environment/malloy-samples/package/faa/models/flights.malloy/sources/${FLIGHTS_SOURCE}`;
+   const validQueryUri = `malloy://environment/malloy-samples/package/faa/models/flights.malloy/queries/${FLIGHTS_CARRIER_QUERY}`;
+   const validViewUri = `malloy://environment/malloy-samples/package/faa/models/flights.malloy/sources/${FLIGHTS_SOURCE}/views/${FLIGHTS_MONTH_VIEW}`;
+   const validNotebookUri = `malloy://environment/malloy-samples/package/faa/notebooks/${OVERVIEW_NOTEBOOK}`;
+   const nonExistentSourceUri = `malloy://environment/malloy-samples/package/faa/models/flights.malloy/sources/non_existent_source`;
+   const nonExistentQueryUri = `malloy://environment/malloy-samples/package/faa/models/flights.malloy/queries/non_existent_query`;
+   const nonExistentViewUri = `malloy://environment/malloy-samples/package/faa/models/flights.malloy/sources/${FLIGHTS_SOURCE}/views/non_existent_view`;
+   const nonExistentNotebookUri = `malloy://environment/malloy-samples/package/faa/notebooks/non_existent.malloynb`;
 
    describe("client.listResources", () => {
       it(
@@ -217,7 +217,7 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
             );
             if (flightsEntry) {
                expect(flightsEntry.uri).toBe(
-                  "malloy://project/malloy-samples/package/faa/sources/flights.malloy",
+                  "malloy://environment/malloy-samples/package/faa/sources/flights.malloy",
                );
                expect(flightsEntry.metadata).toBeDefined();
                expect(flightsEntry.metadata!.description).toContain(
@@ -232,7 +232,7 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
             );
             if (notebookEntry) {
                expect(notebookEntry.uri).toBe(
-                  "malloy://project/malloy-samples/package/faa/notebooks/aircraft_analysis.malloynb",
+                  "malloy://environment/malloy-samples/package/faa/notebooks/aircraft_analysis.malloynb",
                );
                expect(notebookEntry.metadata).toBeDefined();
                expect(notebookEntry.metadata!.description).toContain(
@@ -505,7 +505,7 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
          // Adjust test to expect the generic "Resource not found" error, as the specific
          // "not a notebook" detail isn't easily surfaced in the standard error format.
          expect(errorPayload.error).toMatch(/Notebook 'overview.malloynb'/);
-         expect(errorPayload.error).toMatch(/project 'malloy-samples'/); // Check project name
+         expect(errorPayload.error).toMatch(/environment 'malloy-samples'/);
          expect(errorPayload.suggestions).toBeDefined();
          expect(Array.isArray(errorPayload.suggestions)).toBe(true);
          expect(errorPayload.suggestions.length).toBeGreaterThan(0);
@@ -532,8 +532,7 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
          expect(errorPayload.error).toMatch(/Resource not found/i);
          // Check for the specific source name in the message
          expect(errorPayload.error).toMatch(/Source 'non_existent_source'/);
-         // Adjust project name expectation
-         expect(errorPayload.error).toMatch(/project 'malloy-samples'/);
+         expect(errorPayload.error).toMatch(/environment 'malloy-samples'/);
          expect(errorPayload.suggestions).toBeDefined();
          expect(Array.isArray(errorPayload.suggestions)).toBe(true);
          expect(errorPayload.suggestions.length).toBeGreaterThan(0);
@@ -558,8 +557,7 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
          expect(errorPayload.error).toMatch(/Resource not found/i);
          // Check for the specific query name in the message
          expect(errorPayload.error).toMatch(/Query 'non_existent_query'/);
-         // Adjust project name expectation
-         expect(errorPayload.error).toMatch(/project 'malloy-samples'/);
+         expect(errorPayload.error).toMatch(/environment 'malloy-samples'/);
          expect(errorPayload.suggestions).toBeDefined();
          expect(Array.isArray(errorPayload.suggestions)).toBe(true);
          expect(errorPayload.suggestions.length).toBeGreaterThan(0);
@@ -584,8 +582,7 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
          expect(errorPayload.error).toMatch(/Resource not found/i);
          // Check for the specific view name in the message
          expect(errorPayload.error).toMatch(/View 'non_existent_view'/);
-         // Adjust project name expectation
-         expect(errorPayload.error).toMatch(/project 'malloy-samples'/);
+         expect(errorPayload.error).toMatch(/environment 'malloy-samples'/);
          expect(errorPayload.suggestions).toBeDefined();
          expect(Array.isArray(errorPayload.suggestions)).toBe(true);
          expect(errorPayload.suggestions.length).toBeGreaterThan(0);
@@ -611,8 +608,7 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
          // Check for the specific notebook name and context in the message
          // Adjust test to expect the generic "Resource not found" error
          expect(errorPayload.error).toMatch(/Notebook 'non_existent.malloynb'/);
-         // Adjust project name expectation
-         expect(errorPayload.error).toMatch(/project 'malloy-samples'/);
+         expect(errorPayload.error).toMatch(/environment 'malloy-samples'/);
          expect(errorPayload.suggestions).toBeDefined();
          expect(Array.isArray(errorPayload.suggestions)).toBe(true);
          expect(errorPayload.suggestions.length).toBeGreaterThan(0);
@@ -623,7 +619,7 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
 
       it("should return structured app error when requesting view from wrong source", async () => {
          if (!env) throw new Error("Test environment not initialized");
-         const wrongSourceUri = `malloy://project/malloy-samples/package/faa/models/flights.malloy/sources/aircraft/views/${FLIGHTS_MONTH_VIEW}`;
+         const wrongSourceUri = `malloy://environment/malloy-samples/package/faa/models/flights.malloy/sources/aircraft/views/${FLIGHTS_MONTH_VIEW}`;
          const result = await mcpClient.readResource({ uri: wrongSourceUri });
          expect(result.isError).toBe(true);
          expect(result.contents).toBeDefined();
@@ -633,22 +629,17 @@ describe.serial("MCP Resource Handlers (E2E Integration)", () => {
          // eslint-disable-next-line @typescript-eslint/no-explicit-any
          const errorPayload = JSON.parse((result.contents![0] as any).text);
          expect(errorPayload.error).toBeDefined();
-         // Adjust expectation: The primary error should be the project not found
          expect(errorPayload.error).toMatch(/Resource not found/i);
-         expect(errorPayload.error).toMatch(/project 'malloy-samples'/);
-         // Remove checks for view/source name in the error for this specific case
+         expect(errorPayload.error).toMatch(/View 'flights_by_month'/);
+         expect(errorPayload.error).toMatch(/source 'aircraft'/);
+         expect(errorPayload.error).toMatch(/environment 'malloy-samples'/);
          expect(errorPayload.suggestions).toBeDefined();
          expect(Array.isArray(errorPayload.suggestions)).toBe(true);
-         expect(errorPayload.suggestions.length).toBeGreaterThan(0);
-
-         // The suggestions come as full sentences, so we check for expected content
          expect(errorPayload.suggestions.length).toBe(3);
          expect(errorPayload.suggestions[0]).toContain(
             "Verify the identifier or URI",
          );
-         expect(errorPayload.suggestions[0]).toContain(
-            "project 'malloy-samples'",
-         );
+         expect(errorPayload.suggestions[0]).toContain("flights_by_month");
          expect(errorPayload.suggestions[0]).toContain("is spelled correctly");
          expect(errorPayload.suggestions[0]).toContain(
             "Check capitalization and path separators",

package/tests/unit/duckdb/attached_databases.test.ts

@@ -12,7 +12,7 @@ import fs from "fs/promises";
 import os from "os";
 import path from "path";
 import type { components } from "../../../src/api";
-import { createProjectConnections } from "../../../src/service/connection";
+import { createEnvironmentConnections } from "../../../src/service/connection";
 
 type ApiConnection = components["schemas"]["Connection"];
 
@@ -355,7 +355,7 @@ describe("DuckDB Attached Databases", () => {
    });
 });
 
-describe("createProjectConnections - DuckDB", () => {
+describe("createEnvironmentConnections - DuckDB", () => {
    const PROJECT_TEST_DIR = path.join(os.tmpdir(), "duckdb-project-tests");
    let createdConnections: Map<string, unknown> = new Map();
 
@@ -402,7 +402,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          const { malloyConnections, apiConnections } =
-            await createProjectConnections(connections, PROJECT_TEST_DIR);
+            await createEnvironmentConnections(connections, PROJECT_TEST_DIR);
 
          createdConnections = malloyConnections;
 
@@ -433,7 +433,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          const { malloyConnections, apiConnections } =
-            await createProjectConnections(connections, PROJECT_TEST_DIR);
+            await createEnvironmentConnections(connections, PROJECT_TEST_DIR);
 
          createdConnections = malloyConnections;
 
@@ -462,7 +462,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          const { malloyConnections, apiConnections } =
-            await createProjectConnections(connections, PROJECT_TEST_DIR);
+            await createEnvironmentConnections(connections, PROJECT_TEST_DIR);
 
          createdConnections = malloyConnections;
 
@@ -502,7 +502,7 @@ describe("createProjectConnections - DuckDB", () => {
             },
          ];
 
-         const { malloyConnections } = await createProjectConnections(
+         const { malloyConnections } = await createEnvironmentConnections(
             connections,
             PROJECT_TEST_DIR,
          );
@@ -533,7 +533,7 @@ describe("createProjectConnections - DuckDB", () => {
             },
          ];
 
-         const { malloyConnections } = await createProjectConnections(
+         const { malloyConnections } = await createEnvironmentConnections(
             connections,
             PROJECT_TEST_DIR,
          );
@@ -591,7 +591,7 @@ describe("createProjectConnections - DuckDB", () => {
             },
          ];
 
-         const { malloyConnections } = await createProjectConnections(
+         const { malloyConnections } = await createEnvironmentConnections(
             connections,
             PROJECT_TEST_DIR,
          );
@@ -624,7 +624,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow();
       });
 
@@ -637,7 +637,7 @@ describe("createProjectConnections - DuckDB", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("DuckDB connection configuration is missing");
       });
 
@@ -658,7 +658,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("Unsupported database type");
       });
 
@@ -682,7 +682,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("service account key required");
       });
 
@@ -704,7 +704,7 @@ describe("createProjectConnections - DuckDB", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("keyId and secret are required");
       });
 
@@ -726,7 +726,7 @@ describe("createProjectConnections - DuckDB", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("accessKeyId and secretAccessKey are required");
       });
 
@@ -748,7 +748,7 @@ describe("createProjectConnections - DuckDB", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("PostgreSQL connection configuration is required");
       });
 
@@ -761,7 +761,7 @@ describe("createProjectConnections - DuckDB", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("DuckLake connection configuration is missing");
       });
 
@@ -785,7 +785,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("username is required");
       });
 
@@ -806,7 +806,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("PostgreSQL connection configuration missing");
       });
 
@@ -836,7 +836,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow(
             "DuckDB attached database names cannot conflict with connection name",
          );
@@ -854,7 +854,7 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("DuckDB connection name cannot be 'duckdb'");
       });
 
@@ -884,10 +884,28 @@ describe("createProjectConnections - DuckDB", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("DuckDB connection name cannot be 'duckdb'");
       });
 
+      it("should throw when DuckDB connection has no attached databases", async () => {
+         const connections: ApiConnection[] = [
+            {
+               name: "no_attached_db",
+               type: "duckdb",
+               duckdbConnection: {
+                  attachedDatabases: [],
+               },
+            },
+         ];
+
+         await expect(
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
+         ).rejects.toThrow(
+            "DuckDB connection must have at least one attached database",
+         );
+      });
+
       it("should throw on unsupported connection type", async () => {
          const connections = [
             {
@@ -897,7 +915,7 @@ describe("createProjectConnections - DuckDB", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("Unsupported connection type");
       });
 
@@ -921,13 +939,13 @@ describe("createProjectConnections - DuckDB", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow();
       });
    });
 });
 
-describe("createProjectConnections - Other Connection Types", () => {
+describe("createEnvironmentConnections - Other Connection Types", () => {
    const PROJECT_TEST_DIR = path.join(
       os.tmpdir(),
       "connection-validation-tests",
@@ -970,7 +988,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("Snowflake connection configuration is missing");
       });
 
@@ -988,7 +1006,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("Snowflake account is required");
       });
 
@@ -1006,7 +1024,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("Snowflake username is required");
       });
 
@@ -1024,7 +1042,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow(
             "Snowflake password or private key or private key path is required",
          );
@@ -1044,7 +1062,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("Snowflake warehouse is required");
       });
    });
@@ -1059,7 +1077,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("Trino connection configuration is missing");
       });
 
@@ -1076,7 +1094,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow(
             'Invalid Trino connection: expected "http://server:port" or "https://server:port"',
          );
@@ -1093,7 +1111,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ] as ApiConnection[];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("MotherDuck connection configuration is missing");
       });
 
@@ -1107,7 +1125,7 @@ describe("createProjectConnections - Other Connection Types", () => {
          ];
 
          await expect(
-            createProjectConnections(connections, PROJECT_TEST_DIR),
+            createEnvironmentConnections(connections, PROJECT_TEST_DIR),
          ).rejects.toThrow("MotherDuck access token is required");
       });
    });

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

@@ -4,7 +4,7 @@ import fs from "fs/promises";
 import path from "path";
 import { components } from "../../../src/api";
 import {
-   createProjectConnections,
+   createEnvironmentConnections,
    deleteDuckLakeConnectionFile,
    testConnectionConfig,
 } from "../../../src/service/connection";
@@ -143,7 +143,7 @@ describe("DuckLake Connection Tests", () => {
             };
 
             const { malloyConnections, apiConnections } =
-               await createProjectConnections(
+               await createEnvironmentConnections(
                   [ducklakeConnection],
                   testProjectPath,
                );
@@ -212,7 +212,7 @@ describe("DuckLake Connection Tests", () => {
                },
             };
 
-            const { malloyConnections } = await createProjectConnections(
+            const { malloyConnections } = await createEnvironmentConnections(
                [ducklakeConnection],
                testProjectPath,
             );
@@ -268,7 +268,7 @@ describe("DuckLake Connection Tests", () => {
                },
             };
 
-            const { malloyConnections } = await createProjectConnections(
+            const { malloyConnections } = await createEnvironmentConnections(
                [ducklakeConnection],
                testProjectPath,
             );
@@ -333,7 +333,7 @@ describe("DuckLake Connection Tests", () => {
                },
             };
 
-            const { malloyConnections } = await createProjectConnections(
+            const { malloyConnections } = await createEnvironmentConnections(
                [ducklakeConnection],
                testProjectPath,
             );
@@ -400,7 +400,7 @@ describe("DuckLake Connection Tests", () => {
                },
             };
 
-            const { malloyConnections } = await createProjectConnections(
+            const { malloyConnections } = await createEnvironmentConnections(
                [ducklakeConnection],
                testProjectPath,
             );
@@ -470,7 +470,7 @@ describe("DuckLake Connection Tests", () => {
                },
             };
 
-            const { malloyConnections } = await createProjectConnections(
+            const { malloyConnections } = await createEnvironmentConnections(
                [ducklakeConnection],
                testProjectPath,
             );
@@ -529,7 +529,7 @@ describe("DuckLake Connection Tests", () => {
                },
             };
 
-            const { malloyConnections } = await createProjectConnections(
+            const { malloyConnections } = await createEnvironmentConnections(
                [ducklakeConnection],
                testProjectPath,
             );
@@ -553,7 +553,7 @@ describe("DuckLake Connection Tests", () => {
    describe("Error Handling", () => {
       it("should throw error if DuckLake catalog connection is missing", async () => {
          await expect(
-            createProjectConnections(
+            createEnvironmentConnections(
                [
                   {
                      name: "ducklake_no_catalog",
@@ -576,7 +576,7 @@ describe("DuckLake Connection Tests", () => {
 
       it("should throw error if DuckLake storage bucketUrl is missing", async () => {
          await expect(
-            createProjectConnections(
+            createEnvironmentConnections(
                [
                   {
                      name: "ducklake_no_bucket",
@@ -607,7 +607,7 @@ describe("DuckLake Connection Tests", () => {
 
       it("should throw error if DuckLake connection config is missing", async () => {
          await expect(
-            createProjectConnections(
+            createEnvironmentConnections(
                [
                   {
                      name: "ducklake_missing_config",
@@ -653,19 +653,21 @@ describe("DuckLake Connection Tests", () => {
          };
 
          // Create connection twice - second should handle already attached gracefully
-         const { malloyConnections: conn1 } = await createProjectConnections(
-            [ducklakeConnection],
-            testProjectPath,
-         );
+         const { malloyConnections: conn1 } =
+            await createEnvironmentConnections(
+               [ducklakeConnection],
+               testProjectPath,
+            );
          const connection1 = conn1.get(
             "ducklake_duplicate_test",
          ) as DuckDBConnection;
          createdConnections.push(connection1);
 
-         const { malloyConnections: conn2 } = await createProjectConnections(
-            [ducklakeConnection],
-            testProjectPath,
-         );
+         const { malloyConnections: conn2 } =
+            await createEnvironmentConnections(
+               [ducklakeConnection],
+               testProjectPath,
+            );
          const connection2 = conn2.get(
             "ducklake_duplicate_test",
          ) as DuckDBConnection;
@@ -714,7 +716,7 @@ describe("DuckLake Connection Tests", () => {
                },
             };
 
-            const { malloyConnections } = await createProjectConnections(
+            const { malloyConnections } = await createEnvironmentConnections(
                [ducklakeConnection],
                testProjectPath,
             );
@@ -771,7 +773,7 @@ describe("DuckLake Connection Tests", () => {
             },
          };
 
-         const { malloyConnections } = await createProjectConnections(
+         const { malloyConnections } = await createEnvironmentConnections(
             [ducklakeConnection],
             testProjectPath,
          );
@@ -903,7 +905,7 @@ describe("DuckLake Connection Tests", () => {
                return;
             }
 
-            const { apiConnections } = await createProjectConnections(
+            const { apiConnections } = await createEnvironmentConnections(
                [
                   {
                      name: "ducklake_attrs_test",

package/tests/unit/mcp/prompt_happy.test.ts

@@ -3,13 +3,13 @@ import { describe, it, expect } from "bun:test";
 import { promptHandlerMap } from "../../../src/mcp/prompts/handlers";
 
 describe("Prompt Handlers – happy paths", () => {
-   const projectStore = undefined; // no ProjectStore needed for minimal tests
+   const environmentStore = undefined; // no EnvironmentStore needed for minimal tests
 
    it("explain handler returns text message", async () => {
       const handler = promptHandlerMap["explain-malloy-query@1.0.0"];
       const res = await handler(
          { query_code: "run: t -> { aggregate: c is count() }" },
-         projectStore as any,
+         environmentStore as any,
       );
       expect(res.messages.length).toBe(1);
       expect(res.messages[0].role).toBe("user");
@@ -21,9 +21,9 @@ describe("Prompt Handlers – happy paths", () => {
       const res = await handler(
          {
             natural_language_goal: "total sales by day",
-            target_model_uri: "malloy://project/p/pkg/models/m.malloy",
+            target_model_uri: "malloy://environment/e/pkg/models/m.malloy",
          },
-         projectStore as any,
+         environmentStore as any,
       );
       expect(res.messages[0].role).toBe("user");
    });
@@ -33,9 +33,9 @@ describe("Prompt Handlers – happy paths", () => {
       const res = await handler(
          {
             sql_query: "SELECT 1",
-            target_model_uri: "malloy://project/p/pkg/models/m.malloy",
+            target_model_uri: "malloy://environment/e/pkg/models/m.malloy",
          },
-         projectStore as any,
+         environmentStore as any,
       );
       expect(res.messages[0].content.type).toBe("text");
    });
@@ -43,8 +43,8 @@ describe("Prompt Handlers – happy paths", () => {
    it("summarize handler returns message", async () => {
       const handler = promptHandlerMap["summarize-malloy-model@1.0.0"];
       const res = await handler(
-         { model_uri: "malloy://project/p/pkg/models/m.malloy" },
-         projectStore as any,
+         { model_uri: "malloy://environment/e/pkg/models/m.malloy" },
+         environmentStore as any,
       );
       expect(res.messages[0].content.type).toBe("text");
    });

package/dist/app/assets/HomePage-DbZS0N7G.js

@@ -1 +0,0 @@
-import{d as o,j as a,i as e}from"./index-Bu0ub036.js";function i(){const t=o();return a.jsx(e,{onClickProject:t})}export{i as default};