Skip to content

docs(proposals): when to use which proposal surface#533

Merged
bokelley merged 2 commits into
mainfrom
bokelley/proposal-surface-docs
May 4, 2026
Merged

docs(proposals): when to use which proposal surface#533
bokelley merged 2 commits into
mainfrom
bokelley/proposal-surface-docs

Conversation

@bokelley
Copy link
Copy Markdown
Contributor

@bokelley bokelley commented May 4, 2026

Closes #517.

Summary

Adds docs/proposals/proposal-surfaces.md — a one-page picker that maps adopter goals to the right surface:

  • adcp.server.proposal — request-time helpers (AllocationBuilder, proposals_not_supported())
  • adcp.decisioning.ProposalManager — Protocol for the per-tenant proposal-side platform

Why

Salesagent migration feedback flagged the overlapping names without a one-page picker. The doc is structured as a "if you want to do X, reach for Y" table plus a quick decision rule for the common cases.

Test plan

  • Pre-commit hooks pass (markdown isn't linted but EOF/whitespace checks did)
  • Verified all referenced symbols (AllocationBuilder, proposals_not_supported, ProposalManager, MockProposalManager, ProposalCapabilities) exist in code
  • Verified examples/hello_proposal_manager.py is the canonical multi-platform composition example
  • Reviewer eye on accuracy of decision rule

🤖 Generated with Claude Code

bokelley and others added 2 commits May 3, 2026 21:25
Maps adopter goals to the right surface — adcp.server.proposal
(request-time helpers: AllocationBuilder, proposals_not_supported)
vs adcp.decisioning.ProposalManager (multi-tenant Protocol).
Salesagent migration feedback flagged the overlapping names as
confusing without a picker.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
…lder

- Move docs/proposals/proposal-surfaces.md → docs/proposal-surfaces.md
  (it's an adopter how-to, not an RFC; the proposals/ dir holds design
  documents).
- Add ProposalBuilder as the entry-point inline helper; demote
  AllocationBuilder to "what add_allocation hands back" (the
  higher-level builder was missing from the original).
- Reframe the decision rule so single-tenant + mock-backed proposals
  isn't accidentally framed as "never use ProposalManager."
- Point at adcontextprotocol/adcp for the bin/adcp.js mock-server.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@bokelley bokelley merged commit a9fd18a into main May 4, 2026
15 checks passed
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.

docs: 'when to use which proposal surface' guide

1 participant