Require component name in error logs (#6598)

alexr00 · web-flow · commit 265a18b51281 · 2025-01-16T15:34:00.000+01:00
Part of #6513
diff --git a/src/commands.ts b/src/commands.ts
@@ -149,6 +149,7 @@ export function registerCommands(
 	telemetry: ITelemetry,
 	tree: PullRequestsTreeDataProvider,
 ) {
+	const logId = 'RegisterCommands';
 	context.subscriptions.push(
 		vscode.commands.registerCommand(
 			'pr.openPullRequestOnGitHub',
@@ -290,7 +291,7 @@ export function registerCommands(
 				await vscode.workspace.fs.delete(tempUri);
 			} catch (err) {
 				const moreError = `${err}${err.stderr ? `\n${err.stderr}` : ''}`;
-				Logger.error(`Applying patch failed: ${moreError}`);
+				Logger.error(`Applying patch failed: ${moreError}`, logId);
 				vscode.window.showErrorMessage(vscode.l10n.t('Applying patch failed: {0}', formatError(err)));
 			}
 		}),
@@ -511,7 +512,7 @@ export function registerCommands(
 		vscode.commands.registerCommand('pr.pick', async (pr: PRNode | DescriptionNode | PullRequestModel) => {
 			if (pr === undefined) {
 				// This is unexpected, but has happened a few times.
-				Logger.error('Unexpectedly received undefined when picking a PR.');
+				Logger.error('Unexpectedly received undefined when picking a PR.', logId);
 				return vscode.window.showErrorMessage(vscode.l10n.t('No pull request was selected to checkout, please try again.'));
 			}
 
@@ -552,7 +553,7 @@ export function registerCommands(
 		vscode.commands.registerCommand('pr.openChanges', async (pr: PRNode | DescriptionNode | PullRequestModel) => {
 			if (pr === undefined) {
 				// This is unexpected, but has happened a few times.
-				Logger.error('Unexpectedly received undefined when picking a PR.');
+				Logger.error('Unexpectedly received undefined when picking a PR.', logId);
 				return vscode.window.showErrorMessage(vscode.l10n.t('No pull request was selected to checkout, please try again.'));
 			}
 
@@ -606,7 +607,7 @@ export function registerCommands(
 		vscode.commands.registerCommand('pr.pickOnVscodeDev', async (pr: PRNode | DescriptionNode | PullRequestModel) => {
 			if (pr === undefined) {
 				// This is unexpected, but has happened a few times.
-				Logger.error('Unexpectedly received undefined when picking a PR.');
+				Logger.error('Unexpectedly received undefined when picking a PR.', logId);
 				return vscode.window.showErrorMessage(vscode.l10n.t('No pull request was selected to checkout, please try again.'));
 			}
 
@@ -1090,7 +1091,7 @@ export function registerCommands(
 			const commentEditor = vscode.window.activeTextEditor?.document.uri.scheme === Schemes.Comment ? vscode.window.activeTextEditor
 				: vscode.window.visibleTextEditors.find(visible => (visible.document.uri.scheme === Schemes.Comment) && (visible.document.uri.query === ''));
 			if (!commentEditor) {
-				Logger.error('No comment editor visible for making a suggestion.');
+				Logger.error('No comment editor visible for making a suggestion.', logId);
 				vscode.window.showErrorMessage(vscode.l10n.t('No available comment editor to make a suggestion in.'));
 				return;
 			}
diff --git a/src/common/logger.ts b/src/common/logger.ts
@@ -57,7 +57,7 @@ class Log extends Disposable {
 		this._outputChannel.warn(this.logString(message, component));
 	}
 
-	public error(message: any, component?: string) {
+	public error(message: any, component: string) {
 		this._outputChannel.error(this.logString(message, component));
 	}
 }
diff --git a/src/common/protocol.ts b/src/common/protocol.ts
@@ -17,6 +17,7 @@ export enum ProtocolType {
 }
 
 export class Protocol {
+	private static readonly ID = 'Protocol';
 	public type: ProtocolType = ProtocolType.OTHER;
 	public host: string = '';
 
@@ -44,7 +45,7 @@ export class Protocol {
 				this.owner = this.getOwnerName(this.url.path) || '';
 			}
 		} catch (e) {
-			Logger.error(`Failed to parse '${uriString}'`);
+			Logger.error(`Failed to parse '${uriString}'`, Protocol.ID);
 			vscode.window.showWarningMessage(
 				vscode.l10n.t('Unable to parse remote \'{0}\'. Please check that it is correctly formatted.', uriString)
 			);
diff --git a/src/common/uri.ts b/src/common/uri.ts
@@ -283,7 +283,7 @@ export namespace DataUri {
 				try {
 					await vscode.workspace.fs.delete(vscode.Uri.joinPath(cacheLocation(context), id));
 				} catch (e) {
-					Logger.error(`Failed to delete avatar from cache: ${e}`);
+					Logger.error(`Failed to delete avatar from cache: ${e}`, 'avatarCirclesAsImageDataUris');
 				}
 			}));
 		}
diff --git a/src/github/createPRViewProvider.ts b/src/github/createPRViewProvider.ts
@@ -280,7 +280,7 @@ export abstract class BaseCreatePullRequestViewProvider<T extends BasePullReques
 		try {
 			await pr.addAssignees([resolved]);
 		} catch (e) {
-			Logger.error(`Unable to assign pull request to user ${resolved}.`);
+			Logger.error(`Unable to assign pull request to user ${resolved}.`, BaseCreatePullRequestViewProvider.ID);
 		}
 	}
 
@@ -379,7 +379,7 @@ export abstract class BaseCreatePullRequestViewProvider<T extends BasePullReques
 				});
 			}
 		} catch (e) {
-			Logger.error(formatError(e));
+			Logger.error(`Failed to add reviewers: ${formatError(e)}`, BaseCreatePullRequestViewProvider.ID);
 			vscode.window.showErrorMessage(formatError(e));
 		} finally {
 			quickPick?.hide();
diff --git a/src/github/credentials.ts b/src/github/credentials.ts
@@ -47,6 +47,7 @@ interface AuthResult {
 }
 
 export class CredentialStore extends Disposable {
+	private static readonly ID = 'Authentication';
 	private _githubAPI: GitHub | undefined;
 	private _sessionId: string | undefined;
 	private _githubEnterpriseAPI: GitHub | undefined;
@@ -204,7 +205,7 @@ export class CredentialStore extends Disposable {
 			}
 			return authResult;
 		} else {
-			Logger.debug(`No GitHub${getGitHubSuffix(authProviderId)} token found.`, 'Authentication');
+			Logger.debug(`No GitHub${getGitHubSuffix(authProviderId)} token found.`, CredentialStore.ID);
 			return authResult;
 		}
 	}
@@ -328,9 +329,9 @@ export class CredentialStore extends Disposable {
 			try {
 				await this.initialize(authProviderId, sessionOptions);
 			} catch (e) {
-				Logger.error(`${errorPrefix}: ${e}`);
+				Logger.error(`Login error: ${errorPrefix}: ${e}`, CredentialStore.ID);
 				if (e instanceof Error && e.stack) {
-					Logger.error(e.stack);
+					Logger.error(e.stack, CredentialStore.ID);
 				}
 				if (e.message === 'Cancelled') {
 					isCanceled = true;
diff --git a/src/github/folderRepositoryManager.ts b/src/github/folderRepositoryManager.ts
@@ -276,7 +276,7 @@ export class FolderRepositoryManager extends Disposable {
 		const serverTypes = await Promise.all(
 			potentialRemotes.map(remote => this._githubManager.isGitHub(remote.gitProtocol.normalizeUri()!)),
 		).catch(e => {
-			Logger.error(`Resolving GitHub remotes failed: ${e}`);
+			Logger.error(`Resolving GitHub remotes failed: ${e}`, this.id);
 			vscode.window.showErrorMessage(vscode.l10n.t('Resolving GitHub remotes failed: {0}', formatError(e)));
 			return [];
 		});
@@ -297,7 +297,7 @@ export class FolderRepositoryManager extends Disposable {
 		const serverTypes = await Promise.all(
 			potentialRemotes.map(remote => this._githubManager.isGitHub(remote.gitProtocol.normalizeUri()!)),
 		).catch(e => {
-			Logger.error(`Resolving GitHub remotes failed: ${e}`);
+			Logger.error(`Resolving GitHub remotes failed: ${e}`, this.id);
 			vscode.window.showErrorMessage(vscode.l10n.t('Resolving GitHub remotes failed: {0}', formatError(e)));
 			return [];
 		});
@@ -316,7 +316,7 @@ export class FolderRepositoryManager extends Disposable {
 		const remotesSetting = vscode.workspace.getConfiguration(PR_SETTINGS_NAMESPACE).get<string[]>(REMOTES);
 
 		if (!remotesSetting) {
-			Logger.error(`Unable to read remotes setting`);
+			Logger.error(`Unable to read remotes setting`, this.id);
 			return Promise.resolve([]);
 		}
 
@@ -621,7 +621,7 @@ export class FolderRepositoryManager extends Disposable {
 	}
 
 	private async getCachedFromGlobalState<T>(userKind: 'assignableUsers' | 'teamReviewers' | 'mentionableUsers' | 'orgProjects'): Promise<{ [key: string]: T[] } | undefined> {
-		Logger.appendLine(`Trying to use globalState for ${userKind}.`);
+		Logger.appendLine(`Trying to use globalState for ${userKind}.`, this.id);
 
 		const usersCacheLocation = vscode.Uri.joinPath(this.context.globalStorageUri, userKind);
 		let usersCacheExists;
@@ -631,7 +631,7 @@ export class FolderRepositoryManager extends Disposable {
 			// file doesn't exit
 		}
 		if (!usersCacheExists) {
-			Logger.appendLine(`GlobalState does not exist for ${userKind}.`);
+			Logger.appendLine(`GlobalState does not exist for ${userKind}.`, this.id);
 			return undefined;
 		}
 
@@ -646,7 +646,7 @@ export class FolderRepositoryManager extends Disposable {
 				cacheAsJson = JSON.parse(repoSpecificCache.toString());
 			} catch (e) {
 				if (e instanceof Error && e.message.includes('Unexpected non-whitespace character after JSON')) {
-					Logger.error(`Error parsing ${userKind} cache for ${repo.remote.remoteName}.`);
+					Logger.error(`Error parsing ${userKind} cache for ${repo.remote.remoteName}.`, this.id);
 				}
 				// file doesn't exist
 			}
@@ -656,11 +656,11 @@ export class FolderRepositoryManager extends Disposable {
 			}
 		}))).every(value => value);
 		if (hasAllRepos) {
-			Logger.appendLine(`Using globalState ${userKind} for ${Object.keys(cache).length}.`);
+			Logger.appendLine(`Using globalState ${userKind} for ${Object.keys(cache).length}.`, this.id);
 			return cache;
 		}
 
-		Logger.appendLine(`No globalState for ${userKind}.`);
+		Logger.appendLine(`No globalState for ${userKind}.`, this.id);
 		return undefined;
 	}
 
@@ -861,7 +861,7 @@ export class FolderRepositoryManager extends Disposable {
 		const serverTypes = await Promise.all(
 			remotes.map(remote => this._githubManager.isGitHub(remote.gitProtocol.normalizeUri()!)),
 		).catch(e => {
-			Logger.error(`Resolving GitHub remotes failed: ${e}`);
+			Logger.error(`Resolving GitHub remotes failed: ${e}`, this.id);
 			vscode.window.showErrorMessage(vscode.l10n.t('Resolving GitHub remotes failed: {0}', formatError(e)));
 			return [];
 		});
@@ -1458,7 +1458,7 @@ export class FolderRepositoryManager extends Disposable {
 					return this.createAndAddGitHubRepository(remote, this._credentialStore);
 				}
 
-				Logger.error(`The remote '${upstreamRef.remote}' is not a GitHub repository.`);
+				Logger.error(`The remote '${upstreamRef.remote}' is not a GitHub repository.`, this.id);
 
 				// No GitHubRepository? We currently won't try pushing elsewhere,
 				// so fail.
@@ -2461,7 +2461,7 @@ export class FolderRepositoryManager extends Disposable {
 					return;
 				}
 			}
-			Logger.error(`Exiting failed: ${e}. Target branch ${branch} used to find branch ${branchObj?.name ?? 'unknown'} with upstream ${branchObj?.upstream?.name ?? 'unknown'}.`);
+			Logger.error(`Exiting failed: ${e}. Target branch ${branch} used to find branch ${branchObj?.name ?? 'unknown'} with upstream ${branchObj?.upstream?.name ?? 'unknown'}.`, this.id);
 			vscode.window.showErrorMessage(`Exiting failed: ${e}`);
 		}
 	}
diff --git a/src/github/pullRequestOverview.ts b/src/github/pullRequestOverview.ts
@@ -465,7 +465,7 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode
 				});
 			}
 		} catch (e) {
-			Logger.error(formatError(e));
+			Logger.error(formatError(e), PullRequestOverviewPanel.ID);
 			vscode.window.showErrorMessage(formatError(e));
 		} finally {
 			quickPick?.hide();
diff --git a/src/issues/issueFeatureRegistrar.ts b/src/issues/issueFeatureRegistrar.ts
@@ -69,6 +69,7 @@ import {
 const CREATING_ISSUE_FROM_FILE_CONTEXT = 'issues.creatingFromFile';
 
 export class IssueFeatureRegistrar extends Disposable {
+	private static readonly ID = 'IssueFeatureRegistrar';
 	private _stateManager: StateManager;
 	private _newIssueCache: NewIssueCache;
 
@@ -1260,7 +1261,7 @@ ${options?.body ?? ''}\n
 			}
 			if (!folderManager) {
 				vscode.window.showErrorMessage(vscode.l10n.t(`Could not find the correct repository for the issue; see logs for more details.`));
-				Logger.error(`Could not find the folder manager for the issue originUri: ${originUri.toString()}`);
+				Logger.error(`Could not find the folder manager for the issue originUri: ${originUri.toString()}`, IssueFeatureRegistrar.ID);
 				return false;
 			}
 		}
diff --git a/src/view/compareChangesTreeDataProvider.ts b/src/view/compareChangesTreeDataProvider.ts
@@ -114,6 +114,7 @@ class GitCommitNode extends TreeNode {
 }
 
 abstract class CompareChangesTreeProvider extends Disposable implements vscode.TreeDataProvider<TreeNode>, BaseTreeNode {
+	private static readonly ID = 'CompareChangesTreeProvider';
 	private _view: vscode.TreeView<TreeNode>;
 	private _children: TreeNode[] | undefined;
 	private _onDidChangeTreeData = new vscode.EventEmitter<TreeNode | void>();
@@ -188,7 +189,7 @@ abstract class CompareChangesTreeProvider extends Disposable implements vscode.T
 				this._children = await this.getGitChildren(element);
 			}
 		} catch (e) {
-			Logger.error(`Comparing changes failed: ${e}`);
+			Logger.error(`Comparing changes failed: ${e}`, CompareChangesTreeProvider.ID);
 			return [];
 		}
 		return this._children;
diff --git a/src/view/fileChangeModel.ts b/src/view/fileChangeModel.ts
@@ -13,6 +13,7 @@ import { FolderRepositoryManager } from '../github/folderRepositoryManager';
 import { IResolvedPullRequestModel, PullRequestModel } from '../github/pullRequestModel';
 
 export abstract class FileChangeModel {
+	private static readonly ID = 'FileChangeModel';
 	protected _filePath: vscode.Uri;
 	get filePath(): vscode.Uri {
 		return this._filePath;
@@ -59,7 +60,7 @@ export abstract class FileChangeModel {
 				const patch = await this.folderRepoManager.repository.diffBetween(this.pullRequest.base.sha, commit, this.fileName);
 				diffHunks = parsePatch(patch);
 			} catch (e) {
-				Logger.error(`Failed to parse patch for outdated comments: ${e}`);
+				Logger.error(`Failed to parse patch for outdated comments: ${e}`, FileChangeModel.ID);
 			}
 		}
 		return diffHunks;
diff --git a/src/view/gitContentProvider.ts b/src/view/gitContentProvider.ts
@@ -18,6 +18,7 @@ import { ReviewManager } from './reviewManager';
 import { GitFileChangeNode, RemoteFileChangeNode } from './treeNodes/fileChangeNode';
 
 export class GitContentFileSystemProvider extends RepositoryFileSystemProvider {
+	private static readonly ID = 'GitContentFileSystemProvider';
 	private _fallback?: (uri: vscode.Uri) => Promise<string>;
 
 	constructor(gitAPI: GitApiImpl, credentialStore: CredentialStore, private readonly getReviewManagers: () => ReviewManager[]) {
@@ -72,17 +73,17 @@ export class GitContentFileSystemProvider extends RepositoryFileSystemProvider {
 		const absolutePath = pathLib.join(repository.rootUri.fsPath, path).replace(/\\/g, '/');
 		let content: string | undefined;
 		try {
-			Logger.appendLine(`Getting change model (${repository.rootUri}) content for commit ${commit} and path ${absolutePath}`, 'GitContentFileSystemProvider');
+			Logger.appendLine(`Getting change model (${repository.rootUri}) content for commit ${commit} and path ${absolutePath}`, GitContentFileSystemProvider.ID);
 			content = await this.getChangeModelForFile(uri)?.showBase();
 			if (!content) {
-				Logger.appendLine(`Getting repository (${repository.rootUri}) content for commit ${commit} and path ${absolutePath}`, 'GitContentFileSystemProvider');
+				Logger.appendLine(`Getting repository (${repository.rootUri}) content for commit ${commit} and path ${absolutePath}`, GitContentFileSystemProvider.ID);
 				content = await repository.show(commit, absolutePath);
 			}
 			if (!content) {
 				throw new Error();
 			}
 		} catch (_) {
-			Logger.appendLine('Using fallback content provider.', 'GitContentFileSystemProvider');
+			Logger.appendLine('Using fallback content provider.', GitContentFileSystemProvider.ID);
 			content = await this._fallback(uri);
 			if (!content) {
 				// Content does not exist for the base or modified file for a file deletion or addition.
@@ -91,7 +92,7 @@ export class GitContentFileSystemProvider extends RepositoryFileSystemProvider {
 				try {
 					await repository.getCommit(commit);
 				} catch (err) {
-					Logger.error(err);
+					Logger.error(err, GitContentFileSystemProvider.ID);
 					// Only show the error if we know it's not an outdated commit
 					if (!this.getOutdatedChangeModelForFile(uri)) {
 						vscode.window.showErrorMessage(
diff --git a/src/view/reviewCommentController.ts b/src/view/reviewCommentController.ts
@@ -142,7 +142,7 @@ export class ReviewCommentController extends CommentControllerBase implements Co
 			const adjustedStartLine = startLine - 1;
 			const adjustedEndLine = endLine - 1;
 			if (adjustedStartLine < 0 || adjustedEndLine < 0) {
-				Logger.error(`Mapped new position for workspace comment thread is invalid. Original: (${thread.startLine}, ${thread.endLine}) New: (${adjustedStartLine}, ${adjustedEndLine})`);
+				Logger.error(`Mapped new position for workspace comment thread is invalid. Original: (${thread.startLine}, ${thread.endLine}) New: (${adjustedStartLine}, ${adjustedEndLine})`, ReviewCommentController.ID);
 			}
 			range = threadRange(adjustedStartLine, adjustedEndLine);
 		}
@@ -541,7 +541,7 @@ export class ReviewCommentController extends CommentControllerBase implements Co
 			editor => editor.document.uri.toString() === uri.toString(),
 		);
 		if (!this._folderRepoManager.activePullRequest?.head) {
-			Logger.error('Failed to get content diff. Cannot get content diff without an active pull request head.');
+			Logger.error('Failed to get content diff. Cannot get content diff without an active pull request head.', ReviewCommentController.ID);
 			throw new Error('Cannot get content diff without an active pull request head.');
 		}
 
@@ -561,7 +561,7 @@ export class ReviewCommentController extends CommentControllerBase implements Co
 				return await this._repository.diffWith(this._folderRepoManager.activePullRequest.head.sha, fileName);
 			}
 		} catch (e) {
-			Logger.error(`Failed to get content diff. ${formatError(e)}`);
+			Logger.error(`Failed to get content diff. ${formatError(e)}`, ReviewCommentController.ID);
 			if ((e.stderr as string | undefined)?.includes('bad object')) {
 				if (this._repository.state.HEAD?.upstream && retry) {
 					const pullBeforeCheckoutSetting = vscode.workspace.getConfiguration(PR_SETTINGS_NAMESPACE).get<PullPRBranchVariants>(PULL_PR_BRANCH_BEFORE_CHECKOUT, 'pull');
diff --git a/src/view/treeNodes/pullRequestNode.ts b/src/view/treeNodes/pullRequestNode.ts
@@ -126,7 +126,7 @@ export class PRNode extends TreeNode implements vscode.CommentingRangeProvider2
 
 			return result;
 		} catch (e) {
-			Logger.error(e);
+			Logger.error(`Error getting children ${e}: ${e.message}`, PRNode.ID);
 			return [];
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -57,7 +57,7 @@ class Log extends Disposable {`
`57`	`57`	`this._outputChannel.warn(this.logString(message, component));`
`58`	`58`	`}`
`59`	`59`
`60`		`- public error(message: any, component?: string) {`
	`60`	`+ public error(message: any, component: string) {`
`61`	`61`	`this._outputChannel.error(this.logString(message, component));`
`62`	`62`	`}`
`63`	`63`	`}`
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ export enum ProtocolType {`
`17`	`17`	`}`
`18`	`18`
`19`	`19`	`export class Protocol {`
	`20`	`+ private static readonly ID = 'Protocol';`
`20`	`21`	`public type: ProtocolType = ProtocolType.OTHER;`
`21`	`22`	`public host: string = '';`
`22`	`23`
`@@ -44,7 +45,7 @@ export class Protocol {`
`44`	`45`	`this.owner = this.getOwnerName(this.url.path) \|\| '';`
`45`	`46`	`}`
`46`	`47`	`} catch (e) {`
`47`		- Logger.error(`Failed to parse '${uriString}'`);
	`48`	+ Logger.error(`Failed to parse '${uriString}'`, Protocol.ID);
`48`	`49`	`vscode.window.showWarningMessage(`
`49`	`50`	`vscode.l10n.t('Unable to parse remote \'{0}\'. Please check that it is correctly formatted.', uriString)`
`50`	`51`	`);`
Original file line number	Diff line number	Diff line change
`@@ -283,7 +283,7 @@ export namespace DataUri {`
`283`	`283`	`try {`
`284`	`284`	`await vscode.workspace.fs.delete(vscode.Uri.joinPath(cacheLocation(context), id));`
`285`	`285`	`} catch (e) {`
`286`		- Logger.error(`Failed to delete avatar from cache: ${e}`);
	`286`	+ Logger.error(`Failed to delete avatar from cache: ${e}`, 'avatarCirclesAsImageDataUris');
`287`	`287`	`}`
`288`	`288`	`}));`
`289`	`289`	`}`
Original file line number	Diff line number	Diff line change
`@@ -280,7 +280,7 @@ export abstract class BaseCreatePullRequestViewProvider<T extends BasePullReques`
`280`	`280`	`try {`
`281`	`281`	`await pr.addAssignees([resolved]);`
`282`	`282`	`} catch (e) {`
`283`		- Logger.error(`Unable to assign pull request to user ${resolved}.`);
	`283`	+ Logger.error(`Unable to assign pull request to user ${resolved}.`, BaseCreatePullRequestViewProvider.ID);
`284`	`284`	`}`
`285`	`285`	`}`
`286`	`286`
`@@ -379,7 +379,7 @@ export abstract class BaseCreatePullRequestViewProvider<T extends BasePullReques`
`379`	`379`	`});`
`380`	`380`	`}`
`381`	`381`	`} catch (e) {`
`382`		`- Logger.error(formatError(e));`
	`382`	+ Logger.error(`Failed to add reviewers: ${formatError(e)}`, BaseCreatePullRequestViewProvider.ID);
`383`	`383`	`vscode.window.showErrorMessage(formatError(e));`
`384`	`384`	`} finally {`
`385`	`385`	`quickPick?.hide();`
Original file line number	Diff line number	Diff line change
`@@ -47,6 +47,7 @@ interface AuthResult {`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`export class CredentialStore extends Disposable {`
	`50`	`+ private static readonly ID = 'Authentication';`
`50`	`51`	`private _githubAPI: GitHub \| undefined;`
`51`	`52`	`private _sessionId: string \| undefined;`
`52`	`53`	`private _githubEnterpriseAPI: GitHub \| undefined;`
`@@ -204,7 +205,7 @@ export class CredentialStore extends Disposable {`
`204`	`205`	`}`
`205`	`206`	`return authResult;`
`206`	`207`	`} else {`
`207`		- Logger.debug(`No GitHub${getGitHubSuffix(authProviderId)} token found.`, 'Authentication');
	`208`	+ Logger.debug(`No GitHub${getGitHubSuffix(authProviderId)} token found.`, CredentialStore.ID);
`208`	`209`	`return authResult;`
`209`	`210`	`}`
`210`	`211`	`}`
`@@ -328,9 +329,9 @@ export class CredentialStore extends Disposable {`
`328`	`329`	`try {`
`329`	`330`	`await this.initialize(authProviderId, sessionOptions);`
`330`	`331`	`} catch (e) {`
`331`		- Logger.error(`${errorPrefix}: ${e}`);
	`332`	+ Logger.error(`Login error: ${errorPrefix}: ${e}`, CredentialStore.ID);
`332`	`333`	`if (e instanceof Error && e.stack) {`
`333`		`- Logger.error(e.stack);`
	`334`	`+ Logger.error(e.stack, CredentialStore.ID);`
`334`	`335`	`}`
`335`	`336`	`if (e.message === 'Cancelled') {`
`336`	`337`	`isCanceled = true;`
Original file line number	Diff line number	Diff line change
`@@ -465,7 +465,7 @@ export class PullRequestOverviewPanel extends IssueOverviewPanel<PullRequestMode`
`465`	`465`	`});`
`466`	`466`	`}`
`467`	`467`	`} catch (e) {`
`468`		`- Logger.error(formatError(e));`
	`468`	`+ Logger.error(formatError(e), PullRequestOverviewPanel.ID);`
`469`	`469`	`vscode.window.showErrorMessage(formatError(e));`
`470`	`470`	`} finally {`
`471`	`471`	`quickPick?.hide();`
Original file line number	Diff line number	Diff line change
`@@ -69,6 +69,7 @@ import {`
`69`	`69`	`const CREATING_ISSUE_FROM_FILE_CONTEXT = 'issues.creatingFromFile';`
`70`	`70`
`71`	`71`	`export class IssueFeatureRegistrar extends Disposable {`
	`72`	`+ private static readonly ID = 'IssueFeatureRegistrar';`
`72`	`73`	`private _stateManager: StateManager;`
`73`	`74`	`private _newIssueCache: NewIssueCache;`
`74`	`75`
`@@ -1260,7 +1261,7 @@ ${options?.body ?? ''}\n`
`1260`	`1261`	`}`
`1261`	`1262`	`if (!folderManager) {`
`1262`	`1263`	vscode.window.showErrorMessage(vscode.l10n.t(`Could not find the correct repository for the issue; see logs for more details.`));
`1263`		- Logger.error(`Could not find the folder manager for the issue originUri: ${originUri.toString()}`);
	`1264`	+ Logger.error(`Could not find the folder manager for the issue originUri: ${originUri.toString()}`, IssueFeatureRegistrar.ID);
`1264`	`1265`	`return false;`
`1265`	`1266`	`}`
`1266`	`1267`	`}`
Original file line number	Diff line number	Diff line change
`@@ -114,6 +114,7 @@ class GitCommitNode extends TreeNode {`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`abstract class CompareChangesTreeProvider extends Disposable implements vscode.TreeDataProvider<TreeNode>, BaseTreeNode {`
	`117`	`+ private static readonly ID = 'CompareChangesTreeProvider';`
`117`	`118`	`private _view: vscode.TreeView<TreeNode>;`
`118`	`119`	`private _children: TreeNode[] \| undefined;`
`119`	`120`	`private _onDidChangeTreeData = new vscode.EventEmitter<TreeNode \| void>();`
`@@ -188,7 +189,7 @@ abstract class CompareChangesTreeProvider extends Disposable implements vscode.T`
`188`	`189`	`this._children = await this.getGitChildren(element);`
`189`	`190`	`}`
`190`	`191`	`} catch (e) {`
`191`		- Logger.error(`Comparing changes failed: ${e}`);
	`192`	+ Logger.error(`Comparing changes failed: ${e}`, CompareChangesTreeProvider.ID);
`192`	`193`	`return [];`
`193`	`194`	`}`
`194`	`195`	`return this._children;`