@datarails-ai/dr-cli 1.6.121 → 1.6.122

package/dist/cli.js

@@ -8,7 +8,7 @@ import { fileURLToPath as fileURLToPath3 } from "url";
 // package.json
 var package_default = {
   name: "@datarails-ai/dr-cli",
-  version: "1.6.121",
+  version: "1.6.122",
   description: "Datarails CLI",
   type: "module",
   engines: {
@@ -69,7 +69,7 @@ var package_default = {
 };
 
 // src/buildInfo.ts
-var BUILD_TIME = "2026-10-12T10:41:54.439Z";
+var BUILD_TIME = "2026-10-12T10:51:59.680Z";
 function formatBuildTime(iso) {
   if (iso === "") return "";
   const d = new Date(iso);
@@ -31029,6 +31029,96 @@ function formatCalcFieldError(name, preview2, full) {
   return `${head} looks like placeholder text, not a real expression. The server's ETL compiler rejects formulas with no [field] reference, "quoted" literal, function() call, or numeric value \u2014 replace it before retrying. Common cause: a GET \u2192 PUT round-trip on a pre-kickoff starter mapper carried the placeholder text forward.`;
 }
 
+// src/commands/_mapperConfigShape.ts
+var OBJECT_LIST_KEYS = [
+  "header",
+  "dimension",
+  "footer",
+  "attributes",
+  "calculated_fields",
+  "fx_conversions",
+  "table_id"
+];
+var GET_HINT = '`dr datamappers get <mapper_id> --json` shows the shape the server expects (a mapper with no anchor carries "table_id": []).';
+function describeValue(v) {
+  if (v === null) return "null";
+  if (Array.isArray(v)) return "an array";
+  switch (typeof v) {
+    case "number":
+      return `a number (${String(v)})`;
+    case "string":
+      return `a string (${JSON.stringify(v.length > 40 ? `${v.slice(0, 40)}\u2026` : v)})`;
+    case "boolean":
+      return `a boolean (${String(v)})`;
+    case "object":
+      return "an object";
+    default:
+      return typeof v;
+  }
+}
+function isPlainObject2(v) {
+  return v !== null && typeof v === "object" && !Array.isArray(v);
+}
+function tableIdProblems(value) {
+  if (!Array.isArray(value)) {
+    return [
+      `config.table_id must be an array, got ${describeValue(value)}. It is the table anchor, not the table id: [] for none, or one cell object such as [{"coordinate": "A1", "value": "<text in that cell>", "match_type": "any_text"}]. The table id goes on the command line, not in the body.`
+    ];
+  }
+  const out = [];
+  if (value.length > 1) {
+    out.push(
+      `config.table_id holds ${value.length} entries; the server allows at most one anchor ("Only one anchor is allowed").`
+    );
+  }
+  value.forEach((entry, i) => {
+    if (!isPlainObject2(entry)) {
+      out.push(`config.table_id[${i}] must be an object, got ${describeValue(entry)}.`);
+    } else if (typeof entry["match_type"] !== "string") {
+      out.push(
+        `config.table_id[${i}].match_type is required (a string such as "any_text" or "location").`
+      );
+    }
+  });
+  return out;
+}
+function validateMapperConfigShape(config) {
+  if (config === void 0) return;
+  if (!isPlainObject2(config)) {
+    throw new Error(`config must be a JSON object, got ${describeValue(config)}. ${GET_HINT}`);
+  }
+  const problems = [];
+  for (const key of OBJECT_LIST_KEYS) {
+    if (!(key in config)) continue;
+    const value = config[key];
+    if (key === "table_id") {
+      problems.push(...tableIdProblems(value));
+      continue;
+    }
+    if (!Array.isArray(value)) {
+      problems.push(`config.${key} must be an array of objects, got ${describeValue(value)}.`);
+      continue;
+    }
+    value.forEach((entry, i) => {
+      if (!isPlainObject2(entry)) {
+        problems.push(`config.${key}[${i}] must be an object, got ${describeValue(entry)}.`);
+      }
+    });
+  }
+  if ("include_sheets" in config && !Array.isArray(config["include_sheets"])) {
+    problems.push(
+      `config.include_sheets must be an array, got ${describeValue(config["include_sheets"])}.`
+    );
+  }
+  if (problems.length > 0) {
+    throw new Error(
+      `invalid datamapper config (the server would answer an unattributed HTTP 500 / "Unknown error"):
+  - ${problems.join("\n  - ")}
+${GET_HINT}`
+    );
+  }
+}
+
 // src/commands/_errorLog.ts
 import { promises as fs14 } from "fs";
 import * as os3 from "os";
@@ -31289,6 +31379,7 @@ async function runPreviewData(cfg) {
     throw new Error("--payload-file is required");
   }
   let body = await readJsonPayload(payloadFile);
+  validateMapperConfigShape(body["config"]);
   const { decoded, changeCount } = decodeBodyEntitiesDeep(body);
   if (changeCount > 0) {
     body = decoded;
@@ -31485,7 +31576,12 @@ async function runDatamappersCreate(cfg) {
     parseKeyEqualsValue(cfg.flags.calcs, "--calc");
     parseKeyEqualsValue(cfg.flags.attrs, "--attr");
     parseKeyEqualsValue(cfg.flags.dims, "--dim");
-    parseKeyEqualsValue(cfg.flags.tableIdCells ?? [], "--table-id-cell");
+    const tableIdCells = parseKeyEqualsValue(cfg.flags.tableIdCells ?? [], "--table-id-cell");
+    if (tableIdCells.length > 1) {
+      throw new Error(
+        `--table-id-cell takes one cell; got ${tableIdCells.length}. The server allows one table anchor per mapper.`
+      );
+    }
     parseKeyEqualsValue(cfg.flags.footers, "--footer");
     parseKeyEqualsValue(cfg.flags.renames, "--rename");
   }
@@ -31493,6 +31589,7 @@ async function runDatamappersCreate(cfg) {
   let body;
   if (payloadFile !== "") {
     body = await readJsonPayload(payloadFile);
+    validateMapperConfigShape(body["config"]);
   } else {
     body = await buildConveniencePayload(
       cfg.client,
@@ -31590,6 +31687,7 @@ async function runDatamappersUpdate(cfg) {
   }
   const rescanValue = resolveRescan(cfg.rescan, cfg.noRescan);
   let body = await readJsonPayload(payloadFile);
+  validateMapperConfigShape(body["config"]);
   if (rescanValue === "") {
     delete body["rescan"];
   } else {
@@ -32229,7 +32327,7 @@ function registerDatamappersCommand(root) {
   addAccountFlag(dimensionCmd);
   const multipleCmd = addConvenienceFlags(
     createCmd.command("multiple").description(
-      "Create a multiple-tables datamapper (requires --dim \u2265 1 AND --table-id-cell \u2265 1)"
+      "Create a multiple-tables datamapper (requires --dim \u2265 1 AND one --table-id-cell)"
     ).argument("<table_id>", "Table ID (positive integer)")
   ).requiredOption(
     "--dim <Coord=Label>",
@@ -32238,7 +32336,7 @@ function registerDatamappersCommand(root) {
     []
   ).requiredOption(
     "--table-id-cell <Coord=Label>",
-    "Add table_id entry \u2014 points at the cell carrying a per-section table identifier. Repeatable; at least one is required.",
+    "The table_id (anchor) cell \u2014 the cell carrying the per-section table identifier. Required; the server allows exactly one.",
     collectArrayOpt,
     []
   ).action(async (tableIdStr, opts) => {
@@ -56637,7 +56735,7 @@ async function readHiddenRulesPayload(path12) {
   raw.forEach((entry, idx) => {
     const ctx = `rules[${idx}]`;
     const base = validateCreate(entry, ctx);
-    if (!isPlainObject2(entry)) {
+    if (!isPlainObject3(entry)) {
       throw new Error(`--file: ${ctx} must be a JSON object`);
     }
     const ctxnId = entry["customer_transaction_id"];
@@ -56696,11 +56794,11 @@ function parseReorderShorthand(raw) {
   }
   return { rules: entries };
 }
-function isPlainObject2(v) {
+function isPlainObject3(v) {
   return typeof v === "object" && v !== null && !Array.isArray(v);
 }
 function validateCreate(raw, ctx) {
-  if (!isPlainObject2(raw)) {
+  if (!isPlainObject3(raw)) {
     throw new Error(`--file: ${ctx} must be a JSON object`);
   }
   if (!Array.isArray(raw["conditions"]) || raw["conditions"].length === 0) {
@@ -56756,7 +56854,7 @@ function validateCreate(raw, ctx) {
   return out;
 }
 function validateCondition(raw, ctx) {
-  if (!isPlainObject2(raw)) {
+  if (!isPlainObject3(raw)) {
     throw new Error(`--file: ${ctx} must be a JSON object`);
   }
   const inputField = raw["input_field"];
@@ -56821,7 +56919,7 @@ function validateArgsForOperation(op, args, ctx) {
   }
 }
 function validateReorder(raw, ctx) {
-  if (!isPlainObject2(raw)) {
+  if (!isPlainObject3(raw)) {
     throw new Error(`--file: ${ctx} must be a JSON object`);
   }
   if (!Array.isArray(raw["rules"]) || raw["rules"].length === 0) {
@@ -56830,7 +56928,7 @@ function validateReorder(raw, ctx) {
   const seenIds = /* @__PURE__ */ new Set();
   const entries = [];
   raw["rules"].forEach((entry, idx) => {
-    if (!isPlainObject2(entry)) {
+    if (!isPlainObject3(entry)) {
       throw new Error(`--file: ${ctx}.rules[${idx}] must be a JSON object`);
     }
     const id = entry["id"];

package/dist/scopes-map.json

@@ -1,6 +1,6 @@
 {
   "version": 1,
-  "cliVersion": "1.6.121",
+  "cliVersion": "1.6.122",
   "_contract": "Resolve a command path through `aliases` (canonical prefix + alias segment -> canonical path) before any lookup. A path listed in `pendingCommands`, or whose first segment is a key of `pending`, is UNDECIDED, not unrestricted: fail closed or skip it, and never read a path's absence from `commands` as 'no requirement'. An entry carrying `denied` is UNAVAILABLE to any AGENT-SCOPED credential — one whose `scopes` claim carries at least one `resource:action` entry; a claim that is absent, empty, capability-only or wholly unparseable is UNRESTRICTED, and this mark does not apply to it — whatever else that entry says: it is not a ceiling and it is not always-available, its `denied.reason` says why and `denied.until` names the ticket that lifts it, and the absence of `requires` on it must NEVER be read as 'no requirement'. Only an entry in `commands` with an empty `requires` means always-available.",
   "commands": {
     "login": {

package/dist/skills/dr-cli/reference/datamappers.md

@@ -26,7 +26,8 @@ the matching subcommand. `datamappers create <flat|dimension|multiple|from-file>
   `--version` — `--version` is reserved by the global `dr --version` and silently
   no-ops the subcommand.
 - `dimension` adds required `--dim Coord=Label` (≥1).
-- `multiple` adds required `--dim` and `--table-id-cell Coord=Label` (≥1).
+- `multiple` adds required `--dim` and one `--table-id-cell Coord=Label` (the
+  server allows a single table anchor; a second one is refused).
 - `from-file --file <path>` is the power mode — it ships the JSON body without
   the convenience-flag validation, but **not** strictly verbatim. Three things
   still rewrite it: `--rescan`/`--no-rescan` set or strip `rescan`; a source
@@ -35,6 +36,27 @@ the matching subcommand. `datamappers create <flat|dimension|multiple|from-file>
   Required body fields: `name`, `document_version_id`, `selected_document_ids`
   (GET returns this renamed as `mapped_documents_ids`), `config`, `sheet_name`.
 
+**`config` shape: `table_id` is not the table id (DR-54216).** `config.table_id`
+is the table *anchor*: `[]` for none (what `datamappers get --json` emits), or a
+list with exactly one cell object, e.g.
+`[{"coordinate": "A1", "value": "<text in that cell>", "match_type": "any_text"}]`.
+The table id goes on the command line. The server does not check `config`, so a
+wrong type gets a bare HTTP 500 on create and "Unknown error while processing
+data for preview" on preview, with no field named. `create` (payload-file /
+from-file), `update` and `preview` therefore check these keys before sending and
+fail naming the field:
+
+| Key | Must be |
+|-----|---------|
+| `table_id` | array of at most one object, with a `match_type` string |
+| `header`, `dimension`, `footer`, `attributes` | array of cell objects |
+| `calculated_fields`, `fx_conversions` | array of objects |
+| `include_sheets` | array |
+
+An absent key is fine; `null`, a number, a string or a list of strings is not.
+Writing a body from scratch, start from `datamappers get <mapper_id> --json` of a
+similar mapper.
+
 **Ordinal: mappers that share one are versions of one mapper.** Each mapper has
 an `ordinal` (the ORDINAL column of `datamappers list --table <id>`). The server
 binds documents per (table, ordinal): binding a mapper to a document moves that

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@datarails-ai/dr-cli",
-  "version": "1.6.121",
+  "version": "1.6.122",
   "description": "Datarails CLI",
   "type": "module",
   "engines": {