feat(cli): improve visual formatting and fix GitHub stats

Visual Improvements:
- Add tree branches and file icons to 'dev map' hot paths
- Add tree branches and file icons to 'dev activity' output
- Extract getFileIcon() to shared utility in @lytics/dev-agent-core

GitHub Stats Fix:
- Track issue and PR states separately (issuesByState, prsByState)
- Fix confusing display showing '14 open PRs' when there were 0
- Add proper per-type state counts to GitHubIndexStats interface
- Update display to show accurate counts for issues and PRs

Progress Display Enhancements:
- Add detailed scanning progress with rates (e.g., '1,234/4,567 files (27%, 45 files/sec)')
- Extend detailed progress to 'dev update', 'dev git index', and 'dev github index'
- Add update plan display to 'dev update' (shows changed/added/deleted before starting)
- Refactor progress formatting into reusable updateSectionWithRate() method
- Fix NaN display when totalFiles is 0 (now shows 'Discovering files...')

Test Updates:
- Update map test to match new hot paths format with tree branches

All visual outputs now use consistent tree-based formatting with file icons.

Author: prosdev

packages/cli/src/commands/activity.ts

@@ -5,6 +5,7 @@
 import * as path from 'node:path';
 import {
   type FileMetrics,
+  getFileIcon,
   getMostActive,
   getStoragePath,
   MetricsStore,
@@ -31,43 +32,38 @@ function formatRelativeTime(date: Date): string {
 }
 
 /**
- * Format files as a compact table
+ * Format files with tree branches and icons
  */
 function formatFileMetricsTable(files: FileMetrics[]): string {
   if (files.length === 0) return '';
 
-  // Calculate column widths
-  const maxPathLen = Math.max(...files.map((f) => f.filePath.length), 40);
-  const pathWidth = Math.min(maxPathLen, 55);
+  let output = '';
 
-  // Header
-  let output = chalk.bold(
-    `${'FILE'.padEnd(pathWidth)}  ${'COMMITS'.padStart(7)}  ${'LOC'.padStart(6)}  ${'AUTHORS'.padStart(7)}  ${'LAST CHANGE'}\n`
-  );
+  for (let i = 0; i < files.length; i++) {
+    const file = files[i];
+    const isLast = i === files.length - 1;
+    const branch = isLast ? '└─' : '├─';
+    const connector = isLast ? ' ' : '│'; // Vertical line for non-last items
+    const icon = getFileIcon(file.filePath);
 
-  // Separator line
-  output += chalk.dim(`${'─'.repeat(pathWidth + 2 + 7 + 2 + 6 + 2 + 7 + 2 + 12)}\n`);
-
-  // Rows
-  for (const file of files) {
-    // Truncate path if too long
-    let displayPath = file.filePath;
-    if (displayPath.length > pathWidth) {
-      displayPath = `...${displayPath.slice(-(pathWidth - 3))}`;
-    }
-    displayPath = displayPath.padEnd(pathWidth);
-
-    const commits = String(file.commitCount).padStart(7);
-    const loc = String(file.linesOfCode).padStart(6);
-
-    // Author count with emoji
-    const authorIcon = file.authorCount === 1 ? ' 👤' : file.authorCount === 2 ? ' 👥' : '👥👥';
-    const authors = `${String(file.authorCount).padStart(5)}${authorIcon}`;
+    // Author info
+    const authorIcon = file.authorCount === 1 ? '👤' : file.authorCount === 2 ? '👥' : '👥👥';
 
     // Relative time
     const lastChange = file.lastModified ? formatRelativeTime(file.lastModified) : 'unknown';
 
-    output += `${chalk.dim(displayPath)}  ${chalk.cyan(commits)}  ${chalk.yellow(loc)}  ${chalk.green(authors)}  ${chalk.gray(lastChange)}\n`;
+    // File path line with icon and branch
+    output += `${chalk.dim(branch)} ${icon} ${file.filePath}\n`;
+
+    // Metrics line with vertical connector
+    output += chalk.dim(
+      `${connector}     ${file.commitCount} commits • ${file.authorCount} ${authorIcon} • Last: ${lastChange}\n`
+    );
+
+    // Add vertical line separator between items (except after last)
+    if (!isLast) {
+      output += chalk.dim(`${connector}\n`);
+    }
   }
 
   return output;

packages/cli/src/commands/stats.ts

@@ -100,6 +100,8 @@ async function loadCurrentStats(): Promise<{
       totalDocuments: state.totalDocuments || 0,
       byType: state.byType || {},
       byState: state.byState || {},
+      issuesByState: state.issuesByState,
+      prsByState: state.prsByState,
       lastIndexed: state.lastIndexed || '',
       indexDuration: state.indexDuration || 0,
     };
@@ -217,6 +219,8 @@ What You'll See:
               totalDocuments: number;
               byType: { issue?: number; pull_request?: number };
               byState: { open?: number; closed?: number; merged?: number };
+              issuesByState?: { open: number; closed: number };
+              prsByState?: { open: number; closed: number; merged: number };
               lastIndexed: string;
             } | null) || undefined,
         });

packages/cli/src/utils/output.ts

@@ -142,18 +142,26 @@ export function formatGitHubSummary(githubStats: {
   totalDocuments: number;
   byType: { issue?: number; pull_request?: number };
   byState: { open?: number; closed?: number; merged?: number };
+  issuesByState?: { open: number; closed: number };
+  prsByState?: { open: number; closed: number; merged: number };
   lastIndexed: string;
 }): string {
   const issues = githubStats.byType.issue || 0;
   const prs = githubStats.byType.pull_request || 0;
-  const open = githubStats.byState.open || 0;
-  const merged = githubStats.byState.merged || 0;
+
+  // Use per-type state counts if available (new format), fall back to aggregate (old format)
+  const issuesOpen = githubStats.issuesByState?.open ?? 0;
+  const issuesClosed = githubStats.issuesByState?.closed ?? 0;
+  const prsOpen = githubStats.prsByState?.open ?? 0;
+  const prsMerged = githubStats.prsByState?.merged ?? 0;
 
   const timeSince = getTimeSince(new Date(githubStats.lastIndexed));
 
   return [
     `🔗 ${chalk.bold(githubStats.repository)} • ${formatNumber(githubStats.totalDocuments)} documents`,
-    `   ${chalk.gray(issues.toString())} issues • ${chalk.gray(prs.toString())} PRs • ${chalk.gray(open.toString())} open • ${chalk.gray(merged.toString())} merged • Synced ${timeSince}`,
+    `   Issues: ${chalk.gray(issues.toString())} total (${issuesOpen} open, ${issuesClosed} closed)`,
+    `   Pull Requests: ${chalk.gray(prs.toString())} total (${prsOpen} open, ${prsMerged} merged)`,
+    `   Last synced: ${timeSince}`,
   ].join('\n');
 }
 
@@ -369,6 +377,8 @@ export function printRepositoryStats(data: {
     totalDocuments: number;
     byType: { issue?: number; pull_request?: number };
     byState: { open?: number; closed?: number; merged?: number };
+    issuesByState?: { open: number; closed: number };
+    prsByState?: { open: number; closed: number; merged: number };
     lastIndexed: string;
   } | null;
 }): void {
@@ -471,19 +481,21 @@ export function printRepositoryStats(data: {
     const issues = githubStats.byType.issue || 0;
     const prs = githubStats.byType.pull_request || 0;
 
+    // Use per-type state counts if available (new format), fall back to aggregate (old format)
+    const issuesOpen = githubStats.issuesByState?.open ?? githubStats.byState.open ?? 0;
+    const issuesClosed = githubStats.issuesByState?.closed ?? githubStats.byState.closed ?? 0;
+    const prsOpen = githubStats.prsByState?.open ?? githubStats.byState.open ?? 0;
+    const prsMerged = githubStats.prsByState?.merged ?? githubStats.byState.merged ?? 0;
+
     if (issues > 0) {
-      const openIssues = githubStats.byState.open || 0;
-      const closedIssues = githubStats.byState.closed || 0;
       output.log(
-        `  Issues: ${chalk.bold(issues.toString())} total (${chalk.green(`${openIssues} open`)}, ${chalk.gray(`${closedIssues} closed`)})`
+        `  Issues: ${chalk.bold(issues.toString())} total (${issuesOpen} open, ${issuesClosed} closed)`
       );
     }
 
     if (prs > 0) {
-      const openPRs = githubStats.byState.open || 0;
-      const mergedPRs = githubStats.byState.merged || 0;
       output.log(
-        `  Pull Requests: ${chalk.bold(prs.toString())} total (${chalk.green(`${openPRs} open`)}, ${chalk.magenta(`${mergedPRs} merged`)})`
+        `  Pull Requests: ${chalk.bold(prs.toString())} total (${prsOpen} open, ${prsMerged} merged)`
       );
     }
 
@@ -905,16 +917,21 @@ export function printGitHubStats(githubStats: {
   totalDocuments: number;
   byType: { issue?: number; pull_request?: number; discussion?: number };
   byState: { open?: number; closed?: number; merged?: number };
+  issuesByState?: { open: number; closed: number };
+  prsByState?: { open: number; closed: number; merged: number };
   lastIndexed: string;
   indexDuration?: number;
 }): void {
   const issues = githubStats.byType.issue || 0;
   const prs = githubStats.byType.pull_request || 0;
   const discussions = githubStats.byType.discussion || 0;
 
-  const openCount = githubStats.byState.open || 0;
-  const closedCount = githubStats.byState.closed || 0;
-  const mergedCount = githubStats.byState.merged || 0;
+  // Use per-type state counts if available (new format), fall back to aggregate (old format)
+  const issueOpen = githubStats.issuesByState?.open ?? 0;
+  const issueClosed = githubStats.issuesByState?.closed ?? 0;
+  const prOpen = githubStats.prsByState?.open ?? 0;
+  const prClosed = githubStats.prsByState?.closed ?? 0;
+  const prMerged = githubStats.prsByState?.merged ?? 0;
 
   const timeSince = getTimeSince(new Date(githubStats.lastIndexed));
 
@@ -928,9 +945,6 @@ export function printGitHubStats(githubStats: {
   // Issues breakdown
   if (issues > 0) {
     const issueStates: string[] = [];
-    // Calculate issue-specific counts (open + closed, no merged for issues)
-    const issueOpen = openCount > 0 ? openCount : 0;
-    const issueClosed = closedCount > 0 ? closedCount : 0;
 
     if (issueOpen > 0) {
       issueStates.push(`${chalk.green('●')} ${issueOpen} open`);
@@ -949,13 +963,13 @@ export function printGitHubStats(githubStats: {
   // Pull requests breakdown
   if (prs > 0) {
     const prStates: string[] = [];
-    // For PRs, we show open and merged
-    const prOpen = openCount > 0 ? openCount : 0;
-    const prMerged = mergedCount > 0 ? mergedCount : 0;
 
     if (prOpen > 0) {
       prStates.push(`${chalk.green('●')} ${prOpen} open`);
     }
+    if (prClosed > 0) {
+      prStates.push(`${chalk.gray('●')} ${prClosed} closed`);
+    }
     if (prMerged > 0) {
       prStates.push(`${chalk.magenta('●')} ${prMerged} merged`);
     }

packages/core/src/map/__tests__/map.test.ts

@@ -485,8 +485,9 @@ describe('Codebase Map', () => {
       const output = formatCodebaseMap(map, { includeHotPaths: true });
 
       expect(output).toContain('## Hot Paths');
-      expect(output).toContain('src/core.ts');
+      expect(output).toContain('**core.ts**'); // Filename in bold
       expect(output).toContain('2 refs');
+      expect(output).toContain('src'); // Directory path on separate line
     });
   });
 

packages/core/src/map/index.ts

@@ -7,6 +7,7 @@ import * as path from 'node:path';
 import type { Logger } from '@lytics/kero';
 import type { LocalGitExtractor } from '../git/extractor';
 import type { RepositoryIndexer } from '../indexer';
+import { getFileIcon } from '../utils/icons';
 import type { SearchResult } from '../vector/types';
 import type {
   ChangeFrequency,
@@ -486,26 +487,6 @@ function computeHotPaths(docs: SearchResult[], maxPaths: number): HotPath[] {
   return sorted;
 }
 
-/**
- * Get file icon based on extension
- */
-function getFileIcon(ext: string): string {
-  const iconMap: Record<string, string> = {
-    ts: '📘',
-    tsx: '⚛️',
-    js: '📜',
-    jsx: '⚛️',
-    go: '🐹',
-    py: '🐍',
-    rs: '🦀',
-    md: '📝',
-    json: '📋',
-    yaml: '⚙️',
-    yml: '⚙️',
-  };
-  return iconMap[ext] || '📄';
-}
-
 /**
  * Format codebase map as readable text
  */

packages/core/src/utils/icons.ts

@@ -0,0 +1,27 @@
+/**
+ * Utility functions for file and UI icons
+ */
+
+/**
+ * Get file icon based on extension
+ */
+export function getFileIcon(filePathOrExt: string): string {
+  // Extract extension (handle both full paths and just extensions)
+  const ext = filePathOrExt.includes('.') ? filePathOrExt.split('.').pop() || '' : filePathOrExt;
+
+  const iconMap: Record<string, string> = {
+    ts: '📘',
+    tsx: '⚛️',
+    js: '📜',
+    jsx: '⚛️',
+    go: '🐹',
+    py: '🐍',
+    rs: '🦀',
+    md: '📝',
+    json: '📋',
+    yaml: '⚙️',
+    yml: '⚙️',
+  };
+
+  return iconMap[ext] || '📄';
+}

packages/core/src/utils/index.ts

@@ -4,5 +4,6 @@
 
 export * from './concurrency';
 export * from './file-validator';
+export * from './icons';
 export * from './retry';
 export * from './wasm-resolver';

packages/subagents/src/github/indexer.ts

@@ -178,6 +178,21 @@ export class GitHubIndexer {
       {} as Record<string, number>
     );
 
+    // Calculate states per type for accurate reporting
+    const issuesByState = { open: 0, closed: 0 };
+    const prsByState = { open: 0, closed: 0, merged: 0 };
+
+    for (const doc of enrichedDocs) {
+      if (doc.type === 'issue') {
+        if (doc.state === 'open') issuesByState.open++;
+        else if (doc.state === 'closed') issuesByState.closed++;
+      } else if (doc.type === 'pull_request') {
+        if (doc.state === 'open') prsByState.open++;
+        else if (doc.state === 'closed') prsByState.closed++;
+        else if (doc.state === 'merged') prsByState.merged++;
+      }
+    }
+
     // Update state
     this.state = {
       version: INDEXER_VERSION,
@@ -186,6 +201,8 @@ export class GitHubIndexer {
       totalDocuments: enrichedDocs.length,
       byType: byType as Record<'issue' | 'pull_request' | 'discussion', number>,
       byState: byState as Record<'open' | 'closed' | 'merged', number>,
+      issuesByState,
+      prsByState,
     };
 
     // Save state to disk
@@ -202,6 +219,8 @@ export class GitHubIndexer {
       totalDocuments: enrichedDocs.length,
       byType: byType as Record<'issue' | 'pull_request' | 'discussion', number>,
       byState: byState as Record<'open' | 'closed' | 'merged', number>,
+      issuesByState,
+      prsByState,
       lastIndexed: this.state.lastIndexed,
       indexDuration: durationMs,
     };
@@ -398,6 +417,8 @@ export class GitHubIndexer {
       totalDocuments: this.state.totalDocuments,
       byType: this.state.byType,
       byState: this.state.byState,
+      issuesByState: this.state.issuesByState,
+      prsByState: this.state.prsByState,
       lastIndexed: this.state.lastIndexed,
       indexDuration: 0,
     };

packages/types/src/github.ts

@@ -110,7 +110,9 @@ export interface GitHubIndexerState {
   lastIndexed: string; // ISO date
   totalDocuments: number;
   byType: Record<GitHubDocumentType, number>;
-  byState: Record<GitHubState, number>;
+  byState: Record<GitHubState, number>; // Deprecated: aggregate counts (kept for compatibility)
+  issuesByState: { open: number; closed: number };
+  prsByState: { open: number; closed: number; merged: number };
 }
 
 /**
@@ -145,7 +147,9 @@ export interface GitHubIndexStats {
   repository: string;
   totalDocuments: number;
   byType: Record<GitHubDocumentType, number>;
-  byState: Record<GitHubState, number>;
+  byState: Record<GitHubState, number>; // Deprecated: aggregate counts (kept for compatibility)
+  issuesByState: { open: number; closed: number };
+  prsByState: { open: number; closed: number; merged: number };
   lastIndexed: string; // ISO date
   indexDuration: number; // milliseconds
 }