Skip to content

Sys-8303 co-pilot autofix #37

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
mmaharjan-ccdc merged 3 commits into from
May 7, 2026
Merged

Sys-8303 co-pilot autofix #37

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
d41fac3
SYS-8303 add exception for co-pilot autofix
mmaharjan-ccdc May 7, 2026
fffeab3
SYS-8303 update test with email address
mmaharjan-ccdc May 7, 2026
2b1bb8b
SYS-8303 rename copilot_autofix variable
mmaharjan-ccdc May 7, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,8 +14,8 @@ The commit will be flagged if it includes certain text files with:
* Certain C++ #include patterns and std::exception

The commit will also be flagged if the commit message does not include a Jira
ID (unless marked with NO_JIRA), or if the size of new or modiffied files
exceeds a threshold.
ID (unless marked with NO_JIRA or a Copilot Autofix co-author line), or if the
size of new or modified files exceeds a threshold.


# Github action
Expand Down
25 changes: 20 additions & 5 deletions main/githooks.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,6 +25,11 @@
LARGE_FILE_MARKER = 'LARGE_FILE'
# No jira marker in commit message
NO_JIRA_MARKER = 'NO_JIRA'
# Copilot Autofix co-author in commit message description
copilot_autofix_coauthor_pattern = re.compile(
r'^Co-authored-by:\s+.*<\d+\+Copilot@users\.noreply\.github\.com>$',
re.MULTILINE,
)
# A marker to represent it's a change we don't want to commit
DO_NOT_COMMIT = 'do not' + ' commit'
# Check file content if it has these extensions
Expand Down Expand Up @@ -854,11 +859,16 @@ def check_commit_msg(message, files, repo):
# Do not check for JIRA in opensource repo as we don't want to require external contributors to do this
return 0

if NO_JIRA_MARKER not in message:
if jira_id_pattern.search(message) is None:
_fail('Every commit should contain a Jira issue ID or the text '
f'{NO_JIRA_MARKER}')
return 1
if (
NO_JIRA_MARKER not in message
and copilot_autofix_coauthor_pattern.search(message) is None
and jira_id_pattern.search(message) is None
):
_fail(
'Every commit should contain a Jira issue ID, '
f'{NO_JIRA_MARKER}, or be a Copilot Autofix commit'
)
return 1

for filename in files:
size = Path(filename).stat().st_size / 1024**2
Expand Down Expand Up @@ -904,6 +914,11 @@ def _test(input, is_good=True):
_test('ABC-1234')
_test('Some changes for ABC-1234 ticket')
_test('Trivial change NO_JIRA')
_test(
'Trivial change\n\n'
'Co-authored-by: Copilot Autofix powered by AI '
'<175728472+Copilot@users.noreply.github.com>'
)
_test("Merge branch 'main' into my_branch")
_test("Merge branch 'branch_1' into branch_2")
_test("Merge branch 'jira_pyapi_123_abc' of github.com:ccdc-confidential/cpp-apps-main into jira_pyapi_123_abc")
Expand Down
Loading