pi-extmgr 0.1.20 → 0.1.22

package/src/utils/settings.ts

@@ -7,9 +7,10 @@ import type {
   ExtensionCommandContext,
   ExtensionContext,
 } from "@mariozechner/pi-coding-agent";
-import { readFile, writeFile, mkdir, access, rename, rm } from "node:fs/promises";
+import { readFile, writeFile, mkdir, rename, rm } from "node:fs/promises";
 import { homedir } from "node:os";
 import { join } from "node:path";
+import { fileExists } from "./fs.js";
 
 export interface AutoUpdateConfig {
   intervalMs: number;
@@ -38,6 +39,16 @@ function isRecord(value: unknown): value is Record<string, unknown> {
   return Boolean(value) && typeof value === "object" && !Array.isArray(value);
 }
 
+function isValidStringArray(value: unknown): value is string[] {
+  return Array.isArray(value) && value.every((item): item is string => typeof item === "string");
+}
+
+function sanitizeStringArray(value: unknown): string[] | undefined {
+  if (!isValidStringArray(value)) return undefined;
+  const sanitized = value.map((s) => s.trim()).filter(Boolean);
+  return sanitized.length > 0 ? sanitized : undefined;
+}
+
 function sanitizeAutoUpdateConfig(input: unknown): AutoUpdateConfig {
   if (!isRecord(input)) {
     return { ...DEFAULT_CONFIG };
@@ -74,15 +85,9 @@ function sanitizeAutoUpdateConfig(input: unknown): AutoUpdateConfig {
     config.nextCheck = input.nextCheck;
   }
 
-  if (Array.isArray(input.updatesAvailable)) {
-    const updates = input.updatesAvailable
-      .filter((value): value is string => typeof value === "string")
-      .map((value) => value.trim())
-      .filter(Boolean);
-
-    if (updates.length > 0) {
-      config.updatesAvailable = updates;
-    }
+  const updates = sanitizeStringArray(input.updatesAvailable);
+  if (updates) {
+    config.updatesAvailable = updates;
   }
 
   if (!config.enabled || config.intervalMs === 0) {
@@ -112,18 +117,6 @@ function getSessionConfig(
   return undefined;
 }
 
-/**
- * Checks if a file exists
- */
-async function fileExists(path: string): Promise<boolean> {
-  try {
-    await access(path);
-    return true;
-  } catch {
-    return false;
-  }
-}
-
 /**
  * Ensures the settings directory exists
  */
@@ -198,7 +191,6 @@ async function writeConfigToDisk(config: AutoUpdateConfig): Promise<void> {
 
 function enqueueConfigWrite(config: AutoUpdateConfig): void {
   settingsWriteQueue = settingsWriteQueue
-    .catch(() => undefined)
     .then(() => writeConfigToDisk(config))
     .catch((error) => {
       console.warn("[extmgr] Failed to write settings:", error);

package/src/utils/timer.ts

@@ -0,0 +1,36 @@
+/**
+ * Timer utilities for auto-update functionality.
+ */
+
+export type TimerCallback = () => void;
+
+let timerId: ReturnType<typeof setInterval> | null = null;
+
+/**
+ * Start a recurring timer with the given interval and callback.
+ * Clears any existing timer first.
+ */
+export function startTimer(intervalMs: number, callback: TimerCallback): void {
+  stopTimer();
+
+  if (intervalMs <= 0) return;
+
+  timerId = setInterval(callback, intervalMs);
+}
+
+/**
+ * Stop the current timer if running.
+ */
+export function stopTimer(): void {
+  if (!timerId) return;
+
+  clearInterval(timerId);
+  timerId = null;
+}
+
+/**
+ * Check if a timer is currently running.
+ */
+export function isTimerRunning(): boolean {
+  return timerId !== null;
+}