@objectstack/rest 12.5.0 → 13.0.0

package/dist/index.js

@@ -1,5 +1,6 @@
 // src/rest-server.ts
 import { resolveAuthzContext, resolveLocalizationContext, isAuthGateAllowlisted } from "@objectstack/core";
+import { isMcpServerEnabled } from "@objectstack/types";
 
 // src/route-manager.ts
 var RouteManager = class {
@@ -519,6 +520,17 @@ async function coerceRow(rawRow, metaMap, ctx) {
 }
 
 // src/import-runner.ts
+import { bulkWrite, withTransientRetry } from "@objectstack/core";
+function extractRecordId(rec) {
+  const id = rec?.id ?? rec?.record?.id;
+  return id != null ? String(id) : void 0;
+}
+function toFailedResult(rowNo, err) {
+  const code = err?.code ?? "IMPORT_ROW_FAILED";
+  const message = typeof err?.message === "string" ? err.message.slice(0, 300) : "Row failed";
+  return { row: rowNo, ok: false, action: "failed", error: message, code };
+}
+var MAX_CREATE_BATCH_SIZE = 200;
 function runImport(opts) {
   const {
     p,
@@ -606,7 +618,7 @@ function runImport(opts) {
     return recs[0];
   };
   const writeCtx = { ...context ?? {}, skipAutomations: !runAutomations };
-  const results = [];
+  const results = new Array(rows.length);
   let okCount = 0, errCount = 0, created = 0, updated = 0, skipped = 0;
   let cancelled = false;
   const snapshot = (processed) => ({
@@ -617,6 +629,47 @@ function runImport(opts) {
     skipped,
     errors: errCount
   });
+  const canBulkCreate = typeof p.createManyData === "function";
+  const pendingCreates = [];
+  const flushPendingCreates = async () => {
+    if (pendingCreates.length === 0) return;
+    const batch = pendingCreates.splice(0, pendingCreates.length);
+    const writeResults = await bulkWrite(
+      batch.map((b) => b.data),
+      {
+        // Flush cadence follows progressEvery, but the write batch itself is
+        // capped independently — a caller-supplied progressEvery far above
+        // the issue's suggested 100-500 rows/batch must not translate into
+        // one oversized multi-row INSERT statement.
+        batchSize: Math.min(progressEvery, MAX_CREATE_BATCH_SIZE),
+        writeBatch: (chunk) => p.createManyData({
+          object: objectName,
+          records: chunk,
+          context: writeCtx,
+          ...environmentId ? { environmentId } : {}
+        }).then((r) => r.records),
+        writeOne: (row) => p.createData({
+          object: objectName,
+          data: row,
+          context: writeCtx,
+          ...environmentId ? { environmentId } : {}
+        })
+      }
+    );
+    for (const res of writeResults) {
+      const { index, rowNo } = batch[res.index];
+      if (res.ok) {
+        const id = extractRecordId(res.record);
+        okCount++;
+        created++;
+        if (collectUndo && id != null) undoLog.created.push(id);
+        results[index] = { row: rowNo, ok: true, action: "created", id };
+      } else {
+        errCount++;
+        results[index] = toFailedResult(rowNo, res.error);
+      }
+    }
+  };
   return (async () => {
     for (let i = 0; i < rows.length; i++) {
       const rowNo = i + 1;
@@ -630,7 +683,7 @@ function runImport(opts) {
         if (errors.length > 0) {
           const first2 = errors[0];
           errCount++;
-          results.push({ row: rowNo, ok: false, action: "failed", field: first2.field, code: first2.code, error: first2.message });
+          results[i] = { row: rowNo, ok: false, action: "failed", field: first2.field, code: first2.code, error: first2.message };
         } else {
           let existing = "none";
           let handled = false;
@@ -638,11 +691,11 @@ function runImport(opts) {
             existing = await findExisting(data);
             if (existing === "ambiguous") {
               errCount++;
-              results.push({ row: rowNo, ok: false, action: "failed", code: "AMBIGUOUS_MATCH", error: `matchFields matched more than one ${objectName} record` });
+              results[i] = { row: rowNo, ok: false, action: "failed", code: "AMBIGUOUS_MATCH", error: `matchFields matched more than one ${objectName} record` };
               handled = true;
             } else if (existing === "blank" && (skipBlankMatchKey || writeMode === "update")) {
               skipped++;
-              results.push({ row: rowNo, ok: true, action: "skipped", code: "BLANK_MATCH_KEY" });
+              results[i] = { row: rowNo, ok: true, action: "skipped", code: "BLANK_MATCH_KEY" };
               handled = true;
             }
           }
@@ -651,45 +704,46 @@ function runImport(opts) {
             const willCreate = !willUpdate && (writeMode === "insert" || writeMode === "upsert");
             if (!willUpdate && !willCreate) {
               skipped++;
-              results.push({ row: rowNo, ok: true, action: "skipped", code: "NO_MATCH" });
+              results[i] = { row: rowNo, ok: true, action: "skipped", code: "NO_MATCH" };
             } else if (dryRun) {
               okCount++;
               if (willUpdate) {
                 updated++;
-                results.push({ row: rowNo, ok: true, action: "updated", id: String(existing.id ?? "") || void 0 });
+                results[i] = { row: rowNo, ok: true, action: "updated", id: String(existing.id ?? "") || void 0 };
               } else {
                 created++;
-                results.push({ row: rowNo, ok: true, action: "created" });
+                results[i] = { row: rowNo, ok: true, action: "created" };
               }
             } else if (willUpdate) {
               const target = existing;
-              const res2 = await p.updateData({ object: objectName, id: target.id, data, context: writeCtx, ...environmentId ? { environmentId } : {} });
-              const id = res2?.id ?? res2?.record?.id ?? target.id;
+              const res2 = await withTransientRetry(() => p.updateData({ object: objectName, id: target.id, data, context: writeCtx, ...environmentId ? { environmentId } : {} }));
+              const id = extractRecordId(res2) ?? String(target.id);
               okCount++;
               updated++;
               if (collectUndo && target.id != null) {
                 undoLog.updated.push({ id: String(target.id), before: captureBefore(target, data) });
               }
-              results.push({ row: rowNo, ok: true, action: "updated", id: id != null ? String(id) : void 0 });
+              results[i] = { row: rowNo, ok: true, action: "updated", id };
+            } else if (canBulkCreate) {
+              pendingCreates.push({ index: i, rowNo, data });
             } else {
               const res2 = await p.createData({ object: objectName, data, context: writeCtx, ...environmentId ? { environmentId } : {} });
-              const id = res2?.id ?? res2?.record?.id;
+              const id = extractRecordId(res2);
               okCount++;
               created++;
-              if (collectUndo && id != null) undoLog.created.push(String(id));
-              results.push({ row: rowNo, ok: true, action: "created", id: id != null ? String(id) : void 0 });
+              if (collectUndo && id != null) undoLog.created.push(id);
+              results[i] = { row: rowNo, ok: true, action: "created", id };
             }
           }
         }
       } catch (err) {
         errCount++;
-        const code = err?.code ?? "IMPORT_ROW_FAILED";
-        const message = typeof err?.message === "string" ? err.message.slice(0, 300) : "Row failed";
-        results.push({ row: rowNo, ok: false, action: "failed", error: message, code });
+        results[i] = toFailedResult(rowNo, err);
       }
       const processed = i + 1;
-      if (onProgress && (processed % progressEvery === 0 || processed === rows.length)) {
-        await onProgress(snapshot(processed));
+      if (processed % progressEvery === 0 || processed === rows.length) {
+        await flushPendingCreates();
+        if (onProgress) await onProgress(snapshot(processed));
       }
       if (shouldCancel && processed < rows.length && processed % progressEvery === 0) {
         if (await shouldCancel()) {
@@ -698,10 +752,12 @@ function runImport(opts) {
         }
       }
     }
+    await flushPendingCreates();
+    const compacted = results.filter((r) => r !== void 0);
     return {
-      ...snapshot(results.length),
+      ...snapshot(compacted.length),
       ok: okCount,
-      results,
+      results: compacted,
       cancelled,
       ...collectUndo ? { undoLog } : {}
     };
@@ -1698,7 +1754,7 @@ var RestServer = class {
         userId: authz.userId,
         tenantId: authz.tenantId,
         email: authz.email,
-        roles: authz.roles,
+        positions: authz.positions,
         permissions: authz.permissions,
         systemPermissions: authz.systemPermissions,
         ...authz.tabPermissions ? { tabPermissions: authz.tabPermissions } : {},
@@ -2156,8 +2212,7 @@ var RestServer = class {
           if (this.config.api.enableUi) {
             discovery.routes.ui = `${realBase}/ui`;
           }
-          const mcpEnabled = globalThis?.process?.env?.OS_MCP_SERVER_ENABLED === "true";
-          if (mcpEnabled) {
+          if (isMcpServerEnabled()) {
             const unscopedBase = isScoped ? basePath.replace(/\/(environments|projects)\/:environmentId$/, "") : basePath;
             discovery.routes.mcp = `${unscopedBase}/mcp`;
           } else {