Skip to content

Conversation

@mbaiza27
Copy link
Contributor

@mbaiza27 mbaiza27 commented Apr 4, 2025

Hi there!

Marc here from the VS Code team. This PR adds support for installing this MCP Server in VS Code. We provide a one-click install button, as well as manual instructions.

Thanks so much for creating a great MCP Server!

Copy link
Collaborator

@jirispilka jirispilka left a comment

Choose a reason for hiding this comment

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

Hi, thank you for your contribution, I really appreciate it!
I've added a couple of changes, let me know if you are ok with it.

@mbaiza27
Copy link
Contributor Author

mbaiza27 commented Apr 9, 2025

@jirispilka Changes look good! Thanks for the updates🙂

@jirispilka jirispilka merged commit 5dd679c into apify:master Apr 10, 2025
2 checks passed
MQ37 added a commit that referenced this pull request Apr 15, 2025
* docs: document input schema processing (#58)

document Actor input schema processing in README

* add VS Code instructions to README (#62)

* Add vs code instructions

---------

Co-authored-by: mbaiza27 <[email protected]>
Co-authored-by: Jiří Spilka <[email protected]>

* ci: run prerelase manually (#70)

run prerelase manually

* get apify api url base from env var

---------

Co-authored-by: Marc Baiza <[email protected]>
Co-authored-by: mbaiza27 <[email protected]>
Co-authored-by: Jiří Spilka <[email protected]>
MQ37 added a commit that referenced this pull request Apr 16, 2025
* docs: document input schema processing (#58)

document Actor input schema processing in README

* add VS Code instructions to README (#62)

* Add vs code instructions

---------

Co-authored-by: mbaiza27 <[email protected]>
Co-authored-by: Jiří Spilka <[email protected]>

* initial working draft of proxy actorized mcp servers

* fix tests

* fix proxy tool name handling

* ci: run prerelase manually (#70)

run prerelase manually

* organize, fix passing of apify token

* fix env var name

* fix imports

* get standby url from actor id, check if is mcp server based on actor name for now

* Actor MCP server load default Actors

* get standbyUrlBase from env var

* fix standby url base, use dns friendly owner name

* get mcp path from definition

* Remove unused import from actor.ts

* console.log to logger

* Remove extra slash from MCP server URL construction

* Simplify actor tools loading logic

* Refactor actorOwnerDNSFriendly to handle special characters

* Add TODO comment for reworking actor definition fetch logic

* refactor mcp actor and utils, use real Actor ID as standby URL

* fix: get-actor-definition-default-build (#73)

* fix get default build in get actor definition

* fix tests

* fix double import, lint

---------

Co-authored-by: Marc Baiza <[email protected]>
Co-authored-by: mbaiza27 <[email protected]>
Co-authored-by: Jiří Spilka <[email protected]>
Co-authored-by: Jiri Spilka <[email protected]>
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