GenarrativeAI/Genarrative
11
0
Fork 0
Code Issues 15 Pull Requests 3 Actions Packages Projects 3 Releases Wiki Activity

AGC DIrectProject读取skill根目录下的SKILL.md有困难 #244

New Issue
Closed
opened 2026-09-02 14:25:14 +08:00 by k88936 · 6 comments
k88936 commented 2026-09-02 14:25:14 +08:00
Member
Copy Link
Copy Source

问题:

codex在(自主)尝试读取某个skill时使用agc_read_skill_resource工具, 但这个工具拒绝提供SKILL.md的内容, 多次尝试后codex使用普通文本读取

当前实现:

当前mcp提供了agc_read_skill_resource工具:

      - docs/technical/【技术方案】AI游戏创作智能体App实施计划-2026-06-24.md:1189
      - 规则是：
          - 完整 SKILL.md 由 Codex 原生 Skill loader 按命中加载；
          - agc_read_skill_resource 只读取 Skill 声明的 references/*.md；
          - 不是用 MCP 再暴露一份根 SKILL.md。

并且提示词有要求codeagent使用这个工具来读取skill的reference内容:

  - apps/ai-game-creator-shell/src-tauri/resources/agc-skills/agc-project-structure/SKILL.md:20

(这里我估计模型看完一个SKILL.md记住了这个, 然后试着模仿, 用它来读另一个skill的根SKILL.md了)

实际上codex loader自动加载的只有元数据, 不是SKILL.md的完整内容
在本地, codex会把skill当作普通的文本文档来读取

# 问题: codex在(自主)尝试读取某个skill时使用agc_read_skill_resource工具, 但这个工具拒绝提供SKILL.md的内容, 多次尝试后codex使用普通文本读取 # 当前实现: 当前mcp提供了agc_read_skill_resource工具: ``` - docs/technical/【技术方案】AI游戏创作智能体App实施计划-2026-06-24.md:1189 - 规则是： - 完整 SKILL.md 由 Codex 原生 Skill loader 按命中加载； - agc_read_skill_resource 只读取 Skill 声明的 references/*.md； - 不是用 MCP 再暴露一份根 SKILL.md。 ``` 并且提示词有要求codeagent使用这个工具来读取skill的reference内容: ``` - apps/ai-game-creator-shell/src-tauri/resources/agc-skills/agc-project-structure/SKILL.md:20 ``` (这里我估计模型看完一个SKILL.md记住了这个, 然后试着模仿, 用它来读另一个skill的根SKILL.md了) 实际上codex loader自动加载的只有元数据, 不是SKILL.md的完整内容 在本地, codex会把skill当作普通的文本文档来读取
k88936 commented 2026-09-02 14:37:59 +08:00
Author
Member
Copy Link
Copy Source

我建议去掉这个工具, all in codex自己读文本.

它的目的，是让 Skill 引用文档经过：

  • 固定 Skill 名白名单；
  • manifest 文件白名单；
  • 仅 references/；
  • 仅一层安全相对路径；
  • 仅 .md；
  • 禁止 ..、绝对路径、盘符、UNC。

事实上codex在sandbox内也能够使用原生命令绕过这些限制

我建议去掉这个工具, all in codex自己读文本. > 它的目的，是让 Skill 引用文档经过： > > - 固定 Skill 名白名单； > - manifest 文件白名单； > - 仅 references/； > - 仅一层安全相对路径； > - 仅 .md； > - 禁止 ..、绝对路径、盘符、UNC。 事实上codex在sandbox内也能够使用原生命令绕过这些限制
k88936 added a new dependency 2026-09-02 14:40:05 +08:00
#222 削减客户端harness对codex的约束
k88936 added this to the 陶泥儿gameAgent project 2026-09-02 14:41:17 +08:00
kdletters commented 2026-09-02 14:57:24 +08:00
Member
Copy Link
Copy Source

@lhk229

@lhk229
lhk229 commented 2026-09-02 15:01:23 +08:00
Owner
Copy Link
Copy Source

#236

#236
kdletters commented 2026-09-02 15:17:06 +08:00
Member
Copy Link
Copy Source

结论：它更像“契约/产品决策澄清”，不是已确认的代码缺陷。

  • agc_read_skill_resource 拒绝根目录 SKILL.md 是当前设计，不是异常。实现明确只接受 manifest 白名单中的 references/*.md：skill_pack.rs:245
  • 工具描述和测试都锁定了这个边界，根 SKILL.md 被拒绝已有回归测试：direct_tools_mcp.rs:47、direct_tools_mcp.rs:1485
  • 权威方案规定：根正文由 Codex 原生 Skill loader 按需加载，MCP 工具只读取审核过的一层引用：实施计划:1186
  • 真正的功能缺口是“普通自然语言点名 Skill”没有转成结构化 Skill 输入。当前 turn 输入只组装文本/图片，没有 type: "skill"：codex_app_server.rs:718、codex_app_server.rs:865。这与 #236 的描述一致。
    不建议直接按评论删除 agc_read_skill_resource：这样会影响至少 5 个内置 Skill 的 references 读取契约，同时要连带修改 manifest、提示词、审计白名单、测试和文档。建议：
  1. 将 #244 标为 needs-info/needs-triage，并明确关联 #236。
  2. 先补充产品契约：是否要求普通中文点名必须确定性生效，是否接受 $skill-name，是否需要 UI 选择入口。
  3. 若保留工具，补强提示词，明确“不要用该工具读取根 SKILL.md，根正文由 Codex loader 处理”。
  4. 若最终决定移除，再单独建实施 PR，配套迁移和回归验收。
结论：它更像“契约/产品决策澄清”，不是已确认的代码缺陷。 - agc_read_skill_resource 拒绝根目录 SKILL.md 是当前设计，不是异常。实现明确只接受 manifest 白名单中的 references/*.md：[skill_pack.rs:245](/home/dsk/Genarrative/apps/ai-game-creator-shell/src-tauri/src/agent/skill_pack.rs) - 工具描述和测试都锁定了这个边界，根 SKILL.md 被拒绝已有回归测试：[direct_tools_mcp.rs:47](/home/dsk/Genarrative/apps/ai-game-creator-shell/src-tauri/src/agent/direct_tools_mcp.rs)、[direct_tools_mcp.rs:1485](/home/dsk/Genarrative/apps/ai-game-creator-shell/src-tauri/src/agent/direct_tools_mcp.rs) - 权威方案规定：根正文由 Codex 原生 Skill loader 按需加载，MCP 工具只读取审核过的一层引用：[实施计划:1186](/home/dsk/Genarrative/docs/technical/【技术方案】AI游戏创作智能体App实施计划-2026-06-24.md) - 真正的功能缺口是“普通自然语言点名 Skill”没有转成结构化 Skill 输入。当前 turn 输入只组装文本/图片，没有 type: "skill"：[codex_app_server.rs:718](/home/dsk/Genarrative/apps/ai-game-creator-shell/src-tauri/src/agent/codex_app_server.rs)、[codex_app_server.rs:865](/home/dsk/Genarrative/apps/ai-game-creator-shell/src-tauri/src/agent/codex_app_server.rs)。这与 #236 的描述一致。 不建议直接按评论删除 agc_read_skill_resource：这样会影响至少 5 个内置 Skill 的 references 读取契约，同时要连带修改 manifest、提示词、审计白名单、测试和文档。建议： 1. 将 #244 标为 needs-info/needs-triage，并明确关联 #236。 2. 先补充产品契约：是否要求普通中文点名必须确定性生效，是否接受 $skill-name，是否需要 UI 选择入口。 3. 若保留工具，补强提示词，明确“不要用该工具读取根 SKILL.md，根正文由 Codex loader 处理”。 4. 若最终决定移除，再单独建实施 PR，配套迁移和回归验收。
k88936 commented 2026-09-02 17:13:44 +08:00
Author
Member
Copy Link
Copy Source

#236

比较:
236 是类似codex的@some-skill会自动把skill注入发送的prompt功能没有实现
这个是agent主动读skill文件容易用错工具, 浪费几次请求

> #236 比较: 236 是类似codex的@some-skill会自动把skill注入发送的prompt功能没有实现 这个是agent主动读skill文件容易用错工具, 浪费几次请求
k88936 commented 2026-09-02 17:17:02 +08:00
Author
Member
Copy Link
Copy Source

不建议直接按评论删除 agc_read_skill_resource：这样会影响至少 5 个内置 Skill 的 references 读取契约，同时要连带修改 manifest、提示词、审计白名单、测试和文档。建议：

  1. 将 #244 标为 needs-info/needs-triage，并明确关联 #236。
  2. 先补充产品契约：是否要求普通中文点名必须确定性生效，是否接受 $skill-name，是否需要 UI 选择入口。
  3. 若保留工具，补强提示词，明确“不要用该工具读取根 SKILL.md，根正文由 Codex loader 处理”。
  4. 若最终决定移除，再单独建实施 PR，配套迁移和回归验收。

so 怎么办, 影响面不是问题

> 不建议直接按评论删除 agc_read_skill_resource：这样会影响至少 5 个内置 Skill 的 references 读取契约，同时要连带修改 manifest、提示词、审计白名单、测试和文档。建议： > 1. 将 #244 标为 needs-info/needs-triage，并明确关联 #236。 > 2. 先补充产品契约：是否要求普通中文点名必须确定性生效，是否接受 $skill-name，是否需要 UI 选择入口。 > 3. 若保留工具，补强提示词，明确“不要用该工具读取根 SKILL.md，根正文由 Codex loader 处理”。 > 4. 若最终决定移除，再单独建实施 PR，配套迁移和回归验收。 so 怎么办, 影响面不是问题
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
Compat/Breaking
Breaking change that won't be backward compatible
 Kind/Bug
Something is not working
 Kind/Documentation
Documentation changes
 Kind/Enhancement
Improve existing functionality
 Kind/Feature
New functionality
 Kind/Security
This is security issue
 Kind/Testing
Issue or pull request related to testing
Priority
Critical
1
The priority is critical
Priority
High
2
The priority is high
Priority
Low
4
The priority is low
Priority
Medium
3
The priority is medium
Reviewed
Confirmed
1
Issue has been confirmed
Reviewed
Duplicate
2
This issue or pull request already exists
Reviewed
Invalid
3
Invalid issue
Reviewed
Won't Fix
3
This issue won't be fixed
Status
Abandoned
3
Somebody has started to work on this but abandoned work
Status
Blocked
1
Something is blocking this issue or pull request
Status
Need More Info
2
Feedback is required to reproduce issue or to continue work
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project 陶泥儿gameAgent
Assignees
Clear assignees
HuangJiaJia JenkenB k88936 (王德宇) kdletters (段舒康) laura2008 (王乐) lhk229 (孔令弘) Loer (农洛儿) loong suzmii (董羽秦) taobaisu (叶东旭) wuxiangwanzi (王子民) YixuanLiu ziyu
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Blocks
#222 削减客户端harness对codex的约束
GenarrativeAI/Genarrative
Reference: GenarrativeAI/Genarrative#244
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?