@malloy-publisher/server 0.10.5 → 0.10.7

package/dist/admin-api-doc.yaml

@@ -234,7 +234,10 @@ paths:
         the bytes are rewritten as the catalog's own Parquet, never registered
         in place and never left as rows in the catalog database, so an
         encrypted catalog stays encrypted. One DuckLake transaction:
-        a reader sees the old table or the new one. `201` when the table is new,
+        a reader sees the old table or the new one. A file the reader refuses
+        (not the format it claims, a value it cannot convert) is `400
+        INVALID_REQUEST` carrying the reader's own message, with the object
+        named by its file name. `201` when the table is new,
         `200` when it replaced one; with `If-None-Match: *` an existing table is
         refused with `412 TABLE_EXISTS`. Synchronous: it answers when the table
         is written.

package/dist/instrumentation.mjs

@@ -164831,7 +164831,7 @@ var init_telemetry = __esm(() => {
 });
 
 // package.json
-var version = "0.10.5";
+var version = "0.10.7";
 var init_package = () => {};
 
 // src/version.ts

package/dist/server.mjs

@@ -164832,7 +164832,7 @@ var init_telemetry = __esm(() => {
 });
 
 // package.json
-var version = "0.10.5";
+var version = "0.10.7";
 var init_package = () => {};
 
 // src/version.ts
@@ -349861,6 +349861,9 @@ function carriesAuthorSql(def) {
     if (record.type === "sql_select" || record.node === "genericSQLExpr") {
       return true;
     }
+    if (record.node === "function_call") {
+      return Object.entries(record).some(([key, child]) => key !== "overload" && walk2(child, depth + 1));
+    }
     if (record.type === "table" && typeof record.tablePath === "string" && !CATALOG_TABLE_PATH.test(record.tablePath)) {
       return true;
     }
@@ -350204,7 +350207,7 @@ async function buildSameCatalogIntoStorage(params) {
   const { destinationName, destinationConnection, physicalTableName } = params;
   const alias = params.sourceConnection.name || destinationName;
   if (params.sourceCarriesAuthorSql !== false) {
-    throw new BadRequestError(`Cannot materialize a source that reads storage destination ` + `'${destinationName}' and carries SQL its author wrote (an .sql() source ` + `or a sql_* expression): a build on the catalog runs only Malloy's own SQL. ` + `Express it in Malloy, or persist it without storage=.`);
+    throw new BadRequestError(`Cannot materialize a source that reads storage destination ` + `'${destinationName}' and carries SQL its author wrote (an .sql() source ` + `or a sql_* expression) or reads a file or URL as a table: a build on the catalog ` + `runs only Malloy's own SQL over the catalog's tables. Express it that way. ` + `Dropping storage= does not help when the source's own connection is the catalog.`);
   }
   const { session, dispose, workDir } = createIsolatedBuildSession(`build_${destinationName}`);
   try {
@@ -377213,11 +377216,21 @@ async function loadStorageTable(params) {
     if (params.format === "xlsx") {
       await installAndLoadExtension(session, "excel");
     }
-    return await writeTableAs(session, params, reader);
+    return await writeTableAs(session, params, reader, (error) => readerRefusal(error, params.uri));
   } finally {
     await dispose();
   }
 }
+function readerRefusal(error, uri) {
+  if (!(error instanceof Error))
+    return null;
+  if (!/^(Invalid Input|Conversion) Error:/.test(error.message))
+    return null;
+  logInternalFailure("Load refused by the file reader", error, "warn");
+  const fileName = path24.posix.basename(uri);
+  const words2 = error.message.split(/\n\s*LINE \d+:/)[0].split(uri).join(fileName).trim();
+  return new BadRequestError(words2);
+}
 async function copyStorageTable(params) {
   const { destinationName, destinationConnection, source } = params;
   for (const conn of [destinationConnection, source.connection]) {
@@ -377250,7 +377263,7 @@ async function copyStorageTable(params) {
     await dispose();
   }
 }
-async function writeTableAs(session, params, from) {
+async function writeTableAs(session, params, from, refused) {
   const { destinationName, schemaName, tableName } = params;
   await session.runSQL("SET ducklake_default_data_inlining_row_limit = 0");
   const target = [destinationName, schemaName, tableName].map((part) => quoteIdentifier(part, "duckdb")).join(".");
@@ -377260,7 +377273,7 @@ async function writeTableAs(session, params, from) {
     if (params.mode === "create" && error instanceof Error && /already exists/i.test(error.message)) {
       throw new ConflictError(`Table '${schemaName}.${tableName}' already exists in '${destinationName}'.`);
     }
-    throw error;
+    throw refused?.(error) ?? error;
   }
   await session.runSQL(`CALL ducklake_flush_inlined_data('${escapeSQL(destinationName)}', schema_name => '${escapeSQL(schemaName)}', table_name => '${escapeSQL(tableName)}')`);
   const counted = rows(await session.runSQL(`SELECT count(*) AS n FROM ${target}`));

package/package.json

@@ -19,7 +19,7 @@
       "snowflake"
    ],
    "homepage": "https://docs.malloydata.dev/documentation/user_guides/publishing/mcp_agents",
-   "version": "0.10.5",
+   "version": "0.10.7",
    "main": "dist/server.mjs",
    "bin": {
       "malloy-publisher": "dist/server.mjs"