Merge pull request #139 from keyxmakerx/claude/review-pending-merges-ScVUs

fix: resolve pre-existing lint issues blocking CI

Author: keyxmakerx

internal/plugins/campaigns/invite_service_test.go

@@ -351,10 +351,10 @@ func TestListInvites(t *testing.T) {
 	campaignRepo := newMockCampaignRepoForInvites()
 	svc := NewInviteService(inviteRepo, campaignRepo, nil, "http://localhost:3000")
 
-	svc.CreateInvite(context.Background(), "camp-1", "user-1", CreateInviteInput{
+	_, _ = svc.CreateInvite(context.Background(), "camp-1", "user-1", CreateInviteInput{
 		Email: "p1@example.com", Role: "player",
 	})
-	svc.CreateInvite(context.Background(), "camp-1", "user-1", CreateInviteInput{
+	_, _ = svc.CreateInvite(context.Background(), "camp-1", "user-1", CreateInviteInput{
 		Email: "p2@example.com", Role: "scribe",
 	})
 

internal/plugins/entities/worldbuilding_prompt_service_test.go

@@ -134,11 +134,11 @@ func TestListForType(t *testing.T) {
 	svc := NewWorldbuildingPromptService(repo, typeRepo)
 
 	etID := 1
-	svc.Create(context.Background(), "camp-1", CreatePromptInput{
+	_, _ = svc.Create(context.Background(), "camp-1", CreatePromptInput{
 		Name: "A", PromptText: "prompt a", EntityTypeID: &etID,
 	})
 	etID2 := 2
-	svc.Create(context.Background(), "camp-1", CreatePromptInput{
+	_, _ = svc.Create(context.Background(), "camp-1", CreatePromptInput{
 		Name: "B", PromptText: "prompt b", EntityTypeID: &etID2,
 	})
 

internal/plugins/syncapi/sync_repository.go

@@ -282,11 +282,14 @@ func (r *syncMappingRepo) ListMappingsWithNames(ctx context.Context, campaignID
 	}
 
 	// Sort.
-	orderBy := "sm.last_synced_at DESC"
-	if opts.Sort == "name" {
+	var orderBy string
+	switch opts.Sort {
+	case "name":
 		orderBy = "COALESCE(e.name, m.name, sm.chronicle_id) ASC"
-	} else if opts.Sort == "type" {
+	case "type":
 		orderBy = "sm.chronicle_type ASC, sm.last_synced_at DESC"
+	default:
+		orderBy = "sm.last_synced_at DESC"
 	}
 
 	limit := opts.Limit