@glwhappen/web-code 1.32.6 → 1.32.9

package/server/modules/providers/list/gemini/gemini-session-synchronizer.provider.ts

@@ -86,6 +86,10 @@ export class GeminiSessionSynchronizer implements IProviderSessionSynchronizer {
         continue;
       }
 
+      if (!projectsDb.getProjectPath(ownerUserId, parsed.projectPath)) {
+        continue;
+      }
+
       const timestamps = await readFileTimestamps(filePath);
       sessionsDb.createSession(
         ownerUserId,
@@ -122,6 +126,10 @@ export class GeminiSessionSynchronizer implements IProviderSessionSynchronizer {
       return null;
     }
 
+    if (!projectsDb.getProjectPath(ownerUserId, parsed.projectPath)) {
+      return null;
+    }
+
     const timestamps = await readFileTimestamps(filePath);
     return sessionsDb.createSession(
       ownerUserId,

package/server/modules/providers/services/sessions-watcher.service.ts

@@ -8,22 +8,22 @@ import { sessionSynchronizerService } from '@/modules/providers/services/session
 import { WS_OPEN_STATE, connectedClients } from '@/modules/websocket/index.js';
 import type { LLMProvider } from '@/shared/types.js';
 import { getProjectsWithSessions } from '@/modules/projects/index.js';
-import { getDefaultOwnerUserId } from '@/shared/default-user.js';
+import { userDb } from '@/modules/database/index.js';
 import { AppError } from '@/shared/utils.js';
 
 /**
- * Resolves the user id used to attribute background-watcher synchronizations.
+ * Resolves active app users that should receive background synchronizations.
  *
- * Returns `null` (with a console warning) when no user has registered yet so
+ * Returns an empty list (with a console warning) when no user has registered yet so
  * that the watcher can keep running silently until the first account exists.
  */
-function resolveWatcherOwnerUserId(): number | null {
+function resolveWatcherOwnerUserIds(): number[] {
   try {
-    return getDefaultOwnerUserId();
+    return userDb.listUsers().map((user) => Number(user.id)).filter((id) => Number.isFinite(id));
   } catch (error) {
     if (error instanceof AppError && error.code === 'DEFAULT_USER_NOT_AVAILABLE') {
       console.warn('Session watcher skipping synchronization: no registered user yet');
-      return null;
+      return [];
     }
     throw error;
   }
@@ -161,33 +161,38 @@ async function flushPendingWatcherUpdate(): Promise<void> {
   watcherRefreshInFlight = true;
 
   try {
-    const ownerUserId = resolveWatcherOwnerUserId();
-    const updatedProjects = ownerUserId !== null
-      ? await getProjectsWithSessions(ownerUserId, { skipSynchronization: true })
-      : [];
     const changeTypes = Array.from(queuedUpdate.changeTypes);
     const watchProviders = Array.from(queuedUpdate.providers);
     const updatedSessionIds = Array.from(queuedUpdate.updatedSessionIds);
+    const targetUserIds = Array.from(
+      new Set(
+        Array.from(connectedClients)
+          .map((client) => Number(client.userId))
+          .filter((userId) => Number.isFinite(userId)),
+      ),
+    );
+
+    await Promise.all(targetUserIds.map(async (userId) => {
+      const updatedProjects = await getProjectsWithSessions(userId, { skipSynchronization: true });
+      const updateMessage = JSON.stringify({
+        type: 'projects_updated',
+        projects: updatedProjects,
+        timestamp: new Date().toISOString(),
+        changeType: changeTypes[0] ?? 'change',
+        updatedSessionId: updatedSessionIds[0] ?? undefined,
+        watchProvider: watchProviders[0] ?? undefined,
+        changeTypes,
+        updatedSessionIds,
+        watchProviders,
+        batched: true,
+      });
 
-    // Backward-compatible fields stay populated with the first queued values.
-    const updateMessage = JSON.stringify({
-      type: 'projects_updated',
-      projects: updatedProjects,
-      timestamp: new Date().toISOString(),
-      changeType: changeTypes[0] ?? 'change',
-      updatedSessionId: updatedSessionIds[0] ?? undefined,
-      watchProvider: watchProviders[0] ?? undefined,
-      changeTypes,
-      updatedSessionIds,
-      watchProviders,
-      batched: true,
-    });
-
-    connectedClients.forEach(client => {
-      if (client.readyState === WS_OPEN_STATE) {
-        client.send(updateMessage);
-      }
-    });
+      connectedClients.forEach((client) => {
+        if (client.readyState === WS_OPEN_STATE && Number(client.userId) === userId) {
+          client.send(updateMessage);
+        }
+      });
+    }));
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     console.error('Session watcher refresh failed while broadcasting projects_updated', { error: message });
@@ -214,21 +219,28 @@ async function onUpdate(
   }
 
   try {
-    const ownerUserId = resolveWatcherOwnerUserId();
-    if (ownerUserId === null) {
+    const ownerUserIds = resolveWatcherOwnerUserIds();
+    if (ownerUserIds.length === 0) {
       return;
     }
 
-    const result = await sessionSynchronizerService.synchronizeProviderFile(ownerUserId, provider, filePath);
-    if (!result.indexed) {
+    const results = await Promise.all(
+      ownerUserIds.map((ownerUserId) =>
+        sessionSynchronizerService.synchronizeProviderFile(ownerUserId, provider, filePath),
+      ),
+    );
+    const indexedResults = results.filter((result) => result.indexed);
+    if (indexedResults.length === 0) {
       return;
     }
 
     console.log(`Session synchronization triggered by ${eventType} event for provider "${provider}"`, {
       filePath,
-      sessionId: result.sessionId,
+      sessionIds: indexedResults.map((result) => result.sessionId).filter(Boolean),
+    });
+    indexedResults.forEach((result) => {
+      queuePendingWatcherUpdate(eventType, provider, result.sessionId);
     });
-    queuePendingWatcherUpdate(eventType, provider, result.sessionId);
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     console.error(`Session watcher sync failed for provider "${provider}"`, {
@@ -245,12 +257,20 @@ async function onUpdate(
 export async function initializeSessionsWatcher(): Promise<void> {
   console.log('Setting up session watchers');
 
-  const initialOwnerUserId = resolveWatcherOwnerUserId();
-  if (initialOwnerUserId !== null) {
-    const initialSync = await sessionSynchronizerService.synchronizeSessions(initialOwnerUserId);
+  const initialOwnerUserIds = resolveWatcherOwnerUserIds();
+  if (initialOwnerUserIds.length > 0) {
+    const initialSyncResults = await Promise.all(
+      initialOwnerUserIds.map(async (ownerUserId) => ({
+        userId: ownerUserId,
+        result: await sessionSynchronizerService.synchronizeSessions(ownerUserId),
+      })),
+    );
     console.log('Initial session synchronization complete', {
-      processedByProvider: initialSync.processedByProvider,
-      failures: initialSync.failures,
+      users: initialSyncResults.map(({ userId, result }) => ({
+        userId,
+        processedByProvider: result.processedByProvider,
+        failures: result.failures,
+      })),
     });
   } else {
     console.log('Initial session synchronization skipped: no registered user yet');

package/server/modules/websocket/services/chat-websocket.service.ts

@@ -127,9 +127,11 @@ export function handleChatConnection(
   dependencies: ChatWebSocketDependencies
 ): void {
   console.log('[INFO] Chat WebSocket connected');
-  connectedClients.add(ws);
+  const clientConnection = ws as WebSocket & { userId?: string | number | null };
+  clientConnection.userId = readRequestUserId(request);
+  connectedClients.add(clientConnection);
 
-  const writer = new WebSocketWriter(ws, readRequestUserId(request));
+  const writer = new WebSocketWriter(ws, clientConnection.userId ?? null);
 
   const sendAuthorizationError = (error: AppError, provider: LLMProvider): void => {
     writer.send({
@@ -338,6 +340,6 @@ export function handleChatConnection(
 
   ws.on('close', () => {
     console.log('[INFO] Chat client disconnected');
-    connectedClients.delete(ws);
+    connectedClients.delete(clientConnection);
   });
 }

package/server/modules/websocket/services/websocket-writer.service.ts

@@ -21,6 +21,13 @@ export class WebSocketWriter {
   send(data: unknown): void {
     if (this.ws.readyState === WS_OPEN_STATE) {
       this.ws.send(JSON.stringify(data));
+    } else {
+      const kind = (data as Record<string, unknown>)?.kind;
+      if (kind === 'complete' || kind === 'error') {
+        console.warn(
+          `[WebSocketWriter] Dropped "${kind}" event — socket not open (readyState=${this.ws.readyState}, session=${this.sessionId})`,
+        );
+      }
     }
   }
 

package/server/shared/types.ts

@@ -32,6 +32,7 @@ export type AnyRecord = Record<string, any>;
 export type RealtimeClientConnection = {
   readyState: number;
   send(data: string): void;
+  userId?: string | number | null;
 };
 
 /**