Merge branch 'main' into dependabot/npm_and_yarn/dot-github/actions/conventional-pr/actions/core-1.9.1

Author: OEvgeny

.config/tsaoptions.json

@@ -0,0 +1,6 @@
+{
+  "azurePath": "Composer/Compliance",
+  "notificationAliases": "[email protected]",
+  "codebaseName": "TSABotFrameworkComposer",
+  "ignoreBranchName": true
+}

Composer/packages/adaptive-flow/src/adaptive-flow-editor/AdaptiveFlowEditor.tsx

@@ -137,14 +137,6 @@ const VisualDesigner: React.FC<VisualDesignerProps> = ({ onFocus, onBlur, schema
     editorEvent && handleEditorEvent(editorEvent.type, editorEvent.payload);
   };
 
-  const marqueeStyles = (_) => {
-    return {
-      root: {
-        width: '100%',
-        height: '100%',
-      },
-    };
-  };
   return (
     <CacheProvider value={emotionCache}>
       <NodeRendererContext.Provider value={nodeContext}>
@@ -158,15 +150,15 @@ const VisualDesigner: React.FC<VisualDesignerProps> = ({ onFocus, onBlur, schema
             {...enableKeyboardCommandAttributes(handleCommand)}
             data-testid="visualdesigner-container"
           >
-            <SelectionContext.Provider value={selectionContext}>
-              <MarqueeSelection isDraggingConstrainedToRoot selection={selection} styles={marqueeStyles}>
-                <FlowToolbar
-                  flowCommentsVisible={flowCommentsVisible}
-                  flowZoomRate={flowZoomRate}
-                  focusedId={focusedId}
-                  toggleFlowComments={toggleFlowComments}
-                  updateFlowZoomRate={updateFlowZoomRate}
-                >
+            <FlowToolbar
+              flowCommentsVisible={flowCommentsVisible}
+              flowZoomRate={flowZoomRate}
+              focusedId={focusedId}
+              toggleFlowComments={toggleFlowComments}
+              updateFlowZoomRate={updateFlowZoomRate}
+            >
+              <SelectionContext.Provider value={selectionContext}>
+                <MarqueeSelection isDraggingConstrainedToRoot selection={selection}>
                   <div
                     className="flow-editor-container"
                     css={{
@@ -200,9 +192,9 @@ const VisualDesigner: React.FC<VisualDesignerProps> = ({ onFocus, onBlur, schema
                       }}
                     />
                   </div>
-                </FlowToolbar>
-              </MarqueeSelection>
-            </SelectionContext.Provider>
+                </MarqueeSelection>
+              </SelectionContext.Provider>
+            </FlowToolbar>
           </div>
         </SelfHostContext.Provider>
       </NodeRendererContext.Provider>

Composer/packages/client/src/recoilModel/dispatchers/__tests__/mocks/mockProjectResponse.json

@@ -3085,7 +3085,7 @@
                   "title": "speak"
                 },
                 "inputHint": {
-                  "description": "Indicates whether your bot is accepting,\nexpecting, or ignoring user input after the message is delivered to the client. Possible\nvalues include: 'acceptingInput', 'ignoringInput', 'expectingInput'",
+                  "description": "Indicates whether your bot is accepting,\nexpecting, or ignoring user input after the message is delivered to the client. Possible\nvalues include: 'acceptingInput', 'ignoringInput', 'expectingInput', 'ignoringSpeechInput', 'ignoringNonSpeechInput'",
                   "type": "string",
                   "title": "inputHint"
                 },

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

@@ -102,12 +102,6 @@ export const publisherDispatcher = () => {
     const { set, snapshot } = callbackHelpers;
     const { endpointURL, status, port } = data;
 
-    // remove job id in publish storage if published
-    if (status === PUBLISH_SUCCESS || status === PUBLISH_FAILED) {
-      const publishJobIds = publishStorage.get('jobIds') || {};
-      delete publishJobIds[`${projectId}-${target.name}`];
-      publishStorage.set('jobIds', publishJobIds);
-    }
     // the action below only applies to when a bot is being started using the "start bot" button
     // a check should be added to this that ensures this ONLY applies to the "default" profile.
     if (target.name === defaultPublishConfig.name) {

Composer/packages/electron-server/src/preload.js

@@ -1,9 +1,13 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-const { ipcRenderer } = require('electron'); // eslint-disable-line
+const { ipcRenderer, contextBridge } = require('electron'); // eslint-disable-line
 
 // expose ipcRenderer to the browser
-window.ipcRenderer = ipcRenderer;
+contextBridge.exposeInMainWorld('ipcRenderer', {
+  send: (...args) => ipcRenderer.send(...args),
+  on: (...args) => ipcRenderer.on(...args),
+});
+
 // flag to distinguish electron client from web app client
-window.__IS_ELECTRON__ = true;
+contextBridge.exposeInMainWorld('__IS_ELECTRON__', true);

Composer/packages/integration-tests/cypress/integration/LuisDeploy.spec.ts

@@ -36,7 +36,7 @@ context('Luis Deploy', () => {
       response: 'fixture:luPublish/success',
     });
     cy.findByTestId('startBotButton').click();
-    cy.findByText(/^Starting bot../);
+    cy.findAllByText(/^Starting bot../);
     cy.route('GET', '/api/publish/*/status/default', { endpointURL: 'anything', status: 200 });
     cy.route('PUT', '/api/projects/*/files/appsettings.json', { status: 200 });
   });

Composer/packages/integration-tests/cypress/support/commands.d.ts

@@ -5,6 +5,11 @@
 
 declare namespace Cypress {
   interface Chainable {
+    /**
+     * Get CSRF token
+     * @example cy.getCSRFToken().then(token => ...)
+     */
+    getCSRFToken(): Chainable<string>;
     /**
      * Creates a bot based on empty bot.
      * @example cy.createBot('EmptySample', ()=> {})

Composer/packages/integration-tests/cypress/support/commands.ts

@@ -5,6 +5,49 @@ import '@testing-library/cypress/add-commands';
 
 let TemplateBotProjectId = '';
 
+let csrfToken: string;
+
+Cypress.Commands.add('getCSRFToken', () => {
+  if (csrfToken) {
+    return cy.wrap(csrfToken);
+  }
+  cy.visit('/');
+  return cy
+    .window()
+    .its('__csrf__')
+    .then((csrf) => {
+      csrfToken = csrf;
+      return csrf;
+    });
+});
+
+Cypress.Commands.overwrite('request', (originalFn, ...options) => {
+  return cy.getCSRFToken().then((csrf) => {
+    const headers = {
+      'X-CSRF-Token': csrf,
+    };
+
+    const optionsObject = options[0];
+
+    if (optionsObject === Object(optionsObject)) {
+      optionsObject.headers = {
+        ...headers,
+        ...optionsObject.headers,
+      };
+
+      return originalFn(optionsObject);
+    }
+
+    const [method, url, body] = options;
+    return originalFn({
+      method,
+      url,
+      body,
+      headers,
+    });
+  });
+});
+
 Cypress.Commands.add('createBot', (botName: string, callback?: (bot: any) => void) => {
   const params = {
     description: '',
@@ -60,6 +103,7 @@ Cypress.Commands.add('createTestBot', (botName: string, callback?: (bot: any) =>
     storageId: 'default',
   };
 
+  cy.wrap(TemplateBotProjectId).should('not.be.empty');
   cy.request('post', `/api/projects/${TemplateBotProjectId}/project/saveAs`, params).then((res) => {
     callback?.(res.body);
   });

Composer/packages/integration-tests/cypress/support/index.ts

@@ -5,6 +5,7 @@ import './commands';
 
 before(() => {
   cy.exec('yarn clean-all');
+  cy.getCSRFToken();
   cy.createTemplateBot('EmptySample');
 });
 

Composer/packages/intellisense/src/hooks/useLanguageServer.ts

@@ -52,6 +52,10 @@ export const useLanguageServer = (
     ws.current.onmessage = (messageText) => {
       handleMessage(messageText);
     };
+
+    return () => {
+      ws.current.close();
+    };
   }, [url]);
 
   // If scopes change, update backend with info