Fixed bug with recognizers being called twice.

Author: Stevenic

Node/core/src/bots/Library.ts

@@ -127,12 +127,10 @@ export class Library extends EventEmitter {
         //   top intent is passed along as part of routing. The root libraries recognize()
         //   method will be called a second time so we want to avoid the second recognize
         //   pass.
-        var skipRecognize = (context.intent && context.libraryName === this.name);
-        if (this.recognizers.length > 0 && !skipRecognize) {
+        if (this.recognizers.length > 0 && context.libraryName !== this.name) {
             this.recognizers.recognize(context, callback);
         } else {
-            // Pass through the top intent recognized by the root libraries recognizers.
-            callback(null, context.intent);
+            callback(null, context.intent || { intent: 'None', score: 0.0 });
         }
     }
 

Node/core/src/bots/UniversalBot.ts

@@ -464,12 +464,12 @@ export class UniversalBot extends Library {
         // Run the root libraries recognizers
         var context = session.toRecognizeContext();
         this.recognize(context, (err, topIntent) => {
-            if (topIntent && topIntent.score > 0) {
-                // This intent will be automatically inherited by child libraries
-                // that don't implement their own recognizers.
-                context.intent = topIntent;
-                context.libraryName = this.name;
-            }
+            // This intent will be automatically inherited by child libraries
+            // that don't implement their own recognizers.
+            // - We're passing along the library name to avoid running our own
+            //   recognizer twice.
+            context.intent = topIntent;
+            context.libraryName = this.name;
 
             // Federate across all libraries to find the best route to trigger. 
             var results = Library.addRouteResult({ score: 0.0, libraryName: this.name });