feat: Add environment variable branch support and enhanced pattern ma…

[jamengual]

This pull request introduces several updates to improve code robustness, handle edge cases, and enhance functionality. The most significant changes include better handling of branch references, improved regex detection in glob patterns, and enhanced error handling and fallback mechanisms in the Settings class.

Branch Reference Handling:

• index.js: Added logic to dynamically determine and validate branch references using environment variables, with fallback to the main branch if undefined.

Glob Pattern Matching:

• lib/glob.js: Introduced regex detection for patterns and fallback to glob matching if regex parsing fails. Added a helper method isRegexPattern to identify regex-specific characters.

Enhanced Error Handling and Fallbacks:

• lib/settings.js: Updated Settings class to handle optional chaining for installation_id and repository owner/repo fields, avoiding undefined errors. [1] [2] [3] [4]
• lib/settings.js: Added checks to ensure PR comment and check run updates are only performed in webhook context.
• lib/settings.js: Improved logic for creating PR comments by verifying the presence of pull requests in the webhook payload.

[Copilot]

Pull Request Overview

This PR adds support for configurable branch references via environment variables, improves glob pattern handling by detecting regex usage and falling back on glob matching, and hardens the Settings class with safer defaults and scoped webhook updates.

• Branch Reference Handling: derive and normalize ref from various environment variables, defaulting to main.
• Glob Matching: detect regex-like patterns in Glob.test, try a RegExp, and fallback to minimatch if parsing fails.
• Settings Fallbacks: use optional chaining for installation and repo IDs, guard PR comment/check-run code to webhook context.

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
index.js	Added logic to compute ref from env vars, normalize it, log, and pass to syncAllSettings
lib/glob.js	Added isRegexPattern helper; updated test to try regex then fallback to glob
lib/settings.js	Made installation_id and repo fields null-safe, scoped PR comment/check-run updates to webhook payload, and handled mixed owner formats

Comments suppressed due to low confidence (4)

lib/glob.js:11

• Consider adding unit tests for both regex paths and glob fallback, including invalid regex patterns that trigger the fallback branch.

if (this.isRegexPattern(this.pattern)) {

index.js:239

• Add tests for the branch-detection logic covering scenarios with different environment variable combinations and refs/ prefixes.

let ref = process.env.SAFE_SETTINGS_BRANCH || process.env.GITHUB_HEAD_REF || process.env.GITHUB_REF_NAME || process.env.GITHUB_REF

index.js:238

• [nitpick] It might help to explicitly log or document which branch is assumed when no ref env vars are set (e.g., defaulting to main).

// Use environment variable for branch reference, fallback to undefined (main branch)

lib/settings.js:172

• The variable payload is not defined in this scope. Use this.context.payload or destructure const { payload } = this.context before referencing it.

if (!payload || !payload.check_run) {

[jamengual]

@dmosyan do you have time to review this?