Refactor(architecture): Use Participants API/Models#155
Conversation
|
CLI 没有足够定义,会有新命令么。新旧命令的迁移对照方案? |
|
@GatewayJ updated for cli |
|
And this PR is only for co-review so I've made it draft. Next step will be implementing Recreating Agent warning for outdated image with inheriting skills. |
08a1230 to
f917389
Compare
|
@RussellLuo, ready to review. |
| func (a *App) registerDefaultCommands() { | ||
| a.AddCommand( | ||
| bot.NewCmd(), | ||
| participantcmd.NewCmd(), |
There was a problem hiding this comment.
This removes the existing bot config CLI entrypoint without introducing an equivalent replacement, even though /api/v1/channels/feishu/config is still supported server-side. That means existing Feishu configuration workflows will break on upgrade. If the plan is to move this under participant or channel semantics, it would be safer to add the replacement command first and only remove bot config after the migration path exists.
There was a problem hiding this comment.
Added csgclaw participant/pt config to replace bot config
|
Follow-up idea for a future PR: after the participant migration settles, it may be worth fully retiring |
Participantas the channel-visible identity model, replacing public bot/user surfaces with/api/v1/channels/{channel}/participantsAPIs, CLI commands, and docs.csgclaw participant/pt