Add min-body-length option

When set an error will be emitted if the body does not contain enough
characters. This is useful to require commit bodies with more
information, like impact, motivation, approach, testing results, new
behavior, etc. Default: no minimum. Suggested: at least 150.

Author: soulseekah

Diff for: action.yml

@@ -24,6 +24,10 @@ inputs:
     description: 'Maximum length of the commit subject line'
     required: false
     default: ''
+  min-body-length:
+    description: 'Minimum length of the body of the commit message'
+    required: false
+    default: ''
   max-body-line-length:
     description: 'Maximum length of a line in the body of the commit message'
     required: false

Diff for: dist/index.js

@@ -29029,6 +29029,7 @@ class Inputs {
         this.allowOneLiners = values.allowOneLiners;
         this.additionalVerbs = values.additionalVerbs;
         this.maxSubjectLength = values.maxSubjectLength;
+        this.minBodyLength = values.minBodyLength;
         this.maxBodyLineLength = values.maxBodyLineLength;
         this.enforceSignOff = values.enforceSignOff;
         this.validatePullRequestCommits = values.validatePullRequestCommits;
@@ -29078,7 +29079,7 @@ function parseIntOrInfinity(text) {
     return parseInt(text, 10);
 }
 function parseInputs(rawInputs) {
-    const { additionalVerbsInput = '', pathToAdditionalVerbsInput = '', allowOneLinersInput = '', maxSubjectLengthInput = '', maxBodyLineLengthInput = '', enforceSignOffInput = '', validatePullRequestCommitsInput = '', skipBodyCheckInput = '', ignoreMergeCommitsInput = '', ignorePatternsInput = '', } = rawInputs;
+    const { additionalVerbsInput = '', pathToAdditionalVerbsInput = '', allowOneLinersInput = '', maxSubjectLengthInput = '', minBodyLengthInput = '', maxBodyLineLengthInput = '', enforceSignOffInput = '', validatePullRequestCommitsInput = '', skipBodyCheckInput = '', ignoreMergeCommitsInput = '', ignorePatternsInput = '', } = rawInputs;
     const additionalVerbs = new Set();
     const hasAdditionalVerbsInput = additionalVerbsInput.length > 0;
     if (additionalVerbsInput) {
@@ -29110,6 +29111,13 @@ function parseInputs(rawInputs) {
         return new MaybeInputs(null, 'Unexpected value for max-subject-line-length. ' +
             `Expected a number or nothing, got ${maxSubjectLengthInput}`);
     }
+    const minBodyLength = !minBodyLengthInput
+        ? 0
+        : parseInt(minBodyLengthInput, 10);
+    if (Number.isNaN(minBodyLength)) {
+        return new MaybeInputs(null, 'Unexpected value for min-body-length. ' +
+            `Expected a number or nothing, got ${minBodyLengthInput}`);
+    }
     const maxBodyLineLength = !maxBodyLineLengthInput
         ? 72
         : parseIntOrInfinity(maxBodyLineLengthInput);
@@ -29158,6 +29166,7 @@ function parseInputs(rawInputs) {
         allowOneLiners,
         additionalVerbs,
         maxSubjectLength,
+        minBodyLength,
         maxBodyLineLength,
         enforceSignOff,
         validatePullRequestCommits,
@@ -29380,6 +29389,11 @@ function checkBody(subject, bodyLines, inputs) {
         errors.push('Unexpected empty body');
         return errors;
     }
+    // Minimum character body length
+    if (inputs.minBodyLength && bodyLines.join('').length < inputs.minBodyLength) {
+        errors.push(`Body must contain at least ${inputs.minBodyLength} characters.`);
+        return errors;
+    }
     for (const [i, line] of bodyLines.entries()) {
         if (urlLineRe.test(line) || linkDefinitionRe.test(line)) {
             continue;
@@ -29574,6 +29588,9 @@ async function runWithExceptions() {
     const maxSubjectLengthInput = core.getInput('max-subject-line-length', {
         required: false,
     });
+    const minBodyLengthInput = core.getInput('min-body-length', {
+        required: false,
+    });
     const maxBodyLineLengthInput = core.getInput('max-body-line-length', {
         required: false,
     });
@@ -29597,6 +29614,7 @@ async function runWithExceptions() {
         pathToAdditionalVerbsInput,
         allowOneLinersInput,
         maxSubjectLengthInput,
+        minBodyLengthInput,
         maxBodyLineLengthInput,
         enforceSignOffInput,
         validatePullRequestCommitsInput,

Diff for: src/__tests__/inspection.test.ts

@@ -212,6 +212,7 @@ it(
       allowOneLiners: false,
       additionalVerbs: new Set<string>('table'),
       maxSubjectLength: 50,
+      minBodyLength: 0,
       maxBodyLineLength: 72,
       enforceSignOff: false,
       validatePullRequestCommits: false,

Diff for: src/input.ts

@@ -6,6 +6,7 @@ interface InputValues {
   allowOneLiners: boolean;
   additionalVerbs: Set<string>;
   maxSubjectLength: number;
+  minBodyLength: number;
   maxBodyLineLength: number;
   enforceSignOff: boolean;
   validatePullRequestCommits: boolean;
@@ -19,6 +20,7 @@ export class Inputs implements InputValues {
   public pathToAdditionalVerbs: string;
   public allowOneLiners: boolean;
   public maxSubjectLength: number;
+  public minBodyLength: number;
   public maxBodyLineLength: number;
   public skipBodyCheck: boolean;
   public validatePullRequestCommits: boolean;
@@ -38,6 +40,7 @@ export class Inputs implements InputValues {
     this.allowOneLiners = values.allowOneLiners;
     this.additionalVerbs = values.additionalVerbs;
     this.maxSubjectLength = values.maxSubjectLength;
+    this.minBodyLength = values.minBodyLength;
     this.maxBodyLineLength = values.maxBodyLineLength;
     this.enforceSignOff = values.enforceSignOff;
     this.validatePullRequestCommits = values.validatePullRequestCommits;
@@ -82,6 +85,7 @@ interface RawInputs {
   pathToAdditionalVerbsInput?: string;
   allowOneLinersInput?: string;
   maxSubjectLengthInput?: string;
+  minBodyLengthInput?: string;
   maxBodyLineLengthInput?: string;
   enforceSignOffInput?: string;
   validatePullRequestCommitsInput?: string;
@@ -118,6 +122,7 @@ export function parseInputs(rawInputs: RawInputs): MaybeInputs {
     pathToAdditionalVerbsInput = '',
     allowOneLinersInput = '',
     maxSubjectLengthInput = '',
+    minBodyLengthInput = '',
     maxBodyLineLengthInput = '',
     enforceSignOffInput = '',
     validatePullRequestCommitsInput = '',
@@ -176,6 +181,18 @@ export function parseInputs(rawInputs: RawInputs): MaybeInputs {
     );
   }
 
+  const minBodyLength: number = !minBodyLengthInput
+    ? 0
+    : parseInt(minBodyLengthInput, 10);
+
+  if (Number.isNaN(minBodyLength)) {
+    return new MaybeInputs(
+      null,
+      'Unexpected value for min-body-length. ' +
+        `Expected a number or nothing, got ${minBodyLengthInput}`,
+    );
+  }
+
   const maxBodyLineLength: number = !maxBodyLineLengthInput
     ? 72
     : parseIntOrInfinity(maxBodyLineLengthInput);
@@ -253,6 +270,7 @@ export function parseInputs(rawInputs: RawInputs): MaybeInputs {
       allowOneLiners,
       additionalVerbs,
       maxSubjectLength,
+      minBodyLength,
       maxBodyLineLength,
       enforceSignOff,
       validatePullRequestCommits,

Diff for: src/inspection.ts

@@ -228,6 +228,14 @@ function checkBody(
     return errors;
   }
 
+  // Minimum character body length
+  if (inputs.minBodyLength && bodyLines.join('').length < inputs.minBodyLength) {
+    errors.push(
+      `Body must contain at least ${inputs.minBodyLength} characters.`,
+    );
+    return errors;
+  }
+
   for (const [i, line] of bodyLines.entries()) {
     if (urlLineRe.test(line) || linkDefinitionRe.test(line)) {
       continue;

Diff for: src/mainImpl.ts

@@ -25,6 +25,10 @@ async function runWithExceptions(): Promise<void> {
     required: false,
   });
 
+  const minBodyLengthInput = core.getInput('min-body-length', {
+    required: false,
+  });
+
   const maxBodyLineLengthInput = core.getInput('max-body-line-length', {
     required: false,
   });
@@ -57,6 +61,7 @@ async function runWithExceptions(): Promise<void> {
     pathToAdditionalVerbsInput,
     allowOneLinersInput,
     maxSubjectLengthInput,
+    minBodyLengthInput,
     maxBodyLineLengthInput,
     enforceSignOffInput,
     validatePullRequestCommitsInput,