javi-forge 1.38.4 → 1.38.6

package/dist/commands/plugin.d.ts

@@ -17,7 +17,9 @@ export declare function runPluginList(onStep: StepCallback): Promise<void>;
 /**
  * Search the remote plugin registry.
  */
-export declare function runPluginSearch(query: string | undefined, onStep: StepCallback): Promise<void>;
+export declare function runPluginSearch(query: string | undefined, onStep: StepCallback, options?: {
+    signal?: AbortSignal;
+}): Promise<void>;
 /**
  * Validate a local plugin directory.
  */

package/dist/commands/plugin.js

@@ -58,20 +58,24 @@ export async function runPluginList(onStep) {
 /**
  * Search the remote plugin registry.
  */
-export async function runPluginSearch(query, onStep) {
+export async function runPluginSearch(query, onStep, options = {}) {
     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");
+    const results = await searchRegistry(query, options);
+    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/dist/ui/Plugin.js

@@ -19,6 +19,15 @@ const STATUS_COLOR = {
 export default function Plugin({ action, target, dryRun, codex = false, force = false, }) {
     const [steps, setSteps] = useState([]);
     const [done, setDone] = useState(false);
+    const handleTerminalSignal = (signal, signalController) => {
+        if (signalController.signal.aborted)
+            return;
+        signalController.abort();
+        if (signal === "SIGINT")
+            process.exitCode = 130;
+        if (signal === "SIGTERM")
+            process.exitCode = 143;
+    };
     const onStep = (step) => {
         setSteps((prev) => {
             const idx = prev.findIndex((s) => s.id === step.id);
@@ -31,6 +40,17 @@ export default function Plugin({ action, target, dryRun, codex = false, force =
         });
     };
     useEffect(() => {
+        const controller = action === "search" ? new AbortController() : undefined;
+        const sigintHandler = controller === undefined
+            ? undefined
+            : () => handleTerminalSignal("SIGINT", controller);
+        const sigtermHandler = controller === undefined
+            ? undefined
+            : () => handleTerminalSignal("SIGTERM", controller);
+        if (action === "search" && sigintHandler && sigtermHandler) {
+            process.on("SIGINT", sigintHandler);
+            process.on("SIGTERM", sigtermHandler);
+        }
         const run = async () => {
             try {
                 switch (action) {
@@ -62,7 +82,9 @@ export default function Plugin({ action, target, dryRun, codex = false, force =
                         await runPluginList(onStep);
                         break;
                     case "search":
-                        await runPluginSearch(target, onStep);
+                        await runPluginSearch(target, onStep, {
+                            signal: controller?.signal,
+                        });
                         break;
                     case "validate":
                         if (!target) {
@@ -129,6 +151,14 @@ export default function Plugin({ action, target, dryRun, codex = false, force =
             setDone(true);
         };
         run();
+        return () => {
+            if (sigintHandler) {
+                process.removeListener("SIGINT", sigintHandler);
+            }
+            if (sigtermHandler) {
+                process.removeListener("SIGTERM", sigtermHandler);
+            }
+        };
     }, [action, target, dryRun, force]);
     return (React.createElement(Box, { flexDirection: "column", padding: 1 },
         React.createElement(Box, { marginBottom: 1 },

package/package.json

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