make "compileOnSave" option of tsconfig.json recognized by vscode

[zpdDG4gta8XKpMCd]

Currently "compileOnSave" option of tsconfig.json is ignored by the VS Code:

• http://stackoverflow.com/questions/29996145/visual-studio-code-compile-on-save

There is no a workaround to this problem. Please consider doing something about bringing it to life.

[dbaeumer]

Added a comment to stackoverflow. This is supported right now via a watch task. But I agree that the tsconfig.json option should be supported as well, which has to come from the TS team though.

[dbaeumer]

I opened microsoft/TypeScript#8884 since the support has to go there. I will close this issue as upstream.

[zpdDG4gta8XKpMCd]

the problem with "watch" is that vscode doesn't allow to run the test command while watching typescript, saying there is another command running, so at least with having to run the tests from under vscode "watch" isn't an option

[dbaeumer]

We have feature request to allow more than one command.

[dbaeumer]

As said I close this one since it will be addressed in tsserver.

[zpdDG4gta8XKpMCd]

Merged microsoft/TypeScript#9837

[styfle]

@dbaeumer Now that it is merged in tsserver will it simply work with vscode or will there need to be more work to integrate it?

[dbaeumer]

@styfle it should work with VS Code without additional integration effort. But please note that VS Code still ships with 1.8.10 and I think the feature will not make it out of the box into 2.0 either.

[mrcrowl]

@dbaeumer From what I can see, there is additional integration effort needed to enable compile on save in VSCode for TypeScript. Since v2.0.5, tsserver has supported the new API methods compileOnSaveAffectedFileList and compileOnSaveEmitFile, but they need to be called from the typescript extension. I've done some prelim work on getting this going for myself and would be happy to make a pull request if there was support? (WIP here if you want to have a look: https://github.com/mrcrowl/vscode/tree/master/extensions/typescript)

[dbaeumer]

@mrcrowl you are correct that the current implementation in the tsserver does require quite some work in the extensions. I started a discussion with the TS team about this here: microsoft/TypeScript#11526

The bottom line is that this doesn't belong into the extension since the extension can never implement this in a efficient way (and the TS team agrees on this). Especially since compileOnSave must also do problem checking. Doesn't make to much sense to write a new JS file without telling the user that you now have compile errors in dependent files.

So I would opt that instead of adding this to the extension your effort is better put into the tsserver to make this work. I started this along time ago here: https://github.com/Microsoft/TypeScript/blob/dbaeumer/9125/src/server/builder.ts and parts of the work made it into 2.0.6. But still no full builder. Their current builder only manages dependencies: https://github.com/Microsoft/TypeScript/blob/master/src/server/builder.ts

[Ciantic]

Shouldn't this be re-opened if there is more work to be done in VSCode code base? I'm running 2.2.0-dev in stable VSCode, the compile on save does not work with it.

Edit: Reason this bug would be nice to be open, so one could track when it becomes feasible to use.

Edit2: Or is this the main bug microsoft/TypeScript#11526 one could track for this feature?