codeapp-js 0.2.2 → 1.0.0

package/{dev files/sharepoint.js → examples/apps/kanban/dist/sharepoint.js}

@@ -147,18 +147,167 @@ function initClient() {
 
 // ── Internal: execute a connector operation ────────────────────
 async function execOp(operationName, parameters) {
-  const client = await initClient();
-  const result = await client.executeAsync({
-    connectorOperation: {
-      tableName: DATA_SOURCE,
-      operationName,
-      parameters,
-    },
+  try {
+    const client = await initClient();
+    const result = await client.executeAsync({
+      connectorOperation: {
+        tableName: DATA_SOURCE,
+        operationName,
+        parameters: parameters || {},
+      },
+    });
+
+    if (!result) {
+      throw new Error("No result returned");
+    }
+
+    if (result.success === false) {
+      throw new Error(getSpErrorMessage(result.error));
+    }
+
+    return Object.prototype.hasOwnProperty.call(result, "data") ? result.data : result;
+  } catch (oError) {
+    throw new Error("SharePoint " + operationName + " failed: " + getSpErrorMessage(oError));
+  }
+}
+
+function getSpErrorMessage(oError) {
+  if (!oError) {
+    return "Unknown error";
+  }
+
+  if (typeof oError === "string") {
+    return oError;
+  }
+
+  if (oError instanceof Error && oError.message) {
+    return oError.message;
+  }
+
+  const aCandidates = [
+    oError.message,
+    oError.error && oError.error.message,
+    oError.body && oError.body.message,
+    oError.body && oError.body.error && oError.body.error.message,
+    oError.data && oError.data.message,
+    oError.detail,
+  ];
+
+  const sCandidate = aCandidates.find(function(sValue) {
+    return typeof sValue === "string" && sValue.trim() !== "";
   });
-  if (!result.success) {
-    throw new Error(result.error?.message || "Operation failed");
+
+  if (sCandidate) {
+    return sCandidate;
+  }
+
+  try {
+    return JSON.stringify(oError);
+  } catch (oStringifyError) {
+    return String(oError);
+  }
+}
+
+function requireNonEmptyString(sValue, sLabel) {
+  if (typeof sValue !== "string" || sValue.trim() === "") {
+    throw new Error("SharePoint " + sLabel + " is required.");
+  }
+
+  return sValue.trim();
+}
+
+function requireRecord(oValue, sLabel) {
+  if (!oValue || typeof oValue !== "object" || Array.isArray(oValue)) {
+    throw new Error("SharePoint " + sLabel + " must be an object.");
+  }
+
+  return oValue;
+}
+
+function normalizeHeaders(oHeaders) {
+  if (oHeaders == null) {
+    return {};
+  }
+
+  if (typeof oHeaders !== "object" || Array.isArray(oHeaders)) {
+    throw new Error("SharePoint request headers must be an object.");
+  }
+
+  return oHeaders;
+}
+
+function normalizeNumericQueryValue(value, sLabel) {
+  if (value == null || value === "") {
+    return null;
   }
-  return result.data;
+
+  const iValue = Number(value);
+  if (!Number.isFinite(iValue)) {
+    throw new Error("SharePoint " + sLabel + " must be numeric.");
+  }
+
+  return iValue;
+}
+
+function requireItemId(iItemId) {
+  if ((typeof iItemId !== "string" && typeof iItemId !== "number") || String(iItemId).trim() === "") {
+    throw new Error("SharePoint item ID is required.");
+  }
+
+  return iItemId;
+}
+
+function buildListContext(sSiteUrl, sListId) {
+  return {
+    siteUrl: encodeURIComponent(requireNonEmptyString(sSiteUrl, "site URL")),
+    table: requireNonEmptyString(sListId, "list ID"),
+  };
+}
+
+function buildItemQueryParameters(sSiteUrl, sListId, { filter, orderBy, top, skip } = {}) {
+  const oParams = buildListContext(sSiteUrl, sListId);
+
+  if (filter) {
+    oParams.$filter = String(filter);
+  }
+  if (orderBy) {
+    oParams.$orderby = String(orderBy);
+  }
+
+  const iTop = normalizeNumericQueryValue(top, "top");
+  const iSkip = normalizeNumericQueryValue(skip, "skip");
+  if (iTop != null) {
+    oParams.$top = iTop;
+  }
+  if (iSkip != null) {
+    oParams.$skip = iSkip;
+  }
+
+  return oParams;
+}
+
+function normalizeCollection(oPayload) {
+  if (Array.isArray(oPayload)) {
+    return oPayload;
+  }
+
+  const aCandidates = [
+    oPayload,
+    oPayload && oPayload.value,
+    oPayload && oPayload.items,
+    oPayload && oPayload.results,
+    oPayload && oPayload.body,
+    oPayload && oPayload.data,
+    oPayload && oPayload.result,
+    oPayload && oPayload.response,
+    oPayload && oPayload.d && oPayload.d.results,
+  ];
+
+  const aMatch = aCandidates.find(function(oCandidate) {
+    return Array.isArray(oCandidate);
+  });
+
+  return aMatch || [];
 }
 
 // ═══════════════════════════════════════════════════════════════
@@ -170,13 +319,13 @@ export async function callSharePointOperation(operationName, parameters = {}) {
   return execOp(operationName, parameters);
 }
 
-// ── Send HTTP Request (for list-name-based operations) ─────────
-export async function sendHttpRequest({ method = "GET", uri, headers, body }) {
+// ── Send HTTP Request (raw escape hatch) ───────────────────────
+export async function sendHttpRequest({ method = "GET", uri, headers, body } = {}) {
   return execOp("HttpRequest", {
-    method,
-    uri,
-    headers: headers || {},
-    body: body || "",
+    method: String(method || "GET").toUpperCase(),
+    uri: requireNonEmptyString(uri, "request URI"),
+    headers: normalizeHeaders(headers),
+    body: body == null ? "" : body,
   });
 }
 
@@ -186,89 +335,36 @@ export async function sendHttpRequest({ method = "GET", uri, headers, body }) {
 
 // ── Get Items ──────────────────────────────────────────────────
 export async function getItems(sSiteUrl, sListId, { filter, orderBy, top, skip } = {}) {
-  let params = { siteUrl: encodeURIComponent(sSiteUrl), table: sListId };
-  if (filter)       params.$filter = filter;
-  if (orderBy)      params.$orderby = orderBy;
-  if (top != null)  params.$top = top;
-  if (skip != null) params.$skip = skip;
-  return execOp("GetItems", params);
+  return execOp("GetItems", buildItemQueryParameters(sSiteUrl, sListId, { filter, orderBy, top, skip }));
 }
 
 // ── Get Item ───────────────────────────────────────────────────
 export async function getItem(sSiteUrl, sListId, iItemId) {
-  return execOp("GetItem", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    table: sListId,
-    id: iItemId,
-  });
+  return execOp("GetItem", Object.assign(buildListContext(sSiteUrl, sListId), {
+    id: requireItemId(iItemId),
+  }));
 }
 
 // ── Create Item ────────────────────────────────────────────────
 export async function createItem(sSiteUrl, sListId, oFields) {
-  return execOp("PostItem", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    table: sListId,
-    item: oFields,
-  });
+  return execOp("PostItem", Object.assign(buildListContext(sSiteUrl, sListId), {
+    item: requireRecord(oFields, "item payload"),
+  }));
 }
 
 // ── Update Item ────────────────────────────────────────────────
 export async function updateItem(sSiteUrl, sListId, iItemId, oChangedFields) {
-  return execOp("PatchItem", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    table: sListId,
-    id: iItemId,
-    item: oChangedFields,
-  });
+  return execOp("PatchItem", Object.assign(buildListContext(sSiteUrl, sListId), {
+    id: requireItemId(iItemId),
+    item: requireRecord(oChangedFields, "item payload"),
+  }));
 }
 
 // ── Delete Item ────────────────────────────────────────────────
 export async function deleteItem(sSiteUrl, sListId, iItemId) {
-  return execOp("DeleteItem", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    table: sListId,
-    id: iItemId,
-  });
-}
-
-// ═══════════════════════════════════════════════════════════════
-//  ITEMS (HTTP API — uses list name)
-// ═══════════════════════════════════════════════════════════════
-
-// ── Get Items by List Name ─────────────────────────────────────
-export async function getItemsByName(sSiteUrl, sListName, { filter, orderBy, top, skip } = {}) {
-  let sUri = sSiteUrl + "/_api/web/lists/getbytitle('" + sListName + "')/items";
-  let aQuery = [];
-  if (filter)       aQuery.push("$filter=" + filter);
-  if (orderBy)      aQuery.push("$orderby=" + orderBy);
-  if (top != null)  aQuery.push("$top=" + top);
-  if (skip != null) aQuery.push("$skip=" + skip);
-  if (aQuery.length > 0) sUri = sUri + "?" + aQuery.join("&");
-  return sendHttpRequest({ method: "GET", uri: sUri, headers: { Accept: "application/json;odata=nometadata" } });
-}
-
-// ── Get Item by List Name ──────────────────────────────────────
-export async function getItemByName(sSiteUrl, sListName, iItemId) {
-  let sUri = sSiteUrl + "/_api/web/lists/getbytitle('" + sListName + "')/items(" + iItemId + ")";
-  return sendHttpRequest({ method: "GET", uri: sUri, headers: { Accept: "application/json;odata=nometadata" } });
-}
-
-// ── Create Item by List Name ───────────────────────────────────
-export async function createItemByName(sSiteUrl, sListName, oFields) {
-  let sUri = sSiteUrl + "/_api/web/lists/getbytitle('" + sListName + "')/items";
-  return sendHttpRequest({ method: "POST", uri: sUri, headers: { Accept: "application/json;odata=nometadata", "Content-Type": "application/json;odata=nometadata" }, body: JSON.stringify(oFields) });
-}
-
-// ── Update Item by List Name ───────────────────────────────────
-export async function updateItemByName(sSiteUrl, sListName, iItemId, oChangedFields) {
-  let sUri = sSiteUrl + "/_api/web/lists/getbytitle('" + sListName + "')/items(" + iItemId + ")";
-  return sendHttpRequest({ method: "PATCH", uri: sUri, headers: { Accept: "application/json;odata=nometadata", "Content-Type": "application/json;odata=nometadata", "If-Match": "*" }, body: JSON.stringify(oChangedFields) });
-}
-
-// ── Delete Item by List Name ───────────────────────────────────
-export async function deleteItemByName(sSiteUrl, sListName, iItemId) {
-  let sUri = sSiteUrl + "/_api/web/lists/getbytitle('" + sListName + "')/items(" + iItemId + ")";
-  return sendHttpRequest({ method: "DELETE", uri: sUri, headers: { Accept: "application/json;odata=nometadata", "If-Match": "*" } });
+  return execOp("DeleteItem", Object.assign(buildListContext(sSiteUrl, sListId), {
+    id: requireItemId(iItemId),
+  }));
 }
 
 // ═══════════════════════════════════════════════════════════════
@@ -277,15 +373,15 @@ export async function deleteItemByName(sSiteUrl, sListName, iItemId) {
 
 // ── List Tables (Lists) ────────────────────────────────────────
 export async function listTables(sSiteUrl) {
-  return execOp("GetTables", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-  });
+  return normalizeCollection(await execOp("GetTables", {
+    siteUrl: encodeURIComponent(requireNonEmptyString(sSiteUrl, "site URL")),
+  }));
 }
 
 // ── List Library (Document Libraries) ──────────────────────────
 export async function listLibrary(sSiteUrl) {
   return execOp("GetDataSetsMetadata", {
-    siteUrl: encodeURIComponent(sSiteUrl),
+    siteUrl: encodeURIComponent(requireNonEmptyString(sSiteUrl, "site URL")),
   });
 }
 
@@ -296,9 +392,9 @@ export async function listLibrary(sSiteUrl) {
 // ── Create File ────────────────────────────────────────────────
 export async function createFile(sSiteUrl, sLibraryName, sFileName, fileContent) {
   return execOp("CreateFile", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    folderPath: sLibraryName,
-    name: sFileName,
+    siteUrl: encodeURIComponent(requireNonEmptyString(sSiteUrl, "site URL")),
+    folderPath: requireNonEmptyString(sLibraryName, "folder path"),
+    name: requireNonEmptyString(sFileName, "file name"),
     body: fileContent,
   });
 }
@@ -306,8 +402,8 @@ export async function createFile(sSiteUrl, sLibraryName, sFileName, fileContent)
 // ── Update File ────────────────────────────────────────────────
 export async function updateFile(sSiteUrl, sFileId, fileContent) {
   return execOp("UpdateFile", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    id: sFileId,
+    siteUrl: encodeURIComponent(requireNonEmptyString(sSiteUrl, "site URL")),
+    id: requireNonEmptyString(String(sFileId || ""), "file ID"),
     body: fileContent,
   });
 }
@@ -315,17 +411,17 @@ export async function updateFile(sSiteUrl, sFileId, fileContent) {
 // ── Delete File ────────────────────────────────────────────────
 export async function deleteFile(sSiteUrl, sFileId) {
   return execOp("DeleteFile", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    id: sFileId,
+    siteUrl: encodeURIComponent(requireNonEmptyString(sSiteUrl, "site URL")),
+    id: requireNonEmptyString(String(sFileId || ""), "file ID"),
   });
 }
 
 // ── Move File ──────────────────────────────────────────────────
 export async function moveFile(sSiteUrl, sSourceFileId, sDestinationFolderPath, sNewFileName) {
   return execOp("MoveFile", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    id: sSourceFileId,
-    destinationFolderPath: sDestinationFolderPath,
+    siteUrl: encodeURIComponent(requireNonEmptyString(sSiteUrl, "site URL")),
+    id: requireNonEmptyString(String(sSourceFileId || ""), "source file ID"),
+    destinationFolderPath: requireNonEmptyString(sDestinationFolderPath, "destination folder path"),
     newFileName: sNewFileName || "",
   });
 }
@@ -333,7 +429,7 @@ export async function moveFile(sSiteUrl, sSourceFileId, sDestinationFolderPath,
 // ── Get File Metadata ──────────────────────────────────────────
 export async function getFileMetadata(sSiteUrl, sFileId) {
   return execOp("GetFileMetadata", {
-    siteUrl: encodeURIComponent(sSiteUrl),
-    id: sFileId,
+    siteUrl: encodeURIComponent(requireNonEmptyString(sSiteUrl, "site URL")),
+    id: requireNonEmptyString(String(sFileId || ""), "file ID"),
   });
 }

package/examples/{kanban → apps/kanban}/power.config.json

@@ -1,35 +1,35 @@
-{
-  "appDisplayName": "Kanban",
-  "description": "",
-  "environmentId": "<ENVIRONMENT ID>",
-  "buildPath": "./dist",
-  "buildEntryPoint": "index.html",
-  "logoPath": "<ICON IMAGE PATH>",
-  "localAppUrl": "http://localhost:3000",
-  "region": "prod",
-  "connectionReferences": {
-    "dataverse": {
-      "id": "/providers/Microsoft.PowerApps/apis/shared_commondataserviceforapps",
-      "displayName": "Microsoft Dataverse",
-      "dataSources": [
-        "commondataserviceforapps"
-      ],
-      "authenticationType": null,
-      "sharedConnectionId": null,
-      "dataSets": {}
-    }
-  },
-  "databaseReferences": {
-    "default.cds": {
-      "databaseDetails": null,
-      "dataSources": {
-        "tasks": {
-          "entitySetName": "tasks",
-          "logicalName": "task",
-          "isHidden": false
-        }
-      },
-      "actions": null
-    }
-  }
-}
+{
+  "appDisplayName": "Kanban",
+  "description": "",
+  "environmentId": "<ENVIRONMENT ID>",
+  "buildPath": "./dist",
+  "buildEntryPoint": "index.html",
+  "logoPath": "<ICON IMAGE PATH>",
+  "localAppUrl": "http://localhost:3000",
+  "region": "prod",
+  "connectionReferences": {
+    "dataverse": {
+      "id": "/providers/Microsoft.PowerApps/apis/shared_commondataserviceforapps",
+      "displayName": "Microsoft Dataverse",
+      "dataSources": [
+        "commondataserviceforapps"
+      ],
+      "authenticationType": null,
+      "sharedConnectionId": null,
+      "dataSets": {}
+    }
+  },
+  "databaseReferences": {
+    "default.cds": {
+      "databaseDetails": null,
+      "dataSources": {
+        "tasks": {
+          "entitySetName": "tasks",
+          "logicalName": "task",
+          "isHidden": false
+        }
+      },
+      "actions": null
+    }
+  }
+}