Skip to content

CI hygiene (§5): scrub internal CI backend model identifier from workflow + script comments (follow-up to #100) #108

Description

@vaderyang

Goal

#100 把内部部署细节从 docs/pr-review-agent.md 和 console 注释里脱敏了,但 PR 审查(vivi,CHANGES_REQUESTED)指出同类泄露——内部 CI 后端模型标识——仍残留在 workflow 与脚本的注释/步骤名里。本 issue 收尾:把这些位置的内部后端模型标识(以及纯拓扑描述里的代理/网关工具名)替换为通用措辞(model backend / model gateway),与 #100 之后 docs/pr-review-agent.md 的用词保持一致。语义不变。

Why now

#100 的审查明确把这些残留列为 blocking;#100 的范围限于 docs/注释,没有覆盖 CI 文件。这是 §5(不向任何外发面泄露内部部署细节,含公开的 workflow/脚本)的同类问题。

残留位置(7 处,内部后端模型标识)

  • .github/workflows/pr-review.yml:4(注释)
  • .github/workflows/pr-review-probe.yml:92(step name)
  • .github/workflows/issue-implement.yml:34(注释)
  • scripts/pr-review/run_review.sh:33(注释)
  • scripts/lib/litellm-wait.sh:5(注释)
  • scripts/agent-bot/run_wiwi.sh:52(注释)
  • scripts/agent-bot/run_wiwi.sh:69(注释)

按 §5,本正文不写出该标识本身;上面每个 file:line 上就是要替换的字符串,打开即可见。

In scope

  • 仅改上述 7 处注释/步骤名:把内部后端模型标识替换为 model backendpr-review-probe.yml:92 的 step name 改成 Round-trip claude → model gateway → backend 之类。纯文本脱敏,行为不变。

Out of scope (explicit)

  • 不改 LITELLM_* config secret 名(LITELLM_BASE_URL / LITELLM_API_KEY / LITELLM_NO_PROXY)—— 改名会牵动 repo secrets 配置,属另一项工作。
  • 不改任何 workflow/脚本的行为逻辑。
  • 不动 scripts/pr-review/post_review.py 的 maintainer 默认值(独立且优先级较低,另行处理)。

Acceptance criteria

  • 维护者本地用真实标识跑 git grep -nE '<标识>' -- .github scripts 返回空。
  • 仅注释/step-name 文本变化,workflow 与脚本行为不变;scripts/lint/check-leakage.sh 仍通过、CI 绿。

Suggested approach

逐个 file:line 文本替换,用词对齐 #100 之后的 docs/pr-review-agent.mdmodel gateway / model backend)。

Files touched

  • 3 个 workflow:.github/workflows/{pr-review,pr-review-probe,issue-implement}.yml
  • 3 个脚本:scripts/pr-review/run_review.shscripts/lib/litellm-wait.shscripts/agent-bot/run_wiwi.sh

触发说明(triage)

本改动跨 .github/ + scripts/ 多文件、且是纯文本脱敏(无自然单元测试),自动 triage 很可能因 gate 3(非单一 crate/workflow)或 gate 5(无确定性测试)判 skip——这与 docs 里 body-cap 那个例子同理。若希望 wiwi 直接做这个机械替换,手动打 agent:try 覆盖即可。

Metadata

Metadata

Assignees

No one assigned

    Labels

    agent:trywiwi auto-implement queuebugSomething isn't working

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions