Skip to content

fix(carta-investors): remove unrecognized displayName field#42

Open
bryan-anthropic wants to merge 1 commit into
carta:mainfrom
bryan-anthropic:fix/remove-displayName-from-carta-investors
Open

fix(carta-investors): remove unrecognized displayName field#42
bryan-anthropic wants to merge 1 commit into
carta:mainfrom
bryan-anthropic:fix/remove-displayName-from-carta-investors

Conversation

@bryan-anthropic
Copy link
Copy Markdown

Summary

Removes the displayName field from plugins/carta-investors/.claude-plugin/plugin.json. It's the only Carta plugin that declares this field, and it's not part of the Claude Code plugin manifest schema, so claude plugin validate fails on this file:

❯ root: Unrecognized key: "displayName"

After the fix, validation passes:

$ claude plugin validate plugins/carta-investors
✔ Validation passed

The other two plugins in this repo — carta-cap-table and carta-crm — do not declare displayName and validate cleanly. This change brings carta-investors in line with them.

Why

The strict-schema validation gate on the Anthropic plugin marketplaces blocks distribution when any field outside the documented schema is present. carta-investors was published once but is no longer indexed because of this. Removing the one extra key restores it.

Test plan

  • claude plugin validate plugins/carta-investors — passes
  • No behavior change: the name field already provides the canonical identifier, and the marketplace UI uses name for display.

The 'displayName' key is not part of the Claude Code plugin.json schema and
causes 'claude plugin validate' to fail, which blocks distribution through
the Claude plugin marketplaces.

Verified with:

    $ claude plugin validate plugins/carta-investors
    ✔ Validation passed

The other two plugins in this repo (carta-cap-table, carta-crm) do not
declare displayName and validate cleanly.
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