Reduce notification popups in Roopik extension

Replaces several user-facing notification popups with logger calls or removes them entirely in the Roopik extension. This change aims to reduce notification noise by logging info and warning messages instead of showing popups, and by omitting non-essential notifications for actions like import 'coming soon', canvas deletion, renaming, and project removal. Error notifications remain to inform users of failures.

Author: HumbleBee14

extensions/roopik/src/canvasPanel.ts

@@ -518,12 +518,14 @@ export class CanvasPanel implements vscode.Disposable {
 		level: 'info' | 'warning' | 'error';
 		message: string;
 	}): void {
+		// Log to output channel instead of showing notification popup
 		switch (payload.level) {
 			case 'info':
-				vscode.window.showInformationMessage(payload.message);
+				this.logger.info(payload.message);
 				break;
 			case 'warning':
-				vscode.window.showWarningMessage(payload.message);
+				// vscode.window.showWarningMessage(payload.message);
+				this.logger.warn(payload.message);
 				break;
 			case 'error':
 				vscode.window.showErrorMessage(payload.message);

extensions/roopik/src/config.ts

@@ -5,7 +5,6 @@
 
 import * as fs from 'fs';
 import * as path from 'path';
-import * as vscode from 'vscode';
 import { Logger } from './logger';
 
 /**
@@ -205,10 +204,7 @@ export class ConfigManager {
 				return this.mergeWithDefaults(userConfig);
 			}
 		} catch (error) {
-			this.logger.error('Failed to load config', error);
-			vscode.window.showWarningMessage(
-				'Failed to load Roopik config. Using defaults.'
-			);
+			this.logger.error('Failed to load config, using defaults', error);
 		}
 
 		// Return default config if file doesn't exist or failed to load
@@ -274,7 +270,7 @@ export class ConfigManager {
 			this.logger.info('Configuration updated successfully');
 		} catch (error) {
 			this.logger.error('Failed to save config', error);
-			vscode.window.showErrorMessage('Failed to save Roopik configuration.');
+			throw error;
 		}
 	}
 

src/vs/workbench/contrib/roopik/browser/commands/browserCommands.ts

@@ -212,10 +212,6 @@ export function registerBrowserCommands(): void {
 			}
 
 			const projectPath = args.projectPath;
-			const projectName = projectPath.split(/[\\/]/).pop() || 'Project';
-
-			// Show starting notification
-			notificationService.info(`Starting project: ${projectName}...`);
 
 			try {
 				// ============================================

src/vs/workbench/contrib/roopik/browser/commands/importCommands.ts

@@ -139,9 +139,7 @@ export function registerImportCommands(): void {
 				case 'github':
 				case 'figma':
 				case 'third-party':
-					notificationService.info(
-						localize('roopik.import.comingSoon', '{0} import is coming soon!', selectedSource.label.replace(/\$\([^)]+\)\s*/, ''))
-					);
+					// Coming soon - no notification needed
 					break;
 			}
 		}

src/vs/workbench/contrib/roopik/browser/roopikViewPane.ts

@@ -5,6 +5,7 @@
 
 import { localize, localize2 } from '../../../../nls.js';
 import { Disposable } from '../../../../base/common/lifecycle.js';
+import { mainWindow } from '../../../../base/browser/window.js';
 import { IWorkbenchContribution } from '../../../common/contributions.js';
 import { Registry } from '../../../../platform/registry/common/platform.js';
 import { IViewsRegistry, IViewDescriptor, Extensions, ViewContainer, IViewContainersRegistry, ViewContainerLocation, IViewDescriptorService } from '../../../common/views.js';
@@ -429,7 +430,6 @@ export class RoopikDashboardView extends ViewPane {
 		if (confirmed === canvasName) {
 			try {
 				await this.canvasService.deleteCanvas(canvasId);
-				this.notificationService.info(localize('roopik.deleteCanvas.success', 'Canvas "{0}" deleted', canvasName));
 			} catch (err) {
 				const errorMsg = err instanceof Error ? err.message : String(err);
 				this.notificationService.error(localize('roopik.deleteCanvas.error', 'Failed to delete canvas: {0}', errorMsg));
@@ -456,7 +456,6 @@ export class RoopikDashboardView extends ViewPane {
 		if (newName && newName !== currentName) {
 			try {
 				await this.canvasService.updateCanvas(canvasId, { name: newName });
-				this.notificationService.info(localize('roopik.renameCanvas.success', 'Canvas renamed to "{0}"', newName));
 			} catch (err) {
 				const errorMsg = err instanceof Error ? err.message : String(err);
 				this.notificationService.error(localize('roopik.renameCanvas.error', 'Failed to rename canvas: {0}', errorMsg));
@@ -602,7 +601,6 @@ export class RoopikDashboardView extends ViewPane {
 	private async deleteProject(projectId: string, projectName: string): Promise<void> {
 		try {
 			await this.projectStorageService.deleteProject(projectId);
-			this.notificationService.info(localize('roopik.deleteProject.success', 'Removed "{0}" from recent projects', projectName));
 		} catch (err) {
 			const errorMsg = err instanceof Error ? err.message : String(err);
 			this.notificationService.error(localize('roopik.deleteProject.error', 'Failed to remove project: {0}', errorMsg));
@@ -835,7 +833,7 @@ export class RoopikDashboardView extends ViewPane {
 				50% { transform: scale(1.15); }
 			}
 		`;
-		document.head.appendChild(style);
+		mainWindow.document.head.appendChild(style);
 		RoopikDashboardView.animationsInjected = true;
 	}