-
Notifications
You must be signed in to change notification settings - Fork 16
[Feature]: Standardize indexing content to include comments for both Issues and PRs #47
Copy link
Copy link
Closed
Copy link
Labels
enhancementNew feature or requestNew feature or requesturgentCritical issues that need immediate attentionCritical issues that need immediate attentionv0.2.0Target for v0.2.0Target for v0.2.0
Milestone
Description
Problem Statement
There is a discrepancy between the CLI backfill logic and the live bot pipeline:
- CLI Backfill: Indexes
Title+Body+ Comments. - Live Bot: Indexes
Title+BodyONLY.
This leads to inconsistent search results depending on how an item was indexed. An issue backfilled via CLI will have richer context than one processed by the bot in real-time.
Proposed Solution
- Create a shared function (e.g., in
internal/utils/text) to generate the content string for embedding. - Refactor
internal/steps/indexer.go(Bot) to use this function. - Refactor
cmd/simili/commands/index.go(CLI) to use this function. - The shared function MUST include comments for both Issues and PRs to ensure rich context for similarity search.
Feature Scope
- Pipeline Steps
- CLI
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or requesturgentCritical issues that need immediate attentionCritical issues that need immediate attentionv0.2.0Target for v0.2.0Target for v0.2.0
Type
Projects
Status
Done