validate property types

Author: michelle0927

components/notion/actions/common/base-page-builder.mjs

@@ -87,6 +87,7 @@ export default {
           type: properties[property]?.type ?? property,
           label: properties[property]?.id || property,
           value: this[property] || this.properties?.[property],
+          name: properties[property]?.name || property,
         }));
     },
     /**
@@ -107,7 +108,7 @@ export default {
           try {
             notionProperties[property.label] = notionProperty?.convertToNotion(property);
           } catch {
-            throw new ConfigurationError(`Error converting property with label \`${property.label}\` to Notion format. Must be of type \`${NOTION_CONVERTER[property.type]?.type}\`.`);
+            throw new ConfigurationError(`Error converting property \`${property.name}\` to Notion format. Must be of type \`${NOTION_CONVERTER[property.type]?.type}\`.`);
           }
         }
       }

components/notion/actions/create-page-from-database/create-page-from-database.mjs

@@ -8,7 +8,7 @@ export default {
   key: "notion-create-page-from-database",
   name: "Create Page from Data Source",
   description: "Create a page from a data source. [See the documentation](https://developers.notion.com/reference/post-page)",
-  version: "1.0.1",
+  version: "1.0.2",
   type: "action",
   props: {
     notion,

components/notion/common/notion-page-properties.mjs

@@ -13,43 +13,68 @@ const NOTION_PAGE_PROPERTIES = {
     type: "string",
     example: "New Beauty Title",
     options: () => undefined,
-    convertToNotion: (property) => ({
-      title: utils.buildTextProperty(property.value),
-    }),
+    convertToNotion: (property) => {
+      if (typeof property.value !== "string") {
+        throw new Error;
+      }
+      return {
+        title: utils.buildTextProperty(property.value),
+      };
+    },
   },
   rich_text: {
     type: "string",
     example: "A beauty text value",
     options: () => undefined,
-    convertToNotion: (property) => ({
-      rich_text: utils.buildTextProperty(property.value),
-    }),
+    convertToNotion: (property) => {
+      if (typeof property.value !== "string") {
+        throw new Error;
+      }
+      return {
+        rich_text: utils.buildTextProperty(property.value),
+      };
+    },
   },
   number: {
     type: "integer",
     example: "59",
     options: () => undefined,
-    convertToNotion: (property) => ({
-      number: property.value,
-    }),
+    convertToNotion: (property) => {
+      if (isNaN(property.value)) {
+        throw new Error;
+      }
+      return {
+        number: property.value,
+      };
+    },
   },
   status: {
     type: "string",
     options: (property) => property.status?.options.map((option) => option.name),
-    convertToNotion: (property) => ({
-      status: {
-        name: property.value,
-      },
-    }),
+    convertToNotion: (property) => {
+      if (typeof property.value !== "string") {
+        throw new Error;
+      }
+      return {
+        status: {
+          name: property.value,
+        },
+      };
+    },
   },
   select: {
     type: "string",
     options: (property) => property.select?.options.map((option) => option.name),
-    convertToNotion: (property) => ({
-      select: {
-        name: property.value,
-      },
-    }),
+    convertToNotion: (property) => {
+      if (typeof property.value !== "string") {
+        throw new Error;
+      }
+      return {
+        select: {
+          name: property.value,
+        },
+      };
+    },
   },
   multi_select: {
     type: "string[]",
@@ -113,25 +138,40 @@ const NOTION_PAGE_PROPERTIES = {
     type: "string",
     example: "https://pipedream.com",
     options: () => undefined,
-    convertToNotion: (property) => ({
-      url: property.value,
-    }),
+    convertToNotion: (property) => {
+      if (typeof property.value !== "string") {
+        throw new Error;
+      }
+      return {
+        url: property.value,
+      };
+    },
   },
   email: {
     type: "string",
     example: "[email protected]",
     options: () => undefined,
-    convertToNotion: (property) => ({
-      email: property.value,
-    }),
+    convertToNotion: (property) => {
+      if (typeof property.value !== "string") {
+        throw new Error;
+      }
+      return {
+        email: property.value,
+      };
+    },
   },
   phone_number: {
     type: "string",
     example: "999-999-9999",
     options: () => undefined,
-    convertToNotion: (property) => ({
-      phone_number: property.value,
-    }),
+    convertToNotion: (property) => {
+      if (typeof property.value !== "string") {
+        throw new Error;
+      }
+      return {
+        phone_number: property.value,
+      };
+    },
   },
   relation: {
     type: "string[]",

components/notion/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pipedream/notion",
-  "version": "1.0.1",
+  "version": "1.0.2",
   "description": "Pipedream Notion Components",
   "main": "notion.app.mjs",
   "keywords": [