Hacktoberfest 2026: die Issues, die Maintainer für den Oktober markiert haben – offen und einsteigerfreundlich. Hacktoberfest-Issues durchsuchen

A full skill allowance still admits three retrieved skills

Offen Anfängerfreundlich
  • Stimmt etwas nicht?
#1,450 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
Ein halber Tag
Anfängerfreundlichkeit
88/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
go
Bereich
cli

Rechercherichtung

Beginne in internal/session/skillturn.go bei den Zeilen 102 und 140, führe dann den deterministischen fillSkillRoom-Fall mit Raum null aus. Bestätige, dass codeaf chat genau vier angehängte Skills enthält und dass der Skills-Hinweis keine abgerufenen zusätzlichen Skills enthält; überprüfe das Limit im Skills-Handbuch und vermerke den vorherigen Überlauf in invalidates.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

area:session bug sev:papercut

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 chat with 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.
Vorherrschende Sprache
Go
Sterne
115
Forks
14
Ø Merge
16 Std. 30 Min.
Gemergte PRs (30 T.)
238

Entwicklungsumgebung

Die Einrichtungsdateien dieses Projekts haben wir noch nicht geprüft. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus Agent-Field/CodeAF

Alle Issues in Agent-Field/CodeAF

Ähnliche Issues

Weitere Issues zu Go

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.