Move logic to a helper

Author: triozer

plugins/airtable/src/data.ts

@@ -65,11 +65,15 @@ export async function getTables(baseId: string, signal: AbortSignal): Promise<Ai
 const EMAIL_REGEX = /\S[^\s@]*@\S+\.\S+/
 const PHONE_REGEX = /^(\+?[0-9])[0-9]{7,14}$/
 
-const MAILTO_PREFIX = "mailto:"
-const TEL_PREFIX = "tel:"
-
-const MAILTO_REGEX = new RegExp(MAILTO_PREFIX, "gi")
-const TEL_REGEX = new RegExp(TEL_PREFIX, "gi")
+/**
+ * Helper function to ensure a value has the specified prefix,
+ * removing any existing instances of the prefix first
+ */
+function ensurePrefix(prefix: string, value: string): string {
+    const regex = new RegExp(prefix, "gi")
+    const result = value.replace(regex, "")
+    return `${prefix}${result}`
+}
 
 const NonEmptyArrayOfAttachmentsSchema = v.pipe(
     v.array(v.object({ type: v.optional(v.string()), id: v.string(), url: v.string() })),
@@ -103,14 +107,14 @@ export function getFieldDataEntryForFieldSchema(
             if (typeof value === "string") {
                 if (fieldSchema.airtableType === "email" || EMAIL_REGEX.test(value)) {
                     return {
-                        value: `${MAILTO_PREFIX}${value.replace(MAILTO_REGEX, "")}`,
+                        value: ensurePrefix("mailto:", value),
                         type: "link",
                     }
                 }
 
                 if (fieldSchema.airtableType === "phoneNumber" || PHONE_REGEX.test(value)) {
                     return {
-                        value: `${TEL_PREFIX}${value.replace(TEL_REGEX, "")}`,
+                        value: ensurePrefix("tel:", value),
                         type: "link",
                     }
                 }