Skip to content

fix(governance): relay fork review events and scan non-UTF-8 history - #1

Merged
Pigbibi merged 1 commit into
mainfrom
chore/relay-and-scan-fix
Nov 7, 2026
Merged

Pigbibi merged 1 commit into
mainfrom
chore/relay-and-scan-fix

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Nov 7, 2026

Copy link
Copy Markdown
Contributor

改动内容

  • secret_scan.py:git 历史/范围校验不再把补丁输出当 UTF-8 解码。GdeiAssistant 历史中存在非 UTF-8 文件时,旧实现抛 UnicodeDecodeError 并以 exit 2 结束,导致 Gitleaks 在合法 PR 上误报失败。
  • 新增 templates/pr-review-signal.yml:无权限的信号工作流。fork 与 Dependabot PR 的 review 事件只拿到只读 token,无法发布状态;现在由 workflow_run 在受信任上下文里评估。
  • risk-gate.yml 模板:移除 pull_request_review 直连触发,改为 workflow_run: [PR review signal],并发组包含 relay 的 PR 编号。
  • runner.cjs:支持 workflow_run 目标选择(优先 pull_requests,缺失时按 head_repository:head_branch 回退)。
  • 测试:新增非 UTF-8 历史回归用例、relay 目标选择用例;CI 现在运行 test-labels 与 test-codex(此前未执行)。

验证

  • node --test 五个套件:152 通过
  • test_secret_scan.py:40 通过(真实 Gitleaks 8.30.1)
  • 用 GdeiAssistant 真实历史复现并验证修复:修复前 exit 2,修复后 exit 0
  • actionlint:模板与测试工作流无告警

@Pigbibi
Pigbibi merged commit e7e0175 into main Nov 7, 2026
2 checks passed
@Pigbibi
Pigbibi deleted the chore/relay-and-scan-fix branch November 7, 2026 11:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant