Skip to content

docs(patrol): admin API cron paths, restart endpoint, API key users, CardKit v2#475

Closed
aaronwong1989 wants to merge 1 commit into
hrygo:mainfrom
aaronwong1989:docs/patrol-2026-05-22
Closed

docs(patrol): admin API cron paths, restart endpoint, API key users, CardKit v2#475
aaronwong1989 wants to merge 1 commit into
hrygo:mainfrom
aaronwong1989:docs/patrol-2026-05-22

Conversation

@aaronwong1989
Copy link
Copy Markdown
Contributor

Summary

Test plan

  • make docs-build passes (55 documents, 0 assets)
  • Verify admin-api.md cron paths match current cmd/hotplex/routes.go
  • Verify new endpoint documentation matches handler signatures

🤖 Generated with Claude Code

…l docs

- admin-api.md: fix cron paths /api/cron/ → /admin/cron/, add restart
  endpoint, add API Key User Management section (5 endpoints)
- feishu-integration.md: update Q&A interaction to describe CardKit v2
  copy_text buttons instead of text-only response
- security-model.md: add APIKeyResolver enterprise multi-user session
  isolation to Layer 2 description

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Copy link
Copy Markdown
Owner

@hotplex-ai hotplex-ai left a comment

Choose a reason for hiding this comment

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

审查摘要

文档更新 PR,3 个文件,+47/-17。

审查通过项

  • Cron API 路径修正 /api/cron//admin/cron/#470 breaking change 一致
  • 新增 POST /admin/restart 端点文档(含异步延迟、独立 PGID 隔离说明)
  • API Key 用户管理 5 个 CRUD 端点文档完整(含 501/503 边界说明)
  • Security Model Layer 2 补充 APIKeyResolver 企业模式描述
  • 飞书集成教程更新 Q&A 交互描述匹配 CardKit v2 copy_text

微瑕(不阻塞)

  • PATCH /admin/api-keys/{id}**DELETE /admin/api-keys/{id}** 结尾多余反引号,与已有 PUT /admin/bots/... 行格式一致但属于预存问题

测试项中未勾选的两个验证项建议在 merge 前由作者确认。

@aaronwong1989
Copy link
Copy Markdown
Contributor Author

变更已 cherry-pick 到 #474,合并为一个 PR。

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.

2 participants