Adds more logging

Author: eamodio

src/env/node/git/sub-providers/commits.ts

@@ -49,11 +49,12 @@ import { isRevisionRange, isSha, isUncommitted, isUncommittedStaged } from '../.
 import { isUserMatch } from '../../../../git/utils/user.utils';
 import { configuration } from '../../../../system/-webview/configuration';
 import { splitPath } from '../../../../system/-webview/path';
-import { log } from '../../../../system/decorators/log';
+import { debug, log } from '../../../../system/decorators/log';
 import { filterMap, first, join, last, some } from '../../../../system/iterable';
 import { Logger } from '../../../../system/logger';
 import { getLogScope } from '../../../../system/logger.scope';
 import { isFolderGlob, stripFolderGlob } from '../../../../system/path';
+import { maybeStopWatch } from '../../../../system/stopwatch';
 import type { CachedLog, TrackedGitDocument } from '../../../../trackers/trackedDocument';
 import { GitDocumentState } from '../../../../trackers/trackedDocument';
 import type { Git, GitResult } from '../git';
@@ -380,6 +381,7 @@ export class CommitsGitSubProvider implements GitCommitsSubProvider {
 		return this.getLogCore(repoPath, rev, options, cancellation);
 	}
 
+	@debug({ args: { 2: false, 3: false, 4: false }, exit: true })
 	private async getLogCore(
 		repoPath: string,
 		rev?: string | undefined,
@@ -1326,6 +1328,9 @@ async function parseCommits(
 	if (resultOrStream instanceof Promise) {
 		const result = await resultOrStream;
 
+		const scope = getLogScope();
+		using sw = maybeStopWatch(scope, { log: false, logLevel: 'debug' });
+
 		if (stashes?.size) {
 			const allowFilteredFiles = searchFilters?.files ?? false;
 			const stashesOnly = searchFilters?.type === 'stash';
@@ -1367,6 +1372,8 @@ async function parseCommits(
 			}
 		}
 
+		sw?.stop({ suffix: ` created ${count} commits` });
+
 		return { commits: commits, count: count, countStashChildCommits: countStashChildCommits };
 	}
 

src/views/nodes/branchNode.ts

@@ -13,7 +13,7 @@ import { isStash } from '../../git/models/commit';
 import type { GitLog } from '../../git/models/log';
 import type { PullRequest, PullRequestState } from '../../git/models/pullRequest';
 import type { GitBranchReference } from '../../git/models/reference';
-import type { Repository } from '../../git/models/repository';
+import type { Repository, RepositoryChangeEvent } from '../../git/models/repository';
 import type { GitUser } from '../../git/models/user';
 import type { GitWorktree } from '../../git/models/worktree';
 import { getBranchAheadRange, getBranchMergeTargetName } from '../../git/utils/-webview/branch.utils';
@@ -780,9 +780,7 @@ export class CommitsCurrentBranchNode extends SubscribeableViewNode<'commits-cur
 		const interval = getLastFetchedUpdateInterval(lastFetched);
 		if (lastFetched !== 0 && interval > 0) {
 			return Disposable.from(
-				this.repo != null
-					? weakEvent(this.repo.onDidChange, () => this.view.triggerNodeChange(this), this)
-					: emptyDisposable,
+				this.repo != null ? weakEvent(this.repo.onDidChange, this.onRepositoryChanged, this) : emptyDisposable,
 				disposableInterval(() => {
 					// Check if the interval should change, and if so, reset it
 					if (interval !== getLastFetchedUpdateInterval(lastFetched)) {
@@ -796,4 +794,9 @@ export class CommitsCurrentBranchNode extends SubscribeableViewNode<'commits-cur
 
 		return undefined;
 	}
+
+	@debug<CommitsCurrentBranchNode['onRepositoryChanged']>({ args: { 0: e => e.toString() } })
+	private onRepositoryChanged(_e: RepositoryChangeEvent) {
+		this.view.triggerNodeChange(this);
+	}
 }