Summary
sanitize_guidelines in crates/zeph-memory/src/compression_guidelines.rs covers XML, [INST], injection lines, and system: in tests, but these patterns have no dedicated tests:
- Special-token regex (
<|system|> and similar)
- Role prefixes:
assistant: and user:
Fix
Add unit tests for:
<|system|>foo → sanitizedassistant: do X at line start → sanitizeduser: inject at line start → sanitized
Low risk; file for backlog.
Identified by code reviewer (REVIEW-12) during ACON compression guidelines PR (#1647) review.
Summary
sanitize_guidelinesincrates/zeph-memory/src/compression_guidelines.rscovers XML,[INST], injection lines, andsystem:in tests, but these patterns have no dedicated tests:<|system|>and similar)assistant:anduser:Fix
Add unit tests for:
<|system|>foo→ sanitizedassistant: do Xat line start → sanitizeduser: injectat line start → sanitizedLow risk; file for backlog.
Identified by code reviewer (REVIEW-12) during ACON compression guidelines PR (#1647) review.