Show toast notification when skills are added or removed

.changeset/polite-results-attend.md

@@ -0,0 +1,5 @@
+---
+"kilo-code": minor
+---
+
+Show notifications when skills are added or removed from the project or global config

packages/types/src/context-config.ts

@@ -0,0 +1,25 @@
+// kilocode_change - new file
+
+/**
+ * All discoverable configuration types that affect agent behavior.
+ * These are things that can be discovered from .kilocode directories.
+ */
+export type ContextConfigType = "skill" | "workflow" | "command" | "rule" | "mcp"
+
+/**
+ * Represents a single configuration change (added or removed).
+ */
+export interface ContextConfigChange {
+	/** What kind of configType this is */
+	configType: ContextConfigType
+	/** Whether this item was added or removed */
+	changeType: "added" | "removed"
+	/** Name/identifier of the item */
+	name: string
+	/** Where was it discovered: global (~/.kilocode) or project (.kilocode) */
+	source: "global" | "project"
+	/** Optional mode for mode-specific configs (e.g., 'code', 'architect') */
+	mode?: string
+	/** Optional file path for "click to open" functionality */
+	filePath?: string
+}

packages/types/src/index.ts

@@ -1,5 +1,6 @@
 export * from "./api.js"
 export * from "./auto-purge.js" // kilocode_change
+export * from "./context-config.js" // kilocode_change
 export * from "./cloud.js"
 export * from "./codebase-index.js"
 export * from "./context-management.js"

src/i18n/locales/en/kilocode.json

@@ -2,6 +2,10 @@
 	"info": {
 		"settings_imported": "Settings imported successfully."
 	},
+	"configDiscovery": {
+		"added": "Kilo Code: Added {{source}} {{configType}}: {{name}}{{modeStr}}",
+		"removed": "Kilo Code: Removed {{source}} {{configType}}: {{name}}{{modeStr}}"
+	},
 	"userFeedback": {
 		"message_update_failed": "Failed to update message",
 		"no_checkpoint_found": "No checkpoint found before this message",

src/services/config/ConfigChangeNotifier.spec.ts

@@ -0,0 +1,64 @@
+// kilocode_change - new file
+
+import { describe, it, expect, vi, beforeEach } from "vitest"
+import type { ClineProvider } from "../../core/webview/ClineProvider"
+import { ConfigChangeNotifier } from "./ConfigChangeNotifier"
+
+const { mockShowInformationMessage } = vi.hoisted(() => ({
+	mockShowInformationMessage: vi.fn(),
+}))
+
+vi.mock("vscode", () => ({
+	window: { showInformationMessage: mockShowInformationMessage },
+}))
+
+vi.mock("../../i18n", () => ({
+	t: vi.fn((key, vars) => `${key} ${JSON.stringify(vars)}`),
+}))
+
+describe("ConfigChangeNotifier", () => {
+	let mockProvider: ClineProvider
+	let notifier: ConfigChangeNotifier
+
+	beforeEach(() => {
+		vi.clearAllMocks()
+		mockProvider = { cwd: "/test" } as unknown as ClineProvider
+		notifier = new ConfigChangeNotifier(mockProvider)
+	})
+
+	it("should skip initial discovery without notifying", async () => {
+		await notifier.notifyIfChanged("skill", [{ name: "test-skill", source: "global" }])
+		expect(mockShowInformationMessage).not.toHaveBeenCalled()
+	})
+
+	it("should detect added configurations", async () => {
+		await notifier.notifyIfChanged("skill", [{ name: "existing-skill", source: "global" }])
+		await notifier.notifyIfChanged("skill", [
+			{ name: "existing-skill", source: "global" },
+			{ name: "new-skill", source: "global" },
+		])
+		expect(mockShowInformationMessage).toHaveBeenCalledTimes(1)
+		expect(mockShowInformationMessage).toHaveBeenCalledWith(expect.stringContaining("new-skill"))
+	})
+
+	it("should detect removed configurations", async () => {
+		await notifier.notifyIfChanged("skill", [
+			{ name: "existing-skill", source: "global" },
+			{ name: "removed-skill", source: "global" },
+		])
+		await notifier.notifyIfChanged("skill", [{ name: "existing-skill", source: "global" }])
+		expect(mockShowInformationMessage).toHaveBeenCalledTimes(1)
+		expect(mockShowInformationMessage).toHaveBeenCalledWith(expect.stringContaining("removed-skill"))
+	})
+
+	it("should track different config types separately", async () => {
+		await notifier.notifyIfChanged("skill", [{ name: "skill-a", source: "global" }])
+		await notifier.notifyIfChanged("workflow", [{ name: "workflow-a", source: "global" }])
+		await notifier.notifyIfChanged("skill", [
+			{ name: "skill-a", source: "global" },
+			{ name: "skill-b", source: "global" },
+		])
+		expect(mockShowInformationMessage).toHaveBeenCalledTimes(1)
+		expect(mockShowInformationMessage).toHaveBeenCalledWith(expect.stringContaining("skill-b"))
+	})
+})