Stopping timer to stdin, handling wrong input in lu

Author: munozemilio

packages/command/src/readpipeddata.ts

@@ -10,6 +10,8 @@ const ReadPipedStdin = {
       }
 
       const timer = setTimeout(async () => {
+        clearTimeout(timer)	
+        if (input) return resolve(input)
         reject(new Error('No input'))
       }, 1000)
 

packages/lu/src/commands/luis/translate.ts

@@ -28,14 +28,9 @@ export default class LuisTranslate extends Command {
       const {flags} = this.parse(LuisTranslate)
       // Check if data piped in stdin
       let stdin = await this.readStdin()
-      let outputStat = flags.out ? await fs.stat(flags.out) : null
-
-      if (outputStat && outputStat.isFile()) {
-        throw new CLIError('Output can only be writen to a folder')
-      }
-
       let isLu = await fileHelper.detectLuContent(stdin, flags.in)
       let result: any
+
       if (isLu) {
         let luFiles = await fileHelper.getLuObjects(stdin, flags.in, flags.recurse, fileExtEnum.LUFile)
         result = await luTranslator.translateLuList(luFiles, flags.translatekey, flags.tgtlang, flags.srclang, flags.translate_comments, flags.translate_link_text)

packages/lu/src/commands/qnamaker/translate.ts

@@ -28,11 +28,6 @@ export default class QnamakerTranslate extends Command {
       const {flags} = this.parse(QnamakerTranslate)
       // Check if data piped in stdin
       let stdin = await this.readStdin()
-      let outputStat = flags.out ? await fs.stat(flags.out) : null
-
-      if (outputStat && outputStat.isFile()) {
-        throw new CLIError('Output can only be writen to a folder')
-      }
 
       let isLu = await fileHelper.detectLuContent(stdin, flags.in)
       let result: any

packages/lu/src/utils/filehelper.ts

@@ -82,9 +82,16 @@ export async function generateNewFilePath(outFileName: string, inputfile: string
 
 export async function generateNewTranslatedFilePath(fileName: string, translatedLanguage: string, output: string): Promise<string> {
   let newPath = path.resolve(output)
+
+  let extension = path.extname(newPath)
+  if (extension) {
+    throw new CLIError('Output can only be writen to a folder')
+  }
+
   if (!fs.existsSync(newPath)) {
     throw new CLIError('Path not found: ' + newPath)
   }
+
   newPath = path.join(output, translatedLanguage)
   await fs.mkdirp(newPath)
   return path.join(newPath, fileName)