fix: terminal QProcess-Signal-Disconnect + project_view a11y

B-012: _start_shell() trennt readyReadStandardOutput/Error/finished des alten QProcess vor Ersatz (try/except TypeError,RuntimeError); kill() nur bei state()!=NotRunning. Analog B-004 in output.py. project_view: objectName, Tooltips, accessibleName/Description fuer Ordner-/Aktualisieren-Buttons. 56/56 Tests gruen (3 neue TerminalStartShellTests).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: lukisch

CHANGELOG.md

@@ -9,6 +9,19 @@ Format basiert auf [Keep a Changelog](https://keepachangelog.com/de/1.1.0/).
 
 - `features/remote_editor.py`: SSH/SFTP-Verbindungen laden bekannte Hostkeys und
   lehnen unbekannte Hostkeys jetzt ab, statt sie automatisch zu akzeptieren.
+- `features/terminal.py` (B-012): `_start_shell()` trennte die Signale
+  (`readyReadStandardOutput`, `readyReadStandardError`, `finished`) des alten
+  `QProcess`-Objekts nicht, bevor es durch ein neues ersetzt wurde. Beim
+  Shell-Neustart konnte der alte Prozess nach `kill()` noch `_on_finished` oder
+  `_on_stdout` auslösen und so einen spuriösen „Shell beendet"-Eintrag in den
+  neuen Terminal-Output schreiben. Fix: Signale werden jetzt analog zu B-004
+  (`core/output.py`) getrennt; `kill()` wird nur noch bei `state() != NotRunning`
+  aufgerufen. 3 Regressionstests in `tests/test_terminal_encoding.py` ergänzt.
+- `features/project_view.py`: Die kompakte Sidebar im Projektbaum verlässt sich
+  für Filterfeld und Dateibaum nicht mehr nur auf Placeholder und Position.
+  `Ordner...`, `Aktualisieren`, das Filterfeld und der Dateibaum exponieren
+  jetzt sprechende Accessible Names, Descriptions und Tooltips; Regressionstest
+  in `tests/test_project_view.py` ergänzt.
 - `ui/main_window.py` (B-011): ProjectView blieb beim Öffnen einer Datei aus
   einem anderen Ordner auf dem ersten Root hängen. Der Projektbaum folgt jetzt
   auch bei späteren Dateiwechseln dem aktuellen Dateiverzeichnis; neuer

features/project_view.py

@@ -90,23 +90,42 @@ def _setup_ui(self):
 
         header.addStretch()
 
-        btn_open = QPushButton("Ordner...")
-        btn_open.setFixedWidth(60)
-        btn_open.setStyleSheet("font-size: 10px;")
-        btn_open.clicked.connect(self._open_folder_dialog)
-        header.addWidget(btn_open)
-
-        btn_refresh = QPushButton("Aktualisieren")
-        btn_refresh.setFixedWidth(80)
-        btn_refresh.setStyleSheet("font-size: 10px;")
-        btn_refresh.clicked.connect(self._refresh)
-        header.addWidget(btn_refresh)
+        self.btn_open = QPushButton("Ordner...")
+        self.btn_open.setObjectName("project_view_open_folder_button")
+        self.btn_open.setFixedWidth(60)
+        self.btn_open.setStyleSheet("font-size: 10px;")
+        self.btn_open.setToolTip("Projektordner auswählen")
+        self.btn_open.setAccessibleName("Projektordner auswählen")
+        self.btn_open.setAccessibleDescription(
+            "Öffnet einen Dialog, um den angezeigten Projektordner zu wechseln."
+        )
+        self.btn_open.clicked.connect(self._open_folder_dialog)
+        header.addWidget(self.btn_open)
+
+        self.btn_refresh = QPushButton("Aktualisieren")
+        self.btn_refresh.setObjectName("project_view_refresh_button")
+        self.btn_refresh.setFixedWidth(80)
+        self.btn_refresh.setStyleSheet("font-size: 10px;")
+        self.btn_refresh.setToolTip("Projektbaum neu laden")
+        self.btn_refresh.setAccessibleName("Projektbaum neu laden")
+        self.btn_refresh.setAccessibleDescription(
+            "Lädt den aktuell angezeigten Projektordner und seine Dateien neu."
+        )
+        self.btn_refresh.clicked.connect(self._refresh)
+        header.addWidget(self.btn_refresh)
 
         layout.addLayout(header)
 
         # Filter
         self.filter_input = QLineEdit()
+        self.filter_input.setObjectName("project_view_filter_input")
         self.filter_input.setPlaceholderText("Datei filtern...")
+        self.filter_input.setToolTip("Dateien und Ordner im Projektbaum filtern")
+        self.filter_input.setAccessibleName("Projektdateien filtern")
+        self.filter_input.setAccessibleDescription(
+            "Filtert Dateien und Ordner im Projektbaum. Ordner bleiben sichtbar,"
+            " wenn sie passende Kinder enthalten können."
+        )
         self.filter_input.setStyleSheet(
             "QLineEdit { background: #2a2a2a; color: #ccc; border: 1px solid #444; "
             "padding: 3px; margin: 2px 4px; font-size: 11px; }"
@@ -126,11 +145,17 @@ def _setup_ui(self):
 
         # Tree View
         self.tree = QTreeView()
+        self.tree.setObjectName("project_view_tree")
         self.tree.setModel(self.proxy)
         self.tree.setHeaderHidden(True)
         self.tree.setAnimated(True)
         self.tree.setIndentation(16)
         self.tree.setFont(QFont("Consolas", 10))
+        self.tree.setAccessibleName("Projektdateien")
+        self.tree.setAccessibleDescription(
+            "Dateibaum des aktuellen Projekts. Das Filterfeld grenzt die angezeigten"
+            " Dateien und Ordner ein."
+        )
         self.tree.setStyleSheet("""
             QTreeView {
                 background-color: #1e1e1e;

features/terminal.py

@@ -130,8 +130,21 @@ def _get_shell_command(self) -> tuple:
     def _start_shell(self):
         """Startet den Shell-Prozess."""
         if self.process:
-            self.process.kill()
-            self.process.waitForFinished(1000)
+            # B-012: Signale trennen bevor der Prozess ersetzt wird,
+            # sonst feuert _on_finished/_on_stdout des alten Prozesses auf
+            # den neuen Zustand (analog B-004 in core/output.py).
+            for sig in (
+                self.process.readyReadStandardOutput,
+                self.process.readyReadStandardError,
+                self.process.finished,
+            ):
+                try:
+                    sig.disconnect()
+                except (TypeError, RuntimeError):
+                    pass
+            if self.process.state() != QProcess.ProcessState.NotRunning:
+                self.process.kill()
+                self.process.waitForFinished(1000)
 
         self.process = QProcess(self)
         self.process.setWorkingDirectory(self.working_dir)

tests/test_project_view.py

@@ -81,3 +81,28 @@ def test_project_view_root_follows_current_file_directory(tmp_path):
         assert window.project_view._root_path == second_dir
     finally:
         window.close()
+
+
+def test_project_view_compact_controls_expose_accessible_context():
+    """Die kompakte Sidebar-UI muss Filter- und Baumkontext ohne Dauerlabels
+    für Screenreader und Tastaturhilfen bereitstellen."""
+    _ensure_app()
+    view = ProjectView()
+
+    try:
+        assert view.filter_input.toolTip() == "Dateien und Ordner im Projektbaum filtern"
+        assert view.filter_input.accessibleName() == "Projektdateien filtern"
+        assert "Ordner" in view.filter_input.accessibleDescription()
+
+        assert view.tree.accessibleName() == "Projektdateien"
+        assert "Dateibaum" in view.tree.accessibleDescription()
+
+        assert view.btn_open.toolTip() == "Projektordner auswählen"
+        assert view.btn_open.accessibleName() == "Projektordner auswählen"
+        assert "Dialog" in view.btn_open.accessibleDescription()
+
+        assert view.btn_refresh.toolTip() == "Projektbaum neu laden"
+        assert view.btn_refresh.accessibleName() == "Projektbaum neu laden"
+        assert "neu" in view.btn_refresh.accessibleDescription().lower()
+    finally:
+        view.close()

tests/test_terminal_encoding.py

@@ -68,5 +68,71 @@ def test_set_working_dir_uses_same_encoding_as_execute_command(self):
         widget.close()
 
 
+class TerminalStartShellTests(unittest.TestCase):
+    """Regressionstests für _start_shell() Signal-Verwaltung (B-012)."""
+
+    def setUp(self):
+        _ensure_app()
+
+    def _make_widget_no_shell(self) -> TerminalWidget:
+        """Erzeugt ein TerminalWidget ohne echten Shell-Start."""
+        with patch.object(TerminalWidget, "_start_shell", lambda self: None):
+            return TerminalWidget()
+
+    def test_start_shell_disconnects_old_process_signals(self):
+        """Regression (B-012): _start_shell() muss alle Signale des alten
+        Prozesses trennen, bevor er ersetzt wird — sonst feuert _on_finished
+        des alten Prozesses auf den neuen Zustand und erzeugt z. B. einen
+        spuriösen 'Shell beendet'-Eintrag im neuen Terminal-Output."""
+        from PySide6.QtCore import QProcess as RealQProcess
+
+        widget = self._make_widget_no_shell()
+
+        mock_old = MagicMock()
+        mock_old.state.return_value = RealQProcess.ProcessState.Running
+        widget.process = mock_old
+
+        with patch("features.terminal.QProcess") as mock_cls:
+            mock_cls.return_value = MagicMock()
+            widget._start_shell()
+
+        mock_old.readyReadStandardOutput.disconnect.assert_called_once_with()
+        mock_old.readyReadStandardError.disconnect.assert_called_once_with()
+        mock_old.finished.disconnect.assert_called_once_with()
+        mock_old.kill.assert_called_once()
+        widget.close()
+
+    def test_start_shell_no_kill_when_already_stopped(self):
+        """Ist der alte Prozess bereits beendet, darf kill() nicht aufgerufen werden."""
+        from PySide6.QtCore import QProcess as RealQProcess
+
+        widget = self._make_widget_no_shell()
+
+        mock_old = MagicMock()
+        mock_old.state.return_value = RealQProcess.ProcessState.NotRunning
+        widget.process = mock_old
+
+        with patch("features.terminal.QProcess") as mock_cls:
+            # Echten Enum weiterreichen, damit der state()-Vergleich in
+            # _start_shell korrekt ausgewertet werden kann.
+            mock_cls.ProcessState = RealQProcess.ProcessState
+            mock_cls.return_value = MagicMock()
+            widget._start_shell()
+
+        mock_old.kill.assert_not_called()
+        widget.close()
+
+    def test_start_shell_no_disconnect_when_no_prior_process(self):
+        """Ohne vorherigen Prozess darf kein disconnect() aufgerufen werden."""
+        widget = self._make_widget_no_shell()
+        widget.process = None
+
+        with patch("features.terminal.QProcess") as mock_cls:
+            mock_cls.return_value = MagicMock()
+            widget._start_shell()
+        # Kein AttributeError, kein Kill-Aufruf — Test besteht wenn kein Fehler
+        widget.close()
+
+
 if __name__ == "__main__":
     unittest.main()