[markdennis22]

Why?

The 2.17 and Preview specs for the audience and dynamic content endpoints didn't match the API. The role predicate examples showed values the API never returns, predicate.value was typed as a string, and several error responses were missing or worded differently from what the API sends. The Preview SDK is generated from the Preview spec, so its next regeneration changes Predicate.value from a string to unknown, because the API accepts and returns strings, numbers, booleans, arrays and null. Predicate also gains an optional predicates list, and deleteAudience, createDynamicContent and updateDynamicContent gain a 400 error. The stable SDKs aren't generated from 2.17, so they are unaffected.

How?

Applies the same edit to 2.17 and Preview. Corrects the audience and dynamic content endpoints and schemas to match the API's current behaviour, and adds the OAuth scopes each endpoint accepts to its description.

_{Generated with Claude Code}

[cathalging]

Approving: the 2.17 changes match how the API behaves. Preview (descriptions/0) still has the old Audiences and Dynamic Content text, though, including predicate.value as a string, the eq/user role example and the read-only scope lines. The next version cut would undo this fix, so a Preview change is needed as a follow-up. Once Preview is updated, Predicate.value stops being typed as a string in the Preview SDK, which is worth noting in the release notes.

Optional: per_page on list dynamic contents now has maximum: 50, which contradicts "values above 50 are reduced".

~ Automated via Claude (about: https://github.com/intercom/claude-plugins/blob/main/plugins/base/docs/external-message-attribution.md)

[cathalging]

Re-approving: the Preview changes match 2.17 and how the API behaves. Still optional: per_page with maximum: 50 suggests larger values are rejected, but they're reduced to 50.

~ Automated via Claude (about: https://github.com/intercom/claude-plugins/blob/main/plugins/base/docs/external-message-attribution.md)