refactor: simplify logging to two levels, error and debug

Collapse the five-level scheme (DEBUG/INFO/WARN/ERROR/FEAT) down to just
DEBUG and ERROR on both sides of the WASM boundary. warn/info/feat call
sites become debug (internal diagnostics, not user-actionable) or error
(genuine caught exceptions), matching Obsidian's guideline that only
errors should show by default.

The Rust side binds directly to src/logger/index.ts's `log` singleton via
wasm_bindgen and was calling .info()/.warn() on it — a prior TS-only pass
at removing these methods broke at the WASM boundary (`arg0.info is not a
function`) since Rust still called them. Fixed by updating the extern
binding in wasm/src/utils.rs and every call site, then rebuilding wasm/pkg.

Author: michaelpporter

src/codeblocks/MDRC.ts

@@ -57,7 +57,7 @@ export class CodeblockMDRC extends MarkdownRenderChild {
 		});
 
 		if (!parsed) {
-			log.warn(
+			log.debug(
 				"fatal codeblock errors\n" +
 					errors
 						.map((e) => `  [${e.code}] ${e.path}: ${e.message}`)
@@ -80,7 +80,7 @@ export class CodeblockMDRC extends MarkdownRenderChild {
 		);
 		// Although the postprocessing could also have errors,
 		// they're not fatal at this point, so we can still render the codeblock (which renders the errors as well)
-		if (errors.length) log.warn("non-fatal codeblock errors", errors);
+		if (errors.length) log.debug("non-fatal codeblock errors", errors);
 
 		if (options.type === "tree") {
 			this.component = mount(CodeblockTree, {

src/codeblocks/index.ts

@@ -111,7 +111,7 @@ function postprocess_options(
 	}
 
 	if (parsed["dataview-from"]) {
-		log.warn(
+		log.debug(
 			"Codeblock field `dataview-from` is deprecated; use `from` instead.",
 		);
 	}

src/commands/init.ts

@@ -51,13 +51,11 @@ export function init_all_commands(plugin: BreadcrumbsPlugin) {
 			const stats = get_graph_stats(plugin.graph, {
 				groups: plugin.settings.edge_field_groups,
 			});
-			log.feat("Graph stats >", stats);
+			log.debug("Graph stats >", stats);
 
 			await navigator.clipboard.writeText(JSON.stringify(stats, null, 2));
 
-			new Notice(
-				"Graph stats printed to console and copied to clipboard",
-			);
+			new Notice("Graph stats copied to clipboard");
 		},
 	});
 

src/graph/builders/explicit/dataview_note.ts

@@ -95,7 +95,7 @@ export const _add_explicit_edges_dataview_note: ExplicitEdgeBuilder = (
 					dataview_api.pages(query, dataview_note_file.path),
 				) as IDataview.Page[];
 			} catch (error) {
-				log.warn(
+				log.debug(
 					"dataview-note > DV API error:",
 					error instanceof Error ? error.message : error,
 				);

src/graph/builders/explicit/tag_note.ts

@@ -34,7 +34,7 @@ const get_tag_note_info = (
 		raw_tag = metadata["BC-tag-note"];
 
 		if (raw_tag) {
-			log.warn(
+			log.debug(
 				`'BC-tag-note' is deprecated in favor of ${META_ALIAS["tag-note-tag"]}`,
 			);
 		}

src/logger/index.ts

@@ -1,19 +1,9 @@
-export const LOG_LEVELS = [
-	"DEBUG",
-	"INFO",
-	"WARN",
-	"ERROR",
-	// Some features log data
-	"FEAT",
-] as const;
+export const LOG_LEVELS = ["DEBUG", "ERROR"] as const;
 export type LogLevels = (typeof LOG_LEVELS)[number];
 
 const LEVEL_COLOURS: Record<LogLevels, string | null> = {
 	DEBUG: "#999",
-	INFO: null,
-	WARN: "#f90",
 	ERROR: "#f00",
-	FEAT: "#0f0",
 };
 
 const build_prefix = (level: LogLevels) => {
@@ -41,35 +31,15 @@ class Logger {
 		}
 	}
 
-	info(...args: unknown[]) {
-		if (this.level_i <= 1) {
-			console.log(...build_prefix("INFO"), ...args);
-		}
-	}
-
-	warn(...args: unknown[]) {
-		if (this.level_i <= 2) {
-			// NOTE: Don't actually console.warn
-			// The user doesn't need a stack trace
-			console.log(...build_prefix("WARN"), ...args);
-		}
-	}
-
 	error(...args: unknown[]) {
-		if (this.level_i <= 3) {
+		if (this.level_i <= 1) {
 			console.log(...build_prefix("ERROR"), ...args);
 		}
 	}
 
-	feat(...args: unknown[]) {
-		if (this.level_i === 4) {
-			console.log(...build_prefix("FEAT"), ...args);
-		}
-	}
-
 	set_level(level: LogLevels) {
 		this.level_i = LOG_LEVELS.findIndex((l) => l === level);
 	}
 }
 
-export const log = new Logger("INFO");
+export const log = new Logger("ERROR");

src/main.ts

@@ -104,9 +104,6 @@ export default class BreadcrumbsPlugin extends Plugin {
 		// Logger
 		log.set_level(this.settings.debug.level);
 
-		log.info(
-			`loading plugin "${this.manifest.name}" plugin v${this.manifest.version}`,
-		);
 		log.debug("settings >", this.settings);
 
 		// Init event bus
@@ -358,7 +355,6 @@ export default class BreadcrumbsPlugin extends Plugin {
 				backup_path,
 				JSON.stringify(this.settings, null, "\t"),
 			);
-			log.info(`old settings backed up to ${backup_path}`);
 		}
 	}
 
@@ -386,7 +382,7 @@ export default class BreadcrumbsPlugin extends Plugin {
 			);
 
 		if (Object.keys(explicit_edge_errors).length) {
-			log.warn("explicit_edge_errors >", explicit_edge_errors);
+			log.debug("explicit_edge_errors >", explicit_edge_errors);
 		}
 
 		notice?.setMessage(
@@ -433,7 +429,7 @@ export default class BreadcrumbsPlugin extends Plugin {
 					: workspace.getRightLeaf(false);
 
 			if (!leaf) {
-				log.warn("activate_view > no leaf found");
+				log.debug("activate_view > no leaf found");
 				return;
 			}
 

src/settings/migration.ts

@@ -281,7 +281,7 @@ export function migrate_old_settings(settings: BreadcrumbsSettings) {
 	) {
 		if (old.hierarchyNotes.length > 0) {
 			const msg = `DEPRECATED: The central Hierarchy Notes setting is deprecated in favour of the "${META_ALIAS["list-note-field"]}" field in each hierarchy note.`;
-			log.warn(msg);
+			log.debug(msg);
 		}
 
 		delete old.HNUpField;

src/utils/obsidian.ts

@@ -100,12 +100,12 @@ export const copy_to_clipboard = async (
 	const resolved = Object.assign({ notify: true, log: true }, options);
 
 	if (resolved.log) {
-		log.feat(text);
+		log.debug(text);
 	}
 
 	await navigator.clipboard.writeText(text);
 
 	if (resolved.notify) {
-		new Notice("Copied to clipboard and logged to console.");
+		new Notice("Copied to clipboard.");
 	}
 };