fix: tags parsing issue in frontmatter

Author: Quorafind

src/__tests__/FileMetadataInheritance.test.ts

@@ -343,6 +343,125 @@ describe("File Metadata Inheritance", () => {
 		});
 	});
 
+	describe("Tags Inheritance", () => {
+		test("should inherit tags from file metadata", () => {
+			const content = "- [ ] Task without tags";
+			const fileMetadata = {
+				tags: ["#work", "#urgent", "#meeting"],
+			};
+
+			const tasks = parser.parseLegacy(content, "test.md", fileMetadata);
+
+			expect(tasks).toHaveLength(1);
+			expect(tasks[0].metadata.tags).toBeDefined();
+			expect(tasks[0].metadata.tags).toEqual(["#work", "#urgent", "#meeting"]);
+		});
+
+		test("should merge task tags with inherited tags", () => {
+			const content = "- [ ] Task with existing tags #personal";
+			const fileMetadata = {
+				tags: ["#work", "#urgent"],
+			};
+
+			const tasks = parser.parseLegacy(content, "test.md", fileMetadata);
+
+			expect(tasks).toHaveLength(1);
+			expect(tasks[0].metadata.tags).toBeDefined();
+			expect(tasks[0].metadata.tags).toContain("#personal");
+			expect(tasks[0].metadata.tags).toContain("#work");
+			expect(tasks[0].metadata.tags).toContain("#urgent");
+		});
+
+		test("should not duplicate tags when merging", () => {
+			const content = "- [ ] Task with duplicate tag #work";
+			const fileMetadata = {
+				tags: ["#work", "#urgent"],
+			};
+
+			const tasks = parser.parseLegacy(content, "test.md", fileMetadata);
+
+			expect(tasks).toHaveLength(1);
+			expect(tasks[0].metadata.tags).toBeDefined();
+			// Should only have one instance of #work
+			const workTags = tasks[0].metadata.tags.filter((tag: string) => tag === "#work");
+			expect(workTags).toHaveLength(1);
+			expect(tasks[0].metadata.tags).toContain("#urgent");
+		});
+
+		test("should parse special tag formats from file metadata", () => {
+			const content = "- [ ] Task inheriting project tag";
+			const fileMetadata = {
+				tags: ["#project/myproject", "#area/work", "#@/office"],
+			};
+
+			const tasks = parser.parseLegacy(content, "test.md", fileMetadata);
+
+			expect(tasks).toHaveLength(1);
+			expect(tasks[0].metadata.project).toBe("myproject");
+			expect(tasks[0].metadata.area).toBe("work");
+			expect(tasks[0].metadata.context).toBe("office");
+			expect(tasks[0].metadata.tags).toContain("#project/myproject");
+			expect(tasks[0].metadata.tags).toContain("#area/work");
+			expect(tasks[0].metadata.tags).toContain("#@/office");
+		});
+
+		test("should prioritize task metadata over tag-derived metadata", () => {
+			const content = "- [ ] Task with explicit project [project::taskproject]";
+			const fileMetadata = {
+				tags: ["#project/fileproject"],
+			};
+
+			const tasks = parser.parseLegacy(content, "test.md", fileMetadata);
+
+			expect(tasks).toHaveLength(1);
+			// Task's explicit project should take precedence
+			expect(tasks[0].metadata.project).toBe("taskproject");
+			expect(tasks[0].metadata.tags).toContain("#project/fileproject");
+		});
+
+		test("should handle mixed tag formats in file metadata", () => {
+			const content = "- [ ] Task with mixed tag inheritance";
+			const fileMetadata = {
+				tags: ["#regular-tag", "#project/myproject", "#normalTag", "#area/work"],
+			};
+
+			const tasks = parser.parseLegacy(content, "test.md", fileMetadata);
+
+			expect(tasks).toHaveLength(1);
+			expect(tasks[0].metadata.project).toBe("myproject");
+			expect(tasks[0].metadata.area).toBe("work");
+			expect(tasks[0].metadata.tags).toContain("#regular-tag");
+			expect(tasks[0].metadata.tags).toContain("#normalTag");
+			expect(tasks[0].metadata.tags).toContain("#project/myproject");
+			expect(tasks[0].metadata.tags).toContain("#area/work");
+		});
+
+		test("should handle empty tags array in file metadata", () => {
+			const content = "- [ ] Task with empty tags";
+			const fileMetadata = {
+				tags: [],
+			};
+
+			const tasks = parser.parseLegacy(content, "test.md", fileMetadata);
+
+			expect(tasks).toHaveLength(1);
+			expect(tasks[0].metadata.tags).toEqual([]);
+		});
+
+		test("should handle non-array tags in file metadata", () => {
+			const content = "- [ ] Task with non-array tags";
+			const fileMetadata = {
+				tags: "single-tag",
+			};
+
+			const tasks = parser.parseLegacy(content, "test.md", fileMetadata);
+
+			expect(tasks).toHaveLength(1);
+			// Should inherit as a single tag
+			expect(tasks[0].metadata.tags).toContain("single-tag");
+		});
+	});
+
 	describe("Configuration Migration", () => {
 		test("should work with migrated settings", () => {
 			// 模拟迁移后的设置结构

src/__tests__/TagsInheritanceInvestigation.test.ts

@@ -0,0 +1,67 @@
+/**
+ * Investigation: Tags Inheritance and Metadata Fields
+ */
+
+import { MarkdownTaskParser } from "../utils/workers/ConfigurableTaskParser";
+import { getConfig } from "../common/task-parser-config";
+import { createMockPlugin } from "./mockUtils";
+import { DEFAULT_SETTINGS } from "../common/setting-definition";
+
+describe("Tags Inheritance Investigation", () => {
+	test("should investigate what fields are being inherited incorrectly", () => {
+		const mockPlugin = createMockPlugin({
+			...DEFAULT_SETTINGS,
+			fileMetadataInheritance: {
+				enabled: true,
+				inheritFromFrontmatter: true,
+				inheritFromFrontmatterForSubtasks: false,
+			},
+		});
+
+		const config = getConfig("tasks", mockPlugin);
+		const parser = new MarkdownTaskParser(config);
+
+		const content = `- [>] 12312312
+  - [ ] child task`;
+		
+		// Simulate problematic file metadata that might contain structural fields
+		const fileMetadata = {
+			tags: ["mobility"],
+			children: ["some-other-task"], // This should NOT be inherited
+			parent: "some-parent", // This should NOT be inherited
+			heading: ["Some Heading"], // This should NOT be inherited
+			id: "file-id", // This should NOT be inherited
+			priority: "high", // This SHOULD be inherited
+			area: "work", // This SHOULD be inherited
+		};
+
+		const tasks = parser.parseLegacy(content, "templify-asdasdasd-20250704120358.md", fileMetadata);
+
+		const parentTask = tasks[0];
+		const childTask = tasks[1];
+		
+		// Check parent task
+		console.log("Parent task metadata:", JSON.stringify(parentTask.metadata, null, 2));
+		
+		// Check child task
+		console.log("Child task metadata:", JSON.stringify(childTask.metadata, null, 2));
+		
+		// The problematic fields should NOT be inherited from file metadata
+		expect(parentTask.metadata.children).not.toEqual(["some-other-task"]);
+		expect(parentTask.metadata.parent).not.toBe("some-parent");
+		expect(parentTask.metadata.id).not.toBe("file-id");
+		
+		// But the appropriate fields should be inherited
+		expect(parentTask.metadata.priority).toBe(4); // "high" converted to 4
+		expect(parentTask.metadata.area).toBe("work");
+		expect(parentTask.metadata.tags).toContain("mobility");
+		
+		// Parent task should have correct structural fields
+		expect(parentTask.metadata.children).toEqual([childTask.id]);
+		expect(parentTask.metadata.parent).toBeUndefined();
+		
+		// Child task should have correct structural fields
+		expect(childTask.metadata.children).toEqual([]);
+		expect(childTask.metadata.parent).toBe(parentTask.id);
+	});
+});

src/styles/tag-view.css

@@ -5,6 +5,11 @@
 	height: 100%;
 	width: 100%;
 	overflow: hidden;
+	flex: 1;
+}
+
+.task-genius-view:has(.task-details.visible) .tags-left-column {
+	display: none;
 }
 
 .tags-content {