Impls changed file count for GitHub

src/git/models/diff.ts

@@ -48,9 +48,9 @@ export interface GitDiff {
 }
 
 export interface GitDiffShortStat {
-	readonly files: number;
-	readonly insertions: number;
+	readonly additions: number;
 	readonly deletions: number;
+	readonly changedFiles: number;
 }
 
 export type GitDiffFilter = 'A' | 'C' | 'D' | 'M' | 'R' | 'T' | 'U' | 'X' | 'B' | '*';

src/git/parsers/diffParser.ts

@@ -190,8 +190,8 @@ export class GitDiffParser {
 		const [, files, insertions, deletions] = match;
 
 		const diffShortStat: GitDiffShortStat = {
-			files: files == null ? 0 : parseInt(files, 10),
-			insertions: insertions == null ? 0 : parseInt(insertions, 10),
+			changedFiles: files == null ? 0 : parseInt(files, 10),
+			additions: insertions == null ? 0 : parseInt(insertions, 10),
 			deletions: deletions == null ? 0 : parseInt(deletions, 10),
 		};
 

src/premium/github/githubGitProvider.ts

@@ -680,8 +680,20 @@ export class GitHubGitProvider implements GitProvider, Disposable {
 	}
 
 	@log()
-	async getChangedFilesCount(_repoPath: string, _ref?: string): Promise<GitDiffShortStat | undefined> {
-		return undefined;
+	async getChangedFilesCount(repoPath: string, ref?: string): Promise<GitDiffShortStat | undefined> {
+		// TODO@eamodio if there is no ref we can't return anything, until we can get at the change store from RemoteHub
+		if (!ref) return undefined;
+
+		const commit = await this.getCommit(repoPath, ref);
+		if (commit?.stats == null) return undefined;
+
+		const { stats } = commit;
+
+		const changedFiles =
+			typeof stats.changedFiles === 'number'
+				? stats.changedFiles
+				: stats.changedFiles.added + stats.changedFiles.changed + stats.changedFiles.deleted;
+		return { additions: stats.additions, deletions: stats.deletions, changedFiles: changedFiles };
 	}
 
 	@log()

src/views/nodes/branchTrackingStatusFilesNode.ts

@@ -112,7 +112,7 @@ export class BranchTrackingStatusFilesNode extends ViewNode {
 			this.repoPath,
 			`${this.status.upstream}${this.direction === 'behind' ? '..' : '...'}`,
 		);
-		const files = stats?.files ?? 0;
+		const files = stats?.changedFiles ?? 0;
 
 		const label = `${Strings.pluralize('file', files)} changed`;
 		const item = new TreeItem(label, TreeItemCollapsibleState.Collapsed);

src/views/nodes/statusFilesNode.ts

@@ -136,7 +136,7 @@ export class StatusFilesNode extends ViewNode {
 						`${this.status.upstream}...`,
 					);
 					if (stats != null) {
-						files += stats.files;
+						files += stats.changedFiles;
 					} else {
 						files = -1;
 					}