From 007f93422cbddc0ec7f6f9c8c620304f79566067 Mon Sep 17 00:00:00 2001
From: svtter <svtter@qq.com>
Date: Mon, 25 May 2026 11:12:56 +0800
Subject: [PATCH] fix: remove unused configPath parameter from loadReviewers()
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

configPath was declared in the function signature but never referenced
in the body — all reviewers are loaded from the built-in directory
(actionPath/reviewers/). The sole call site (index.ts) only passes
{ actionPath }, confirming the parameter is dead.

Also clarified in the issue that `team` is NOT a ghost parameter — it
is actively used on line 42 for team resolution.

Refs: #110
---
 multi-review/src/reviewers.ts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/multi-review/src/reviewers.ts b/multi-review/src/reviewers.ts
index fd3de93..69d104d 100644
--- a/multi-review/src/reviewers.ts
+++ b/multi-review/src/reviewers.ts
@@ -34,7 +34,6 @@ function loadBuiltInReviewers(reviewersDir: string): Map<string, PersonaYAML> {
 export function loadReviewers(opts: {
   actionPath: string;
   team?: string;
-  configPath?: string;
 }): Reviewer[] {
   const builtInDir = join(opts.actionPath, "reviewers");
   const personas = loadBuiltInReviewers(builtInDir);