Skip to content

feat(cipp): scaffold CIPP plugin#26

Draft
asachs01 wants to merge 1 commit intomainfrom
feature/cipp-plugin
Draft

feat(cipp): scaffold CIPP plugin#26
asachs01 wants to merge 1 commit intomainfrom
feature/cipp-plugin

Conversation

@asachs01
Copy link
Copy Markdown
Collaborator

Summary

  • Scaffolds CIPP (CyberDrain Improved Partner Portal) plugin for M365 multi-tenant management
  • 6 skills: api-patterns, tenants, users, standards, security, alerts
  • 5 commands: tenant-summary, user-offboard, standards-check, security-posture, alert-review
  • Registered in docs site under security category with alpha maturity

Test plan

  • Verify npm run build in msp-claude-plugins/docs/ succeeds with new plugin entry
  • Check CIPP appears under "Security" in the sidebar
  • Verify all SKILL.md files have proper frontmatter (description + triggers)
  • Verify all command .md files have proper frontmatter (name, description, arguments)
  • Verify plugin.json follows the validated schema

Closes #24

🤖 Generated with Claude Code

…stry

Add CIPP (CyberDrain Improved Partner Portal) plugin for M365 multi-tenant
management. Includes 6 skills (api-patterns, tenants, users, standards,
security, alerts) and 5 commands. Registered in docs site under security
category.

Closes #24

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Collaborator Author

@asachs01 asachs01 left a comment

Choose a reason for hiding this comment

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

Review: CIPP Plugin Scaffold

Good content quality — skills have proper API endpoint tables, request/response examples, and error handling. Two blockers before this can leave draft:

Blockers

1. .mcp.json does not follow the gateway URL pattern and is missing auth headers

Currently points to ${CIPP_BASE_URL}/api with no headers block. The CIPP_API_KEY credential is never injected — the plugin will make unauthenticated requests. Should follow the established pattern:

{
  "mcpServers": {
    "cipp": {
      "type": "http",
      "url": "https://mcp.wyretechnology.com/v1/cipp/mcp",
      "headers": {
        "X-API-Key": "${CIPP_API_KEY}"
      }
    }
  }
}

If the gateway slug doesn't exist yet for CIPP, add a TODO noting that.

2. Double-nested cipp/cipp/ directory path

Should be cyberdrain/cipp/ (matching vendor/product pattern like kaseya/autotask/) or flat if treating CIPP as its own vendor namespace.

Other items

  • Missing type: "http" in .mcp.json
  • Inconsistent required field on command arguments (some commands omit it)
  • inforcer-tenant-summary.md has no arguments: [] in frontmatter
  • No Related Skills cross-linking between the 6 skills
  • agents/.gitkeep placeholder has no precedent in other plugins — remove unless there's a plan

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.

[Plugin Request] CIPP (CyberDrain Improved Partner Portal)

1 participant