Skip to content

Conversation

@bokelley
Copy link
Contributor

Summary

Improves developer experience by providing semantic type names that match the AdCP specification. Users can now import cleanly from the main adcp package without relying on internal modules.

  • Semantic aliases: CreateMediaBuySuccessResponse instead of CreateMediaBuyResponse1
  • Public API exports: All commonly-used request/response types re-exported from main package
  • Type safety: Added PEP 561 py.typed marker for full type checker support
  • Documentation: Added import guidelines and examples to README

Test plan

  • ✅ All 220 existing tests pass
  • ✅ Added 9 new tests for type alias functionality
  • ✅ Updated discriminated union tests to use semantic aliases
  • ✅ All linters and formatters pass
  • ✅ Type checking passes

🤖 Generated with Claude Code

Improve developer experience by providing semantic type names that match the AdCP spec:
- CreateMediaBuySuccessResponse / CreateMediaBuyErrorResponse instead of Response1/2
- Re-export all commonly-used request/response types from main package
- Add PEP 561 py.typed marker for full type checker support
- Update documentation with import guidelines and examples

Users can now import cleanly from adcp without relying on internal modules.

🤖 Generated with Claude Code

Co-Authored-By: Claude <noreply@anthropic.com>
@bokelley bokelley merged commit 5c63dec into main Nov 16, 2025
7 checks passed
bokelley added a commit that referenced this pull request Nov 18, 2025
Improve developer experience by providing semantic type names that match the AdCP spec:
- CreateMediaBuySuccessResponse / CreateMediaBuyErrorResponse instead of Response1/2
- Re-export all commonly-used request/response types from main package
- Add PEP 561 py.typed marker for full type checker support
- Update documentation with import guidelines and examples

Users can now import cleanly from adcp without relying on internal modules.

🤖 Generated with Claude Code

Co-authored-by: Claude <noreply@anthropic.com>
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.

2 participants