MoonshotAI / MoonshotAI/kimi-code

Unnecessary "DISREGARD any earlier skill listings" line in the system prompt

Open
#3,628 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
TypeScript
Stars
7.5k
Forks
1.2k
Avg merge
11h 53m
Merged PRs (30d)
350

Description

What version of Kimi Code is running?

Observed in source at main branch (latest commit at time of report).

Which open platform/subscription were you using?

N/A — issue identified through source-code inspection.

Which model were you using?

N/A.

What platform is your computer?

N/A.

What issue are you seeing?

Every rendered system prompt that contains skills begins with:

DISREGARD any earlier skill listings. Current available skills:

This line is hard-coded in packages/agent-core-v2/src/features/skill/catalog/registry.ts:

getModelSkillListing(): string {
  const lines = ['DISREGARD any earlier skill listings. Current available skills:'];
  ...
}

Tracing the lifecycle shows this is unnecessary: the system prompt is rendered once at agent bind time and is effectively immutable for the rest of the session. Skill listings are cached (frozenSkillListing / frozenPluginSections), compaction does not mutate the system prompt, and model switches do not re-bind the profile. There are no "earlier skill listings" to disregard on the initial bind, so the line is confusing dead weight in the rendered prompt.

What steps can reproduce the bug?
  1. Start a session with a default profile.
  2. Inspect the rendered system prompt (e.g. via kimi vis or ~/.kimi-code/sessions/<id>/agents/main/wire.jsonl).
  3. Look at the # Skills / ## Available skills section.
  4. The first line is DISREGARD any earlier skill listings. Current available skills:.
What is the expected behavior?

The skills section should list available skills directly, without a disclaimer about earlier listings, because no earlier listings exist in the current architecture.

Additional information

The only test coverage for this string is a presence assertion in packages/agent-core-v2/test/features/skill/catalog/registry.test.ts. I have a branch ready that removes the line and updates the test.

Contribution
  • I am willing to submit a PR for this bug fix myself (please wait for maintainer approval in this issue first)

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in packages/agent-core-v2/src/features/skill/catalog/registry.ts at getModelSkillListing(), then review the related assertion in packages/agent-core-v2/test/features/skill/catalog/registry.test.ts. Run that test to confirm the skills section no longer includes the unnecessary disclaimer and still lists available skills correctly.

Written by the indexing model from the issue text.

Assessment

Tech stack
typescript
Domain
cli
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Active
Clarity
Clearly specified
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.