Merge pull request #3387 from Kilo-Org/kevinvandijk/notification-types

Add support for notifications per app

Author: RSO

cli/src/state/atoms/notifications.ts

@@ -11,6 +11,7 @@ export interface KilocodeNotification {
 		actionText: string
 		actionURL: string
 	}
+	showIn?: string[]
 }
 
 /**

cli/src/utils/notifications.ts

@@ -52,7 +52,7 @@ export async function fetchKilocodeNotifications({
 
 	const { notifications } = (await response.json()) as NotificationsResponse
 
-	return notifications
+	return notifications.filter(({ showIn }) => !showIn || showIn.includes("cli"))
 }
 
 /**

src/core/kilocode/webview/webviewMessageHandlerUtils.ts

@@ -111,13 +111,13 @@ export const fetchKilocodeNotificationsHandler = async (provider: ClineProvider)
 			(notification: any) =>
 				!dismissedIds.includes(notification.id) &&
 				!shownNativeNotificationIds.has(notification.id) &&
-				notification.showAsNative === true,
+				(notification.showIn ?? []).includes("extension-native"),
 		)
 
 		provider.postMessageToWebview({
 			type: "kilocodeNotificationsResponse",
 			notifications: (response.data?.notifications || []).filter(
-				(notification: any) => notification.showAsNative !== true,
+				({ showIn }: { showIn?: string[] }) => !showIn || showIn.includes("extension"),
 			),
 		})