[[FIX]] Re-enable library checking (#14)

Also, drop src from distro
gitter-badger · Dec 19, 2016 · a3be946 · a3be946
1 parent 84f1568
commit a3be946
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 2 deletions.
diff --git a/.npmignore b/.npmignore
@@ -4,6 +4,10 @@ node_modules/
 coverage/
 test/
 typings/
+src/
+tsconfig.json
+tslint.json
+typings.json
 
 npm-debug.log
 .eslintrc
@@ -12,3 +16,6 @@ npm-debug.log
 .travis.yml
 
 CONTRIBUTING.md
+
+*.spec.js
+*.spec.ts
diff --git a/package.json b/package.json
@@ -4,6 +4,9 @@
   "description": "Bot Core package to run bots",
   "main": "lib/index.js",
   "typings": "lib/index.d.ts",
+  "files": [
+    "lib"
+  ],
   "scripts-info": {
     "info": "List available scripts",
     "build": "Builds the plugin, by cleaning and compiling TypesScript into Javascript",

diff --git a/src/loader.ts b/src/loader.ts
@@ -33,8 +33,7 @@ export class PluginLoader {
     protected requireLibrary(libpath: string): BotBuilder.Library {
         let lib = require(libpath).default as BotBuilder.Library;
 
-        // XXX this validation could be improved checking that lib instanceof BotBuilder.Library
-        if (!lib.name) {
+        if (!(lib instanceof BotBuilder.Library)) {
             logger.warn(`Skip plugin at ${libpath}: not a valid Library`);
             return null;
         }