fix: fix bug in summary table construction

Author: hesreallyhim

.gitignore

@@ -1,2 +1,4 @@
 node_modules/
 coverage/
+*.social-preview*
+.DS_Store

dist/post/index.js

@@ -27845,16 +27845,18 @@ function formatMs(ms) {
 
 function makeSummaryTable(resources) {
   const summaryTable = [
-    { data: 'Bucket', header: true },
-    { data: 'Used', header: true },
-    { data: 'Remaining', header: true }
+    [
+      { data: 'Bucket', header: true },
+      { data: 'Used', header: true },
+      { data: 'Remaining', header: true }
+    ]
   ];
   for (const [bucket, info] of Object.entries(resources)) {
-    summaryTable.push(
+    summaryTable.push([
       { data: bucket },
       { data: String(info.used) },
       { data: String(info.remaining) }
-    );
+    ]);
   }
 
   return summaryTable;
@@ -28079,6 +28081,9 @@ async function run() {
     const outPath = (core.getInput('output_path') || '').trim();
     maybeWrite(outPath, output);
 
+    log(
+      `[github-api-usage-tracker] Preparing summary table for ${Object.keys(data).length} bucket(s)`
+    );
     core.summary
       .addHeading('GitHub API Usage Tracker Summary')
       .addTable(makeSummaryTable(data))

src/post-utils.js

@@ -18,16 +18,18 @@ function formatMs(ms) {
 
 function makeSummaryTable(resources) {
   const summaryTable = [
-    { data: 'Bucket', header: true },
-    { data: 'Used', header: true },
-    { data: 'Remaining', header: true }
+    [
+      { data: 'Bucket', header: true },
+      { data: 'Used', header: true },
+      { data: 'Remaining', header: true }
+    ]
   ];
   for (const [bucket, info] of Object.entries(resources)) {
-    summaryTable.push(
+    summaryTable.push([
       { data: bucket },
       { data: String(info.used) },
       { data: String(info.remaining) }
-    );
+    ]);
   }
 
   return summaryTable;

src/post.js

@@ -124,6 +124,9 @@ async function run() {
     const outPath = (core.getInput('output_path') || '').trim();
     maybeWrite(outPath, output);
 
+    log(
+      `[github-api-usage-tracker] Preparing summary table for ${Object.keys(data).length} bucket(s)`
+    );
     core.summary
       .addHeading('GitHub API Usage Tracker Summary')
       .addTable(makeSummaryTable(data))

tests/post-utils.test.mjs

@@ -27,15 +27,13 @@ describe('post utils', () => {
     });
 
     expect(table).toEqual([
-      { data: 'Bucket', header: true },
-      { data: 'Used', header: true },
-      { data: 'Remaining', header: true },
-      { data: 'core' },
-      { data: '3' },
-      { data: '10' },
-      { data: 'search' },
-      { data: '1' },
-      { data: '2' }
+      [
+        { data: 'Bucket', header: true },
+        { data: 'Used', header: true },
+        { data: 'Remaining', header: true }
+      ],
+      [{ data: 'core' }, { data: '3' }, { data: '10' }],
+      [{ data: 'search' }, { data: '1' }, { data: '2' }]
     ]);
   });
 });