fix: improve error handling in main.ts and quickAddApi.ts

Author: chhoumann

src/main.ts

@@ -2,10 +2,11 @@ import type { TFile } from "obsidian";
 import { Plugin } from "obsidian";
 import { DEFAULT_SETTINGS, QuickAddSettingsTab } from "./quickAddSettingsTab";
 import type { QuickAddSettings } from "./quickAddSettingsTab";
-import { log, toError } from "./logger/logManager";
+import { log } from "./logger/logManager";
 import { ConsoleErrorLogger } from "./logger/consoleErrorLogger";
 import { GuiLogger } from "./logger/guiLogger";
 import { LogManager } from "./logger/logManager";
+import { reportError } from "./utils/errorUtils";
 import { StartupMacroEngine } from "./engine/StartupMacroEngine";
 import { ChoiceExecutor } from "./choiceExecutor";
 import type IChoice from "./types/choices/IChoice";
@@ -109,8 +110,9 @@ export default class QuickAdd extends Plugin {
 			const choice = this.getChoice("name", parameters.choice);
 
 			if (!choice) {
-				log.logError(
-					`URI could not find any choice named '${parameters.choice}'`,
+				reportError(
+					new Error(`URI could not find any choice named '${parameters.choice}'`),
+					"URI handler error"
 				);
 				return;
 			}
@@ -125,7 +127,7 @@ export default class QuickAdd extends Plugin {
 			try {
 				await choiceExecutor.execute(choice);
 			} catch (err) {
-				log.logError(toError(err));
+				reportError(err, "Error executing choice from URI");
 			}
 		});
 

src/quickAddApi.ts

@@ -7,7 +7,7 @@ import GenericCheckboxPrompt from "./gui/GenericCheckboxPrompt/genericCheckboxPr
 import type { IChoiceExecutor } from "./IChoiceExecutor";
 import type QuickAdd from "./main";
 import type IChoice from "./types/choices/IChoice";
-import { log } from "./logger/logManager";
+import { reportError } from "./utils/errorUtils";
 import { CompleteFormatter } from "./formatters/completeFormatter";
 import { getDate } from "./utilityObsidian";
 import { MarkdownView } from "obsidian";
@@ -70,7 +70,7 @@ export class QuickAddApi {
 			) => {
 				const choice: IChoice = plugin.getChoiceByName(choiceName);
 				if (!choice)
-					log.logError(`choice named '${choiceName}' not found`);
+					reportError(new Error(`Choice named '${choiceName}' not found`), "API executeChoice error");
 
 				if (variables) {
 					Object.keys(variables).forEach((key) => {
@@ -171,7 +171,7 @@ export class QuickAddApi {
 					);
 
 					if (!assistantRes) {
-						log.logError("AI Assistant returned null");
+						reportError(new Error("AI Assistant returned null"), "AI Prompt error");
 						return {};
 					}
 
@@ -262,7 +262,7 @@ export class QuickAddApi {
 					);
 
 					if (!assistantRes) {
-						log.logError("AI Assistant returned null");
+						reportError(new Error("AI Assistant returned null"), "Chunked AI Prompt error");
 						return {};
 					}
 
@@ -301,14 +301,12 @@ export class QuickAddApi {
 						app.workspace.getActiveViewOfType(MarkdownView);
 
 					if (!activeView) {
-						log.logError(
-							"no active view - could not get selected text."
-						);
+						reportError(new Error("No active view"), "Could not get selected text");
 						return;
 					}
 
 					if (!activeView.editor.somethingSelected()) {
-						log.logError("no text selected.");
+						reportError(new Error("No text selected"), "Could not get selected text");
 						return;
 					}
 
@@ -422,4 +420,4 @@ export class QuickAddApi {
 			return undefined;
 		}
 	}
-}
+}