registry icon indicating copy to clipboard operation
registry copied to clipboard

Named arguments should identify the argument name

Open dmartinol opened this issue 5 months ago • 1 comments

Is your feature request related to a problem? Please describe. Package arguments with named arguments should have a clear identification of the argument name and the possible default value.

Definitions like the following should not be allowed because we need to parse the name field to extract the argument name:

        "package_arguments": [
          {
            "description": "Specify directory to work in.",
            "is_required": true,
            "format": "string",
            "value": "--directory path/to/cloned/repo/mcp-server-couchbase/",
            "default": "--directory path/to/cloned/repo/mcp-server-couchbase/",
            "type": "named",
            "name": "--directory path/to/cloned/repo/mcp-server-couchbase/",
            "value_hint": "path/to/cloned/repo/mcp-server-couchbase/"
          },

Describe the solution you'd like The above should be converted to a simpler configuration with a clear definition of the argument name:

        "package_arguments": [
          {
            "description": "Specify directory to work in.",
            "is_required": true,
            "format": "string",
            "value": "<VALUE>",
            "default": "<DEFAULT VALUE>",
            "type": "named",
            "name": "--directory",
            "value_hint": "path/to/cloned/repo/mcp-server-couchbase/"
          },

Describe alternatives you've considered

Additional context After looking again at the seed file, I see that all the entries have package arguments where value is equal to default, so I cannot understand the difference between these fields, are we sure both of them are needed? The schema is also not clear in specifying their purpose:

        value:
          type: string
          description: |
            The default value for the input. If this is not set, the user may be prompted to provide a value.
...
        default:
          type: string
          description: The default value for the input.

dmartinol avatar Jun 20 '25 09:06 dmartinol

Definitions like the following should not be allowed because we need to parse the name field to extract the argument name:

They are not, that is a bug in the seed data.

connor4312 avatar Jun 20 '25 14:06 connor4312

related (I think?): https://github.com/modelcontextprotocol/registry/issues/49

domdomegg avatar Aug 21 '25 03:08 domdomegg

related (I think?): #49

Related, but I think it focuses on the invalid package definition, rather than the specific issue with parameter names.

dmartinol avatar Aug 27 '25 08:08 dmartinol