feat: Add DON ID type validation and Rev Orgs service (CUSS-396, CUSS-397)

[mgmonteleone]

Overview

This PR implements two related enhancements to improve the DevRev MCP tools and SDK:

1. CUSS-396: DON ID type validation with helpful error messages
2. CUSS-397: Rev Orgs (revenue organizations) service

Problem

Users were getting generic "Validation error: Bad Request" when passing the wrong type of DON ID to MCP tools (e.g., passing a rev_org ID revo/... to devrev_accounts_get which expects an account ID account/...). Additionally, there was no way to look up rev_orgs directly.

What Changed

CUSS-396: DON ID Type Validation

New file: src/devrev_mcp/utils/don_id.py

• parse_don_type(don_id) — extracts the object type segment from a DON ID
• validate_don_id(don_id, expected_types, tool_name) — soft-validates DON IDs with clear error messages
• DON_TYPE_MAP — maps DON type segments to human-friendly names
• TOOL_SUGGESTIONS — maps DON types to the correct MCP tool for helpful suggestions

Modified: 14 MCP tool files — added validate_don_id() calls to all _get, _update, _delete operations:

• accounts.py, articles.py, conversations.py, engagements.py, groups.py, incidents.py, links.py, parts.py, question_answers.py, slas.py, tags.py, timeline.py, users.py, works.py

Example error message:

The provided ID 'don:identity:dvrv-us-1:devo/11Ca9baGrM:revo/CHjPMe8K' appears to be a rev_org ID,
but devrev_accounts_get expects an account ID. Try using devrev_rev_orgs_get instead.

CUSS-397: Rev Orgs Service

New files:

• src/devrev/models/rev_orgs.py — Pydantic v2 models (RevOrg, RevOrgSummary, request/response models)
• src/devrev/services/rev_orgs.py — Sync RevOrgsService and async AsyncRevOrgsService
• src/devrev_mcp/tools/rev_orgs.py — MCP tools (list, get, create, update, delete)

Modified:

• src/devrev/client.py — Added rev_orgs property to both clients
• src/devrev/models/__init__.py — Exported rev_orgs models
• src/devrev/services/__init__.py — Exported rev_orgs services
• src/devrev_mcp/server.py — Registered rev_orgs tools

Testing

• 22 new tests for DON ID validation (tests/unit/test_don_id_validation.py)
• 19 new tests for Rev Orgs models and services (tests/unit/test_rev_orgs.py)
• All 1091 tests pass — zero regressions
• Ruff linting clean

Deployment Notes

• No breaking changes — validation is soft (non-DON IDs pass through silently)
• New MCP tools are automatically registered
• Destructive rev_orgs tools (create/update/delete) are behind the enable_destructive_tools config flag

Related Issues

• Implements: CUSS-396
• Implements: CUSS-397

---

Pull Request opened by Augment Code with guidance from the PR author