Skip to content

chore: update the MCP tools' argument description - #55

Merged
sbillinge merged 2 commits into
diffpy:mainfrom
ycexiao:clean-output
Oct 31, 2026
Merged

sbillinge merged 2 commits into
diffpy:mainfrom
ycexiao:clean-output

Conversation

@ycexiao

@ycexiao ycexiao commented Oct 30, 2026

Copy link
Copy Markdown
Contributor
  • chore: add clear and informative outputs
  • chore: update the MCP tool interface

What problem does this PR address?

Update the error information and MCP tool interface so the agent can get more information.

What should the reviewer(s) do?

@codecov

codecov Bot commented Oct 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.69%. Comparing base (4e25b63) to head (93fcf3c).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #55   +/-   ##
=======================================
  Coverage   99.69%   99.69%           
=======================================
  Files           8        8           
  Lines         648      648           
=======================================
  Hits          646      646           
  Misses          2        2           
Files with missing lines Coverage Δ
tests/test_refinement_server.py 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@ycexiao
ycexiao marked this pull request as ready for review October 30, 2026 13:04
@ycexiao ycexiao changed the title clean output chore: update the MCP interface description Oct 30, 2026
@ycexiao ycexiao changed the title chore: update the MCP interface description chore: update the MCP tools' argument description Oct 30, 2026
@ycexiao

ycexiao commented Oct 30, 2026

Copy link
Copy Markdown
Contributor Author

@sbillinge, it's ready for review.

@sbillinge
sbillinge merged commit 109ba11 into diffpy:main Oct 31, 2026
6 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.

2 participants