Skip to content

Conversation

@RichardHightower
Copy link
Contributor

Summary

  • Phase 26: 19 E2E tests across 3 files (multi-agent, degradation, error paths)
  • Phase 27: Dedicated E2E CI job in GitHub Actions with separate reporting
  • Milestone archive: v2.2 Production Hardening shipped and archived

What's included

  • multi_agent_test.rs: 3 tests for cross-agent queries, filtered queries, agent discovery
  • degradation_test.rs: 4 tests for graceful fallback when indexes missing
  • error_path_test.rs: 12 tests for malformed input handling and useful error messages
  • CI workflow: dedicated e2e job, excluded from test job, required by ci-success gate
  • Planning docs: milestone archived, MILESTONES.md updated, STATE.md reset

Test plan

  • All 29 E2E tests pass (cargo test -p e2e-tests --all-features)
  • CI pipeline shows separate E2E job in checks
  • ci-success gate requires E2E job

🤖 Generated with Claude Code

RichardHightower and others added 17 commits February 11, 2026 00:29
- New helper function to create test events for a specific named agent
- Same as create_test_events but accepts explicit agent parameter
- Enables multi-agent test data creation without modifying existing API

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- test_degradation_all_indexes_missing: worst-case Agentic-only mode
- test_degradation_no_bm25_index: BM25 missing, system still responds
- test_degradation_bm25_present_vector_missing: Keyword tier with BM25
- test_degradation_capabilities_warnings_contain_context: warning quality

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- test_multi_agent_cross_agent_query: 3-agent ingest, TOC build, BM25 index,
  route_query returns results with agent attribution
- test_multi_agent_filtered_query: verifies BM25 agent attribution from
  contributing_agents, route_query accepts agent_filter parameter
- test_multi_agent_discovery: ListAgents reports correct agent counts,
  session counts, and descending last_seen_ms ordering
- build_toc_with_agent helper sets contributing_agents after TocBuilder
  (TocBuilder does not propagate agent from events automatically)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- test_ingest_missing_event: validates None event returns InvalidArgument
- test_ingest_missing_event_id: validates empty event_id returns InvalidArgument
- test_ingest_missing_session_id: validates empty session_id returns InvalidArgument
- test_ingest_invalid_timestamp: validates overflow timestamp returns InvalidArgument
- test_ingest_valid_event_succeeds: positive control proves validation not overly aggressive

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- SUMMARY.md with 3 test descriptions, 2 deviations documented
- STATE.md updated with position, decisions, metrics

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- test_route_query_empty_query: validates empty query returns InvalidArgument
- test_classify_intent_empty_query: validates empty intent query returns InvalidArgument
- test_get_node_empty_id: validates empty node_id returns InvalidArgument
- test_expand_grip_empty_id: validates empty grip_id returns InvalidArgument
- test_expand_grip_nonexistent_graceful: validates nonexistent grip returns empty (no panic)
- test_browse_toc_empty_parent_id: validates empty parent_id returns InvalidArgument
- test_get_agent_activity_invalid_bucket: validates invalid bucket returns InvalidArgument

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- 12 error path tests covering ingest, query, lookup, navigation, agent activity
- STATE.md updated: Phase 26 complete, ready for Phase 27

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Mark Phase 26 complete in ROADMAP.md
- Mark E2E-05, E2E-06, E2E-08 as Done in REQUIREMENTS.md
- Add VERIFICATION.md (14/14 must-haves passed)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Add e2e job running cargo test -p e2e-tests with step summary reporting
- Exclude e2e-tests from workspace test job to avoid redundant execution
- Add e2e to ci-success gate job needs array and result check
- E2E job uses continue-on-error pattern for proper failure reporting

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Summary: dedicated E2E job in CI with step summary and gate integration
- STATE.md: phase 27 complete, v2.2 milestone done, all tech debt resolved

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Mark Phase 27 complete in ROADMAP.md
- Mark CI-01, CI-02, CI-03 as Done in REQUIREMENTS.md
- Add VERIFICATION.md (4/4 must-haves passed)
- All 17/17 v2.2 requirements satisfied

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Archive ROADMAP, REQUIREMENTS, AUDIT to milestones/
- Update MILESTONES.md with v2.2 accomplishments and stats
- Evolve PROJECT.md for v2.2 completion (43,932 LOC, 29 E2E tests)
- Reset STATE.md for between-milestones
- Delete active REQUIREMENTS.md (archived copy preserved)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@RichardHightower RichardHightower merged commit 64510e8 into main Feb 11, 2026
9 checks passed
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