Problem
Buddy's visible buddy_observe bubble currently uses deterministic template text from templateFallback, and some of those templates sound like concrete code review findings even when Buddy only received a short summary.
Examples from src/lib/observer.ts include:
Missing error handling there.No input validation?Tests passing — nice.
When no max-mode finding is present and no diff-aware structured context was provided, these lines can read like false positives.
Why this matters
The product voice suggests Buddy is reacting to the work that just happened, but the current fallback templates can imply evidence that Buddy does not actually have. That creates trust erosion even when the system is technically behaving as designed.
Request
Adjust the fallback template pools in src/lib/observer.ts to use safer, less code-specific language unless there is real evidence to support a concrete comment.
Example direction
Instead of:
Missing error handling there.No input validation?Tests passing — nice.
Prefer something like:
Might be worth a second pass.I'd double-check the edges on that.That sounds like progress.
Scope
This is a copy/tone improvement, not a request to make Buddy diff-aware by itself. The goal is to stop the base fallback path from overclaiming.
Problem
Buddy's visible
buddy_observebubble currently uses deterministic template text fromtemplateFallback, and some of those templates sound like concrete code review findings even when Buddy only received a short summary.Examples from
src/lib/observer.tsinclude:Missing error handling there.No input validation?Tests passing — nice.When no max-mode finding is present and no diff-aware structured context was provided, these lines can read like false positives.
Why this matters
The product voice suggests Buddy is reacting to the work that just happened, but the current fallback templates can imply evidence that Buddy does not actually have. That creates trust erosion even when the system is technically behaving as designed.
Request
Adjust the fallback template pools in
src/lib/observer.tsto use safer, less code-specific language unless there is real evidence to support a concrete comment.Example direction
Instead of:
Missing error handling there.No input validation?Tests passing — nice.Prefer something like:
Might be worth a second pass.I'd double-check the edges on that.That sounds like progress.Scope
This is a copy/tone improvement, not a request to make Buddy diff-aware by itself. The goal is to stop the base fallback path from overclaiming.