fc24fb699b
The one-liner entry docs were untouched by the backend upgrade — new users following "帮我安装 Agent Reach" would still get the old world. Now: - docs/install.md: xiaohongshu section rewritten (desktop OpenCLI with the one manual extension click / server xiaohongshu-mcp with QR + 150MB first-run warning / legacy xhs-cli kept as fallback); Reddit section says plainly there is no zero-config path; optional-channel menu gains OpenCLI; upstream table matches the new routing with a doctor --json pointer - docs/update.md: full rewrite — upgrade-only-what-exists tool refresh (incl. npm update for OpenCLI/mcporter, pinned git source for rdt-cli), one-time OpenCLI offer for desktops, skill sync now rides doctor's auto-sync instead of an inline python blob, WeChat-era content removed - check-update / watch: "update available" now leads with the one-liner (帮我更新 Agent Reach:<update.md>) which updates package + tools + skill; bare pip stays as the package-only alternative - fix: version compare was `!=` — a local build AHEAD of the latest release (installed from main during a release window) was told to "update" into a downgrade; now semantic tuple compare with fallback (+4 tests) - skill (zh+en): description gains a NOT-for boundary (fetch-only, no write ops, defer to dedicated skills); new standing rule — after substantial tasks run check-update and append a one-liner update hint to the wrap-up, never interrupting or nagging - SKILL_en.md restructured to the same three-layer skeleton as the Chinese version (124-line body, shared references/) instead of a 263-line everything-inline page Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
202 lines
7.0 KiB
Python
202 lines
7.0 KiB
Python
# -*- coding: utf-8 -*-
|
|
"""Tests for Agent Reach CLI."""
|
|
|
|
import shutil
|
|
import subprocess
|
|
from unittest.mock import patch
|
|
|
|
import pytest
|
|
import requests
|
|
import agent_reach.cli as cli
|
|
from agent_reach.cli import main
|
|
|
|
|
|
class TestCLI:
|
|
def test_version(self, capsys):
|
|
with pytest.raises(SystemExit) as exc_info:
|
|
with patch("sys.argv", ["agent-reach", "version"]):
|
|
main()
|
|
assert exc_info.value.code == 0
|
|
captured = capsys.readouterr()
|
|
assert "Agent Reach v" in captured.out
|
|
|
|
def test_no_command_shows_help(self, capsys):
|
|
with pytest.raises(SystemExit) as exc_info:
|
|
with patch("sys.argv", ["agent-reach"]):
|
|
main()
|
|
assert exc_info.value.code == 0
|
|
|
|
def test_doctor_runs(self, capsys):
|
|
with patch("sys.argv", ["agent-reach", "doctor"]):
|
|
main()
|
|
captured = capsys.readouterr()
|
|
assert "Agent Reach" in captured.out
|
|
assert "✅" in captured.out
|
|
|
|
def test_transcribe_command_prints_text(self, capsys):
|
|
with patch("agent_reach.transcribe.transcribe", return_value="hello transcript"):
|
|
with patch("sys.argv", ["agent-reach", "transcribe", "audio.mp3"]):
|
|
main()
|
|
captured = capsys.readouterr()
|
|
assert "hello transcript" in captured.out
|
|
|
|
def test_transcribe_command_writes_output_file(self, capsys, tmp_path):
|
|
out_file = tmp_path / "t.txt"
|
|
with patch("agent_reach.transcribe.transcribe", return_value="saved text"):
|
|
with patch("sys.argv", ["agent-reach", "transcribe", "audio.mp3", "-o", str(out_file)]):
|
|
main()
|
|
assert out_file.read_text(encoding="utf-8").strip() == "saved text"
|
|
assert "Transcript written" in capsys.readouterr().out
|
|
|
|
def test_parse_twitter_cookie_input_separate_values(self):
|
|
auth_token, ct0 = cli._parse_twitter_cookie_input("token123 ct0abc")
|
|
assert auth_token == "token123"
|
|
assert ct0 == "ct0abc"
|
|
|
|
def test_parse_twitter_cookie_input_cookie_header(self):
|
|
auth_token, ct0 = cli._parse_twitter_cookie_input(
|
|
"auth_token=token123; ct0=ct0abc; other=value"
|
|
)
|
|
assert auth_token == "token123"
|
|
assert ct0 == "ct0abc"
|
|
|
|
def test_install_rdt_cli_prefers_github_source(self, monkeypatch, capsys):
|
|
state = {"rdt_installed": False}
|
|
commands = []
|
|
|
|
def fake_which(name):
|
|
if name == "rdt":
|
|
return "/usr/local/bin/rdt" if state["rdt_installed"] else None
|
|
if name == "pipx":
|
|
return "/usr/local/bin/pipx"
|
|
return None
|
|
|
|
def fake_run(cmd, **kwargs):
|
|
commands.append(cmd)
|
|
state["rdt_installed"] = True
|
|
return subprocess.CompletedProcess(cmd, 0, "", "")
|
|
|
|
monkeypatch.setattr(shutil, "which", fake_which)
|
|
monkeypatch.setattr(subprocess, "run", fake_run)
|
|
|
|
cli._install_rdt_cli()
|
|
|
|
out = capsys.readouterr().out
|
|
assert commands == [["pipx", "install", cli._RDT_GIT_SOURCE]]
|
|
assert "✅ rdt-cli installed" in out
|
|
|
|
def test_install_reddit_deps_routes_by_environment(self, monkeypatch):
|
|
"""桌面 → OpenCLI;服务器 → rdt-cli(钉 git 源)。"""
|
|
calls = []
|
|
monkeypatch.setattr(cli, "_install_opencli_deps", lambda: calls.append("opencli"))
|
|
monkeypatch.setattr(cli, "_install_rdt_cli", lambda: calls.append("rdt"))
|
|
monkeypatch.setattr(shutil, "which", lambda _: None)
|
|
|
|
monkeypatch.setattr(cli, "_detect_environment", lambda: "local")
|
|
cli._install_reddit_deps()
|
|
assert calls == ["opencli"]
|
|
|
|
calls.clear()
|
|
monkeypatch.setattr(cli, "_detect_environment", lambda: "server")
|
|
cli._install_reddit_deps()
|
|
assert calls == ["rdt"]
|
|
|
|
|
|
class TestCheckUpdateRetry:
|
|
def test_retry_timeout_classification(self):
|
|
sleeps = []
|
|
|
|
def fake_sleep(seconds):
|
|
sleeps.append(seconds)
|
|
|
|
with patch("requests.get", side_effect=requests.exceptions.Timeout("timed out")):
|
|
resp, err, attempts = cli._github_get_with_retry(
|
|
"https://api.github.com/test",
|
|
timeout=1,
|
|
retries=3,
|
|
sleeper=fake_sleep,
|
|
)
|
|
|
|
assert resp is None
|
|
assert err == "timeout"
|
|
assert attempts == 3
|
|
assert sleeps == [1, 2]
|
|
|
|
def test_retry_dns_classification(self):
|
|
error = requests.exceptions.ConnectionError("getaddrinfo failed for api.github.com")
|
|
with patch("requests.get", side_effect=error):
|
|
resp, err, attempts = cli._github_get_with_retry(
|
|
"https://api.github.com/test",
|
|
retries=1,
|
|
sleeper=lambda _x: None,
|
|
)
|
|
assert resp is None
|
|
assert err == "dns"
|
|
assert attempts == 1
|
|
|
|
def test_retry_rate_limit_then_success(self):
|
|
sleeps = []
|
|
|
|
class R:
|
|
def __init__(self, code, payload=None, headers=None):
|
|
self.status_code = code
|
|
self._payload = payload or {}
|
|
self.headers = headers or {}
|
|
|
|
def json(self):
|
|
return self._payload
|
|
|
|
sequence = [
|
|
R(429, headers={"Retry-After": "3"}),
|
|
R(200, payload={"tag_name": "v1.5.0"}),
|
|
]
|
|
|
|
with patch("requests.get", side_effect=sequence):
|
|
resp, err, attempts = cli._github_get_with_retry(
|
|
"https://api.github.com/test",
|
|
retries=3,
|
|
sleeper=lambda s: sleeps.append(s),
|
|
)
|
|
|
|
assert err is None
|
|
assert resp is not None
|
|
assert resp.status_code == 200
|
|
assert attempts == 2
|
|
assert sleeps == [3.0]
|
|
|
|
def test_classify_rate_limit_from_403(self):
|
|
class R:
|
|
status_code = 403
|
|
headers = {"X-RateLimit-Remaining": "0"}
|
|
|
|
@staticmethod
|
|
def json():
|
|
return {"message": "API rate limit exceeded"}
|
|
|
|
assert cli._classify_github_response_error(R()) == "rate_limit"
|
|
|
|
def test_check_update_reports_classified_error(self, capsys):
|
|
with patch("agent_reach.cli._github_get_with_retry", return_value=(None, "timeout", 3)):
|
|
result = cli._cmd_check_update()
|
|
|
|
captured = capsys.readouterr()
|
|
assert result == "error"
|
|
assert "网络超时" in captured.out
|
|
assert "已重试 3 次" in captured.out
|
|
|
|
|
|
class TestVersionCompare:
|
|
def test_newer_remote_triggers_update(self):
|
|
assert cli._is_newer_version("1.5.0", "1.4.2") is True
|
|
|
|
def test_equal_versions_no_update(self):
|
|
assert cli._is_newer_version("1.5.0", "1.5.0") is False
|
|
|
|
def test_local_ahead_of_release_no_downgrade_prompt(self):
|
|
"""发版窗口期本地装了 main(更新)时,不能提示"有更新"诱导降级。"""
|
|
assert cli._is_newer_version("1.4.2", "1.5.0") is False
|
|
|
|
def test_unparseable_falls_back_to_inequality(self):
|
|
assert cli._is_newer_version("2026.06-beta", "1.5.0") is True
|
|
assert cli._is_newer_version("1.5.0", "1.5.0-dev") is True
|