refactor(docs-ui): doc custom-range & hyperlink

packages/core/package.json

@@ -62,6 +62,7 @@
     "dependencies": {
         "@univerjs/protocol": "0.1.39-alpha.38",
         "@wendellhu/redi": "0.16.1",
+        "fast-diff": "1.3.0",
         "kdbush": "^4.0.2",
         "lodash-es": "^4.17.21",
         "nanoid": "5.0.8",

packages/core/src/docs/data-model/__tests__/replacement.spec.ts

@@ -14,11 +14,11 @@
  * limitations under the License.
  */
 
-import { describe, expect, it } from 'vitest';
+import type { IDocumentBody } from '../../../types/interfaces';
 
+import { describe, expect, it } from 'vitest';
 import { BooleanNumber } from '../../../types/enum/text-style';
 import { replaceInDocumentBody } from '../replacement';
-import type { IDocumentBody } from '../../../types/interfaces';
 
 function getTestDocumentBody() {
     const documentBody = {
@@ -147,7 +147,7 @@ describe('test case in replaceInDocumentBody utils', () => {
             ],
         } as IDocumentBody;
 
-        expect(replaceInDocumentBody(documentBody, '月色', 'Jocs')).toEqual(expectedBody);
+        expect(replaceInDocumentBody(documentBody, '月色', 'Jocs', true)).toEqual(expectedBody);
     });
 
     it('Should replace all `query` to `target` when `target` is empty', () => {
@@ -205,12 +205,12 @@ describe('test case in replaceInDocumentBody utils', () => {
             ],
         } as IDocumentBody;
 
-        expect(replaceInDocumentBody(documentBody, '月色', '')).toEqual(expectedBody);
+        expect(replaceInDocumentBody(documentBody, '月色', '', true)).toEqual(expectedBody);
     });
 
     it('Should return the origin body when the query is empty', () => {
         const documentBody = getTestDocumentBody();
 
-        expect(replaceInDocumentBody(documentBody, '', 'Jocs')).toEqual(documentBody);
+        expect(replaceInDocumentBody(documentBody, '', 'Jocs', true)).toEqual(documentBody);
     });
 });

packages/core/src/docs/data-model/replacement.ts

@@ -19,7 +19,8 @@ import { DocumentDataModel } from './document-data-model';
 import { JSONX } from './json-x/json-x';
 import { TextX } from './text-x/text-x';
 
-export function replaceInDocumentBody(body: IDocumentBody, query: string, target: string): IDocumentBody {
+// TODO: this function should be replaced by replaceSelection
+export function replaceInDocumentBody(body: IDocumentBody, query: string, target: string, caseSensitive: boolean): IDocumentBody {
     if (query === '') {
         return body;
     }
@@ -36,7 +37,7 @@ export function replaceInDocumentBody(body: IDocumentBody, query: string, target
     let index;
 
     // eslint-disable-next-line no-cond-assign
-    while ((index = documentDataModel.getBody()!.dataStream.indexOf(query)) >= 0) {
+    while ((index = (caseSensitive ? documentDataModel.getBody()!.dataStream : documentDataModel.getBody()!.dataStream.toLowerCase()).indexOf(query)) >= 0) {
         const textX = new TextX();
         const jsonX = JSONX.getInstance();
 
@@ -58,6 +59,15 @@ export function replaceInDocumentBody(body: IDocumentBody, query: string, target
                 }];
             }
 
+            if (sliceBody?.customRanges?.length) {
+                const customRange = sliceBody.customRanges[0];
+                replaceBody.customRanges = [{
+                    ...customRange,
+                    startIndex: 0,
+                    endIndex: target.length - 1,
+                }];
+            }
+
             textX.insert(target.length, replaceBody);
         }
 

packages/core/src/docs/data-model/text-x/__tests__/transform.spec.ts

@@ -724,7 +724,7 @@ describe('transform()', () => {
                     dataStream: '',
                     customRanges: [
                         {
-                            startIndex: -1,
+                            startIndex: 0,
                             endIndex: 0,
                             rangeId: 'rangeId',
                             rangeType: CustomRangeType.HYPERLINK,
@@ -746,7 +746,7 @@ describe('transform()', () => {
                     dataStream: '',
                     customRanges: [
                         {
-                            startIndex: -1,
+                            startIndex: 0,
                             endIndex: 0,
                             rangeId: 'rangeId',
                             rangeType: CustomRangeType.HYPERLINK,
@@ -768,7 +768,7 @@ describe('transform()', () => {
                     dataStream: '',
                     customRanges: [
                         {
-                            startIndex: -1,
+                            startIndex: 0,
                             endIndex: 0,
                             rangeId: 'rangeId',
                             rangeType: CustomRangeType.HYPERLINK,
@@ -790,7 +790,7 @@ describe('transform()', () => {
                     dataStream: '',
                     customRanges: [
                         {
-                            startIndex: -1,
+                            startIndex: 0,
                             endIndex: 0,
                             rangeId: 'rangeId',
                             rangeType: CustomRangeType.HYPERLINK,