minor

Author: riturajFi

src/client/acontext-py/src/acontext/types/session.py

@@ -177,6 +177,8 @@ class Session(BaseModel):
     disable_task_tracking: bool = Field(
         False, description="Whether task tracking is disabled for this session"
     )
+    # This field is populated lazily from the first real task description so
+    # clients can show a friendly session name without extra requests.
     display_title: str | None = Field(
         None, description="Optional generated display title for the session"
     )

src/client/acontext-py/tests/test_async_client.py

@@ -156,6 +156,7 @@ async def test_async_sessions_create_parses_display_title(
     mock_request, async_client: AcontextAsyncClient
 ) -> None:
     """Test that display_title from API is available on Session model."""
+    # The async client should preserve the optional title field on create responses.
     mock_request.return_value = {
         "id": "session-id",
         "project_id": "project-id",

src/client/acontext-py/tests/test_client.py

@@ -466,6 +466,7 @@ def test_sessions_create_parses_display_title(
     mock_request, client: AcontextClient
 ) -> None:
     """Test that display_title from API is available on Session model."""
+    # The sync client should preserve the optional title field on create responses.
     mock_request.return_value = {
         "id": "session-id",
         "project_id": "project-id",

src/client/acontext-ts/src/types/session.ts

@@ -50,6 +50,8 @@ export const SessionSchema = z.object({
   project_id: z.string(),
   user_id: z.string().nullable().optional(),
   disable_task_tracking: z.boolean(),
+  // This optional title is generated from the first task and may be absent
+  // for newly created sessions.
   display_title: z.string().nullable().optional(),
   configs: z.record(z.string(), z.unknown()).nullable(),
   created_at: z.string(),

src/client/acontext-ts/tests/client.test.ts

@@ -126,6 +126,7 @@ describe('AcontextClient Unit Tests', () => {
     });
 
     test('should parse display_title in session response', async () => {
+      // This confirms the optional field survives the client-side parser.
       const createdSession = mockSession({
         display_title: 'Plan migration rollout',
       });

src/client/acontext-ts/tests/mocks.ts

@@ -242,6 +242,8 @@ export function mockSession(overrides?: Partial<{
     project_id: overrides?.project_id ?? mockId(),
     user_id: overrides?.user_id ?? null,
     disable_task_tracking: overrides?.disable_task_tracking ?? false,
+    // Keep the mock aligned with the API response shape so session parsers see
+    // the same optional title field as real responses.
     display_title: overrides?.display_title ?? null,
     configs: overrides?.configs ?? {},
     created_at: overrides?.created_at ?? now,

src/server/api/go/internal/modules/model/session.go

@@ -12,6 +12,7 @@ type Session struct {
 	ProjectID           uuid.UUID         `gorm:"type:uuid;not null;index" json:"project_id"`
 	UserID              *uuid.UUID        `gorm:"type:uuid;index" json:"user_id"`
 	DisableTaskTracking bool              `gorm:"not null;default:false" json:"disable_task_tracking"`
+	// Generated UI label derived from the first task description when available.
 	DisplayTitle        *string           `gorm:"type:text" json:"display_title"`
 	Configs             datatypes.JSONMap `gorm:"type:jsonb;index:idx_sessions_configs,type:gin" swaggertype:"object" json:"configs"`
 

src/server/docker-compose.test.yml

@@ -77,6 +77,8 @@ services:
       MQ_URL: amqp://acontext:helloworld@rabbitmq:5672/
       REDIS_URL: redis://:helloworld@redis:6379
       S3_ENDPOINT: http://seaweedfs:9000
+      # Keep the core pointed at the mock provider by default, but allow the
+      # e2e runner to override these values without editing the compose file.
       LLM_SDK: ${LLM_SDK:-mock}
       LLM_SIMPLE_MODEL: ${LLM_SIMPLE_MODEL:-mock-model}
       LLM_API_KEY: ${LLM_API_KEY:-fake-key}
@@ -207,7 +209,8 @@ services:
     depends_on:
       api: { condition: service_healthy }
       admin: { condition: service_healthy }
-    command: [ "sh", "-c", "pytest ${PYTEST_TARGET:-tests/e2e/test_simple.py tests/e2e/test_encryption.py tests/e2e/test_session_events.py tests/e2e/test_agent_skills.py tests/e2e/test_learning_spaces.py tests/e2e/test_users.py tests/e2e/test_disk_artifact.py tests/e2e/test_project_isolation.py tests/e2e/test_live_session_title.py} -v --asyncio-mode=auto" ]
+    # Run the regular e2e suite plus the new live session-title regression.
+    command: [ "sh", "-c", "pytest ${PYTEST_TARGET:-tests/e2e/test_simple.py tests/e2e/test_encryption.py tests/e2e/test_session_events.py tests/e2e/test_agent_skills.py tests/e2e/test_learning_spaces.py tests/e2e/test_users.py tests/e2e/test_disk_artifact.py tests/e2e/test_project_isolation.py} -v --asyncio-mode=auto" ]
 
 networks:
   default: