fix: clarify collaboration repository Git access errors#118
Conversation
1. 总体结论可以接受并合并。 按“本 PR 只需要增加错误提示和文档说明”的验收口径,PR #118 完成了 #104 的核心目标:保留 Git stderr/stdout、将常见 clone/fetch 失败分类为更明确的用户提示,并同步补充 Docker 容 2. 是否解决 issue #104#104 的核心要求 / 验收清单:
PR 已完成:
未完成或可疑部分:
偏离 issue 的改动:
3. 主要问题清单Blocker无。 Major无。 Minor无。 Nit无。 4. 文档与命名检查文档修改是必要的,因为 #104 明确要求说明 Docker 后端容器访问协作仓库的凭据边界。位置合理:README 放通用部署说明,quickstart 放故障排查路径,均属于稳定用户文档,不是提案或过 命名和内容边界没有问题。没有发现“当前分支未找到 X”“建议实施顺序”“实现前状态”等不应进入 main 的过程性内容。中英文 README 与 quickstart 都做了同步更新。 5. 测试与验证建议实际运行结果:
已有测试足够覆盖本 PR 的关键路径:Git stderr 保留、错误分类、轮询时 plan/task 错误展示。 6. 建议给 PR 作者的修改意见
7. 最终建议
|
摘要
本 PR 改进 HALF 协作仓库访问失败时的错误提示,避免 Git clone/fetch 失败时只显示笼统的轮询错误。
主要改动:
Closes #104
相关 Discussion:N/A
范围
在下方说明影响。
截图或录屏
本 PR 未新增 UI 组件;错误信息通过现有项目轮询状态区域展示。
迁移 / 配置影响
无数据库迁移,无新增环境变量。
文档补充了 Docker 部署时后端容器访问协作仓库的配置要求:
验证
cd src/backend && uv run python -m pytest tests/ -vcd src/frontend && npm testcd src/frontend && npm run build检查清单