A full skill allowance still admits three retrieved skills
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
从 internal/session/skillturn.go 的第 102 行和第 140 行开始，然后使用 room zero 运行确定性的 fillSkillRoom 用例。确认 codeaf chat 恰好携带四个附加 skill，并确认 skills notice 没有检索到的额外 skill；验证 skills manual 中的上限，并将之前的溢出记录到 invalidates 中。
由索引模型根据 Issue 内容生成。
描述
Found on santos/dev2 at 008363c98 (#1410). It reaches dev when #1410 merges.
What happened
When attachments and explicitly named skills fill the four-skill turn allowance, room is zero. fillSkillRoom returns the entire retrieved list for zero, so a turn can carry up to three more skills and reach seven. This bypasses the allowance precisely when a person has filled it deliberately.
Replication
Deterministic (no model). In internal/session, test fillSkillRoom([]string{"extra-one", "extra-two", "extra-three"}, 0). Today it returns all three. Build a turn with four attached or pinned skills and matching retrieval candidates; the · skills · notice names more than four.
Field (real models). Put seven distinct valid skills in a fresh home, attach four through /skill, then send a message that matches three other descriptions. Use OPENROUTER_API_KEY and deepseek/deepseek-v4-flash; budget about a minute and a small charge. Inspect the turn's skills notice.
Where
internal/session/skillturn.go:102 computes the remaining room; fillSkillRoom at :140 returns retrieved when room <= 0.
The fix
Return no retrieved names when room is zero. Keep attached names first, then explicitly named names, then retrieval only in remaining seats.
Acceptance
- e2e:
codeaf chatwith four attached skills carries exactly those four on the next turn; the skills notice names no retrieved extras. - Unit: room zero returns an empty slice; every composition path stays within the declared cap.
- State the cap in the skills manual and record the old overflow in
invalidates.
- 主要语言
- Go
- 星标
- 115
- 派生
- 14
- 平均合并
- 9 小时 38 分钟
- 30 天内合并 PR
- 749
环境准备
我们还没有检查这个项目的环境配置文件。先看它的 README，通用步骤见我们的新手贡献指南。
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
Agent-Field/CodeAF 的其他 Issue
-
area:chat feature
难度 2/5 1-3 小时 新手友好度 88/100
Agent-Field/CodeAF#1510 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
Agent-Field/CodeAF#1489 ·
维护者通常 1 天内回复
-
area:chat bug good first issue sev:papercut
难度 2/5 1-3 小时 新手友好度 78/100
Agent-Field/CodeAF#1470 ·
维护者通常 1 天内回复
-
area:chat bug good first issue sev:papercut
难度 2/5 1-3 小时 新手友好度 85/100
Agent-Field/CodeAF#1469 ·
维护者通常 1 天内回复
-
area:chat bug sev:papercut
难度 2/5 1-3 小时 新手友好度 88/100
Agent-Field/CodeAF#1468 ·
维护者通常 1 天内回复
查看 Agent-Field/CodeAF 的全部 Issue
相似的 Issue
-
enhancement low priority
难度 2/5 1-3 小时 新手友好度 85/100
eugenioenko/ttt#674 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 84/100
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 92/100
GoogleCloudPlatform/k8s-config-connector#13462 ·
维护者通常 1 天内回复
-
bug good first issue
难度 2/5 1-3 小时 新手友好度 85/100
vavallee/bindery#2793 · 1 条评论 ·
维护者通常 1 天内回复