Skip to content

Commit 8a00881

Browse files
authored
fix: Rename tool name, sent notifications/tools/list_changed (#37)
* fix: Handle dots and slash in tool names. Send notification when list of available tools is changed * feat: Add tests * fix: revert tool auto-loading * feat: add build and test to the workflow * fix: actorNameToToolName
1 parent bcc57a8 commit 8a00881

File tree

16 files changed

+1406
-70
lines changed

16 files changed

+1406
-70
lines changed

.actor/input_schema.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
"enableActorAutoLoading": {
1818
"title": "Enable automatic loading of Actors based on context and use-case (experimental, check if it supported by your client)",
1919
"type": "boolean",
20-
"description": "When enabled, the server can dynamically add Actors as tools based on user requests and context. \n\nNote: Not all MCP clients support this feature. To try it, you can use the [Tester MCP Client](https://apify.com/jiri.spilka/tester-mcp-client). This is an experimental feature and may require client-specific support.",
20+
"description": "When enabled, the server can dynamically add Actors as tools based on user requests and context. \n\nNote: MCP client must support notification on tool updates. To try it, you can use the [Tester MCP Client](https://apify.com/jiri.spilka/tester-mcp-client). This is an experimental feature and may require client-specific support.",
2121
"default": false
2222
},
2323
"maxActorMemoryBytes": {

.github/workflows/check.yaml

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -13,8 +13,8 @@ on:
1313
- "**" # Ignore all tags to prevent duplicate builds when tags are pushed.
1414

1515
jobs:
16-
lint:
17-
name: Lint
16+
lint_and_test:
17+
name: Build & Test
1818
runs-on: ubuntu-latest
1919

2020
steps:
@@ -27,4 +27,12 @@ jobs:
2727
cache-dependency-path: 'package-lock.json'
2828
- name: Install Dependencies
2929
run: npm ci
30-
- run: npm run lint
30+
31+
- name: Lint
32+
run: npm run lint
33+
34+
- name: Build
35+
run: npm run build
36+
37+
- name: Test
38+
run: npm run test

README.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ The server provides a set of helper tools to discover available Actors and retri
9696
- `get-actor-details`: Retrieves documentation, input schema, and details about a specific Actor.
9797
- `discover-actors`: Searches for relevant Actors using keywords and returns their details.
9898

99-
There are also tools to manage the available tools list. However, dynamically adding and removing tools requires the MCP client to have the capability to manage the tools list, which is typically not supported.
99+
There are also tools to manage the available tools list. However, dynamically adding and removing tools requires the MCP client to have the capability to update tools list (handle `ToolListChangedNotificationSchema`), which is typically not supported.
100100

101101
You can try this functionality using the [Apify Tester MCP Client](https://apify.com/jiri.spilka/tester-mcp-client) Actor.
102102
To enable it, set the `enableActorAutoLoading` parameter.
@@ -393,3 +393,4 @@ If you need other features or have any feedback, [submit an issue](https://conso
393393
- [What is MCP and why does it matter?](https://blog.apify.com/what-is-model-context-protocol/)
394394
- [Tester MCP Client](https://apify.com/jiri.spilka/tester-mcp-client)
395395
- [AI agent workflow: building an agent to query Apify datasets](https://blog.apify.com/ai-agent-workflow/)
396+
- [MCP Client development guide](https://github.com/cyanheads/model-context-protocol-resources/blob/main/guides/mcp-client-development-guide.md)

0 commit comments

Comments
 (0)