Add Last-Translators to TX pull commit message (#88)

StanFromIreland · web-flow · commit 3620d73df8e7 · 2025-06-30T14:25:57.000+01:00
diff --git a/.github/workflows/update-lint-and-build.yml b/.github/workflows/update-lint-and-build.yml
@@ -56,7 +56,7 @@ jobs:
         run: >
           ! git diff -I'^"POT-Creation-Date: ' -I'^"Language-Team: ' -I'^# ' -I'^"Last-Translator: ' -I'^"Project-Id-Version: ' --exit-code && echo "SIGNIFICANT_CHANGES=1" >> "$GITHUB_ENV" || exit 0
       - run: git add .
-      - run: git commit -m 'Update translation from Transifex'
+      - run: git commit -m "$(python manage_translation.py generate_commit_msg)"
         if: env.SIGNIFICANT_CHANGES
       - name: Push commit
         uses: ad-m/github-push-action@master
diff --git a/manage_translation.py b/manage_translation.py
@@ -11,6 +11,7 @@
 #          files.
 # * recreate_tx_config: recreate configuration for all resources.
 # * warn_about_files_to_delete: lists files that are not available upstream
+# * generate_commit_msg: generates commit message with co-authors
 
 from argparse import ArgumentParser
 import os
@@ -19,13 +20,13 @@
 from difflib import SequenceMatcher
 from logging import info
 from pathlib import Path
-from subprocess import call
+from subprocess import call, run, CalledProcessError
 import sys
 from tempfile import TemporaryDirectory
 from typing import Self, Generator, Iterable
 from warnings import warn
 
-from polib import pofile
+from polib import pofile, POFile
 from transifex.api import transifex_api
 
 LANGUAGE = 'pl'
@@ -202,8 +203,62 @@ def language_switcher(entry: ResourceLanguageStatistics) -> bool:
     )
 
 
+def generate_commit_msg():
+    """Generate a commit message
+    Parses staged files and generates a commit message with Last-Translator's as
+    co-authors.
+    """
+    translators: set[str] = set()
+
+    result = run(
+        ['git', 'diff', '--cached', '--name-only', '--diff-filter=ACM'],
+        capture_output=True,
+        text=True,
+        check=True,
+    )
+    staged = [
+        filename for filename in result.stdout.splitlines() if filename.endswith('.po')
+    ]
+
+    for file in staged:
+        staged_file = run(
+            ['git', 'show', f':{file}'], capture_output=True, text=True, check=True
+        ).stdout
+        try:
+            old_file = run(
+                ['git', 'show', f'HEAD:{file}'],
+                capture_output=True,
+                text=True,
+                check=True,
+            ).stdout
+        except CalledProcessError:
+            old_file = ''
+
+        new_po = pofile(staged_file)
+        old_po = pofile(old_file) if old_file else POFile()
+        old_entries = {entry.msgid: entry.msgstr for entry in old_po}
+
+        for entry in new_po:
+            if entry.msgstr and (
+                entry.msgid not in old_entries
+                or old_entries[entry.msgid] != entry.msgstr
+            ):
+                translator = new_po.metadata.get('Last-Translator')
+                translator = translator.split(',')[0].strip()
+                if translator:
+                    translators.add(f'Co-Authored-By: {translator}')
+                break
+
+    print('Update translation from Transifex\n\n' + '\n'.join(translators))
+
+
 if __name__ == '__main__':
-    RUNNABLE_SCRIPTS = ('fetch', 'recreate_tx_config', 'warn_about_files_to_delete')
+    RUNNABLE_SCRIPTS = (
+        'fetch',
+        'recreate_tx_config',
+        'warn_about_files_to_delete',
+        'generate_commit_msg',
+    )
 
     parser = ArgumentParser()
     parser.add_argument('cmd', choices=RUNNABLE_SCRIPTS)
