agent 发现的每处问题都可以被继续讨论
让 code review 更加通透彻底
| 17 | for (const seg of segments) { | |
| 18 | − | await this.encode(seg); |
| 19 | − | done += 1; |
| 18 | ＋ | this.spawn(async () => { |
| 19 | ＋ | await this.encode(seg); |
| 20 | ＋ | counter.set(counter.get() + 1); // track progress |
| 21 | ＋ | }); |
| 22 | } | |
| 23 | async retry(task: Task) { | |
| 22 | − | while (true) await this.run(task); |
| 24 | ＋ | while (!task.done) await this.run(task); |
Cell<usize> 不是线程安全的,不能跨 spawn 共享;每个 task 在独立线程执行,counter.set 无原子性也不满足 Sync。
| 18 | − | await this.encode(seg); |
| 19 | − | done += 1; |
| 18 | ＋ | this.spawn(async () => { |
| 19 | ＋ | await this.encode(seg); |
| 20 | counter.set(counter.get() + 1); // track progress |
Cell<usize> 不是线程安全的,不能跨 spawn 共享;每个 task 在独立线程执行,counter.set 无原子性也不满足 Sync。用 Arc<AtomicUsize> + fetch_add(1, Ordering::Relaxed) 替代。
Reviewer 和 agent 同步进行审查，agent 扫出的 finding 会逐步落到 diff 上。
每条 finding 都可以被继续讨论；行内 ✎ 或框选一段代码也可新开讨论。彻底搞懂实现细节。
每轮换一个新会话全量重扫，复核之前发现的问题，同时对于 Reviewer 或者作者明确表态的意见处置会被保留。
人工筛一遍真正要提交的内容，提交成 GitHub PR review。本地分支就导出成一份 Markdown 报告。