feat: add translation orchestration to PR Fixer mode

[hannesrudolph]

Description

This PR adds minimal translation orchestration capabilities to the PR Fixer mode, allowing it to delegate translation updates to the translate mode when changes affect user-facing content.

Changes Made

• Added step 5 in the implementation phase to check for i18n file changes and delegate to translate mode using new_task tool
• Added translation completion criterion to ensure all required translations are handled
• Maintains existing non-interactive command usage (--force, --json flags)

Key Features

1. Translation Detection: Automatically detects when changes affect:

   • webview-ui/src/i18n/locales/en/*.json
   • src/i18n/locales/en/*.json
   • UI components
   • Announcements

2. Delegation Pattern: Uses the new_task tool to create a translation task, following the same pattern as the Orchestrator mode

3. Minimal Implementation: Avoided adding unnecessary complexity - integrated within existing workflow structure

Testing

The changes are minimal and focused on workflow orchestration. The PR Fixer mode will now:

• Continue to work as before for non-translation changes
• Delegate to translate mode when i18n files are modified
• Maintain non-interactive operation throughout

Notes

This implementation follows the principle of minimal changes - we added only what was necessary without creating new sections or files that would bloat the mode.

---

Important

Adds translation orchestration to PR Fixer mode, delegating i18n updates to translate mode for user-facing content changes.

• Behavior:
  • Adds translation orchestration to PR Fixer mode, delegating i18n updates to translate mode using new_task tool.
  • Detects changes in webview-ui/src/i18n/locales/en/*.json, src/i18n/locales/en/*.json, UI components, and announcements.
  • Ensures all required translations are completed before PR completion.
• Workflow:
  • Adds step in 1_workflow.xml to check for i18n changes and delegate to translate mode.
  • Updates completion_criteria in 1_workflow.xml to include translation completion.
• Best Practices:
  • Adds principle in 2_best_practices.xml to handle fork remotes correctly.
• Patterns:
  • Updates 3_common_patterns.xml with patterns for non-interactive rebase and determining push remote.
• Tool Usage:
  • Adds new_task (mode: translate) to 4_tool_usage.xml for handling translation tasks.
• Examples:
  • Adds example in 5_examples.xml for fixing PRs with translation requirements.

^{This description was created by for 6d5cbb6. You can customize this summary. It will automatically update as commits are pushed.}

[Copilot]

Pull Request Overview

This PR introduces translation orchestration into the PR Fixer mode, enabling non‐interactive delegation of translation updates for changes that affect user-facing content.

• Added a new step for detecting changes in i18n files and delegating translation tasks using the new_task tool.
• Updated tool usage and workflow documentation to enforce non-interactive commands.
• Revised common patterns to support non-interactive operations in conflicts detection, rebase, and commit workflows.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File	Description
.roo/rules-pr-fixer/5_examples.xml	Updated examples to illustrate the use of non-interactive commands and translation delegation.
.roo/rules-pr-fixer/4_tool_usage.xml	Introduced new priority for the new_task (mode: translate) tool and clarified non-interactive usage.
.roo/rules-pr-fixer/3_common_patterns.xml	Renamed and refined conflict detection and rebase patterns to support non-interactive workflows.
.roo/rules-pr-fixer/1_workflow.xml	Updated workflow steps to integrate non-interactive commands and translation orchestration.

Comments suppressed due to low confidence (2)

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

• [nitpick] The new pattern name 'detecting_conflicts' is somewhat ambiguous; consider renaming it to 'conflict_detection_dry_run' to better reflect its purpose.

  <pattern name="detecting_conflicts">

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

• Consider referencing the corresponding non-interactive conflict detection patterns (e.g., 'detecting_conflicts' or 'non_interactive_rebase') to clarify the recommended approach for managing conflicts.

        <step>For conflict resolution: Use automated detection first, then manual resolution via file editing, avoiding interactive rebase.</step>

[delve-auditor]

✅ No security or compliance issues detected. Reviewed everything up to 6d5cbb6.

Security Overview

• 🔎 Scanned files: 5 changed file(s)

Detected Code Changes

Change Type

Relevant files

Enhancement

► rules-pr-fixer/1_workflow.xml     Update implementation and validation steps ► rules-pr-fixer/2_best_practices.xml     Add fork remote handling practices ► rules-pr-fixer/3_common_patterns.xml     Add new command patterns for conflict detection and remote handling ► rules-pr-fixer/4_tool_usage.xml     Add translation tool guidance and enhanced command usage ► rules-pr-fixer/5_examples.xml     Update examples with translation requirements

Reply to this PR with @delve-auditor followed by a description of what change you want and we'll auto-submit a change to this PR to implement it.