fix: apply suggestions from code review

Co-authored-by: Espen Hovlandsdal <espen@hovlandsdal.com>

Author: ricokahler

src/diffPatch.ts

@@ -24,20 +24,20 @@ import {
 const SYSTEM_KEYS = ['_id', '_type', '_createdAt', '_updatedAt', '_rev']
 
 /**
- * Maximum document size to consider for diff-match-patch (1MB)
- * Based on testing showing consistent 0-2ms performance up to this size
+ * Maximum size of strings to consider for diff-match-patch (1MB)
+ * Based on testing showing consistently good performance up to this size
  */
-const DMP_MAX_DOCUMENT_SIZE = 1_000_000
+const DMP_MAX_STRING_SIZE = 1_000_000
 
 /**
- * Maximum change ratio before falling back to set operations (40%)
+ * Maximum difference in string length before falling back to set operations (40%)
  * Above this threshold, likely indicates text replacement which can be slow
  */
-const DMP_MAX_CHANGE_RATIO = 0.4
+const DMP_MAX_STRING_LENGTH_CHANGE_RATIO = 0.4
 
 /**
- * Minimum document size to apply change ratio check (10KB)
- * Small documents are always fast regardless of change ratio
+ * Minimum string size to apply change ratio check (10KB)
+ * Small strings are always fast regardless of change ratio
  */
 const DMP_MIN_SIZE_FOR_RATIO_CHECK = 10_000
 
@@ -405,8 +405,6 @@ function diffArrayByKey(
  * Compatible with @sanity/diff-match-patch@3.2.0
  */
 export function shouldUseDiffMatchPatch(source: string, target: string): boolean {
-  // const sourceLength = sourceText.length
-  // const targetLength = targetText.length
   const maxLength = Math.max(source.length, target.length)
 
   // Always reject documents larger than our tested size limit