fix-tag-processor-same-name

Author: Dev Agent

component/tagparser/tag_processor.go

@@ -124,13 +124,23 @@ func (p *tagProcessor) processTags(existingCategoryTagMap map[string]map[string]
 		}
 		for _, tagName := range tagNames {
 			if tag, ok := existingTaskTags[tagName]; !ok {
-				tagsToCreate = append(tagsToCreate, &database.Tag{
-					Name:     tagName,
-					Category: category,
-					Scope:    p.tagScope,
-					BuiltIn:  false, // new tag is absolutely not built-in
-					Group:    "",    // keep empty
-				})
+				// Before creating a new tag, search for an existing built-in tag with the
+				// same name across all categories. This handles the case where a tag like
+				// "jax" (framework) appears under the generic "tags" frontmatter key, which
+				// gets mapped to the "task" category by formatCategoryName. Without this
+				// lookup, the system would incorrectly create a new task-category tag
+				// instead of reusing the existing built-in framework tag.
+				if builtIn := findBuiltInTagByName(existingCategoryTagMap, tagName); builtIn != nil {
+					tagsMatched = append(tagsMatched, builtIn)
+				} else {
+					tagsToCreate = append(tagsToCreate, &database.Tag{
+						Name:     tagName,
+						Category: category,
+						Scope:    p.tagScope,
+						BuiltIn:  false, // new tag is absolutely not built-in
+						Group:    "",    // keep empty
+					})
+				}
 			} else {
 				tagsMatched = append(tagsMatched, tag)
 			}
@@ -141,6 +151,17 @@ func (p *tagProcessor) processTags(existingCategoryTagMap map[string]map[string]
 	return tagsMatched, tagsToCreate
 }
 
+// findBuiltInTagByName searches for a built-in tag with the given name across all categories.
+// Returns the first built-in tag found, or nil if none exists.
+func findBuiltInTagByName(existingCategoryTagMap map[string]map[string]*database.Tag, name string) *database.Tag {
+	for _, tags := range existingCategoryTagMap {
+		if tag, ok := tags[name]; ok && tag.BuiltIn {
+			return tag
+		}
+	}
+	return nil
+}
+
 func (p *tagProcessor) mapCategoryTag(tags []*database.Tag) map[string]map[string]*database.Tag {
 	predefinedCategoryTagMap := make(map[string]map[string]*database.Tag)
 	for _, tag := range tags {

component/tagparser/tag_processor_test.go

@@ -60,6 +60,47 @@ func Test_ProcessReadme(t *testing.T) {
 		t.Fail()
 	}
 }
+
+// Test_processTags_BuiltInCrossCategory verifies that when a tag name appears under a
+// different category in the README frontmatter (e.g. "jax" under "tags" which maps to
+// "task"), the processor reuses the existing built-in tag from the correct category
+// ("framework") instead of creating a new tag with the wrong category.
+func Test_processTags_BuiltInCrossCategory(t *testing.T) {
+	p := new(tagProcessor)
+	p.tagScope = types.ModelTagScope
+
+	existingCategoryTagMap := make(map[string]map[string]*database.Tag)
+
+	existingCategoryTagMap[categoryNameTask] = make(map[string]*database.Tag)
+	existingCategoryTagMap[categoryNameTask]["text-generation"] = &database.Tag{Name: "text-generation", Category: categoryNameTask, BuiltIn: true}
+
+	existingCategoryTagMap[categoryNameFramework] = make(map[string]*database.Tag)
+	existingCategoryTagMap[categoryNameFramework]["jax"] = &database.Tag{Name: "jax", Category: categoryNameFramework, BuiltIn: true}
+
+	// "tags" key in frontmatter maps to "task" category, but "jax" is a framework tag
+	categoryTagMap := make(map[string][]string)
+	categoryTagMap[categoryNameTask] = []string{"text-generation", "jax"}
+
+	tagsMatched, tagsToCreate := p.processTags(existingCategoryTagMap, categoryTagMap)
+
+	if len(tagsToCreate) != 0 {
+		t.Logf("expected no new tags, got %d: %+v", len(tagsToCreate), tagsToCreate)
+		t.FailNow()
+	}
+
+	if len(tagsMatched) != 2 {
+		t.Logf("expected 2 matched tags, got %d", len(tagsMatched))
+		t.FailNow()
+	}
+
+	if !slices.ContainsFunc(tagsMatched, func(e *database.Tag) bool {
+		return e.Name == "jax" && e.Category == categoryNameFramework
+	}) {
+		t.Logf("expected matched tag framework/jax, not found in matched: %+v", tagsMatched)
+		t.FailNow()
+	}
+}
+
 func Test_processTags(t *testing.T) {
 	p := new(tagProcessor)
 	p.tagScope = types.DatasetTagScope