Performance: Reduce waterfall fetching by making repo and resolvedRev optional (!40244) · Merge requests · Warren Gifford / sourcegraph

Created by: umpox

Actions before merging this PR:

Fix TS errors where repo or resolvedRev are now undefined
Remove any more code that blocks rendering if repo or resolvedRev are undefined
Use repoName and revision from URL (parseBrowserRepoURL) in components instead of waiting for these queries.
- E.g., here we do this for the fetchBlob query.
- Check that this is OK, with code intel. That there are no issues with doing this. The only one I can think is if our backend expects a specific commit ID or something, and we don't have that because the URL only contains a branch, or nothing.
  - Slack thread

Makes repo and resolvedRev optional.

Components should try to render as much as they can to improve perceived perf here
Queries such as fetchBlob should use the URL as a source of truth rather than the output of this query for improved perf.