weflow-cli 1.9.0 → 1.11.0

package/scripts/health_check.py

@@ -10,6 +10,8 @@ thing has silently broken at least once, and the failure was invisible:
   导出格式         excel 全程从未成功过, 报错只有"导出失败"四个字
   Python 依赖     缺 sqlcipher3 时命令报的是"数据库连接失败"
   计划任务        提醒脚本静默停跑, 没人会发现
+  微信客户端      从没报过客户端版本; 而"读 exe 里的版本字面量"是错的判据(实测那只是兜底),
+                  磁盘上还可能留着一个不能用的版本目录(指向它的加载会失败)
 
 Run it ad hoc, or schedule it and read the summary line:
 
@@ -136,14 +138,41 @@ def check_environment(report):
 
 
 def pick_talkers(report):
-    """The most recently active sessions - the likeliest to span shards."""
+    """The most recently active sessions - the likeliest to span shards.
+
+    **挑不出来时要说出来**：没有会话是"没登录/没 init"（会报红），而**有会话、却一条都拿不到
+    `username`** —— 那是"字段长变了"或"列表是半截的"。这一支原来直接返回空，`main()` 里
+    `if talkers:` 于是**静默跳过**后两项检查，报告照样写"全部通过"（只是项数悄悄少了两个）。
+    同一个假绿在"计划任务"那一项也出现过（见那一段的注释）。
+    """
     _, data = run_json(['sessions', '--limit', str(SAMPLE_LIMIT)])
     sessions = ((data or {}).get('sessions') or [])
     if not sessions:
         report.add('选择样本会话', False, '会话列表为空', '先确认微信已登录且 init 成功')
         return []
     ordered = sorted(sessions, key=lambda s: s.get('sortTimestamp') or 0, reverse=True)
-    return [s['username'] for s in ordered[:2] if s.get('username')]
+    picked = [s['username'] for s in ordered[:2] if s.get('username')]
+    if not picked:
+        # 有会话却挑不出样本 ⇒ 后面两项（读取路径一致 / 导出格式）会整段不跑，
+        # 不说的话这一份报告就是"少了两个检查的全绿"
+        report.add('选择样本会话', False, f'会话有 {len(sessions)} 条，但都没有 username 字段',
+                   'sessions 的输出字段可能变了 —— 这两项检查因此没跑，不代表通过')
+    return picked
+
+
+def read_newest_json(out_dir):
+    """那个目录里 json 导出的最新消息时间（秒）；读不出来给 None。"""
+    newest = None
+    for path in Path(out_dir).glob('*.json'):
+        try:
+            rows = json.loads(path.read_text(encoding='utf-8'))
+        except (OSError, ValueError):
+            continue
+        times = [int(r.get('createTime') or 0) for r in rows]
+        if times:
+            newest = max(times)
+            break
+    return newest
 
 
 def check_read_paths(report, talker):
@@ -152,21 +181,24 @@ def check_read_paths(report, talker):
     They disagreed by 1114 messages when the JSON path read only the first
     shard, so comparing their newest message is the cheapest way to notice the
     read paths drifting apart again.
+
+    **但"最新消息"是会动的**：json 先导、html 后导，中间隔着一两分钟 —— 这期间聊天里来了
+    一条新消息，两条路径就会"不一致"，而那是**聊天活着**，不是读取断了。原来判据是精确到分钟
+    的相等（2026-10-11 实测撞到过：`json 13:38 / html 13:40:14` 判红，紧接着同一条聊天
+    `13:40 vs 13:40:14` 判绿）。放宽容差解决不了：繁忙的群里分片轮转本来就在分钟级，
+    放宽就会把这个检查唯一要抓的东西放过去。
+
+    所以不一致时**再读一次 json 做仲裁**：那一读能看到 html 那条 ⇒ 差别来自"消息是新的"；
+    看不到 ⇒ 两条路径真的不一致（这才是要报的）。仲裁只在**本来就不一致**时才花这一次导出。
     """
     with tempfile.TemporaryDirectory(prefix='wf-health-') as tmp:
         out_json = Path(tmp) / 'json'
-        code_j, err_j = run(['export', talker, 'json', '--limit', '5',
-                             '--output', str(out_json), '--non-interactive'])[0::2]
-        newest_json = None
-        for path in out_json.glob('*.json'):
-            try:
-                rows = json.loads(path.read_text(encoding='utf-8'))
-            except (OSError, ValueError):
-                continue
-            times = [int(r.get('createTime') or 0) for r in rows]
-            if times:
-                newest_json = max(times)
-                break
+        # 这次导出的退出码与 stderr **不看**：这一项判的是"两条路径读到的内容一致不一致"，
+        # 而 json 读不出来时 `read_newest_json` 会给 None、下面照样报红。留两个不用的变量
+        # 只会让"检查了什么"看起来比实际多。
+        run(['export', talker, 'json', '--limit', '5',
+             '--output', str(out_json), '--non-interactive'])
+        newest_json = read_newest_json(out_json)
 
         out_html = Path(tmp) / 'html'
         code_h, out_h, _ = run(['export', talker, 'html', '--limit', '5',
@@ -193,14 +225,37 @@ def check_read_paths(report, talker):
         return
 
     newest_txt = datetime.datetime.fromtimestamp(newest_json).strftime('%Y-%m-%d %H:%M')
-    if newest_html:
-        match = newest_html[:16] == newest_txt
-        report.add(label, match,
-                   f'json 最新 {newest_txt} / html 最新 {newest_html}',
-                   '两条路径读到不同的最新消息，通常又只剩一个分片被读了')
-    else:
+    if not newest_html:
         # No timestamp in the page (empty chat); the count comparison still holds.
         report.add(label, total_html > 0, f'json 最新 {newest_txt} / html 共 {total_html} 条')
+        return
+
+    if newest_html[:16] == newest_txt:
+        report.add(label, True, f'json 最新 {newest_txt} / html 最新 {newest_html}')
+        return
+
+    # 不一致 —— **仲裁**：再读一次 json。它能看到 html 那条，说明差别只是"消息是新的"。
+    with tempfile.TemporaryDirectory(prefix='wf-health-arbiter-') as tmp:
+        again = Path(tmp) / 'json'
+        run(['export', talker, 'json', '--limit', '5',
+             '--output', str(again), '--non-interactive'])
+        arbiter = read_newest_json(again)
+    if arbiter is not None and arbiter >= newest_json:
+        arbiter_txt = datetime.datetime.fromtimestamp(arbiter).strftime('%Y-%m-%d %H:%M')
+        if arbiter_txt == newest_html[:16]:
+            report.add(label, True,
+                       f'json 最新 {newest_txt} → 复查 {arbiter_txt} / html 最新 {newest_html}',
+                       '差别来自检查期间来了新消息（这一项防的是"只剩一个分片被读了"）')
+            return
+
+    # 仲裁没说通（复查也没读到 html 那条，或者根本没读出来）⇒ 这是真的不一致
+    if arbiter is None:
+        reason = '复查 json 也没读出来'
+    else:
+        reason = '复查 json 仍是 %s，没有更近的消息' % \
+            datetime.datetime.fromtimestamp(arbiter).strftime('%Y-%m-%d %H:%M')
+    report.add(label, False, f'json 最新 {newest_txt} / html 最新 {newest_html}',
+               '两条路径读到不同的最新消息，通常又只剩一个分片被读了（%s）' % reason)
 
 
 def html_newest_stamp(out_dir):
@@ -219,7 +274,13 @@ def html_newest_stamp(out_dir):
 
 
 def check_session_freshness(report):
-    """A read path that silently stops at an old shard shows up here first."""
+    """A read path that silently stops at an old shard shows up here first.
+
+    ⚠️ **但"最旧"与"只是没聊天"是同一个观测**：这一项看的是会话列表里最新那条消息有多旧，
+    而一个人一周没发消息，读数与"读取停在某个旧分片"**完全一样**。所以提示里必须把两种解释
+    都说出来，并且点出判别器在隔壁：`读取路径一致` 绿着 ⇒ 多半只是这段时间没聊天；
+    两项一起红才更像读取断了。**别在这里替人下结论。**
+    """
     _, data = run_json(['sessions', '--limit', '5'])
     sessions = ((data or {}).get('sessions') or [])
     stamps = [s.get('sortTimestamp') or 0 for s in sessions]
@@ -231,7 +292,9 @@ def check_session_freshness(report):
     when = datetime.datetime.fromtimestamp(newest).strftime('%Y-%m-%d %H:%M')
     report.add('会话新鲜度', age <= STALE_SESSION_DAYS,
                f'最新会话 {when}（{age:.1f} 天前）',
-               f'超过 {STALE_SESSION_DAYS} 天未更新，通常意味着读取只覆盖了部分分片')
+               f'超过 {STALE_SESSION_DAYS} 天没有新消息。两种解释：**这段时间没聊天**（最可能）'
+               f'或**读取只覆盖了部分分片** —— 看「读取路径一致」那一项：它绿着就多半是前者，'
+               f'两项一起红才更像读取断了')
 
 
 def check_export_formats(report, talker):
@@ -267,7 +330,13 @@ def check_python_deps(report):
 
 
 def check_scheduled_tasks(report):
-    """The watcher tasks must exist and their last run must have succeeded."""
+    """The watcher tasks must exist and their last run must have succeeded.
+
+    **"读不出来"必须与"上次成功"分开**：这一项的判据是从 `schtasks` 的文本输出里找
+    `Last Result` / `上次运行结果` 那一行 —— 系统语言或输出格式一变，那一行就找不到了，
+    而**两个清单都会是空的**，于是它落到 `else`、报"均在位且上次成功"。
+    那正是把"没验证"说成"通过"（同一类：`docs/DECISIONS.md` D-107 里那条"取不到就说未知"）。
+    """
     if os.name != 'nt':
         report.add('计划任务', True, '非 Windows，跳过')
         return
@@ -275,7 +344,7 @@ def check_scheduled_tasks(report):
         report.add('计划任务', True, '找不到 schtasks，跳过')
         return
     names = ['WeFlow Issue Watch', 'WeFlow Mail Watch']
-    broken, absent = [], []
+    broken, absent, unreadable, not_yet_run = [], [], [], []
     for name in names:
         try:
             proc = subprocess.run(['schtasks', '/query', '/tn', name, '/fo', 'LIST', '/v'],
@@ -287,21 +356,88 @@ def check_scheduled_tasks(report):
             absent.append(name)
             continue
         text = proc.stdout or ''
+        result = None
         for line in text.split('\n'):
             if 'Last Result' in line or '上次运行结果' in line:
                 result = line.split(':', 1)[-1].strip()
-                if result not in ('0', '0x0'):
-                    broken.append(f'{name}={result}')
+                break
+        if result is None:
+            # 输出里**没有那个字段**：要么语言/格式变了，要么它没在汇报这一项。
+            # 这时该说的是"读不出来"，不是"上次成功"。
+            unreadable.append(name)
+        elif result in ('267011', '0x41303'):
+            # `SCHED_S_TASK_HAS_NOT_RUN`（267011 / 0x41303）—— **不是失败**：注册之后还没到
+            # 第一次触发。原来它落进下面那支"上次运行失败"，提示还写着"脚本可能抛异常了"，把刚
+            # 注册的人引去查一个不存在的异常（2026-10-11 本机注册这两个任务时实测到）。
+            # 但也**不算通过**：这一项要的是"跑过一次且成功"，而任务一直"没跑过"正是它要防的事
+            # （触发条件写坏，或设成了"仅在用户登录时运行"而没人登录）。
+            not_yet_run.append(name)
+        elif result not in ('0', '0x0'):
+            broken.append(f'{name}={result}')
     if broken:
         report.add('计划任务', False, f'上次运行失败: {", ".join(broken)}',
                    '脚本可能抛异常了，手动跑一次看输出')
     elif absent:
         report.add('计划任务', False, f'不存在: {", ".join(absent)}',
                    '提醒不会触发；见 docs/HEALTH-CHECK.md 的注册命令')
+    elif unreadable:
+        # **不算通过**：这一项没被验证过（不是"上次成功"）
+        report.add('计划任务', False, f'读不出上次运行结果: {", ".join(unreadable)}',
+                   'schtasks 的输出格式或系统语言变了 —— 这一项**没有验证**，不代表成功；'
+                   '手动跑 `schtasks /query /tn "WeFlow Issue Watch" /fo LIST /v` 看一眼')
+    elif not_yet_run:
+        report.add('计划任务', False, f'还没跑过: {", ".join(not_yet_run)}',
+                   '不算通过 —— 这一项要的是"跑过一次且成功"。刚注册的话等一次触发（30 分钟）；'
+                   '一直如此就查触发条件，以及任务是不是设成了"仅在用户登录时运行"')
     else:
         report.add('计划任务', True, '均在位且上次成功')
 
 
+def check_wechat_client(report):
+    """**微信客户端版本**，以及磁盘上有没有"不能用的版本目录"。
+
+    判据在 CLI 那一侧（`src/services/wechatClient.ts`）：版本来自**活进程命令行的
+    `--user-lib-dir`** —— 不是 exe 里的版本字面量（实测那只是兜底），也不是"磁盘上有没有
+    这个版本目录"（本机就有一个只剩 1 个文件的残留目录，指向它的加载会失败）。
+
+    取不到就写"未知"，**不写"没问题"**：与仓库里那条"没找到必须报 UNKNOWN，别报 CLEAR"同一条纪律。
+    字段整个不在 = CLI 是该重建的旧版本 ⇒ **不算通过**（这正是"打开成功≠能用"那一类：
+    少了一项却一片绿）。
+    """
+    _, data = run_json(['check'])
+    if not data:
+        report.add('微信客户端', False, 'check 未返回 JSON', '先跑 weflow-cli check 看原始输出')
+        return
+    client = data.get('wechatClient')
+    if not isinstance(client, dict):
+        report.add('微信客户端', False, 'check 没报这一项',
+                   'CLI 可能没重新构建（npm run build），或 dist 是旧的')
+        return
+    version = client.get('version')
+    loaded = client.get('loadedDirs') or []
+    if version:
+        detail = str(version) + (f'（{loaded[0]}）' if loaded else '')
+    else:
+        detail = '未知 —— ' + str(client.get('note') or '')
+    dead = client.get('unusableVersionDirs') or []
+    if dead:
+        shown = '、'.join(f'{d.get("dir")}({d.get("files")} 个文件)' for d in dead)
+        detail += f'　⚠ 残留版本目录（没有 Weixin.dll，加载会失败）: {shown}'
+    # 客户端自报的数据目录 vs 配置推出来的根：不一致 = 两边可能不是同一份库（CLI 那边已按真身比过，
+    # 本机数据目录有 D 盘软链，所以这里只报结论）
+    if client.get('dataDirMatchesConfig') is False:
+        detail += '　⚠ 客户端自报的数据目录与配置里那个数据库推出来的根**不是同一份**'
+    crashes = client.get('crashRecords') or {}
+    count = int(crashes.get('reports') or 0) + int(crashes.get('records') or 0)
+    if count:
+        newest = str(crashes.get('newest') or '时间读不到')[:16]
+        detail += f'　· 客户端崩溃记录 {count} 份（最近 {newest}）—— 崩过时数据读不出来先看这条'
+    # `update.data` 在场 ⇒ 上面那个版本**未必**是加载的那个：这句必须跟着版本一起出现
+    for caveat in client.get('caveats') or []:
+        detail += f'　⚠ {caveat}'
+    report.add('微信客户端', True, detail)
+
+
 def main():
     parser = argparse.ArgumentParser(description='weflow-cli periodic health check')
     parser.add_argument('--json', action='store_true', help='机器可读输出')
@@ -314,6 +450,7 @@ def main():
     report = Report()
     check_version(report)
     check_environment(report)
+    check_wechat_client(report)
     check_python_deps(report)
     check_session_freshness(report)
 

package/scripts/nt_decrypt.py

@@ -91,7 +91,7 @@ def _is_linux_wechat_process(pid):
     if pid == os.getpid():
         return False
     try:
-        with open(f'/proc/{pid}/comm') as f:
+        with open(f'/proc/{pid}/comm', encoding='utf-8') as f:
             comm = f.read().strip().lower()
         if comm in _LINUX_WECHAT_COMMS:
             return True
@@ -122,7 +122,7 @@ def find_weixin_pid_linux():
         if not _is_linux_wechat_process(pid):
             continue
         try:
-            with open(f'/proc/{pid}/statm') as f:
+            with open(f'/proc/{pid}/statm', encoding='utf-8') as f:
                 rss_kb = int(f.read().split()[1]) * 4
         except (OSError, IndexError, ValueError):
             rss_kb = 0
@@ -205,7 +205,7 @@ def scan_memory_keys_linux(pid):
     """
     regions = []
     try:
-        with open(f'/proc/{pid}/maps') as f:
+        with open(f'/proc/{pid}/maps', encoding='utf-8') as f:
             for line in f:
                 parts = line.split()
                 if len(parts) < 2 or 'r' not in parts[1]: