feat: add filter by reviewer # 39

Author: LonoxX

src/filter-manager.ts

@@ -38,6 +38,11 @@ export class FilterManager {
 			filteredPullRequests = this.applyAssigneeFilter(filteredPullRequests, repo.prAssigneeFilterMode ?? "assigned-to-me", repo.prAssigneeFilters ?? []);
 		}
 
+		// Apply reviewer filtering
+		if ((repo.enablePrReviewerFilter ?? false)) {
+			filteredPullRequests = this.applyReviewerFilter(filteredPullRequests, repo.prReviewerFilterMode ?? "review-requested-from-me", repo.prReviewerFilters ?? []);
+		}
+
 		return filteredPullRequests;
 	}
 
@@ -91,6 +96,31 @@ export class FilterManager {
 		});
 	}
 
+	private applyReviewerFilter(items: any[], filterMode: "review-requested-from-me" | "review-requested-from-specific" | "no-review-requested" | "any-review-requested", reviewerFilters: string[]): any[] {
+		return items.filter((item) => {
+			const reviewers = item.requested_reviewers || [];
+			const reviewerUsernames = reviewers.map((reviewer: any) => reviewer.login || reviewer);
+
+			switch (filterMode) {
+				case "review-requested-from-me":
+					const currentUser = this.getCurrentUser();
+					return reviewerUsernames.includes(currentUser);
+
+				case "review-requested-from-specific":
+					return reviewerFilters.some(filterUser => reviewerUsernames.includes(filterUser));
+
+				case "no-review-requested":
+					return reviewerUsernames.length === 0;
+
+				case "any-review-requested":
+					return reviewerUsernames.length > 0;
+
+				default:
+					return true;
+			}
+		});
+	}
+
 	private getCurrentUser(): string {
 		// Access the current user from the GitHubClient through the main plugin
 		return this.gitHubClient ? this.gitHubClient.getCurrentUser() : "";

src/settings/modal-manager.ts

@@ -240,7 +240,7 @@ export class ModalManager {
 	async fetchAndShowRepositoryCollaborators(
 		repositoryName: string,
 		repo: RepositoryTracking,
-		filterType: 'assigneeFilters' | 'prAssigneeFilters',
+		filterType: 'assigneeFilters' | 'prAssigneeFilters' | 'prReviewerFilters',
 		textAreaElement: HTMLTextAreaElement,
 	): Promise<void> {
 		if (!this.plugin.gitHubClient?.isReady()) {

src/settings/repository-renderer.ts

@@ -7,7 +7,7 @@ export class RepositoryRenderer {
 		private app: App,
 		private plugin: GitHubTrackerPlugin,
 		private fetchLabels?: (repo: string, repoObj: RepositoryTracking, filterType: 'labelFilters' | 'prLabelFilters', textArea: HTMLTextAreaElement) => Promise<void>,
-		private fetchCollaborators?: (repo: string, repoObj: RepositoryTracking, filterType: 'assigneeFilters' | 'prAssigneeFilters', textArea: HTMLTextAreaElement) => Promise<void>,
+		private fetchCollaborators?: (repo: string, repoObj: RepositoryTracking, filterType: 'assigneeFilters' | 'prAssigneeFilters' | 'prReviewerFilters', textArea: HTMLTextAreaElement) => Promise<void>,
 	) {}
 
 	renderIssueSettings(
@@ -46,6 +46,9 @@ export class RepositoryRenderer {
 
 		// Assignee filtering settings for pull requests
 		this.renderAssigneeFilter(container, repo, 'pr');
+
+		// Reviewer filtering settings for pull requests
+		this.renderReviewerFilter(container, repo);
 	}
 
 	private renderLabelFilter(
@@ -126,6 +129,92 @@ export class RepositoryRenderer {
 			);
 	}
 
+	private renderReviewerFilter(
+		container: HTMLElement,
+		repo: RepositoryTracking,
+	): void {
+		new Setting(container)
+			.setName("Filter pull requests by reviewers")
+			.setDesc("Enable filtering pull requests based on requested reviewers")
+			.addToggle((toggle) =>
+				toggle
+					.setValue(repo.enablePrReviewerFilter ?? false)
+					.onChange(async (value) => {
+						repo.enablePrReviewerFilter = value;
+						reviewerFilterContainer.classList.toggle(
+							"github-issues-hidden",
+							!value,
+						);
+						await this.plugin.saveSettings();
+					}),
+			);
+
+		const reviewerFilterContainer = container.createDiv(
+			"github-issues-settings-group github-issues-nested",
+		);
+		reviewerFilterContainer.classList.toggle(
+			"github-issues-hidden",
+			!(repo.enablePrReviewerFilter ?? false),
+		);
+
+		new Setting(reviewerFilterContainer)
+			.setName("Reviewer filter mode")
+			.setDesc("Choose how to filter pull requests by reviewers")
+			.addDropdown((dropdown) =>
+				dropdown
+					.addOption("review-requested-from-me", "Review requested from me")
+					.addOption("review-requested-from-specific", "Review requested from specific users")
+					.addOption("no-review-requested", "No review requested")
+					.addOption("any-review-requested", "Any review requested")
+					.setValue(repo.prReviewerFilterMode ?? "review-requested-from-me")
+					.onChange(async (value) => {
+						repo.prReviewerFilterMode = value as "review-requested-from-me" | "review-requested-from-specific" | "no-review-requested" | "any-review-requested";
+						reviewerSpecificContainer.classList.toggle(
+							"github-issues-hidden",
+							value !== "review-requested-from-specific",
+						);
+						await this.plugin.saveSettings();
+					}),
+			);
+
+		const reviewerSpecificContainer = reviewerFilterContainer.createDiv(
+			"github-issues-settings-group github-issues-nested",
+		);
+		reviewerSpecificContainer.classList.toggle(
+			"github-issues-hidden",
+			(repo.prReviewerFilterMode ?? "review-requested-from-me") !== "review-requested-from-specific",
+		);
+
+		new Setting(reviewerSpecificContainer)
+			.setName("Specific reviewers")
+			.setDesc("Comma-separated list of GitHub usernames to filter by")
+			.addTextArea((text) => {
+				text
+					.setPlaceholder("username1, username2, username3")
+					.setValue((repo.prReviewerFilters || []).join(", "))
+					.onChange(async (value) => {
+						repo.prReviewerFilters = value
+							.split(",")
+							.map(username => username.trim())
+							.filter(username => username.length > 0);
+						await this.plugin.saveSettings();
+					});
+
+				return text;
+			})
+			.addButton((button) =>
+				button
+					.setButtonText("Fetch collaborators")
+					.setTooltip("Load collaborators from this repository to help with configuration")
+					.onClick(async () => {
+						const textArea = button.buttonEl.closest('.setting-item')?.querySelector('textarea');
+						if (textArea && this.fetchCollaborators) {
+							await this.fetchCollaborators(repo.repository, repo, 'prReviewerFilters', textArea as HTMLTextAreaElement);
+						}
+					}),
+			);
+	}
+
 	private renderAssigneeFilter(
 		container: HTMLElement,
 		repo: RepositoryTracking,

src/types.ts

@@ -63,6 +63,9 @@ export interface RepositoryTracking {
 	enablePrAssigneeFilter: boolean;
 	prAssigneeFilterMode: "assigned-to-me" | "assigned-to-specific" | "unassigned" | "any-assigned";
 	prAssigneeFilters: string[];
+	enablePrReviewerFilter: boolean;
+	prReviewerFilterMode: "review-requested-from-me" | "review-requested-from-specific" | "no-review-requested" | "any-review-requested";
+	prReviewerFilters: string[];
 	escapeHashTags: boolean;
 
 	// Profile-managed fields (optional - hydrated from profile at runtime)
@@ -311,5 +314,8 @@ export const DEFAULT_REPOSITORY_TRACKING: RepositoryTracking = {
 	enablePrAssigneeFilter: false,
 	prAssigneeFilterMode: "assigned-to-me",
 	prAssigneeFilters: [],
+	enablePrReviewerFilter: false,
+	prReviewerFilterMode: "review-requested-from-me",
+	prReviewerFilters: [],
 	escapeHashTags: false,
 };