feat: add disallowScopes option

README.md

@@ -55,6 +55,10 @@ The action works without configuration, however you can provide options for cust
             ui
           # Configure that a scope must always be provided.
           requireScope: true
+          # Configure which scopes are disallowed in a PR title.
+          # For instance, by setting the value below, `chore(release): ...` or `ci(e2e,release): ...` will be rejected
+          disallowScopes: |
+            release
           # Configure additional validation for the subject based on a regex.
           # This example ensures the subject doesn't start with an uppercase character.
           subjectPattern: ^(?![A-Z]).+$

action.yml

@@ -17,6 +17,9 @@ inputs:
   requireScope:
     description: "Configure that a scope must always be provided."
     required: false
+  disallowScopes:
+    description: 'Configure a list of disallowed scopes to be used on a PR.'
+    required: false
   subjectPattern:
     description: "Configure additional validation for the subject based on a regex. E.g. '^(?![A-Z]).+$' ensures the subject doesn't start with an uppercase character."
     required: false

src/parseConfig.js

@@ -16,6 +16,11 @@ module.exports = function parseConfig() {
     requireScope = ConfigParser.parseBoolean(process.env.INPUT_REQUIRESCOPE);
   }
 
+  let disallowScopes;
+  if (process.env.INPUT_DISALLOWSCOPES) {
+    disallowScopes = ConfigParser.parseEnum(process.env.INPUT_DISALLOWSCOPES);
+  }
+
   let subjectPattern;
   if (process.env.INPUT_SUBJECTPATTERN) {
     subjectPattern = ConfigParser.parseString(process.env.INPUT_SUBJECTPATTERN);
@@ -73,6 +78,7 @@ module.exports = function parseConfig() {
     types,
     scopes,
     requireScope,
+    disallowScopes,
     wip,
     subjectPattern,
     subjectPatternError,

src/validatePrTitle.js

@@ -11,6 +11,7 @@ module.exports = async function validatePrTitle(
     types,
     scopes,
     requireScope,
+    disallowScopes,
     subjectPattern,
     subjectPatternError,
     headerPattern,
@@ -46,6 +47,10 @@ module.exports = async function validatePrTitle(
     return scopes && !scopes.includes(s);
   }
 
+  function isDisallowedScope(s) {
+    return disallowScopes && disallowScopes.includes(s);
+  }
+
   if (!result.type) {
     throw new Error(
       `No release type found in pull request title "${prTitle}". Add a prefix to indicate what kind of release this pull request corresponds to. For reference, see https://www.conventionalcommits.org/\n\n${printAvailableTypes()}`
@@ -76,6 +81,7 @@ module.exports = async function validatePrTitle(
   const givenScopes = result.scope
     ? result.scope.split(',').map((scope) => scope.trim())
     : undefined;
+
   const unknownScopes = givenScopes ? givenScopes.filter(isUnknownScope) : [];
   if (scopes && unknownScopes.length > 0) {
     throw new Error(
@@ -89,6 +95,15 @@ module.exports = async function validatePrTitle(
     );
   }
 
+  const disallowedScopes = givenScopes
+    ? givenScopes.filter(isDisallowedScope)
+    : [];
+  if (disallowScopes && disallowedScopes.length > 0) {
+    throw new Error(
+      `Disallowed scope(s) are found: ${disallowScopes.join(', ')}`
+    );
+  }
+
   function throwSubjectPatternError(message) {
     if (subjectPatternError) {
       message = formatMessage(subjectPatternError, {

src/validatePrTitle.test.js

@@ -98,6 +98,68 @@ describe('defined scopes', () => {
       });
     });
 
+    describe('disallow scopes', () => {
+      it('passes when a single scope is provided, but not present in disallowScopes with one item', async () => {
+        await validatePrTitle('fix(core): Bar', {disallowScopes: ['release']});
+      });
+
+      it('passes when multiple scopes are provided, but not present in disallowScopes with one item', async () => {
+        await validatePrTitle('fix(core,e2e,bar): Bar', {
+          disallowScopes: ['release']
+        });
+      });
+
+      it('passes when a single scope is provided, but not present in disallowScopes with multiple items', async () => {
+        await validatePrTitle('fix(core): Bar', {
+          disallowScopes: ['release', 'test']
+        });
+      });
+
+      it('passes when multiple scopes are provided, but not present in disallowScopes with multiple items', async () => {
+        await validatePrTitle('fix(core,e2e,bar): Bar', {
+          disallowScopes: ['release', 'test']
+        });
+      });
+
+      it('throws when a single scope is provided and it is present in disallowScopes with one item', async () => {
+        await expect(
+          validatePrTitle('fix(release): Bar', {disallowScopes: ['release']})
+        ).rejects.toThrow('Disallowed scope(s) are found: release');
+      });
+
+      it('throws when a single scope is provided and it is present in disallowScopes with multiple item', async () => {
+        await expect(
+          validatePrTitle('fix(release): Bar', {
+            disallowScopes: ['release', 'test']
+          })
+        ).rejects.toThrow('Disallowed scope(s) are found: release');
+      });
+
+      it('throws when multiple scopes are provided and one of them is present in disallowScopes with one item ', async () => {
+        await expect(
+          validatePrTitle('fix(release,e2e): Bar', {
+            disallowScopes: ['release']
+          })
+        ).rejects.toThrow('Disallowed scope(s) are found: release');
+      });
+
+      it('throws when multiple scopes are provided and one of them is present in disallowScopes with multiple items ', async () => {
+        await expect(
+          validatePrTitle('fix(release,e2e): Bar', {
+            disallowScopes: ['release', 'test']
+          })
+        ).rejects.toThrow('Disallowed scope(s) are found: release');
+      });
+
+      it('throws when multiple scopes are provided and more than one of them are present in disallowScopes', async () => {
+        await expect(
+          validatePrTitle('fix(release,test): Bar', {
+            disallowScopes: ['release', 'test']
+          })
+        ).rejects.toThrow('Disallowed scope(s) are found: release, test');
+      });
+    });
+
     describe('scope allowlist not defined', () => {
       it('passes when a scope is provided', async () => {
         await validatePrTitle('fix(core): Bar', {