Exclude special search URLs from isRepoRoot and isRepoHome

[kang8]

Fixes #173

[fregante]

This issue is similar in complexity to #131 because /user/repo?search=1 is technically a repo home but doesn't have the DOM we expect it to have.

Since we don't actually need a standalone isRepoTreeFileFinder, we can avoid exporting it and rename it to hasSearchParameter.

I think we need:

• _isRepoRoot (not exported), which runs the necessary checks for the repo root, except hasSearchParameter
• isRepoRoot = () => !hasSearchParameter() && _isRepoRoot()
• isRepoTree = () => _isRepoRoot() || Boolean(getRepo(url)?.path.startsWith('tree/'))

This is because currently isRepoTree is first excluding isRepoTreeFileFinder inside isRepoRoot and then adding it again later. With a private _isRepoRoot we don't need this awkward dance.

[kang8]

It's very strange that none of the commits after the first one triggered a GitHub Action.

e.g.

• e901a63
• c198d35
• ...

[fregante]

Thanks @kang8!

[fregante]

v8 has been released. Would you like to send a PR to RG to update it? There are a few changes to make, considering all the updates included:

https://github.com/refined-github/github-url-detection/releases/tag/v8.0.0

[kang8]

Would you like to send a PR to RG to update it?

Yes, I will submit a PR after work.