ywxzm03 commented on PR #933:
URL: https://github.com/apache/dubbo-go-pixiu/pull/933#issuecomment-4666437748
> 我看了当前 head，并对照了已有 review comments；之前关于
`mapType`、`opt.values`/`opt.types`、`resources[].source.content`
条件必填等问题看起来已经在当前版本里处理了。还有几个未覆盖的正确性点建议补一下：
>
> 1. `skills/pixiu-filter-author/SKILL.md` 的 Workflow 只要求实现 SPI 和在
`pkg/pluginregistry/registry.go` 加 blank import，但没有明确要求插件包在 `init()` 里注册。blank
import 只是触发 `init()`；HTTP filter 仍需要
`filter.RegisterHttpFilter(&Plugin{})`，Network filter 需要
`filter.RegisterNetworkFilterPlugin(&Plugin{})`。否则 agent 可能生成可编译、也被 blank
import 的 filter，但运行时没有注册到 Pixiu。
> 2. 同一个 skill 里 `Kind` 现在只描述成 constant/YAML name，缺少按当前 Pixiu Kind
约定校验的要求。建议明确生成前必须对照 `pkg/common/constant/key.go`：新增 Kind 应放在包内 `const Kind =
...`，YAML `name` 使用同一个值，并避免无效前缀/未注册名称。否则配置可解析，但 filter 可能匹配不上。
> 3. `skills/pixiu-mcp-gateway/SKILL.md` 会引导生成 `resources[].source` 的
inline/file/url/template 配置，但当前 `resources/read` 仍是 TODO，占位返回 `Resource content
for ...`，没有实际读取 `source.content/path/url/template`。建议 skill 标注这是当前 runtime
limitation，或暂时不要让 agent 承诺资源读取会返回配置的真实内容。
>
> 整体方向我觉得是对的，主要是这些会导致“生成内容看起来完整，但运行不生效”的约束最好补回。
fixed
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
