Skip to content

Conversation

@KedoKudo
Copy link
Collaborator

@KedoKudo KedoKudo commented Dec 9, 2025

Summary

  • Add comprehensive MCP documentation to docs/mcp/
  • README.md: Installation, quick start, tool reference, troubleshooting
  • manifest-format.md: YAML frontmatter specification with validation rules
  • integration-pattern.md: Reusable pattern for adding MCP to scientific packages

Changes

docs/mcp/README.md

  • Installation instructions (PyPI, editable, Pixi)
  • Quick start guide with .mcp.json configuration
  • Complete tool reference with parameters and return values
  • Mermaid workflow diagram
  • Backend options and isotope selection priority
  • Security notes and troubleshooting section

docs/mcp/manifest-format.md

  • YAML frontmatter specification
  • Field reference (required, optional, material properties)
  • Isotope composition options (explicit list, enrichment, natural abundance)
  • Priority order documentation
  • Complete and minimal examples

docs/mcp/integration-pattern.md

  • Architecture overview with Mermaid diagram
  • Decorator approaches (FastMCP built-in vs custom registry)
  • Step-by-step implementation guide
  • Design principles and common pitfalls
  • Realistic code size estimates

Test plan

  • Verify Mermaid diagrams render correctly on GitHub
  • Check all internal links work
  • Review for technical accuracy against implementation

Closes #170

🤖 Generated with Claude Code

Add comprehensive documentation for the PLEIADES MCP server integration:

- README.md: Installation, quick start, tool reference, troubleshooting
- manifest-format.md: YAML frontmatter specification with validation rules
- integration-pattern.md: Reusable pattern for adding MCP to scientific packages

Changes include:
- Mermaid diagrams for better web rendering
- Clarified decorator approaches (FastMCP built-in vs custom registry)
- Documented isotope selection priority hierarchy

Closes #170

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Copy link
Contributor

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 comprehensive documentation for the MCP (Model Context Protocol) module in PLEIADES, enabling AI-assisted neutron resonance analysis. The documentation provides installation guides, API references, manifest specifications, and reusable integration patterns for adding MCP capabilities to other scientific packages.

Key Changes

  • Complete MCP module documentation with installation, configuration, and usage instructions
  • Manifest format specification detailing YAML frontmatter structure and validation rules
  • Integration pattern guide providing a reusable architecture template for other scientific software packages

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.

File Description
docs/mcp/README.md Installation instructions, quick start guide, complete tool reference with parameters/returns, workflow diagrams, backend options, and troubleshooting
docs/mcp/manifest-format.md YAML frontmatter specification with field references, isotope composition options, validation rules, and complete examples
docs/mcp/integration-pattern.md Architecture overview, decorator pattern implementation, step-by-step integration guide, design principles, and code size estimates

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- Add timezone Z to ISO-8601 timestamp example (manifest-format.md)
- Change URL to next branch instead of feature branch (integration-pattern.md)
- Remove nova from backend options in MCP tool description (tools.py)
- Use _mcp_registry instead of _tool_registry for accuracy (integration-pattern.md)
- Use sys.exit() instead of exit() (integration-pattern.md)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <[email protected]>
Copy link
Contributor

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

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@KedoKudo KedoKudo merged commit 6a7a6b6 into feature/163-mcp-integration Dec 9, 2025
2 checks passed
@KedoKudo KedoKudo deleted the feature/170-mcp-documentation branch December 9, 2025 16:56
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