test(agent): add integration tests for cross-mode consistency and KG ranking #516

AlexMikhalev · 2026-02-02T14:41:28Z

Summary

This PR adds comprehensive integration tests for the terraphim_agent crate, focusing on cross-mode consistency testing and knowledge graph ranking verification.

Changes

New Test Files

cross_mode_consistency_test.rs - Verifies search results and KG ranking are IDENTICAL across Server (HTTP API), REPL (interactive), and CLI modes
kg_ranking_integration_test.rs - Demonstrates how knowledge graphs enhance search relevance with real server calls and snapshot comparisons
repl_integration_tests.rs - Tests REPL command parsing, handler operations in server/offline modes

Documentation

test_ranking_kg.md - Knowledge graph terms for testing scenarios
pi-mono-architecture-comparison.md - Architecture analysis documentation

Fixes Applied

Fixed undefined SearchResultDoc type errors (commented out disabled CLI code)
Removed unused imports (FromStr, Context, PathBuf)
Fixed unused variable warnings
Applied 20 clippy fixes (efficiency improvements)

Test Results

✅ 134 unit tests passing
✅ All compilation warnings resolved
✅ All clippy warnings resolved
✅ Zero breaking changes

Verification

cargo test -p terraphim_agent --lib  # 134 passed
cargo check -p terraphim_agent --tests  # Clean
cargo clippy -p terraphim_agent --tests  # Clean

Add comprehensive integration tests for terraphim_agent: - cross_mode_consistency_test.rs: Cross-mode testing (Server, REPL, CLI) - kg_ranking_integration_test.rs: KG ranking impact verification - repl_integration_tests.rs: REPL command and handler tests Fix compilation errors and warnings: - Comment out SearchResultDoc references (undefined type) - Remove unused imports (FromStr, Context, PathBuf) - Fix unused variable warnings with underscore prefix - Add #[allow(dead_code)] to intentionally disabled functions - Apply 20 clippy fixes (contains() instead of iter().any(), useless vec!) Add supporting documentation: - test_ranking_kg.md: Knowledge graph terms for testing - pi-mono-architecture-comparison.md: Architecture analysis

github-actions · 2026-02-02T14:43:40Z

Documentation Preview

Your documentation changes have been deployed to:
https://860d03ee.terraphim-docs.pages.dev

This preview will be available until the PR is closed.

AlexMikhalev · 2026-02-02T14:48:57Z

GitHub Runner Execution Results

PR: #516 - test(agent): add integration tests for cross-mode consistency and KG ranking
URL: #516