From 35ff3623f2b9ec49233270b63e3a7412f75cf3cf Mon Sep 17 00:00:00 2001
From: Marcus Forsberg <marcus@winter.ax>
Date: Mon, 6 May 2024 07:52:04 +0200
Subject: [PATCH 1/2] fix: don't crash when interacting with empty link field

---
 src/components/CustomLinkInput.tsx | 18 ++++++++++--------
 1 file changed, 10 insertions(+), 8 deletions(-)

diff --git a/src/components/CustomLinkInput.tsx b/src/components/CustomLinkInput.tsx
index 3704f26..01f6432 100644
--- a/src/components/CustomLinkInput.tsx
+++ b/src/components/CustomLinkInput.tsx
@@ -23,18 +23,20 @@ export function CustomLinkInput(
   const linkValue = useFormValue(props.path.slice(0, -1)) as LinkValue | null
   const [options, setOptions] = useState<CustomLinkTypeOptions[] | null>(null)
 
-  const customLinkType = props.customLinkTypes.find((type) => type.value === linkValue!.type)!
+  const customLinkType = props.customLinkTypes.find((type) => type.value === linkValue!.type)
 
   useEffect(() => {
-    if (Array.isArray(customLinkType.options)) {
-      setOptions(customLinkType.options)
-    } else {
-      customLinkType
-        .options(document, props.path, workspace.currentUser)
-        .then((options) => setOptions(options))
+    if (customLinkType) {
+      if (Array.isArray(customLinkType?.options)) {
+        setOptions(customLinkType.options)
+      } else {
+        customLinkType
+          .options(document, props.path, workspace.currentUser)
+          .then((options) => setOptions(options))
+      }
     }
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [customLinkType.options, props.path, workspace.currentUser])
+  }, [customLinkType, props.path, workspace.currentUser])
 
   return options ? (
     <Select

From 1839f3b037ef02f0880d2d01e12fdaa12e3a185c Mon Sep 17 00:00:00 2001
From: Marcus Forsberg <marcus@winter.ax>
Date: Mon, 6 May 2024 08:14:10 +0200
Subject: [PATCH 2/2] fix: set default type to "internal" if it's somehow
 missing

---
 src/components/LinkTypeInput.tsx | 8 ++++++++
 src/helpers/typeGuards.ts        | 2 +-
 2 files changed, 9 insertions(+), 1 deletion(-)

diff --git a/src/components/LinkTypeInput.tsx b/src/components/LinkTypeInput.tsx
index 816c704..6e7ffb7 100644
--- a/src/components/LinkTypeInput.tsx
+++ b/src/components/LinkTypeInput.tsx
@@ -5,6 +5,7 @@ import {set, type StringInputProps} from 'sanity'
 import styled from 'styled-components'
 
 import {CustomLinkType, LinkFieldPluginOptions, LinkType} from '../types'
+import {useEffect} from 'react'
 
 const defaultLinkTypes: LinkType[] = [
   {title: 'Internal', value: 'internal', icon: LinkIcon},
@@ -52,6 +53,13 @@ export function LinkTypeInput({
 
   const selectedType = linkTypes.find((type) => type.value === value) || linkTypes[0]
 
+  useEffect(() => {
+    // Make sure there's always a default value
+    if (!value) {
+      onChange(set('internal'))
+    }
+  }, [onChange, value])
+
   return (
     <MenuButton
       button={
diff --git a/src/helpers/typeGuards.ts b/src/helpers/typeGuards.ts
index c8bfbe2..2a78256 100644
--- a/src/helpers/typeGuards.ts
+++ b/src/helpers/typeGuards.ts
@@ -16,4 +16,4 @@ export const isEmailLink = (link: LinkValue): link is EmailLink => link.type ===
 export const isPhoneLink = (link: LinkValue): link is PhoneLink => link.type === 'phone'
 
 export const isCustomLink = (link: LinkValue): link is CustomLink =>
-  !['internal', 'external', 'email', 'phone'].includes(link.type)
+  !!link.type && !['internal', 'external', 'email', 'phone'].includes(link.type)