Skip to content

fix(chapter13): repair trip planner health check - #899

Open
Ling-ye wants to merge 1 commit into
datawhalechina:recovery/pr-base-20260917from
Ling-ye:fix/ch13-trip-health-reapply
Open

Ling-ye wants to merge 1 commit into
datawhalechina:recovery/pr-base-20260917from
Ling-ye:fix/ch13-trip-health-reapply

Conversation

@Ling-ye

@Ling-ye Ling-ye commented Sep 17, 2026

Copy link
Copy Markdown

这是已关闭 PR #824 的重新提交。2026-09-17 上游 main 历史重写后,旧 PR 与新 main 失去共同祖先;本分支已从当前 main 重新构建,并重新完成验证。

问题

get_trip_planner_agent() 返回 MultiAgentTripPlanner,但健康检查访问了不存在的统一 .agent 属性。规划器成功初始化后,/trip/health 仍会因 AttributeError 返回 HTTP 503。

修改

  • attraction_agentweather_agenthotel_agentplanner_agent 四个真实组件读取工具清单。
  • 对组件间共享的同名工具去重后计算 tools_count
  • 使用实际规划器类型名填充现有 agent_name 字段。
  • 保持响应字段和初始化失败时的 HTTP 503 行为不变。

验证

  • 四组件离线替身下,健康检查返回 HTTP 200 语义、agent_name=StubMultiAgentTripPlannertools_count=3
  • 工厂抛出 RuntimeError("initialization failed") 时仍转换为 HTTP 503,并保留错误详情。
  • 核对 hello-agents==0.2.40.2.9 wheel:SimpleAgent.list_tools() 均返回工具名称列表。
  • 路由文件通过 Python 编译检查。
  • git diff --check:通过。

验证边界

验证未初始化真实 LLM,也未连接 MCP 或地图服务,因此不代表完整旅行规划端到端验证。

@jjyaoao
jjyaoao changed the base branch from main to recovery/pr-base-20260917 September 18, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant