Skip to content

Commit 02a46cd

Browse files
Copilotpelikhan
andcommitted
Fix mention resolution to include known authors from payload
Issue: resolveAllowedMentionsFromPayload was passing empty string to resolveMentionsLazily, which meant no mentions were extracted from text, so known authors were never added to the allowed list. This caused "@pelikhan" and other team members to not be recognized as allowed mentions. Fix: Create fake text with @mentions of known authors so they get processed by the mention resolution logic. This ensures known authors (PR author, issue author, assignees) are always included in the allowed mentions list along with collaborators. The mention resolution now correctly: 1. Extracts known authors from event payload (issue/PR author, assignees, etc.) 2. Creates mention text from known authors 3. Resolves mentions which checks against collaborators cache 4. Returns combined list of allowed mentions for output sanitization Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
1 parent 00e1266 commit 02a46cd

111 files changed

Lines changed: 27260 additions & 6600 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/ai-moderator.lock.yml

Lines changed: 2 additions & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/archie.lock.yml

Lines changed: 302 additions & 116 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/artifacts-summary.lock.yml

Lines changed: 243 additions & 53 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/audit-workflows.lock.yml

Lines changed: 243 additions & 53 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/blog-auditor.lock.yml

Lines changed: 243 additions & 53 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/brave.lock.yml

Lines changed: 302 additions & 116 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/breaking-change-checker.lock.yml

Lines changed: 243 additions & 53 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/changeset.lock.yml

Lines changed: 302 additions & 116 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/ci-coach.lock.yml

Lines changed: 243 additions & 53 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/ci-doctor.lock.yml

Lines changed: 243 additions & 53 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)