fix: use import resolver in LU parsing

Composer/packages/client/src/recoilModel/dispatchers/__tests__/lu.test.tsx

@@ -30,7 +30,7 @@ const file1 = {
   content: `\r\n# Hello\r\n-hi`,
 };
 
-const luFiles = [luUtil.parse(file1.id, file1.content, luFeatures)] as LuFile[];
+const luFiles = [luUtil.parse(file1.id, file1.content, luFeatures, [])] as LuFile[];
 
 const getLuIntent = (Name, Body): LuIntentSection =>
   ({

Composer/packages/client/src/recoilModel/dispatchers/lu.ts

@@ -95,11 +95,12 @@ const getRelatedLuFileChanges = async (
   // sync add/remove intents
   if (onlyAdds || onlyDeletes) {
     for (const item of sameIdOtherLocaleFiles) {
-      let newLuFile = (await luWorker.addIntents(item, addedIntents, luFeatures)) as LuFile;
+      let newLuFile = (await luWorker.addIntents(item, addedIntents, luFeatures, luFiles)) as LuFile;
       newLuFile = (await luWorker.removeIntents(
         newLuFile,
         deletedIntents.map(({ Name }) => Name),
-        luFeatures
+        luFeatures,
+        luFiles
       )) as LuFile;
       changes.push(newLuFile);
     }
@@ -124,7 +125,7 @@ export const createLuFileState = async (
   const locale = await snapshot.getPromise(localeState(projectId));
   const { languages, luFeatures } = await snapshot.getPromise(settingsState(projectId));
   const createdLuId = `${id}.${locale}`;
-  const createdLuFile = (await luWorker.parse(id, content, luFeatures)) as LuFile;
+  const createdLuFile = (await luWorker.parse(id, content, luFeatures, luFiles)) as LuFile;
   if (luFiles.find((lu) => lu.id === createdLuId)) {
     setError(callbackHelpers, new Error(formatMessage('lu file already exist')));
     return;
@@ -196,7 +197,7 @@ export const luDispatcher = () => {
       const { luFeatures } = await snapshot.getPromise(settingsState(projectId));
 
       try {
-        const updatedFile = (await luWorker.parse(id, content, luFeatures)) as LuFile;
+        const updatedFile = (await luWorker.parse(id, content, luFeatures, luFiles)) as LuFile;
         const updatedFiles = await getRelatedLuFileChanges(luFiles, updatedFile, projectId, luFeatures);
         // compare to drop expired change on current id file.
         /**
@@ -259,7 +260,8 @@ export const luDispatcher = () => {
               item,
               intentName,
               { Name: intent.Name },
-              luFeatures
+              luFeatures,
+              luFiles
             )) as LuFile;
             changes.push(updatedFile);
           }
@@ -271,7 +273,8 @@ export const luDispatcher = () => {
             luFile,
             intentName,
             { Body: intent.Body },
-            luFeatures
+            luFeatures,
+            luFiles
           )) as LuFile;
           set(luFilesState(projectId), luFilesAtomUpdater({ updates: [updatedFile] }));
         }
@@ -298,7 +301,7 @@ export const luDispatcher = () => {
       const file = luFiles.find((temp) => temp.id === id);
       if (!file) return luFiles;
       try {
-        const updatedFile = (await luWorker.addIntent(file, intent, luFeatures)) as LuFile;
+        const updatedFile = (await luWorker.addIntent(file, intent, luFeatures, luFiles)) as LuFile;
         const updatedFiles = await getRelatedLuFileChanges(luFiles, updatedFile, projectId, luFeatures);
         set(luFilesState(projectId), luFilesAtomUpdater({ updates: updatedFiles }));
       } catch (error) {
@@ -324,7 +327,7 @@ export const luDispatcher = () => {
       const file = luFiles.find((temp) => temp.id === id);
       if (!file) return luFiles;
       try {
-        const updatedFile = (await luWorker.removeIntent(file, intentName, luFeatures)) as LuFile;
+        const updatedFile = (await luWorker.removeIntent(file, intentName, luFeatures, luFiles)) as LuFile;
         const updatedFiles = await getRelatedLuFileChanges(luFiles, updatedFile, projectId, luFeatures);
         set(luFilesState(projectId), luFilesAtomUpdater({ updates: updatedFiles }));
       } catch (error) {

Composer/packages/client/src/recoilModel/parsers/__test__/luWorker.test.ts

@@ -31,7 +31,7 @@ describe('test lu worker', () => {
   it('get expected parse result', async () => {
     const content = `# Hello
 - hi`;
-    const result: any = await luWorker.parse('', content, luFeatures);
+    const result: any = await luWorker.parse('', content, luFeatures, []);
     const expected = [
       { Body: '- hi', Entities: [], Name: 'Hello', range: new Range(new Position(1, 0), new Position(2, 4)) },
     ];
@@ -47,7 +47,7 @@ hi
 @ simple friendsName
 
 `;
-    const { intents, diagnostics }: any = await luWorker.parse('', content, luFeatures);
+    const { intents, diagnostics }: any = await luWorker.parse('', content, luFeatures, []);
     expect(intents.length).toEqual(1);
     expect(diagnostics.length).toEqual(1);
     expect(diagnostics[0].range.start.line).toEqual(2);
@@ -57,7 +57,7 @@ hi
   });
 
   it('get expected add intent result', async () => {
-    const result: any = await luWorker.addIntent(luFile, getLuIntent('New', '-IntentValue'), luFeatures);
+    const result: any = await luWorker.addIntent(luFile, getLuIntent('New', '-IntentValue'), luFeatures, []);
     const expected = {
       Body: '-IntentValue',
       Entities: [],
@@ -73,7 +73,8 @@ hi
     const result: any = await luWorker.addIntents(
       luFile,
       [getLuIntent('New1', '-IntentValue1'), getLuIntent('New2', '-IntentValue2')],
-      luFeatures
+      luFeatures,
+      []
     );
     const expected = {
       Body: '-IntentValue2',
@@ -87,7 +88,7 @@ hi
   });
 
   it('get expected update intent result', async () => {
-    const result: any = await luWorker.updateIntent(luFile, 'New', getLuIntent('New', '-update'), luFeatures);
+    const result: any = await luWorker.updateIntent(luFile, 'New', getLuIntent('New', '-update'), luFeatures, []);
     const expected = {
       Body: '-update',
       Entities: [],
@@ -100,14 +101,14 @@ hi
   });
 
   it('get expected remove intent result', async () => {
-    const result: any = await luWorker.removeIntent(luFile, 'New2', luFeatures);
+    const result: any = await luWorker.removeIntent(luFile, 'New2', luFeatures, []);
     expect(result.intents.length).toBe(3);
     expect(result.intents[3]).toBeUndefined();
     luFile = result;
   });
 
   it('get expected remove intent result', async () => {
-    const result: any = await luWorker.removeIntents(luFile, ['New1', 'New'], luFeatures);
+    const result: any = await luWorker.removeIntents(luFile, ['New1', 'New'], luFeatures, []);
     expect(result.intents.length).toBe(1);
   });
 });

Composer/packages/client/src/recoilModel/parsers/luWorker.ts

@@ -15,33 +15,39 @@ import {
 } from './types';
 // Wrapper class
 class LuWorker extends BaseWorker<LuActionType> {
-  parse(id: string, content: string, luFeatures) {
-    const payload = { id, content, luFeatures };
+  parse(id: string, content: string, luFeatures, luFiles: LuFile[]) {
+    const payload = { id, content, luFeatures, luFiles };
     return this.sendMsg<LuParsePayload>(LuActionType.Parse, payload);
   }
 
-  addIntent(luFile: LuFile, intent: LuIntentSection, luFeatures) {
-    const payload = { luFile, intent, luFeatures };
+  addIntent(luFile: LuFile, intent: LuIntentSection, luFeatures, luFiles: LuFile[]) {
+    const payload = { luFile, intent, luFeatures, luFiles };
     return this.sendMsg<LuAddIntentPayload>(LuActionType.AddIntent, payload);
   }
 
-  updateIntent(luFile: LuFile, intentName: string, intent: { Name?: string; Body?: string }, luFeatures) {
-    const payload = { luFile, intentName, intent, luFeatures };
+  updateIntent(
+    luFile: LuFile,
+    intentName: string,
+    intent: { Name?: string; Body?: string },
+    luFeatures,
+    luFiles: LuFile[]
+  ) {
+    const payload = { luFile, intentName, intent, luFeatures, luFiles };
     return this.sendMsg<LuUpdateIntentPayload>(LuActionType.UpdateIntent, payload);
   }
 
-  removeIntent(luFile: LuFile, intentName: string, luFeatures) {
-    const payload = { luFile, intentName, luFeatures };
+  removeIntent(luFile: LuFile, intentName: string, luFeatures, luFiles: LuFile[]) {
+    const payload = { luFile, intentName, luFeatures, luFiles };
     return this.sendMsg<LuRemoveIntentPayload>(LuActionType.RemoveIntent, payload);
   }
 
-  addIntents(luFile: LuFile, intents: LuIntentSection[], luFeatures) {
-    const payload = { luFile, intents, luFeatures };
+  addIntents(luFile: LuFile, intents: LuIntentSection[], luFeatures, luFiles: LuFile[]) {
+    const payload = { luFile, intents, luFeatures, luFiles };
     return this.sendMsg<LuAddIntentsPayload>(LuActionType.AddIntents, payload);
   }
 
-  removeIntents(luFile: LuFile, intentNames: string[], luFeatures) {
-    const payload = { luFile, intentNames, luFeatures };
+  removeIntents(luFile: LuFile, intentNames: string[], luFeatures, luFiles: LuFile[]) {
+    const payload = { luFile, intentNames, luFeatures, luFiles };
     return this.sendMsg<LuRemoveIntentsPayload>(LuActionType.RemoveIntents, payload);
   }
 }

Composer/packages/client/src/recoilModel/parsers/types.ts

@@ -8,37 +8,43 @@ export type LuParsePayload = {
   id: string;
   content: string;
   luFeatures: ILUFeaturesConfig;
+  luFiles: LuFile[];
 };
 
 export type LuAddIntentPayload = {
   luFile: LuFile;
   intent: LuIntentSection;
   luFeatures: ILUFeaturesConfig;
+  luFiles: LuFile[];
 };
 
 export type LuAddIntentsPayload = {
   luFile: LuFile;
   intents: LuIntentSection[];
   luFeatures: ILUFeaturesConfig;
+  luFiles: LuFile[];
 };
 
 export type LuUpdateIntentPayload = {
   luFile: LuFile;
   intentName: string;
   intent?: { Name?: string; Body?: string };
   luFeatures: ILUFeaturesConfig;
+  luFiles: LuFile[];
 };
 
 export type LuRemoveIntentPayload = {
   luFile: LuFile;
   intentName: string;
   luFeatures: ILUFeaturesConfig;
+  luFiles: LuFile[];
 };
 
 export type LuRemoveIntentsPayload = {
   luFile: LuFile;
   intentNames: string[];
   luFeatures: ILUFeaturesConfig;
+  luFiles: LuFile[];
 };
 
 export type LgParsePayload = {

Composer/packages/client/src/recoilModel/parsers/workers/luParser.worker.ts

@@ -2,6 +2,7 @@
 // Licensed under the MIT License.
 
 import { luUtil } from '@bfc/indexers';
+import { luImportResolverGenerator } from '@bfc/shared';
 
 import {
   LuActionType,
@@ -52,42 +53,46 @@ interface RemoveIntentsMessage {
 
 type LuMessageEvent = ParseMessage | AddMessage | AddsMessage | UpdateMessage | RemoveMessage | RemoveIntentsMessage;
 
+const luFileResolver = (luFiles) => {
+  return luImportResolverGenerator(luFiles, '.lu');
+};
+
 export const handleMessage = (msg: LuMessageEvent) => {
   let result: any = null;
   switch (msg.type) {
     case LuActionType.Parse: {
-      const { id, content, luFeatures } = msg.payload;
-      result = luUtil.parse(id, content, luFeatures);
+      const { id, content, luFeatures, luFiles } = msg.payload;
+      result = luUtil.parse(id, content, luFeatures, luFiles);
       break;
     }
 
     case LuActionType.AddIntent: {
-      const { luFile, intent, luFeatures } = msg.payload;
-      result = luUtil.addIntent(luFile, intent, luFeatures);
+      const { luFile, intent, luFeatures, luFiles } = msg.payload;
+      result = luUtil.addIntent(luFile, intent, luFeatures, luFileResolver(luFiles));
       break;
     }
 
     case LuActionType.AddIntents: {
-      const { luFile, intents, luFeatures } = msg.payload;
-      result = luUtil.addIntents(luFile, intents, luFeatures);
+      const { luFile, intents, luFeatures, luFiles } = msg.payload;
+      result = luUtil.addIntents(luFile, intents, luFeatures, luFileResolver(luFiles));
       break;
     }
 
     case LuActionType.UpdateIntent: {
-      const { luFile, intentName, intent, luFeatures } = msg.payload;
-      result = luUtil.updateIntent(luFile, intentName, intent || null, luFeatures);
+      const { luFile, intentName, intent, luFeatures, luFiles } = msg.payload;
+      result = luUtil.updateIntent(luFile, intentName, intent || null, luFeatures, luFileResolver(luFiles));
       break;
     }
 
     case LuActionType.RemoveIntent: {
-      const { luFile, intentName, luFeatures } = msg.payload;
-      result = luUtil.removeIntent(luFile, intentName, luFeatures);
+      const { luFile, intentName, luFeatures, luFiles } = msg.payload;
+      result = luUtil.removeIntent(luFile, intentName, luFeatures, luFileResolver(luFiles));
       break;
     }
 
     case LuActionType.RemoveIntents: {
-      const { luFile, intentNames, luFeatures } = msg.payload;
-      result = luUtil.removeIntents(luFile, intentNames, luFeatures);
+      const { luFile, intentNames, luFeatures, luFiles } = msg.payload;
+      result = luUtil.removeIntents(luFile, intentNames, luFeatures, luFileResolver(luFiles));
       break;
     }
   }

Composer/packages/lib/code-editor/src/lu/__test__/InsertEntityButton.test.tsx

@@ -35,6 +35,7 @@ const luFile: LuFile = {
       Entities: [{ Name: 'target', Type: 'ml' }],
     },
   ],
+  allIntents: [],
   empty: false,
   content: '',
   imports: [],

Composer/packages/lib/indexers/__tests__/luIndexer.test.ts

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-import { FileInfo } from '@bfc/shared';
+import { FileInfo, luImportResolverGenerator } from '@bfc/shared';
 
 import { luIndexer } from '../src/luIndexer';
 
@@ -45,13 +45,25 @@ describe('parse', () => {
   });
 
   it('should parse LU file with import', () => {
-    const content = `[import](greating.lu)
+    const content = `[import](greeting.lu)
   [import](../common/aks.lu)
   `;
 
-    const result = parse(content, '', {});
+    const luFiles = [
+      {
+        id: 'greeting',
+        content: '',
+      },
+      {
+        id: 'aks',
+        content: '',
+      },
+    ];
+    const luImportResolver = luImportResolverGenerator(luFiles, '.lu');
+
+    const result = parse(content, 'a.lu', {}, luImportResolver);
     expect(result.imports.length).toEqual(2);
-    expect(result.imports[0]).toEqual({ id: 'greating.lu', path: 'greating.lu', description: 'import' });
+    expect(result.imports[0]).toEqual({ id: 'greeting.lu', path: 'greeting.lu', description: 'import' });
     expect(result.imports[1]).toEqual({ id: 'aks.lu', path: '../common/aks.lu', description: 'import' });
     expect(result.empty).toEqual(false);
     expect(result.intents.length).toEqual(0);