fix(integrations): address review comments — stale names and docs

- retry.go: update package-level doc from "Package gemini" to
  "Package ai" to match the declared package name
- vdb_router.go: rename NewVDBRouterFromGemini →
  NewVDBRouterFromEmbedder and fix its stale doc comment
- index.go: rename local variable geminiClient → embedder to
  match the provider-neutral *ai.Embedder type
- gemini_test.go → prompts_test.go: rename test file to a
  provider-neutral name consistent with the package rename

Relates to #68

Signed-off-by: Kavirubc <hapuarachchikaviru@gmail.com>

Author: Kavirubc

cmd/simili/commands/index.go

@@ -89,13 +89,13 @@ func runIndex(cmd *cobra.Command, args []string) {
 
 	ghClient := similiGithub.NewClient(ctx, token)
 
-	geminiClient, err := ai.NewEmbedder(cfg.Embedding.APIKey, cfg.Embedding.Model)
+	embedder, err := ai.NewEmbedder(cfg.Embedding.APIKey, cfg.Embedding.Model)
 	if err != nil {
 		log.Fatalf("Failed to init embedder: %v", err)
 	}
-	defer geminiClient.Close()
+	defer embedder.Close()
 	embeddingDimensions := cfg.Embedding.Dimensions
-	if dim := geminiClient.Dimensions(); dim > 0 {
+	if dim := embedder.Dimensions(); dim > 0 {
 		embeddingDimensions = dim
 	}
 
@@ -144,7 +144,7 @@ func runIndex(cmd *cobra.Command, args []string) {
 		go func(id int) {
 			defer wg.Done()
 			for job := range jobs {
-				processIssue(ctx, id, job.Issue, ghClient, geminiClient, qdrantClient, splitter, cfg.Qdrant.Collection, org, repoName, indexDryRun)
+				processIssue(ctx, id, job.Issue, ghClient, embedder, qdrantClient, splitter, cfg.Qdrant.Collection, org, repoName, indexDryRun)
 			}
 		}(i)
 	}

internal/integrations/ai/prompts_test.go

@@ -0,0 +1,155 @@
+// Author: Kaviru Hapuarachchi
+// GitHub: https://github.com/Kavirubc
+// Created: 2026-02-02
+// Last Modified: 2026-02-02
+
+package ai
+
+import (
+	"strings"
+	"testing"
+)
+
+func TestBuildTriagePrompt(t *testing.T) {
+	issue := &IssueInput{
+		Title:  "Bug: Application crashes on startup",
+		Body:   "When I start the application, it immediately crashes with error XYZ",
+		Author: "testuser",
+		Labels: []string{"bug", "needs-triage"},
+	}
+
+	prompt := buildTriagePrompt(issue)
+
+	if prompt == "" {
+		t.Error("Expected non-empty prompt")
+	}
+
+	// Check that prompt contains key information
+	if !strings.Contains(prompt, issue.Title) {
+		t.Error("Prompt should contain issue title")
+	}
+	if !strings.Contains(prompt, issue.Author) {
+		t.Error("Prompt should contain author")
+	}
+}
+
+func TestBuildResponsePrompt(t *testing.T) {
+	similar := []SimilarIssueInput{
+		{
+			Number:     123,
+			Title:      "Similar issue",
+			URL:        "https://github.com/org/repo/issues/123",
+			Similarity: 0.85,
+			State:      "closed",
+		},
+		{
+			Number:     456,
+			Title:      "Another similar issue",
+			URL:        "https://github.com/org/repo/issues/456",
+			Similarity: 0.75,
+			State:      "open",
+		},
+	}
+
+	prompt := buildResponsePrompt(similar)
+
+	if prompt == "" {
+		t.Error("Expected non-empty prompt")
+	}
+
+	// Check that prompt contains issue information
+	if !strings.Contains(prompt, "#123") {
+		t.Error("Prompt should contain first issue number")
+	}
+	if !strings.Contains(prompt, "#456") {
+		t.Error("Prompt should contain second issue number")
+	}
+}
+
+func TestTruncate(t *testing.T) {
+	tests := []struct {
+		name     string
+		input    string
+		maxLen   int
+		expected string
+	}{
+		{
+			name:     "short string",
+			input:    "hello",
+			maxLen:   10,
+			expected: "hello",
+		},
+		{
+			name:     "exact length",
+			input:    "hello",
+			maxLen:   5,
+			expected: "hello",
+		},
+		{
+			name:     "long string",
+			input:    "hello world this is a long string",
+			maxLen:   10,
+			expected: "hello worl...",
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result := truncate(tt.input, tt.maxLen)
+			if result != tt.expected {
+				t.Errorf("Expected %q, got %q", tt.expected, result)
+			}
+		})
+	}
+}
+
+func TestParseTriageResponse(t *testing.T) {
+	tests := []struct {
+		name            string
+		response        string
+		expectedQuality string
+		expectedLabels  int
+		expectedDupe    bool
+	}{
+		{
+			name:            "good quality bug",
+			response:        "Quality: good\nLabels: bug\nReasoning: Well described issue with clear steps",
+			expectedQuality: "good",
+			expectedLabels:  1,
+			expectedDupe:    false,
+		},
+		{
+			name:            "poor quality",
+			response:        "Quality: poor\nLabels: question\nReasoning: Vague description, no details",
+			expectedQuality: "poor",
+			expectedLabels:  1,
+			expectedDupe:    false,
+		},
+		{
+			name:            "duplicate detection",
+			response:        "Quality: good\nThis appears to be a duplicate of issue #123",
+			expectedQuality: "good",
+			expectedLabels:  0,
+			expectedDupe:    true,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			result := parseTriageResponseLegacy(tt.response)
+
+			if result.Quality != tt.expectedQuality {
+				t.Errorf("Expected quality %q, got %q", tt.expectedQuality, result.Quality)
+			}
+
+			if len(result.SuggestedLabels) != tt.expectedLabels {
+				t.Errorf("Expected %d labels, got %d", tt.expectedLabels, len(result.SuggestedLabels))
+			}
+
+			if result.IsDuplicate != tt.expectedDupe {
+				t.Errorf("Expected duplicate=%v, got %v", tt.expectedDupe, result.IsDuplicate)
+			}
+		})
+	}
+}
+

internal/integrations/ai/retry.go

@@ -3,7 +3,7 @@
 // Created: 2026-02-15
 // Last Modified: 2026-02-17
 
-// Package gemini provides Gemini AI integration for embeddings and LLM.
+// Package ai provides provider-neutral AI integration for embeddings and LLM.
 package ai
 
 import (

internal/transfer/vdb_router.go

@@ -137,7 +137,7 @@ func (r *VDBRouter) SuggestTransfer(ctx context.Context, issue *IssueInput, curr
 	}, nil
 }
 
-// VDBRouterWithGeminiEmbedder is a convenience constructor accepting *ai.Embedder directly.
-func NewVDBRouterFromGemini(embedder *ai.Embedder, store qdrant.VectorStore, collection string, maxResults int) *VDBRouter {
+// NewVDBRouterFromEmbedder is a convenience constructor accepting *ai.Embedder directly.
+func NewVDBRouterFromEmbedder(embedder *ai.Embedder, store qdrant.VectorStore, collection string, maxResults int) *VDBRouter {
 	return NewVDBRouter(embedder, store, collection, maxResults)
 }