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

Skill-suitability block leaks into the visible transcript — keep it model-context only

Offen
  • Stimmt etwas nicht?
#1,504 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
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
68/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
go
Bereich
cli, developer-experience, documentation

Rechercherichtung

Start with attachTurnSkillsLocked in internal/session/skillturn.go and the message rendering path in internal/session/chatlog.go; run the relevant skillturn tests first. Check the existing user.said assertion and turnSkillsNotice behavior, then update the manual page under internal/manual/chat/. Done means the model still receives the suitability block while every displayed transcript shows only the person's words and the specified e2e, unit, and documentation checks pass.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

area:session bug sev:papercut

What happened

Every time a turn's words match skills on the active shelf, the person's own message in the visible transcript grows a block they never asked for:

Skills suited to this message:
- Use for Codex models/pricing, scheduled tasks, skills, settings, setup, troubleshooting, customization, automations, and self-knowledge—including 'you,' 'your,' 'this app,' or 'this coding agent' when they refer to Codex—and for OpenAI APIs/products and ChatGPT Work. Also use for model choice/migration, prompting, SDKs, Responses, Realtime, agents, evals, and Chat/Work/Codex comparisons. Do not use for generic app/software tasks that merely mention Codex. [/Users/santoshkumar/.codex/skills/.system/openai-docs/SKILL.md — body in this file]
- Build multiple genuinely different versions of a UI piece you describe, rendered behind a visual picker so you can flip through them live and promote the one that feels right. Only runs when explicitly invoked; it does not trigger on its own. [/Users/santoshkumar/.agents/skills/prototype/SKILL.md — body in this file]
- Survey a codebase's animation and motion code as a senior motion advisor, then produce a prioritized audit and self-contained implementation plans for other agents (or cheaper models) to execute. Read-only on source code — it plans improvements, it does not apply them. Use when the user asks to "improve the animations", "audit the motion", "make this app feel better", or wants a roadmap of animation fixes rather than a review of a single diff. [/Users/santoshkumar/.agents/skills/improve-animations/SKILL.md — body in this file]
Earlier-listed skills win when two skills conflict.

Observed 2026-09-25 on dev@7f136b3 in an ordinary chat turn (no flags). The skill-suitability list exists to inform the model which skills fit the message's words; it is not something the person said or asked to see, yet it is spliced into the user-role message, so every surface that draws the conversation prints it.

Replication

Deterministic (no model).

The splice is pure and needs no model. attachTurnSkillsLocked appends "\n\nSkills suited to this message:\n" + block to user.message whenever turnSkills returns a non-empty block (internal/session/skillturn.go, search Skills suited to this message). With an active shelf holding one skill whose doc shares words with a message, the message content messageContentText(user.message) equals the person's words plus the block; the display path renders that same text (internal/session/chatlog.go builds chat lines from messageContentText). A developer today can run a conversation with a one-skill shelf, send a message using that skill's words, and quote the block appearing in the printed transcript — that wrong output is the evidence.

Field (real models).

Any chat turn with an active skill shelf and a message matching a skill doc; reproduces on the first matching turn. No keys beyond a normal chat's; a couple of minutes of wall time.

Where

  • internal/session/skillturn.go — attachTurnSkillsLocked (searchable string: Skills suited to this message; the splice is user.message = textMessage("user", messageContentText(user.message)+block)). The block rides inside the user-role message content, so anything that renders the conversation renders it.
  • internal/session/chatlog.go — user messages are rendered from messageContentText(...); the defect is not the skill choice but that the block lives in message text instead of a part the display path strips.
  • The person's true words are already kept separately (user.said, asserted in skillturn_test.go), and turnSkillsNotice already emits a dim skills carried: … event that surfaces may draw or ignore — so today two channels carry the same fact, one loud and visible.

The fix

The person's message in the transcript shows exactly the words they typed. Skill suitability is model context only: the block still reaches the model, but no surface prints it. Deliver it to the model in a form the display path does not render (a separate content part, or a tool-shaped entry), and use exactly one surface-visible channel — the existing dim "skills carried" notice — so the loud block and the notice never both appear.

Acceptance

  • e2e: through the real chat door with an active shelf and a matching message — the transcript the person sees renders their message as their words only: the string Skills suited to this message: appears nowhere the surface prints, while the model's copy of the message still carries the block (the existing skillturn_test.go assertion on the model's copy stays green), exit 0.
  • e2e: the control run — codeaf do --json (or the non-TUI surface) for the same turn emits no text containing Skills suited to this message:, exit 0.
  • Unit: the display path — rendering the stored user message for a turn that carried skills never includes the block; the journal/store still hold the person's words unchanged (user.said, already covered by skillturn_test.go).
  • The manual page that covers what appears in the transcript (internal/manual/chat/) quotes the new wording — the transcript shows only what the person typed — and the change entry's invalidates names what people believed before.

Drafted with CodeAF · reviewed and owned by the author

Vorherrschende Sprache
Go
Sterne
115
Forks
14
Ø Merge
9 Std. 35 Min.
Gemergte PRs (30 T.)
752

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.