agentsight-code-review

name	agentsight-code-review
description	对当前分支的变更执行 AgentSight 专属代码审查。覆盖 6 个维度：硬性规则合规、eBPF 安全、FFI 边界、Footprint Ladder、流水线测试覆盖、文档同步。输出编号的 findings 列表，每条包含文件路径和行号。

AgentSight Code Review

目标

对当前分支相对于 main 的全部变更执行代码审查，输出所有 findings。

触发时自动执行

步骤 1：收集变更

git diff origin/main..HEAD
git diff --stat origin/main..HEAD
git log --oneline origin/main..HEAD

步骤 2：按维度审查

对 diff 中每个变更文件，依次检查以下 5 个维度。不要在发现第一个问题后停止，必须遍历全部文件和全部维度。

维度 1：硬性规则合规

对照 AGENTS.md ## 0. 硬性规则：

• 非测试代码中是否使用了 unwrap() / expect() / dbg!()
• 是否添加了 #[allow(clippy::...)] 但没有注释说明
• 单个模块是否超过 500 行（不含测试），超过 2000 行的文件是否有拆分计划
• PR diff 是否超过 800 行，复杂逻辑变更是否超过 500 行

维度 2：eBPF 安全

仅当 diff 涉及 src/bpf/ 或 src/probes/ 时检查：

• BPF 程序是否兼容 kernel >= 5.8（不使用高版本才有的 helper）
• ring buffer 大小是否合理（参考现有探针配置）
• uprobe attach 的符号名是否正确，是否处理了符号不存在的情况
• BPF map 的 key/value 类型是否与 Rust 侧定义一致

维度 3：FFI 边界

仅当 diff 涉及 src/ffi.rs 或 cbindgen.toml 时检查：

• 新增/修改的 extern "C" 函数是否在 cbindgen.toml 的 after_includes 中同步声明
• FFI 类型是否标注了 #[repr(C)]
• 是否有 panic 可能穿越 FFI 边界（缺少 catch_unwind）
• 指针参数是否做了 null check

维度 4：Footprint Ladder

对照 ：