Summary

The Mastra target-state diagram in docs/design/crm-custom-object-linkage-leads.md showed boundObjectApiName: "deals" on Mastra's Deals AOP. The actual implementation never does this , DEAL_OBJECT_API_NAME (bound-object-resolution.ts) is a single 'Opportunity' sentinel shared across all three provider drivers, not each provider's own wire slug for its Deal-equivalent object. attio.ts's updateOpportunityFields explicitly checks against that shared sentinel, confirmed by its own inline comment.

Literally following the stale diagram later (e.g. setting Mastra's real Deals AOP's boundObjectApiName to the literal string "deals") would make resolveBoundAopForObject/resolveAopReassignmentTarget treat it as a non-Deal object and silently stop resolving it as the Deals AOP , the exact class of regression this doc's Phase 0 fix exists to prevent, just in the opposite direction. Fixed the diagram to show null (the correct, implicit default) with an explanation.

Docs-only change, no code touched.

🤖 Generated with Claude Code

Confidence Score: 4/5

The documentation correction is safe to merge, with a non-blocking wording fix recommended so it does not imply all provider drivers import the shared constant.

Findings

1.  Provider constant usage overstated ▶

### Issue 1
docs/design/crm-custom-object-linkage-leads.md:110-113
This says every provider driver compares against the shared `DEAL_OBJECT_API_NAME` constant, but only Attio imports it. Salesforce and HubSpot each compare against a hardcoded `'Opportunity'` literal. Describing this as a shared sentinel value rather than a shared constant would avoid leading maintainers to assume that changing the constant updates all three drivers.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

This documentation-only PR corrects the Mastra Deals AOP diagram to show its implicit null binding and explains how Deal resolution uses the 'Opportunity' sentinel.

• The diagram now matches the stored default and resolver behavior.
• The explanation accurately warns that using Attio's "deals" wire slug would disable Deals fallback resolution.
• One sentence overstates use of the shared constant across provider implementations.

_{Reviews (1) · Last reviewed commit: "docs(crm): fix stale Mastra Deals-AOP di..."}