Add operation-type labels to MRs for tracking statistics#452
Add operation-type labels to MRs for tracking statistics#452lbarcziova wants to merge 1 commit intopackit:mainfrom
Conversation
Label each MR with ymir_backport, ymir_rebase, or ymir_rebuild so we can query GitLab for per-operation statistics. Assisted-by: Claude Opus 4.6
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request introduces the ability to apply labels to merge requests generated by the backport, rebase, and rebuild agents. The commit_push_and_open_mr function was updated to handle label assignment through a new tool call. Feedback focuses on moving hardcoded label strings to a centralized constants file to improve code maintainability and prevent errors.
| ), | ||
| available_tools=gateway_tools, | ||
| commit_only=dry_run, | ||
| labels=["ymir_backport"], |
There was a problem hiding this comment.
The label "ymir_backport" is a hardcoded string used for tracking statistics. It should be defined as a constant in ymir/agents/constants.py to ensure consistency across the codebase and avoid potential typos in future updates.
| ), | ||
| available_tools=gateway_tools, | ||
| commit_only=dry_run, | ||
| labels=["ymir_rebase"], |
There was a problem hiding this comment.
The label "ymir_rebase" is a hardcoded string used for tracking statistics. It should be defined as a constant in ymir/agents/constants.py to ensure consistency across the codebase and avoid potential typos in future updates.
| available_tools=gateway_tools, | ||
| commit_only=dry_run, | ||
| allow_empty=is_empty_commit, | ||
| labels=["ymir_rebuild"], |
There was a problem hiding this comment.
The label "ymir_rebuild" is a hardcoded string used for tracking statistics. It should be defined as a constant in ymir/agents/constants.py to ensure consistency across the codebase and avoid potential typos in future updates.
1 participant
Label each MR with ymir_backport, ymir_rebase, or ymir_rebuild so we can query GitLab for per-operation statistics.
Assisted-by: Claude Opus 4.6