javi-forge 1.38.4 → 1.38.5

package/dist/commands/plugin.js

@@ -62,16 +62,20 @@ export async function runPluginSearch(query, onStep) {
     const stepId = "plugin-search";
     report(onStep, stepId, `Search plugins${query ? `: ${query}` : ""}`, "running");
     const results = await searchRegistry(query);
-    if (results.length === 0) {
-        report(onStep, stepId, `Search plugins${query ? `: ${query}` : ""}`, "done", query
-            ? `no plugins matching "${query}"`
-            : "registry empty or unreachable");
+    if (results.status === "cancelled") {
+        report(onStep, stepId, `Search plugins${query ? `: ${query}` : ""}`, "error", "registry search cancelled");
+    }
+    else if (results.status === "unavailable") {
+        report(onStep, stepId, `Search plugins${query ? `: ${query}` : ""}`, "error", "registry unavailable");
+    }
+    else if (results.entries.length === 0) {
+        report(onStep, stepId, `Search plugins${query ? `: ${query}` : ""}`, "done", query ? `no plugins matching "${query}"` : "registry empty");
     }
     else {
-        const summary = results
+        const summary = results.entries
             .map((p) => `${p.id} — ${p.description}`)
             .join("\n  ");
-        report(onStep, stepId, `Search plugins${query ? `: ${query}` : ""}`, "done", `${results.length} results:\n  ${summary}`);
+        report(onStep, stepId, `Search plugins${query ? `: ${query}` : ""}`, "done", `${results.entries.length} results:\n  ${summary}`);
     }
 }
 /**

package/dist/lib/plugin.d.ts

@@ -1,4 +1,12 @@
 import type { InstalledPlugin, PluginRegistryEntry, PluginSyncResult, PluginValidationResult } from "../types/index.js";
+export type RegistrySearchResult = {
+    status: "success";
+    entries: PluginRegistryEntry[];
+} | {
+    status: "unavailable";
+} | {
+    status: "cancelled";
+};
 /**
  * Validate a plugin directory structure and manifest.
  */
@@ -38,8 +46,14 @@ export declare function removePlugin(name: string, options?: {
 export declare function listInstalledPlugins(): Promise<InstalledPlugin[]>;
 /**
  * Fetch the remote plugin registry and optionally filter by query.
+ *
+ * A registry that cannot be read or validated is deliberately distinct from a
+ * valid registry with no matches. Callers need that distinction to avoid
+ * presenting a network failure as a successful empty search.
  */
-export declare function searchRegistry(query?: string): Promise<PluginRegistryEntry[]>;
+export declare function searchRegistry(query?: string, options?: {
+    signal?: AbortSignal;
+}): Promise<RegistrySearchResult>;
 /**
  * Detect installed plugins in a project's .javi-forge/plugins/ directory.
  * Returns an array of plugin names (sorted alphabetically).

package/dist/lib/plugin.js

@@ -8,6 +8,7 @@ import { evaluateCoverageGate, scanFailureMessage, } from "./skill-install-gate.
 import { scanSkillsWithCoverage } from "./skill-scanner.js";
 const KEBAB_RE = /^[a-z0-9]+(-[a-z0-9]+)*$/;
 const SEMVER_RE = /^\d+\.\d+\.\d+$/;
+const REGISTRY_REQUEST_TIMEOUT_MS = 10_000;
 // ── Validation ──────────────────────────────────────────────────────────────
 /**
  * Validate a plugin directory structure and manifest.
@@ -251,26 +252,90 @@ export async function listInstalledPlugins() {
 }
 /**
  * Fetch the remote plugin registry and optionally filter by query.
+ *
+ * A registry that cannot be read or validated is deliberately distinct from a
+ * valid registry with no matches. Callers need that distinction to avoid
+ * presenting a network failure as a successful empty search.
  */
-export async function searchRegistry(query) {
+export async function searchRegistry(query, options = {}) {
+    const { signal: callerSignal } = options;
+    if (callerSignal?.aborted)
+        return { status: "cancelled" };
+    const controller = new AbortController();
+    let timedOut = false;
+    let callerCancelled = false;
+    let resolveDeadline;
+    const deadline = new Promise((resolve) => {
+        resolveDeadline = resolve;
+    });
+    const timeout = setTimeout(() => {
+        timedOut = true;
+        controller.abort();
+        resolveDeadline();
+    }, REGISTRY_REQUEST_TIMEOUT_MS);
+    const cancelFromCaller = () => {
+        callerCancelled = true;
+        controller.abort();
+        resolveDeadline();
+    };
+    callerSignal?.addEventListener("abort", cancelFromCaller, { once: true });
     try {
-        const response = await fetch(PLUGIN_REGISTRY_URL);
-        if (!response.ok) {
-            return [];
-        }
-        const registry = (await response.json());
-        let plugins = registry.plugins ?? [];
-        if (query) {
-            const q = query.toLowerCase();
-            plugins = plugins.filter((p) => p.id.toLowerCase().includes(q) ||
-                p.description.toLowerCase().includes(q) ||
-                p.tags.some((t) => t.toLowerCase().includes(q)));
-        }
-        return plugins;
+        const response = await Promise.race([
+            Promise.resolve().then(() => fetch(PLUGIN_REGISTRY_URL, { signal: controller.signal })),
+            deadline,
+        ]);
+        if (callerCancelled || callerSignal?.aborted)
+            return { status: "cancelled" };
+        if (timedOut || !response || !response.ok)
+            return { status: "unavailable" };
+        const body = await Promise.race([response.json(), deadline]);
+        if (callerCancelled || callerSignal?.aborted)
+            return { status: "cancelled" };
+        if (timedOut || !isPluginRegistry(body))
+            return { status: "unavailable" };
+        const entries = query
+            ? filterRegistryEntries(body.plugins, query)
+            : body.plugins;
+        return { status: "success", entries };
     }
     catch {
-        return [];
+        return callerCancelled || callerSignal?.aborted
+            ? { status: "cancelled" }
+            : { status: "unavailable" };
     }
+    finally {
+        clearTimeout(timeout);
+        callerSignal?.removeEventListener("abort", cancelFromCaller);
+    }
+}
+function filterRegistryEntries(plugins, query) {
+    const normalizedQuery = query.toLowerCase();
+    return plugins.filter((plugin) => plugin.id.toLowerCase().includes(normalizedQuery) ||
+        plugin.description.toLowerCase().includes(normalizedQuery) ||
+        plugin.tags.some((tag) => tag.toLowerCase().includes(normalizedQuery)));
+}
+function isPluginRegistry(value) {
+    if (!isRecord(value))
+        return false;
+    return (typeof value.version === "string" &&
+        typeof value.updatedAt === "string" &&
+        Array.isArray(value.plugins) &&
+        value.plugins.every(isPluginRegistryEntry));
+}
+function isPluginRegistryEntry(value) {
+    if (!isRecord(value))
+        return false;
+    return (typeof value.id === "string" &&
+        typeof value.repository === "string" &&
+        typeof value.description === "string" &&
+        Array.isArray(value.tags) &&
+        value.tags.every((tag) => typeof tag === "string") &&
+        (value.stars === undefined ||
+            (typeof value.stars === "number" && Number.isFinite(value.stars))) &&
+        (value.updatedAt === undefined || typeof value.updatedAt === "string"));
+}
+function isRecord(value) {
+    return typeof value === "object" && value !== null;
 }
 // ── Sync ───────────────────────────────────────────────────────────────
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "javi-forge",
-	"version": "1.38.4",
+	"version": "1.38.5",
 	"description": "Project scaffolding and AI-ready CI bootstrap",
 	"type": "module",
 	"bin": {