Skip to content

Commit e3c5ff3

Browse files
committed
fix: simplify git commit commands and remove explicit author configuration
1 parent d5e7ac2 commit e3c5ff3

File tree

4 files changed

+7
-11
lines changed

4 files changed

+7
-11
lines changed

.roo/rules-pr-fixer/1_workflow.xml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@
4646
<step>Fix failing tests by modifying test files or source code as needed.</step>
4747
<step>For conflict resolution: Use automated detection first, then manual resolution via file editing, avoiding interactive rebase.</step>
4848
<step>If changes affect user-facing content (i18n files, UI components, announcements), delegate translation updates using the new_task tool with translate mode.</step>
49-
<step>Commit changes using automated git commands with explicit author information.</step>
50-
<step>Push changes using 'git push --force-with-lease' to avoid interactive authentication prompts.</step>
49+
<step>Commit changes using git commands.</step>
50+
<step>Push changes using 'git push --force-with-lease' to safely update the pull request.</step>
5151
</steps>
5252
</phase>
5353

.roo/rules-pr-fixer/3_common_patterns.xml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -91,9 +91,7 @@
9191
<usage>Commit operations that work in automated environments.</usage>
9292
<template>
9393
<command tool="git">git add .</command>
94-
<command tool="git">git -c user.name="PR Fixer" -c user.email="pr-fixer@local" commit -m "<commit_message>"</command>
95-
<comment>Alternative with explicit author if needed</comment>
96-
<command tool="git">git commit --author="PR Fixer <pr-fixer@local>" -m "<commit_message>"</command>
94+
<command tool="git">git commit -m "<commit_message>"</command>
9795
</template>
9896
</pattern>
9997
</common_patterns>

.roo/rules-pr-fixer/4_tool_usage.xml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -58,8 +58,6 @@
5858

5959
<tool name="git operations">
6060
<best_practices>
61-
<practice>Use explicit git configuration for automated commits.</practice>
62-
<practice>Format: 'git -c user.name="PR Fixer" -c user.email="pr-fixer@local" commit'</practice>
6361
<practice>Use --force-with-lease for safer force pushing.</practice>
6462
<practice>Check git status programmatically with --porcelain flag.</practice>
6563
<practice>Use git rebase --dry-run to detect conflicts before attempting rebase.</practice>
@@ -69,7 +67,7 @@
6967
<step>If conflicts detected, resolve manually via file editing</step>
7068
<step>Use 'git status --porcelain' to identify conflicted files</step>
7169
<step>Edit files to resolve conflicts, removing conflict markers</step>
72-
<step>Use 'git add .' and commit with explicit author info</step>
70+
<step>Use 'git add .' and commit the resolved changes</step>
7371
</conflict_resolution>
7472
</tool>
7573

.roo/rules-pr-fixer/5_examples.xml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -161,10 +161,10 @@ Ensure consistency across all language files and maintain the same context and t
161161
<description>Commit all changes including translations with automated git configuration.</description>
162162
<tool_use>
163163
<execute_command>
164-
<command>git add . && git -c user.name="PR Fixer" -c user.email="pr-fixer@local" commit -m "fix: update button text and translations as requested in review"</command>
164+
<command>git add . && git commit -m "fix: update button text and translations as requested in review"</command>
165165
</execute_command>
166166
</tool_use>
167-
<analysis>All code changes and translation updates are now committed with explicit author info.</analysis>
167+
<analysis>All code changes and translation updates are now committed.</analysis>
168168
</step>
169169

170170
<step number="5">
@@ -174,7 +174,7 @@ Ensure consistency across all language files and maintain the same context and t
174174
<command>git push --force-with-lease</command>
175175
</execute_command>
176176
</tool_use>
177-
<analysis>Push changes safely, then check status programmatically.</analysis>
177+
<analysis>Push changes safely to update the pull request.</analysis>
178178
</step>
179179

180180
<step number="6">

0 commit comments

Comments
 (0)