fix: make TodoItemSchema tolerant of Claude's TodoWrite payload

[GeT-LeFt]

Problem

Claude's TodoWrite tool sends todo items as:

{ "content": "Fix the bug", "status": "in_progress", "activeForm": "Fixing the bug" }

But TodoItemSchema requires priority and id as mandatory fields:

export const TodoItemSchema = z.object({
    content: z.string(),
    status: z.enum(['pending', 'in_progress', 'completed']),
    priority: z.enum(['high', 'medium', 'low']),  // ← required
    id: z.string()                                 // ← required
})

This causes TodosSchema.safeParse() to return { success: false } in all three extraction paths (extractTodosFromClaudeOutput, extractTodosFromCodexMessage, extractTodosFromAcpMessage), so session.todos is never populated and the web UI todo panel stays permanently empty.

Solution

Make priority and id optional with sensible defaults, and add activeForm as an optional field:

export const TodoItemSchema = z.object({
    content: z.string(),
    status: z.enum(['pending', 'in_progress', 'completed']),
    priority: z.enum(['high', 'medium', 'low']).optional().default('medium'),
    id: z.string().optional().default(''),
    activeForm: z.string().optional()
})

Backwards-compatible: existing data with all fields present still parses correctly (Zod defaults only apply when the field is missing).

Changes

File	Change
shared/src/schemas.ts	Make priority and id optional with defaults, add activeForm

Fixes #501

[github-actions]

Findings

• No high-confidence issues found in the added/modified lines.

Summary

• Review mode: initial
• No issues found in shared/src/schemas.ts:90. Residual risk: this fixes the parsing mismatch, but I did not find automated coverage in repo/docs for TodoWrite payloads that omit priority/id or include activeForm, so a regression here could slip through silently.

Testing

• Not run (automation)

HAPI Bot