Merge pull request #136 from keyxmakerx/claude/phase-sprint-planning-MEZXp

keyxmakerx · web-flow · commit 44f316ad8be6 · 2026-03-12T10:57:29.000-05:00
fix: use type conversion instead of struct literal for NPCTagInfo (S1…
diff --git a/internal/plugins/npcs/service.go b/internal/plugins/npcs/service.go
@@ -76,9 +76,7 @@ func (s *npcService) ListNPCs(ctx context.Context, campaignID string, role int,
 			for i := range cards {
 				if infos, ok := tagMap[cards[i].ID]; ok {
 					for _, t := range infos {
-						cards[i].Tags = append(cards[i].Tags, NPCTagInfo{
-							ID: t.ID, Name: t.Name, Slug: t.Slug, Color: t.Color,
-						})
+						cards[i].Tags = append(cards[i].Tags, NPCTagInfo(t))
 					}
 				}
 			}

Original file line number	Diff line number	Diff line change
`@@ -76,9 +76,7 @@ func (s *npcService) ListNPCs(ctx context.Context, campaignID string, role int,`
`76`	`76`	`for i := range cards {`
`77`	`77`	`if infos, ok := tagMap[cards[i].ID]; ok {`
`78`	`78`	`for _, t := range infos {`
`79`		`- cards[i].Tags = append(cards[i].Tags, NPCTagInfo{`
`80`		`- ID: t.ID, Name: t.Name, Slug: t.Slug, Color: t.Color,`
`81`		`- })`
	`79`	`+ cards[i].Tags = append(cards[i].Tags, NPCTagInfo(t))`
`82`	`80`	`}`
`83`	`81`	`}`
`84`	`82`	`}`