chore(capture): apply review polish

Author: chhoumann

src/engine/canvasCapture.test.ts

@@ -1,7 +1,6 @@
 import { describe, expect, it } from "vitest";
 import type { App } from "obsidian";
 import {
-	applyCaptureToCanvasTextNode,
 	getCanvasActionUnsupportedReason,
 	getCanvasTextCaptureContent,
 	mergeCanvasTextAtBottom,
@@ -260,28 +259,6 @@ describe("canvasCapture", () => {
 		expect(mergeCanvasTextAtBottom("line1\n", "line2")).toBe("line1\nline2");
 	});
 
-	it("applies capture to text nodes via setText when available", () => {
-		let lastValue = "";
-		const target = {
-			kind: "text",
-			source: "active",
-			canvas: {
-				requestSave: () => {},
-			},
-			canvasFile: { path: "Canvas.canvas", basename: "Canvas" },
-			node: {
-				type: "text",
-				text: "existing",
-				setText: (value: string) => {
-					lastValue = value;
-				},
-			},
-			nodeData: null,
-		} as const;
-
-		applyCaptureToCanvasTextNode(target as any, "capture", "activeFileTop");
-		expect(lastValue).toBe("capture\nexisting");
-	});
 
 	it("reports unsupported reasons by action", () => {
 		expect(

src/engine/canvasCapture.ts

@@ -436,22 +436,3 @@ export function mergeCanvasTextAtBottom(
 	return `${existingText}${needsSeparator ? "\n" : ""}${captureContent}`;
 }
 
-export function applyCaptureToCanvasTextNode(
-	target: Extract<CanvasCaptureTarget, { kind: "text" }>,
-	captureContent: string,
-	action: CaptureAction,
-): void {
-	const existingText = getCanvasTextCaptureContent(target);
-	const nextText =
-		action === "activeFileTop"
-			? mergeCanvasTextAtTop(existingText, captureContent)
-			: mergeCanvasTextAtBottom(existingText, captureContent);
-
-	if (typeof target.node.setText === "function") {
-		target.node.setText(nextText);
-	} else {
-		target.node.text = nextText;
-	}
-
-	target.canvas.requestSave?.();
-}

src/formatters/captureChoiceFormatter.ts

@@ -120,6 +120,8 @@ export class CaptureChoiceFormatter extends CompleteFormatter {
 		const formattedContentIsEmpty = formatted.trim() === "";
 		if (formattedContentIsEmpty) return this.fileContent;
 
+		// Historical note: `prepend` is a legacy flag name that means
+		// append-to-bottom behavior.
 		const shouldAppendToBottom =
 			this.choice.prepend ||
 			(this.choice.captureToActiveFile &&

src/gui/ChoiceBuilder/captureChoiceBuilder.ts

@@ -440,7 +440,7 @@ export class CaptureChoiceBuilder extends ChoiceBuilder {
 			const canvasFile = this.resolveStaticCanvasTargetFile(canvasTargetPath);
 			if (!canvasFile) {
 				renderEmpty(
-					"Node picker works for direct .canvas file paths that exist in your vault.",
+					"Node picker works for direct .canvas paths that already exist in your vault. Format syntax paths cannot be listed here.",
 				);
 				return;
 			}

src/styles.css

@@ -209,7 +209,6 @@
 	background: var(--background-primary);
 }
 
-
 .checkboxRowContainer {
 	margin: 30px 0px;
 	display: grid;