The issue has been closed
Any feature that makes use of pathnameParts[4] or getReference don't support branch names that contain /.
The problem with extracting the branch from the URL is that branch/folder/folder/folder is indistinguishable from branch/still/branch/folder, so the branch cannot be safely determined from the URL.
fix-view-file-link-in-pr and link-to-file-in-file-history both have solutions for it, but I think they're not easily copy-able.
For example getReference could be changed to:
// Tested on the Code tab
return select('#branch-select-menu summary .css-truncate-target').textContent
Originally discussed in https://github.com/sindresorhus/refined-github/pull/2998#issuecomment-613125320
cc @yakov116
It fails anywhere a branch can appear in the URL. deep-reblame is also affected: https://github.com/sindresorhus/refined-github/blame/broken/reblame/demo/test/utils.ts
deep-reblame will need to avoid using getReference altogether since we only use it to get the filename and generate a URL.
- the filename is in the breadcrumbs
- the URL can be generated from scratch: user/repo/blame/prBlameCommit/path
I think the best way forward would be to have getReference always use the DOM and error if the DOM isn't found. This is better than giving false promises that the received reference is correct.
Can use the last part of RSS link tag to extract branch name, similar to how we use link tags to get currently logged in user (if that is what we are still doing).
<link href="https://github.com/microsoft/PowerToys/commits/dev/PowerLauncher.atom"
rel="alternate" title="Recent Commits to PowerToys:dev/PowerLauncher"
type="application/atom+xml">
The last part after /commits/ and before .atom in href above is always the branch name, same goes for /commits/master.atom.
Example: https://github.com/microsoft/PowerToys/tree/dev/PowerLauncher
posted by fregante almost 5 years ago 
posted by yakov116 almost 5 years ago Can you also add every route where getReference is expected to work? e.g.
/** Should work on isCommitList, isSingleFile, blah blah */
function getCurrentBranch() {……..}
Can you also add every route where getReference is expected to work? e.g.
/** Should work on isCommitList, isSingleFile, blah blah */
function getCurrentBranch() {……..}
Sure.
I dont see a place it does not work. (Even if your on the settings page it will give you the current branch)
posted by fregante almost 5 years ago @fregante we are doing a select.last. That will give you .atom
IssueHunt