release: v1.2.0 — About+update checker, log viewer, restart button, dup-name fix, stale-file hardening #26

Merged

the_og merged 34 commits from release/v1.2.0 into main 2026-07-08 11:34:35 -04:00

Conversation 0 Commits 34 Files Changed 4

+1103 -39

the_og commented 2026-07-08 11:33:54 -04:00

Owner

Copy Link

Copy Source

Integration of the five reviewed feature branches for the v1.2.0 release, merged with real git 3-way merges, conflicts resolved, and verified green locally (ruff format --check, ruff check, pytest ~138 tests pass).

Closes #6
Closes #8
Closes #9
Closes #17
Closes #18
Closes #19

Included

• #18 + #19 — About dialog (Help menu, repo/issues/license links) + notify-only update checker (__version__ single source, numeric version compare, fail-quiet tokenless releases fetch, throttled once/day startup check). (PR #25)
• #9 — Restart Claude Desktop button after save, gated to Desktop profiles only. (PR #24)
• #6 — In-app MCP server log viewer (server_log_path() + read-only auto-tailing LogViewerDialog). (PR #23)
• #17 — Stale-file detection hardened with a mtime+size fingerprint (ConfigStat/config_fingerprint). (PR #22)
• #8 — Duplicate-name collision fix: closes a real silent-overwrite on drag-and-drop import; shared _import_server prompt + pure resolve_name_collision. (PR #21)

Conflict resolutions

• _normalize_unicode NBSP line — kept the single out.replace(chr(0xA0), " ") form (all branches were equivalent).
• bcc.py — LogViewerDialog (#6) and AboutDialog (#18/19) both inserted a class at the same anchor; kept both. Imports and MainWindow hooks merged cleanly.
• tests/test_core.py — each branch appended tests at EOF; concatenated all.
• Version bumped 1.1.0 → 1.2.0 in __version__, pyproject.toml, and test_dunder_version_matches_pyproject.

Supersedes the individual feature PRs #21–#25 (to be closed on merge).

Integration of the five reviewed feature branches for the **v1.2.0** release, merged with real git 3-way merges, conflicts resolved, and verified green locally (`ruff format --check`, `ruff check`, `pytest` ~138 tests pass). Closes #6 Closes #8 Closes #9 Closes #17 Closes #18 Closes #19 ## Included - **#18 + #19** — About dialog (Help menu, repo/issues/license links) + notify-only update checker (`__version__` single source, numeric version compare, fail-quiet tokenless releases fetch, throttled once/day startup check). *(PR #25)* - **#9** — Restart Claude Desktop button after save, gated to Desktop profiles only. *(PR #24)* - **#6** — In-app MCP server log viewer (`server_log_path()` + read-only auto-tailing `LogViewerDialog`). *(PR #23)* - **#17** — Stale-file detection hardened with a mtime+size fingerprint (`ConfigStat`/`config_fingerprint`). *(PR #22)* - **#8** — Duplicate-name collision fix: closes a real silent-overwrite on drag-and-drop import; shared `_import_server` prompt + pure `resolve_name_collision`. *(PR #21)* ## Conflict resolutions - `_normalize_unicode` NBSP line — kept the single `out.replace(chr(0xA0), " ")` form (all branches were equivalent). - `bcc.py` — `LogViewerDialog` (#6) and `AboutDialog` (#18/19) both inserted a class at the same anchor; kept both. Imports and `MainWindow` hooks merged cleanly. - `tests/test_core.py` — each branch appended tests at EOF; concatenated all. - Version bumped 1.1.0 → 1.2.0 in `__version__`, `pyproject.toml`, and `test_dunder_version_matches_pyproject`. Supersedes the individual feature PRs #21–#25 (to be closed on merge).

the_og added 34 commits 2026-07-08 11:33:55 -04:00

feat: harden stale-file detection with mtime+size fingerprint (#17) ... 3c65657d2f

Bare mtime equality can miss a concurrent external write that lands
within the filesystem's mtime resolution (same-second writes), or
where the writer restores the original mtime. Add config_fingerprint()
returning a (mtime, size) ConfigStat pair; the stale-file check in
bcc.py now compares both fields instead of mtime alone. config_mtime()
is kept as-is (still used/tested independently).

bcc_core.py: add resolve_name_collision() for paste/import name-collision handling 165c65be5f

bcc_core.py: fix non-breaking-space byte lost in transcription 0f1cdbef3c

fix: restore non-breaking-space literal lost in transcription ... 9760b1537e

The previous commit accidentally normalized the non-breaking space
(U+00A0) in _normalize_unicode's replace() call to a regular space
during a copy/paste, turning that replace() into a no-op. Use an
explicit   escape instead of the literal character so it can't
be silently corrupted again.

feat: add server_log_path() for in-app MCP log viewer (#6) 8d90ab449d

feat: __version__ constant + notify-only update checker logic (#19), version source for About dialog (#18) ... fe66d53e9f

- Add `__version__ = "1.1.0"` as the single source of truth (matches pyproject.toml).
- Add parse_version()/is_newer_version() for numeric (never lexical) version
  comparison, handling v-prefix, pre-release suffixes, and malformed input.
- Add fetch_latest_release(): reads the public Gitea releases API
  (anonymous, no token) and returns {version, url} or None on any failure.
  Never downloads or touches a binary — metadata only.

bcc.py: unify paste/drop collision handling via shared _import_server() prompt 2a0802b22f

fix: actually apply the non-breaking-space escape fix ... 4b45251682

The previous commit message said this was fixed but the content still
had the literal (unescaped) line. Apply the   escape for real
this time.

feat: restart_claude_desktop() core logic (#9) ... 6c51bac1e0

Platform-abstracted restart: macOS uses pkill -x Claude + open -a Claude,
Windows uses taskkill + relaunch via the Start-menu shortcut, Linux uses
pkill claude + a detached Popen relaunch. Not finding a running process is
not an error -- only a failed relaunch is reported as failure.

Also adds profile_targets_claude_desktop() to distinguish Claude Desktop
profiles (claude_desktop_config.json) from Claude Code profiles, used to
gate the restart button in the GUI.

fix: use chr(0xA0) instead of a literal/escaped NBSP ... 82cec27c11

Two prior attempts to fix this line via a literal or  -escaped
non-breaking space both silently reverted back to a no-op regular-space
replace during transcription. Using chr(0xA0) instead removes any
non-ASCII or backslash-escape character from the source line entirely.

tests/test_core.py: add resolve_name_collision unit tests 0843c51c7d

chore: restore trailing newline in bcc_core.py cffdee8a40

fix: restore actual file content (previous commit had placeholder text by mistake) 5d59c1c423

fix: use   escape for non-breaking space (byte-exact transcription fix) 346d0aabb6

feat: Restart Claude Desktop button after save (#9) ... fd2c3567a0

After a successful save, shows a 'Restart Claude Desktop' button next to
the status line. Only shown when the saved profile targets Claude Desktop
(core.profile_targets_claude_desktop) -- never for Claude Code, which has
no GUI process to bounce. Clicking it calls core.restart_claude_desktop()
and reports success/failure. The button hides again on further edits or
when switching/reloading profiles.

chore: restore trailing newline in bcc_core.py (attempt 2) 0ffc6a1fb6

feat: use mtime+size fingerprint for stale-file check (#17) ... f1935fe320

MainWindow._loaded_mtime -> _loaded_stat (core.ConfigStat), populated
via core.config_fingerprint() at load/save time. The save-path stale
check now compares the full fingerprint (mtime AND size) instead of
bare mtime equality.

fix: use explicit backslash-u-0-0-a-0 escape for non-breaking space literal 7c8fd6d0bb

test: add coverage for config_fingerprint mtime+size hardening (#17) ... f9752211a2

Includes the required regression case: rewrite a file with different
content, force the original mtime back via os.utime, and assert the
fingerprint still differs (because size changed).

test: add server_log_path coverage (macOS, Windows, missing, unsupported) (#6) 85d47aea97

fix: use \xa0 escape instead of a literal NBSP byte (transmission-safe, same behavior); drop stray trailing blank line 256827eaf3

feat: add View logs button + LogViewerDialog for in-app MCP log viewer (#6) 2b3843a714

test: cover restart_claude_desktop() and profile_targets_claude_desktop() (#9) ... 16961a5cc8

Mocks subprocess.run/Popen and patches sys.platform per-OS branch (darwin,
win32, linux) -- never actually kills or launches anything. Covers: correct
command sequence per platform, pkill/taskkill exiting non-zero (nothing to
kill) is NOT treated as failure, a failed relaunch IS reported as failure,
and profile_targets_claude_desktop() correctly distinguishes Claude Desktop
configs from Claude Code / legacy settings.json profiles.

feat: About dialog with menu bar + notify-only update checker UI (#18, #19) ... 67c898cd35

- Add a real QMenuBar (Help -> About...).
- AboutDialog: app icon, name, version (core.__version__), and links to the
  repo/issues/license opened via QDesktopServices.openUrl (system browser,
  never in-app navigation). macOS unsigned-app note.
- "Check for updates" button + UpdateCheckWorker (QThread) runs
  core.fetch_latest_release() off the UI thread; shows "up to date" or
  "vX.Y.Z available" with a button to open the releases page. No binary
  download, ever.
- Optional quiet startup auto-check, throttled to once/day via a QSettings
  timestamp, off-thread, silent on failure, toggle lives in the About dialog.

test: cover __version__, parse_version/is_newer_version, and fetch_latest_release (#19) ... 3cd18392c9

All network calls are monkeypatched (urllib.request.urlopen) — no live
network in tests. Covers older/newer/equal comparisons, v-prefix,
differing-length tuples, malformed input on both sides, and
fetch_latest_release success/timeout/network-failure/malformed-response
paths.

Fix: rebuild from clean main (previous push included unrelated contaminated content) a811e323e6

Fix: rebuild from clean main (previous push included unrelated contaminated content) 3e07b51134

Fix: rebuild from clean main (previous push included unrelated contaminated content) 4ab3c3b00a

Fix: correct smart-quote/nbsp characters mangled in previous push 2d9fb083dc

Merge branch 'feat/17-stale-size-hardening' into integration/v1.2.0 8fdcbda681

Merge branch 'feat/6-log-viewer' into integration/v1.2.0 ... 70b865be8f

# Conflicts:
#	bcc_core.py
#	tests/test_core.py

Merge branch 'feat/9-restart-claude-desktop' into integration/v1.2.0 ... c56dec8051

# Conflicts:
#	bcc_core.py
#	tests/test_core.py

Merge branch 'feat/18-19-about-update-checker' into integration/v1.2.0 ... 15a30fb986

# Conflicts:
#	bcc.py
#	bcc_core.py
#	tests/test_core.py

chore: bump version to 1.2.0 for release 8c718387c0

the_og merged commit 4c6fe7c5aa into main 2026-07-08 11:34:35 -04:00

the_og deleted branch release/v1.2.0 2026-07-08 11:34:35 -04:00

the_og referenced this pull request 2026-07-08 11:35:37 -04:00

Fix silent name-collision overwrite on drag-and-drop import (#8) #21

the_og referenced this pull request 2026-07-08 11:35:39 -04:00

feat: harden stale-file change detection with mtime+size fingerprint (#17) #22

the_og referenced this pull request 2026-07-08 11:35:42 -04:00

feat: in-app MCP server log viewer (#6) #23

the_og referenced this pull request 2026-07-08 11:35:44 -04:00

feat: Restart Claude Desktop convenience button (#9) #24

the_og referenced this pull request 2026-07-08 11:35:47 -04:00

feat: About dialog + notify-only update checker (#18, #19) #25

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

P0

P1

P2

blocked

Waiting on an external thing before it can move.

decision

A decision record (D-007 style) — settled call, referenceable by number from any repo.

needs-decision

Waiting on AJ's call.

next

Current focus / next actionable item.

status

The single pinned STATUS issue for this repo — live state, next action, open items, decisions needed.

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

the_og

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: the_og/better-claude-config#26