Skip to content

feat: add --feedback flag for search result quality feedback loop (#604) - #611

Open
lb1192176991-lab wants to merge 1 commit into
Ikalus1988:mainfrom
lb1192176991-lab:feat/search-feedback-604
Open

feat: add --feedback flag for search result quality feedback loop (#604)#611
lb1192176991-lab wants to merge 1 commit into
Ikalus1988:mainfrom
lb1192176991-lab:feat/search-feedback-604

Conversation

@lb1192176991-lab

Copy link
Copy Markdown
Contributor

What

Adds a --feedback flag to search_knowledge.py so users/agents can report whether search results were useful.

  • After displaying results, prompts: Was this helpful? (y/n/c=comment/Enter=skip)
  • Logs feedback to data/search-feedback.jsonl with: query, found_results, feedback, timestamp
  • Supports: helpful, not_helpful, comment modes
  • Gracefully handles non-interactive environments (EOFError/KeyboardInterrupt)

Why

This feeds into #574 (feedback hub) and #591 (demand board) — enabling systematic tracking of search result quality and identification of knowledge gaps.

Testing

# With feedback prompt
python3 search_knowledge.py "pip install timeout" --feedback

# Without feedback (unchanged behavior)
python3 search_knowledge.py "pip install timeout"

# No results case
python3 search_knowledge.py "nonexistent_xyz_123" --feedback
  • Verified data/search-feedback.jsonl contains valid JSONL entries
  • No PII in logs (only query + result status + feedback)
  • Existing behavior unchanged when --feedback not used

- Add --feedback flag to search_knowledge.py
- After displaying results, prompt: 'Was this helpful? (y/n/c=comment)'
- Log feedback to data/search-feedback.jsonl
- No PII in logs (only query + result info + feedback)
- No external dependencies
- Existing search behavior unchanged when --feedback not used

Closes Ikalus1988#604
@Ikalus1988 Ikalus1988 added needs-dco Missing Signed-off-by in commit risk:high High risk change, needs maintainer review shape-safe labels Jul 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-dco Missing Signed-off-by in commit risk:high High risk change, needs maintainer review shape-safe

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants