Create a fresh copy of the base repository config

lib/glob.js

@@ -4,13 +4,20 @@ class Glob {
 
     // If not a glob pattern then just match the string.
     if (!this.glob.includes('*')) {
-      this.regexp = new RegExp(`.*${this.glob}.*`, 'u')
+      // Escape special regex characters for non-glob patterns
+      const escapedGlob = this.escapeRegExp(this.glob)
+      this.regexp = new RegExp(`.*${escapedGlob}.*`, 'u')
       return
     }
     this.regexptText = this.globize(this.glob)
     this.regexp = new RegExp(`^${this.regexptText}$`, 'u')
   }
 
+  // Helper method to escape special regex characters
+  escapeRegExp (string) {
+    return string.replace(/[.*+?^${}()|[\]\\]/g, '\\$&') // $& means the whole matched string
+  }
+
   globize (glob) {
     return glob
       .replace(/\\/g, '\\\\') // escape backslashes
@@ -41,4 +48,4 @@ class Glob {
     return s.replaceAll(this.regexp, replacement)
   }
 }
-module.exports = Glob
+module.exports = Glob

lib/settings.js

@@ -298,9 +298,10 @@ ${this.results.reduce((x, y) => {
 
   async updateRepos(repo) {
     this.subOrgConfigs = this.subOrgConfigs || await this.getSubOrgConfigs()
-    let repoConfig = this.config.repository
+    // Create a fresh copy of the base repository config
+    let repoConfig = this.config.repository ? Object.assign({}, this.config.repository) : {}
     if (repoConfig) {
-      repoConfig = Object.assign(repoConfig, { name: repo.repo, org: repo.owner })
+      repoConfig = Object.assign({}, repoConfig, { name: repo.repo, org: repo.owner })
     }
 
     const subOrgConfig = this.getSubOrgConfig(repo.repo)

test/unit/lib/glob.test.ts

@@ -74,5 +74,29 @@ describe('glob test', function () {
     str = 'java/csrf-protection-disabled'
     expect(str.search(pattern)>=0).toBeTruthy()
   })
-
+
+  test('Test Glob with special regex characters', () => {
+    // Test for repository names with a period (.)
+    let pattern = new Glob('.sentry')
+    let str = '.sentry'
+    expect(str.search(pattern)>=0).toBeTruthy()
+    str = 'some-repo'
+    expect(str.search(pattern)>=0).toBeFalsy()
+    str = 'other-sentry-repo'
+    expect(str.search(pattern)>=0).toBeFalsy()
+
+    // Test for other special regex characters
+    pattern = new Glob('repo+name')
+    str = 'repo+name'
+    expect(str.search(pattern)>=0).toBeTruthy()
+    str = 'reponame'
+    expect(str.search(pattern)>=0).toBeFalsy()
+
+    pattern = new Glob('repo[1-3]')
+    str = 'repo[1-3]'
+    expect(str.search(pattern)>=0).toBeTruthy()
+    str = 'repo1'
+    expect(str.search(pattern)>=0).toBeFalsy()
+  })
+
 })