Skip to content

feat: tools return in the TOON format instead of JSON.#40

Merged
flacatus merged 2 commits into
konflux-ci:mainfrom
lipka28:reTOONing
Jan 27, 2026
Merged

feat: tools return in the TOON format instead of JSON.#40
flacatus merged 2 commits into
konflux-ci:mainfrom
lipka28:reTOONing

Conversation

@lipka28

@lipka28 lipka28 commented Jan 23, 2026

Copy link
Copy Markdown
Member

This unifies the behavior across the mcp tools to always return in TOON format instead of JSON for reduced token use. The main changes are mostly in database_tools.py and tools_manager.py. The other changes are:

  • Make all the affected tests expect a TOON formated string as a result.
  • Variable name consistency (variable containing TOON formated string shouldn't be called result_json)
  • Inline documentation fixes for some functions which specify returning JSON which is no longer true after this change.

@snyk-io

snyk-io Bot commented Jan 23, 2026

Copy link
Copy Markdown
Contributor

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 66.66667% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.45%. Comparing base (bef5536) to head (24bd7cb).
⚠️ Report is 32 commits behind head on main.

Files with missing lines Patch % Lines
tools/database_tools.py 66.66% 1 Missing ⚠️
tools/tools_manager.py 66.66% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
+ Coverage   47.54%   51.45%   +3.90%     
==========================================
  Files          27       38      +11     
  Lines        1424     2859    +1435     
  Branches      178      391     +213     
==========================================
+ Hits          677     1471     +794     
- Misses        724     1326     +602     
- Partials       23       62      +39     
Flag Coverage Δ
unit-tests 51.45% <66.66%> (+3.90%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 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.

@lipka28

lipka28 commented Jan 26, 2026

Copy link
Copy Markdown
Member Author

/ok-to-test

1 similar comment
@flacatus

Copy link
Copy Markdown
Member

/ok-to-test

@flacatus flacatus merged commit 86f63d6 into konflux-ci:main Jan 27, 2026
11 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.

3 participants