Make repo compare in activation case insensitive (#8223)

Fixes #8221

Author: alexr00

src/common/utils.ts

@@ -102,9 +102,9 @@ function isWindowsPath(path: string): boolean {
 	return /^[a-zA-Z]:\\/.test(path);
 }
 
-export function isDescendant(parent: string, descendant: string, separator: string = sep): boolean {
+export function isDescendant(parent: string, descendant: string, caseInsensitive: boolean = false, separator: string = sep): boolean {
 	// Windows is case insensitive
-	if (isWindowsPath(parent)) {
+	if (isWindowsPath(parent) || caseInsensitive) {
 		parent = parent.toLowerCase();
 		descendant = descendant.toLowerCase();
 	}

src/extension.ts

@@ -232,7 +232,9 @@ async function init(
 		}
 
 		// Check if repo is in one of the workspace folders or vice versa
-		if (workspaceFolders && !workspaceFolders.some(folder => isDescendant(folder.uri.fsPath, repo.rootUri.fsPath) || isDescendant(repo.rootUri.fsPath, folder.uri.fsPath))) {
+		Logger.debug(`Checking if repo ${repo.rootUri.fsPath} is in a workspace folder.`, ACTIVATION);
+		Logger.debug(`Workspace folders: ${workspaceFolders?.map(folder => folder.uri.fsPath).join(', ')}`, ACTIVATION);
+		if (workspaceFolders && !workspaceFolders.some(folder => isDescendant(folder.uri.fsPath, repo.rootUri.fsPath, true) || isDescendant(repo.rootUri.fsPath, folder.uri.fsPath, true))) {
 			Logger.appendLine(`Repo ${repo.rootUri} is not in a workspace folder, ignoring.`, ACTIVATION);
 			return;
 		}

src/github/pullRequestModel.ts

@@ -2160,7 +2160,7 @@ export class PullRequestModel extends IssueModel<PullRequest> implements IPullRe
 	async markFiles(filePathOrSubpaths: string[], event: boolean, state: 'viewed' | 'unviewed'): Promise<void> {
 		const allFilenames = filePathOrSubpaths
 			.map((f) =>
-				isDescendant(this.githubRepository.rootUri.path, f, '/')
+				isDescendant(this.githubRepository.rootUri.path, f, false, '/')
 					? f.substring(this.githubRepository.rootUri.path.length + 1)
 					: f
 			);