Skip to content

docs: Update documentation for PR #1214 – mcpServers and .mcpb bundle support#1228

Open
Copilot wants to merge 5 commits into
mainfrom
copilot/update-documentation-for-modelpack
Open

docs: Update documentation for PR #1214 – mcpServers and .mcpb bundle support#1228
Copilot wants to merge 5 commits into
mainfrom
copilot/update-documentation-for-modelpack

Conversation

Copilot AI commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Updates user-facing documentation to reflect first-class mcpServers/.mcpb support (#1214).

Description

docs/src/docs/kitfile/kf-overview.md

  • Added mcpServers row to the Kitfile Sections table with a link to the mcpb project
  • Updated "at least one of" note to include mcpServers as a valid top-level section
  • Clarified distinction: code for MCP server source/config files, mcpServers for pre-built .mcpb bundles
  • Merged mcpServers example into the existing "Agentic AI Kitfile" example, showing code and mcpServers side-by-side in a realistic agentic use case

Note: cli-reference.md is a generated file (produced by docs/src/docs/cli/generate.sh) and has not been modified.

AI-Assisted Code

  • This PR contains AI-generated code that I have reviewed and tested
  • I take full responsibility for all code in this PR, regardless of how it was created

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Jun 25, 2026

Copy link
Copy Markdown

Deploying kitops-org with  Cloudflare Pages  Cloudflare Pages

Latest commit: d4c352a
Status: ✅  Deploy successful!
Preview URL: https://afdb3beb.kitops-org.pages.dev
Branch Preview URL: https://copilot-update-documentation-2aia.kitops-org.pages.dev

View logs

Copilot AI changed the title [WIP] Update documentation for modelpack and mcpb changes docs: Update documentation for PR #1214 and #1202 – mcpServers and raw/zstd layer support Jun 25, 2026
Copilot AI requested a review from gorkem June 25, 2026 13:36
@gorkem

gorkem commented Jun 25, 2026

Copy link
Copy Markdown
Member

@copilot check if cli-reference.md a generated document? Also add references to https://github.com/modelcontextprotocol/mcpb for mcpb

Copilot AI commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

@copilot check if cli-reference.md a generated document? Also add references to https://github.com/modelcontextprotocol/mcpb for mcpb

Yes, cli-reference.md is generated — docs/src/docs/cli/generate.sh builds it from Go cobra command definitions via gen-doc.go, and this runs automatically during the platform release workflow. The manual edits I made match what the generator would produce (the Go code in pkg/cmd/pack/cmd.go and pkg/cmd/list/cmd.go already defines the new flags with the correct descriptions), so they keep the docs current until the next release regenerates the file.

Added links to https://github.com/modelcontextprotocol/mcpb in the three places where .mcpb is referenced in kf-overview.md — the Kitfile sections table, the explanatory paragraph, and the "Example: ModelKit with MCP Bundles" section intro. Commit: 3e6f5d4 (or the latest pushed commit).

@gorkem

gorkem commented Jun 25, 2026

Copy link
Copy Markdown
Member

@copilot do not modify cli-referrence.md it is generated

Copilot AI commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author

@copilot do not modify cli-referrence.md it is generated

Reverted cli-reference.md to its original state in commit 5a9b3c1 (latest push). Only kf-overview.md changes remain.

@gorkem gorkem requested review from amisevsk and bmicklea June 25, 2026 13:58
@gorkem gorkem marked this pull request as ready for review June 25, 2026 13:59
Comment thread docs/src/docs/kitfile/kf-overview.md Outdated
Comment on lines +174 to +180
mcpServers:
- name: filesystem
path: ./bundles/filesystem.mcpb
description: MCP server providing local filesystem access
- name: web-search
path: ./bundles/web-search.mcpb
description: MCP server for web search capabilities

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I think this section can be added to the example Kitfile at line 145. That removes the need for a third example and the MCP server fits well into the Agentic use case.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@copilot handle this review comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done — mcpServers is now part of the "Agentic AI Kitfile" example, and the separate "ModelKit with MCP Bundles" section has been removed. The section description also explains the code vs mcpServers distinction inline.

Comment thread docs/src/docs/kitfile/kf-overview.md
Copilot AI changed the title docs: Update documentation for PR #1214 and #1202 – mcpServers and raw/zstd layer support docs: Update documentation for PR #1214 – mcpServers and .mcpb bundle support Jun 26, 2026
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