fix(is-production): crucial fix to detect main correctly as production

add some tests to prevent future fails..

Author: mumenthalers

packages/branch-utilities/src/lib/base.utils.spec.ts

@@ -1,4 +1,4 @@
-import { BranchInfo, getBranchInfo, parseBranchName } from './base.utils.js'
+import { BranchInfo, getBranchInfo, isProduction, isPullRequest, parseBranchName } from './base.utils.js'
 import { CustomGitHubContext, GithubActionEnv, GitHubContext } from './types/index.js'
 import { CustomScOverrideEnv } from './types/sc-override-env-var.type.js'
 
@@ -109,4 +109,31 @@ describe('base utils', () => {
       expect(() => parseBranchName('feat/copilot/fix-123')).toThrow()
     })
   })
+
+  describe('isProduction', () => {
+    test.each([
+      ['main', true],
+      ['master', true],
+      ['prod', false],
+      ['xx1', false],
+      ['pr1', false],
+      ['pr1-main', false],
+      ['xx1-master', false],
+    ])("when '%s' it returns %s", (stageName, expected) => {
+      expect(isProduction(stageName)).toBe(expected)
+    })
+  })
+
+  describe('isPullRequest', () => {
+    test.each([
+      ['pr1', true],
+      ['pr123456', true],
+      ['xx1', false],
+      ['xx123', false],
+      ['main', false],
+      ['master', false],
+    ])("when '%s' it returns %s", (stageName, expected) => {
+      expect(isPullRequest(stageName)).toBe(expected)
+    })
+  })
 })

packages/branch-utilities/src/lib/base.utils.ts

@@ -165,7 +165,7 @@ export function parseBranchName(branchName: string): { branchId: number; branchN
  * @return returns true if the stage is 'master' or 'main', false if not
  */
 export function isProduction(stageName: string): boolean {
-  return REGEX_MASTER.test(stageName) ?? REGEX_MAIN.test(stageName)
+  return REGEX_MASTER.test(stageName) || REGEX_MAIN.test(stageName)
 }
 
 /**