feat: Orchestrator Recognizer Preview

Composer/packages/adaptive-form/src/components/fields/RecognizerField/RecognizerField.tsx

@@ -20,7 +20,7 @@ export const RecognizerField: React.FC<FieldProps<MicrosoftIRecognizer>> = (prop
 
   useMigrationEffect(value, onChange);
   const { recognizers: recognizerConfigs, currentRecognizer } = useRecognizerConfig();
-  const dropdownOptions = useMemo(() => getDropdownOptions(recognizerConfigs), [recognizerConfigs]);
+  const dropdownOptions = useMemo(() => getDropdownOptions(recognizerConfigs, shellApi), [recognizerConfigs]);
 
   const RecognizerEditor = currentRecognizer?.recognizerEditor;
   const widget = RecognizerEditor ? <RecognizerEditor {...props} /> : null;

Composer/packages/adaptive-form/src/components/fields/RecognizerField/getDropdownOptions.ts

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-import { RecognizerSchema, FallbackRecognizerKey } from '@bfc/extension-client';
+import { RecognizerSchema, FallbackRecognizerKey, ShellApi } from '@bfc/extension-client';
 
 import { recognizerOrderMap } from './defaultRecognizerOrder';
 import { mapRecognizerSchemaToDropdownOption } from './mappers';
@@ -16,9 +16,9 @@ const getRankScore = (r: RecognizerSchema) => {
   return recognizerOrderMap[r.id] ?? Number.MAX_VALUE - 1;
 };
 
-export const getDropdownOptions = (recognizerConfigs: RecognizerSchema[]) => {
+export const getDropdownOptions = (recognizerConfigs: RecognizerSchema[], shellApi: ShellApi) => {
   return recognizerConfigs
-    .filter((r) => !r.disabled)
+    .filter((r) => (typeof r.disabled === 'function' && r.disabled(shellApi)) || !r.disabled)
     .sort((r1, r2) => {
       return getRankScore(r1) - getRankScore(r2);
     })

Composer/packages/client/package.json

@@ -31,6 +31,7 @@
     "@bfc/ui-plugin-dialog-schema-editor": "*",
     "@bfc/ui-plugin-lg": "*",
     "@bfc/ui-plugin-luis": "*",
+    "@bfc/ui-plugin-orchestrator": "*",
     "@bfc/ui-plugin-prompts": "*",
     "@bfc/ui-plugin-select-dialog": "*",
     "@bfc/ui-plugin-select-skill-dialog": "*",

Composer/packages/client/src/components/TestController/TestController.tsx

@@ -31,7 +31,7 @@ import { navigateTo, openInEmulator } from '../../utils/navigation';
 import plugins from '../../plugins';
 import { useShell } from '../../shell/useShell';
 
-import { isBuildConfigComplete, needsBuild } from './../../utils/buildUtil';
+import { isBuildConfigComplete, isKeyRequired, needsBuild } from './../../utils/buildUtil';
 import { PublishDialog } from './publishDialog';
 import { ErrorCallout } from './errorCallout';
 import { EmulatorOpenButton } from './emulatorOpenButton';
@@ -181,25 +181,27 @@ export const TestControllerContent: React.FC<{ projectId: string }> = (props) =>
 
   async function handleStart() {
     dismissCallout();
-    const config = Object.assign(
-      {},
-      {
-        luis: settings.luis,
-        qna: settings.qna,
-      }
-    );
-    if (!isAbsHosted() && needsBuild(dialogs)) {
-      if (
-        botStatus === BotStatus.failed ||
-        botStatus === BotStatus.pending ||
-        !isBuildConfigComplete(config, dialogs, luFiles, qnaFiles)
-      ) {
-        openDialog();
-      } else {
-        await handleBuild(config);
-      }
+    const config = {
+      luis: { ...settings.luis },
+      qna: { ...settings.qna },
+    };
+
+    if (isAbsHosted() || !needsBuild(dialogs)) {
+      return await handleLoadBot();
+    }
+
+    if (!isKeyRequired(dialogs, luFiles, qnaFiles)) {
+      return await handleBuild(config);
+    }
+
+    if (
+      botStatus === BotStatus.failed ||
+      botStatus === BotStatus.pending ||
+      !isBuildConfigComplete(config, dialogs, luFiles, qnaFiles)
+    ) {
+      openDialog();
     } else {
-      await handleLoadBot();
+      await handleBuild(config);
     }
   }
 

Composer/packages/client/src/components/TestController/publishDialog.tsx

@@ -21,8 +21,8 @@ import { Dropdown, ResponsiveMode, IDropdownOption } from 'office-ui-fabric-reac
 import { Text, Tips, Links, nameRegex } from '../../constants';
 import { FieldConfig, useForm } from '../../hooks/useForm';
 import { getReferredQnaFiles } from '../../utils/qnaUtil';
-import { getReferredLuFiles } from '../../utils/luUtil';
-import { dialogsSelectorFamily, luFilesState, qnaFilesState } from '../../recoilModel';
+import { getLuisBuildLuFiles } from '../../utils/luUtil';
+import { luFilesState, qnaFilesState, validateDialogsSelectorFamily } from '../../recoilModel';
 
 // -------------------- Styles -------------------- //
 const textFieldLabel = css`
@@ -105,11 +105,11 @@ interface IPublishDialogProps {
 
 export const PublishDialog: React.FC<IPublishDialogProps> = (props) => {
   const { isOpen, onDismiss, onPublish, botName, config, projectId } = props;
-  const dialogs = useRecoilValue(dialogsSelectorFamily(projectId));
+  const dialogs = useRecoilValue(validateDialogsSelectorFamily(projectId));
   const luFiles = useRecoilValue(luFilesState(projectId));
   const qnaFiles = useRecoilValue(qnaFilesState(projectId));
   const qnaConfigShow = getReferredQnaFiles(qnaFiles, dialogs).length > 0;
-  const luConfigShow = getReferredLuFiles(luFiles, dialogs).length > 0;
+  const luConfigShow = getLuisBuildLuFiles(luFiles, dialogs).length > 0;
 
   const formConfig: FieldConfig<FormData> = {
     name: {

Composer/packages/client/src/plugins.ts

@@ -11,6 +11,7 @@ import prompts from '@bfc/ui-plugin-prompts';
 import schemaEditor from '@bfc/ui-plugin-dialog-schema-editor';
 import selectDialog from '@bfc/ui-plugin-select-dialog';
 import selectSkillDialog from '@bfc/ui-plugin-select-skill-dialog';
+import orchestrator from '@bfc/ui-plugin-orchestrator';
 import lg from '@bfc/ui-plugin-lg';
 import lu from '@bfc/ui-plugin-luis';
 import crossTrained from '@bfc/ui-plugin-cross-trained';
@@ -43,6 +44,7 @@ export default mergePluginConfigs(
   selectSkillDialog,
   lg,
   lu,
+  orchestrator,
   crossTrained,
   schemaEditor
 );

Composer/packages/client/src/recoilModel/Recognizers.tsx

@@ -15,7 +15,7 @@ import { crossTrainConfigState, luFilesState, qnaFilesState, settingsState } fro
 import { dialogsSelectorFamily } from './selectors';
 import { recognizersSelectorFamily } from './selectors/recognizers';
 
-const LuisRecognizerTemplate = (target: string, fileName: string) => ({
+export const LuisRecognizerTemplate = (target: string, fileName: string) => ({
   $kind: SDKKinds.LuisRecognizer,
   id: `LUIS_${target}`,
   applicationId: `=settings.luis.${fileName.replace(/[.-]/g, '_')}_lu.appId`,
@@ -24,28 +24,34 @@ const LuisRecognizerTemplate = (target: string, fileName: string) => ({
   endpointKey: '=settings.luis.endpointKey',
 });
 
-const QnAMakerRecognizerTemplate = (target: string, fileName: string) => ({
+export const QnAMakerRecognizerTemplate = (target: string, fileName: string) => ({
   $kind: SDKKinds.QnAMakerRecognizer,
   id: `QnA_${target}`,
   knowledgeBaseId: `=settings.qna.${fileName.replace(/[.-]/g, '_')}_qna`,
   hostname: '=settings.qna.hostname',
   endpointKey: '=settings.qna.endpointKey',
 });
 
-const MultiLanguageRecognizerTemplate = (target: string, fileType: 'lu' | 'qna') => ({
+export const MultiLanguageRecognizerTemplate = (target: string, fileType: 'lu' | 'qna') => ({
   $kind: SDKKinds.MultiLanguageRecognizer,
   id: `${fileType === 'lu' ? 'LUIS' : 'QnA'}_${target}`,
   recognizers: {},
 });
 
-const CrossTrainedRecognizerTemplate = (): {
-  $kind: string;
+export const CrossTrainedRecognizerTemplate = (): {
+  $kind: SDKKinds.CrossTrainedRecognizerSet;
   recognizers: string[];
 } => ({
   $kind: SDKKinds.CrossTrainedRecognizerSet,
   recognizers: [],
 });
 
+export const OrchestratorRecognizerTemplate = (target: string, fileName: string) => ({
+  $kind: SDKKinds.OrchestratorRecognizer,
+  modelPath: '=settings.orchestrator.modelPath',
+  snapshotPath: `=settings.orchestrator.snapshots.${fileName.replace(/[.-]/g, '_')}`,
+});
+
 export const getMultiLanguagueRecognizerDialog = (
   target: string,
   files: { empty: boolean; id: string }[],
@@ -140,6 +146,7 @@ export const Recognizer = React.memo((props: { projectId: string }) => {
   const luFiles = useRecoilValue(luFilesState(projectId));
   const qnaFiles = useRecoilValue(qnaFilesState(projectId));
   const settings = useRecoilValue(settingsState(projectId));
+  const curRecognizers = useRecoilValue(recognizersSelectorFamily(projectId));
 
   useEffect(() => {
     let recognizers: RecognizerFile[] = [];
@@ -160,17 +167,19 @@ export const Recognizer = React.memo((props: { projectId: string }) => {
 
         if (luisRecognizers.length) {
           recognizers.push(luMultiLanguagueRecognizer);
-          recognizers = [...recognizers, ...preserveRecognizer(luisRecognizers, [])];
+          recognizers = [...recognizers, ...preserveRecognizer(luisRecognizers, curRecognizers)];
         }
         if (isCrossTrain) {
           recognizers.push(crossTrainedRecognizer);
         }
         if (isCrossTrain && qnaMakeRecognizers.length) {
           recognizers.push(qnaMultiLanguagueRecognizer);
-          recognizers = [...recognizers, ...preserveRecognizer(qnaMakeRecognizers, [])];
+          recognizers = [...recognizers, ...preserveRecognizer(qnaMakeRecognizers, curRecognizers)];
         }
       });
-    setRecognizers(recognizers);
+    if (recognizers.length != curRecognizers.length) {
+      setRecognizers(recognizers);
+    }
   }, [dialogs, luFiles, qnaFiles]);
 
   useEffect(() => {

Composer/packages/client/src/recoilModel/atoms/botState.ts

@@ -45,6 +45,7 @@ const emptyDialog: DialogInfo = {
   triggers: [],
   intentTriggers: [],
   skills: [],
+  luProvider: '',
   isFormDialog: false,
 };
 type dialogStateParams = { projectId: string; dialogId: string };
@@ -155,7 +156,7 @@ export const recognizerIdsState = atomFamily<string[], string>({
 export const recognizerState = atomFamily<RecognizerFile, { projectId: string; id: string }>({
   key: getFullyQualifiedKey('recognizer'),
   default: () => {
-    return { id: '', content: {}, lastModified: '' };
+    return {} as RecognizerFile;
   },
 });
 

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

@@ -1,14 +1,39 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 /* eslint-disable react-hooks/rules-of-hooks */
+import { SDKKinds } from '@bfc/shared';
 import { CallbackInterface, useRecoilCallback } from 'recoil';
 
 import { recognizerState } from '../atoms';
+import { LuisRecognizerTemplate, OrchestratorRecognizerTemplate } from '../Recognizers';
+
+import { recognizersSelectorFamily } from './../selectors/recognizers';
+
+export type RecognizerKindType = SDKKinds.OrchestratorRecognizer | SDKKinds.LuisRecognizer;
+const LuProviderRecognizer = [SDKKinds.OrchestratorRecognizer, SDKKinds.LuisRecognizer];
+
+const templates = {
+  [SDKKinds.OrchestratorRecognizer]: OrchestratorRecognizerTemplate,
+  [SDKKinds.LuisRecognizer]: LuisRecognizerTemplate,
+};
 
 export const recognizerDispatcher = () => {
   const updateRecognizer = useRecoilCallback(
-    ({ set }: CallbackInterface) => async (projectId: string, recognizerId: string, content: any) => {
-      set(recognizerState({ projectId, id: recognizerId }), content);
+    ({ set, snapshot }: CallbackInterface) => (projectId: string, dialogId: string, kind: RecognizerKindType) => {
+      const recognizersLoadable = snapshot.getLoadable(recognizersSelectorFamily(projectId));
+      if (recognizersLoadable.state === 'hasValue') {
+        const updates = recognizersLoadable.contents.filter(
+          ({ id, content }) =>
+            id.split('.')[0] === dialogId && LuProviderRecognizer.includes(content.$kind) && content.$kind !== kind
+        );
+
+        updates.forEach(({ id }) => {
+          set(recognizerState({ projectId, id }), {
+            id,
+            content: templates[kind](dialogId, id.replace('.lu.dialog', '')),
+          });
+        });
+      }
     }
   );
 

Composer/packages/client/src/recoilModel/selectors/validatedDialogs.ts

@@ -5,6 +5,9 @@ import { selectorFamily } from 'recoil';
 import { validateDialog } from '@bfc/indexers';
 
 import { botProjectIdsState, dialogIdsState, schemasState, lgFilesState, luFilesState, dialogState } from '../atoms';
+import { getLuProvider } from '../../utils/dialogUtil';
+
+import { recognizersSelectorFamily } from './recognizers';
 
 type validateDialogSelectorFamilyParams = { projectId: string; dialogId: string };
 const validateDialogSelectorFamily = selectorFamily({
@@ -14,8 +17,9 @@ const validateDialogSelectorFamily = selectorFamily({
     const schemas = get(schemasState(projectId));
     const lgFiles = get(lgFilesState(projectId));
     const luFiles = get(luFilesState(projectId));
-
-    return { ...dialog, diagnostics: validateDialog(dialog, schemas.sdk.content, lgFiles, luFiles) };
+    const recognizers = get(recognizersSelectorFamily(projectId));
+    const luProvider = getLuProvider(dialogId, recognizers);
+    return { ...dialog, diagnostics: validateDialog(dialog, schemas.sdk.content, lgFiles, luFiles), luProvider };
   },
 });
 

Composer/packages/client/src/shell/useShell.ts

@@ -27,6 +27,7 @@ import {
   luFilesState,
   rateInfoState,
   rootBotProjectIdSelector,
+  featureFlagsState,
 } from '../recoilModel';
 import { undoFunctionState } from '../recoilModel/undo/history';
 import httpClient from '../utils/httpUtil';
@@ -60,6 +61,7 @@ const stubDialog = (): DialogInfo => ({
   triggers: [],
   intentTriggers: [],
   skills: [],
+  luProvider: '',
   isFormDialog: false,
 });
 
@@ -82,9 +84,9 @@ export function useShell(source: EventSource, projectId: string): Shell {
   const settings = useRecoilValue(settingsState(projectId));
   const flowZoomRate = useRecoilValue(rateInfoState);
   const rootBotProjectId = useRecoilValue(rootBotProjectIdSelector);
-
   const userSettings = useRecoilValue(userSettingsState);
   const clipboardActions = useRecoilValue(clipboardActionsState);
+  const featureFlags = useRecoilValue(featureFlagsState);
   const {
     updateDialog,
     updateDialogSchema,
@@ -101,6 +103,7 @@ export function useShell(source: EventSource, projectId: string): Shell {
     displayManifestModal,
     updateSkill,
     updateZoomRate,
+    updateRecognizer,
     reloadProject,
     setApplicationLevelError,
   } = useRecoilValue(dispatcherState);
@@ -202,6 +205,7 @@ export function useShell(source: EventSource, projectId: string): Shell {
         commitChanges();
       });
     },
+    updateRecognizer,
     updateRegExIntent: updateRegExIntentHandler,
     renameRegExIntent: renameRegExIntentHandler,
     updateIntentTrigger: updateIntentTriggerHandler,
@@ -232,6 +236,7 @@ export function useShell(source: EventSource, projectId: string): Shell {
     redo,
     commitChanges,
     displayManifestModal: (skillId) => displayManifestModal(skillId, projectId),
+    featureFlags: (featureFlagKey: string): boolean => featureFlags?.[featureFlagKey]?.enabled ?? false,
     updateDialogSchema: async (dialogSchema: DialogSchemaFile) => {
       updateDialogSchema(dialogSchema, projectId);
     },

Composer/packages/client/src/utils/buildUtil.ts

@@ -1,11 +1,11 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-import { DialogInfo, LuFile } from '@bfc/shared';
+import { DialogInfo, LuFile, QnAFile } from '@bfc/shared';
 
 import { LuisConfig, QnaConfig } from '../constants';
 
-import { getReferredLuFiles } from './luUtil';
+import { getLuisBuildLuFiles } from './luUtil';
 import { getReferredQnaFiles } from './qnaUtil';
 import { getBaseName } from './fileUtil';
 
@@ -45,7 +45,7 @@ export function createCrossTrainConfig(dialogs: DialogInfo[], luFiles: LuFile[],
 
 export function isBuildConfigComplete(config, dialogs, luFiles, qnaFiles) {
   let complete = true;
-  if (getReferredLuFiles(luFiles, dialogs).length > 0) {
+  if (getLuisBuildLuFiles(luFiles, dialogs).length > 0) {
     if (Object.values(LuisConfig).some((luisConfigKey) => config.luis[luisConfigKey] === '')) {
       complete = false;
     }
@@ -58,9 +58,11 @@ export function isBuildConfigComplete(config, dialogs, luFiles, qnaFiles) {
   return complete;
 }
 
+export function isKeyRequired(dialogs: DialogInfo[], luFiles: LuFile[], qnaFiles: QnAFile[]) {
+  return getLuisBuildLuFiles(luFiles, dialogs).length || getReferredQnaFiles(qnaFiles, dialogs).length;
+}
+
 // return true if dialogs have one with default recognizer.
 export function needsBuild(dialogs) {
   return dialogs.some((dialog) => typeof dialog.content.recognizer === 'string');
 }
-
-export function createRecognizerTypeMap(dialogs: DialogInfo[]) {}