Skip to content

MCP tool function cleanup #73

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 86 commits into from
Jul 9, 2025
Merged

MCP tool function cleanup #73

merged 86 commits into from
Jul 9, 2025

Conversation

Prasidh
Copy link
Contributor

@Prasidh Prasidh commented Jul 2, 2025

No description provided.

markmiro and others added 30 commits June 6, 2025 09:01
@markmiro
wip
e1d04ae
@markmiro
add minimal mcp structure
cee4e9b
@markmiro
anaconda_assistant_conda -> anaconda_assistant_mcp
2c089ca
@markmiro
anaconda-assistant-conda -> anaconda-assistant-mcp
864e7a2
@markmiro
rename dir
6d1ebe0
@markmiro
make tests run
aadc4cb
@markmiro
module = "conda.*" -> module = "mcp.*"
6e15130
@markmiro
add add/subtract mcp tools and add tests for them
9358cbc
@markmiro
add tools list_packages and list_envs that don't work
1809094
@markmiro
clean up tests and better file name
3c16959
@markmiro
test list packages command
3aafc1e
@markmiro
add fastmcp 2.0 and make tools actually work
7e0e113
@markmiro
update fastmcp
6529bcf
@markmiro
add 5 sec timeout
8a66505
@markmiro
fix test
c5172d8
@markmiro
Install latest fastmcp version that has no issues
78874eb
@markmiro
Remove 5 second delay from list_packages function
f7f6f53
@markmiro
Rename server_start command to serve in CLI
7e779dc
@markmiro
correct types
8e5fc87
@markmiro
remove mcp commands, tools is enough
d05627d
@Prasidh
Aas 183/mcp server scaffolding (#65)
4a49f2e 
* MCP Server Scaffolding
@markmiro
wip test mcp client e2e
acd36d7
@Prasidh
Merge branch 'mcp' of https://github.com/anaconda/assistant-sdk into mcp
b246f9a
@markmiro
remove unneeded readme content
7f390e8
@markmiro
Update readme with info for dev in claude and cursor
fc30e05
@markmiro
Add details on seeing code changes
0ad9134
@markmiro
update readme
af626d3
@markmiro
clean up, add more experimental stuff, clean up stub
d5b3d66
@markmiro
use return value instead
4e9bc42
@markmiro
Update README
3955994
Prasidh added 21 commits June 26, 2025 17:48
@Prasidh
PR revisions
429b1ca
@Prasidh
Merge branch 'main' into AAS-175/mcp-create-tool
63ca354
@Prasidh
Update environment MCP tool
df2c1de
@Prasidh
Merge branch 'main' into AAS-177/mcp-update-env
4fb3db2
@Prasidh
Delete test_server.py
df6e43c
@Prasidh
Added additional details about each param using Pydantic annotated class
b5ffa0c
@Prasidh
Modified to use context.report_progress in create_environment
8fd16f0
@Prasidh
Remove duplicated function
caa6c4b
@Prasidh
Update report_progress function with new method signature
cd9e546
@Prasidh
Bugfixes
c1c9101
@Prasidh
Added unit tests for create_environment
9737179
@Prasidh
Merge branch 'AAS-175/mcp-create-tool' into AAS-177/mcp-update-env
709dd3b
@Prasidh
Merge branch 'main' into AAS-177/mcp-update-env
0817691
@Prasidh
Added unit tests for update_environment
e9d5487
@Prasidh
Bugfixes
2af6e3c
@Prasidh
Resolve mypy errors
ecfeca5
@Prasidh
Resolve mypy errors
7f261a9
@Prasidh
Resolve mypy errors
0b8a54a
@Prasidh
Merge branch 'main' into AAS-177/mcp-update-env
7fb3e45
@Prasidh
Merge branch 'main' into mcp-tool-function-cleanup
ed7ebe7
@Prasidh
Compile shared functions into shared.py
ca3618b
@Prasidh Prasidh requested a review from markmiro July 2, 2025 14:21
markmiro
markmiro requested changes Jul 2, 2025
View reviewed changes
Copy link
Contributor

@markmiro markmiro left a comment

Choose a reason for hiding this comment

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

added a few comments, I remember the channels func you had didn't get the right channels when I tried on my machine

@Prasidh Prasidh added this pull request to the merge queue Jul 9, 2025
Merged via the queue into main with commit c449bb2 Jul 9, 2025
18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@markmiro markmiro markmiro approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Prasidh @markmiro @jkong-anaconda