feat: Allow passing Github title as arg to skip API request

src/lint.test.ts

@@ -1,4 +1,5 @@
 import { setFailed, warning, error, info } from '@actions/core';
+import { getOctokit } from '@actions/github';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
 import { lint } from './lint';
 
@@ -49,22 +50,46 @@ vi.mock('@actions/github', async importOriginal => {
   };
 });
 
-const mockArgs = ['TOKEN', './', './src/fixtures/commitlint.rules.js'];
+const mockArgs = [
+  'TOKEN',
+  './',
+  undefined,
+  './src/fixtures/commitlint.rules.js'
+];
 
 describe('Linter', () => {
   beforeEach(() => {
     vi.clearAllMocks();
     vi.restoreAllMocks();
   });
 
-  it('should find and log the PR title', async () => {
+  it('should skip the API request if PR title provided as an arg and log the PR title', async () => {
+    mocks.getOctokit.mockImplementation(vi.fn());
+
+    const mockArgsWithManualTitle = [
+      ...mockArgs.filter(arg => arg).slice(0, 2),
+      'chore(FOO-1234): hello i am a valid title passed manually',
+      ...mockArgs
+        .filter(arg => arg)
+        .slice(2, mockArgs.filter(arg => arg).length)
+    ];
+
+    await lint.apply(null, mockArgsWithManualTitle);
+
+    expect(getOctokit).not.toHaveBeenCalled();
+    expect(info).toHaveBeenCalledWith(
+      '🕵️ Found PR title in action args: "chore(FOO-1234): hello i am a valid title passed manually"'
+    );
+  });
+
+  it('should retrieve the pull request from the API if not passed as an arg and log the PR title', async () => {
     mocks.getOctokit.mockReturnValue({
       rest: {
         pulls: {
           get: vi.fn().mockReturnValue({
             data: {
               commits: 1,
-              title: 'feat(BAR-1234): hello i am a valid title'
+              title: 'feat(BAR-1234): hello i am a valid title from the API'
             }
           })
         }
@@ -73,8 +98,9 @@ describe('Linter', () => {
 
     await lint.apply(null, mockArgs);
 
+    expect(getOctokit).toHaveBeenCalledWith(mockArgs[0]);
     expect(info).toHaveBeenCalledWith(
-      '🕵️ Found PR title: "feat(BAR-1234): hello i am a valid title"'
+      '🕵️ Found PR title from Github API: "feat(BAR-1234): hello i am a valid title from the API"'
     );
   });
 
@@ -129,7 +155,7 @@ describe('Linter', () => {
 
     await lint.apply(
       null,
-      mockArgs.filter(arg => arg !== mockArgs[1])
+      mockArgs.filter(arg => arg && !arg.includes('commitlint.rules.js'))
     );
 
     expect(info).toHaveBeenCalledWith('📋 Checking PR title with commitlint');

src/lint.ts

@@ -7,7 +7,8 @@ import {
   logLintableScopeFound,
   logLintingPrTitle,
   logLintingPrTitleWithCustomRules,
-  logPrTitleFound,
+  logPrTitleFoundArg,
+  logPrTitleFoundApi,
   logScopeCheckSkipped
 } from './outputs/logs';
 import {
@@ -32,6 +33,7 @@ import { errorLinting } from './outputs/errors';
 const lint = async (
   githubToken?: string,
   githubWorkspace?: string,
+  prTitle?: string,
   rulesPath?: string,
   enforcedScopeTypes?: Array<string>,
   scopeRegex?: RegExp
@@ -40,27 +42,35 @@ const lint = async (
     return setFailedMissingToken();
   }
 
-  const octokit = github.getOctokit(githubToken);
+  let pullRequestTitle = prTitle;
 
-  if (!github.context.payload.pull_request) {
-    return setFailedPrNotFound();
-  }
+  if (pullRequestTitle) {
+    logPrTitleFoundArg(pullRequestTitle);
+  } else {
+    const octokit = github.getOctokit(githubToken);
 
-  const {
-    number: pullNumber,
-    base: {
-      user: { login: owner },
-      repo: { name: repo }
+    if (!github.context.payload.pull_request) {
+      return setFailedPrNotFound();
     }
-  } = github.context.payload.pull_request;
 
-  const { data: pullRequest } = await octokit.rest.pulls.get({
-    owner,
-    repo,
-    pull_number: pullNumber
-  });
+    const {
+      number: pullNumber,
+      base: {
+        user: { login: owner },
+        repo: { name: repo }
+      }
+    } = github.context.payload.pull_request;
 
-  logPrTitleFound(pullRequest.title);
+    const { data: pullRequest } = await octokit.rest.pulls.get({
+      owner,
+      repo,
+      pull_number: pullNumber
+    });
+
+    pullRequestTitle = pullRequest.title;
+
+    logPrTitleFoundApi(pullRequestTitle);
+  }
 
   const commitlintRules = await getLintRules(rulesPath, githubWorkspace);
 
@@ -78,13 +88,9 @@ const lint = async (
     conventionalChangelog: { parserOpts }
   } = await createPreset(null, null);
 
-  const lintOutput = await commitlint(
-    pullRequest.title,
-    commitlintRules.rules,
-    {
-      parserOpts
-    }
-  );
+  const lintOutput = await commitlint(pullRequestTitle, commitlintRules.rules, {
+    parserOpts
+  });
   lintOutput.warnings.forEach(warn => warnLinting(warn.message));
   lintOutput.errors.forEach(err => errorLinting(err.message));
 
@@ -94,7 +100,7 @@ const lint = async (
     return setFailedDoesNotMatchSpec();
   }
 
-  const { scope, type } = conventionalCommitsParser.sync(pullRequest.title);
+  const { scope, type } = conventionalCommitsParser.sync(pullRequestTitle);
 
   if (
     !enforcedScopeTypes ||

src/main.ts

@@ -6,12 +6,13 @@ try {
   const {
     githubToken,
     githubWorkspace,
+    prTitle,
     rulesPath,
     enforcedScopeTypes,
-    scopeRegex
+    scopeRegex,
   } = getActionConfig();
 
-  lint(githubToken, githubWorkspace, rulesPath, enforcedScopeTypes, scopeRegex);
+  lint(githubToken, githubWorkspace, prTitle, rulesPath, enforcedScopeTypes, scopeRegex);
 } catch (e) {
   core.setFailed(`Failed to run action with error: ${e}`);
 }

src/outputs/logs.test.ts

@@ -5,7 +5,8 @@ import {
   logLintableScopeFound,
   logLintingPrTitle,
   logLintingPrTitleWithCustomRules,
-  logPrTitleFound,
+  logPrTitleFoundArg,
+  logPrTitleFoundApi,
   logScopeCheckSkipped
 } from './logs';
 
@@ -24,10 +25,17 @@ describe('Log outputs', () => {
     vi.resetAllMocks();
   });
 
-  it('`logPrTitleFound` should pass the expected log to the output', () => {
-    logPrTitleFound(`fix(CDV-2812): Get with friends`);
+  it('`logPrTitleFoundArg` should pass the expected log to the output', () => {
+    logPrTitleFoundArg(`fix(CDV-2812): Get with friends`);
     expect(info).toHaveBeenCalledWith(
-      `🕵️ Found PR title: "fix(CDV-2812): Get with friends"`
+      `🕵️ Found PR title in action args: "fix(CDV-2812): Get with friends"`
+    );
+  });
+
+  it('`logPrTitleFoundApi` should pass the expected log to the output', () => {
+    logPrTitleFoundApi(`fix(CDV-2812): Get with friends`);
+    expect(info).toHaveBeenCalledWith(
+      `🕵️ Found PR title from Github API: "fix(CDV-2812): Get with friends"`
     );
   });
 

src/outputs/logs.ts

@@ -1,8 +1,11 @@
 import * as core from '@actions/core';
 import { Commit } from 'conventional-commits-parser';
 
-export const logPrTitleFound = (title: string) =>
-  core.info(`🕵️ Found PR title: "${title}"`);
+export const logPrTitleFoundArg = (title: string) =>
+  core.info(`🕵️ Found PR title in action args: "${title}"`);
+
+export const logPrTitleFoundApi = (title: string) =>
+  core.info(`🕵️ Found PR title from Github API: "${title}"`);
 
 export const logLintingPrTitle = () =>
   core.info(`📋 Checking PR title with commitlint`);

src/utils/config.test.ts

@@ -8,14 +8,16 @@ describe('Config utils', () => {
     process.env.INPUT_COMMITLINTRULESPATH = './commitlint.rules.js';
     process.env.GITHUB_TOKEN = 'asdf';
     process.env.GITHUB_WORKSPACE = './';
+    process.env.INPUT_PR_TITLE = 'chore(TEST-1234): Manually pass a PR title'
   });
 
   it('`getActionConfig` returns a valid config object with required values', () => {
     const config = getActionConfig();
     expect(config).toMatchObject({
       rulesPath: expect.any(String),
       githubToken: expect.any(String),
-      githubWorkspace: expect.any(String)
+      githubWorkspace: expect.any(String),
+      prTitle: expect.any(String)
     });
   });
 
@@ -27,7 +29,8 @@ describe('Config utils', () => {
       enforcedScopeTypes: expect.any(Array),
       rulesPath: expect.any(String),
       githubToken: expect.any(String),
-      githubWorkspace: expect.any(String)
+      githubWorkspace: expect.any(String),
+      prTitle: expect.any(String)
     });
     expect(config.enforcedScopeTypes).toEqual(['feat', 'fix']);
   });
@@ -40,7 +43,20 @@ describe('Config utils', () => {
       scopeRegex: expect.any(RegExp),
       rulesPath: expect.any(String),
       githubToken: expect.any(String),
-      githubWorkspace: expect.any(String)
+      githubWorkspace: expect.any(String),
+      prTitle: expect.any(String)
+    });
+  });
+
+  it('`getActionConfig` returns a valid config object with `undefined` PR title if not provided', () => {
+    delete process.env.INPUT_PR_TITLE;
+
+    const config = getActionConfig();
+    expect(config).toMatchObject({
+      rulesPath: expect.any(String),
+      githubToken: expect.any(String),
+      githubWorkspace: expect.any(String),
+      prTitle: undefined
     });
   });
 });

src/utils/config.ts

@@ -23,6 +23,7 @@ export const getActionConfig = () => {
   return {
     githubToken: process.env.GITHUB_TOKEN,
     githubWorkspace: process.env.GITHUB_WORKSPACE,
+    prTitle: process.env.INPUT_PR_TITLE,
     rulesPath: process.env.INPUT_COMMITLINTRULESPATH,
     ...(enforcedScopeTypes ? { enforcedScopeTypes } : {}),
     ...(scopeRegex ? { scopeRegex } : {})