@databrainhq/mcp-server 0.3.0-rc.5 → 0.3.0-rc.6

package/dist/apps/server.js

@@ -1,9 +1,9 @@
 #!/usr/bin/env node
 import {
   createChartAppServer
-} from "../chunk-4DYPD273.js";
-import "../chunk-5IQ7MVUH.js";
-import "../chunk-PRE4CGOV.js";
+} from "../chunk-7G5QBEPC.js";
+import "../chunk-Z2SF3K6P.js";
+import "../chunk-SKU37UVJ.js";
 export {
   createChartAppServer
 };

package/dist/apps/stdio.js

@@ -1,12 +1,12 @@
 #!/usr/bin/env node
 import {
   loadConfig
-} from "../chunk-V64ZQINV.js";
+} from "../chunk-SDFC7PKT.js";
 import {
   createChartAppServer
-} from "../chunk-4DYPD273.js";
-import "../chunk-5IQ7MVUH.js";
-import "../chunk-PRE4CGOV.js";
+} from "../chunk-7G5QBEPC.js";
+import "../chunk-Z2SF3K6P.js";
+import "../chunk-SKU37UVJ.js";
 
 // src/apps/stdio.ts
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";

package/dist/{chunk-4DYPD273.js → chunk-7G5QBEPC.js}

@@ -22,13 +22,13 @@ import {
   saveDraftAsDashboard,
   viewSchema,
   workspacesForDatamart
-} from "./chunk-5IQ7MVUH.js";
+} from "./chunk-Z2SF3K6P.js";
 import {
   DatabrainApiError,
   DatabrainClient,
   MCP_SERVER_VERSION,
   toolContext
-} from "./chunk-PRE4CGOV.js";
+} from "./chunk-SKU37UVJ.js";
 
 // src/apps/server.ts
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
@@ -513,4 +513,4 @@ function createChartAppServer(config, backend) {
 export {
   createChartAppServer
 };
-//# sourceMappingURL=chunk-4DYPD273.js.map
+//# sourceMappingURL=chunk-7G5QBEPC.js.map

package/dist/{chunk-V64ZQINV.js → chunk-SDFC7PKT.js}

@@ -2,7 +2,7 @@
 import {
   oauthConfigSchema,
   oauthHttpsUrl
-} from "./chunk-PRE4CGOV.js";
+} from "./chunk-SKU37UVJ.js";
 
 // src/utils/config.ts
 import { z, ZodError } from "zod";
@@ -72,4 +72,4 @@ export {
   DEFAULT_POSTHOG_HOST,
   loadConfig
 };
-//# sourceMappingURL=chunk-V64ZQINV.js.map
+//# sourceMappingURL=chunk-SDFC7PKT.js.map

package/dist/{chunk-PRE4CGOV.js → chunk-SKU37UVJ.js}

@@ -2,18 +2,20 @@
 
 // src/utils/errors.ts
 var DatabrainApiError = class extends Error {
-  constructor(status, code, message, details) {
+  constructor(status, code, message, details, serverHint, retryable) {
     super(message);
     this.status = status;
     this.code = code;
     this.details = details;
+    this.serverHint = serverHint;
+    this.retryable = retryable;
     this.name = "DatabrainApiError";
   }
   toToolResponse() {
     const parts = [`Error: ${this.message}`];
     const detailDump = this.formatDetails();
     if (detailDump) parts.push(detailDump);
-    const hint = this.hint();
+    const hint = this.serverHint ? `Hint: ${this.serverHint}` : this.hint();
     if (hint) parts.push(hint);
     return {
       content: [{ type: "text", text: parts.join("\n\n") }],
@@ -29,7 +31,7 @@ var DatabrainApiError = class extends Error {
   formatDetails() {
     if (this.status < 400 || this.status >= 500) return "";
     if (!this.details || typeof this.details !== "object") return "";
-    const skipKeys = /* @__PURE__ */ new Set(["message", "detail", "error", "code"]);
+    const skipKeys = /* @__PURE__ */ new Set(["message", "detail", "error", "code", "data", "hint", "retryable"]);
     const extra = {};
     for (const [k, v] of Object.entries(this.details)) {
       if (skipKeys.has(k)) continue;
@@ -44,6 +46,9 @@ var DatabrainApiError = class extends Error {
     }
   }
   hint() {
+    if (this.code === "UNAUTHORIZED_ORIGIN") {
+      return "Hint: The embed origin is not on the allowed domains list. Add the domain to the allowed domains in the Databrain settings; the token itself is fine.";
+    }
     if (this.status === 401 || this.status === 403) {
       if (this.code === "OAUTH_API_ERROR") {
         return "Hint: Check that the backend accepts the configured OAuth issuer and resource, and that the service principal has permission for this operation. Static fallback applies only to token acquisition failures.";
@@ -123,7 +128,7 @@ var logger = {
 // package.json
 var package_default = {
   name: "@databrainhq/mcp-server",
-  version: "0.3.0-rc.5",
+  version: "0.3.0-rc.6",
   description: "MCP server for Databrain embedded analytics - connect databases, create datamarts, configure embeds, and generate frontend code",
   type: "module",
   bin: {
@@ -214,6 +219,29 @@ var TtlCache = class {
   }
 };
 
+// src/client/error-body.ts
+var isRecord = (v) => typeof v === "object" && v !== null && !Array.isArray(v);
+var str = (v) => typeof v === "string" && v.trim() ? v : void 0;
+var fromEnvelope = (v) => {
+  if (!isRecord(v)) return void 0;
+  const out = {
+    code: str(v.code),
+    message: str(v.message) ?? str(v.detail),
+    hint: str(v.hint),
+    retryable: typeof v.retryable === "boolean" ? v.retryable : void 0,
+    details: v.details
+  };
+  return out.code || out.message || out.hint ? out : void 0;
+};
+function parseErrorBody(body) {
+  if (!isRecord(body)) return {};
+  const nested = fromEnvelope(isRecord(body.data) ? body.data.error : void 0) ?? fromEnvelope(body.error) ?? fromEnvelope(body);
+  if (nested) {
+    return { ...nested, message: nested.message ?? str(body.error) };
+  }
+  return { message: str(body.error) };
+}
+
 // src/utils/tool-context.ts
 import { AsyncLocalStorage } from "async_hooks";
 var toolContext = new AsyncLocalStorage();
@@ -531,14 +559,17 @@ var DatabrainClient = class {
             parsed = JSON.parse(errorBody);
           } catch {
           }
-          const errorMsg = parsed.message || parsed.detail || (typeof parsed.error === "string" ? parsed.error : JSON.stringify(parsed.error)) || `API request failed with status ${response.status}`;
+          const envelope = parseErrorBody(parsed);
+          const errorMsg = envelope.message || `API request failed with status ${response.status}`;
           const error = new DatabrainApiError(
             response.status,
-            parsed.code || `HTTP_${response.status}`,
+            envelope.code || `HTTP_${response.status}`,
             errorMsg,
-            parsed
+            parsed,
+            envelope.hint,
+            envelope.retryable
           );
-          const isTransient = response.status === 429 || response.status >= 500 && !NON_TRANSIENT_PATTERNS.some((p) => errorMsg.includes(p));
+          const isTransient = envelope.retryable !== void 0 ? envelope.retryable && (response.status === 429 || response.status >= 500) : response.status === 429 || response.status >= 500 && !NON_TRANSIENT_PATTERNS.some((p) => errorMsg.includes(p));
           if (isTransient && attempt < MAX_RETRIES) {
             const delay = Math.min(1e3 * Math.pow(2, attempt - 1), 8e3);
             logger.warn(`Retrying in ${delay}ms (attempt ${attempt}/${MAX_RETRIES})`, {
@@ -669,7 +700,7 @@ var V2_OPTION_EXTRA_KEYS = {
 var TRANSLATION_KEYS = ["name", "description", "slots", "options"];
 var SLOT_TRANSLATION_KEYS = ["label", "description"];
 var OPTION_TRANSLATION_KEYS = ["label", "group", "choices"];
-var isRecord = (value) => value !== null && typeof value === "object" && !Array.isArray(value) && Object.getPrototypeOf(value) === Object.prototype;
+var isRecord2 = (value) => value !== null && typeof value === "object" && !Array.isArray(value) && Object.getPrototypeOf(value) === Object.prototype;
 var isId = (value) => typeof value === "string" && ID_PATTERN.test(value) && !FORBIDDEN_IDS.includes(value);
 var isText = (value, max = MAX_TEXT) => typeof value === "string" && value.length > 0 && value.length <= max;
 var isTranslationText = (value) => typeof value === "string" && value.length <= MAX_TEXT;
@@ -704,7 +735,7 @@ var Collector = class {
   }
 };
 function validateSlot(c, value, path, v2) {
-  if (!isRecord(value)) {
+  if (!isRecord2(value)) {
     c.add("manifest.slot.invalid", path, "A slot must be an object.");
     return;
   }
@@ -734,7 +765,7 @@ function validateSlot(c, value, path, v2) {
   }
 }
 function validateOption(c, value, path, v2) {
-  if (!isRecord(value)) {
+  if (!isRecord2(value)) {
     c.add("manifest.option.invalid", path, "An option must be an object.");
     return;
   }
@@ -799,7 +830,7 @@ function validateOption(c, value, path, v2) {
     case "select": {
       const choices = value.choices;
       const choicesOk = Array.isArray(choices) && choices.length > 0 && choices.length <= MAX_CHOICES && choices.every(
-        (choice) => isRecord(choice) && Object.keys(choice).length === 2 && isId(choice.value) && isText(choice.label)
+        (choice) => isRecord2(choice) && Object.keys(choice).length === 2 && isId(choice.value) && isText(choice.label)
       ) && new Set(choices.map((choice) => choice.value)).size === choices.length;
       if (!choicesOk) {
         c.add("manifest.option.choices", `${path}/choices`, `choices is a list of 1 to ${MAX_CHOICES} distinct { value, label } pairs; a value follows the id rule.`);
@@ -821,7 +852,7 @@ function validateOption(c, value, path, v2) {
 }
 function validateTranslations(c, value, slots, options) {
   const path = "/translations";
-  if (!isRecord(value)) {
+  if (!isRecord2(value)) {
     c.add("manifest.translations.invalid", path, "translations is an object keyed by language code.");
     return;
   }
@@ -831,7 +862,7 @@ function validateTranslations(c, value, slots, options) {
       c.add("manifest.translations.language", langPath, `"${language}" is not a language code like "fr" or "pt-BR".`);
       continue;
     }
-    if (!isRecord(entry)) {
+    if (!isRecord2(entry)) {
       c.add("manifest.translations.invalid", langPath, "A translation is an object with optional name, description, slots and options.");
       continue;
     }
@@ -843,7 +874,7 @@ function validateTranslations(c, value, slots, options) {
     }
     if (Object.hasOwn(entry, "slots")) {
       const entrySlots = entry.slots;
-      if (!isRecord(entrySlots)) {
+      if (!isRecord2(entrySlots)) {
         c.add("manifest.translations.invalid", `${langPath}/slots`, "slots is an object keyed by slot id.");
       } else {
         for (const [slotId, slotEntry] of Object.entries(entrySlots)) {
@@ -852,7 +883,7 @@ function validateTranslations(c, value, slots, options) {
             c.add("manifest.translations.slot", slotPath, `"${slotId}" is not a slot id in this manifest.`);
             continue;
           }
-          if (!isRecord(slotEntry)) {
+          if (!isRecord2(slotEntry)) {
             c.add("manifest.translations.invalid", slotPath, "A slot translation is an object with optional label and description.");
             continue;
           }
@@ -867,7 +898,7 @@ function validateTranslations(c, value, slots, options) {
     }
     if (Object.hasOwn(entry, "options")) {
       const entryOptions = entry.options;
-      if (!isRecord(entryOptions)) {
+      if (!isRecord2(entryOptions)) {
         c.add("manifest.translations.invalid", `${langPath}/options`, "options is an object keyed by option id.");
       } else {
         for (const [optionId, optionEntry] of Object.entries(entryOptions)) {
@@ -877,7 +908,7 @@ function validateTranslations(c, value, slots, options) {
             c.add("manifest.translations.option", optionPath, `"${optionId}" is not an option id in this manifest.`);
             continue;
           }
-          if (!isRecord(optionEntry)) {
+          if (!isRecord2(optionEntry)) {
             c.add("manifest.translations.invalid", optionPath, "An option translation is an object with optional label, group and choices.");
             continue;
           }
@@ -889,7 +920,7 @@ function validateTranslations(c, value, slots, options) {
           }
           if (!Object.hasOwn(optionEntry, "choices")) continue;
           const choices = optionEntry.choices;
-          if (!isRecord(choices)) {
+          if (!isRecord2(choices)) {
             c.add("manifest.translations.invalid", `${optionPath}/choices`, "choices is an object keyed by choice value.");
             continue;
           }
@@ -908,7 +939,7 @@ function validateTranslations(c, value, slots, options) {
 }
 function validateManifest(value) {
   const c = new Collector();
-  if (!isRecord(value)) {
+  if (!isRecord2(value)) {
     c.add("manifest.invalid", "", "manifest.json must be a JSON object.");
     return c.problems;
   }
@@ -944,7 +975,7 @@ function validateManifest(value) {
     slots.forEach((slot, index) => validateSlot(c, slot, `/slots/${index}`, v2));
     const seen = /* @__PURE__ */ new Set();
     slots.forEach((slot, index) => {
-      if (!isRecord(slot) || typeof slot.id !== "string") return;
+      if (!isRecord2(slot) || typeof slot.id !== "string") return;
       if (seen.has(slot.id)) {
         c.add("manifest.slot.duplicate", `/slots/${index}/id`, `Slot id "${slot.id}" is used more than once.`);
       }
@@ -960,7 +991,7 @@ function validateManifest(value) {
     options.forEach((option, index) => validateOption(c, option, `/options/${index}`, v2));
     const seen = /* @__PURE__ */ new Set();
     options.forEach((option, index) => {
-      if (!isRecord(option) || typeof option.id !== "string") return;
+      if (!isRecord2(option) || typeof option.id !== "string") return;
       if (seen.has(option.id)) {
         c.add("manifest.option.duplicate", `/options/${index}/id`, `Option id "${option.id}" is used more than once.`);
       }
@@ -1012,7 +1043,7 @@ var manifestSchema = z2.object({
 });
 function sortKeys(value) {
   if (Array.isArray(value)) return value.map((item) => sortKeys(item));
-  if (isRecord(value)) {
+  if (isRecord2(value)) {
     const out = {};
     for (const key of Object.keys(value).sort()) out[key] = sortKeys(value[key]);
     return out;
@@ -1900,4 +1931,4 @@ export {
   apiProblems,
   planUpload
 };
-//# sourceMappingURL=chunk-PRE4CGOV.js.map
+//# sourceMappingURL=chunk-SKU37UVJ.js.map