Clarify toolbar collapse counts

Author: TennyZhuang

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "show-me-talk",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Chrome extension that auto-collapses code in GitHub PRs so only the talk stays open.",
   "type": "module",
   "private": false,

package.json

@@ -8,7 +8,7 @@ interface ScanStats {
   totalFiles: number;
   talkFiles: number;
   codeFiles: number;
-  collapsedFiles: number;
+  collapsedCodeFiles: number;
 }
 
 function isPullRequestPage(url: URL): boolean {
@@ -186,7 +186,7 @@ export class GitHubPullRequestController {
       totalFiles: files.length,
       talkFiles: 0,
       codeFiles: 0,
-      collapsedFiles: 0,
+      collapsedCodeFiles: 0,
     };
 
     for (const file of files) {
@@ -210,10 +210,17 @@ export class GitHubPullRequestController {
       }
 
       const toggleButton = file.querySelector<HTMLButtonElement>('.file-header .js-details-target');
-      if (toggleButton && toggleButton.getAttribute('aria-expanded') === 'true') {
+      if (!toggleButton) {
+        continue;
+      }
+
+      if (toggleButton.getAttribute('aria-expanded') === 'true') {
         this.programmaticToggleButtons.add(toggleButton);
         toggleButton.click();
-        stats.collapsedFiles += 1;
+      }
+
+      if (toggleButton.getAttribute('aria-expanded') === 'false') {
+        stats.collapsedCodeFiles += 1;
       }
     }
 
@@ -284,7 +291,7 @@ export class GitHubPullRequestController {
 
     const summary = toolbar.querySelector<HTMLElement>('.show-me-talk-toolbar__summary');
     if (summary) {
-      summary.textContent = `${stats.collapsedFiles} collapsed · ${stats.talkFiles} talk files kept open · ${stats.codeFiles} code files detected`;
+      summary.textContent = `${stats.collapsedCodeFiles}/${stats.codeFiles} code files collapsed · ${stats.talkFiles} talk files kept open`;
     }
   }
 

src/content/pullRequestController.ts

@@ -9,7 +9,7 @@ test('keeps docs open on docs-only PRs', async ({ context }) => {
   const fileHeader = page.locator('.file-header[data-path="docs/src/chrome-extensions-js-python.md"]');
   await expect(fileHeader).toBeVisible();
   await expect(page.locator('#show-me-talk-toolbar')).toBeVisible();
-  await expect(page.locator('#show-me-talk-toolbar')).toContainText('1 talk files');
+  await expect(page.locator('#show-me-talk-toolbar')).toContainText('0/0 code files collapsed · 1 talk files kept open');
   await expect(fileHeader.locator('.js-details-target')).toHaveAttribute('aria-expanded', 'true');
   await page.close();
 });
@@ -23,6 +23,7 @@ test('collapses code on code-only PRs', async ({ context }) => {
   const fileHeader = page.locator('.file-header[data-path="packages/playwright-core/src/cli/client/program.ts"]');
   await expect(fileHeader).toBeVisible();
   await expect(page.locator('#show-me-talk-toolbar')).toBeVisible();
+  await expect(page.locator('#show-me-talk-toolbar')).toContainText('1/1 code files collapsed');
   await expect(fileHeader.locator('.js-details-target')).toHaveAttribute('aria-expanded', 'false');
   await page.close();
 });
@@ -39,6 +40,7 @@ test('keeps docs open and collapses code on mixed PRs', async ({ context }) => {
   await expect(docsHeader).toBeVisible();
   await expect(codeHeader).toBeVisible();
   await expect(page.locator('#show-me-talk-toolbar')).toBeVisible();
+  await expect(page.locator('#show-me-talk-toolbar')).toContainText('15/15 code files collapsed · 2 talk files kept open');
   await expect(docsHeader.locator('.js-details-target')).toHaveAttribute('aria-expanded', 'true');
   await expect(codeHeader.locator('.js-details-target')).toHaveAttribute('aria-expanded', 'false');
   await page.close();