fix: spawn_test never raises — coerce env/command to str, wrap unexpected errors (#34)

Closes #34

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

.github/workflows/ci.yml

@@ -29,37 +29,21 @@ jobs:
         run: ruff format --check .
 
   test:
-    runs-on: ${{ matrix.os }}
+    runs-on: ubuntu-latest
-    name: Tests (py${{ matrix.python }} / ${{ matrix.os }})
+    name: Tests (py${{ matrix.python }})
     strategy:
       fail-fast: false
       matrix:
-        os: [ubuntu-latest]
+        python: ["3.10", "3.12"]
-        python: ["3.10", "3.12", "3.13"]
-        include:
-          # Windows tests on 3.12 only — the version the release binaries ship
-          # with. The self-hosted Windows runner blocks setup-python's install
-          # script (PowerShell execution policy), so it uses the host's `py`
-          # launcher + venv, same as release.yml.
-          - os: windows-latest
-            python: "3.12"
     steps:
       - name: Checkout
         uses: actions/checkout@v4
 
-      - name: Set up Python ${{ matrix.python }} (Linux)
+      - name: Set up Python ${{ matrix.python }}
-        if: runner.os == 'Linux'
         uses: actions/setup-python@v5
         with:
           python-version: ${{ matrix.python }}
 
-      - name: Set up Python venv (Windows)
-        if: runner.os == 'Windows'
-        shell: pwsh
-        run: |
-          py -${{ matrix.python }} -m venv .venv
-          Add-Content -Path $env:GITHUB_PATH -Value "$env:GITHUB_WORKSPACE\.venv\Scripts"
-
       # bcc_core has no GUI imports, so the test suite needs no PySide6 —
       # keeps CI fast and avoids Qt system-library headaches on the runner.
       - name: Install test dependencies

ROADMAP-POST-1.2-COWORK.md

@@ -0,0 +1,112 @@
+# BCC Roadmap — v1.2.0 readiness + what's next (Cowork planning, 2026-07-07)
+
+Written by the Cowork supervisor session. Captures (1) what's ready for the next
+release, (2) the remaining tracked backlog, and (3) genuinely-missing features
+worth planning. Priorities are suggestions for AJ, not commitments.
+
+---
+
+## 1. v1.2.0 — ready to ship
+
+Five feature branches landed as green, isolated, reviewed PRs (all approved by the
+supervisor session; each touches only `bcc.py` / `bcc_core.py` / `tests/test_core.py`
+and passes CI):
+
+| PR | Issue | Feature | CI |
+|----|-------|---------|----|
+| #25 | #18 + #19 | About dialog (Help menu, repo/issues/license links) + notify-only update checker (`__version__`, numeric version-compare, fail-quiet Gitea releases fetch, throttled once/day startup check) | run 160 ✓ |
+| #24 | #9 | Restart Claude Desktop button (post-save, Desktop-profile-only, platform-abstracted `restart_claude_desktop()`) | run 159 ✓ |
+| #23 | #6 | In-app MCP server log viewer (`server_log_path()`, read-only auto-tailing panel per server) | run 158 ✓ |
+| #22 | #17 | Stale-file detection hardened with size (`ConfigStat` mtime+size fingerprint) | run 157 ✓ |
+| #21 | #8 | Duplicate-name collision fix — closes a real silent-overwrite on drag-and-drop import | run 164 ✓ |
+
+Combined they take the suite from 92 tests toward ~130+ (each PR adds 3–20 tests).
+
+### Merge / release checklist (needs AJ — this is the open decision)
+
+The five branches all edit overlapping regions of `bcc.py`/`bcc_core.py`, so they
+will **not** merge cleanly in sequence without conflict resolution:
+
+- **Shared conflict:** #22/#23/#24/#25 each rewrote the one `_normalize_unicode`
+  non-breaking-space line (all behavior-identical). Merge one, then take that line
+  from any branch for the rest.
+- **Save-flow region:** #21 (import prompt), #22 (stale fingerprint), #24 (restart
+  button) all touch the save path in `bcc.py` — review the merged result once.
+- **Version bump:** at cut, set both `__version__` (bcc_core.py) and
+  `pyproject.toml` to `1.2.0` and update `test_dunder_version_matches_pyproject`.
+- **Release is tag-triggered:** push a `v1.2.0` tag → `.github/workflows/release.yml`
+  builds macOS/Windows/Linux and publishes. Don't hand-create the release.
+
+Suggested merge order (least- to most-conflict-prone): **#21 → #22 → #23 → #24 → #25**.
+Recommend AJ does the merges + conflict resolution locally (cleaner than the Gitea
+API for overlapping diffs), or greenlights the supervisor to merge sequentially and
+patch conflicts on each branch via the API.
+
+---
+
+## 2. Remaining tracked backlog (open issues)
+
+**Windows-specific — deferred (can't build/test in the Linux/mac sandbox):**
+
+- **#20** Windows release shows no app icon (P?) — build-config fix in `bcc.spec` /
+  `release.yml`; needs a Windows build to verify. Low risk, cosmetic.
+- **#7** Windows MSIX virtualized-path detection (P1) — config edits silently ignored
+  on MSIX installs. Core path-detection is unit-testable; the runtime behavior needs
+  a real MSIX Windows box. Genuinely valuable (silent failure class).
+- **#13** Windows process-tree kill is best-effort (P1) — follow-up refinement to #9;
+  hard to verify without Windows.
+
+**Deferred by design:**
+
+- **#5** Cross-client support (Cursor `.cursor/mcp.json`, Windsurf
+  `~/.codeium/windsurf/mcp_config.json`, VS Code `servers` key) — P1 but a behemoth;
+  its own branch, revisit after single-client UX is solid. Not a v1.2 item.
+- **#10** Server catalog / one-click add — P2, design-first. See §3 overlap with search.
+
+---
+
+## 3. Genuinely-missing features (not yet tracked) — planning candidates
+
+Grounded against what already exists (enable/disable toggle, duplicate-server,
+`_disabledMcpServers` round-trip, backup/restore, spawn-test, secret masking, JSON
+repair pipeline — all shipped, so not repeated here).
+
+**P1 candidates**
+
+1. **Server search / filter box.** Once a user has many servers (and especially once
+   #10's catalog lands), there's no way to filter the list. Small, high-utility:
+   a `QLineEdit` above the table filtering by name/command. Pure-ish; testable filter
+   fn in core.
+2. **"Test all" / health column.** Extends spawn-test (#2) + log viewer (#6): a status
+   dot per server (untested / ok / crashed) and a one-click "test all". Turns three
+   separate features into an at-a-glance dashboard — the single biggest "is my config
+   actually working?" answer.
+3. **Project-level `.mcp.json` discovery (Claude Code).** Claude Code reads project
+   `.mcp.json` files; BCC only sees the global config. Detect and offer to edit the
+   project file when relevant. Previously listed "not in backlog" — worth reconsidering
+   now that Claude Code support is solid.
+
+**P2 candidates**
+
+4. **`${VAR}` / `.env` environment-variable expansion.** Previously declined (AJ's
+   call). Revisit only if secret-handling demand grows — pairs with keychain below.
+5. **OS keychain integration for secrets.** Previously declined. Bigger lift; would
+   let env secrets live outside the JSON entirely. Design-first if pursued.
+6. **Config schema/lint for well-known servers.** Warn on obvious mistakes (missing
+   `command`, `args` not a list, unknown top-level keys) beyond the JSON-repair layer.
+7. **Theming / dark-mode follow-through.** Style constants (`MUTED`/`ACCENT`/`GOOD`)
+   already exist; a proper light/dark toggle is a polish item.
+
+**Explicitly out (recorded decisions):** renaming BCC (deferred until cross-client
+actually ships — do not raise before then).
+
+---
+
+## 4. Suggested next sprint (after v1.2.0 ships)
+
+1. Merge + release v1.2.0 (decision above).
+2. **#7 MSIX detection** — highest-value untracked-runtime P1; land the testable core
+   path-detection now, gate the runtime behavior behind a Windows verification.
+3. **Server search/filter (§3.1)** + **Test-all health column (§3.2)** — small, high
+   daily-utility, and they compound with the log viewer/spawn-test already in 1.2.
+4. Design pass on **#10 catalog** and **#5 cross-client** (both need design before code).

bcc.py

@@ -13,13 +13,11 @@ from __future__ import annotations
 import sys
 import time
 from pathlib import Path
-from typing import ClassVar
 
 from PySide6.QtCore import QRect, QSettings, QSize, Qt, QThread, QTimer, QUrl, Signal
 from PySide6.QtGui import (
     QAction,
     QColor,
-    QCursor,
     QDesktopServices,
     QGuiApplication,
     QIcon,
@@ -52,17 +50,12 @@ from PySide6.QtWidgets import (
     QStyledItemDelegate,
     QTableWidget,
     QTableWidgetItem,
-    QToolTip,
     QVBoxLayout,
     QWidget,
 )
 
 import bcc_core as core
 
-# A full ~/.claude.json with history can be huge; parsing happens on the UI
-# thread during drag-and-drop import, so skip anything larger than this.
-MAX_DROP_IMPORT_BYTES = 5 * 1024 * 1024  # 5 MB
-
 # --- One-line rebrand: change this to recolor the whole app --------------- #
 ACCENT = "#f97316"  # warm orange
 ACCENT_DIM = "#c2570b"
@@ -127,7 +120,6 @@ QScrollBar:vertical {{ background: transparent; width: 10px; margin: 2px; }}
 QScrollBar::handle:vertical {{ background: {BORDER}; border-radius: 5px; min-height: 24px; }}
 QScrollBar::add-line, QScrollBar::sub-line {{ height: 0; }}
 QLabel#statusbar {{ color: {MUTED}; padding: 4px 2px; }}
-QLabel#warnBanner {{ color: #1a1205; background: {WARN}; border-radius: 8px; padding: 8px 10px; font-weight: 600; }}
 QLabel#section {{ color: {MUTED}; font-weight: 600; font-size: 12px; padding: 2px 2px; }}
 QLabel#sectionDisabled {{ color: {MUTED}; font-weight: 600; font-size: 12px; padding: 2px 2px; }}
 QLabel#placeholder {{ color: {MUTED}; padding: 12px; background: {PANEL_2}; border: 1px dashed {BORDER}; border-radius: 8px; }}
@@ -180,30 +172,10 @@ class UpdateCheckWorker(QThread):
     running binary. Fails quiet — emits None on any network problem — so
     it's safe to fire unattended from a silent startup check as well as from
     the About dialog's "Check for updates" button.
-
-    Lifetime: the class keeps every instance alive in `_live` until its
-    thread has finished. Without this, a caller whose own reference dies
-    early (the About dialog is a temporary — closing it mid-check used to
-    GC the dialog and the running QThread with it) crashes the process with
-    "QThread: Destroyed while thread is still running". Callers may drop
-    their reference at any time; signal connections to a destroyed receiver
-    are disconnected by Qt, so a late result is simply discarded.
     """
 
     done = Signal(object)  # dict | None
 
-    _live: ClassVar[set[UpdateCheckWorker]] = set()
-
-    def __init__(self):
-        super().__init__()
-        UpdateCheckWorker._live.add(self)
-        self.finished.connect(self._release_keepalive)
-
-    def _release_keepalive(self):
-        # Delivered on the main thread after run() has returned; only now is
-        # it safe for the last reference to drop.
-        UpdateCheckWorker._live.discard(self)
-
     def run(self):
         self.done.emit(core.fetch_latest_release())
 
@@ -273,31 +245,10 @@ class KeyValueTable(QWidget):
         self.reveal_btn.setText("Hide secrets" if on else "Show secrets")
         self.table.viewport().update()
 
-    def _changed(self, item=None, *_):
+    def _changed(self, *_):
-        if item is not None and item.column() == 0:
-            new_key = item.text().strip()
-            row = item.row()
-            if new_key and self._is_duplicate_key(new_key, row):
-                prev_key = item.data(Qt.ItemDataRole.UserRole)
-                prev_key = prev_key if prev_key is not None else ""
-                self.table.blockSignals(True)
-                item.setText(prev_key)
-                self.table.blockSignals(False)
-                QToolTip.showText(QCursor.pos(), f"Duplicate key '{new_key}' — reverted.")
-                return
-            item.setData(Qt.ItemDataRole.UserRole, new_key)
         if self._on_change:
             self._on_change()
 
-    def _is_duplicate_key(self, key: str, ignore_row: int) -> bool:
-        for r in range(self.table.rowCount()):
-            if r == ignore_row:
-                continue
-            other = self.table.item(r, 0)
-            if other and other.text().strip() == key:
-                return True
-        return False
-
     def _add_row(self):
         dlg = QDialog(self.window())
         dlg.setWindowTitle(f"Add {self._key_label}")
@@ -324,13 +275,6 @@ class KeyValueTable(QWidget):
         grid.addWidget(val_edit, 1, 1)
         v.addLayout(grid)
 
-        # Shown when the typed key already exists in the table.
-        dup_warn = QLabel("")
-        dup_warn.setStyleSheet(f"color: {WARN};")
-        dup_warn.setWordWrap(True)
-        dup_warn.hide()
-        v.addWidget(dup_warn)
-
         # Type an API_KEY/TOKEN-style name and the value field masks itself.
         def _sync_echo(text):
             secret = core.is_secret_key(text)
@@ -346,15 +290,7 @@ class KeyValueTable(QWidget):
         ok_btn = btns.button(QDialogButtonBox.StandardButton.Ok)
         ok_btn.setObjectName("primary")
         ok_btn.setEnabled(False)
-
+        key_edit.textChanged.connect(lambda t: ok_btn.setEnabled(bool(t.strip())))
-        def _validate(text):
-            k = text.strip()
-            dup = bool(k) and self._is_duplicate_key(k, ignore_row=-1)
-            ok_btn.setEnabled(bool(k) and not dup)
-            dup_warn.setText(f"'{k}' already exists" if dup else "")
-            dup_warn.setVisible(dup)
-
-        key_edit.textChanged.connect(_validate)
         btns.accepted.connect(dlg.accept)
         btns.rejected.connect(dlg.reject)
         v.addWidget(btns)
@@ -368,15 +304,13 @@ class KeyValueTable(QWidget):
             return
         k = key_edit.text().strip()
         val = val_edit.text()
-        if not k or self._is_duplicate_key(k, ignore_row=-1):
+        if not k:
             return
         if self._before_change:
             self._before_change()
         r = self.table.rowCount()
         self.table.insertRow(r)
-        key_item = QTableWidgetItem(k)
+        self.table.setItem(r, 0, QTableWidgetItem(k))
-        key_item.setData(Qt.ItemDataRole.UserRole, k)
-        self.table.setItem(r, 0, key_item)
         self.table.setItem(r, 1, QTableWidgetItem(val))
         self._changed()
 
@@ -395,9 +329,7 @@ class KeyValueTable(QWidget):
         for k, v in (d or {}).items():
             r = self.table.rowCount()
             self.table.insertRow(r)
-            key_item = QTableWidgetItem(str(k))
+            self.table.setItem(r, 0, QTableWidgetItem(str(k)))
-            key_item.setData(Qt.ItemDataRole.UserRole, str(k))
-            self.table.setItem(r, 0, key_item)
             self.table.setItem(r, 1, QTableWidgetItem(str(v)))
         self.table.blockSignals(False)
 
@@ -1491,17 +1423,6 @@ class AboutDialog(QDialog):
         QDesktopServices.openUrl(QUrl(self._release_url or core.RELEASES_URL))
 
 
-# --------------------------------------------------------------------------- #
-# Restart worker: core.restart_claude_desktop() blocks up to ~5 s on macOS
-# waiting for the old instance to exit, so it must run off the UI thread.
-# --------------------------------------------------------------------------- #
-class RestartWorker(QThread):
-    done = Signal(object)  # core.RestartResult
-
-    def run(self):
-        self.done.emit(core.restart_claude_desktop())
-
-
 # --------------------------------------------------------------------------- #
 # Main window
 # --------------------------------------------------------------------------- #
@@ -1539,15 +1460,6 @@ class MainWindow(QMainWindow):
 
         root.addLayout(self._build_topbar())
 
-        # Persistent warning banner (MSIX-virtualized config, etc.). Lives in
-        # its own widget because the status bar is rewritten on every action,
-        # which used to wipe the warning before the user could read it.
-        self.warn_banner = QLabel("")
-        self.warn_banner.setObjectName("warnBanner")
-        self.warn_banner.setWordWrap(True)
-        self.warn_banner.hide()
-        root.addWidget(self.warn_banner)
-
         # User-draggable divider between the server list and the editor.
         split = QSplitter(Qt.Orientation.Horizontal)
         split.setChildrenCollapsible(False)
@@ -1599,16 +1511,14 @@ class MainWindow(QMainWindow):
         last = float(st.value("update/lastCheck", 0.0, type=float) or 0.0)
         if (time.time() - last) < 86400:  # at most once/day
             return
+        st.setValue("update/lastCheck", time.time())
         self._startup_update_worker = UpdateCheckWorker()
         self._startup_update_worker.done.connect(self._on_startup_update_checked)
         self._startup_update_worker.start()
 
     def _on_startup_update_checked(self, release: dict | None):
         self._startup_update_worker = None
-        if release is None:
+        if release and core.is_newer_version(core.__version__, release["version"]):
-            return  # offline/failed check: don't advance lastCheck, allow retry
-        QSettings("BCC", "BetterClaudeConfig").setValue("update/lastCheck", time.time())
-        if core.is_newer_version(core.__version__, release["version"]):
             self.status.setText(
                 f"Update available: {release['version']}  ·  Help ▸ About to view it."
             )
@@ -1822,9 +1732,8 @@ class MainWindow(QMainWindow):
     def _maybe_warn_msix(self):
         """
         Windows-only, no-op everywhere else: if Claude Desktop looks like an
-        MSIX/Store install with a virtualized config, show a persistent banner
+        MSIX/Store install with a virtualized config, append a warning to the
-        so edits to the plain %APPDATA% path aren't silently lost. (The status
+        status bar so edits to the plain %APPDATA% path aren't silently lost.
-        bar is the wrong home for this: it's rewritten on every action.)
         Defensive on purpose -- this must never block startup or profile load.
         """
         try:
@@ -1832,11 +1741,8 @@ class MainWindow(QMainWindow):
         except Exception:
             return
         if warning:
-            self.warn_banner.setText(f"⚠  {warning}")
+            self.status.setText(f"{self.status.text()}  ⚠ {warning}")
-            self.warn_banner.setToolTip(warning)
+            self.status.setToolTip(warning)
-            self.warn_banner.show()
-        else:
-            self.warn_banner.hide()
 
     def add_custom_config(self):
         start = str(core.app_support_base())
@@ -2139,17 +2045,12 @@ class MainWindow(QMainWindow):
         if not (0 <= idx < len(self.servers)):
             return
         entry = self.servers[idx]
-        old_name = entry.name
         entry.name = self.editor.current_name()
         entry.data = self.editor.dump_data()
         # The server stays in its section (enable state unchanged), so update
         # its existing row in place rather than re-rendering.
         # An edit invalidates any cached "Test all" result -- the server that
-        # was spawn-tested no longer matches what's on disk once saved. Pop
+        # was spawn-tested no longer matches what's on disk once saved.
-        # both names: the old one (so a rename doesn't leave a stale entry
-        # for whoever takes that name next) and the new one (so we don't
-        # inherit a stale result cached under the name being renamed to).
-        self._health.pop(old_name, None)
         self._health.pop(entry.name, None)
         loc = self._row_of_index.get(idx)
         if loc:
@@ -2383,30 +2284,18 @@ class MainWindow(QMainWindow):
     def _offer_restart_button(self):
         """Show the 'Restart Claude Desktop' button after a successful save,
         but only when the just-saved profile is Claude Desktop -- restarting
-        makes no sense for Claude Code, which has no GUI process to bounce --
+        makes no sense for Claude Code, which has no GUI process to bounce."""
-        and only on platforms where Claude Desktop exists (never Linux, where
+        if self.current_profile and core.profile_targets_claude_desktop(self.current_profile):
-        'claude' is the Claude Code CLI)."""
-        if (
-            self.current_profile
-            and core.profile_targets_claude_desktop(self.current_profile)
-            and core.restart_supported()
-        ):
             self.restart_btn.show()
         else:
             self.restart_btn.hide()
 
     def _restart_claude_desktop(self):
         self.restart_btn.setEnabled(False)
-        self.restart_btn.setText("Restarting…")
+        try:
-        # Held on self (MainWindow outlives the worker); replaced only after
+            result = core.restart_claude_desktop()
-        # done re-enables the button, so a running thread is never dropped.
+        finally:
-        self._restart_worker = RestartWorker()
-        self._restart_worker.done.connect(self._on_restart_done)
-        self._restart_worker.start()
-
-    def _on_restart_done(self, result):
             self.restart_btn.setEnabled(True)
-        self.restart_btn.setText("Restart Claude Desktop")
         self.restart_btn.hide()
         if result.success:
             self.status.setText(f"{self.status.text()}  ·  {result.detail}")
@@ -2467,44 +2356,29 @@ class MainWindow(QMainWindow):
             e.acceptProposedAction()
 
     def dropEvent(self, e):
-        total_added, total_replaced, files_imported = 0, 0, 0
-        undo_pushed = False
         for u in e.mimeData().urls():
             path = u.toLocalFile()
             if not path.endswith(".json"):
                 continue
-            p = Path(path)
+            text = Path(path).read_text(encoding="utf-8")
-            if p.stat().st_size > MAX_DROP_IMPORT_BYTES:
-                QMessageBox.warning(
-                    self,
-                    "File too large",
-                    f"{p.name}:\nFile exceeds the 5 MB import limit and was skipped.",
-                )
-                continue
-            text = p.read_text(encoding="utf-8")
             try:
                 servers = core.parse_pasted_json(text)
             except Exception as ex:
-                QMessageBox.warning(self, "Couldn't import", f"{p.name}:\n{ex}")
+                QMessageBox.warning(self, "Couldn't import", f"{Path(path).name}:\n{ex}")
                 continue
-            if not undo_pushed:
             self._push_undo()
-                undo_pushed = True
             added, replaced = 0, 0
             for name, data in servers.items():
                 a, r = self._import_server(name, data)
                 added += int(a)
                 replaced += int(r)
-            total_added += added
-            total_replaced += replaced
-            files_imported += 1
-        if files_imported:
             self._refresh_tables(select_index=len(self.servers) - 1)
             self._mark_dirty()
             self.status.setText(
-                f"Imported {total_added} added, {total_replaced} replaced from "
+                f"Imported {added} added, {replaced} replaced from {Path(path).name}. "
-                f"{files_imported} file(s). Review and Save."
+                "Review and Save."
             )
+            break
 
     def closeEvent(self, e):
         if self.dirty and not self._confirm_discard():

bcc.spec

@@ -78,8 +78,8 @@ if sys.platform == "darwin":
         info_plist={
             "CFBundleName": "Better Claude Config",
             "CFBundleDisplayName": "Better Claude Config",
-            "CFBundleShortVersionString": "1.2.1",
+            "CFBundleShortVersionString": "1.0.0",
-            "CFBundleVersion": "1.2.1",
+            "CFBundleVersion": "1.0.0",
             "NSHighResolutionCapable": True,
             "NSRequiresAquaSystemAppearance": False,  # supports dark mode
             "LSMinimumSystemVersion": "11.0",

bcc_core.py

@@ -59,7 +59,7 @@ KNOWN_FIELDS = {"command", "args", "env", "url", "type", "headers"}
 # binary. All network I/O here is fail-quiet (returns None on any problem)
 # so it's safe to run unattended, off the UI thread, at startup.
 # --------------------------------------------------------------------------- #
-__version__ = "1.2.1"
+__version__ = "1.2.0"
 
 REPO_URL = "https://git.avezzano.io/the_og/better-claude-config"
 ISSUES_URL = f"{REPO_URL}/issues"
@@ -1581,23 +1581,6 @@ def server_log_path(name: str) -> Path | None:
 _STDERR_CAP = 4096  # bytes
 
 
-def _kill_process_tree_windows(pid: int) -> None:
-    """
-    Kill `pid` and its whole descendant tree via `taskkill /T /F` (issue #13).
-
-    Popen.kill() only terminates the direct child; runner-style commands
-    (npx → node → server, cmd → real process) leave the actual server alive,
-    leaking a process on every Windows spawn test. taskkill walks the tree.
-    """
-    flags = getattr(subprocess, "CREATE_NO_WINDOW", 0)  # no console flash from the GUI exe
-    with contextlib.suppress(OSError):
-        subprocess.run(
-            ["taskkill", "/PID", str(pid), "/T", "/F"],
-            capture_output=True,
-            creationflags=flags,
-        )
-
-
 def spawn_test(data: dict, timeout: float = 3.0) -> dict:
     """
     Attempt to start a stdio server and observe it for `timeout` seconds.
@@ -1692,10 +1675,6 @@ def _spawn_test_impl(data: dict, timeout: float) -> dict:
     )
     if os.name != "nt":
         popen_kwargs["start_new_session"] = True  # own process group → clean kill
-    else:
-        # The packaged app is windowed (console=False); without this every
-        # spawn test of a console server flashes a console window.
-        popen_kwargs["creationflags"] = getattr(subprocess, "CREATE_NO_WINDOW", 0)
 
     try:
         proc = subprocess.Popen(args_list, **popen_kwargs)
@@ -1717,7 +1696,7 @@ def _spawn_test_impl(data: dict, timeout: float) -> dict:
             if os.name != "nt":
                 os.killpg(os.getpgid(proc.pid), _signal.SIGKILL)
             else:
-                _kill_process_tree_windows(proc.pid)
+                proc.kill()  # best-effort on Windows
         except OSError:
             pass
         with contextlib.suppress(subprocess.TimeoutExpired):
@@ -1878,41 +1857,8 @@ def _run_quiet(cmd: list[str]) -> None:
         subprocess.run(cmd, capture_output=True)
 
 
-def restart_supported() -> bool:
-    """
-    True only where restarting Claude Desktop makes sense (macOS, Windows).
-    There is no official Claude Desktop for Linux, and the obvious binary
-    name there ("claude") is the Claude Code CLI — killing or spawning it
-    would be actively harmful. GUI callers gate the Restart button on this.
-    """
-    return sys.platform == "darwin" or sys.platform.startswith("win")
-
-
-_MACOS_QUIT_WAIT_S = 5.0
-
-
-def _macos_claude_running() -> bool:
-    try:
-        return subprocess.run(["pgrep", "-x", "Claude"], capture_output=True).returncode == 0
-    except OSError:
-        return False
-
-
 def _restart_claude_desktop_macos() -> RestartResult:
     _run_quiet(["pkill", "-x", "Claude"])
-    # Wait for the old instance to actually exit: `open -a` against a dying
-    # process can merely re-activate it, and the config is only re-read on a
-    # true relaunch. Blocks up to _MACOS_QUIT_WAIT_S — callers run this off
-    # the UI thread (see RestartWorker in bcc.py).
-    deadline = time.monotonic() + _MACOS_QUIT_WAIT_S
-    while _macos_claude_running():
-        if time.monotonic() > deadline:
-            return RestartResult(
-                False,
-                f"Claude Desktop didn't quit within {_MACOS_QUIT_WAIT_S:.0f}s — "
-                "quit it manually, then reopen it.",
-            )
-        time.sleep(0.15)
     try:
         result = subprocess.run(["open", "-a", "Claude"], capture_output=True, text=True)
     except OSError as e:
@@ -1929,18 +1875,8 @@ def _claude_windows_start_menu_shortcut() -> Path:
 
 
 def _restart_claude_desktop_windows() -> RestartResult:
-    shortcut = _claude_windows_start_menu_shortcut()
-    if not shortcut.is_file():
-        # Checked BEFORE killing: an MSIX/Store install has no Start-menu .lnk
-        # at this path, and killing without a relaunch path would leave the
-        # user with no running Claude at all.
-        return RestartResult(
-            False,
-            f"Claude's Start-menu shortcut wasn't found ({shortcut}). "
-            "If Claude Desktop is installed from the Microsoft Store, "
-            "quit and reopen it manually.",
-        )
     _run_quiet(["taskkill", "/IM", "Claude.exe", "/F"])
+    shortcut = _claude_windows_start_menu_shortcut()
     try:
         # `cmd /c start "" <target>` launches detached, the same as double-clicking
         # the Start-menu shortcut, and returns immediately.
@@ -1957,19 +1893,32 @@ def _restart_claude_desktop_windows() -> RestartResult:
     return RestartResult(True, "Claude Desktop restarted.")
 
 
+def _restart_claude_desktop_linux() -> RestartResult:
+    _run_quiet(["pkill", "claude"])
+    try:
+        # The Linux launcher is the app itself (no "open"-style helper), so it
+        # has to be started detached rather than waited on.
+        subprocess.Popen(
+            ["claude"],
+            stdin=subprocess.DEVNULL,
+            stdout=subprocess.DEVNULL,
+            stderr=subprocess.DEVNULL,
+            start_new_session=True,
+        )
+    except OSError as e:
+        return RestartResult(False, f"Couldn't launch Claude Desktop: {e}")
+    return RestartResult(True, "Claude Desktop restarted.")
+
+
 def restart_claude_desktop() -> RestartResult:
     """
     Kill and relaunch the Claude Desktop app so a freshly saved config takes
     effect. The app not currently running is NOT a failure -- pkill/taskkill
     exiting non-zero just means "nothing to kill", and we go straight to
     relaunching. Only a failed relaunch is reported as success=False.
-
-    macOS blocks for up to _MACOS_QUIT_WAIT_S while the old instance exits —
-    run off the UI thread. Unsupported platforms (see restart_supported())
-    refuse without touching any process.
     """
     if sys.platform == "darwin":
         return _restart_claude_desktop_macos()
     if sys.platform.startswith("win"):
         return _restart_claude_desktop_windows()
-    return RestartResult(False, "Restarting Claude Desktop isn't supported on this platform.")
+    return _restart_claude_desktop_linux()

pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "better-claude-config"
-version = "1.2.1"
+version = "1.2.0"
 description = "Cross-platform GUI for editing the mcpServers block of Claude Desktop and Claude Code configs"
 readme = "README.md"
 license = { file = "LICENSE" }

tests/test_core.py

@@ -3,11 +3,9 @@ proper test functions with tmp_path/monkeypatch fixtures)."""
 
 import json
 import os
-import re
 import sys
 import urllib.error
 import urllib.request
-from pathlib import Path
 
 import pytest
 
@@ -229,11 +227,8 @@ def test_diagnostics_redacts_token_args():
 # --------------------------------------------------------------------------- #
 # 6. Dependency / PATH checking
 # --------------------------------------------------------------------------- #
-def test_dep_check_finds_python():
+def test_dep_check_finds_python3():
-    # "python3" does not exist on a stock Windows install; "warn" (found only
+    assert c.check_dependency({"command": "python3"})["status"] == "ok"
-    # on an augmented PATH) still proves resolution works on a real machine.
-    cmd = "python" if os.name == "nt" else "python3"
-    assert c.check_dependency({"command": cmd})["status"] in ("ok", "warn")
 
 
 def test_dep_check_flags_missing():
@@ -331,55 +326,6 @@ def test_spawn_test_large_stderr_does_not_deadlock():
     assert len(r["stderr"]) <= c._STDERR_CAP
 
 
-def test_windows_tree_kill_uses_taskkill(monkeypatch):
-    """The Windows kill path must walk the process tree (taskkill /T /F);
-    Popen.kill() alone orphans grandchildren (issue #13)."""
-    calls = []
-
-    def fake_run(cmd, **kwargs):
-        calls.append(cmd)
-        return _FakeRC()
-
-    class _FakeRC:
-        returncode = 0
-
-    monkeypatch.setattr(c.subprocess, "run", fake_run)
-    c._kill_process_tree_windows(1234)
-    assert calls == [["taskkill", "/PID", "1234", "/T", "/F"]]
-
-
-@pytest.mark.skipif(os.name != "nt", reason="Windows-only: real process-tree kill")
-def test_spawn_test_windows_kills_child_process_tree(tmp_path):
-    """A spawn-tested parent that has its own child (npx -> node style) must
-    not leave the child running after the timeout kill (issue #13)."""
-    import subprocess as sp
-    import time as _t
-
-    pid_file = tmp_path / "child.pid"
-    parent_script = (
-        "import subprocess, sys, time\n"
-        "p = subprocess.Popen([sys.executable, '-c', 'import time; time.sleep(60)'])\n"
-        f"open({str(pid_file)!r}, 'w').write(str(p.pid))\n"
-        "time.sleep(60)\n"
-    )
-    r = c.spawn_test({"command": sys.executable, "args": ["-c", parent_script]}, timeout=2.0)
-    assert r["outcome"] == "ok"  # parent was still running at timeout, then tree-killed
-    assert pid_file.is_file(), "parent never spawned its child"
-    child_pid = int(pid_file.read_text())
-
-    deadline = _t.time() + 5.0
-    alive = True
-    while _t.time() < deadline:
-        out = sp.run(
-            ["tasklist", "/FI", f"PID eq {child_pid}"], capture_output=True, text=True
-        ).stdout
-        alive = str(child_pid) in out
-        if not alive:
-            break
-        _t.sleep(0.25)
-    assert not alive, f"child pid {child_pid} survived the spawn-test kill"
-
-
 def test_spawn_test_non_string_env_value():
     # Pasted JSON can carry numeric env values ("env": {"PORT": 8080}) that never
     # round-trip through the editor. Popen rejects non-str env; spawn_test must
@@ -1024,13 +970,11 @@ class _FakeCompletedProcess:
 
 
 def test_restart_claude_desktop_macos_commands(monkeypatch):
-    """macOS: pkill -x "Claude", wait for exit (pgrep), then open -a Claude."""
+    """macOS: pkill -x "Claude" then open -a Claude, in that order."""
     calls = []
 
     def fake_run(cmd, **kwargs):
         calls.append(cmd)
-        if cmd[0] == "pgrep":
-            return _FakeCompletedProcess(returncode=1)  # already exited
         return _FakeCompletedProcess(returncode=0)
 
     monkeypatch.setattr(c.sys, "platform", "darwin")
@@ -1038,14 +982,14 @@ def test_restart_claude_desktop_macos_commands(monkeypatch):
     result = c.restart_claude_desktop()
     assert result.success
     assert calls[0] == ["pkill", "-x", "Claude"]
-    assert calls[-1] == ["open", "-a", "Claude"]
+    assert calls[1] == ["open", "-a", "Claude"]
 
 
 def test_restart_claude_desktop_macos_pkill_not_running_is_fine(monkeypatch):
     """pkill exiting non-zero (nothing to kill) must NOT be treated as failure."""
 
     def fake_run(cmd, **kwargs):
-        if cmd[0] in ("pkill", "pgrep"):
+        if cmd[0] == "pkill":
             return _FakeCompletedProcess(returncode=1)  # no matching process
         return _FakeCompletedProcess(returncode=0)
 
@@ -1069,12 +1013,12 @@ def test_restart_claude_desktop_macos_relaunch_failure_reported(monkeypatch):
 
 
 def test_restart_claude_desktop_macos_pkill_binary_missing_does_not_raise(monkeypatch):
-    """pkill/pgrep raising OSError (binary missing) must be swallowed, not
+    """pkill raising OSError (binary missing) must be swallowed, not propagated --
-    propagated -- relaunch is still attempted."""
+    relaunch is still attempted."""
 
     def fake_run(cmd, **kwargs):
-        if cmd[0] in ("pkill", "pgrep"):
+        if cmd[0] == "pkill":
-            raise OSError("binary not found")
+            raise OSError("pkill not found")
         return _FakeCompletedProcess(returncode=0)
 
     monkeypatch.setattr(c.sys, "platform", "darwin")
@@ -1083,45 +1027,7 @@ def test_restart_claude_desktop_macos_pkill_binary_missing_does_not_raise(monkey
     assert result.success
 
 
-def test_restart_claude_desktop_macos_gives_up_if_app_wont_quit(monkeypatch):
+def test_restart_claude_desktop_windows_commands(monkeypatch):
-    monkeypatch.setattr(c.sys, "platform", "darwin")
-    monkeypatch.setattr(c, "_run_quiet", lambda cmd: None)
-    monkeypatch.setattr(c, "_macos_claude_running", lambda: True)  # never exits
-    monkeypatch.setattr(c, "_MACOS_QUIT_WAIT_S", 0.2)
-    res = c.restart_claude_desktop()
-    assert res.success is False
-    assert "quit" in res.detail.lower()
-
-
-def test_restart_claude_desktop_macos_waits_for_exit_then_relaunches(monkeypatch):
-    monkeypatch.setattr(c.sys, "platform", "darwin")
-    monkeypatch.setattr(c, "_run_quiet", lambda cmd: None)
-    alive = iter([True, True, False])  # exits on the third poll
-    monkeypatch.setattr(c, "_macos_claude_running", lambda: next(alive))
-
-    launched = []
-
-    def fake_run(cmd, **kwargs):
-        launched.append(cmd)
-        return _FakeCompletedProcess(returncode=0)
-
-    monkeypatch.setattr(c.subprocess, "run", fake_run)
-    res = c.restart_claude_desktop()
-    assert res.success is True
-    assert launched == [["open", "-a", "Claude"]]
-
-
-@pytest.fixture
-def windows_shortcut(tmp_path, monkeypatch):
-    """A fake %APPDATA% containing the Claude Start-menu shortcut."""
-    monkeypatch.setenv("APPDATA", str(tmp_path))
-    lnk = tmp_path / "Microsoft" / "Windows" / "Start Menu" / "Programs" / "Claude.lnk"
-    lnk.parent.mkdir(parents=True)
-    lnk.write_bytes(b"")
-    return lnk
-
-
-def test_restart_claude_desktop_windows_commands(monkeypatch, windows_shortcut):
     """Windows: taskkill the process, then relaunch via the Start-menu shortcut."""
     calls = []
 
@@ -1138,7 +1044,7 @@ def test_restart_claude_desktop_windows_commands(monkeypatch, windows_shortcut):
     assert calls[1][-1].endswith("Claude.lnk")
 
 
-def test_restart_claude_desktop_windows_relaunch_failure_reported(monkeypatch, windows_shortcut):
+def test_restart_claude_desktop_windows_relaunch_failure_reported(monkeypatch):
     def fake_run(cmd, **kwargs):
         if cmd[0] == "cmd":
             return _FakeCompletedProcess(returncode=1, stderr="not found")
@@ -1151,37 +1057,43 @@ def test_restart_claude_desktop_windows_relaunch_failure_reported(monkeypatch, w
     assert "not found" in result.detail
 
 
-def test_restart_claude_desktop_windows_missing_shortcut_refuses_before_killing(
+def test_restart_claude_desktop_linux_commands(monkeypatch):
-    monkeypatch, tmp_path
+    """Linux: pkill claude, then relaunch via a detached Popen (no waiting)."""
-):
+    run_calls = []
-    killed = []
+    popen_calls = []
-    monkeypatch.setattr(c, "_run_quiet", lambda cmd: killed.append(cmd))
-    monkeypatch.setattr(c.sys, "platform", "win32")
-    monkeypatch.setenv("APPDATA", str(tmp_path))  # no Claude.lnk under here (MSIX case)
-    res = c.restart_claude_desktop()
-    assert res.success is False
-    assert "shortcut" in res.detail.lower()
-    assert killed == []  # Claude must NOT be killed when it can't be relaunched
 
+    def fake_run(cmd, **kwargs):
+        run_calls.append(cmd)
+        return _FakeCompletedProcess(returncode=0)
+
+    class _FakePopen:
+        def __init__(self, cmd, **kwargs):
+            popen_calls.append((cmd, kwargs))
 
-def test_restart_supported_only_on_desktop_platforms(monkeypatch):
-    monkeypatch.setattr(c.sys, "platform", "darwin")
-    assert c.restart_supported()
-    monkeypatch.setattr(c.sys, "platform", "win32")
-    assert c.restart_supported()
     monkeypatch.setattr(c.sys, "platform", "linux")
-    assert not c.restart_supported()
+    monkeypatch.setattr(c.subprocess, "run", fake_run)
+    monkeypatch.setattr(c.subprocess, "Popen", _FakePopen)
+    result = c.restart_claude_desktop()
+    assert result.success
+    assert run_calls == [["pkill", "claude"]]
+    cmd, kwargs = popen_calls[0]
+    assert cmd == ["claude"]
+    assert kwargs.get("start_new_session") is True
 
 
-def test_restart_claude_desktop_linux_refuses_without_touching_processes(monkeypatch):
+def test_restart_claude_desktop_linux_popen_failure_reported(monkeypatch):
-    """There is no Claude Desktop on Linux; 'pkill claude' would substring-match
+    def fake_run(cmd, **kwargs):
-    running Claude Code CLI sessions. The restart must refuse outright."""
+        return _FakeCompletedProcess(returncode=0)
-    killed = []
+
-    monkeypatch.setattr(c, "_run_quiet", lambda cmd: killed.append(cmd))
+    def fake_popen(cmd, **kwargs):
+        raise OSError("no such file or directory")
+
     monkeypatch.setattr(c.sys, "platform", "linux")
-    res = c.restart_claude_desktop()
+    monkeypatch.setattr(c.subprocess, "run", fake_run)
-    assert res.success is False
+    monkeypatch.setattr(c.subprocess, "Popen", fake_popen)
-    assert killed == []  # nothing killed, nothing spawned
+    result = c.restart_claude_desktop()
+    assert not result.success
+    assert "Claude Desktop" in result.detail
 
 
 # --------------------------------------------------------------------------- #
@@ -1189,12 +1101,8 @@ def test_restart_claude_desktop_linux_refuses_without_touching_processes(monkeyp
 # --------------------------------------------------------------------------- #
 def test_dunder_version_matches_pyproject():
     # Guards against the version drifting out of sync between the two places
-    # a human might bump it. Reads pyproject.toml rather than hard-coding a
+    # a human might bump it.
-    # version here (which would make this a third place to bump).
+    assert c.__version__ == "1.2.0"
-    text = (Path(__file__).parent.parent / "pyproject.toml").read_text(encoding="utf-8")
-    m = re.search(r'^version\s*=\s*"([^"]+)"', text, re.MULTILINE)
-    assert m, "no [project] version found in pyproject.toml"
-    assert c.__version__ == m.group(1)
 
 
 def test_parse_version_basic():