fix(types): metadata should be optional (#646)

dungdong-aws · web-flow · commit c8cb47fc9aa8 · 2025-08-01T13:07:41.000-07:00
## Problem

This new field must be optional to not break other feature

## Solution

update this to optional

&lt;!---
    REMINDER:
    - Read CONTRIBUTING.md first.
    - Add test coverage for your changes.
    - Link to related issues/commits.
    - Testing: how did you test your changes?
    - Screenshots if applicable
--&gt;

## License

By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache 2.0 license.
diff --git a/types/chat.ts b/types/chat.ts
@@ -401,7 +401,7 @@ export interface ButtonClickParams {
     tabId: string
     messageId: string
     buttonId: string
-    metadata: Record<string, string>
+    metadata?: Record<string, string>
 }
 
 export interface ButtonClickResult {

Original file line number	Diff line number	Diff line change
`@@ -401,7 +401,7 @@ export interface ButtonClickParams {`
`401`	`401`	`tabId: string`
`402`	`402`	`messageId: string`
`403`	`403`	`buttonId: string`
`404`		`- metadata: Record<string, string>`
	`404`	`+ metadata?: Record<string, string>`
`405`	`405`	`}`
`406`	`406`
`407`	`407`	`export interface ButtonClickResult {`