@zephyrdeng/pi-review 0.6.1 → 0.6.2

package/extensions/rv-completions.test.ts

@@ -177,6 +177,21 @@ describe("buildRvCompletions", () => {
     assert.ok(tmpl.value.endsWith(" @"));
   });
 
+  it("does not duplicate list-models completion at top level", () => {
+    for (const locale of ["en", "zh"] as const) {
+      for (const prefix of ["", "models", "list", "mod"]) {
+        const items = buildRvCompletions(prefix, { ...deps, locale });
+        if (!items?.length) continue;
+        const ui = rvUi(locale);
+        const listModelItems = items.filter((i) => i.label === ui.listModels);
+        assert.ok(
+          listModelItems.length <= 1,
+          `locale=${locale} prefix=${JSON.stringify(prefix)} got ${listModelItems.length} "${ui.listModels}" items`,
+        );
+      }
+    }
+  });
+
   it("every returned value preserves the head prefix (整段替换安全)", () => {
     const cases = ["@a.ts --mode ", "@a.ts --model ", "@a.ts --thinking ", "@a.ts --"];
     for (const c of cases) {

package/extensions/rv-completions.ts

@@ -424,7 +424,13 @@ function topLevelCompletions(
   items.push(...semanticCompletionItems(localeOf(deps), tail, headPrefix, head));
 
   const ui = rvUi(localeOf(deps));
-  if (fuzzyMatch(ui.modelsKeyword, tail) || ui.modelsKeyword.startsWith(tail) || fuzzyMatch("models", tail)) {
+  const hasListModelsItem = items.some((i) => i.label === ui.listModels);
+  if (
+    !hasListModelsItem &&
+    (fuzzyMatch(ui.modelsKeyword, tail) ||
+      ui.modelsKeyword.startsWith(tail) ||
+      fuzzyMatch("models", tail))
+  ) {
     items.push({
       value: `${headPrefix}${ui.listModels}`,
       label: ui.listModels,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@zephyrdeng/pi-review",
-  "version": "0.6.1",
+  "version": "0.6.2",
   "description": "Isolated AI-powered code and plan reviews from the command line",
   "type": "module",
   "author": "ZephyrDeng <zephyrTang@aliyun.com>",