Skip to content

test: Case 2 (mixed tools) integration tests for recall interception#141

Merged
BYK merged 1 commit intomainfrom
test/case2-recall-integration
May 7, 2026
Merged

test: Case 2 (mixed tools) integration tests for recall interception#141
BYK merged 1 commit intomainfrom
test/case2-recall-integration

Conversation

@BYK
Copy link
Copy Markdown
Owner

@BYK BYK commented May 7, 2026

Summary

  • Adds 2 end-to-end integration tests for Case 2 recall interception (mixed tools: recall + other tool_use in the same response)
  • Exercises the full pipeline flow: accumulator suppression → recall extraction → response stripping → pending storage → next-request injection
  • Covers single other tool (recall + Read) and multiple other tools (recall + Read + Bash) scenarios

Testing

All 161 gateway tests pass (159 existing + 2 new).

End-to-end tests that exercise the full Case 2 pipeline flow:
accumulator suppresses recall blocks with re-indexing, pipeline extracts
recall tool_use from accumulated response, strips recall for post-processing,
stores pending recall, and injects it into the next request's conversation
history with correct tool_use/tool_result ordering.

Two scenarios tested:
- Single other tool (recall + Read)
- Multiple other tools (recall + Read + Bash)
@BYK BYK merged commit bcbde61 into main May 7, 2026
1 check failed
@BYK BYK deleted the test/case2-recall-integration branch May 7, 2026 19:32
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.

1 participant