@clinebot/core 0.0.3 → 0.0.5

package/src/session/unified-session-persistence-service.ts

@@ -10,7 +10,7 @@ import type {
 	SubAgentStartContext,
 } from "@clinebot/agents";
 import type { providers as LlmsProviders } from "@clinebot/llms";
-import { resolveRootSessionId } from "@clinebot/shared";
+import { normalizeUserInput, resolveRootSessionId } from "@clinebot/shared";
 import { nanoid } from "nanoid";
 import { z } from "zod";
 import type { SessionStatus } from "../types/common";
@@ -48,6 +48,58 @@ function stringifyMetadataJson(
 	return JSON.stringify(metadata);
 }
 
+function normalizeSessionTitle(title?: string | null): string | undefined {
+	const trimmed = title?.trim();
+	return trimmed ? trimmed.slice(0, 120) : undefined;
+}
+
+function deriveSessionTitleFromPrompt(
+	prompt?: string | null,
+): string | undefined {
+	const normalizedPrompt = normalizeUserInput(prompt ?? "").trim();
+	if (!normalizedPrompt) {
+		return undefined;
+	}
+	const firstLine = normalizedPrompt.split("\n")[0]?.trim();
+	return normalizeSessionTitle(firstLine);
+}
+
+function normalizeMetadataForStorage(
+	metadata: Record<string, unknown> | null | undefined,
+): Record<string, unknown> | undefined {
+	if (!metadata) {
+		return undefined;
+	}
+	const next = { ...metadata };
+	if (typeof next.title === "string") {
+		const normalizedTitle = normalizeSessionTitle(next.title);
+		if (normalizedTitle) {
+			next.title = normalizedTitle;
+		} else {
+			delete next.title;
+		}
+	} else {
+		delete next.title;
+	}
+	return Object.keys(next).length > 0 ? next : undefined;
+}
+
+function metadataWithResolvedTitle(input: {
+	metadata?: Record<string, unknown> | null;
+	title?: string | null;
+	prompt?: string | null;
+}): Record<string, unknown> | undefined {
+	const next = { ...(normalizeMetadataForStorage(input.metadata) ?? {}) };
+	const resolvedTitle =
+		input.title !== undefined
+			? normalizeSessionTitle(input.title)
+			: deriveSessionTitleFromPrompt(input.prompt);
+	if (resolvedTitle) {
+		next.title = resolvedTitle;
+	}
+	return Object.keys(next).length > 0 ? next : undefined;
+}
+
 export interface PersistedSessionUpdateInput {
 	sessionId: string;
 	expectedStatusLock?: number;
@@ -56,6 +108,7 @@ export interface PersistedSessionUpdateInput {
 	exitCode?: number | null;
 	prompt?: string | null;
 	metadataJson?: string | null;
+	title?: string | null;
 	parentSessionId?: string | null;
 	parentAgentId?: string | null;
 	agentId?: string | null;
@@ -173,6 +226,63 @@ export class UnifiedSessionPersistenceService {
 		);
 	}
 
+	private readSessionManifestFile(sessionId: string): {
+		path: string;
+		manifest?: SessionManifest;
+	} {
+		const manifestPath = this.sessionManifestPath(sessionId, false);
+		if (!existsSync(manifestPath)) {
+			return { path: manifestPath };
+		}
+		try {
+			const manifest = SessionManifestSchema.parse(
+				JSON.parse(readFileSync(manifestPath, "utf8")) as SessionManifest,
+			);
+			return { path: manifestPath, manifest };
+		} catch {
+			return { path: manifestPath };
+		}
+	}
+
+	private applyResolvedTitleToRow(row: SessionRowShape): SessionRowShape {
+		const existingMetadata =
+			typeof row.metadata_json === "string" &&
+			row.metadata_json.trim().length > 0
+				? (() => {
+						try {
+							const parsed = JSON.parse(row.metadata_json) as unknown;
+							if (
+								parsed &&
+								typeof parsed === "object" &&
+								!Array.isArray(parsed)
+							) {
+								return parsed as Record<string, unknown>;
+							}
+						} catch {
+							// Ignore malformed metadata payloads.
+						}
+						return undefined;
+					})()
+				: undefined;
+		const sanitizedMetadata = normalizeMetadataForStorage(existingMetadata);
+		const { manifest } = this.readSessionManifestFile(row.session_id);
+		const manifestTitle = normalizeSessionTitle(
+			typeof manifest?.metadata?.title === "string"
+				? (manifest.metadata.title as string)
+				: undefined,
+		);
+		const resolvedMetadata = manifestTitle
+			? {
+					...(sanitizedMetadata ?? {}),
+					title: manifestTitle,
+				}
+			: sanitizedMetadata;
+		return {
+			...row,
+			metadata_json: stringifyMetadataJson(resolvedMetadata),
+		};
+	}
+
 	private createRootSessionId(): string {
 		return `${Date.now()}_${nanoid(5)}`;
 	}
@@ -207,9 +317,13 @@ export class UnifiedSessionPersistenceService {
 			enable_spawn: input.enableSpawn,
 			enable_teams: input.enableTeams,
 			prompt: input.prompt?.trim() || undefined,
-			metadata: input.metadata,
+			metadata: metadataWithResolvedTitle({
+				metadata: input.metadata,
+				prompt: input.prompt,
+			}),
 			messages_path: messagesPath,
 		});
+		const storedMetadata = normalizeMetadataForStorage(manifest.metadata);
 
 		await this.adapter.upsertSession({
 			session_id: sessionId,
@@ -235,7 +349,7 @@ export class UnifiedSessionPersistenceService {
 			conversation_id: null,
 			is_subagent: 0,
 			prompt: manifest.prompt ?? null,
-			metadata_json: stringifyMetadataJson(manifest.metadata),
+			metadata_json: stringifyMetadataJson(storedMetadata),
 			transcript_path: transcriptPath,
 			hook_path: hookPath,
 			messages_path: messagesPath,
@@ -293,40 +407,86 @@ export class UnifiedSessionPersistenceService {
 		sessionId: string;
 		prompt?: string | null;
 		metadata?: Record<string, unknown> | null;
+		title?: string | null;
 	}): Promise<{ updated: boolean }> {
 		for (let attempt = 0; attempt < 4; attempt++) {
 			const row = await this.adapter.getSession(input.sessionId);
 			if (!row || typeof row.status_lock !== "number") {
 				return { updated: false };
 			}
+			const sanitizedMetadata =
+				input.metadata === undefined
+					? undefined
+					: normalizeMetadataForStorage(input.metadata);
+			const existingMetadata = (() => {
+				const raw = row.metadata_json?.trim();
+				if (!raw) {
+					return undefined;
+				}
+				try {
+					const parsed = JSON.parse(raw) as unknown;
+					if (parsed && typeof parsed === "object" && !Array.isArray(parsed)) {
+						return normalizeMetadataForStorage(
+							parsed as Record<string, unknown>,
+						);
+					}
+				} catch {
+					// Ignore malformed metadata payloads.
+				}
+				return undefined;
+			})();
+			const existingTitle = normalizeSessionTitle(
+				typeof existingMetadata?.title === "string"
+					? (existingMetadata.title as string)
+					: undefined,
+			);
+			const nextTitle =
+				input.title !== undefined
+					? normalizeSessionTitle(input.title)
+					: input.prompt !== undefined
+						? deriveSessionTitleFromPrompt(input.prompt)
+						: existingTitle;
+			const nextMetadata =
+				input.metadata !== undefined
+					? { ...(sanitizedMetadata ?? {}) }
+					: { ...(existingMetadata ?? {}) };
+			if (nextTitle) {
+				nextMetadata.title = nextTitle;
+			} else {
+				delete nextMetadata.title;
+			}
 			const changed = await this.adapter.updateSession({
 				sessionId: input.sessionId,
 				prompt: input.prompt,
 				metadataJson:
-					input.metadata === undefined
+					input.metadata === undefined &&
+					input.prompt === undefined &&
+					input.title === undefined
 						? undefined
-						: stringifyMetadataJson(input.metadata),
+						: stringifyMetadataJson(nextMetadata),
+				title: nextTitle,
 				expectedStatusLock: row.status_lock,
 			});
 			if (!changed.updated) {
 				continue;
 			}
-			const manifestPath = this.sessionManifestPath(input.sessionId, false);
-			if (existsSync(manifestPath)) {
-				try {
-					const manifest = SessionManifestSchema.parse(
-						JSON.parse(readFileSync(manifestPath, "utf8")) as SessionManifest,
-					);
-					if (input.prompt !== undefined) {
-						manifest.prompt = input.prompt ?? undefined;
-					}
-					if (input.metadata !== undefined) {
-						manifest.metadata = input.metadata ?? undefined;
-					}
-					this.writeSessionManifestFile(manifestPath, manifest);
-				} catch {
-					// Ignore malformed manifests and keep backend session state as source of truth.
+			const { path: manifestPath, manifest } = this.readSessionManifestFile(
+				input.sessionId,
+			);
+			if (manifest) {
+				if (input.prompt !== undefined) {
+					manifest.prompt = input.prompt ?? undefined;
+				}
+				const nextMetadata =
+					input.metadata !== undefined
+						? { ...(normalizeMetadataForStorage(input.metadata) ?? {}) }
+						: { ...(normalizeMetadataForStorage(manifest.metadata) ?? {}) };
+				if (nextTitle) {
+					nextMetadata.title = nextTitle;
 				}
+				manifest.metadata =
+					Object.keys(nextMetadata).length > 0 ? nextMetadata : undefined;
+				this.writeSessionManifestFile(manifestPath, manifest);
 			}
 			return { updated: true };
 		}
@@ -422,7 +582,9 @@ export class UnifiedSessionPersistenceService {
 				conversation_id: input.conversationId,
 				is_subagent: 1,
 				prompt,
-				metadata_json: null,
+				metadata_json: stringifyMetadataJson(
+					metadataWithResolvedTitle({ prompt }),
+				),
 				transcript_path: artifactPaths.transcriptPath,
 				hook_path: artifactPaths.hookPath,
 				messages_path: artifactPaths.messagesPath,
@@ -444,6 +606,30 @@ export class UnifiedSessionPersistenceService {
 			agentId: input.agentId,
 			conversationId: input.conversationId,
 			prompt: existing.prompt ?? prompt ?? null,
+			metadataJson: stringifyMetadataJson(
+				metadataWithResolvedTitle({
+					metadata: (() => {
+						const raw = existing.metadata_json?.trim();
+						if (!raw) {
+							return undefined;
+						}
+						try {
+							const parsed = JSON.parse(raw) as unknown;
+							if (
+								parsed &&
+								typeof parsed === "object" &&
+								!Array.isArray(parsed)
+							) {
+								return parsed as Record<string, unknown>;
+							}
+						} catch {
+							// Ignore malformed metadata payloads.
+						}
+						return undefined;
+					})(),
+					prompt: existing.prompt ?? prompt ?? null,
+				}),
+			),
 			expectedStatusLock: existing.status_lock,
 		});
 		return sessionId;
@@ -601,7 +787,9 @@ export class UnifiedSessionPersistenceService {
 			conversation_id: null,
 			is_subagent: 1,
 			prompt: message || `Team task for ${agentId}`,
-			metadata_json: null,
+			metadata_json: stringifyMetadataJson(
+				metadataWithResolvedTitle({ prompt: message }),
+			),
 			transcript_path: transcriptPath,
 			hook_path: hookPath,
 			messages_path: messagesPath,
@@ -750,7 +938,9 @@ export class UnifiedSessionPersistenceService {
 			}
 			rows = await this.adapter.listSessions({ limit: scanLimit });
 		}
-		return rows.slice(0, requestedLimit);
+		return rows
+			.slice(0, requestedLimit)
+			.map((row) => this.applyResolvedTitleToRow(row));
 	}
 
 	async deleteSession(sessionId: string): Promise<{ deleted: boolean }> {

package/src/types/config.ts

@@ -15,7 +15,7 @@ import type {
 	SessionPromptConfig,
 	SessionWorkspaceConfig,
 } from "@clinebot/shared";
-import type { ToolRoutingRule } from "../default-tools/model-tool-routing.js";
+import type { ToolRoutingRule } from "../tools/model-tool-routing.js";
 
 export type CoreAgentMode = AgentMode;
 

package/dist/index.browser.d.ts

@@ -1 +0,0 @@
-export * from "./index";