thth 2.3.1__tar.gz → 2.3.3__tar.gz

{thth-2.3.1 → thth-2.3.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: thth
-Version: 2.3.1
+Version: 2.3.3
 Summary: 承認済みの投稿だけを投げる門（Threads・Bluesky・Mastodon）。人が承認し、記録は利用者の git に残る。
 Project-URL: Homepage, https://thth.me
 Project-URL: Source, https://github.com/aokings/thth
@@ -45,7 +45,7 @@ watchtower（`~/Developer/watchtower`）の隣に同じ流儀で並べる。watc
 
 ## いまの状態（2026-09-13）
 
-版 **2.3.1**。全件テスト **2287 件**（`python -m pytest tests/ -q -n auto -p no:cacheprovider`・1 skip・rc=0）。
+版 **2.3.3**。全件テスト **2320 件**（`python -m pytest tests/ -q -n auto -p no:cacheprovider`・1 skip・rc=0）。
 
 **媒体**: Threads（稼働）・Bluesky・Mastodon（同席用の台帳あり・未稼働）。
 
@@ -79,7 +79,7 @@ timer（systemd・`thth systemd` で生成）で毎時投稿、返信の採集
 - **英語の文書**: [README.en.md](README.en.md)・[docs/usage.en.md](docs/usage.en.md)・[llms.txt](llms.txt)。
 - **skill**: `skills/thth/SKILL.md`（wheel にも入る）。
 
-- **動くもの**（`thth --help` の全 34 サブコマンド）: `lint`・`preview`・`approve`・`account`・`revoke`・`posts`・`replies`・`measured`・`threads`・`after`・`topics`・`forms`・`queue`・`schedule`・`throw`・`run`・`systemd`・`board`・`collect`・`auth`・`refresh`・`maintain`・`send`・`doctor`・`app`・`token`・`ask`・`mentions`・`profile`・`thread`・`where`・`who`・`retract`・`location`。
+- **動くもの**（`thth --help` の全 35 サブコマンド）: `lint`・`preview`・`approve`・`account`・`revoke`・`posts`・`replies`・`measured`・`threads`・`after`・`topics`・`forms`・`queue`・`schedule`・`throw`・`run`・`systemd`・`board`・`collect`・`pull`・`auth`・`refresh`・`maintain`・`send`・`doctor`・`app`・`token`・`ask`・`mentions`・`profile`・`thread`・`where`・`who`・`retract`・`location`。
 - **最初の本番投稿の記録**: 2026-09-09、@aoking に疎通確認を 1 本（`17916074118445631`）。
 - **未着手**: X・Facebook ページ・Instagram の各アダプタ。トピック検索の権限（tester には降りない）。泉のサーバ（v2-5）。
 - **権限の制約**: tester に降りる scope は 5 つ。削除はできない。

{thth-2.3.1 → thth-2.3.3}/README.md

@@ -21,7 +21,7 @@ watchtower（`~/Developer/watchtower`）の隣に同じ流儀で並べる。watc
 
 ## いまの状態（2026-09-13）
 
-版 **2.3.1**。全件テスト **2287 件**（`python -m pytest tests/ -q -n auto -p no:cacheprovider`・1 skip・rc=0）。
+版 **2.3.3**。全件テスト **2320 件**（`python -m pytest tests/ -q -n auto -p no:cacheprovider`・1 skip・rc=0）。
 
 **媒体**: Threads（稼働）・Bluesky・Mastodon（同席用の台帳あり・未稼働）。
 
@@ -55,7 +55,7 @@ timer（systemd・`thth systemd` で生成）で毎時投稿、返信の採集
 - **英語の文書**: [README.en.md](README.en.md)・[docs/usage.en.md](docs/usage.en.md)・[llms.txt](llms.txt)。
 - **skill**: `skills/thth/SKILL.md`（wheel にも入る）。
 
-- **動くもの**（`thth --help` の全 34 サブコマンド）: `lint`・`preview`・`approve`・`account`・`revoke`・`posts`・`replies`・`measured`・`threads`・`after`・`topics`・`forms`・`queue`・`schedule`・`throw`・`run`・`systemd`・`board`・`collect`・`auth`・`refresh`・`maintain`・`send`・`doctor`・`app`・`token`・`ask`・`mentions`・`profile`・`thread`・`where`・`who`・`retract`・`location`。
+- **動くもの**（`thth --help` の全 35 サブコマンド）: `lint`・`preview`・`approve`・`account`・`revoke`・`posts`・`replies`・`measured`・`threads`・`after`・`topics`・`forms`・`queue`・`schedule`・`throw`・`run`・`systemd`・`board`・`collect`・`pull`・`auth`・`refresh`・`maintain`・`send`・`doctor`・`app`・`token`・`ask`・`mentions`・`profile`・`thread`・`where`・`who`・`retract`・`location`。
 - **最初の本番投稿の記録**: 2026-09-09、@aoking に疎通確認を 1 本（`17916074118445631`）。
 - **未着手**: X・Facebook ページ・Instagram の各アダプタ。トピック検索の権限（tester には降りない）。泉のサーバ（v2-5）。
 - **権限の制約**: tester に降りる scope は 5 つ。削除はできない。

{thth-2.3.1 → thth-2.3.3}/skills/thth/SKILL.md

@@ -12,7 +12,8 @@ description: 投稿する前・返信を書く前・出したあと・絡みに
 - **`--production` を付けるまで出ない。** 付けない実行は、投げるはずの本文と digest を
   表示して終わる。
 - **記録は利用者の git に残る。** post_id・承認者・承認時刻・返信・実測は、あなたの repo の
-  commit として残る。THTH のサーバには何も無い。
+  commit として残る。THTH のサーバには何も無い。絡みの台帳（`data/sns/engagements/`）は
+  公開の commit とは別で、**次の採取（`thth collect`・最大 10 分の遅れ）**で commit される。
 - **数値には分母が付く。** 件数・期間・揃えた条件が無い数字は返さない。取れていない刻みは
   `null` であって `0` ではない。
 - **黙って間違えない。** 拒むときは理由と次の一手を言って非ゼロで終わる（loud reject）。
@@ -30,8 +31,11 @@ description: 投稿する前・返信を書く前・出したあと・絡みに
 - **何を書くかを決めない。** 文体・頻度・時刻・語の選び方はあなたのプロジェクトのもの。
   渡された本文を**整形も切り詰めもせず**そのまま出す。
 - **読む口は何も保存しない。** `thread_read`・`where_to_appear`・`who_is_this` はどれも、
-  読んで見せるだけで**台帳に 1 バイトも書かない**。残るのは**自分の行為と反応**
-  （絡みの台帳）だけで、**相手の本文・相手の名前・あなたの判断は台帳に入らない**。
+  読んで見せるだけで**台帳に 1 バイトも書かない**。この 3 つと絡みの台帳
+  （`data/sns/engagements/`）だけを見れば、残るのは**自分の行為と反応**だけで、
+  **相手の本文・相手の名前・あなたの判断は入らない**。**別の台帳**——自分の投稿への
+  返信の台帳（`thth collect` が採り、`thth replies` で読む）——は、この規律の外で
+  相手の `username` と本文をそのまま残す（読む口でもなく、絡みの台帳でもない）。
 - **人の代わりに承認しない。** あなた（エージェント）は下書きと digest を人に見せ、
   人が `--confirm` を打つ。
 

thth-2.3.3/tests/test_behind_remote.py

@@ -0,0 +1,142 @@
+"""`writeback.behind_remote()` と `thth queue`/`schedule`/`board` の遅れ表示
+（発注 T8-2・kopicha 続報）。
+
+`thth approve`（と `revoke`・`throw`・`collect`・`retract`）は既に
+`writeback.sync_repo()` を通しているが、**読むだけの口**（`queue`・
+`schedule`・`board`）は通していない。push した直後に読むだけの口を見ると、
+timer の `collect` が pull するまでの間、まだ取り込み前の一覧が出る
+——**読むだけの口が勝手に pull するのは避ける**のが設計の芯なので、
+ここでは「遅れていることを言う」だけを確かめる（取り込みは T8-3 の
+`thth pull` か `thth approve` が行う）。
+"""
+from __future__ import annotations
+
+from tests.conftest import init_git_pair, run_git, run_thth, write_queue_file
+from thth import writeback
+
+
+def test_behind_remoteは追いついていれば0(tmp_path):
+    pair = init_git_pair(tmp_path / "a", seed_content="", seed_name=".gitkeep")
+    info = writeback.behind_remote(pair["work"])
+    assert info == {"behind": 0, "ahead": 0, "head": info["head"],
+                     "fetched_at": info["fetched_at"], "reason": None}
+    assert info["head"]
+    assert info["reason"] is None
+
+
+def test_behind_remoteはfetchだけでpullしない(tmp_path):
+    """**pull はしない**——fetch のあと、work の作業ツリー（HEAD）は動かない。"""
+    pair = init_git_pair(tmp_path / "a", seed_content="旧", seed_name="a.md")
+    head_before = run_git(pair["work"], ["rev-parse", "HEAD"]).stdout.strip()
+
+    run_git(pair["seed"], ["commit", "--allow-empty", "-m", "second"])
+    run_git(pair["seed"], ["push"])
+
+    info = writeback.behind_remote(pair["work"])
+    assert info["behind"] == 1
+    assert info["ahead"] == 0
+
+    head_after = run_git(pair["work"], ["rev-parse", "HEAD"]).stdout.strip()
+    assert head_after == head_before, "fetch だけのはずが HEAD が動いた（pull してしまった）"
+
+
+def test_behind_remoteはfetchできないとNoneとreason(tmp_path):
+    broken = tmp_path / "broken"
+    run_git_init = __import__("subprocess").run(
+        ["git", "init", "-b", "main", str(broken)], capture_output=True, text=True)
+    assert run_git_init.returncode == 0
+    run_git(str(broken), ["remote", "add", "origin", "/does/not/exist/repo.git"])
+    # **git の名前とメールを repo に持たせる**（`conftest.init_git_pair()` と同じ）。
+    # 開発機（macOS）は全体の設定があるので通っていたが、GitHub Actions の
+    # Linux には無く、`commit` が "Author identity unknown" で落ちた（2026-09-16）。
+    run_git(str(broken), ["config", "user.email", "thth-test@example.invalid"])
+    run_git(str(broken), ["config", "user.name", "thth-test"])
+    run_git(str(broken), ["commit", "--allow-empty", "-m", "x"])
+
+    info = writeback.behind_remote(str(broken))
+    assert info["behind"] is None
+    assert info["ahead"] is None
+    assert info["reason"]
+    assert "0" != info["reason"]  # **0 と混ぜない**: reason があるのに behind: 0 ではない
+
+
+def test_queueは遅れているときに先頭に1行(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="", seed_name=".gitkeep")
+    account = isolated_account_factory(repo_dir=pair["work"])
+    write_queue_file(account["queue_dir"], "a.md",
+                      fm_overrides={"status": "draft", "account": account["name"]})
+
+    # **`seed` を origin の最新に合わせてから**新しい commit を積む（`work` の
+    # `write_queue_file()` push で origin が進んでいるため、合わせないと
+    # `seed` の push 自体が non-fast-forward で失敗する）。
+    run_git(pair["seed"], ["pull"])
+    run_git(pair["seed"], ["commit", "--allow-empty", "-m", "second"])
+    run_git(pair["seed"], ["push"])
+
+    result = run_thth(["queue", account["name"]])
+    assert result.returncode == 0, result.stderr
+    assert "remote に 1 commit 分の新しいものがあります" in result.stdout
+    assert f"thth pull {account['name']}" in result.stdout
+
+
+def test_queueは遅れていなければ何も出さない(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="", seed_name=".gitkeep")
+    account = isolated_account_factory(repo_dir=pair["work"])
+    write_queue_file(account["queue_dir"], "a.md",
+                      fm_overrides={"status": "draft", "account": account["name"]})
+
+    result = run_thth(["queue", account["name"]])
+    assert result.returncode == 0, result.stderr
+    assert "remote に" not in result.stdout
+    assert "thth pull" not in result.stdout
+
+
+def test_queueのjsonにrepoが乗る(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="", seed_name=".gitkeep")
+    account = isolated_account_factory(repo_dir=pair["work"])
+    write_queue_file(account["queue_dir"], "a.md",
+                      fm_overrides={"status": "draft", "account": account["name"]})
+
+    result = run_thth(["queue", account["name"], "--json"])
+    assert result.returncode == 0, result.stderr
+    import json
+    out = json.loads(result.stdout)
+    assert out[account["name"]]["repo"]["behind"] == 0
+
+
+def test_scheduleは遅れているときに先頭に1行(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="", seed_name=".gitkeep")
+    account = isolated_account_factory(repo_dir=pair["work"])
+    write_queue_file(account["queue_dir"], "a.md",
+                      fm_overrides={"status": "draft", "account": account["name"],
+                                    "publish_at": "2026-09-20T08:00:00+09:00"})
+
+    run_git(pair["seed"], ["pull"])
+    run_git(pair["seed"], ["commit", "--allow-empty", "-m", "second"])
+    run_git(pair["seed"], ["push"])
+
+    result = run_thth(["schedule", account["name"]])
+    assert result.returncode == 0, result.stderr
+    assert "remote に 1 commit 分の新しいものがあります" in result.stdout
+
+
+def test_boardは遅れているaccountの行の下に1行(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="", seed_name=".gitkeep")
+    account = isolated_account_factory(repo_dir=pair["work"])
+
+    run_git(pair["seed"], ["commit", "--allow-empty", "-m", "second"])
+    run_git(pair["seed"], ["push"])
+
+    result = run_thth(["board"])
+    assert result.returncode == 0, result.stderr
+    assert "repo が 1 commit 遅れています" in result.stdout
+    assert f"thth pull {account['name']}" in result.stdout
+
+
+def test_boardは遅れていなければ何も出さない(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="", seed_name=".gitkeep")
+    isolated_account_factory(repo_dir=pair["work"])
+
+    result = run_thth(["board"])
+    assert result.returncode == 0, result.stderr
+    assert "repo が" not in result.stdout

thth-2.3.3/tests/test_cli_vm_path.py

@@ -0,0 +1,80 @@
+"""VM に無いパスを渡したときの断り方（発注 T8-1・kopicha 続報）。
+
+`~/.local/bin/thth` は Mac から VM へ ssh する薄いラッパで、引数はそのまま
+VM 側の `thth` に渡る。**Mac 側の相対パスも絶対パスも VM には無い**ので、
+いままでは `FileNotFoundError` の素の traceback が出ていた（`--article
+/tmp/x.json` で踏んだのと同じ根）。
+
+ファイルを受け取る全ての引数（`lint`・`approve`・`revoke`・`preview`・
+`send --text-file`・`topics suggest` の queue ファイル・`--article`・
+`--proposal`・`--input`・`--draft`・`--revised-draft`）が、共通の
+`cli._require_vm_path()` を通って rc=2・案内つきで断ることを確かめる。
+存在するパスは従来どおり動くことも確かめる（回帰）。
+"""
+from __future__ import annotations
+
+import os
+
+from tests.conftest import run_thth, write_queue_file
+
+MISSING = "/tmp/thth-t8-does-not-exist-まさかここには無い.md"
+MISSING_JSON = "/tmp/thth-t8-does-not-exist-まさかここには無い.json"
+
+
+def _assert_vm_refusal(result, *, rc=2):
+    combined = result.stdout + result.stderr
+    assert result.returncode == rc, f"rc={result.returncode}: {combined}"
+    assert "VM" in combined, combined
+    assert "thth queue" in combined, combined
+    assert "Traceback" not in result.stderr, result.stderr
+
+
+def test_lintにVMに無いパスを渡すとrc2で案内する():
+    result = run_thth(["lint", MISSING])
+    _assert_vm_refusal(result)
+
+
+def test_approveにVMに無いパスを渡すとrc2で案内する():
+    result = run_thth(["approve", MISSING])
+    _assert_vm_refusal(result)
+
+
+def test_sendのtext_fileにVMに無いパスを渡すとrc2で案内する(isolated_account):
+    result = run_thth(["send", isolated_account["name"], "--text-file", MISSING])
+    _assert_vm_refusal(result)
+
+
+def test_topics_suggestのarticleにVMに無いパスを渡すとrc2で案内する(isolated_account):
+    queue_path = write_queue_file(
+        isolated_account["queue_dir"], "q.md",
+        body="## threads\n\nhello https://example.test/x\n",
+        fm_overrides={"status": "draft", "account": isolated_account["name"]})
+    result = run_thth(["topics", "suggest", queue_path, "--article", MISSING_JSON])
+    _assert_vm_refusal(result)
+
+
+def test_revokeにVMに無いパスを渡すとrc2で案内する():
+    result = run_thth(["revoke", MISSING, "--by", "test"])
+    _assert_vm_refusal(result)
+
+
+def test_previewにVMに無いパスを渡すとrc2で案内する():
+    result = run_thth(["preview", MISSING])
+    _assert_vm_refusal(result)
+
+
+def test_存在するパスは従来どおり動く(isolated_account):
+    """回帰: 存在するパスには案内を出さない（発注書「存在するパスは従来どおり」）。"""
+    queue_path = write_queue_file(
+        isolated_account["queue_dir"], "ok.md",
+        body="## threads\n\nこんにちは https://example.test/x\n",
+        fm_overrides={"status": "draft", "account": isolated_account["name"]})
+    assert os.path.exists(queue_path)
+
+    result = run_thth(["lint", queue_path])
+    assert "VM" not in (result.stdout + result.stderr)
+    assert "thth queue" not in (result.stdout + result.stderr)
+
+    result = run_thth(["preview", queue_path])
+    assert result.returncode == 0, result.stderr
+    assert "こんにちは" in result.stdout

thth-2.3.3/tests/test_collect_engagements.py

@@ -0,0 +1,175 @@
+"""絡みの台帳（`data/sns/engagements/`）を採取の commit に含める（T9-1・
+照合「X Developer Agreement と自分の泉」検収 1）。
+
+**事実**: 投稿の commit（`core.py` の `writeback.commit_and_push(rel_path=原稿)`）は
+原稿ファイルだけ、採取の commit（`collect.py` の `result["touched"]`）は採取で
+触ったファイルだけを add する。同席送信（`core._send_locked`）は commit しない。
+`thth/engagements.py::append()` が書く `data/sns/engagements/<YYYY-MM>.ndjson` は
+どこからも add されず、VM の repo に未追跡のまま溜まっていた。
+
+固定するのは 4 つ（発注 T9-1 の (a)〜(d)）:
+
+  (a) `engagements.append` で 1 行書いた repo で `collect` を 1 回 → origin に
+      `data/sns/engagements/<月>.ndjson` が push されている。
+  (b) 採取で触るものが 0 でも (a) と同じ。
+  (c) repo が無い account では git を呼ばない。
+  (d) 何も変わっていなければ空の commit を作らない。
+"""
+from __future__ import annotations
+
+import datetime
+import json
+import os
+import subprocess
+
+from tests.conftest import init_git_pair, make_queue_text
+from tests.test_collect import FakeAdapter
+from thth import accounts as accounts_mod
+from thth import collect as collect_mod
+from thth import engagements as engagements_mod
+from thth import jst
+
+NOW = datetime.datetime(2026, 9, 16, 12, 0, tzinfo=jst.JST)
+
+
+class NoOpAdapter:
+    """能力を何も名乗らない偽アダプタ（`collect_once()` に何も触らせない）。
+
+    `tests/test_collect_sent.py` の `FakeAdapter` と違い `views` すら持たない
+    ——**採取が触るものを 0 にする**ためだけの道具。
+    """
+
+    CAPABILITIES = frozenset()
+
+    @classmethod
+    def capabilities(cls):
+        return set(cls.CAPABILITIES)
+
+    def insights(self, post_id):
+        raise AssertionError("能力を名乗っていないのに insights を呼んだ")
+
+    def conversation(self, post_id, *, since=None):
+        raise AssertionError("能力を名乗っていないのに conversation を呼んだ")
+
+    def account_insights(self, user_id, *, since, until):
+        raise AssertionError("能力を名乗っていないのに account_insights を呼んだ")
+
+
+def _engagement_row(**over):
+    row = {
+        "schema": engagements_mod.SCHEMA,
+        "post_id": "111",
+        "reply_to": "222",
+        "root_post": None,
+        "author_key": None,
+        "account": None,  # append() 側の既定に任せる
+        "medium": "threads",
+        "topic": None,
+        "form": None,
+        "hour_band": "朝",
+        "posted_at": "2026-09-16T08:00:00+09:00",
+        "found_by": None,
+    }
+    row.update(over)
+    return row
+
+
+def _in_origin(bare: str) -> str:
+    return subprocess.run(
+        ["git", "-C", bare, "ls-tree", "-r", "--name-only", "main"],
+        capture_output=True, text=True,
+    ).stdout
+
+
+def test_a_絡みの台帳が採取のcommitでpushされる(tmp_path, isolated_account_factory):
+    """採取が投稿本体も触る、ふつうの回。絡みの台帳も同じ commit に乗って
+    origin まで届く。"""
+    pair = init_git_pair(tmp_path, seed_content=make_queue_text({
+        "status": "posted", "post_id": "POST1",
+        "posted_at": "2026-09-16T10:00:00+09:00"}))  # 2 時間前・1h の刻みを跨ぐ
+    account = isolated_account_factory(repo_dir=pair["work"], production=True)
+
+    cfg = accounts_mod.load_account(account["name"])
+    engagements_mod.append(cfg, account["name"],
+                            _engagement_row(account=account["name"]), now=NOW)
+
+    rc = collect_mod.run_collect(account["name"], adapter=FakeAdapter(), now=NOW,
+                                  log=lambda _l: None)
+
+    assert rc == 0
+    in_origin = _in_origin(pair["bare"])
+    assert "data/sns/engagements/2026-09.ndjson" in in_origin, in_origin
+    # ついでに、投稿の実測も同じ commit の対象として届いていること
+    # （足した処理が既存の touched を壊していないこと）。
+    assert "data/sns/insights/posts/POST1.ndjson" in in_origin, in_origin
+
+
+def test_b_採取で触るものが0でも絡みの台帳はpushされる(tmp_path, isolated_account_factory):
+    """`collect_once()` の `touched` が空でも、絡みの台帳の未追跡・変更だけで
+    commit する。"""
+    pair = init_git_pair(tmp_path, seed_content=make_queue_text({
+        "status": "posted", "post_id": "POST1",
+        # collect_days（既定 14 日）をとうに過ぎている → posts_seen は 0、
+        # touched も空のまま（NoOpAdapter は何も能力を名乗らないので
+        # アカウント日次も呼ばれない）。
+        "posted_at": "2026-08-01T10:00:00+09:00"}))
+    account = isolated_account_factory(repo_dir=pair["work"], production=True)
+
+    cfg = accounts_mod.load_account(account["name"])
+    engagements_mod.append(cfg, account["name"],
+                            _engagement_row(account=account["name"]), now=NOW)
+
+    rc = collect_mod.run_collect(account["name"], adapter=NoOpAdapter(), now=NOW,
+                                  log=lambda _l: None)
+
+    assert rc == 0
+    in_origin = _in_origin(pair["bare"])
+    assert "data/sns/engagements/2026-09.ndjson" in in_origin, in_origin
+    # 投稿側は本当に何も触っていないこと（この test が (a) の焼き直しに
+    # なっていないことの確認）。
+    assert "data/sns/insights/posts/POST1.ndjson" not in in_origin, in_origin
+
+
+def test_c_repoが無いaccountでは絡みの台帳も含めてgitを呼ばない(
+        tmp_path, isolated_account_factory, monkeypatch):
+    """`repo_dir` が実在しない台帳（`repos/_none` 相当）では、絡みの台帳の
+    判定そのもの（`_pending_engagement_paths`）を呼ばない——呼べば
+    `git status` を repo の外で走らせる危険がある。"""
+    account = isolated_account_factory(
+        repo_dir=str(tmp_path / "repos" / "_none"), production=True, scheduled=False)
+
+    def 呼ぶな(*a, **k):
+        raise AssertionError("repo が無いのに絡みの台帳の git 判定を呼んだ")
+
+    monkeypatch.setattr(collect_mod, "_pending_engagement_paths", 呼ぶな)
+    monkeypatch.setattr(collect_mod, "_git", 呼ぶな)
+    monkeypatch.setattr(collect_mod.writeback, "sync_repo", 呼ぶな)
+    monkeypatch.setattr(collect_mod.writeback, "commit_and_push", 呼ぶな)
+
+    rc = collect_mod.run_collect(account["name"], adapter=NoOpAdapter(), now=NOW,
+                                  log=lambda _l: None)
+    assert rc == 0
+
+
+def test_d_変わっていなければ空のcommitを作らない(tmp_path, isolated_account_factory,
+                                                    monkeypatch):
+    """絡みの台帳に変更が無い回は、`commit_and_push` を一切呼ばない。"""
+    pair = init_git_pair(tmp_path, seed_content=make_queue_text({
+        "status": "posted", "post_id": "POST1",
+        "posted_at": "2026-08-01T10:00:00+09:00"}))  # collect_days を過ぎている
+    account = isolated_account_factory(repo_dir=pair["work"], production=True)
+
+    calls = []
+    real_commit_and_push = collect_mod.writeback.commit_and_push
+
+    def 数える(*a, **k):
+        calls.append((a, k))
+        return real_commit_and_push(*a, **k)
+
+    monkeypatch.setattr(collect_mod.writeback, "commit_and_push", 数える)
+
+    rc = collect_mod.run_collect(account["name"], adapter=NoOpAdapter(), now=NOW,
+                                  log=lambda _l: None)
+
+    assert rc == 0
+    assert calls == [], "変更が無いのに commit_and_push を呼んだ"

thth-2.3.3/tests/test_pull_cli.py

@@ -0,0 +1,96 @@
+"""`thth pull <account|--project P>`（発注 T8-3・kopicha 続報）。
+
+読むだけの口（`queue`・`schedule`・`board`。T8-2）は遅れを**言うだけ**で
+pull しない。この道具は、その案内から誘導される**明示の**取り込み口——
+`writeback.sync_repo()` を呼ぶだけで、`thth approve` が既に通しているのと
+同じ関数・同じ repo ロックを使う。
+"""
+from __future__ import annotations
+
+import json
+
+from tests.conftest import init_git_pair, run_git, run_thth
+
+
+def test_pullはffして取り込む(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="旧", seed_name="a.md")
+    account = isolated_account_factory(repo_dir=pair["work"])
+    head_before = run_git(pair["work"], ["rev-parse", "--short", "HEAD"]).stdout.strip()
+
+    run_git(pair["seed"], ["commit", "--allow-empty", "-m", "second"])
+    run_git(pair["seed"], ["push"])
+    head_after_remote = run_git(pair["seed"], ["rev-parse", "--short", "HEAD"]).stdout.strip()
+
+    result = run_thth(["pull", account["name"]])
+    assert result.returncode == 0, result.stderr
+    assert f"取り込みました: {head_before} → {head_after_remote}（1 commit）" in result.stdout
+
+    # 本当に取り込まれている（work の HEAD が動いた）。
+    head_now = run_git(pair["work"], ["rev-parse", "--short", "HEAD"]).stdout.strip()
+    assert head_now == head_after_remote
+
+
+def test_pullはすでに最新なら何も変えない(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="", seed_name=".gitkeep")
+    account = isolated_account_factory(repo_dir=pair["work"])
+    head_before = run_git(pair["work"], ["rev-parse", "--short", "HEAD"]).stdout.strip()
+
+    result = run_thth(["pull", account["name"]])
+    assert result.returncode == 0, result.stderr
+    assert "すでに最新です" in result.stdout
+
+    head_after = run_git(pair["work"], ["rev-parse", "--short", "HEAD"]).stdout.strip()
+    assert head_after == head_before
+
+
+def test_pullのjsonはbehind_remoteの形(isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="", seed_name=".gitkeep")
+    account = isolated_account_factory(repo_dir=pair["work"])
+
+    result = run_thth(["pull", account["name"], "--json"])
+    assert result.returncode == 0, result.stderr
+    out = json.loads(result.stdout)
+    assert len(out) == 1
+    row = out[0]
+    assert row["account"] == account["name"]
+    assert row["ok"] is True
+    assert set(row["repo"].keys()) == {"behind", "ahead", "head", "fetched_at", "reason"}
+    assert row["repo"]["behind"] == 0
+
+
+def test_pullのprojectは同じrepoを共有する2accountで1回だけ引く(
+        isolated_account_factory, tmp_path):
+    pair = init_git_pair(tmp_path / "acct", seed_content="旧", seed_name="a.md")
+    account_a = isolated_account_factory("a-threads", repo_dir=pair["work"], project="shared")
+    account_b = isolated_account_factory("b-threads", repo_dir=pair["work"], project="shared")
+    assert account_a["repo_dir"] == account_b["repo_dir"]
+
+    run_git(pair["seed"], ["commit", "--allow-empty", "-m", "second"])
+    run_git(pair["seed"], ["push"])
+
+    result = run_thth(["pull", "--project", "shared", "--json"])
+    assert result.returncode == 0, result.stderr
+    out = json.loads(result.stdout)
+    # **重複なく 1 回だけ**——2 account だが repo は 1 つ。
+    assert len(out) == 1
+    assert out[0]["repo"]["behind"] == 0
+
+
+def test_pullは失敗をsync_repoの理由のままrc1(isolated_account_factory, tmp_path):
+    account = isolated_account_factory(repo_dir=str(tmp_path / "no-such-repo-dir"))
+
+    result = run_thth(["pull", account["name"]])
+    assert result.returncode == 1, result.stdout
+    assert "取り込めませんでした" in result.stderr or "見当たりません" in result.stderr
+
+
+def test_pullはaccountもprojectも無ければ断る(isolated_account_factory):
+    result = run_thth(["pull"])
+    assert result.returncode == 2
+    assert "account" in result.stderr
+
+
+def test_pullは未知のprojectを断る(isolated_account_factory):
+    result = run_thth(["pull", "--project", "no-such-project"])
+    assert result.returncode == 2
+    assert "no-such-project" in result.stderr