@portel/photon 1.32.3 → 1.32.4

package/dist/daemon/server.js

@@ -57,27 +57,34 @@ async function isSocketResponsive(target) {
         return false;
     return new Promise((resolve) => {
         const client = new net.Socket();
+        let done = false;
+        const finish = (alive) => {
+            if (done)
+                return;
+            done = true;
+            clearTimeout(timer);
+            if (alive) {
+                client.end();
+            }
+            else {
+                client.destroy();
+            }
+            resolve(alive);
+        };
         const timer = setTimeout(() => {
-            client.destroy();
-            resolve(false);
+            finish(false);
         }, 1000);
         client.on('error', () => {
-            clearTimeout(timer);
-            client.destroy();
-            resolve(false);
+            finish(false);
         });
         client.on('connect', () => {
-            clearTimeout(timer);
-            client.destroy();
-            resolve(true);
+            finish(true);
         });
         try {
             client.connect(target);
         }
         catch {
-            clearTimeout(timer);
-            client.destroy();
-            resolve(false);
+            finish(false);
         }
     });
 }
@@ -286,6 +293,9 @@ const stateDirWatchers = new Map();
 // @scheduled cron changes and @webhook additions reach the declared-set
 // without a daemon restart. See watchBaseForProactiveMetadata().
 const baseDirWatchers = new Map();
+// photonDir -> SafeWatcher: startup watcher for new .photon.ts files in each
+// active base. Tracked separately so Bun poll timers are closed on shutdown.
+const startupPhotonDirWatchers = new Map();
 import { compositeKey as _compositeKey, declaredKey as _declaredKey, webhookKey as _webhookKey, locationKey as _locationKey, findByPhoton, asScheduleKey, } from './registry-keys.js';
 /**
  * Create a composite key from photonName + workingDir for map lookups.
@@ -5155,15 +5165,33 @@ function startupWatchPhotons() {
     const defaultBase = getDefaultContext().baseDir;
     const bases = new Set([defaultBase]);
     for (const base of listActiveBases()) {
+        if (!shouldStartupWatchBase(base.path, defaultBase))
+            continue;
         bases.add(base.path);
     }
     for (const photonDir of bases) {
         startupWatchPhotonDir(photonDir, defaultBase);
     }
 }
+function shouldStartupWatchBase(basePath, defaultBase) {
+    const resolved = path.resolve(basePath);
+    if (resolved === path.resolve(defaultBase))
+        return true;
+    // Temporary directories are common in Beam and daemon regression tests. The
+    // bases registry keeps them while the OS temp cleaner has not removed them,
+    // but a production daemon should not spend long-lived watchers on them at
+    // every startup. If a user actively invokes a temp photon later, the normal
+    // request path still loads it and registers on-demand watchers.
+    const tmpRoot = path.resolve(os.tmpdir());
+    if (resolved === tmpRoot || resolved.startsWith(tmpRoot + path.sep))
+        return false;
+    return true;
+}
 function startupWatchPhotonDir(photonDir, defaultBase) {
     if (!fs.existsSync(photonDir))
         return;
+    if (startupPhotonDirWatchers.has(photonDir))
+        return;
     // Host mode: when the default base is host-disabled, skip the file
     // watcher, the eager onInitialize loader, and the directory watcher
     // entirely. They all activate background work that the marker is
@@ -5288,6 +5316,7 @@ function startupWatchPhotonDir(photonDir, defaultBase) {
         if (typeof dirWatcher.on === 'function') {
             dirWatcher.on('error', () => { }); // Non-fatal
         }
+        startupPhotonDirWatchers.set(photonDir, dirWatcher);
     }
     catch {
         logger.warn('Failed to watch photon directory for new files', { dir: photonDir });
@@ -5335,7 +5364,13 @@ function startServer() {
             cleanupSocketSubscriptions(socket);
         });
         socket.on('error', (error) => {
-            logger.warn('Socket error', { error: getErrorMessage(error) });
+            const code = error?.code;
+            if (code === 'ECONNRESET' || code === 'EPIPE') {
+                logger.debug('Socket closed by client', { error: getErrorMessage(error) });
+            }
+            else {
+                logger.warn('Socket error', { error: getErrorMessage(error) });
+            }
             connectedSockets.delete(socket);
             cleanupSocketSubscriptions(socket);
         });
@@ -5670,6 +5705,22 @@ function shutdown() {
         for (const photonPath of fileWatchers.keys()) {
             unwatchPhotonFile(photonPath);
         }
+        for (const watcher of startupPhotonDirWatchers.values()) {
+            watcher.close();
+        }
+        startupPhotonDirWatchers.clear();
+        for (const watcher of baseDirWatchers.values()) {
+            watcher.close();
+        }
+        baseDirWatchers.clear();
+        for (const watcher of parentDirWatchers.values()) {
+            watcher.close();
+        }
+        parentDirWatchers.clear();
+        for (const watcher of stateDirWatchers.values()) {
+            watcher.close();
+        }
+        stateDirWatchers.clear();
         // Clean up poll-based watchers (bun fallback)
         for (const timer of pollTimers) {
             clearInterval(timer);