New files can't be added to a Node project while the server is running

From this reddit thread, the options to add new files e.g. new JavaScript, CSS, or HTML files is removed while the project is running: While there is no purpose to adding new server-side JavaScr...

Id #1583 | Release: None | Updated: Today at 12:03 AM by crwilcox | Created: Wed at 1:22 AM by TMiguelT

GetPathToNpm doesn't find npm properly

Was looking at the code for this method as part of another review and there are a few cases where it will find the wrong path: If InstallPath is set in the registry but does not contain npm.cmd, w...

Id #1582 | Release: None | Updated: Today at 12:02 AM by crwilcox | Created: Tue at 7:25 PM by Zooba

Please remove the "Node.js Interactive Window" menu item

Hi, NTVS: 1.0 RC VS: 2013 Ultimate Update 4 This option appears to have been added to the general context menu and hence appears for all projects everywhere. The NodeJS interactive window is...

Id #1581 | Release: None | Updated: Tue at 6:19 PM by nabog | Created: Tue at 6:18 PM by nabog

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: Wed at 10:05 PM 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: Nov 21 at 11:17 PM by crwilcox | Created: Nov 21 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: Nov 21 at 10:58 PM by crwilcox | Created: Nov 21 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: Nov 21 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: Nov 20 at 6:33 PM by Zooba | Created: Nov 20 at 6:05 PM by RickWinter