{"action":"created","issue":{"url":"https://api.github.com/repos/aurelia/cli/issues/862","repository_url":"https://api.github.com/repos/aurelia/cli","labels_url":"https://api.github.com/repos/aurelia/cli/issues/862/labels{/name}","comments_url":"https://api.github.com/repos/aurelia/cli/issues/862/comments","events_url":"https://api.github.com/repos/aurelia/cli/issues/862/events","html_url":"https://github.com/aurelia/cli/pull/862","id":310115576,"number":862,"title":"feat(bundler): auto tracing for requirejs/systemjs","user":{"login":"huochunpeng","id":1188806,"avatar_url":"https://avatars3.githubusercontent.com/u/1188806?v=4","gravatar_id":"","url":"https://api.github.com/users/huochunpeng","html_url":"https://github.com/huochunpeng","followers_url":"https://api.github.com/users/huochunpeng/followers","following_url":"https://api.github.com/users/huochunpeng/following{/other_user}","gists_url":"https://api.github.com/users/huochunpeng/gists{/gist_id}","starred_url":"https://api.github.com/users/huochunpeng/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/huochunpeng/subscriptions","organizations_url":"https://api.github.com/users/huochunpeng/orgs","repos_url":"https://api.github.com/users/huochunpeng/repos","events_url":"https://api.github.com/users/huochunpeng/events{/privacy}","received_events_url":"https://api.github.com/users/huochunpeng/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":18,"created_at":"2018-03-30T16:58:15Z","updated_at":"2018-03-31T22:25:05Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/aurelia/cli/pulls/862","html_url":"https://github.com/aurelia/cli/pull/862","diff_url":"https://github.com/aurelia/cli/pull/862.diff","patch_url":"https://github.com/aurelia/cli/pull/862.patch"},"body":"remove all lib/importer files.\r\nremove most of amodro-tracing.\r\nmanual efficient js and html tracing for aurelia.\r\nremove irrelevant tracing cache.\r\ncover all aurelia-webpack-plugin covers, plus more edge cases.\r\nmore permissive, usage of PLATFORM.moduleName is NOT required.\r\nalmost zero manual maintenance of aurelia.json dependencies.\r\nfix node id compatibility issue #831 for both requirejs and systemjs.\r\nfix bundles moduleIds format for modules need plugin.\r\nsimplify and reduce source code, but increase test coverge.\r\nadd \"au run --watch --auto-install\" for frictionless dev flow.\r\nbest guess on package manager 'yarn' to avoid 'npm i' error on yarn\r\ncontrolled node_modules.\r\novertake webpack on simplicity, usability, compatibility and flexibility,\r\nbut probably not on performance.\r\nperform slightly faster than existing amodro-trace + cache.\r\n\r\nSupersede #853, #842, #831\r\nBREAKING CHANGE: require minor user code change to support non-js main, like \"main\": \"font-awesome.css\"\r\nBREAKING CHANGE: remove support of undocumented \"main\": false, replace with generic \"lazyMain\": true"},"comment":{"url":"https://api.github.com/repos/aurelia/cli/issues/comments/377727757","html_url":"https://github.com/aurelia/cli/pull/862#issuecomment-377727757","issue_url":"https://api.github.com/repos/aurelia/cli/issues/862","id":377727757,"user":{"login":"huochunpeng","id":1188806,"avatar_url":"https://avatars3.githubusercontent.com/u/1188806?v=4","gravatar_id":"","url":"https://api.github.com/users/huochunpeng","html_url":"https://github.com/huochunpeng","followers_url":"https://api.github.com/users/huochunpeng/followers","following_url":"https://api.github.com/users/huochunpeng/following{/other_user}","gists_url":"https://api.github.com/users/huochunpeng/gists{/gist_id}","starred_url":"https://api.github.com/users/huochunpeng/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/huochunpeng/subscriptions","organizations_url":"https://api.github.com/users/huochunpeng/orgs","repos_url":"https://api.github.com/users/huochunpeng/repos","events_url":"https://api.github.com/users/huochunpeng/events{/privacy}","received_events_url":"https://api.github.com/users/huochunpeng/received_events","type":"User","site_admin":false},"created_at":"2018-03-31T22:25:04Z","updated_at":"2018-03-31T22:25:04Z","author_association":"CONTRIBUTOR","body":"@CasiOo thx for the feedback. I have been considering the option, it is not an important feature to keep, just a showcase of extra flexibility of auto tracing. Nothing is set yet, but I sided on removing it.\r\n\r\nSome additional notes:\r\n* Users need explicitly turn on auto-install on command line if they want to use it, by default it's off.\r\n* `aurelia-cli` already has installer built in, it can help to install dependencies after `au new`. So there is actually only very little code for me to support auto-install for `au run` (and `au build`).\r\n* The auto-install also behaves differently from normal `npm install` or `yarn add`. A normal install only install the package and its dependencies specified in its `package.json`. But auto-install is based on code tracing, so it can auto install `jquery` and `popper.js` (they are peerDependencies in `package.json`) for `bootstrap`.\r\n"}}
{"actor":{"display_login":"huochunpeng"}}