Skip to content

Add configurable MCP tools and implicit agent replies - #107

Merged
timkpaine merged 1 commit into
mainfrom
tkp/agent-replies-and-mcp
Oct 9, 2026
Merged

timkpaine merged 1 commit into
mainfrom
tkp/agent-replies-and-mcp

Conversation

@timkpaine

Copy link
Copy Markdown
Member

Description

Brief description of the changes in this PR.

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactor / code cleanup
  • CI / build configuration
  • Other (describe below)

Checklist

  • Linting passes (make lint)
  • Tests pass (make test)
  • New tests added for new functionality
  • Documentation updated (if applicable)
  • Changelog / version bump (if applicable)

@timkpaine
timkpaine requested a review from ptomecek as a code owner October 9, 2026 20:45
@github-actions

github-actions Bot commented Oct 9, 2026

Copy link
Copy Markdown
Contributor

Test Results

307 tests  +7   307 ✅ +7   8s ⏱️ -2s
  1 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit f464a75. ± Comparison against base commit 9259759.

This pull request removes 1 and adds 8 tests. Note that renamed tests count towards both.
csp_bot.tests.test_agent_command.TestSessionStore ‑ test_update_response_id_removes_old_mapping
csp_bot.tests.test_agent_command.TestMultimodalPrompt ‑ test_incoming_workbook_is_exposed_to_read_file
csp_bot.tests.test_agent_command.TestSessionStore ‑ test_update_response_id_preserves_old_mapping
csp_bot.tests.test_agent_command.TestSetBackends ‑ test_reply_index_retains_older_responses_and_thread_root
csp_bot.tests.test_agent_command.TestSetBackends ‑ test_reply_prompt_identifies_deletion_target
csp_bot.tests.test_agent_mcp ‑ test_mcp_config_resolves_environment_headers_and_filters_tools
csp_bot.tests.test_agent_mcp ‑ test_mcp_requires_https
csp_bot.tests.test_agent_mcp ‑ test_missing_mcp_credential_fails_without_value
csp_bot.tests.test_bot_integration.TestAgentSessionReplyIsolation ‑ test_unmentioned_channel_reply_to_agent_is_routed

@codecov

codecov Bot commented Oct 9, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 89.04110% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.43%. Comparing base (9259759) to head (f464a75).

Files with missing lines Patch % Lines
csp_bot/commands/agent.py 76.92% 4 Missing and 5 partials ⚠️
csp_bot/bot.py 68.42% 6 Missing ⚠️
csp_bot/commands/mcp.py 96.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #107      +/-   ##
==========================================
+ Coverage   84.68%   85.43%   +0.74%     
==========================================
  Files          44       46       +2     
  Lines        4604     4730     +126     
  Branches      338      353      +15     
==========================================
+ Hits         3899     4041     +142     
+ Misses        598      579      -19     
- Partials      107      110       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@timkpaine
timkpaine merged commit a0dc2c4 into main Oct 9, 2026
5 checks passed
@timkpaine
timkpaine deleted the tkp/agent-replies-and-mcp branch October 9, 2026 21:46
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