worklog-for-claude-code 0.3.0__tar.gz → 0.3.2__tar.gz

{worklog_for_claude_code-0.3.0/worklog_for_claude_code.egg-info → worklog_for_claude_code-0.3.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: worklog-for-claude-code
-Version: 0.3.0
+Version: 0.3.2
 Summary: See what you did with Claude Code each day and how many tokens it used: an automatic daily work log across accounts and devices, private by default
 Author: Urvin
 License-Expression: MIT
@@ -33,7 +33,7 @@ Provides-Extra: test
 Requires-Dist: pytest>=7; extra == "test"
 Provides-Extra: dev
 Requires-Dist: pytest>=7; extra == "dev"
-Requires-Dist: ruff==0.17.0; extra == "dev"
+Requires-Dist: ruff==0.17.3; extra == "dev"
 Requires-Dist: build; extra == "dev"
 Dynamic: license-file
 
@@ -113,7 +113,11 @@ A checklist then shows anything still missing, with the exact command for your c
 
 **Several accounts or computers?** Settings walks you through it. For several computers, see [Full setup](https://github.com/us-urvin/worklog-for-claude-code/blob/main/docs/GUIDE.md#full-setup).
 
-**How do I update?** Click **Update now** when the dashboard says a new version is out, or run `pipx upgrade worklog-for-claude-code`.
+**Using several GitHub accounts?** GitHub knows you by your SSH key. In Settings → Private data repo, click **Test connection** to see which account signs in, and open *Several GitHub accounts on this computer?* for the steps. [More →](https://github.com/us-urvin/worklog-for-claude-code/blob/main/docs/GUIDE.md#full-setup)
+
+**Uploads to my data repo fail?** Click **Test connection** in Settings: it says what's wrong in plain words. See [Troubleshooting](https://github.com/us-urvin/worklog-for-claude-code/blob/main/docs/TROUBLESHOOTING.md#uploads-fail-or-the-wrong-github-account-is-used).
+
+**How do I update?** Click **Update now** when the dashboard says a new version is out, or run `pipx upgrade --pip-args=--no-cache-dir worklog-for-claude-code`.
 
 **Something isn't working?** Run `claude-worklog doctor`, or see [Troubleshooting](https://github.com/us-urvin/worklog-for-claude-code/blob/main/docs/TROUBLESHOOTING.md).
 

{worklog_for_claude_code-0.3.0 → worklog_for_claude_code-0.3.2}/README.md

@@ -74,7 +74,11 @@ A checklist then shows anything still missing, with the exact command for your c
 
 **Several accounts or computers?** Settings walks you through it. For several computers, see [Full setup](https://github.com/us-urvin/worklog-for-claude-code/blob/main/docs/GUIDE.md#full-setup).
 
-**How do I update?** Click **Update now** when the dashboard says a new version is out, or run `pipx upgrade worklog-for-claude-code`.
+**Using several GitHub accounts?** GitHub knows you by your SSH key. In Settings → Private data repo, click **Test connection** to see which account signs in, and open *Several GitHub accounts on this computer?* for the steps. [More →](https://github.com/us-urvin/worklog-for-claude-code/blob/main/docs/GUIDE.md#full-setup)
+
+**Uploads to my data repo fail?** Click **Test connection** in Settings: it says what's wrong in plain words. See [Troubleshooting](https://github.com/us-urvin/worklog-for-claude-code/blob/main/docs/TROUBLESHOOTING.md#uploads-fail-or-the-wrong-github-account-is-used).
+
+**How do I update?** Click **Update now** when the dashboard says a new version is out, or run `pipx upgrade --pip-args=--no-cache-dir worklog-for-claude-code`.
 
 **Something isn't working?** Run `claude-worklog doctor`, or see [Troubleshooting](https://github.com/us-urvin/worklog-for-claude-code/blob/main/docs/TROUBLESHOOTING.md).
 

{worklog_for_claude_code-0.3.0 → worklog_for_claude_code-0.3.2}/pyproject.toml

@@ -5,7 +5,7 @@ build-backend = "setuptools.build_meta"
 [project]
 # Distribution name on PyPI. The command stays `claude-worklog` and the package stays `worklog`.
 name = "worklog-for-claude-code"
-version = "0.3.0"
+version = "0.3.2"
 description = "See what you did with Claude Code each day and how many tokens it used: an automatic daily work log across accounts and devices, private by default"
 readme = "README.md"
 requires-python = ">=3.10"
@@ -35,7 +35,7 @@ dependencies = ['tomli>=2.0; python_version < "3.11"']
 
 [project.optional-dependencies]
 test = ["pytest>=7"]
-dev = ["pytest>=7", "ruff==0.17.0", "build"]
+dev = ["pytest>=7", "ruff==0.17.3", "build"]
 
 [project.urls]
 Homepage = "https://github.com/us-urvin/worklog-for-claude-code"

{worklog_for_claude_code-0.3.0 → worklog_for_claude_code-0.3.2}/tests/test_chats.py

@@ -13,7 +13,7 @@ from pathlib import Path
 
 import pytest
 
-from worklog import chats, cli, identity, store, web
+from worklog import chats, cli, identity, store, syncstatus, web
 from worklog.config import ConfigError, dump_config, load_config
 from worklog.report import aggregate, render
 
@@ -73,7 +73,7 @@ def home(tmp_path, monkeypatch):
                      ("APPDATA", "AppData/Roaming"), ("LOCALAPPDATA", "AppData/Local")):
         monkeypatch.setenv(var, str(h / sub) if sub else str(h))
     state = h / ".local/state/claude-worklog"
-    for mod in (identity, store, cli, chats):
+    for mod in (identity, store, cli, chats, syncstatus):
         monkeypatch.setattr(mod, "STATE_DIR", state)
     assert Path.home() == h
     return h

worklog_for_claude_code-0.3.2/tests/test_forget.py

@@ -0,0 +1,170 @@
+"""Forget a login: deletes exactly its Worklog history on this device, after a confirmation."""
+import json
+from datetime import datetime, timedelta
+
+import pytest
+from test_accounts import EMAIL_A, EMAIL_B, UID_A, UID_B, login, make_cfg, records, start_session, transcript
+
+from worklog import collector, forget, identity, watcher, web
+from worklog.config import load_config, save_config
+
+UID_C, EMAIL_C = "33333333-cccc-4ccc-8ccc-333333333333", "stranger@viitor.example"
+
+
+@pytest.fixture
+def isolated(home, monkeypatch):
+    wpath = home / ".local/state/claude-worklog/watch.json"
+    monkeypatch.setattr(watcher, "state_path", lambda: wpath)
+    monkeypatch.setattr(collector, "watch_state", lambda: watcher.load_state(wpath))
+    return home
+
+
+def setup_user_case(home, repo):
+    """Like the reported setup: three logins in ~/.claude, two named, one ("stranger") without a name."""
+    cfg = make_cfg(home, repo, f'[[accounts]]\nname = "laracopilot"\nconfig_dir = "~/.claude"\n'
+                               f'match_account_id = "{UID_B}"\n')
+    cfg.accounts[0] = cfg.accounts[0].__class__("personal", cfg.accounts[0].config_dir, match_account_id=UID_A)
+    cfg.accounts.reverse()  # laracopilot first: it is the folder default for logins without a name
+    save_config(cfg, cfg.path)
+    cfg = load_config(cfg.path)
+    now = datetime.now().astimezone().replace(microsecond=0)
+    for uid, email, sid, days_ago in ((UID_A, EMAIL_A, "s-personal", 0), (UID_B, EMAIL_B, "s-lara", 0),
+                                      (UID_C, EMAIL_C, "s-stranger-old", 6), (UID_C, EMAIL_C, "s-stranger", 0)):
+        login(home, uid, email)
+        start_session(home, sid)
+        transcript(home, sid, now - timedelta(days=days_ago), home)
+    collector.collect(cfg, [now.date() - timedelta(days=6), now.date()])
+    return cfg, now
+
+
+def paths(repo):
+    return sorted(records(repo))
+
+
+def test_forget_an_unnamed_login_deletes_only_its_sessions(isolated, repo):
+    cfg, now = setup_user_case(isolated, repo)
+    before = paths(repo)
+    assert len(before) == 4 and sum("s-stranger" in p for p in before) == 2
+    p = forget.plan(cfg, identity.ref(UID_C))
+    assert p.sessions == {"s-stranger", "s-stranger-old"} and len(p.files) == 2 and len(p.days) == 2
+    assert p.account is None  # it was never named: no account goes away
+
+    msg = forget.apply(load_config(cfg.path), p)
+    assert "deleted 2 session record(s) from 2 day(s)" in msg
+    after = paths(repo)
+    assert not any("s-stranger" in x for x in after) and len(after) == 2
+    assert {x.split("/")[3] for x in after} == {"personal", "laracopilot"}
+    assert UID_C not in {x["account_id"] for x in identity.detected_logins()}
+
+    # a later sync never writes the forgotten sessions again
+    collector.collect(load_config(cfg.path), [now.date() - timedelta(days=6), now.date()])
+    assert not any("s-stranger" in x for x in paths(repo))
+
+
+def test_forget_a_named_login_removes_its_account_and_rehomes_the_rest(isolated, repo):
+    cfg, now = setup_user_case(isolated, repo)
+    cfg.summarizer_account = "laracopilot"
+    save_config(cfg, cfg.path)
+    p = forget.plan(load_config(cfg.path), identity.ref(UID_B))
+    assert p.account == "laracopilot" and p.sessions == {"s-lara"}
+    msg = forget.apply(load_config(cfg.path), p)
+    new = load_config(cfg.path)
+    assert new.account("laracopilot") is None and new.summarizer_account == "personal"
+    assert "removed account laracopilot" in msg and "moved 2 other record(s)" in msg
+    after = paths(repo)
+    assert not any("s-lara" in x for x in after)
+    # the stranger's sessions had fallen back to laracopilot: now with the folder's new default, not orphaned
+    assert all("/laracopilot/" not in x for x in after)
+    assert sum("/personal/s-stranger" in x for x in after) == 2
+    assert records(repo)[next(x for x in after if "s-stranger" in x)]["account"] == "personal"
+
+
+def test_forget_the_only_account_keeps_it_but_unlinks(isolated, repo):
+    cfg = make_cfg(isolated, repo, "")
+    cfg.accounts[0] = cfg.accounts[0].__class__("solo", cfg.accounts[0].config_dir, match_account_id=UID_A)
+    cfg.summarizer_account = "solo"
+    save_config(cfg, cfg.path)
+    cfg = load_config(cfg.path)
+    login(isolated, UID_A, EMAIL_A)
+    start_session(isolated, "s1")
+    transcript(isolated, "s1", datetime.now().astimezone(), isolated)
+    collector.collect(cfg, [datetime.now().date()])
+    p = forget.plan(cfg, identity.ref(UID_A))
+    assert p.keep_account
+    forget.apply(cfg, p)
+    new = load_config(cfg.path)
+    assert [a.name for a in new.accounts] == ["solo"] and new.accounts[0].match_account_id is None
+    assert paths(repo) == []
+
+
+def test_forget_finds_sessions_the_watcher_matched(isolated, repo):
+    cfg = make_cfg(isolated, repo,
+                   f'[[accounts]]\nname = "work"\nconfig_dir = "~/.claude"\nmatch_account_id = "{UID_B}"\n')
+    now = datetime.now().astimezone().replace(microsecond=0)
+    w = watcher.Watcher(watcher.state_path())
+    t = now - timedelta(minutes=20)
+    login(isolated, UID_B, EMAIL_B)  # switched by some tool; no hook anywhere
+    while t <= now:
+        w._last_discover = float("inf")
+        w.step(now=t)
+        t += timedelta(seconds=30)
+    transcript(isolated, "s-w", now - timedelta(minutes=10), isolated)
+    collector.collect(cfg, [now.date()])
+    rec = records(repo)[f"records/{now.date().isoformat()}/dev1/work/s-w.json"]
+    assert rec["account_source"] == "watcher"
+    p = forget.plan(cfg, identity.ref(UID_B))
+    assert p.sessions == {"s-w"} and p.account == "work"
+
+
+def test_forget_drops_the_saved_summaries_of_those_projects(isolated, repo):
+    cfg, now = setup_user_case(isolated, repo)
+    d = now.date().isoformat()
+    rep = repo / "reports" / d
+    rep.mkdir(parents=True)
+    (rep / "dev1.json").write_text(json.dumps({"summaries": {"misc:R&D / misc": {"bullets": ["- secret work"]},
+                                                             "other": {"bullets": ["- keep me"]}}}))
+    (rep / "dev1.md").write_text("- secret work\n")
+    forget.apply(load_config(cfg.path), forget.plan(cfg, identity.ref(UID_C)))
+    meta = json.loads((rep / "dev1.json").read_text())
+    assert "misc:R&D / misc" not in meta["summaries"] and "other" in meta["summaries"]
+    assert not (rep / "dev1.md").exists()
+
+
+def test_the_same_account_logging_in_again_is_tracked_again(isolated, repo):
+    cfg, now = setup_user_case(isolated, repo)
+    forget.apply(load_config(cfg.path), forget.plan(cfg, identity.ref(UID_C)))
+    login(isolated, UID_C, EMAIL_C)
+    start_session(isolated, "s-stranger-new")
+    transcript(isolated, "s-stranger-new", now, isolated)
+    collector.collect(load_config(cfg.path), [now.date()])
+    assert any("s-stranger-new" in x for x in paths(repo))
+    assert UID_C in {x["account_id"] for x in identity.detected_logins()}
+
+
+def test_ui_asks_first_and_explains_what_is_lost(isolated, repo, monkeypatch):
+    cfg, _ = setup_user_case(isolated, repo)
+    monkeypatch.setattr(web, "_try_config", lambda: load_config(cfg.path))
+    app = web.App(8765)
+    ref = identity.ref(UID_C)
+    settings = web.page_settings(app, cfg)
+    assert f'href="/settings?forget={ref}#forget"' in settings and "Forget…" in settings
+
+    card = web.page_settings(app, cfg, ref)
+    assert "Forget s***@viitor.example?" in card and "2 Claude Code session records" in card
+    assert "can&#39;t be undone" in card and "Claude Code&#39;s own files" in card and 'required' in card
+    assert EMAIL_C not in card and UID_C not in card  # masked, as everywhere
+
+    n = len(paths(repo))
+    loc, err, _ = app.post("/accounts/forget", {"ref": ref})  # the box isn't ticked: nothing happens
+    assert err and len(paths(repo)) == n
+    loc, err, _ = app.post("/accounts/forget", {"ref": "000000000000", "confirm": "1"})
+    assert err and len(paths(repo)) == n
+    loc, err, _ = app.post("/accounts/forget", {"ref": ref, "confirm": "1"})
+    assert err is None and loc.startswith("/settings?ok=job")
+    import time
+    for _ in range(200):
+        if not app.jobs.snapshot()["running"]:
+            break
+        time.sleep(0.02)
+    assert app.jobs.snapshot()["ok"], app.jobs.snapshot()["message"]
+    assert not any("s-stranger" in x for x in paths(repo))

worklog_for_claude_code-0.3.2/tests/test_syncstatus.py

@@ -0,0 +1,208 @@
+"""Settings → Private data repo: connected or not, which repo, private or public, and whether syncing works."""
+import io
+import json
+import subprocess
+from urllib.error import HTTPError
+
+import pytest
+from test_accounts import make_cfg
+
+from worklog import syncstatus, web
+from worklog.store import GitStore
+
+
+def git(repo, *args):
+    subprocess.run(["git", "-C", str(repo), *args], check=True, capture_output=True)
+
+
+def section(cfg):
+    return web.data_repo_section(web.App(1), cfg)
+
+
+@pytest.fixture
+def no_network(monkeypatch):
+    def fail(*a, **k):
+        raise AssertionError("no network expected here")
+    monkeypatch.setattr(syncstatus, "urlopen", fail)
+
+
+def test_not_a_git_folder_says_not_set_up(home, repo, no_network):
+    cfg = make_cfg(home, repo)
+    cfg.store_repo = home / "nowhere"
+    html = section(cfg)
+    assert "Not set up" in html and "git init" in html
+
+
+def test_local_only_explains_how_to_connect(home, repo, no_network):
+    html = section(make_cfg(home, repo))
+    assert "Local only" in html and "Not connected" in html and "remote add origin" in html and "Copy" in html
+
+
+def test_connected_shows_which_repo_and_never_the_credentials(home, repo, monkeypatch):
+    git(repo, "remote", "add", "origin", "https://me:ghp_SECRET123@github.com/me/claude-worklog-data.git")
+    monkeypatch.setattr(syncstatus, "urlopen", lambda *a, **k: (_ for _ in ()).throw(
+        HTTPError("u", 404, "Not Found", {}, io.BytesIO(b""))))
+    syncstatus.visibility("github.com", "me/claude-worklog-data")  # what the background check does
+    html = section(make_cfg(home, repo))
+    assert "Connected" in html and "github.com/me/claude-worklog-data" in html
+    assert 'href="https://github.com/me/claude-worklog-data"' in html
+    assert "SECRET" not in html and "ghp_" not in html and "me:ghp" not in html
+    assert "🔒 Private" in html and "Not uploaded yet" in html
+
+
+def test_public_repo_is_flagged_everywhere(home, repo, monkeypatch):
+    git(repo, "remote", "add", "origin", "git@github.com:me/claude-worklog-data.git")
+    monkeypatch.setattr(syncstatus, "urlopen", lambda *a, **k: io.BytesIO(json.dumps({"private": False}).encode()))
+    assert syncstatus.visibility("github.com", "me/claude-worklog-data") == "public"
+    cfg = make_cfg(home, repo)
+    assert "PUBLIC" in section(cfg) and "Change visibility" in section(cfg)
+    assert "is public" in web._public_repo_warning(cfg)  # the warning on every page
+
+
+def test_visibility_is_cached_and_only_asked_for_github(home, monkeypatch):
+    calls = []
+
+    def fake(req, timeout):
+        calls.append(req.full_url)
+        return io.BytesIO(b'{"private": true}')
+    monkeypatch.setattr(syncstatus, "urlopen", fake)
+    assert syncstatus.visibility("github.com", "me/r") == "private"
+    assert syncstatus.visibility("github.com", "me/r") == "private" and len(calls) == 1  # cached
+    assert syncstatus.visibility("gitlab.com", "me/r") == "unknown" and len(calls) == 1  # never asks other hosts
+    assert calls == ["https://api.github.com/repos/me/r"]
+    monkeypatch.setattr(syncstatus, "urlopen", lambda *a, **k: (_ for _ in ()).throw(OSError("offline")))
+    assert syncstatus.visibility("github.com", "me/other") == "unknown"
+
+
+def test_store_records_uploads_and_failures(home, repo, tmp_path, no_network):
+    bare = tmp_path / "remote.git"
+    subprocess.run(["git", "init", "-q", "--bare", str(bare)], check=True)
+    git(repo, "remote", "add", "origin", str(bare))
+    store = GitStore(repo)
+    store.write_json("records/2026-11-01/dev/personal/a.json", {"a": 1})
+    store.commit_and_push("test")
+    assert syncstatus.last("push")["ok"] is True
+    git(repo, "remote", "set-url", "origin", str(tmp_path / "missing.git"))
+    store.write_json("records/2026-11-01/dev/personal/b.json", {"b": 1})
+    store.commit_and_push("test 2")
+    failed = syncstatus.last("push")
+    assert failed["ok"] is False and failed["error"]
+
+
+def test_failed_upload_and_waiting_changes_are_shown(home, repo, no_network):
+    git(repo, "remote", "add", "origin", "git@gitlab.com:me/data.git")
+    GitStore(repo).write_json("records/2026-11-01/dev/personal/a.json", {"a": 1})
+    git(repo, "add", "-A")
+    subprocess.run(["git", "-C", str(repo), "-c", "user.name=t", "-c", "user.email=t@t", "commit", "-qm", "x"],
+                   check=True)
+    syncstatus.record("push", False, "fatal: Could not read from remote repository.")
+    html = section(make_cfg(home, repo))
+    assert "Last upload failed" in html and "Could not read from remote repository" in html
+    assert "1 change waiting to upload" in html
+    assert "Privacy not checked" in html  # not GitHub: no request is made
+    syncstatus.record("push", True)
+    assert "Synced" in section(make_cfg(home, repo))
+
+
+def test_test_connection_works_and_explains_failures(home, repo, tmp_path):
+    bare = tmp_path / "remote.git"
+    subprocess.run(["git", "init", "-q", "--bare", str(bare)], check=True)
+    git(repo, "remote", "add", "origin", str(bare))
+    cfg = make_cfg(home, repo)
+    ok, msg = syncstatus.test_connection(cfg)
+    assert ok and "Connection works" in msg
+    git(repo, "remote", "set-url", "origin", str(tmp_path / "missing.git"))
+    ok, msg = syncstatus.test_connection(cfg)
+    assert not ok and msg.startswith("Can't reach your data repo")
+
+
+def test_sync_works_without_a_git_identity_on_this_computer(home, repo, tmp_path, no_network):
+    """A pull that replays local commits needs a committer name; this computer has none set."""
+    bare = tmp_path / "remote.git"
+    subprocess.run(["git", "init", "-q", "--bare", str(bare)], check=True)
+    git(repo, "remote", "add", "origin", str(bare))
+    git(repo, "config", "user.useConfigOnly", "true")  # like a computer without `git config --global user.email`
+    a = GitStore(repo)
+    a.write_json("records/2026-11-01/a/personal/1.json", {"n": 1})
+    a.commit_and_push("first")
+    other = tmp_path / "other"
+    subprocess.run(["git", "clone", "-q", str(bare), str(other)], check=True)
+    b = GitStore(other)
+    b.write_json("records/2026-11-01/b/personal/2.json", {"n": 2})
+    b.commit_and_push("from the other computer")
+    git(repo, "remote", "set-url", "origin", str(tmp_path / "offline.git"))  # local commits pile up while offline
+    a.write_json("records/2026-11-01/a/personal/3.json", {"n": 3})
+    a.commit_and_push("offline")
+    git(repo, "remote", "set-url", "origin", str(bare))
+    a.pull()
+    assert syncstatus.last("pull")["ok"] is True, syncstatus.last("pull")
+    assert (repo / "records/2026-11-01/b/personal/2.json").exists()
+    a.commit_and_push("nothing new")  # nothing staged, but the waiting commit still goes up
+    assert syncstatus.status(make_cfg(home, repo))["waiting"] == 0
+
+
+def _fake_ssh(monkeypatch, *, who="us-urvin", ls_remote=(0, "", "")):
+    """ssh -G answers from a ~/.ssh/config alias; ssh -T says who the key is; git ls-remote as given."""
+    real_run, real_git = syncstatus.subprocess.run, syncstatus._git
+    syncstatus._ssh_cache.clear()
+    monkeypatch.setattr(syncstatus.shutil, "which", lambda n: "/usr/bin/" + n)
+
+    def run(cmd, *a, **k):
+        if cmd[0] == "ssh" and "-G" in cmd:
+            out = "user git\nhostname github.com\nport 22\nidentityfile ~/.ssh/id_work\n"
+            return subprocess.CompletedProcess(cmd, 0, out, "")
+        if cmd[0] == "ssh" and "-T" in cmd:
+            return subprocess.CompletedProcess(cmd, 1, "", f"Hi {who}! You've successfully authenticated, but GitHub "
+                                                            "does not provide shell access.")
+        return real_run(cmd, *a, **k)
+    monkeypatch.setattr(syncstatus.subprocess, "run", run)
+    monkeypatch.setattr(syncstatus, "_git", lambda repo, *args, **k: ls_remote if args[0] == "ls-remote"
+                        else real_git(repo, *args, **k))
+
+
+def test_ssh_nickname_links_to_the_real_host_and_names_the_key(home, repo, monkeypatch):
+    git(repo, "remote", "add", "origin", "git@worklog-git:us-urvin/claude-worklog-data.git")
+    _fake_ssh(monkeypatch)
+    asked = []
+    monkeypatch.setattr(syncstatus, "urlopen", lambda req, timeout: asked.append(req.full_url) or (_ for _ in ()).throw(
+        HTTPError("u", 404, "Not Found", {}, io.BytesIO(b""))))
+    cfg = make_cfg(home, repo)
+    r = syncstatus.remote(cfg)
+    assert r["host"] == "github.com" and r["alias"] == "worklog-git" and r["key"] == "~/.ssh/id_work"
+    assert syncstatus.visibility(r["host"], r["slug"]) == "private"  # the privacy check works through the nickname
+    html = section(cfg)
+    assert 'href="https://github.com/us-urvin/claude-worklog-data"' in html and "https://worklog-git" not in html
+    assert "worklog-git" in html and "id_work" in html and "Test connection</b> to see which GitHub account" in html
+
+
+def test_test_connection_says_which_account_signs_in(home, repo, monkeypatch):
+    git(repo, "remote", "add", "origin", "git@worklog-git:us-urvin/claude-worklog-data.git")
+    _fake_ssh(monkeypatch)
+    cfg = make_cfg(home, repo)
+    ok, msg = syncstatus.test_connection(cfg)
+    assert ok and "signs in to GitHub as us-urvin" in msg
+    assert "Signs in as us-urvin" in section(cfg)
+
+
+def test_wrong_account_is_explained(home, repo, monkeypatch):
+    git(repo, "remote", "add", "origin", "git@gh-office:us-urvin/claude-worklog-data.git")
+    err = "ERROR: Repository not found.\nfatal: Could not read from remote repository."
+    _fake_ssh(monkeypatch, who="office-u", ls_remote=(128, "", err))
+    ok, msg = syncstatus.test_connection(make_cfg(home, repo))
+    assert not ok and "knows this computer's key as office-u" in msg
+    assert "no access to us-urvin/claude-worklog-data" in msg
+
+
+def test_https_and_local_remotes_are_not_treated_as_ssh(home, repo):
+    git(repo, "remote", "add", "origin", "https://us-urvin@github.com/us-urvin/data.git")
+    r = syncstatus.remote(make_cfg(home, repo))
+    assert r["via"] == "https" and r["user"] == "us-urvin" and r["host"] == "github.com"
+    assert syncstatus._ssh_parts("C:/repos/data.git") is None and syncstatus._ssh_parts("/srv/data.git") is None
+    assert syncstatus._ssh_parts("ssh://git@work-gh:22/me/r.git") == ("git", "work-gh")
+
+
+def test_errors_show_the_line_that_matters():
+    err = ("ssh: connect to host github.com port 22: Connection timed out\nfatal: Could not read from remote "
+           "repository.\n\nPlease make sure you have the correct access rights\nand the repository exists.\n")
+    assert syncstatus._main_line(err).startswith("ssh: connect to host github.com port 22")
+    assert syncstatus._main_line("") == "no answer"