Merge pull request #478 from crazy-max/commit-date-request

commiter_date: fix github api request fallback

__mocks__/@actions/github.ts

@@ -205,5 +205,19 @@ export const context = {
 };
 
 export const getOctokit = jest.fn(() => ({
-  request: () => Promise.resolve({data: {committer: {date: '2024-11-13T13:42:28Z'}}})
+  rest: {
+    repos: {
+      getCommit: jest.fn(() =>
+        Promise.resolve({
+          data: {
+            commit: {
+              committer: {
+                date: '2024-11-13T13:42:28Z'
+              }
+            }
+          }
+        })
+      )
+    }
+  }
 }));

src/context.ts

@@ -119,11 +119,18 @@ async function getCommitDateFromWorkflow(sha: string, toolkit: Toolkit): Promise
   }
 
   // fallback to github api for commit date
-  const commit = await toolkit.github.octokit.request('GET /repos/{owner}/{repo}/commits/{commit_sha}', {
+  try {
-    commit_sha: sha,
+    const commit = await toolkit.github.octokit.rest.repos.getCommit({
-    owner: GitHub.context.repo.owner,
+      owner: GitHub.context.repo.owner,
-    repo: GitHub.context.repo.repo
+      repo: GitHub.context.repo.repo,
-  });
+      ref: sha
-
+    });
-  return new Date(commit.data.committer.date);
+    if (commit.data.commit.committer?.date) {
+      return new Date(commit.data.commit.committer.date);
+    }
+    throw new Error('Committer date not found');
+  } catch (error) {
+    core.debug(`Failed to get commit date from GitHub API: ${error.message}`);
+    return new Date();
+  }
 }