{"action":"created","issue":{"url":"https://api.github.com/repos/madskristensen/NpmTaskRunner/issues/20","repository_url":"https://api.github.com/repos/madskristensen/NpmTaskRunner","labels_url":"https://api.github.com/repos/madskristensen/NpmTaskRunner/issues/20/labels{/name}","comments_url":"https://api.github.com/repos/madskristensen/NpmTaskRunner/issues/20/comments","events_url":"https://api.github.com/repos/madskristensen/NpmTaskRunner/issues/20/events","html_url":"https://github.com/madskristensen/NpmTaskRunner/issues/20","id":141943889,"number":20,"title":"[Feature request] Run default npm scripts from Task Runner","user":{"login":"BrainCrumbz","id":3185573,"avatar_url":"https://avatars.githubusercontent.com/u/3185573?v=3","gravatar_id":"","url":"https://api.github.com/users/BrainCrumbz","html_url":"https://github.com/BrainCrumbz","followers_url":"https://api.github.com/users/BrainCrumbz/followers","following_url":"https://api.github.com/users/BrainCrumbz/following{/other_user}","gists_url":"https://api.github.com/users/BrainCrumbz/gists{/gist_id}","starred_url":"https://api.github.com/users/BrainCrumbz/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BrainCrumbz/subscriptions","organizations_url":"https://api.github.com/users/BrainCrumbz/orgs","repos_url":"https://api.github.com/users/BrainCrumbz/repos","events_url":"https://api.github.com/users/BrainCrumbz/events{/privacy}","received_events_url":"https://api.github.com/users/BrainCrumbz/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":25,"created_at":"2016-03-18T18:26:35Z","updated_at":"2016-03-21T20:47:11Z","closed_at":null,"body":"### Installed product versions\r\n-  Visual Studio: 2015 Community ed.\r\n- This extension: 1.1.44\r\n\r\n### Description\r\nThinking back, [your comment](https://github.com/madskristensen/NpmTaskRunner/issues/19#issuecomment-198397980) on issue #19 got us thinking:\r\n\r\n> Great idea. What do we do if there is a postinstall script but no install? Do we create the install parent node in that case since it has to be executable for the postinstall to work?\r\n\r\nAt the moment there's no way to run default npm commands from the Task Runner, unless they are actually listed. So e.g. we have defined `start` in our scripts, and we can run it, but e.g. we did not define `install`, but still we can run that from console. \r\n\r\nMaybe it could be worth it to add them to the Runner as well. They'd be some kind of *ghost* items, or *implicit* items, that are listed even if not actually present.\r\n\r\nIn this scenario, your earlier comment comes back to place: if scripts contains `postinstall`, that would be nested under the *implicit* `install`.\r\n\r\nDoes that make sense? (For this same release, in case?)"},"comment":{"url":"https://api.github.com/repos/madskristensen/NpmTaskRunner/issues/comments/199474705","html_url":"https://github.com/madskristensen/NpmTaskRunner/issues/20#issuecomment-199474705","issue_url":"https://api.github.com/repos/madskristensen/NpmTaskRunner/issues/20","id":199474705,"user":{"login":"scottaddie","id":10702007,"avatar_url":"https://avatars.githubusercontent.com/u/10702007?v=3","gravatar_id":"","url":"https://api.github.com/users/scottaddie","html_url":"https://github.com/scottaddie","followers_url":"https://api.github.com/users/scottaddie/followers","following_url":"https://api.github.com/users/scottaddie/following{/other_user}","gists_url":"https://api.github.com/users/scottaddie/gists{/gist_id}","starred_url":"https://api.github.com/users/scottaddie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/scottaddie/subscriptions","organizations_url":"https://api.github.com/users/scottaddie/orgs","repos_url":"https://api.github.com/users/scottaddie/repos","events_url":"https://api.github.com/users/scottaddie/events{/privacy}","received_events_url":"https://api.github.com/users/scottaddie/received_events","type":"User","site_admin":false},"created_at":"2016-03-21T20:47:11Z","updated_at":"2016-03-21T20:47:11Z","body":"@madskristensen Here's one other potential problem I noticed. The user shouldn't be allowed to bind, for example, the entire `dependencies` node and 1 of its children (e.g., `babel-polyfill`) to the same Visual Studio event. Here's a screenshot to clarify:\r\n\r\n![bindings](https://cloud.githubusercontent.com/assets/10702007/13933571/d6894c9c-ef7b-11e5-99e5-c6f16dd60d20.png)\r\n\r\nSince `dependencies` is already attached to the binding, `babel-polyfill` would be accounted for in the uninstall which runs during the \"Before Build\" event. How should this be handled? "}}