Add Noted output dropdown option (#128)

* feat: Add dedicated Noted output channel for debug logging

- Created new LogService with output channel support
- Supports multiple log levels: Debug, Info, Warning, Error
- Added 'noted.debug' configuration setting to enable debug mode
- Integrated logging into extension activation and key operations
- Added 'Noted: Show Output Channel' command (noted.showOutput)
- Logger appears in Output panel dropdown (View > Output > Noted)
- Automatic disposal on extension deactivation
- Example logging added to openDailyNote function

This provides a centralized location for extension diagnostics and debug information.

* Update src/services/logService.ts

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Update src/extension.ts

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

* Update src/services/logService.ts

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

---------

Co-authored-by: Claude <[email protected]>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

Author: 3 people

package.json

@@ -393,6 +393,11 @@
         "title": "Noted: View What's New",
         "icon": "$(info)"
       },
+      {
+        "command": "noted.showOutput",
+        "title": "Noted: Show Output Channel",
+        "icon": "$(output)"
+      },
       {
         "command": "noted.showPreview",
         "title": "Noted: Open Preview (with Diagram Support)",
@@ -1156,6 +1161,11 @@
           "default": "md",
           "description": "File format for notes"
         },
+        "noted.debug": {
+          "type": "boolean",
+          "default": false,
+          "description": "Enable debug logging to the 'Noted' output channel (View > Output > Noted)"
+        },
         "noted.useTemplate": {
           "type": "boolean",
           "default": false,

src/extension.ts

@@ -93,6 +93,7 @@ import {
     handleEditBundle,
     handleDeleteBundle
 } from './commands/bundleCommands';
+import { logger, LogLevel } from './services/logService';
 import {
     handleCreatePromptTemplate,
     handleEditPromptTemplate,
@@ -119,9 +120,16 @@ import { FOLDER_PATTERNS, SPECIAL_FOLDERS, MONTH_NAMES, SUPPORTED_EXTENSIONS, ge
 import { showModalWarning, showModalInfo, StandardButtons, StandardDetails } from './utils/dialogHelpers';
 
 export function activate(context: vscode.ExtensionContext) {
-    console.log('Noted extension is now active');
-
+    // Initialize logger and add to subscriptions for proper disposal
     const config = vscode.workspace.getConfiguration('noted');
+    const debugMode = config.get<boolean>('debug', false);
+    if (debugMode) {
+        logger.setMinLogLevel(LogLevel.Debug);
+    }
+    context.subscriptions.push(logger);
+
+    logger.info('Noted extension activated');
+    logger.debug('Configuration loaded', { notesFolder: config.get('notesFolder') });
     const notesFolder = config.get<string>('notesFolder', 'Notes');
     const workspaceFolders = vscode.workspace.workspaceFolders;
 
@@ -1747,6 +1755,11 @@ export function activate(context: vscode.ExtensionContext) {
         await showChangelogView(context);
     });
 
+    // Command to show the Noted output channel (for debugging)
+    let showOutput = vscode.commands.registerCommand('noted.showOutput', () => {
+        logger.show(false); // Show the output channel and focus it
+    });
+
     // Command to show current notes folder configuration (for debugging)
     let showNotesConfig = vscode.commands.registerCommand('noted.showConfig', async () => {
         const config = vscode.workspace.getConfiguration('noted');
@@ -2204,7 +2217,7 @@ export function activate(context: vscode.ExtensionContext) {
         createTemplateWithAI, enhanceTemplate, selectAIModel,
         createBundle, createBundleFromTemplates, editBundle, deleteBundle,
         migrateTemplates, createUserStoryWithAI, showTemplateBrowserCmd,
-        createFolder, moveNote, renameFolder, deleteFolder, showCalendar, showGraph, showActivity, showVersionChangelog,
+        createFolder, moveNote, renameFolder, deleteFolder, showCalendar, showGraph, showActivity, showVersionChangelog, showOutput,
         togglePinNote, archiveNote, unarchiveNote, archiveOldNotes, rebuildBacklinks, clearBacklinks,
         toggleSelectMode, toggleNoteSelection, selectAllNotes, clearSelection, bulkDelete, bulkMove, bulkArchive, bulkMerge,
         createHierarchicalNote, openHierarchicalNote, searchHierarchicalNotes,
@@ -2632,6 +2645,7 @@ async function openDailyNote(templateType?: string) {
     const notesPath = getNotesPath();
     if (!notesPath) {
         vscode.window.showErrorMessage('Please open a workspace folder first');
+        logger.warn('openDailyNote: No workspace folder');
         return;
     }
 
@@ -2651,6 +2665,8 @@ async function openDailyNote(templateType?: string) {
         const noteFolder = path.join(notesPath, year, folderName);
         const filePath = path.join(noteFolder, fileName);
 
+        logger.debug('openDailyNote', { fileName, templateType });
+
         try {
             await fsp.access(noteFolder);
         } catch {
@@ -2660,10 +2676,12 @@ async function openDailyNote(templateType?: string) {
         let isNewNote = false;
         try {
             await fsp.access(filePath);
+            logger.info('Opening existing daily note', { fileName });
         } catch {
             const content = await generateTemplate(templateType, now);
             await fsp.writeFile(filePath, content);
             isNewNote = true;
+            logger.info('Created new daily note', { fileName, templateType });
         }
 
         const document = await vscode.workspace.openTextDocument(filePath);
@@ -2679,6 +2697,7 @@ async function openDailyNote(templateType?: string) {
             autoTagNewNote(filePath);
         }
     } catch (error) {
+        logger.error('Failed to open daily note', error instanceof Error ? error : new Error(String(error)));
         vscode.window.showErrorMessage(`Failed to open daily note: ${error instanceof Error ? error.message : String(error)}`);
     }
 }
@@ -3289,4 +3308,6 @@ async function previewTemplate() {
     `;
 }
 
-export function deactivate() {}
+export function deactivate() {
+    logger.info('Noted extension deactivated');
+}

src/services/logService.ts

@@ -0,0 +1,148 @@
+import * as vscode from 'vscode';
+
+/**
+ * Log levels for the output channel
+ */
+export enum LogLevel {
+	Debug = 'DEBUG',
+	Info = 'INFO',
+	Warning = 'WARN',
+	Error = 'ERROR'
+}
+
+/**
+ * Service for logging messages to the Noted output channel
+ * Provides a centralized logging mechanism visible in VS Code's Output panel
+ */
+export class LogService {
+	private static instance: LogService;
+	private outputChannel: vscode.OutputChannel;
+	private minLogLevel: LogLevel = LogLevel.Info;
+
+	private constructor() {
+		this.outputChannel = vscode.window.createOutputChannel('Noted');
+	}
+
+	/**
+	 * Get the singleton instance of LogService
+	 */
+	public static getInstance(): LogService {
+		if (!LogService.instance) {
+			LogService.instance = new LogService();
+		}
+		return LogService.instance;
+	}
+
+	/**
+	 * Set the minimum log level to display
+	 */
+	public setMinLogLevel(level: LogLevel): void {
+		this.minLogLevel = level;
+	}
+
+	/**
+	 * Get the current minimum log level
+	 */
+	public getMinLogLevel(): LogLevel {
+		return this.minLogLevel;
+	}
+
+	/**
+	 * Check if a log level should be logged based on current min level
+	 */
+	private shouldLog(level: LogLevel): boolean {
+		const levelOrder: Record<LogLevel, number> = {
+			[LogLevel.Debug]: 0, [LogLevel.Info]: 1,
+			[LogLevel.Warning]: 2, [LogLevel.Error]: 3
+		};
+		return levelOrder[level] >= levelOrder[this.minLogLevel];
+	}
+
+	/**
+	 * Format a log message with timestamp and level
+	 */
+	private formatMessage(level: LogLevel, message: string, data?: any): string {
+		const timestamp = new Date().toISOString().slice(11, 23); // Extracts HH:mm:ss.sss from YYYY-MM-DDTHH:mm:ss.sssZ
+		let formatted = `[${timestamp}] [${level}] ${message}`;
+
+		if (data !== undefined) {
+			if (data instanceof Error) {
+				formatted += `\n  Error: ${data.message}\n  Stack: ${data.stack}`;
+			} else if (typeof data === 'object') {
+				try {
+					formatted += `\n  ${JSON.stringify(data, null, 2)}`;
+				} catch (e) {
+					formatted += `\n  [Unable to stringify object]`;
+				}
+			} else {
+				formatted += `\n  ${data}`;
+			}
+		}
+
+		return formatted;
+	}
+
+	/**
+	 * Log a message to the output channel
+	 */
+	private log(level: LogLevel, message: string, data?: any): void {
+		if (!this.shouldLog(level)) {
+			return;
+		}
+
+		const formatted = this.formatMessage(level, message, data);
+		this.outputChannel.appendLine(formatted);
+	}
+
+	/**
+	 * Log a debug message (lowest priority)
+	 */
+	public debug(message: string, data?: any): void {
+		this.log(LogLevel.Debug, message, data);
+	}
+
+	/**
+	 * Log an info message
+	 */
+	public info(message: string, data?: any): void {
+		this.log(LogLevel.Info, message, data);
+	}
+
+	/**
+	 * Log a warning message
+	 */
+	public warn(message: string, data?: any): void {
+		this.log(LogLevel.Warning, message, data);
+	}
+
+	/**
+	 * Log an error message (highest priority)
+	 */
+	public error(message: string, error?: Error | any): void {
+		this.log(LogLevel.Error, message, error);
+	}
+
+	/**
+	 * Show the output channel in the UI
+	 */
+	public show(preserveFocus: boolean = true): void {
+		this.outputChannel.show(preserveFocus);
+	}
+
+	/**
+	 * Clear all messages from the output channel
+	 */
+	public clear(): void {
+		this.outputChannel.clear();
+	}
+
+	/**
+	 * Dispose of the output channel
+	 */
+	public dispose(): void {
+		this.outputChannel.dispose();
+	}
+}
+
+// Export a singleton instance for easy access
+export const logger = LogService.getInstance();