❓ Just want to validate that I'm understanding correctly, this updates ChatPostMessageArguments from

interface ChatPostMessageArguments {
  ...
  metadata?:  {
    event_type: string;
    event_payload: {
      [key: string]: string | number | boolean | MessageMetadataEventPayloadObject | MessageMetadataEventPayloadObject[];
    };
}

To

interface ChatPostMessageArguments {
  ...
  metadata?:  {
    event_type?: string;
    event_payload?: {
      [key: string]: string | number | boolean | MessageMetadataEventPayloadObject | MessageMetadataEventPayloadObject[];
    };
    entities?: EntityMetadata[];
}

Since this type is used to define and input for a function, making event_type and event_payload optional should not be a breaking change?

Yup; what I'm trying to show is that one can provide event and / or entity metadata in chat.postMessage

interface ChatPostMessageArguments {
  ...
  metadata?:  {

   // if providing event metadata, both of these params are still required (but they're optional in the type bc one can provide entity metadata instead)
    event_type: string;
    event_payload: {
      [key: string]: string | number | boolean | MessageMetadataEventPayloadObject | MessageMetadataEventPayloadObject[];
    };


AND / OR

   // if providing entity metadata, this param is required (but it's optional in the type bc one can provide event metadata instead)
    entities: EntityMetadata[];
}

Since this type is used to define and input for a function, making event_type and event_payload optional should not be a breaking change?

Passing an object into the metadata param with just event_type and event_payload is still valid and should still be accepted by the SDK; if users are typing the object as Metadata though they would need to change that to ChatPostMessageMetadata. Similar thing came up for the Java update; changing the type of an existing field does seem like a breaking change 😅 Will work object support require a major version bump because of this?

Will work object support require a major version bump because of this?

The the Java SDK I need to take a look there might be a way around this

++ on praise for this 🙏

QQ: In ChatPostMessageMetadata, entities is optional, just want to confirm that here is should be required

Nice 💯