feat: Add array traversal support to sumNestedValue function and update tests

Author: austenstone

__tests__/main.test.ts

@@ -27,6 +27,7 @@ const exampleResponseEnterprise = JSON.parse(sample);
 test('createJobSummaryUsage(enterpriseUsage)', async () => {
   const summary = await createJobSummaryUsage(exampleResponseEnterprise, 'enterprise');
   writeFileSync('./__tests__/mock/sample-output.md', summary.stringify());
+  console.log('Summary:', summary.stringify());
   expect(summary).toBeDefined();
 });
 
@@ -78,6 +79,29 @@ test('sumNestedValue with completely missing path', () => {
   expect(sumNestedValue(data, ['a', 'b'])).toBe(0); // Path doesn't exist at all
 });
 
+// New test for array traversal
+test('sumNestedValue with array traversal', () => {
+  const data = [
+    { 
+      a: { 
+        items: [
+          { value: 5 },
+          { value: 10 }
+        ] 
+      } 
+    },
+    { 
+      a: { 
+        items: [
+          { value: 15 },
+          { value: 20 }
+        ] 
+      } 
+    }
+  ];
+  expect(sumNestedValue(data, ['a', 'items', 'value'])).toBe(50); // Should sum all values in the arrays
+});
+
 test('sumNestedValue with exampleResponseEnterprise data', () => {
   // Test with real data paths
   const totalChatEngagedUsers = sumNestedValue(exampleResponseEnterprise, ['copilot_ide_chat', 'total_engaged_users']);

__tests__/mock/sample-output.md

@@ -1,6 +1,6 @@
 <h1>Copilot Usage for enterprise<br>3/1/2025 - 3/27/2025</h1>
 <h1>Usage Summary</h1>
-<ul><li>Total Code Suggestions: 243,150</li><li>Total Code Acceptances: 57,011</li><li>Acceptance Rate: 23.45%</li><li>Total Lines of Code Accepted: 75,177</li><li>Total Chat Interactions: 2,937</li><li>Total Chat Copy Events: 0</li><li>Total Chat Insertion Events: 0</li></ul>
+<ul><li>Total Code Suggestions: 243,150</li><li>Total Code Acceptances: 57,011</li><li>Acceptance Rate: 23.45%</li><li>Total Lines of Code Accepted: 75,177</li><li>Total Chat Interactions: 2,937</li><li>Total Chat Copy Events: 2,989</li><li>Total Chat Insertion Events: 2,989</li></ul>
 
 ```mermaid
 ---

package.json

@@ -5,7 +5,7 @@
   "private": false,
   "scripts": {
     "build": "tsc && ncc build src/index.ts -o dist --source-map --license LICENSE",
-    "test": "vitest",
+    "test": "vitest run",
     "lint": "eslint src/**/*.ts"
   },
   "keywords": [

src/job-summary.ts

@@ -39,13 +39,36 @@ const dateFormat = (date: string, options: Intl.DateTimeFormatOptions = {
 // Enhanced data aggregation utilities
 export const sumNestedValue = <T extends object>(data: T[], path: string[]): number => {
   return data.reduce((sum, obj) => {
-    let current: Record<string, unknown> = obj as Record<string, unknown>;
-    for (const key of path) {
-      console.log(`Checking key: ${key}, current value: ${current}`);
-      if (!current?.[key]) return sum;
-      current = current[key] as Record<string, unknown>;
-    }
-    return sum + (typeof current === 'number' ? current : 0);
+    let result = 0;
+    
+    // Helper function to recursively traverse the object/array
+    const traverse = (current: any, pathIndex: number) => {
+      // Return if we've reached an invalid path
+      if (current === undefined || current === null) return;
+      
+      if (pathIndex >= path.length) {
+        // We've reached the end of the path, add the value if it's a number
+        if (typeof current === 'number') {
+          result += current;
+        }
+        return;
+      }
+      
+      const key = path[pathIndex];
+      
+      if (Array.isArray(current)) {
+        // If current is an array, traverse each element
+        current.forEach(item => traverse(item, pathIndex));
+      } else if (typeof current === 'object') {
+        // If current has the key, traverse deeper
+        if (key in current) {
+          traverse(current[key], pathIndex + 1);
+        }
+      }
+    };
+    
+    traverse(obj, 0);
+    return sum + result;
   }, 0);
 };