Support partially URL-encoded paths

Author: SunsetTechuila

index.test.ts

@@ -234,7 +234,36 @@ test('getRepositoryInfo', () => {
 			  ],
 			}
 		`);
+		expect(getRepositoryInfoAdapter('https://github.com/refined-github/sandbox/blob/bracket-in-path/foo%2F%5Bbar%5D%2Fbaz')).toMatchInlineSnapshot(`
+			{
+			  "name": "sandbox",
+			  "nameWithOwner": "refined-github/sandbox",
+			  "owner": "refined-github",
+			  "path": "blob/bracket-in-path/foo/[bar]/baz",
+			  "pathParts": [
+			    "blob",
+			    "bracket-in-path",
+			    "foo",
+			    "[bar]",
+			    "baz",
+			  ],
+			}
+		`);
+		expect(getRepositoryInfoAdapter('https://github.com/refined-github/sandbox/blob/bracket-in-path/foo%252F%5Bbar%5D%252Fbaz')).toMatchInlineSnapshot(`
+			{
+			  "name": "sandbox",
+			  "nameWithOwner": "refined-github/sandbox",
+			  "owner": "refined-github",
+			  "path": "blob/bracket-in-path/foo%2F[bar]%2Fbaz",
+			  "pathParts": [
+			    "blob",
+			    "bracket-in-path",
+			    "foo%2F[bar]%2Fbaz",
+			  ],
+			}
+		`);
 	}
+	// foo%252F%5Bbar%5D%252Fbaz
 });
 
 test('parseRepoExplorerTitle', () => {

index.ts

@@ -55,13 +55,13 @@ TEST: addTests('isCompare', [
 	'isQuickPR',
 ]);
 
-export const isCompareWikiPage = (url: URL | HTMLAnchorElement | Location = location): boolean => isRepoWiki(url) && getCleanPathname(url).split('/').slice(3, 5).includes('_compare');
+export const isCompareWikiPage = (url: URL | HTMLAnchorElement | Location = location): boolean => isRepoWiki(url) && processPathname(url).split('/').slice(3, 5).includes('_compare');
 TEST: addTests('isCompareWikiPage', [
 	'https://github.com/brookhong/Surfingkeys/wiki/_compare/8ebb46b1a12d16fc1af442b7df0ca13ca3bb34dc...80e51eeabe69b15a3f23880ecc36f800b71e6c6d',
 	'https://github.com/brookhong/Surfingkeys/wiki/Color-Themes/_compare/8ebb46b1a12d16fc1af442b7df0ca13ca3bb34dc...80e51eeabe69b15a3f23880ecc36f800b71e6c6d',
 ]);
 
-export const isDashboard = (url: URL | HTMLAnchorElement | Location = location): boolean => !isGist(url) && /^$|^(orgs\/[^/]+\/)?dashboard(-feed)?(\/|$)/.test(getCleanPathname(url));
+export const isDashboard = (url: URL | HTMLAnchorElement | Location = location): boolean => !isGist(url) && /^$|^(orgs\/[^/]+\/)?dashboard(-feed)?(\/|$)/.test(processPathname(url));
 TEST: addTests('isDashboard', [
 	'https://github.com///',
 	'https://github.com//',
@@ -174,12 +174,12 @@ TEST: addTests('isNewRelease', [
 	'https://github.com/sindresorhus/refined-github/releases/new',
 ]);
 
-export const isNewWikiPage = (url: URL | HTMLAnchorElement | Location = location): boolean => isRepoWiki(url) && getCleanPathname(url).endsWith('/_new');
+export const isNewWikiPage = (url: URL | HTMLAnchorElement | Location = location): boolean => isRepoWiki(url) && processPathname(url).endsWith('/_new');
 TEST: addTests('isNewWikiPage', [
 	'https://github.com/tooomm/wikitest/wiki/_new',
 ]);
 
-export const isNotifications = (url: URL | HTMLAnchorElement | Location = location): boolean => getCleanPathname(url) === 'notifications';
+export const isNotifications = (url: URL | HTMLAnchorElement | Location = location): boolean => processPathname(url) === 'notifications';
 TEST: addTests('isNotifications', [
 	'https://github.com/notifications',
 ]);
@@ -199,7 +199,7 @@ TEST: addTests('isTeamDiscussion', [
 	'https://github.com/orgs/refined-github/teams/core-team',
 ]);
 
-export const isOwnUserProfile = (): boolean => getCleanPathname() === getLoggedInUser();
+export const isOwnUserProfile = (): boolean => processPathname() === getLoggedInUser();
 
 // If there's a Report Abuse link, we're not part of the org
 export const isOwnOrganizationProfile = (): boolean => isOrganizationProfile() && !exists('[href*="contact/report-abuse?report="]');
@@ -365,7 +365,7 @@ TEST: addTests('isEditingRelease', [
 export const hasReleaseEditor = (url: URL | HTMLAnchorElement | Location = location): boolean => isEditingRelease(url) || isNewRelease(url);
 TEST: addTests('hasReleaseEditor', combinedTestOnly);
 
-export const isEditingWikiPage = (url: URL | HTMLAnchorElement | Location = location): boolean => isRepoWiki(url) && getCleanPathname(url).endsWith('/_edit');
+export const isEditingWikiPage = (url: URL | HTMLAnchorElement | Location = location): boolean => isRepoWiki(url) && processPathname(url).endsWith('/_edit');
 TEST: addTests('isEditingWikiPage', [
 	'https://github.com/tooomm/wikitest/wiki/Getting-Started/_edit',
 ]);
@@ -374,7 +374,7 @@ export const hasWikiPageEditor = (url: URL | HTMLAnchorElement | Location = loca
 TEST: addTests('hasWikiPageEditor', combinedTestOnly);
 
 export const isRepo = (url: URL | HTMLAnchorElement | Location = location): boolean => {
-	const [user, repo, extra] = getCleanPathname(url).split('/');
+	const [user, repo, extra] = processPathname(url).split('/');
 	return Boolean(user
 		&& repo
 		&& !reservedNames.includes(user)
@@ -704,7 +704,7 @@ const doesLookLikeAProfile = (string: string | undefined): boolean =>
 
 export const isProfile = (url: URL | HTMLAnchorElement | Location = location): boolean =>
 	!isGist(url)
-	&& doesLookLikeAProfile(getCleanPathname(url));
+	&& doesLookLikeAProfile(processPathname(url));
 
 TEST: addTests('isProfile', [
 	'https://github.com/fregante',
@@ -822,7 +822,7 @@ TEST: addTests('isRepoGitObject', [
 /** Covers blob, trees and blame pages */
 export const isRepoGitObject = (url: URL | HTMLAnchorElement | Location = location): boolean =>
 	isRepo(url)
-	&& [undefined, 'blob', 'tree', 'blame'].includes(getCleanPathname(url).split('/')[2]);
+	&& [undefined, 'blob', 'tree', 'blame'].includes(processPathname(url).split('/')[2]);
 
 TEST: addTests('hasFiles', combinedTestOnly);
 /** Has a list of files */
@@ -874,7 +874,7 @@ export const canUserAdminRepo = (): boolean => {
 	].join(',')}) a[href="/${repo.nameWithOwner}/settings"]`));
 };
 
-export const isNewRepo = (url: URL | HTMLAnchorElement | Location = location): boolean => !isGist(url) && (url.pathname === '/new' || /^organizations\/[^/]+\/repositories\/new$/.test(getCleanPathname(url)));
+export const isNewRepo = (url: URL | HTMLAnchorElement | Location = location): boolean => !isGist(url) && (url.pathname === '/new' || /^organizations\/[^/]+\/repositories\/new$/.test(processPathname(url)));
 TEST: addTests('isNewRepo', [
 	'https://github.com/new',
 	'https://github.com/organizations/npmhub/repositories/new',
@@ -889,11 +889,19 @@ TEST: addTests('isNewRepoTemplate', [
 /** Get the logged-in user’s username */
 const getLoggedInUser = (): string | undefined => $('meta[name="user-login"]')?.getAttribute('content') ?? undefined;
 
-/** Drop all redundant slashes */
-const getCleanPathname = (url: URL | HTMLAnchorElement | Location = location): string => url.pathname.replaceAll(/\/\/+/g, '/').replace(/\/$/, '').slice(1);
+/** Decode it and drop all redundant slashes */
+const processPathname = (url: URL | HTMLAnchorElement | Location = location): string =>
+	url.pathname
+	  .split("/")
+	  .map((part) => decodeURIComponent(part))
+	  .join('/')
+	  .replaceAll(/\/\/+/g, "/")
+	  .replace(/\/$/, "")
+	  .slice(1);
+
 
 const getCleanGistPathname = (url: URL | HTMLAnchorElement | Location = location): string | undefined => {
-	const pathname = getCleanPathname(url);
+	const pathname = processPathname(url);
 	if (url.hostname.startsWith('gist.')) {
 		return pathname;
 	}
@@ -903,7 +911,7 @@ const getCleanGistPathname = (url: URL | HTMLAnchorElement | Location = location
 };
 
 const getOrg = (url: URL | HTMLAnchorElement | Location = location): {name: string; path: string} | undefined => {
-	const [orgs, name, ...path] = getCleanPathname(url).split('/');
+	const [orgs, name, ...path] = processPathname(url).split('/');
 	if (orgs === 'orgs' && name) {
 		return {name, path: path.join('/')};
 	}
@@ -949,7 +957,7 @@ const getRepo = (url?: URL | HTMLAnchorElement | Location | string): RepositoryI
 		if (canonical) {
 			const canonicalUrl = new URL(canonical.content, location.origin);
 			// Sometimes GitHub sets the canonical to an incomplete URL, so it can't be used
-			if (getCleanPathname(canonicalUrl).toLowerCase() === getCleanPathname(location).toLowerCase()) {
+			if (processPathname(canonicalUrl).toLowerCase() === processPathname(location).toLowerCase()) {
 				url = canonicalUrl;
 			}
 		}
@@ -963,7 +971,7 @@ const getRepo = (url?: URL | HTMLAnchorElement | Location | string): RepositoryI
 		return;
 	}
 
-	const [owner, name, ...pathParts] = getCleanPathname(url).split('/') as [string, string, string];
+	const [owner, name, ...pathParts] = processPathname(decodePathname(url)).split('/') as [string, string, string];
 	return {
 		owner,
 		name,
@@ -976,7 +984,7 @@ const getRepo = (url?: URL | HTMLAnchorElement | Location | string): RepositoryI
 export const utils = {
 	getOrg,
 	getLoggedInUser,
-	getCleanPathname,
+	processPathname,
 	getCleanGistPathname,
 	getRepositoryInfo: getRepo,
 	parseRepoExplorerTitle,