SuperAPI AI 工具集

factory-review

安全与审计

查看项目

更新于 10-16 17:39

技能介绍

Factory Review 是一个用于审查拉取请求(PR)的技能,以熟悉 Cloudflare Workers、Durable Objects、Cloudflare Workflows 以及 Flue agent 框架的资深工程师视角进行代码审查。该技能针对的代码库是一个由 webhook 驱动的 Worker,它针对 GitHub 仓库编排持久化工作流和沙箱化 AI agent。

审查重点

控制流的正确性:工作流步骤不得嵌套在其他步骤内部。步骤结果会被持久化并重放,因此 step.do 返回的任何内容都必须可 JSON 序列化且保持稳定。需要识别那些在重试或重放时会表现不同的逻辑。

凭据卫生:GitHub token 绝不能到达 agent、被持久化到步骤状态、写入 git config,或在日志与错误信息中以未脱敏形式出现。短期作用域 token 应仅保留在使用它的单个步骤内。

Webhook 驱动的边界情况:投递会被重试,可能并发或乱序到达。需要识别缺失的幂等性、每个实体协调器未覆盖的竞态,以及自我触发循环(机器人对自身评论、标签或推送作出反应)。

不可信输入:issue 与 PR 文本、评论、仓库文件内容以及 LLM 输出均为不可信。需要标记任何未经验证或未加引号就将其插入 shell 命令、URL 或 API 调用的情况。

Schema/契约漂移:Valibot schema、工作流参数、agent 初始数据及其 TypeScript 类型必须保持一致,包括与协调器已持久化状态的向后兼容性。

无需优先关注的内容

现有代码未遵循的风格偏好;对与已测试代码机械相同的代码提出测试覆盖建议;需要攻击者已控制 Worker 密钥的假设性场景。

严重性分级

critical:凭据泄露、沙箱外的远程代码执行,或协调器状态的数据丢失/损坏。

high:破坏某能力正常路径或向 GitHub 重复发布的 bug。

medium:重试、重复投递或异常但可达输入下的不正确行为。

low:其他值得提及的问题。

仅在阅读周边代码后对确有把握的发现进行报告。简短而正确的审查优于冗长而推测性的审查。