-
Notifications
You must be signed in to change notification settings - Fork 16
[Feature]: Support indexing Pull Requests in CLI index command #46
Copy link
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
The current CLI index command explicitly skips Pull Requests:
if issue.IsPullRequest() {
continue // Skip PRs
}This makes it impossible to use the CLI to backfill or test on historical PRs, which is critical for ensuring the bot has context on past code changes and discussions.
Proposed Solution
Update cmd/simili/commands/index.go to:
- Remove the hardcoded check that skips PRs.
- Ensure the
processIssuefunction handles PR objects correctly (likely just treating them as issues since they share the same structure for Title/Body). - Optionally add a flag
--include-prs(defaulting to true or false) to control this behavior.
Alternatives Considered
- Using a separate command
index-prs, but this duplicates logic.
Feature Scope
- 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