commitlint: add new rule

Add a rule to reject obvious words in the commit title.
nblockchain · Feb 14, 2023 · 62e1a2d · 62e1a2d
1 parent 1cf8fc7
commit 62e1a2d
Show file tree

Hide file tree

Showing 2 changed files with 46 additions and 0 deletions.
diff --git a/commitlint.config.ts b/commitlint.config.ts
@@ -47,6 +47,7 @@ module.exports = {
         "too-many-spaces": [RuleConfigSeverity.Error, "always"],
         "commit-hash-alone": [RuleConfigSeverity.Error, "always"],
         "title-uppercase": [RuleConfigSeverity.Error, "always"],
+        "reject-obvious-words": [RuleConfigSeverity.Error, "always"],
     },
     plugins: [
         // TODO (ideas for more rules):
@@ -81,6 +82,21 @@ module.exports = {
                     return Plugins.commitHashAlone(rawStr);
                 },
 
+                "reject-obvious-words": ({
+                    header,
+                    body,
+                }: {
+                    header: any;
+                    body: any;
+                }) => {
+                    let headerStr = Helpers.convertAnyToString(
+                        header,
+                        "header"
+                    );
+                    let bodyStr = Helpers.convertAnyToString(body, "header");
+                    return Plugins.rejectObviousWords(headerStr, bodyStr);
+                },
+
                 "empty-wip": ({ header }: { header: any }) => {
                     let headerUncastedStr = Helpers.convertAnyToString(
                         header,

diff --git a/commitlint/plugins.ts b/commitlint/plugins.ts
@@ -1,5 +1,6 @@
 import { abbr } from "./abbreviations";
 import { Helpers } from "./helpers";
+const obviousWords = ["change", "update", "modify"];
 
 export abstract class Plugins {
     public static bodyProse(rawStr: string) {
@@ -260,6 +261,35 @@ export abstract class Plugins {
         ];
     }
 
+    public static rejectObviousWords(
+        headerStr: string | null,
+        bodyStr: string | null
+    ) {
+        let offence = false;
+        let firstWordInTitle = "";
+
+        if (headerStr !== null) {
+            let colonFirstIndex = headerStr.indexOf(":");
+            let titleStartIndex = Math.max(0, colonFirstIndex + 1);
+            let title = headerStr
+                .substring(titleStartIndex, headerStr.length)
+                .trim();
+            let titleWords = title.split(" ");
+            firstWordInTitle = titleWords[0];
+
+            if (firstWordInTitle === "update") {
+                offence = titleWords.length > 1 && bodyStr === null;
+            } else {
+                offence = obviousWords.includes(firstWordInTitle);
+            }
+        }
+
+        return [
+            !offence,
+            `Please don't use obvious words such as ${firstWordInTitle} in the commit title`,
+        ];
+    }
+
     public static titleUppercase(headerStr: string) {
         let firstWord = headerStr.split(" ")[0];
         let offence =