@dbx-tools/appkit-mastra 0.6.61 → 0.6.63

package/package.json

@@ -31,18 +31,18 @@
   "dependencies": {
     "@databricks/appkit": "^0.43.0",
     "@databricks/sdk-experimental": "^0.17.0",
-    "@dbx-tools/appkit": "0.6.61",
-    "@dbx-tools/core": "0.6.61",
-    "@dbx-tools/databricks": "0.6.61",
-    "@dbx-tools/fs": "0.6.61",
-    "@dbx-tools/genie": "0.6.61",
-    "@dbx-tools/model": "0.6.61",
-    "@dbx-tools/path": "0.6.61",
-    "@dbx-tools/shared-core": "0.6.61",
-    "@dbx-tools/shared-fs": "0.6.61",
-    "@dbx-tools/shared-genie": "0.6.61",
-    "@dbx-tools/shared-mastra": "0.6.61",
-    "@dbx-tools/shared-model": "0.6.61",
+    "@dbx-tools/appkit": "0.6.63",
+    "@dbx-tools/core": "0.6.63",
+    "@dbx-tools/databricks": "0.6.63",
+    "@dbx-tools/fs": "0.6.63",
+    "@dbx-tools/genie": "0.6.63",
+    "@dbx-tools/model": "0.6.63",
+    "@dbx-tools/path": "0.6.63",
+    "@dbx-tools/shared-core": "0.6.63",
+    "@dbx-tools/shared-fs": "0.6.63",
+    "@dbx-tools/shared-genie": "0.6.63",
+    "@dbx-tools/shared-mastra": "0.6.63",
+    "@dbx-tools/shared-model": "0.6.63",
     "@mastra/ai-sdk": "^1.6.0",
     "@mastra/core": "^1.47.0",
     "@mastra/express": "^1.4.2",
@@ -71,7 +71,7 @@
       "./package.json": "./package.json"
     }
   },
-  "version": "0.6.61",
+  "version": "0.6.63",
   "types": "./lib/index.d.ts",
   "type": "module",
   "exports": {

package/src/chart.ts

@@ -28,7 +28,16 @@
  */
 
 import { AppKitError, CacheManager, ExecutionError } from "@databricks/appkit";
-import { async, error, hash, json, log, string, type BrandContext } from "@dbx-tools/shared-core";
+import {
+  async,
+  error,
+  hash,
+  json,
+  log,
+  object,
+  string,
+  type BrandContext,
+} from "@dbx-tools/shared-core";
 import { marker, wire, type Chart, type ChartResult } from "@dbx-tools/shared-mastra";
 import { model } from "@dbx-tools/shared-model";
 import { Agent } from "@mastra/core/agent";
@@ -99,19 +108,18 @@ const chartDataPointSchema = z
       if (v === null || v === undefined) return null;
       if (typeof v === "number") return Number.isFinite(v) ? v : null;
       if (typeof v === "string") {
-        const n = Number(v);
-        return Number.isFinite(n) ? n : null;
+        return object.toNumber(v) ?? null;
       }
       // Scatter `[x, y]` or heatmap `[xIndex, yIndex, value]`. Coerce
       // stringified components; reject if any component is non-finite.
       if (Array.isArray(v) && (v.length === 2 || v.length === 3)) {
-        const nums = v.map((c) => (typeof c === "number" ? c : Number(c)));
-        return nums.every((n) => Number.isFinite(n)) ? nums : null;
+        const nums = v.map((c) => object.toNumber(c));
+        return nums.every((n) => n !== undefined) ? nums : null;
       }
       if (typeof v === "object" && v !== null && "value" in v) {
         const obj = v as { name?: unknown; value: unknown };
-        const val = typeof obj.value === "number" ? obj.value : Number(obj.value);
-        if (!Number.isFinite(val)) return null;
+        const val = object.toNumber(obj.value);
+        if (val === undefined) return null;
         // Coerce numeric / boolean / nullish names to strings so a
         // pie slice keyed on a year (`2024`) or category id is
         // accepted without round-tripping through the catch arm.

package/src/pagination.ts

@@ -7,6 +7,8 @@
  * @module
  */
 
+import { object } from "@dbx-tools/shared-core";
+
 /** Bounds for {@link clampPerPage}. */
 export interface PerPageBounds {
   /** Page size used for empty / non-positive / non-numeric inputs. */
@@ -29,8 +31,7 @@ export function clampPerPage(value: number | undefined, bounds: PerPageBounds):
  * can apply its built-in defaults.
  */
 export function parseIntParam(value: string | undefined): number | undefined {
-  if (!value) return undefined;
-  const n = Number(value);
-  if (!Number.isFinite(n) || n < 0) return undefined;
+  const n = object.toNumber(value);
+  if (n === undefined || n < 0) return undefined;
   return Math.trunc(n);
 }

package/src/plugin.ts

@@ -58,7 +58,7 @@ import {
 } from "@databricks/appkit";
 import { plugin } from "@dbx-tools/appkit";
 import { serving as nodeServing } from "@dbx-tools/model";
-import { async, error, log, string } from "@dbx-tools/shared-core";
+import { async, error, log, object, string } from "@dbx-tools/shared-core";
 import {
   feedback,
   routes,
@@ -1164,8 +1164,8 @@ type EmbedResolver = (
 function parseTimeoutMs(raw: unknown): number | undefined {
   const v = Array.isArray(raw) ? raw[0] : raw;
   if (typeof v !== "string") return undefined;
-  const n = Number(v);
-  if (!Number.isFinite(n) || n <= 0) return undefined;
+  const n = object.toNumber(v);
+  if (n === undefined || n <= 0) return undefined;
   return Math.min(Math.floor(n), MAX_EMBED_POLL_TIMEOUT_MS);
 }
 
@@ -1179,8 +1179,8 @@ function parseTimeoutMs(raw: unknown): number | undefined {
 function parseStatementLimit(raw: unknown): number | undefined {
   const v = Array.isArray(raw) ? raw[0] : raw;
   if (typeof v !== "string") return undefined;
-  const n = Number(v);
-  if (!Number.isFinite(n) || n < 0) return undefined;
+  const n = object.toNumber(v);
+  if (n === undefined || n < 0) return undefined;
   return Math.floor(n);
 }
 

package/src/statement.ts

@@ -21,6 +21,7 @@
 
 import { WorkspaceClient } from "@databricks/sdk-experimental";
 import { databricks } from "@dbx-tools/appkit";
+import { object } from "@dbx-tools/shared-core";
 import type { GenieDatasetData } from "@dbx-tools/shared-mastra";
 
 /**
@@ -35,15 +36,16 @@ export const STATEMENT_ROW_CAP = 500;
 /**
  * Best-effort numeric coercion for the Statement Execution API's
  * all-strings cells. Leaves non-numeric strings (and explicit
- * `null`s) intact; everything else flows through `Number`.
+ * `null`s) intact.
+ *
+ * Both of {@link object.toNumber}'s string leniencies are disabled, because a
+ * cell's other characters belong to the VALUE: a `"1,000"` in a text column is
+ * the string the query returned, and a `"25%"` must not silently arrive as
+ * `0.25` in a table the user is reading.
  */
 function coerceCell(cell: string | null): unknown {
   if (cell === null) return null;
-  if (/^-?\d+(\.\d+)?$/.test(cell)) {
-    const n = Number(cell);
-    if (Number.isFinite(n)) return n;
-  }
-  return cell;
+  return object.toNumber(cell, { separators: false, percent: false }) ?? cell;
 }
 
 /**