Project references should be automatically npm installed

(from https://nodejstools.codeplex.com/discussions/572866) pminaev wrote: BTW, one thing that NTVS could do in this case is automatically npm install the referenced project as if it >were a modul...

Id #1580 | Release: None | Updated: Sun at 5:38 AM by pminaev | Created: Sun at 5:38 AM by pminaev

Handle GetPathToNpm throwing exceptions or don't throw.

We throw an exception when we can't find Npm which causes us not to display the correct dialog. throw new NpmNotFoundException( string.Format( "Cannot find npm.cmd at ...

Id #1579 | Release: None | Updated: Sat at 1:26 AM by crwilcox | Created: Sat at 1:26 AM by crwilcox

Navigation to typescript test opens the .js instead of .ts

From test explorer, double-click on a test. It will open the wrong file.

Id #1578 | Release: None | Updated: Sat at 1:17 AM by huguesv | Created: Sat at 1:17 AM by huguesv

For loops indent comments when they shouldn't

Format Below: function f() { for(var i = 0; i<0; i++) /* var x; var y; */ var z; var w; } Expect: function f() { for(var i = 0; i<0; i++) /* var x; var y; */ var z; var w; } Actual: funct...

Id #1577 | Release: None | Updated: Fri at 11:17 PM by crwilcox | Created: Fri at 11:17 PM by crwilcox

Improve indenting of multiline comments

And incorrectly indents things that follow them. Format: { /* foo bar **/ baz } Current: { /* foo bar **/ baz; } Best Result would be: { /* * foo * bar **/ baz; }

Id #1576 | Release: None | Updated: Fri at 10:58 PM by crwilcox | Created: Fri at 10:57 PM by crwilcox

npm commands in repl or ui often fail to exit

this locks up the repl/dialog

Id #1575 | Release: None | Updated: Sat at 12:45 AM by mousetraps | Created: Fri at 12:14 PM by mousetraps

Crash in require

<description>System.NullReferenceException: Object reference not set to an instance of an object. at Microsoft.NodejsTools.Analysis.ModuleAnalysis.GetModules() in d:\DevDiv\Offcycle\xTVS\Open_So...

Id #1574 | Release: None | Updated: Thu at 6:33 PM by Zooba | Created: Thu at 6:05 PM by RickWinter

Change npm install output message

When we are downloading the message should state "npm install output: Downloading npm list..."

Id #1573 | Release: None | Updated: Thu at 5:51 PM by RickWinter | Created: Thu at 5:51 PM by RickWinter

Selecting "never again" in path too long dialog doesn't change setting

Option in Tools Options didn't change.

Id #1572 | Release: None | Updated: Thu at 6:36 PM by pminaev | Created: Thu at 12:49 AM by Zooba

Comments after braces misformatted

Before: if ('open' == this.conn.readyState) { debug('writing packet %j', packet); if(!preEncoded) { // not broadcasting, need to encode this.encoder.encode(packet, function (encodedPackets) { /...

Id #1571 | Release: None | Updated: Fri at 8:10 PM by crwilcox | Created: Thu at 12:33 AM by Zooba