Skip to content

Conversation

@bokelley
Copy link
Contributor

Summary

Add four commonly-used types to the stable public API that were previously only accessible from internal modules:

  • FormatId: Format identifier (agent_url + id)
  • PackageRequest: Package creation request structure
  • PriceGuidance: Pricing guidance for auction-based options
  • PushNotificationConfig: Webhook push notification configuration

These types are referenced in workflows and should be available from the stable API to prevent users from importing internal generated types.

Test plan

  • ✅ All types import successfully from adcp package
  • ✅ All types import successfully from adcp.types.stable
  • ✅ All types appear in __all__ exports
  • ✅ Types are correctly aliased to generated counterparts
  • ✅ All types are valid Pydantic models with expected fields

…iceGuidance from stable API

Add missing types to stable.py public API:
- FormatId: Format identifier type (agent_url + id)
- PackageRequest: Request structure for creating packages
- PushNotificationConfig: Webhook push notification configuration
- PriceGuidance: Pricing guidance for auction-based options (floor, percentiles)

These types are commonly used in AdCP workflows and should be part of
the stable public API to avoid users importing from internal generated modules.

Changes:
- Add types to adcp.types.stable imports and __all__
- Add types to adcp.__init__.py imports and __all__
- Group PriceGuidance with other pricing types
- Add new "Configuration types" section for PushNotificationConfig

All types are properly exported from both stable.py and main package.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@bokelley bokelley merged commit 654f882 into main Nov 19, 2025
7 checks passed
@bokelley bokelley deleted the add-missing-stable-types branch November 19, 2025 02:24
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