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
processIssue function 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
Problem Statement
The current CLI
indexcommand explicitly skips Pull Requests: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.goto:processIssuefunction handles PR objects correctly (likely just treating them as issues since they share the same structure for Title/Body).--include-prs(defaulting to true or false) to control this behavior.Alternatives Considered
index-prs, but this duplicates logic.Feature Scope