Skip to content

Misc MCP improvements#134

Merged
paulgb merged 6 commits intomainfrom
paulgb/inherit-base-url-from-config
Mar 20, 2025
Merged

Misc MCP improvements#134
paulgb merged 6 commits intomainfrom
paulgb/inherit-base-url-from-config

Conversation

@paulgb
Copy link
Member

@paulgb paulgb commented Mar 19, 2025

  • Adds a note pointing to the docs for other MCP clients
  • Adds a note about installing for local development
  • Reads server base URL from global forevervm CLI config if that's where the token comes from
  • Adds endpoints for resources and prompts, even though we don't have any, which makes the logs cleaner (otherwise there is an error when the client hits them)
  • When a request fails, we send an error message to the client instead of exiting
  • Moves MCP installation-related code to a separate file

@paulgb paulgb changed the title Inherit base url from config Misc MPC improvements Mar 20, 2025
@paulgb paulgb marked this pull request as ready for review March 20, 2025 21:22
async function makeCreateMachineRequest(forevervmOptions: ForeverVMOptions): Promise<string> {
try {
const fvm = new ForeverVM({ token: forevervmToken })
console.error('using options', forevervmOptions)
Copy link
Contributor

Choose a reason for hiding this comment

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

Debug logging alert: Consider removing or replacing the console.error debug statement printing options with a proper logger.

@rolyatmax rolyatmax changed the title Misc MPC improvements Misc MCP improvements Mar 20, 2025
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
@paulgb paulgb merged commit 6847499 into main Mar 20, 2025
4 checks passed
@paulgb paulgb deleted the paulgb/inherit-base-url-from-config branch March 20, 2025 21:52
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