fix: Bugs #9-1 bis #9-7 — Bugsweep Lauf #9 Korrekturen

#9-1: german_hints bereinigt (ASCII-Digraphe + false-positive 'ok' entfernt)
#9-2: _save_translations() OSError-Guard ergaenzt
#9-3: GERMAN_HINTS in manage_translations.py bereinigt
#9-4: JSON-Lesen mit try/except JSONDecodeError abgesichert
#9-5: JSON-Schreiben mit try/except OSError abgesichert
#9-6: setUp() in TranslatorIsGermanTests Hint-Liste bereinigt
#9-7: subprocess.run() timeout=30 in beiden run_cli()-Helpern ergaenzt

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Lukas Geiger

manage_translations.py

@@ -23,10 +23,10 @@
 ]
 
 GERMAN_HINTS = [
-    "datei", "filter", "fehler", "laden", "speichern",
-    "ansicht", "optionen", "zurueck", "anzeigen", "export",
-    "import", "einstellungen", "abbrechen", "hilfe", "bearbeiten",
-    "oeffnen", "schliessen", "start", "aktualisieren",
+    "datei", "fehler", "laden", "speichern",
+    "ansicht", "optionen", "anzeigen",
+    "einstellungen", "abbrechen", "hilfe", "bearbeiten",
+    "aktualisieren",
 ]
 
 
@@ -62,8 +62,11 @@ def manage_translations(source_dir="."):
     trans_file = os.path.join(source_dir, TRANSLATION_FILE)
 
     if os.path.exists(trans_file):
-        with open(trans_file, "r", encoding="utf-8") as f:
-            translations = json.load(f)
+        try:
+            with open(trans_file, "r", encoding="utf-8") as f:
+                translations = json.load(f)
+        except (OSError, json.JSONDecodeError):
+            translations = {}
     else:
         translations = {}
 
@@ -75,9 +78,12 @@ def manage_translations(source_dir="."):
             translations[s] = {"de": s, "en": ""}
             added.append(s)
 
-    os.makedirs(os.path.dirname(trans_file), exist_ok=True)
-    with open(trans_file, "w", encoding="utf-8") as f:
-        json.dump(translations, f, indent=2, ensure_ascii=False)
+    try:
+        os.makedirs(os.path.dirname(trans_file), exist_ok=True)
+        with open(trans_file, "w", encoding="utf-8") as f:
+            json.dump(translations, f, indent=2, ensure_ascii=False)
+    except OSError as e:
+        print(f"[FEHLER] translations.json konnte nicht geschrieben werden: {e}", file=sys.stderr)
 
     if added:
         print(f"[+] {len(added)} neue Eintraege hinzugefuegt:")

tests/test_cli.py

@@ -30,6 +30,7 @@ def run_cli(
             encoding="utf-8",
             env=env,
             check=False,
+            timeout=30,
         )
 
     def test_file_mode_returns_zero_for_clean_file(self) -> None:
@@ -215,6 +216,7 @@ def run_cli(self, *args: str, input_text: str | None = None, extra_env: dict | N
             encoding="utf-8",
             env=env,
             check=False,
+            timeout=30,
         )
 
     def test_report_text_contains_no_emojis(self) -> None:
@@ -545,8 +547,8 @@ def setUp(self):
         from translator import TranslationSystem
         self.tr = TranslationSystem.__new__(TranslationSystem)
         self.tr.german_hints = [
-            "datei", "bearbeiten", "ansicht", "hilfe", "oeffnen", "speichern",
-            "schliessen", "einstellungen", "abbrechen", "ok", "ja", "nein",
+            "datei", "bearbeiten", "ansicht", "hilfe", "speichern",
+            "einstellungen", "abbrechen", "ja", "nein",
         ]
 
     def test_english_words_not_classified_as_german(self) -> None:
@@ -569,5 +571,55 @@ def test_german_umlaut_words_classified_as_german(self) -> None:
                 )
 
 
+class GuiShortcutTests(unittest.TestCase):
+    def test_keyboard_shortcut_hint_mentions_primary_actions(self) -> None:
+        sys.path.insert(0, str(PROJECT_ROOT))
+        from MethodenAnalyser3 import _get_keyboard_shortcut_hint
+
+        hint = _get_keyboard_shortcut_hint()
+
+        self.assertIn("Alt+D", hint)
+        self.assertIn("Alt+P", hint)
+        self.assertIn("Alt+F", hint)
+        self.assertIn("F1", hint)
+
+    def test_register_gui_shortcuts_binds_and_invokes_callbacks(self) -> None:
+        sys.path.insert(0, str(PROJECT_ROOT))
+        from MethodenAnalyser3 import _register_gui_shortcuts
+
+        class FakeRoot:
+            def __init__(self) -> None:
+                self.bindings = {}
+
+            def bind_all(self, sequence, handler) -> None:
+                self.bindings[sequence] = handler
+
+        calls = []
+
+        def mark(name: str):
+            def _callback() -> None:
+                calls.append(name)
+
+            return _callback
+
+        root = FakeRoot()
+        _register_gui_shortcuts(
+            root,
+            analyze_file_cb=mark("file"),
+            info_cb=mark("info"),
+            auto_fix_cb=mark("fix"),
+            analyze_project_cb=mark("project"),
+        )
+
+        for sequence in ("<Alt-d>", "<Alt-D>", "<Alt-p>", "<Alt-P>", "<Alt-f>", "<Alt-F>", "<F1>"):
+            self.assertIn(sequence, root.bindings)
+
+        self.assertEqual(root.bindings["<Alt-d>"](None), "break")
+        self.assertEqual(root.bindings["<Alt-P>"](None), "break")
+        self.assertEqual(root.bindings["<Alt-f>"](None), "break")
+        self.assertEqual(root.bindings["<F1>"](None), "break")
+        self.assertEqual(calls, ["file", "project", "fix", "info"])
+
+
 if __name__ == "__main__":
     unittest.main()

translator.py

@@ -49,10 +49,10 @@ def __init__(self, default_lang: str = 'de', app_dir: Path = None):
         ]
 
         self.german_hints = [
-            "datei", "bearbeiten", "ansicht", "hilfe", "oeffnen", "speichern",
-            "schliessen", "einstellungen", "abbrechen", "ok", "ja", "nein",
-            "start", "stop", "pause", "fortsetzen", "laden", "aktualisieren",
-            "filter", "fehler", "export", "import", "optionen", "anzeigen",
+            "datei", "bearbeiten", "ansicht", "hilfe", "speichern",
+            "einstellungen", "abbrechen", "nein", "ja",
+            "fortsetzen", "laden", "aktualisieren",
+            "fehler", "optionen", "anzeigen",
         ]
 
         self.translations = {}
@@ -69,9 +69,12 @@ def _load_translations(self):
             self.translations = {}
 
     def _save_translations(self):
-        self.translations_file.parent.mkdir(parents=True, exist_ok=True)
-        with open(self.translations_file, 'w', encoding='utf-8') as f:
-            json.dump(self.translations, f, indent=2, ensure_ascii=False)
+        try:
+            self.translations_file.parent.mkdir(parents=True, exist_ok=True)
+            with open(self.translations_file, 'w', encoding='utf-8') as f:
+                json.dump(self.translations, f, indent=2, ensure_ascii=False)
+        except OSError:
+            pass
 
     def t(self, key: str) -> str:
         """