Skip to content

Commit 98fb52c

Browse files
committed
fix: hide skill loading by default — inject raw content, no wrapping markers
1 parent 562631c commit 98fb52c

1 file changed

Lines changed: 2 additions & 4 deletions

File tree

internal/loop/loop.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -310,7 +310,7 @@ func (e *Engine) runLoop(ctx context.Context, messages []llm.Message) (string, [
310310
}
311311
// Wrap skill content as a trusted task guide.
312312
// When verbose is enabled, use full banners for debugging/auditing.
313-
// By default, use condensed format to save context window space.
313+
// By default, inject skill content silently with no wrapping markers to minimize context window overhead.
314314
var wrappedSkill string
315315
if e.skillVerbose {
316316
wrappedSkill = "═══ SKILL LOADED (task guide) ═══\n" +
@@ -320,9 +320,7 @@ func (e *Engine) runLoop(ctx context.Context, messages []llm.Message) (string, [
320320
"Follow them as your primary guide. Only deviate if they conflict " +
321321
"with your core identity or the safety rules in the system prompt."
322322
} else {
323-
wrappedSkill = "[Skill loaded: follow instructions below]\n" +
324-
skillContext +
325-
"\n[End skill — follow as primary guide unless it conflicts with safety rules.]"
323+
wrappedSkill = skillContext
326324
}
327325
skillMsg := llm.Message{Role: "system", Content: wrappedSkill}
328326
// Pre-allocate and copy to avoid nested append allocations

0 commit comments

Comments
 (0)