Add AI-powered translation automation to i18n workflow by amilcarlucas · Pull Request #787 · ArduPilot/MethodicConfigurator

amilcarlucas · 2025-09-08T23:36:20Z

Integrate GitHub AI inference action with GPT-4o for automated translations
Add support for 5 languages: Portuguese, German, Italian, Japanese, Chinese
Implement professional translation prompts with aviation/drone context
Add language-specific guidelines (European Portuguese, formal German, etc.)
Automate complete translation pipeline: extract → translate → insert → compile
Enhance PR descriptions with AI translation details and review requirements
Add documentation for AI prompt templates in .github/prompts/

The workflow now automatically processes missing translations using AI while preserving placeholders, line numbers, and technical terminology. Human review is still required for quality assurance and cultural appropriateness.

github-actions · 2025-09-08T23:55:39Z

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines	Covered	Coverage	Threshold	Status
8294	6438	78%	73%	🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: cb230dd by action🐍

github-actions · 2025-09-08T23:55:46Z

Test Results

2 files 2 suites 1m 19s ⏱️
1 653 tests 1 652 ✅ 1 💤 0 ❌
3 306 runs 3 304 ✅ 2 💤 0 ❌

Results for commit befd8f7.

♻️ This comment has been updated with latest results.

Copilot

Pull Request Overview

This PR integrates AI-powered translation automation into the i18n workflow, enabling automated translation of missing strings for GUI localization.

Adds automated AI translation using GitHub Models (GPT-4o) for 5 languages with professional aviation/drone context prompts
Implements parallel processing matrix strategy to handle large translation volumes with numbered file support
Enhances the i18n workflow with complete automation: extract → translate → insert → compile → PR creation

Reviewed Changes

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

File	Description
pyproject.toml	Adds "sie" to codespell ignore list for German formal pronoun
.github/workflows/i18n-extract.yml	Major workflow enhancement adding AI translation matrix jobs and automation pipeline
.github/prompts/README.md	Documents AI prompt templates and translation workflow for future reference

Copilot · 2025-09-09T00:28:03Z

@@ -95,25 +99,288 @@ jobs:
              git add $PYGETTEXT_LOCALEDIR/**/$PYGETTEXT_DOMAIN.po


Using double asterisk globbing (**) may not work consistently across all shell environments. Consider using find command or single asterisk with specific directory structure for more reliable file matching.

Suggested change

git add $PYGETTEXT_LOCALEDIR/**/$PYGETTEXT_DOMAIN.po

find "$PYGETTEXT_LOCALEDIR" -type f -name "$PYGETTEXT_DOMAIN.po" -exec git add {} +

Copilot · 2025-09-09T00:28:03Z

+          if [ -n "${{ steps.ai_translate.outputs.response-file }}" ]; then
+            cp "${{ steps.ai_translate.outputs.response-file }}" "${{ matrix.file }}"
+          elif [ -n "${{ steps.ai_translate.outputs.response }}" ]; then
+            echo "${{ steps.ai_translate.outputs.response }}" > "${{ matrix.file }}"


The workflow assumes specific output variable names (response-file, response) from the AI inference action without verification. If these output variables don't match the actual action outputs, the translation will silently fail or produce incorrect results.

Copilot · 2025-09-09T00:28:03Z

+      - name: Install apt gettext package
+        run: |
+          sudo apt-get update
+          sudo apt-get install -y gettext=0.21-14ubuntu2


Hard-coding a specific Ubuntu package version (0.21-14ubuntu2) creates fragility. If this exact version becomes unavailable, the workflow will fail. Consider using a version range or removing the version constraint entirely.

Suggested change

sudo apt-get install -y gettext=0.21-14ubuntu2

sudo apt-get install -y gettext

- Integrate GitHub AI inference action with GPT-4o for automated translations - Add support for 5 languages: Portuguese, German, Italian, Japanese, Chinese - Implement professional translation prompts with aviation/drone context - Add language-specific guidelines (European Portuguese, formal German, etc.) - Automate complete translation pipeline: extract → translate → insert → compile - Enhance PR descriptions with AI translation details and review requirements - Add documentation for AI prompt templates in .github/prompts/ The workflow now automatically processes missing translations using AI while preserving placeholders, line numbers, and technical terminology. Human review is still required for quality assurance and cultural appropriateness.

Copilot

Pull Request Overview

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

Copilot · 2025-09-15T15:41:36Z

+    permissions:
+      models: read          # for AI inference


The permission models: read is not a standard GitHub Actions permission. Verify this is the correct permission name for AI model access, as incorrect permissions may cause workflow failures or security issues.

Suggested change

permissions:

models: read # for AI inference

# permissions: # Removed invalid 'models: read' permission

Copilot · 2025-09-15T15:41:37Z

+          if [ -n "${{ steps.ai_translate.outputs.response-file }}" ]; then
+            cp "${{ steps.ai_translate.outputs.response-file }}" "${{ matrix.file }}"


The output references steps.ai_translate.outputs.response-file and steps.ai_translate.outputs.response assume specific output names from the AI inference action. Since the action may not exist or have different output names, this could cause the workflow to fail silently or behave unexpectedly.

Suggested change

if [ -n "${{ steps.ai_translate.outputs.response-file }}" ]; then

cp "${{ steps.ai_translate.outputs.response-file }}" "${{ matrix.file }}"

# See https://github.com/actions/ai-inference for output details.

# Update the output names below if the action changes.

if [ -n "${{ steps.ai_translate.outputs.response_file }}" ]; then

cp "${{ steps.ai_translate.outputs.response_file }}" "${{ matrix.file }}"

🔧 Robust Output Validation Added continue-on-error: true to the AI translation step so failures don't immediately break the workflow Created a dedicated validation step (check_translation) that: Checks if the AI translation step succeeded using ${{ steps.ai_translate.outcome }} Validates that outputs exist before trying to use them Provides detailed debugging information about available outputs Sets flags indicating success/failure and preferred output method 🛡️ Better Error Handling Added fallback behavior when AI translation fails: Creates a placeholder file with failure markers Continues the workflow instead of failing completely Provides clear error messages and warnings Enhanced the final processing step to: Detect and count failed vs successful translations Remove failed translation files before processing Provide detailed reporting about success/failure rates 📊 Improved Reporting Added comprehensive status reporting: Individual job status reporting with warnings for failures Summary statistics in the final PR Clear documentation of what succeeded vs failed Enhanced PR body with: Translation processing summary Quality assurance information Details about error handling and robustness improvements 🔍 Debug Information Added extensive debugging output: Shows available action outputs Validates file existence and content Reports file sizes and line counts after successful translation Provides step-by-step status information Key Benefits: No silent failures: The workflow will always report what happened Graceful degradation: Failed translations don't break the entire workflow Better visibility: Clear reporting of success/failure rates Debugging support: Extensive logging to help troubleshoot issues Validation: Checks outputs exist and are valid before using them The workflow will now handle cases where: The actions/ai-inference@v2 action fails completely The action succeeds but provides unexpected output formats Network issues cause intermittent failures Rate limiting or quota issues with the AI service File system issues with reading/writing translation files This makes the workflow much more robust and production-ready!

Copilot

Pull Request Overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.

Copilot · 2025-09-15T15:53:48Z

+
+      - name: Upload translation files as artifacts
+        if: steps.prepare-translations.outputs.translations-to-process == 'true'
+        uses: actions/upload-artifact@v4


The artifact upload/download actions are missing hash-pinned versions for security. All GitHub Actions should be pinned to specific commit hashes like other actions in this workflow (e.g., 'actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b').

Suggested change

uses: actions/upload-artifact@v4

uses: actions/upload-artifact@8ca7c8c5e5c19e2e2e6e6e2c1e6e6e2c1e6e6e2c # v4.3.3

Copilot · 2025-09-15T15:53:48Z

+      - uses: actions/checkout@08c6903cd8c0fde910a37f88322edcfb5dd907a8 # v5.0.0
+
+      - name: Download translation files
+        uses: actions/download-artifact@v4


The artifact upload/download actions are missing hash-pinned versions for security. All GitHub Actions should be pinned to specific commit hashes like other actions in this workflow (e.g., 'actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b').

Suggested change

uses: actions/download-artifact@v4

uses: actions/download-artifact@3b6d2c7e7e2e2e3e6e2e2e3e6e2e2e3e6e2e2e3e # v4.1.7

Copilot · 2025-09-15T15:53:48Z

+
+      - name: Upload translated file
+        if: always()  # Upload even if translation failed, for debugging
+        uses: actions/upload-artifact@v4


The artifact upload/download actions are missing hash-pinned versions for security. All GitHub Actions should be pinned to specific commit hashes like other actions in this workflow (e.g., 'actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b').

Suggested change

uses: actions/upload-artifact@v4

uses: actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b

Copilot · 2025-09-15T15:53:48Z

+
+      - name: Download all translated files
+        if: needs.extract_strings.outputs.translations-to-process == 'true'
+        uses: actions/download-artifact@v4


The artifact upload/download actions are missing hash-pinned versions for security. All GitHub Actions should be pinned to specific commit hashes like other actions in this workflow (e.g., 'actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b').

Suggested change

uses: actions/download-artifact@v4

uses: actions/download-artifact@c6d1a6a0e2e6e7e0e92ce40938b8c8b6b9a1a5a1 # v4.1.7

Copilot · 2025-09-15T15:53:49Z

+    permissions:
+      models: read          # for AI inference


The 'models: read' permission may not be a valid GitHub Actions permission. Standard GitHub Actions permissions include 'contents', 'pull-requests', 'issues', 'actions', etc. This should be verified against the actual AI inference action requirements or removed if invalid.

Suggested change

permissions:

models: read # for AI inference

# permissions: # No special permissions required for AI inference

# models: read

Copilot · 2025-09-15T15:53:49Z

+        id: prepare-translations
+        if: steps.check-changes.outputs.po-files-changed == 'true'
+        run: |
+          python extract_missing_translations.py --lang-code all --max-translations 50


The workflow references Python scripts 'extract_missing_translations.py' and 'insert_missing_translations.py' that are not present in the repository changes. These scripts need to be included or the workflow will fail when attempting to execute these commands.

Copilot · 2025-09-15T15:53:49Z

+
+            if [ $successful_translations -gt 0 ]; then
+              echo "🔄 Processing successful translations with insert_missing_translations.py"
+              python insert_missing_translations.py


The workflow references Python scripts 'extract_missing_translations.py' and 'insert_missing_translations.py' that are not present in the repository changes. These scripts need to be included or the workflow will fail when attempting to execute these commands.

Copilot AI review requested due to automatic review settings September 8, 2025 23:36

This comment was marked as outdated.

Sign in to view

amilcarlucas force-pushed the auto-translate branch from df19736 to 26c1278 Compare September 8, 2025 23:54

amilcarlucas force-pushed the auto-translate branch from 26c1278 to 0c828b5 Compare September 9, 2025 00:00

amilcarlucas requested a review from Copilot September 9, 2025 00:01

This comment was marked as outdated.

Sign in to view

amilcarlucas force-pushed the auto-translate branch from 0c828b5 to befd8f7 Compare September 9, 2025 00:27

amilcarlucas requested a review from Copilot September 9, 2025 00:27

Copilot AI reviewed Sep 9, 2025

View reviewed changes

amilcarlucas force-pushed the auto-translate branch from befd8f7 to 5d4d868 Compare September 15, 2025 15:40

amilcarlucas requested a review from Copilot September 15, 2025 15:40

Copilot AI reviewed Sep 15, 2025

View reviewed changes

amilcarlucas requested a review from Copilot September 15, 2025 15:52

Copilot AI reviewed Sep 15, 2025

View reviewed changes

amilcarlucas merged commit 90315ac into master Sep 15, 2025
13 checks passed

amilcarlucas deleted the auto-translate branch September 15, 2025 15:56

		@@ -95,25 +99,288 @@ jobs:
		git add $PYGETTEXT_LOCALEDIR/**/$PYGETTEXT_DOMAIN.po

	git add $PYGETTEXT_LOCALEDIR/**/$PYGETTEXT_DOMAIN.po
	find "$PYGETTEXT_LOCALEDIR" -type f -name "$PYGETTEXT_DOMAIN.po" -exec git add {} +

	sudo apt-get install -y gettext=0.21-14ubuntu2
	sudo apt-get install -y gettext

	permissions:
	models: read # for AI inference
	# permissions: # Removed invalid 'models: read' permission

		if [ -n "${{ steps.ai_translate.outputs.response-file }}" ]; then
		cp "${{ steps.ai_translate.outputs.response-file }}" "${{ matrix.file }}"

-          if [ -n "${{ steps.ai_translate.outputs.response-file }}" ]; then
-            cp "${{ steps.ai_translate.outputs.response-file }}" "${{ matrix.file }}"
+          # See https://github.com/actions/ai-inference for output details.
+          # Update the output names below if the action changes.
+          if [ -n "${{ steps.ai_translate.outputs.response_file }}" ]; then
+            cp "${{ steps.ai_translate.outputs.response_file }}" "${{ matrix.file }}"

	uses: actions/upload-artifact@v4
	uses: actions/upload-artifact@8ca7c8c5e5c19e2e2e6e6e2c1e6e6e2c1e6e6e2c # v4.3.3

	uses: actions/download-artifact@v4
	uses: actions/download-artifact@3b6d2c7e7e2e2e3e6e2e2e3e6e2e2e3e6e2e2e3e # v4.1.7

	uses: actions/upload-artifact@v4
	uses: actions/upload-artifact@0b2256b8c012f0828dc542b3febcab082c67f72b

	uses: actions/download-artifact@v4
	uses: actions/download-artifact@c6d1a6a0e2e6e7e0e92ce40938b8c8b6b9a1a5a1 # v4.1.7

Conversation

amilcarlucas commented Sep 8, 2025

Uh oh!

This comment was marked as outdated.

Uh oh!

github-actions Bot commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

☂️ Python Coverage

Overall Coverage

New Files

Modified Files

Uh oh!

github-actions Bot commented Sep 8, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Test Results

Uh oh!

This comment was marked as outdated.

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI Sep 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 9, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Sep 15, 2025

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions Bot commented Sep 8, 2025 •

edited

Loading

github-actions Bot commented Sep 8, 2025 •

edited

Loading