@malloy-publisher/server 0.10.5 → 0.10.6

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.6";
 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.6";
 var init_package = () => {};
 
 // src/version.ts
@@ -268347,23 +268347,12 @@ var require_bcrypt_pbkdf = __commonJS((exports, module) => {
   };
 });
 
-// ../../node_modules/cpu-features/build/Release/cpufeatures.node
-var require_cpufeatures = __commonJS((exports, module) => {
-  module.exports = __require("./cpufeatures-gknxfsk5.node");
-});
-
-// ../../node_modules/cpu-features/lib/index.js
-var require_lib5 = __commonJS((exports, module) => {
-  var binding = require_cpufeatures();
-  module.exports = binding.getCPUInfo;
-});
-
 // ../../node_modules/ssh2/lib/protocol/constants.js
 var require_constants12 = __commonJS((exports, module) => {
   var crypto3 = __require("crypto");
   var cpuInfo;
   try {
-    cpuInfo = require_lib5()();
+    cpuInfo = (()=>{throw new Error("Cannot require module "+"cpu-features");})()();
   } catch {}
   var { bindingAvailable, CIPHER_INFO, MAC_INFO } = require_crypto();
   var eddsaSupported = (() => {
@@ -283006,7 +282995,7 @@ var require_keygen = __commonJS((exports, module) => {
 });
 
 // ../../node_modules/ssh2/lib/index.js
-var require_lib6 = __commonJS((exports, module) => {
+var require_lib5 = __commonJS((exports, module) => {
   var {
     AgentProtocol,
     BaseAgent,
@@ -283189,7 +283178,7 @@ function openSshProxy(ssh, target) {
 var import_ssh2, ALLOW_UNVERIFIED_SSH_HOST_KEY_ENV = "PUBLISHER_ALLOW_UNVERIFIED_SSH_HOST_KEY", TRUTHY_ENV_VALUES, SSH_CONNECT_TIMEOUT_MS = 15000, SSH_KEEPALIVE_INTERVAL_MS = 15000;
 var init_proxy = __esm(() => {
   init_logger();
-  import_ssh2 = __toESM(require_lib6(), 1);
+  import_ssh2 = __toESM(require_lib5(), 1);
   TRUTHY_ENV_VALUES = new Set(["1", "true", "yes", "on"]);
 });
 
@@ -284261,7 +284250,7 @@ var init_connection_config = __esm(() => {
 });
 
 // ../../node_modules/webidl-conversions/lib/index.js
-var require_lib7 = __commonJS((exports, module) => {
+var require_lib6 = __commonJS((exports, module) => {
   var conversions = {};
   module.exports = conversions;
   function sign(x2) {
@@ -285818,7 +285807,7 @@ var require_URL_impl = __commonJS((exports) => {
 
 // ../../node_modules/whatwg-url/lib/URL.js
 var require_URL = __commonJS((exports, module) => {
-  var conversions = require_lib7();
+  var conversions = require_lib6();
   var utils = require_utils95();
   var Impl = require_URL_impl();
   var impl = utils.implSymbol;
@@ -286010,7 +285999,7 @@ var require_public_api2 = __commonJS((exports) => {
 });
 
 // ../../node_modules/node-fetch/lib/index.js
-var require_lib8 = __commonJS((exports, module) => {
+var require_lib7 = __commonJS((exports, module) => {
   Object.defineProperty(exports, "__esModule", { value: true });
   function _interopDefault(ex) {
     return ex && typeof ex === "object" && "default" in ex ? ex["default"] : ex;
@@ -288483,7 +288472,7 @@ var require_gaxios2 = __commonJS((exports) => {
   exports.Gaxios = undefined;
   var extend_1 = __importDefault(require_extend());
   var https_1 = __require("https");
-  var node_fetch_1 = __importDefault(require_lib8());
+  var node_fetch_1 = __importDefault(require_lib7());
   var querystring_1 = __importDefault(__require("querystring"));
   var is_stream_1 = __importDefault(require_is_stream());
   var url_1 = __require("url");
@@ -298539,7 +298528,7 @@ var require_stream_events = __commonJS((exports, module) => {
 var require_src140 = __commonJS((exports) => {
   Object.defineProperty(exports, "__esModule", { value: true });
   exports.teenyRequest = exports.RequestError = undefined;
-  var node_fetch_1 = require_lib8();
+  var node_fetch_1 = require_lib7();
   var stream_1 = __require("stream");
   var uuid = require_dist10();
   var agents_1 = require_agents();
@@ -349861,6 +349850,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 +350196,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 +377205,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 +377252,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 +377262,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.6",
    "main": "dist/server.mjs",
    "bin": {
       "malloy-publisher": "dist/server.mjs"