Skip to content

docs: 新增 AI 健身教练用例#47

Open
AlexAnys wants to merge 2 commits intomainfrom
feat/fitness-coach
Open

docs: 新增 AI 健身教练用例#47
AlexAnys wants to merge 2 commits intomainfrom
feat/fitness-coach

Conversation

@AlexAnys
Copy link
Copy Markdown
Owner

Summary

  • 新增 usecases/fitness-coach.md — Garmin MCP 驱动的 AI 健身教练
  • 覆盖:准备度评分、ACWR 监控、赛事信心指数、Obsidian 记忆集成
  • 国内适配:garmin.cn 中国区配置、国产手表替代方案(华为/小米)、推送渠道(钉钉/飞书)
  • 注意:仅用例文件,README 更新后续处理

Test plan

  • 格式符合仓库模板
  • 术语首次出现括号注释
  • 凭证环境变量传递
  • 代码注释中文

Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: db14fe5448

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread usecases/fitness-coach.md Outdated

### 第二步:配置 MCP

在 `~/.claude/settings.json` 中添加 Garmin MCP Server:
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Use OpenClaw config path for MCP setup

This step points users to ~/.claude/settings.json, but the rest of this repository’s OpenClaw guides configure runtime integrations via ~/.openclaw/openclaw.json (for example usecases/cn-dingtalk-ai-assistant.md lines 40-51). If users follow the current path, they configure a different client and OpenClaw won’t load the Garmin MCP server, so the workflow fails at data retrieval.

Useful? React with 👍 / 👎.

Comment thread usecases/fitness-coach.md
git clone https://github.com/iflow-mcp/borisbw-claude-fitness-cn.git

# 将技能文件复制到 OpenClaw 命令目录
cp borisbw-claude-fitness-cn/fitness-coach.md ~/.claude/commands/
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Install the skill into an OpenClaw-recognized directory

The copy command installs fitness-coach.md into ~/.claude/commands/, which is not an OpenClaw path; this repo’s OpenClaw examples use ~/.openclaw/... directories (e.g. usecases/cn-xiaohongshu-automation.md line 35). In an OpenClaw setup this leaves the command unavailable, so users cannot execute the documented /fitness-coach flow after setup.

Useful? React with 👍 / 👎.

@AlexAnys
Copy link
Copy Markdown
Owner Author

🧪 E2E 测试报告:AI 健身教练

测试时间:2026-03-29
测试环境:macOS arm64 / OpenClaw 2026.3.8 / uvx 0.9.27 / git 2.50.1
结果⚠️ PASS(有改进建议)


步骤验证

# 步骤 结果 说明
1 Garmin MCP 安装 (国际版) uvx --python 3.12 自动下载 Python 3.12,构建成功(37 packages),正确提示需先认证
2 Garmin MCP 安装 (中国区) BorisBW/garmin-mcp-cn 构建成功(27 packages),GARMIN_IS_CN=true 生效
3 MCP 配置 JSON 格式正确
4 Coach Paddy 克隆 iflow-mcp/borisbw-claude-fitness-cn 克隆成功,fitness-coach.md(12KB)存在
5 复制技能文件 ~/.claude/commands/ 目录可写
6 Garmin 认证 ⏭️ SKIP 需要 Garmin 凭证
7 /fitness-coach morning ⏭️ SKIP 依赖认证

所有外部仓库均可访问(HTTP 200)。


改进建议

1. MCP 配置路径说明

文档第二步说"在 ~/.claude/settings.json 中添加 Garmin MCP Server",第三步说"将技能文件复制到 ~/.claude/commands/"。

这两个路径是 Claude CLI / Claude Desktop 的专属路径。作为 OpenClaw 用例合集中的文档,建议:

  • 补充说明这些配置路径适用于哪些客户端(如 Claude CLI、Claude Desktop)
  • 或提供 OpenClaw 等效的配置方式(如通过 mcporter 管理 MCP 服务器)

这不是错误(配置确实能在 Claude CLI 下工作),但可能让只用 OpenClaw 的读者困惑。

2. Python 3.12 依赖说明

uvx --python 3.12 会自动下载 Python 3.12(如果本地没有),这个行为是正常的,但建议在"所需技能"章节提一句需要 Python 3.12(uvx 会自动处理),让用户有预期。

3. 小建议

  • 第一步的 garmin-mcp-auth(认证命令)和第二步 MCP 配置中的 garmin-mcp(服务器命令)是不同的入口点,文档已经区分清楚了 ✅
  • Obsidian 目录结构示例很直观 ✅
  • 国内适配章节的华为 Health Kit / 小米 API 信息实用 ✅

- MCP 配置路径区分 OpenClaw / Claude Desktop
- 技能文件路径区分 ~/.openclaw/commands/ 和 ~/.claude/commands/
- 所需技能补充 Python 3.12 说明
@AlexAnys
Copy link
Copy Markdown
Owner Author

🧪 E2E 重测报告:AI 健身教练(v2)

测试时间:2026-03-29 19:20
结果⚠️ PASS(1 个建议)


修复验证

项目 结果 说明
Python 3.12 说明 已修复 "所需技能"章节新增"Python 3.12(uvx 会自动下载,无需手动安装)"
MCP 配置路径 已修复 第二步现在列出两个路径:OpenClaw ~/.openclaw/openclaw.json / Claude ~/.claude/settings.json
~/.openclaw/openclaw.json 路径 已验证该文件存在且支持 mcpServers 配置

剩余建议

第三步的 ~/.openclaw/commands/ 目录不存在

文档改为:

OpenClaw 用户:cp borisbw-claude-fitness-cn/fitness-coach.md ~/.openclaw/commands/

~/.openclaw/commands/ 不是 OpenClaw 的约定目录。OpenClaw 的 slash 命令通过 Gateway 管理(native commands + text commands + skill commands),没有文件目录形式的自定义命令机制。

实测 ls ~/.openclaw/commands/目录不存在

Coach Paddy 的 fitness-coach.md 本质上是一个 system prompt/指令文件,它依赖 Claude CLI 的 ~/.claude/commands/ 目录约定来注册 /fitness-coach 命令。这个机制在 OpenClaw 中没有等价物。

建议

  • 方案 A:将 OpenClaw 路径改为提示用户手动创建该目录(如果 OpenClaw 未来支持)
  • 方案 B:说明 Coach Paddy 的 slash 命令目前仅适用于 Claude Code / Claude Desktop,OpenClaw 用户可以将 fitness-coach.md 内容作为提示词直接发送
  • 方案 C:将文件内容集成为 OpenClaw skill(通过 clawhub publish)

这不是阻断性问题(用户仍可手动使用提示词),但 cp ... ~/.openclaw/commands/ 这一步在纯 OpenClaw 环境下无效。

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