[austin-rt]

Issue(s)

Closes #160
Related to #19, #21, #140, #141

Acceptance Criteria fulfillment

• A logged-in user sees a private-repo pull request in the PR view, from the pulls command and from the View Changes button in search results
• View Changes and View File show a private repository's diff and files
• A revoked or expired token no longer breaks the PR view on public repositories
• A logged-out user who opens a private-repo pull request is told to log in

Proposed changes (including videos or screenshots)

• modals/pullDetailsModal.ts takes the user's token and sends it on both requests, the way lib/pullReqeustListMessage.ts and lib/repoDataMessage.ts already do. If GitHub answers 401, it retries without the token, so a revoked or expired token does not break public repositories. When the request had no token, the "doesn't exist" message also says to log in.
• modals/fileCodeModal.ts fetches diffs and files through the GitHub API with the token. The button value is a github.com link that comes back from the client, so the token only goes to an API URL rebuilt from that link's owner, repo, number or commit and path, never to the link itself. Any other link is fetched without the token, as before.
• lib/commandUtility.ts and handlers/ExecuteBlockActionHandler.ts pass the logged-in user's token.

Further comments

• Tested by hand on Rocket.Chat 8.8 in Docker against a private repository. The pulls command and the View Changes button both opened the PR view with the title, author, state and files, and View Changes and View File showed the diff and the file content.
• The third caller, handlers/ExecuteViewClosedHandler.ts, opens the view without data and is left as it is.
• The API host stays api.github.com, like the rest of these files. Reading it from the BaseApiHostID setting added in [Feat]: Add new GithubSDK class #144 belongs with GitHub Enterprise Server Support #111 and feat: Migrate GithubSDK To class Based. #143.
• Touches modals/pullDetailsModal.ts and handlers/ExecuteBlockActionHandler.ts like [FEAT]: Settings to hide individual actions in the pull request view #166, so whichever merges second needs a small rebase.

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}