should have better tests for DocUrl but this prevents errors for non-…

…string falsy inputs
automerge · Oct 30, 2023 · 7b20344 · 7b20344
1 parent 254bad1
commit 7b20344
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/packages/automerge-repo/src/DocUrl.ts b/packages/automerge-repo/src/DocUrl.ts
@@ -47,8 +47,10 @@ export const stringifyAutomergeUrl = ({
  * @param str: URL candidate
  * @returns boolean
  */
-export const isValidAutomergeUrl = (str: string): str is AutomergeUrl => {
-  if (!str.startsWith(urlPrefix)) return false
+export const isValidAutomergeUrl = (
+  str: string | undefined | null
+): str is AutomergeUrl => {
+  if (!str || !str.startsWith(urlPrefix)) return false
 
   const { binaryDocumentId: documentId } = parts(str)
   return documentId ? true : false