{"action":"created","comment":{"url":"https://api.github.com/repos/atom/atom/pulls/comments/189913550","pull_request_review_id":122183976,"id":189913550,"diff_hunk":"@@ -0,0 +1,100 @@\n+# Updateable Bundled Packages\n+\n+## Status\n+\n+Proposed\n+\n+## Summary\n+\n+> One paragraph explanation of the feature.\n+\n+This feature will enable an opt-in subset of bundled Atom packages to be updated with `apm` outside of the Atom release cycle.  This will enable users to receive new functionality and bug fixes for some bundled packages as regularly as needed without waiting for them to be included in a new Atom release.  This is especially important for packages like [GitHub](https://github.com/atom/github/) and  [Teletype](https://github.com/atom/teletype/) which provide essential Atom functionality and could be improved independently of Atom.\n+\n+## Motivation\n+\n+> Why are we doing this? What use cases does it support? What is the expected outcome?\n+\n+Atom currently uses a monthly release cycle with staged Stable and Beta releases so that major issues get caught early in Beta before reaching the Stable release.  Because Atom releases updates monthly, this means that a new feature merged into `master` right after a new Atom release could take one month to reach the next Beta and then another month to reach Stable.\n+\n+Since a large part of Atom's built-in functionality is provided by bundled packages, it makes sense to allow some of those packages to be updated independently of Atom's monthly release cycle so that users can receive new features and fixes whenever they become available.\n+\n+The primary use case for this improvement is enabling the GitHub package to ship improvements more frequently than Atom's release cycle since many of its improvements can be done without changes to Atom itself.  If this approach is proven to work well for the GitHub package, we might also consider using it to ship Teletype as a bundled Atom package.\n+\n+## Explanation\n+\n+> Explain the proposal as if it was already implemented in Atom and you were describing it to an Atom user. That generally means:\n+> - Introducing new named concepts.\n+> - Explaining the feature largely in terms of examples.\n+> - Explaining any changes to existing workflows.\n+\n+Bundled packages are treated differently than community packages that you can install using `apm`:\n+\n+- You are not prompted to update them when new versions are released on `apm`\n+- `apm` will warn you at the command line when you try to install or update a bundled package\n+- If a user intentionally installs a bundled package from `apm` the [Dalek package](https://github.com/atom/dalek/) will show a warning in the \"deprecations\" view asking the user to remove the offending package\n+\n+Despite all this, if the user *does* install a bundled package using `apm`, it will be loaded into the editor and updated dutifully as releases occur.\n+\n+### Implementation Details\n+\n+Because the necessary infrastructure is already in place to enable updates to bundled packages using `apm`, the only work required is to provide a way for packages to opt in to this behavior and for `apm` to include those packages in its update checks if they haven't already been installed in the user's packages folder.\n+\n+Any bundled Atom package will be able to opt in to updates by adding `\"updateable\": true` to its `package.json` file.  This will cause `apm` to consider it as part of the list of packages it checks for updates.  If a community (non-bundled) package sets this field to `true` or `false` it will be ignored as it's only relevant to bundled packages.\n+\n+`apm` will be updated to include the list of bundled packages with `\"updateable\": true` set in their `package.json` so that the user will be notified of new package versions that support the engine version of their current Atom build.\n+\n+### User Experience Examples\n+\n+1. The user downloads Atom 1.28.0 from atom.io which includes GitHub package version 0.15.0.  After Atom 1.28.0 was released, a hotfix release was shipped for the GitHub package as 0.15.1.  When the user installs and starts Atom, they are prompted to install the update to the GitHub package.\n+\n+2. The user downloads and installs Atom 1.28.0 from atom.io which includes GitHub package version 0.15.0.  Two weeks later, GitHub package 0.16.0 is released with a few new features.  The user is prompted to update to the new version and gets the new features even though Atom 1.29.0 hasn't been released yet.\n+\n+3. In the future, a user has an old install of Atom 1.28.0 and waits a long time between installing Atom updates.  The GitHub package releases version 0.25.0 but the user is not prompted to install it because the GitHub package has set `engines` in `package.json` to restrict to Atom 1.32.0 and above.\n+\n+### Rules for Updateable Bundled Packages\n+\n+Any package that opts into this behavior must follow one rule: **its `engines` field must be regularly updated to reflect the necessary Atom version for the Atom, Electron, and Node.js APIs used in the package**.  This field defines the range of Atom versions in which the package is expected to work.  The field should always be set to the lowest possible Atom version that the package supports.\n+\n+If a package wants to use API features of a newer version of Atom while still supporting older Atom versions, it must do so in a way that is aware of the user's version and adjust itself accordingly.\n+\n+## Drawbacks\n+\n+> Why should we *not* do this?\n+\n+The primary drawback of this approach is that updateable bundled packages might exhibit problems on older Atom versions due to missing or changed APIs in Atom, Electron, or Node.js.  The solution for these packages is to keep their `engines` field updated appropriately, but there's still a chance that some updates will slip through without the necessary engine version changes.\n+\n+One other possible drawback is that an updated version of a bundled package might not be compatible across two different Atom channels.  For example, if the user installs a new update to a bundled package that only supports the current Atom Beta release or higher, the user will no longer have access to that package if they open Atom Stable.\n+\n+However, this drawback is no different than what the user would face today installing a community package under the same circumstances, so this could be considered a general problem in the Atom package ecosystem.\n+\n+## Rationale and alternatives\n+\n+> - Why is this approach the best in the space of possible approaches?\n+> - What other approaches have been considered and what is the rationale for not choosing them?\n+> - What is the impact of not doing this?\n+\n+This is the best approach for updating bundled packages because it allows those packages to take control of their own release cycle so long as they manage their Atom engine version correctly.  It also does so in a way that allows us to decide which packages can be updated independently, reducing the likelihood of problems for users.\n+\n+The primary alternative to this approach is to speed up the Atom release cycle so that bundled Atom package updates will reach users more frequently.  This approach will be investigated independently of this RFC as it may still be valuable even with updateable bundled packages.\n+\n+## Unresolved questions\n+\n+> - What unresolved questions do you expect to resolve through the RFC process before this gets merged?\n+\n+Is it enough to just depend on the `engines` field of `package.json` to protect users from installing a package update that doesn't work with their version of Atom?\n+\n+Is `updateable` the right name for the field in `package.json`?  Is there a clearer name?","path":"docs/rfcs/001-updateable-bundled-packages.md","position":86,"original_position":86,"commit_id":"391bf82a419aad390b4b6e847d47cce423ea89ba","original_commit_id":"391bf82a419aad390b4b6e847d47cce423ea89ba","user":{"login":"jasonrudolph","id":2988,"avatar_url":"https://avatars3.githubusercontent.com/u/2988?v=4","gravatar_id":"","url":"https://api.github.com/users/jasonrudolph","html_url":"https://github.com/jasonrudolph","followers_url":"https://api.github.com/users/jasonrudolph/followers","following_url":"https://api.github.com/users/jasonrudolph/following{/other_user}","gists_url":"https://api.github.com/users/jasonrudolph/gists{/gist_id}","starred_url":"https://api.github.com/users/jasonrudolph/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasonrudolph/subscriptions","organizations_url":"https://api.github.com/users/jasonrudolph/orgs","repos_url":"https://api.github.com/users/jasonrudolph/repos","events_url":"https://api.github.com/users/jasonrudolph/events{/privacy}","received_events_url":"https://api.github.com/users/jasonrudolph/received_events","type":"User","site_admin":true},"body":"![screen shot 2018-05-22 at 10 04 03 am](https://user-images.githubusercontent.com/2988/40367492-90b82412-5da7-11e8-9bd2-8e68a788b194.png)\r\n\r\n😆 :trollface: ","created_at":"2018-05-22T14:05:11Z","updated_at":"2018-05-22T14:05:11Z","html_url":"https://github.com/atom/atom/pull/17379#discussion_r189913550","pull_request_url":"https://api.github.com/repos/atom/atom/pulls/17379","author_association":"MEMBER","_links":{"self":{"href":"https://api.github.com/repos/atom/atom/pulls/comments/189913550"},"html":{"href":"https://github.com/atom/atom/pull/17379#discussion_r189913550"},"pull_request":{"href":"https://api.github.com/repos/atom/atom/pulls/17379"}},"in_reply_to_id":189599497},"pull_request":{"url":"https://api.github.com/repos/atom/atom/pulls/17379","id":189386488,"html_url":"https://github.com/atom/atom/pull/17379","diff_url":"https://github.com/atom/atom/pull/17379.diff","patch_url":"https://github.com/atom/atom/pull/17379.patch","issue_url":"https://api.github.com/repos/atom/atom/issues/17379","number":17379,"state":"open","locked":false,"title":"Add proposed RFC 'Updateable Bundled Packages'","user":{"login":"daviwil","id":79405,"avatar_url":"https://avatars3.githubusercontent.com/u/79405?v=4","gravatar_id":"","url":"https://api.github.com/users/daviwil","html_url":"https://github.com/daviwil","followers_url":"https://api.github.com/users/daviwil/followers","following_url":"https://api.github.com/users/daviwil/following{/other_user}","gists_url":"https://api.github.com/users/daviwil/gists{/gist_id}","starred_url":"https://api.github.com/users/daviwil/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/daviwil/subscriptions","organizations_url":"https://api.github.com/users/daviwil/orgs","repos_url":"https://api.github.com/users/daviwil/repos","events_url":"https://api.github.com/users/daviwil/events{/privacy}","received_events_url":"https://api.github.com/users/daviwil/received_events","type":"User","site_admin":true},"body":"This feature will enable an opt-in subset of bundled Atom packages to be updated with `apm` outside of the Atom release cycle.  This will enable users to receive new functionality and bug fixes for some bundled packages as regularly as needed without waiting for them to be included in a new Atom release. \r\n\r\n[**Rendered view**](https://github.com/atom/atom/blob/dw-updateable-packages-rfc/docs/rfcs/001-updateable-bundled-packages.md)","created_at":"2018-05-21T13:56:56Z","updated_at":"2018-05-22T14:05:11Z","closed_at":null,"merged_at":null,"merge_commit_sha":"2ac8cb0ff872dbe5e2dbf58fee622e2faaebe056","assignee":null,"assignees":[],"requested_reviewers":[{"login":"jasonrudolph","id":2988,"avatar_url":"https://avatars3.githubusercontent.com/u/2988?v=4","gravatar_id":"","url":"https://api.github.com/users/jasonrudolph","html_url":"https://github.com/jasonrudolph","followers_url":"https://api.github.com/users/jasonrudolph/followers","following_url":"https://api.github.com/users/jasonrudolph/following{/other_user}","gists_url":"https://api.github.com/users/jasonrudolph/gists{/gist_id}","starred_url":"https://api.github.com/users/jasonrudolph/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jasonrudolph/subscriptions","organizations_url":"https://api.github.com/users/jasonrudolph/orgs","repos_url":"https://api.github.com/users/jasonrudolph/repos","events_url":"https://api.github.com/users/jasonrudolph/events{/privacy}","received_events_url":"https://api.github.com/users/jasonrudolph/received_events","type":"User","site_admin":true},{"login":"maxbrunsfeld","id":326587,"avatar_url":"https://avatars3.githubusercontent.com/u/326587?v=4","gravatar_id":"","url":"https://api.github.com/users/maxbrunsfeld","html_url":"https://github.com/maxbrunsfeld","followers_url":"https://api.github.com/users/maxbrunsfeld/followers","following_url":"https://api.github.com/users/maxbrunsfeld/following{/other_user}","gists_url":"https://api.github.com/users/maxbrunsfeld/gists{/gist_id}","starred_url":"https://api.github.com/users/maxbrunsfeld/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/maxbrunsfeld/subscriptions","organizations_url":"https://api.github.com/users/maxbrunsfeld/orgs","repos_url":"https://api.github.com/users/maxbrunsfeld/repos","events_url":"https://api.github.com/users/maxbrunsfeld/events{/privacy}","received_events_url":"https://api.github.com/users/maxbrunsfeld/received_events","type":"User","site_admin":true}],"requested_teams":[],"labels":[],"milestone":null,"commits_url":"https://api.github.com/repos/atom/atom/pulls/17379/commits","review_comments_url":"https://api.github.com/repos/atom/atom/pulls/17379/comments","review_comment_url":"https://api.github.com/repos/atom/atom/pulls/comments{/number}","comments_url":"https://api.github.com/repos/atom/atom/issues/17379/comments","statuses_url":"https://api.github.com/repos/atom/atom/statuses/391bf82a419aad390b4b6e847d47cce423ea89ba","head":{"label":"atom:dw-updateable-packages-rfc","ref":"dw-updateable-packages-rfc","sha":"391bf82a419aad390b4b6e847d47cce423ea89ba","user":{"login":"atom","id":1089146,"avatar_url":"https://avatars2.githubusercontent.com/u/1089146?v=4","gravatar_id":"","url":"https://api.github.com/users/atom","html_url":"https://github.com/atom","followers_url":"https://api.github.com/users/atom/followers","following_url":"https://api.github.com/users/atom/following{/other_user}","gists_url":"https://api.github.com/users/atom/gists{/gist_id}","starred_url":"https://api.github.com/users/atom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atom/subscriptions","organizations_url":"https://api.github.com/users/atom/orgs","repos_url":"https://api.github.com/users/atom/repos","events_url":"https://api.github.com/users/atom/events{/privacy}","received_events_url":"https://api.github.com/users/atom/received_events","type":"Organization","site_admin":false},"repo":{"id":3228505,"name":"atom","full_name":"atom/atom","owner":{"login":"atom","id":1089146,"avatar_url":"https://avatars2.githubusercontent.com/u/1089146?v=4","gravatar_id":"","url":"https://api.github.com/users/atom","html_url":"https://github.com/atom","followers_url":"https://api.github.com/users/atom/followers","following_url":"https://api.github.com/users/atom/following{/other_user}","gists_url":"https://api.github.com/users/atom/gists{/gist_id}","starred_url":"https://api.github.com/users/atom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atom/subscriptions","organizations_url":"https://api.github.com/users/atom/orgs","repos_url":"https://api.github.com/users/atom/repos","events_url":"https://api.github.com/users/atom/events{/privacy}","received_events_url":"https://api.github.com/users/atom/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/atom/atom","description":":atom: The hackable text editor","fork":false,"url":"https://api.github.com/repos/atom/atom","forks_url":"https://api.github.com/repos/atom/atom/forks","keys_url":"https://api.github.com/repos/atom/atom/keys{/key_id}","collaborators_url":"https://api.github.com/repos/atom/atom/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/atom/atom/teams","hooks_url":"https://api.github.com/repos/atom/atom/hooks","issue_events_url":"https://api.github.com/repos/atom/atom/issues/events{/number}","events_url":"https://api.github.com/repos/atom/atom/events","assignees_url":"https://api.github.com/repos/atom/atom/assignees{/user}","branches_url":"https://api.github.com/repos/atom/atom/branches{/branch}","tags_url":"https://api.github.com/repos/atom/atom/tags","blobs_url":"https://api.github.com/repos/atom/atom/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/atom/atom/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/atom/atom/git/refs{/sha}","trees_url":"https://api.github.com/repos/atom/atom/git/trees{/sha}","statuses_url":"https://api.github.com/repos/atom/atom/statuses/{sha}","languages_url":"https://api.github.com/repos/atom/atom/languages","stargazers_url":"https://api.github.com/repos/atom/atom/stargazers","contributors_url":"https://api.github.com/repos/atom/atom/contributors","subscribers_url":"https://api.github.com/repos/atom/atom/subscribers","subscription_url":"https://api.github.com/repos/atom/atom/subscription","commits_url":"https://api.github.com/repos/atom/atom/commits{/sha}","git_commits_url":"https://api.github.com/repos/atom/atom/git/commits{/sha}","comments_url":"https://api.github.com/repos/atom/atom/comments{/number}","issue_comment_url":"https://api.github.com/repos/atom/atom/issues/comments{/number}","contents_url":"https://api.github.com/repos/atom/atom/contents/{+path}","compare_url":"https://api.github.com/repos/atom/atom/compare/{base}...{head}","merges_url":"https://api.github.com/repos/atom/atom/merges","archive_url":"https://api.github.com/repos/atom/atom/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/atom/atom/downloads","issues_url":"https://api.github.com/repos/atom/atom/issues{/number}","pulls_url":"https://api.github.com/repos/atom/atom/pulls{/number}","milestones_url":"https://api.github.com/repos/atom/atom/milestones{/number}","notifications_url":"https://api.github.com/repos/atom/atom/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/atom/atom/labels{/name}","releases_url":"https://api.github.com/repos/atom/atom/releases{/id}","deployments_url":"https://api.github.com/repos/atom/atom/deployments","created_at":"2012-01-20T18:18:21Z","updated_at":"2018-05-22T13:13:16Z","pushed_at":"2018-05-22T03:05:29Z","git_url":"git://github.com/atom/atom.git","ssh_url":"git@github.com:atom/atom.git","clone_url":"https://github.com/atom/atom.git","svn_url":"https://github.com/atom/atom","homepage":"https://atom.io","size":298076,"stargazers_count":44994,"watchers_count":44994,"language":"JavaScript","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":9268,"mirror_url":null,"archived":false,"open_issues_count":856,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit"},"forks":9268,"open_issues":856,"watchers":44994,"default_branch":"master"}},"base":{"label":"atom:master","ref":"master","sha":"9ba737de29f5ec160440cce19f588529f2e6eeab","user":{"login":"atom","id":1089146,"avatar_url":"https://avatars2.githubusercontent.com/u/1089146?v=4","gravatar_id":"","url":"https://api.github.com/users/atom","html_url":"https://github.com/atom","followers_url":"https://api.github.com/users/atom/followers","following_url":"https://api.github.com/users/atom/following{/other_user}","gists_url":"https://api.github.com/users/atom/gists{/gist_id}","starred_url":"https://api.github.com/users/atom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atom/subscriptions","organizations_url":"https://api.github.com/users/atom/orgs","repos_url":"https://api.github.com/users/atom/repos","events_url":"https://api.github.com/users/atom/events{/privacy}","received_events_url":"https://api.github.com/users/atom/received_events","type":"Organization","site_admin":false},"repo":{"id":3228505,"name":"atom","full_name":"atom/atom","owner":{"login":"atom","id":1089146,"avatar_url":"https://avatars2.githubusercontent.com/u/1089146?v=4","gravatar_id":"","url":"https://api.github.com/users/atom","html_url":"https://github.com/atom","followers_url":"https://api.github.com/users/atom/followers","following_url":"https://api.github.com/users/atom/following{/other_user}","gists_url":"https://api.github.com/users/atom/gists{/gist_id}","starred_url":"https://api.github.com/users/atom/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/atom/subscriptions","organizations_url":"https://api.github.com/users/atom/orgs","repos_url":"https://api.github.com/users/atom/repos","events_url":"https://api.github.com/users/atom/events{/privacy}","received_events_url":"https://api.github.com/users/atom/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/atom/atom","description":":atom: The hackable text editor","fork":false,"url":"https://api.github.com/repos/atom/atom","forks_url":"https://api.github.com/repos/atom/atom/forks","keys_url":"https://api.github.com/repos/atom/atom/keys{/key_id}","collaborators_url":"https://api.github.com/repos/atom/atom/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/atom/atom/teams","hooks_url":"https://api.github.com/repos/atom/atom/hooks","issue_events_url":"https://api.github.com/repos/atom/atom/issues/events{/number}","events_url":"https://api.github.com/repos/atom/atom/events","assignees_url":"https://api.github.com/repos/atom/atom/assignees{/user}","branches_url":"https://api.github.com/repos/atom/atom/branches{/branch}","tags_url":"https://api.github.com/repos/atom/atom/tags","blobs_url":"https://api.github.com/repos/atom/atom/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/atom/atom/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/atom/atom/git/refs{/sha}","trees_url":"https://api.github.com/repos/atom/atom/git/trees{/sha}","statuses_url":"https://api.github.com/repos/atom/atom/statuses/{sha}","languages_url":"https://api.github.com/repos/atom/atom/languages","stargazers_url":"https://api.github.com/repos/atom/atom/stargazers","contributors_url":"https://api.github.com/repos/atom/atom/contributors","subscribers_url":"https://api.github.com/repos/atom/atom/subscribers","subscription_url":"https://api.github.com/repos/atom/atom/subscription","commits_url":"https://api.github.com/repos/atom/atom/commits{/sha}","git_commits_url":"https://api.github.com/repos/atom/atom/git/commits{/sha}","comments_url":"https://api.github.com/repos/atom/atom/comments{/number}","issue_comment_url":"https://api.github.com/repos/atom/atom/issues/comments{/number}","contents_url":"https://api.github.com/repos/atom/atom/contents/{+path}","compare_url":"https://api.github.com/repos/atom/atom/compare/{base}...{head}","merges_url":"https://api.github.com/repos/atom/atom/merges","archive_url":"https://api.github.com/repos/atom/atom/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/atom/atom/downloads","issues_url":"https://api.github.com/repos/atom/atom/issues{/number}","pulls_url":"https://api.github.com/repos/atom/atom/pulls{/number}","milestones_url":"https://api.github.com/repos/atom/atom/milestones{/number}","notifications_url":"https://api.github.com/repos/atom/atom/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/atom/atom/labels{/name}","releases_url":"https://api.github.com/repos/atom/atom/releases{/id}","deployments_url":"https://api.github.com/repos/atom/atom/deployments","created_at":"2012-01-20T18:18:21Z","updated_at":"2018-05-22T13:13:16Z","pushed_at":"2018-05-22T03:05:29Z","git_url":"git://github.com/atom/atom.git","ssh_url":"git@github.com:atom/atom.git","clone_url":"https://github.com/atom/atom.git","svn_url":"https://github.com/atom/atom","homepage":"https://atom.io","size":298076,"stargazers_count":44994,"watchers_count":44994,"language":"JavaScript","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":9268,"mirror_url":null,"archived":false,"open_issues_count":856,"license":{"key":"mit","name":"MIT License","spdx_id":"MIT","url":"https://api.github.com/licenses/mit"},"forks":9268,"open_issues":856,"watchers":44994,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/atom/atom/pulls/17379"},"html":{"href":"https://github.com/atom/atom/pull/17379"},"issue":{"href":"https://api.github.com/repos/atom/atom/issues/17379"},"comments":{"href":"https://api.github.com/repos/atom/atom/issues/17379/comments"},"review_comments":{"href":"https://api.github.com/repos/atom/atom/pulls/17379/comments"},"review_comment":{"href":"https://api.github.com/repos/atom/atom/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/atom/atom/pulls/17379/commits"},"statuses":{"href":"https://api.github.com/repos/atom/atom/statuses/391bf82a419aad390b4b6e847d47cce423ea89ba"}},"author_association":"MEMBER"}}
{"actor":{"display_login":"jasonrudolph"}}