A linked skill folder can make chat hash an entire repository at launch
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 48/100
- Tipo di issue
- Bug
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Attiva
- Stack tecnologico
- go
- Ambito
- cli, performance
Direzione di ricerca
Start with internal/resident/skills.go at contentDigest and line 705, then inspect internal/tui3/skillpick.go:515. Reproduce the linked-folder case by calling skills.Discover and ReconcileImportedSkills, counting files passed through contentDigest, and reopening /skill. Done means bounded chat and picker work, unrelated root files are excluded from the digest, and the boundary is documented in the skills manual and invalidates.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Found on santos/dev2 at 008363c98 (#1410). It reaches dev when #1410 merges.
What happened
Discovery follows linked skill folders, while import digesting walks every regular file beneath a resolved folder. A committed .agents/skills/x -> .. link and a valid root SKILL.md make the digest cover the repository. /skill also reruns discovery on the update loop. The path is confirmed by reading the code; no slow launch was measured in the small hand setup.
Replication
Deterministic (no model). In a throwaway Git repository, write a root SKILL.md, make .agents/skills/x link to .., and add many regular files under the root. Call skills.Discover and ReconcileImportedSkills from internal/resident; count files passed through contentDigest. The count grows with unrelated repository files. Open /skill again and observe another discovery pass.
Where
internal/resident/skills.go:226, contentDigest, walks all regular files; :705 resolves a linked folder before digesting. internal/tui3/skillpick.go:515, discoveredSkills, scans on picker updates.
The fix
Bound imported-skill content to the skill's allowed folder and files, and cache or schedule picker discovery so an update cannot repeatedly walk a large tree.
Acceptance
- e2e:
codeaf chatand/skillin the linked fixture finish within a bounded time as unrelated files are added, while still listing the valid skill. - Unit: the digest does not include unrelated root files through a linked folder.
- State the link boundary in the skills manual and
invalidates.
- Lingua principale
- Go
- Stelle
- 115
- Fork
- 14
- Merge medio
- 9h 38m
- PR unite (30g)
- 730
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Agent-Field/CodeAF
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
Agent-Field/CodeAF#1489 ·
-
area:chat bug good first issue sev:papercut
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
Agent-Field/CodeAF#1470 ·
-
area:chat bug good first issue sev:papercut
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
Agent-Field/CodeAF#1469 ·
-
area:chat bug sev:papercut
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
Agent-Field/CodeAF#1468 ·
-
area:tests bug hygiene sev:papercut
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
Agent-Field/CodeAF#1460 ·
Tutte le issue di Agent-Field/CodeAF
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
bazel-contrib/rules_go#4726 · 1 commento ·
-
area/auto-scaling area/monitoring area/ops-productivity kind/enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Type/Improvement
Difficoltà 1/5 1-3 ore Idoneità per principianti 90/100
OpenNSW/nsw-srilanka#522 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 92/100