Skip to content

feat: clarify on empty or ambiguous tool results#65

Merged
khaliqgant merged 2 commits intomainfrom
fix/clarify-on-empty-tool-results
Apr 26, 2026
Merged

feat: clarify on empty or ambiguous tool results#65
khaliqgant merged 2 commits intomainfrom
fix/clarify-on-empty-tool-results

Conversation

@kjgbot
Copy link
Copy Markdown

@kjgbot kjgbot commented Apr 26, 2026

Summary

  • add a harness hook for turning tool-result evidence into a targeted clarification
  • export a reusable tool-evidence clarification helper with empty, ambiguous, and transient-error heuristics
  • preserve limit enforcement and add regression coverage for malformed hook payloads and missing error codes

Test Plan

  • npm run build -w @agent-assistant/harness
  • npm test -w @agent-assistant/harness

@kjgbot kjgbot force-pushed the fix/clarify-on-empty-tool-results branch from d828343 to 174e027 Compare April 26, 2026 09:10
@khaliqgant khaliqgant merged commit c23a497 into main Apr 26, 2026
1 check passed
@khaliqgant khaliqgant deleted the fix/clarify-on-empty-tool-results branch April 26, 2026 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants