Skip to content

Temp: Investigating dxt format some more #166

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.

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

Merged
merged 1 commit into from
Jun 28, 2025
Merged

Temp: Investigating dxt format some more #166

merged 1 commit into from
Jun 28, 2025

Conversation

Flux159
Copy link
Owner

@Flux159 Flux159 commented Jun 26, 2025

Summary:

Test Plan:

manifest.json Outdated
Comment on lines 26 to 32
"allowed_directories": {
"type": "directory",
"title": "Allow access to .kube directory",
"description": ".kube directory access for mcp server kubernetes",
"required": true,
"default": ["${HOME}/.kube"]
},
Copy link
Owner Author

Choose a reason for hiding this comment

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

I'm not sure if this is needed or not?

Mac OS definitely blocks access to Documents, Downloads, and Desktop, but I'm not sure if it's needed to get approval for hidden directories inside of home.

Copy link
Contributor

Choose a reason for hiding this comment

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

Hey, I'm the developer behind dxt and owner of Claude Desktop! This isn't required - macOS might show some dialogs but this config would only be if you, the dev, wanted the user to configure some directories.

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds new configuration properties in the manifest.json file to allow user-specific configurations related to Kubernetes setup.

  • Added a user_config section with allowed_directories and kubeconfig_path settings.
  • Updated manifest.json to include these user-defined fields.

@felixrieseberg
Copy link
Contributor

Hey! Love your MCP server, it's cool stuff. Do you want to add a long_description to the manifest.json? I'd love to put it straight into the directory.

@Flux159
Copy link
Owner Author

Flux159 commented Jun 28, 2025

@rr-paras-patel Fixed up typo.

@felixrieseberg Added a long_description & removed the allowed_directories object. Thanks for the feedback!

Going to merge this in & create a new release to get the updated dxt. @felixrieseberg Should I submit the google form again with the new DXT?

Edit: New dxt is here: https://github.com/Flux159/mcp-server-kubernetes/releases/tag/v2.4.6 ; will resubmit google form if needed.

@Flux159 Flux159 requested a review from rr-paras-patel June 28, 2025 17:40
@Flux159 Flux159 merged commit cdf896d into main Jun 28, 2025
2 checks passed
@felixrieseberg
Copy link
Contributor

@Flux159 All good, I'll take it straight from your url. Thank you!

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.

3 participants