70 lines
2.5 KiB
Markdown
70 lines
2.5 KiB
Markdown
# Phase 1: P0 紧急修复 - Discussion Log
|
||
|
||
> **Audit trail only.** Do not use as input to planning, research, or execution agents.
|
||
> Decisions are captured in CONTEXT.md — this log preserves the alternatives considered.
|
||
|
||
**Date:** 2026-03-27
|
||
**Phase:** 01-P0 紧急修复
|
||
**Areas discussed:** CRITICAL-06 修复范围, Bug 执行顺序与分组, Bootstrap 动态限制
|
||
|
||
---
|
||
|
||
## CRITICAL-06 修复范围
|
||
|
||
| Option | Description | Selected |
|
||
|--------|-------------|----------|
|
||
| 逐一语义审查，按场景修复 | 对每处确认"卖家是谁"再决定是否改；有些场景 buyerCostPrice 可能是正确的 | ✓ |
|
||
| 模式匹配全量改为 operatorCostPrice | 出现即改，快但风险高；场景 1/2/3 可能并不需要改 | |
|
||
|
||
**User's choice:** 逐一语义审查
|
||
**Notes:** 审查依据为修正业务文档中的五种购买场景对应表
|
||
|
||
---
|
||
|
||
## Bug 执行顺序与分组
|
||
|
||
| Option | Description | Selected |
|
||
|--------|-------------|----------|
|
||
| 按优先级顺序分 2 批 | 第 1 批主链路（01→02→03→04→05），第 2 批独立 Bug（06/07/08） | ✓ |
|
||
| 全部串行，一个 agent 按顶部清单执行 | 简单，但失败时难定位哪一步 | |
|
||
|
||
**每个 Bug 一个 commit：**
|
||
| Option | Description | Selected |
|
||
|--------|-------------|----------|
|
||
| 每个 Bug 一个 commit | 回滚粒度细 | ✓ |
|
||
| 每批 Bug 一个 commit | 提交记录少 | |
|
||
|
||
**User's choice:** 分 2 批，每个 Bug 独立 commit
|
||
**Notes:** 无
|
||
|
||
---
|
||
|
||
## Bootstrap 动态限制
|
||
|
||
| Option | Description | Selected |
|
||
|--------|-------------|----------|
|
||
| 分开处理，各自一个 commit | CRITICAL-03 改 polling handler 注入；CRITICAL-04 改 recharge service + AutoPurchase 注册 | ✓ |
|
||
| 合并处理，一个 commit | Worker 初始化路径相关 | |
|
||
|
||
**API vs Worker bootstrap 确认：**
|
||
经代码确认：`recharge.Service` 只在 API 侧 `services.go` 初始化，`QueueClient` 已在 `deps` 中。Worker 侧 `worker_services.go` 不需要改。
|
||
|
||
| Option | Description | Selected |
|
||
|--------|-------------|----------|
|
||
| 只改 worker bootstrap（worker_services.go） | | |
|
||
| 确认 API 侧是否也用到 recharge.Service | 经查：recharge 在 API services.go，不在 worker | ✓ |
|
||
|
||
**User's choice:** API services.go + worker handler.go 各改一处，两个独立 commit
|
||
**Notes:** `QueueClient` 已在 `deps` 中，无需新增字段
|
||
|
||
---
|
||
|
||
## the Agent's Discretion
|
||
|
||
- 日志消息使用中文
|
||
- 具体代码实现参照规格书提供的代码片段
|
||
|
||
## Deferred Ideas
|
||
|
||
None
|