I like the flow of this, I like having the whole review grouped on the Description page and being able to navigate to that from any of the draft badges.

I think it would be nice to be able to start and end a review in the same place. So in place of the Send button after a review has been started, I think there should be a Submit Review button the comment. Likewise, should there be a button to start a review on the Description page?

I would opt to not have the n comments in review text. Though it provides some reminder of where you are in the review, I think it doesn't add too much value weighed against the cost of making an API change to support it - it doesn't seem like a typical contribution point to the UI.

I have a couple different thoughts here, one is on the proposal above and one is on how it merges with the current review mechanism.

1. In terms of making clear the distinction between single comment and review, should we just try to match closely to the GH model? Their comment box looks like this below. I think the "Add single comment" button is clearer in intent than saying Send.
   

GitHub uses the language "Pending" instead of draft, I like either but unless we have a strong reason to be different might as well match.

The other thing I noticed on GitHub is that they put the submit review button on every file.

This might be a convenient thing so you don't need to bounce around to the description page but I think what @queerviolet proposes is simpler ,especially since we will also need to support things like approval status and that would be easier in the description page:

2. Today the PR extension has a review mode where after you click checkout you are now in "Review Mode"
   

When I first used the extension, I thought that this was the same thing as the GitHub review, but this is different. An option could be to make this the actual bulk review mechanism, but that would mean we require you to checkout the branch to do a review, and I am not sure if everyone would want that. But my concern is the confusion over the two concepts.

@RMacfarlane — Concur, we don't need the n comments in review text. Mostly, I had it there to make it clear that there's a review ongoing, but I think we have enough other affordances that it's unnecessary

@mmanela — I actually don't think it would be too bad to require the branch be checked out, since then we could easily make multi-file suggested changes, something we've been discussing. Also, agree that we should just have a Finish review button.

What contribution points do we need to add to the VS Code API to enable this? Any chance of adding those for the early Dec release?

While playing around with microsoft/vscode#64010, I found that GitHub v3 API is limited in Review comments management, with which we can't archive the same experience as github.com. Some blockers/limitations I have noticed

• We can create a review with draft comments, but once we created the pending review, there is no way to add more comments to it
• We don't know if there is a pending review or not. GitHub for Visual Studio fetches all reviews and check if the latest one is pending, but it would be great if we can have an explicit api.
• To fetch all comments visible to myself, we at least need to make two API calls
  1. Fetch all comments through /repo/**/pulls/**/comments
  2. Fetch all reviews, check if the first one is pending or not, and then fetch all comments in this pending review, as they are not shown in the first step, even if I already logged in.

For now to test the API, I wrote a sample extension https://github.com/rebornix/comments-provider-sample .

Any timeline for this feature? We are using VSCode extensively for TensorFlow.js and our reviews can sometimes have 20-30 comments. We are currently using Reviewable, but if we had a review mode with draft comments in VSCode I can see us completely switching to VSCode for reviews.

Any updates coming for the new year ?

This is on our plan for January

Completed by 6d1f0de and cd7f73a