Skip to content

Drop Moonshot AI from the recipe catalog's vendors - #559

Open
dennis-upbound wants to merge 1 commit into
modelplaneai:mainfrom
dennis-upbound:dennis/drop-moonshot-vendor
Open

dennis-upbound wants to merge 1 commit into
modelplaneai:mainfrom
dennis-upbound:dennis/drop-moonshot-vendor

Conversation

@dennis-upbound

@dennis-upbound dennis-upbound commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Description of your changes

The docs home page shows an empty Moonshot AI card. The recipe catalog renders every vendor in docs/data/recipes.yaml, graying out one that no recipe carries, and Kimi-K2 was the only Moonshot AI recipe. #529 (24a5cdd) removed it because it used the endpoint picker's old command line.

This removes the vendor entry. The brand icon stays, so bringing the Kimi-K2 recipe back, once it's ported to the endpoint picker template and tested again, only needs the entry back.

I have:

  • Read and followed Modelplane's contribution process.
  • Run nix flake check (or ./nix.sh flake check) and made sure it passes. Not run locally for a three-line data change; CI runs it.
  • Added or updated tests covering any composition function changes. Docs data only.
  • Signed off every commit with git commit -s.

🤖 Generated with Claude Code

The catalog's browse cards render every vendor in docs/data/recipes.yaml,
graying out one no recipe carries. Kimi-K2 was the only Moonshot AI recipe,
and it was removed in 24a5cdd because it used the endpoint picker's old
command line. The docs home page now shows an empty Moonshot AI card.

Remove the entry. The brand icon stays, so restoring the recipe only needs
the entry back.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Signed-off-by: Dennis Ramdass <dennis@upbound.io>
@github-actions

Copy link
Copy Markdown

Docs preview: https://modelplane-docs-pr-559.vercel.app (ready once the site's Content workflow finishes)

@dennis-upbound

Copy link
Copy Markdown
Collaborator Author

Alternative: #560 restores the Kimi-K2 recipe instead. Merge one and I'll close the other.

This branch has not been deployed

No deployments
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