ignore files inside node_modules when removing declarations files

.gitignore

@@ -12,6 +12,7 @@
 /test/tmp
 /test/build
 !/test/src/typings/*.d.ts
+!/test/src/node_modules/*.d.ts
 /test/src/**/*.d.ts
 /test/src/**/*.js
 /test/src/**/*.js.map

lib/index.ts

@@ -22,6 +22,7 @@ const identifierExp = /^\w+(?:[\.-]\w+)*$/;
 const fileExp = /^([\./].*|.:.*)$/;
 const privateExp = /^[ \t]*(?:static )?private (?:static )?/;
 const publicExp = /^([ \t]*)(static |)(public |)(static |)(.*)/;
+const nodeModulesExp = /\/node_modules\//;
 
 export interface Options {
     main: string;
@@ -374,11 +375,11 @@ export function bundle(options: Options): BundleResult {
     if (removeSource) {
         trace('\n### remove source typings ###');
 
-        sourceTypings.forEach(p => {
+        sourceTypings.forEach((filePath: string) => {
             // safety check, only delete .d.ts files, leave our outFile intact for now
-            if (p !== outFile && dtsExp.test(p) && fs.statSync(p).isFile()) {
-                trace(' - %s', p);
-                fs.unlinkSync(p);
+            if (filePath !== outFile && dtsExp.test(filePath) && !nodeModulesExp.test(filePath) && fs.statSync(filePath).isFile()) {
+                trace(' - %s', filePath);
+                fs.unlinkSync(filePath);
             }
         });
     }

package.json

@@ -41,7 +41,7 @@
     "@types/detect-indent": "0.1.30",
     "@types/glob": "5.0.30",
     "@types/mkdirp": "0.3.29",
-    "@types/node": "8.0.0",
+    "@types/node": "8.0.14",
     "commander": "^2.9.0",
     "detect-indent": "^0.2.0",
     "glob": "^6.0.4",

test/expected/default/foo-mx.d.ts

@@ -1,13 +1,16 @@
 // Dependencies for this module:
 //   ../../src/typings/external.d.ts
+//   ../../src/node_modules/modules.d.ts
 
 declare module 'foo-mx' {
     import exp = require('foo-mx/lib/exported-sub');
     import mod1 = require('external1');
+    import nodeModule = require('node_module');
     export import Foo = require('foo-mx/Foo');
     export function run(foo?: Foo): Foo;
     export function flep(): exp.ExternalContainer;
     export function bar(): mod1.SomeType;
+    export function baz(): nodeModule.SomeType;
 }
 
 declare module 'foo-mx/lib/exported-sub' {

test/expected/excludeExp/foo-mx.d.ts

@@ -1,13 +1,16 @@
 // Dependencies for this module:
 //   ../../src/typings/external.d.ts
+//   ../../src/node_modules/modules.d.ts
 
 declare module 'foo-mx' {
     import exp = require('foo-mx/lib/exported-sub');
     import mod1 = require('external1');
+    import nodeModule = require('node_module');
     export import Foo = require('foo-mx/Foo');
     export function run(foo?: Foo): Foo;
     export function flep(): exp.ExternalContainer;
     export function bar(): mod1.SomeType;
+    export function baz(): nodeModule.SomeType;
 }
 
 declare module 'foo-mx/Foo' {

test/expected/excludeFunc/foo-mx.d.ts

@@ -1,13 +1,16 @@
 // Dependencies for this module:
 //   ../../src/typings/external.d.ts
+//   ../../src/node_modules/modules.d.ts
 
 declare module 'foo-mx' {
     import exp = require('foo-mx/lib/exported-sub');
     import mod1 = require('external1');
+    import nodeModule = require('node_module');
     export import Foo = require('foo-mx/Foo');
     export function run(foo?: Foo): Foo;
     export function flep(): exp.ExternalContainer;
     export function bar(): mod1.SomeType;
+    export function baz(): nodeModule.SomeType;
 }
 
 declare module 'foo-mx/Foo' {

test/expected/externals/foo-mx.d.ts

@@ -2,10 +2,12 @@
 declare module 'foo-mx' {
     import exp = require('foo-mx/lib/exported-sub');
     import mod1 = require('foo-mx/index//external1');
+    import nodeModule = require('foo-mx/index//node_module');
     export import Foo = require('foo-mx/Foo');
     export function run(foo?: Foo): Foo;
     export function flep(): exp.ExternalContainer;
     export function bar(): mod1.SomeType;
+    export function baz(): nodeModule.SomeType;
 }
 
 declare module 'foo-mx/index//external1' {
@@ -20,6 +22,12 @@ declare module 'foo-mx/index//external2' {
     }
 }
 
+declare module 'foo-mx/index//node_module' {
+    export class SomeType {
+        foo(): void;
+    }
+}
+
 declare module 'foo-mx/lib/exported-sub' {
     import Foo = require('foo-mx/Foo');
     import mod2 = require('foo-mx/index//external2');

test/expected/includeExclude/foo-mx.d.ts

@@ -2,10 +2,12 @@
 declare module 'foo-mx' {
     import exp = require('foo-mx/lib/exported-sub');
     import mod1 = require('foo-mx/index//external1');
+    import nodeModule = require('foo-mx/index//node_module');
     export import Foo = require('foo-mx/Foo');
     export function run(foo?: Foo): Foo;
     export function flep(): exp.ExternalContainer;
     export function bar(): mod1.SomeType;
+    export function baz(): nodeModule.SomeType;
 }
 
 declare module 'foo-mx/index//external1' {
@@ -20,6 +22,12 @@ declare module 'foo-mx/index//external2' {
     }
 }
 
+declare module 'foo-mx/index//node_module' {
+    export class SomeType {
+        foo(): void;
+    }
+}
+
 declare module 'foo-mx/Foo' {
     class Foo {
             foo: string;

test/expected/out/fizz/buzz.d.ts

@@ -1,13 +1,16 @@
 // Dependencies for this module:
 //   ../../src/typings/external.d.ts
+//   ../../src/node_modules/modules.d.ts
 
 declare module 'foo-mx' {
     import exp = require('foo-mx/lib/exported-sub');
     import mod1 = require('external1');
+    import nodeModule = require('node_module');
     export import Foo = require('foo-mx/Foo');
     export function run(foo?: Foo): Foo;
     export function flep(): exp.ExternalContainer;
     export function bar(): mod1.SomeType;
+    export function baz(): nodeModule.SomeType;
 }
 
 declare module 'foo-mx/lib/exported-sub' {

test/expected/remove/foo-mx.d.ts

@@ -1,13 +1,16 @@
 // Dependencies for this module:
 //   ../../src/typings/external.d.ts
+//   ../../src/node_modules/modules.d.ts
 
 declare module 'foo-mx' {
     import exp = require('foo-mx/lib/exported-sub');
     import mod1 = require('external1');
+    import nodeModule = require('node_module');
     export import Foo = require('foo-mx/Foo');
     export function run(foo?: Foo): Foo;
     export function flep(): exp.ExternalContainer;
     export function bar(): mod1.SomeType;
+    export function baz(): nodeModule.SomeType;
 }
 
 declare module 'foo-mx/lib/exported-sub' {

test/expected/seprinnew/bar-mx.d.ts

@@ -1,13 +1,16 @@
 // Dependencies for this module: //$
 //   ../../src/typings/external.d.ts //$
+//   ../../src/node_modules/modules.d.ts //$
  //$
 declare module 'bar-mx' { //$
 	import exp = require('--bar-mx#lib#exported-sub'); //$
 	import mod1 = require('external1'); //$
+	import nodeModule = require('node_module'); //$
 	export import Foo = require('--bar-mx#Foo'); //$
 	export function run(foo?: Foo): Foo; //$
 	export function flep(): exp.ExternalContainer; //$
 	export function bar(): mod1.SomeType; //$
+	export function baz(): nodeModule.SomeType; //$
 } //$
  //$
 declare module '--bar-mx#lib#exported-sub' { //$

test/src/main/index.ts

@@ -1,8 +1,10 @@
 /// <reference path="./../typings/external.d.ts" />
+/// <reference path="./../node_modules/modules.d.ts" />
 
 import int = require('./lib/only-internal');
 import exp = require('./lib/exported-sub');
 import mod1 = require('external1');
+import nodeModule = require('node_module');
 
 export import Foo = require('./Foo');
 /*
@@ -23,3 +25,8 @@ export function flep(): exp.ExternalContainer {
 export function bar(): mod1.SomeType {
     return new mod1.SomeType();
 }
+
+// baz some of that
+export function baz(): nodeModule.SomeType {
+    return new nodeModule.SomeType();
+}