clawwork-ai/ClawWork

[Bug] Conductor prompt injection surface in room-store initConductor

Chiusa

#211 aperta il 31 mar 2026

 (1 commento) (0 reazioni) (0 assegnatari)TypeScript (74 fork)github user discovery
area/coregood first issuehelp wantedkind/bugsecurity

Metriche repository

Star
 (525 stelle)
Metriche merge PR
 (Merge medio 22g) (2 PR mergiate in 30 g)

Descrizione

Summary

packages/core/src/stores/room-store.tsinitConductor() concatenates agentCatalog (from gateway) and userMessage (from user input) directly into the conductor system prompt without any escaping, validation, or length limits.

let prompt = buildConductorPrompt(agentCatalog);
if (userMessage) {
  prompt += `\n\n---\nUser task:\n${userMessage}`;
}

Expected behavior

  • userMessage should be truncated to a reasonable max length
  • agentCatalog should be validated for expected structure before embedding
  • Consider wrapping user content in a fenced block to reduce injection surface

Files

  • packages/core/src/stores/room-store.tsinitConductor() (line ~97)
  • packages/shared/src/constants.tsbuildConductorPrompt()

Context

Introduced in PR #210.

Guida contributor