Skip to content

Implement MCP tool listing support#911

Merged
jviotti merged 7 commits into
mainfrom
tools-list-2
May 18, 2026
Merged

Implement MCP tool listing support#911
jviotti merged 7 commits into
mainfrom
tools-list-2

Conversation

@jviotti
Copy link
Copy Markdown
Member

@jviotti jviotti commented May 17, 2026

Signed-off-by: Juan Cruz Viotti jv@jviotti.com

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark Index (community)

Details
Benchmark suite Current: 7362982 Previous: 0018c8a Ratio
Add one schema (0 existing) 286 ms 237 ms 1.21
Add one schema (100 existing) 26 ms 24 ms 1.08
Add one schema (1000 existing) 79 ms 80 ms 0.99
Add one schema (10000 existing) 679 ms 758 ms 0.90
Update one schema (1 existing) 19 ms 19 ms 1
Update one schema (101 existing) 27 ms 25 ms 1.08
Update one schema (1001 existing) 81 ms 81 ms 1
Update one schema (10001 existing) 683 ms 853 ms 0.80
Cached rebuild (1 existing) 5 ms 5 ms 1
Cached rebuild (101 existing) 7 ms 7 ms 1
Cached rebuild (1001 existing) 27 ms 27 ms 1
Cached rebuild (10001 existing) 255 ms 254 ms 1.00
Index 100 schemas 144 ms 114 ms 1.26
Index 1000 schemas 1143 ms 896 ms 1.28
Index 10000 schemas 13663 ms 13416 ms 1.02

This comment was automatically generated by workflow using github-action-benchmark.

Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Benchmark Index (enterprise)

Details
Benchmark suite Current: 7362982 Previous: 0018c8a Ratio
Add one schema (0 existing) 267 ms 210 ms 1.27
Add one schema (100 existing) 29 ms 20 ms 1.45
Add one schema (1000 existing) 87 ms 62 ms 1.40
Add one schema (10000 existing) 906 ms 562 ms 1.61
Update one schema (1 existing) 22 ms 15 ms 1.47
Update one schema (101 existing) 29 ms 20 ms 1.45
Update one schema (1001 existing) 89 ms 63 ms 1.41
Update one schema (10001 existing) 727 ms 562 ms 1.29
Cached rebuild (1 existing) 7 ms 4 ms 1.75
Cached rebuild (101 existing) 9 ms 5 ms 1.80
Cached rebuild (1001 existing) 33 ms 18 ms 1.83
Cached rebuild (10001 existing) 282 ms 168 ms 1.68
Index 100 schemas 123 ms 99 ms 1.24
Index 1000 schemas 1174 ms 788 ms 1.49
Index 10000 schemas 14023 ms 11797 ms 1.19

This comment was automatically generated by workflow using github-action-benchmark.

jviotti added 3 commits May 18, 2026 10:04
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti marked this pull request as ready for review May 18, 2026 14:27
@jviotti jviotti changed the title [WIP] Implement MCP tool listing support Implement MCP tool listing support May 18, 2026
@augmentcode
Copy link
Copy Markdown

augmentcode Bot commented May 18, 2026

This pull request is too large for Augment to review. The PR exceeds the maximum size limit of 75000 tokens (approximately 300000 characters) for automated code review. Please consider breaking this PR into smaller, more focused changes.

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 123 files

Note: This PR contains a large number of files. cubic only reviews up to 100 files per PR, so some files may not have been reviewed. cubic prioritizes the most important files to review.
On a pro plan you can use ultrareview for larger PRs.

Re-trigger cubic

Comment thread enterprise/index/enterprise_index.cc Outdated
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 issue found across 2 files (changes from recent commits).

Reply with feedback, questions, or to request a fix.

Re-trigger cubic

Comment thread enterprise/index/enterprise_index.cc Outdated
Signed-off-by: Juan Cruz Viotti <jv@jviotti.com>
@jviotti jviotti merged commit 2bcebf9 into main May 18, 2026
5 checks passed
@jviotti jviotti deleted the tools-list-2 branch May 18, 2026 16:45
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