Switch luis sdk to node-fetch (#829)

* Switch luis sdk to node-fetch

* Fixing export in lubuild

Co-authored-by: Vishwac Sena Kannan <[email protected]>

Author: munozemilio

packages/lu/src/parser/lubuild/core.ts

@@ -126,13 +126,20 @@ export class LuBuildCore {
   }
 
   public async exportApplication(appId: string, versionId: string) {
-    let response
+    const url = this.endpoint + '/luis/authoring/v3.0-preview/apps/' + appId + '/versions/' + versionId + '/export?format=json'
+    const headers = {
+      'Content-Type': 'application/json',
+      'Ocp-Apim-Subscription-Key': this.subscriptionKey
+    }
+
+    let messageData
     let retryCount = this.retryCount + 1
     let error
     while (retryCount > 0) {
       if (error === undefined || error.statusCode === rateLimitErrorCode) {
         try {
-          response = await this.client.versions.exportMethod(appId, versionId)
+          const response = await fetch(url, {method: 'GET', headers})
+          messageData = await response.json()
           break
         } catch (e) {
           error = e
@@ -148,7 +155,11 @@ export class LuBuildCore {
       throw error
     }
 
-    return response
+    if (messageData.error) {
+      throw (new exception(retCode.errorCode.LUIS_API_CALL_FAILED, messageData.error.message))
+    }
+
+    return messageData
   }
 
   public compareApplications(currentApp: any, existingApp: any) {

packages/luis/src/commands/luis/version/export.ts

@@ -4,6 +4,7 @@
  */
 
 import {CLIError, Command, flags} from '@microsoft/bf-cli-command'
+import fetch from 'node-fetch'
 
 const utils = require('../../../utils/index')
 
@@ -41,16 +42,25 @@ export default class LuisVersionExport extends Command {
     const requiredProps = {appId, versionId, endpoint, subscriptionKey}
     utils.validateRequiredProps(requiredProps)
 
-    const client = utils.getLUISClient(subscriptionKey, endpoint)
-
     try {
-      const appJSON = await client.versions.exportMethod(appId, versionId)
-      if (!appJSON) throw new CLIError('Failed to export file')
+      let url = endpoint + '/luis/authoring/v3.0-preview/apps/' + appId + '/versions/' + versionId + '/export?format=json'
+      const headers = {
+        'Content-Type': 'application/json',
+        'Ocp-Apim-Subscription-Key': subscriptionKey
+      }
+
+      const response = await fetch(url, {method: 'GET', headers})
+      const messageData = await response.json()
+
+      if (messageData.error) {
+        throw new CLIError(messageData.error.message)
+      }
+
       if (out) {
-        const writtenFilePath: string = await utils.writeToFile(out, appJSON, force)
+        const writtenFilePath: string = await utils.writeToFile(out, messageData, force)
         this.log(`File successfully written: ${writtenFilePath}`)
       } else {
-        await utils.writeToConsole(appJSON)
+        await utils.writeToConsole(messageData)
       }
     } catch (error) {
       throw new CLIError(error)

packages/luis/test/commands/luis/version/export.test.ts

@@ -51,12 +51,19 @@ describe('luis:version:export', () => {
   test
   .nock('https://westus.api.cognitive.microsoft.com', api => api
   .get(uri => uri.includes('export'))
-  .reply(200, {name: 'testname'})
+  .reply(200, {
+    name: "Utilities.Cancel",
+    inherits: {
+      domain_name: "Utilities",
+      model_name: "Cancel"
+    },
+    "features": []
+})
   )
   .stdout()
   .command(['luis:version:export', '--appId', uuidv1(), '--versionId', '0.1', '--subscriptionKey', uuidv1(), '--endpoint', 'https://westus.api.cognitive.microsoft.com'])
   .it('exports a luis app and displays the export contents in the console', ctx => {
-    expect(ctx.stdout).to.contain('testname')
+    expect(ctx.stdout).to.contain('domain_name')
   })
 
   test