Skip to content

升级 wecom-notifier 以修复超长通知分段(上游 #2 #3) #195

Closed

Description

@zj1123581321

背景

PR #194 把总结正文和"完成"合并成一条通知,超长正文的分段完全交给 wecom-notifier 库处理。目前库里有两个分段缺口,修复不在业务层做:

这两个缺口在 PR #194 之前就已存在:旧的 send_long_text 也是直接调用 send_rich。

影响

生产总结长度:p50 4610 字,p99 12650 字。

  • 飞书:总结约超过 1 万字时会被拒收。
  • 企业微信:正文里出现超长单行时,个别分段会超限。

待办

  • 上游两个 issue 修复并发布新版本后,把本仓 pyproject.toml 里的 wecom-notifier 版本下限升到修复版。
  • 升级后用一条超长总结做一次真实推送验证。

Refs #194

Activity

  1. added
    优先级:P2质量债、待拍板的产品决策、可选优化
    落点:本仓修复代码在本仓 git 跟踪路径内
    on Oct 10, 2026
  2. zj1123581321 commented on Oct 10, 2026

    @zj1123581321
    CollaboratorAuthor

    分诊核实(2026-10-10):上游阻塞已解除——zlxlabs/wecom-notifier#2、#3 均由 PR #7(main 合并 a97e29dc)修复关闭,PyPI 已发布 0.3.2(2026-10-08T12:26 上传,含两修复)。本仓 pyproject.toml:35 仍是 wecom-notifier>=0.3.1。第一项待办(升版本下限)已派 quick 卡;第二项真实推送验证需主 checkout 未跟踪的 config/config.jsonc 里的真实 webhook,worktree 卡无法执行,升级合并后由 owner/主脑侧做一次超长总结真推。

  3. zj1123581321 commented on Oct 10, 2026

    @zj1123581321
    CollaboratorAuthor

    升级已合并(2026-10-10):

    • PR 将 wecom-notifier 升至 0.3.2 并锁定超长分段预算 #221(merge commit 59ee6ca):pyproject.toml:35 → wecom-notifier>=0.3.2,uv.lock 仅动该包
    • 新增离线回归 tests/unit/test_wecom_notifier_chunking.py:短行+4.5KB 超长单行、≥10KB 输入走公开 MessageSegmenter.segment(..., "markdown_v2"),断言每段 ≤3800 字节;base 红验抽查通过(0.3.1 上同测试 FAILED,0.3.2 绿——测试能区分坏/好版本)
    • 待办一(升版本下限)✅ 完成;待办二(超长总结真实推送验证)仍开放:需主 checkout 未跟踪 config/config.jsonc 的真实 webhook,worktree 无法执行,由 owner 侧下次真实任务推送时顺带核验(飞书卡片应正常分页、企微 markdown 不再超限段),验证完成即可关单
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    优先级:P2质量债、待拍板的产品决策、可选优化落点:本仓修复代码在本仓 git 跟踪路径内

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions