@jingyi0605/codingns 0.9.7 → 0.9.8

package/dist/server/modules/sessions/session-history-service.d.ts

@@ -75,6 +75,12 @@ interface SessionHistoryAdapterOverrides {
     providerSessionDeleteCli?: ProviderSessionDeleteCli;
 }
 type LiveActivityObservationResolver = (sessionId: string) => SessionActivityObservation | null;
+type SessionTitleChangedObserver = (input: {
+    sessionId: string;
+    userId: string;
+    workspaceId: string;
+    title: string;
+}) => Promise<void> | void;
 type SessionDeletedObserver = (input: {
     sessionId: string;
     userId: string;
@@ -116,6 +122,7 @@ export declare class SessionHistoryService {
     private readonly codexDirtyBindingRepairStates;
     private readonly streamingDeltaSuppressionDebugState;
     private readonly liveActivityObservationResolvers;
+    private readonly sessionTitleChangedObservers;
     private readonly sessionDeletedObservers;
     private readonly workspaceSessionRelations;
     private workspaceStateRefreshTaskSequence;
@@ -127,6 +134,9 @@ export declare class SessionHistoryService {
     registerSessionDeletedObserver(observer: SessionDeletedObserver): {
         close(): void;
     };
+    registerSessionTitleChangedObserver(observer: SessionTitleChangedObserver): {
+        close(): void;
+    };
     private registerBackgroundTasks;
     discoverWorkspaceSessions(workspaceId: string, userId: string, options?: {
         maxAgeMs?: number;
@@ -218,9 +228,10 @@ export declare class SessionHistoryService {
     private shouldSuppressStreamingSessionDelta;
     private publishHistoryEnvelope;
     private syncSessionTitleFromProvider;
-    private requestCodexTitleGenerationIfNeeded;
+    requestCodexTitleGenerationForNewSession(sessionId: string, firstUserMessage: string): void;
     private shouldRequestCodexTitleGeneration;
     private runCodexSessionTitleGeneration;
+    private notifySessionTitleChanged;
     private readFirstUserMessageTitleForSync;
     private resolvePersistedParentSessionId;
     private ensureSessionChangedFilesIndexed;

package/dist/server/modules/sessions/session-history-service.js

@@ -9,6 +9,7 @@ import { isTerminalDebugEnabled, logTerminalDebug, terminalDebugNowMs } from "..
 import { nowIso } from "../../shared/utils/time.js";
 import { inspectSessionActivity } from "./session-activity-inspector.js";
 import { SessionActivityAuthorityService } from "./session-activity-authority-service.js";
+import { buildSessionTitleFromContent, normalizeRuntimePromptTitle } from "./session-title-utils.js";
 import { mapSessionProviderError } from "./session-provider-error-mapper.js";
 import { SessionForkRepository } from "../../storage/repositories/session-fork-repository.js";
 import { buildParallelGroupColorToken, resolveParallelDisplayParentSessionId } from "../parallel-sessions/parallel-session-group-service.js";
@@ -91,6 +92,7 @@ export class SessionHistoryService {
     codexDirtyBindingRepairStates = new Map();
     streamingDeltaSuppressionDebugState = new Map();
     liveActivityObservationResolvers = new Set();
+    sessionTitleChangedObservers = new Set();
     sessionDeletedObservers = new Set();
     workspaceSessionRelations = new Map();
     workspaceStateRefreshTaskSequence = 0;
@@ -204,6 +206,14 @@ export class SessionHistoryService {
             }
         };
     }
+    registerSessionTitleChangedObserver(observer) {
+        this.sessionTitleChangedObservers.add(observer);
+        return {
+            close: () => {
+                this.sessionTitleChangedObservers.delete(observer);
+            }
+        };
+    }
     registerBackgroundTasks() {
         if (!this.taskManager.has(HOST_TASK_TYPES.workspaceDiscovery)) {
             this.taskManager.register({
@@ -238,8 +248,9 @@ export class SessionHistoryService {
             this.taskManager.register({
                 taskType: HOST_TASK_TYPES.sessionCodexTitleGenerate,
                 executionLane: "external_process",
+                concurrency: 1,
                 timeoutMs: 45_000,
-                run: async ({ sessionId }, context) => this.runCodexSessionTitleGeneration(sessionId, context.signal)
+                run: async ({ sessionId, firstUserMessage }, context) => this.runCodexSessionTitleGeneration(sessionId, firstUserMessage, context.signal)
             });
         }
     }
@@ -461,7 +472,6 @@ export class SessionHistoryService {
     async syncSessionTitle(sessionId, signal) {
         const binding = this.getBindingOrThrow(sessionId);
         await this.syncSessionTitleFromProvider(sessionId, binding, signal);
-        this.requestCodexTitleGenerationIfNeeded(sessionId, "session_history.sync_session_title");
     }
     async syncWorkspaceSessionTitles(workspaceId, userId, concurrency = 1, signal) {
         const sessions = this.sessionIndexRepository.listByWorkspace(workspaceId, userId);
@@ -1167,7 +1177,6 @@ export class SessionHistoryService {
             return null;
         }
         await this.syncSessionTitleFromProvider(sessionId, binding);
-        this.requestCodexTitleGenerationIfNeeded(sessionId, "session_history.read_recent_history");
         const snapshot = this.sessionStatusSnapshotRepository.findBySessionId(sessionId);
         this.upsertSnapshot(sessionId, {
             syncStatus: "idle",
@@ -1641,9 +1650,6 @@ export class SessionHistoryService {
             }
             this.workspaceSessionRelations.set(workspaceId, relationMap);
             this.observeDiscoveredProviderActivity(sessions, discoveredSessionIds, userId, timestamp);
-            for (const persisted of persistedSessions) {
-                this.requestCodexTitleGenerationIfNeeded(persisted.sessionId, "session_history.discover_workspace_sessions.codex_title");
-            }
             const listItemsStartedAt = Date.now();
             const items = this.sessionIndexRepository.listByWorkspace(workspaceId, userId);
             listItemsDurationMs = Date.now() - listItemsStartedAt;
@@ -2221,7 +2227,6 @@ export class SessionHistoryService {
             return;
         }
         await this.syncSessionTitleFromProvider(sessionId, binding);
-        this.requestCodexTitleGenerationIfNeeded(sessionId, "session_history.publish_history_envelope");
         const snapshot = this.sessionStatusSnapshotRepository.findBySessionId(sessionId);
         this.upsertSnapshot(sessionId, {
             syncStatus: "idle",
@@ -2274,15 +2279,20 @@ export class SessionHistoryService {
             updatedAt: nowIso()
         });
     }
-    requestCodexTitleGenerationIfNeeded(sessionId, source) {
-        if (!this.shouldRequestCodexTitleGeneration(sessionId)) {
+    requestCodexTitleGenerationForNewSession(sessionId, firstUserMessage) {
+        const normalizedFirstUserMessage = firstUserMessage.trim();
+        if (!normalizedFirstUserMessage) {
+            return;
+        }
+        if (!this.shouldRequestCodexTitleGeneration(sessionId, normalizedFirstUserMessage)) {
             return;
         }
         const handle = this.taskManager.enqueue(HOST_TASK_TYPES.sessionCodexTitleGenerate, {
             key: sessionId,
-            source,
+            source: "session_history.new_codex_session_title",
             input: {
-                sessionId
+                sessionId,
+                firstUserMessage: normalizedFirstUserMessage
             }
         });
         if (handle.deduped) {
@@ -2298,7 +2308,7 @@ export class SessionHistoryService {
             });
         });
     }
-    shouldRequestCodexTitleGeneration(sessionId) {
+    shouldRequestCodexTitleGeneration(sessionId, firstUserMessage) {
         const binding = this.sessionBindingRepository.findBySessionId(sessionId);
         const index = this.sessionIndexRepository.findIndexRecordBySessionId(sessionId);
         if (!binding || !index || binding.provider !== "codex") {
@@ -2307,12 +2317,9 @@ export class SessionHistoryService {
         if (isPendingBindingValue(binding.providerSessionId) || isPendingBindingValue(binding.rawStoreRef)) {
             return false;
         }
-        if (index.messageCount <= 0) {
-            return false;
-        }
-        return true;
+        return shouldGenerateCodexSessionTitle(index.title, firstUserMessage);
     }
-    async runCodexSessionTitleGeneration(sessionId, signal) {
+    async runCodexSessionTitleGeneration(sessionId, firstUserMessage, signal) {
         const binding = this.sessionBindingRepository.findBySessionId(sessionId);
         const index = this.sessionIndexRepository.findIndexRecordBySessionId(sessionId);
         if (!binding || !index || binding.provider !== "codex") {
@@ -2321,20 +2328,17 @@ export class SessionHistoryService {
         if (isPendingBindingValue(binding.providerSessionId) || isPendingBindingValue(binding.rawStoreRef)) {
             return { title: null };
         }
-        const page = await this.sessionSyncService.readHistory(binding.provider, binding.providerSessionId, binding.rawStoreRef, null, 16, "forward").catch(() => null);
-        if (!page) {
-            return { title: null };
-        }
-        const firstUserMessage = page.messages.find((message) => message.role === "user")?.content ?? null;
         if (!shouldGenerateCodexSessionTitle(index.title, firstUserMessage)) {
             return { title: null };
         }
         const title = await this.codexSessionTitleGenerator.generate({
             currentTitle: index.title,
-            messages: page.messages.map((message) => ({
-                role: message.role,
-                content: message.content
-            })),
+            messages: [
+                {
+                    role: "user",
+                    content: firstUserMessage
+                }
+            ],
             signal
         });
         if (!title || !shouldApplyGeneratedCodexSessionTitle(title, firstUserMessage)) {
@@ -2348,9 +2352,29 @@ export class SessionHistoryService {
                 title,
                 updatedAt: nowIso()
             });
+            await this.notifySessionTitleChanged({
+                sessionId,
+                userId: binding.userId,
+                workspaceId: binding.workspaceId,
+                title
+            });
         }
         return { title };
     }
+    async notifySessionTitleChanged(input) {
+        const userId = input.userId ?? this.workspaceRepository.findById(input.workspaceId)?.ownerUserId;
+        if (!userId) {
+            return;
+        }
+        await Promise.allSettled(Array.from(this.sessionTitleChangedObservers).map(async (observer) => {
+            await observer({
+                sessionId: input.sessionId,
+                userId,
+                workspaceId: input.workspaceId,
+                title: input.title
+            });
+        }));
+    }
     async readFirstUserMessageTitleForSync(binding) {
         const pageSize = 20;
         const maxPages = 3;
@@ -4265,7 +4289,7 @@ function shouldMatchSessionBindingByRawStoreRef(provider) {
 function resolveSessionListTitle(provider, existingTitle, fallbackContent, parentTitle = null) {
     const normalizedExistingTitle = existingTitle?.trim() ?? "";
     const normalizedParentTitle = parentTitle?.trim() ?? "";
-    const fallbackTitle = buildUserMessageTitle(fallbackContent, normalizedExistingTitle || "继续对话");
+    const fallbackTitle = buildSessionTitleFromContent(fallbackContent, normalizedExistingTitle || "继续对话");
     if (normalizedExistingTitle.length > 0 &&
         !isSyntheticCodexSessionTitle(normalizedExistingTitle) &&
         (normalizedParentTitle.length === 0 ||
@@ -4280,17 +4304,6 @@ function resolveSessionListTitle(provider, existingTitle, fallbackContent, paren
     }
     return normalizedExistingTitle || fallbackTitle;
 }
-function buildUserMessageTitle(content, fallbackTitle) {
-    const title = content.trim().replace(/\s+/g, " ");
-    return title.slice(0, 48) || fallbackTitle;
-}
-function normalizeRuntimePromptTitle(content) {
-    const normalized = (typeof content === "string" ? content : "").trim().replace(/\s+/g, " ");
-    if (normalized.length === 0) {
-        return null;
-    }
-    return normalized.slice(0, 48);
-}
 function buildRecoveredSessionTitle(provider, providerSessionId) {
     if (isPendingBindingValue(providerSessionId)) {
         return "新会话";