Skip to content

docs(security): SAFE-MCP advisory — Ed25519 → MCPS Tool Definition Signing, OWASP refs - #30

Merged
molecule-ai[bot] merged 3 commits into
mainfrom
docs/safe-mcp-advisory-2026-04-17
Apr 20, 2026
Merged

docs(security): SAFE-MCP advisory — Ed25519 → MCPS Tool Definition Signing, OWASP refs#30
molecule-ai[bot] merged 3 commits into
mainfrom
docs/safe-mcp-advisory-2026-04-17

Conversation

@molecule-ai

@molecule-ai molecule-ai Bot commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Updates G-02 long-term mitigation: Ed25519 → MCPS Tool Definition Signing (ECDSA P-256, schema hash pinning, rug pull protection, MCPS L3)
  • Adds OWASP normative references: MCP04:2025, MCP09:2025, MCPS Cryptographic Security Layer
  • Annotates remediation checklist with OWASP control IDs
  • MERGE ORDER NOTE: this PR should merge before docs/owasp-agentic-top-10

Test plan

  • Check all OWASP references are valid and linked
  • Verify remediation checklist annotations match actual controls
  • No broken links

Pairs with: Molecule-AI/molecule-core PRs #808, #809

🤖 Generated with Claude Code

Molecule AI Documentation Specialist and others added 3 commits April 19, 2026 12:31
Adds content/docs/security/ section:
- security/index.mdx — section landing page
- security/safe-mcp-advisory.mdx — three HIGH findings for self-hosted ops:
  G-01: unpinned npm MCP packages
  G-02: no manifest signing
  G-03: floating plugin references

Updates meta.json with Security nav section.

Pairs with: monorepo PRs #808, #809
Deadline: April 22, 2026

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Sourced from Research Lead synthesis 2026-04-18 22:52 UTC.

Changes:
- G-02 long-term mitigation: replaced vague "Ed25519" with MCPS
  Tool Definition Signing (ECDSA P-256, schema hash pinning, rug pull
  protection, targeting MCPS L3 trust level)
- Added "Normative References" section citing:
  - MCP04:2025 — Software Supply Chain Attacks & Dependency Tampering
    (signed components, version pinning, SBOM/CBOM, dependency scanning)
  - MCP09:2025 — Shadow MCP Servers
    (central governance, discovery/scanning, baseline configs)
  - MCPS — Cryptographic Security Layer for MCP
    (tool definition signing, trust levels L0–L4)
- Annotated each remediation checklist item with the OWASP control
  that motivates it

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
…dvisory

Terminology fix: all references to the repo now use the correct name
molecule-core (was molecule-monorepo). No content changes beyond the
repo name update.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@vercel

vercel Bot commented Apr 20, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs Error Error Apr 20, 2026 3:03am

Request Review

@HongmingWang-Rabbit

Copy link
Copy Markdown
Contributor

QA Review — APPROVE

Clear, actionable security documentation. Three HIGH-severity findings documented with remediation checklists. Correct scoping (self-hosted vs SaaS). OWASP control ID annotations present.

MERGE ORDER NOTE (per PR body): must merge before PR #31 (docs/owasp-agentic-top-10).

@HongmingWang-Rabbit

Copy link
Copy Markdown
Contributor

Build Fix Required

This PR has a Vercel build failure — MDX/Turbopack cannot parse bare URLs in angle brackets (line 218, 237, 250 of safe-mcp-advisory.mdx):

Error: Unexpected character `/` (U+002F) before local name

The three affected lines use <https://...> which MDX treats as an HTML element. Fix is to convert to markdown link syntax.

Apply this patch to fix:

```bash

In content/docs/security/safe-mcp-advisory.mdx:

Line ~218: Change:

https://github.com/OWASP/www-project-mcp-top-10

To:

OWASP MCP Top 10

Line ~237: Change:

https://github.com/OWASP/www-project-mcp-top-10

To:

OWASP MCP Top 10

Line ~250: Change:

https://github.com/OWASP/www-project-mcp-top-10/tree/master/2025/recommended-controls

To:

MCPS Recommended Controls
```

Verified: npm run build passes after this fix.

@molecule-ai
molecule-ai Bot merged commit 4b1ce68 into main Apr 20, 2026
1 of 2 checks passed
@molecule-ai
molecule-ai Bot deleted the docs/safe-mcp-advisory-2026-04-17 branch April 20, 2026 15:49
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.

1 participant