fix(capture): allow vault picker and clear stale canvas node ids

Author: chhoumann

src/engine/CaptureChoiceEngine.ts

@@ -314,12 +314,12 @@ export class CaptureChoiceEngine extends QuickAddChoiceEngine {
 			return null;
 		}
 
-		invariant(
-			this.choice.captureTo?.trim().length > 0,
-			"Canvas node capture requires a target .canvas file path.",
-		);
+		const rawCaptureTo = this.choice.captureTo?.trim() ?? "";
+		if (!rawCaptureTo) {
+			return null;
+		}
 
-		const targetPath = await this.formatFilePath(this.choice.captureTo);
+		const targetPath = await this.formatFilePath(rawCaptureTo);
 		const isCanvasTarget = CANVAS_FILE_EXTENSION_REGEX.test(targetPath);
 		const nodeId = this.choice.captureToCanvasNodeId?.trim() ?? "";
 

src/gui/ChoiceBuilder/captureChoiceBuilder.ts

@@ -208,11 +208,17 @@ export class CaptureChoiceBuilder extends ChoiceBuilder {
 					(el) => new FormatSyntaxSuggester(this.app, el, this.plugin),
 				],
 				onChange: async (value) => {
+					const previousCanvasPath = this.normalizeVaultPath(this.choice.captureTo);
 					const wasCanvasTarget = this.isCanvasTargetPath(this.choice.captureTo);
 					this.choice.captureTo = value;
+					const nextCanvasPath = this.normalizeVaultPath(value);
 					const isCanvasTarget = this.isCanvasTargetPath(value);
+					const canvasPathChanged =
+						wasCanvasTarget &&
+						isCanvasTarget &&
+						previousCanvasPath !== nextCanvasPath;
 
-					if (!isCanvasTarget) {
+					if (!isCanvasTarget || canvasPathChanged) {
 						this.choice.captureToCanvasNodeId = "";
 					}
 
@@ -222,7 +228,7 @@ export class CaptureChoiceBuilder extends ChoiceBuilder {
 						formatDisplay.textContent = "Preview unavailable";
 					}
 
-					if (wasCanvasTarget !== isCanvasTarget) {
+					if (wasCanvasTarget !== isCanvasTarget || canvasPathChanged) {
 						this.reload();
 					}
 				},