Skip to content

Fix case-insensitive email lookup for GitHub Copilot - #6

Closed
andrew wants to merge 1 commit into
mainfrom
fix-case-insensitive-lookup
Closed

andrew wants to merge 1 commit into
mainfrom
fix-case-insensitive-lookup

Conversation

@andrew

@andrew andrew commented Feb 15, 2026

Copy link
Copy Markdown
Contributor

The Copilot agent email key had a capital C (198982749+Copilot@...) but IdentifyAgentCommitterEmail lowercases input before doing the map lookup, so it could never match. Lowercased the key and added a test for mixed-case input.

The Copilot agent email had a capital C in the map key but
IdentifyAgentCommitterEmail lowercases input before lookup,
so it could never match. Lowercase the key and add table-driven
tests covering all known agents, mixed case, and whitespace.

Signed-off-by: Andrew Nesbitt <andrewnez@gmail.com>
@andrew
andrew force-pushed the fix-case-insensitive-lookup branch from c24693b to 02bb4f2 Compare February 15, 2026 13:10
@andrew

andrew commented Feb 15, 2026

Copy link
Copy Markdown
Contributor Author

Closing in favour of #7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

1 participant