fix: round-trip task assignee through mirror layer

The mirror writer hard-coded 'assignee: null' regardless of the actual
record.assigneeId, and the mirror reader (importTaskFromFile) parsed the
frontmatter assignee slug into ParsedTaskFile but never applied it back to
the store on create/update. End result: assignee was invisible in the
markdown mirror and survived neither edit-via-file nor a re-import from a
mirror checkout.

Wire both directions through the team_members table:

* store-manager.buildMirrorTaskAttrs(record): resolve record.assigneeId via
  store.team.get(...).slug for the markdown frontmatter. Falls back to null
  when the team_members row is missing (orphaned FK from a deleted member).
  All 6 task mirror call sites (createTask, updateTask, moveTask,
  reorderTask, bulkMoveTasks, bulkPriorityTasks) now go through this helper
  instead of duplicating the inline literal — eliminates ~50 lines of repetition.

* importTaskFromFile: resolve parsed.assignee (slug) → numeric assigneeId
  via store.team.getBySlug. Unknown slug or empty string both produce
  assigneeId=null (orphaned reference, not an error — the alternative would
  be silently dropping the import). Apply the resolved id in both the
  create and update branches.

Tests (4 new, in store-manager.test.ts under tasks):
* createTask with assigneeId writes 'assignee: <slug>' to task.md frontmatter
* updateTask after team.delete(member) writes 'assignee: null' (orphan handling)
* importTaskFromFile with known slug round-trips to numeric assigneeId on the record
* importTaskFromFile with unknown slug yields assigneeId=null without throwing

Author: prih

src/lib/store-manager.ts

@@ -188,17 +188,31 @@ export class StoreManager {
   // Tasks
   // =========================================================================
 
-  async createTask(data: TaskCreate): Promise<TaskRecord> {
-    const embedding = await this.embedFn(`${data.title} ${data.description ?? ''}`);
-    const record = this.scoped.tasks.create(data, embedding);
-    mirrorTaskCreate(`${this.projectDir}/.tasks`, record.slug, {
+  /**
+   * Build the TaskAttrs object passed to mirrorTaskCreate/mirrorTaskUpdate.
+   * Resolves the numeric `assigneeId` to a human-readable team-member slug for
+   * the markdown frontmatter — falls back to `null` when the team_members row
+   * has been removed (orphaned assignment).
+   */
+  private buildMirrorTaskAttrs(record: TaskRecord) {
+    let assignee: string | null = null;
+    if (record.assigneeId != null) {
+      assignee = this.store.team.get(record.assigneeId)?.slug ?? null;
+    }
+    return {
       title: record.title, description: record.description,
       status: record.status, priority: record.priority,
-      tags: record.tags, order: record.order, assignee: null,
+      tags: record.tags, order: record.order, assignee,
       dueDate: record.dueDate, estimate: record.estimate,
       completedAt: record.completedAt,
       createdAt: record.createdAt, updatedAt: record.updatedAt, version: record.version,
-    }, []);
+    };
+  }
+
+  async createTask(data: TaskCreate): Promise<TaskRecord> {
+    const embedding = await this.embedFn(`${data.title} ${data.description ?? ''}`);
+    const record = this.scoped.tasks.create(data, embedding);
+    mirrorTaskCreate(`${this.projectDir}/.tasks`, record.slug, this.buildMirrorTaskAttrs(record), []);
     this.recordTaskMirrorWrites(record.slug);
     this.emit('task:created', { projectId: this.projectId, taskId: record.id });
     return record;
@@ -215,14 +229,7 @@ export class StoreManager {
       embedding = await this.embedFn(`${title} ${description}`);
     }
     const record = this.scoped.tasks.update(taskId, patch, embedding, authorId, expectedVersion);
-    mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, patch, {
-      title: record.title, description: record.description,
-      status: record.status, priority: record.priority,
-      tags: record.tags, order: record.order, assignee: null,
-      dueDate: record.dueDate, estimate: record.estimate,
-      completedAt: record.completedAt,
-      createdAt: record.createdAt, updatedAt: record.updatedAt, version: record.version,
-    }, []);
+    mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, patch, this.buildMirrorTaskAttrs(record), []);
     this.recordTaskMirrorWrites(record.slug);
     this.emit('task:updated', { projectId: this.projectId, taskId });
     return record;
@@ -238,14 +245,7 @@ export class StoreManager {
 
   moveTask(taskId: number, status: TaskStatus, targetOrder?: number, authorId?: number, expectedVersion?: number): TaskRecord {
     const record = this.scoped.tasks.move(taskId, status, targetOrder, authorId, expectedVersion);
-    mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, { status }, {
-      title: record.title, description: record.description,
-      status: record.status, priority: record.priority,
-      tags: record.tags, order: record.order, assignee: null,
-      dueDate: record.dueDate, estimate: record.estimate,
-      completedAt: record.completedAt,
-      createdAt: record.createdAt, updatedAt: record.updatedAt, version: record.version,
-    }, []);
+    mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, { status }, this.buildMirrorTaskAttrs(record), []);
     this.recordTaskMirrorWrites(record.slug);
     this.emit('task:updated', { projectId: this.projectId, taskId });
     this.emit('task:moved', { projectId: this.projectId, taskId, status });
@@ -256,14 +256,7 @@ export class StoreManager {
     const record = this.scoped.tasks.reorder(taskId, order, status, authorId);
     const patch: TaskPatch = { order };
     if (status) patch.status = status;
-    mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, patch, {
-      title: record.title, description: record.description,
-      status: record.status, priority: record.priority,
-      tags: record.tags, order: record.order, assignee: null,
-      dueDate: record.dueDate, estimate: record.estimate,
-      completedAt: record.completedAt,
-      createdAt: record.createdAt, updatedAt: record.updatedAt, version: record.version,
-    }, []);
+    mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, patch, this.buildMirrorTaskAttrs(record), []);
     this.recordTaskMirrorWrites(record.slug);
     this.emit('task:updated', { projectId: this.projectId, taskId });
     this.emit('task:reordered', { projectId: this.projectId, taskId, order });
@@ -310,14 +303,7 @@ export class StoreManager {
     for (const id of taskIds) {
       const record = this.scoped.tasks.get(id);
       if (!record) continue;
-      mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, { status }, {
-        title: record.title, description: record.description,
-        status: record.status, priority: record.priority,
-        tags: record.tags, order: record.order, assignee: null,
-        dueDate: record.dueDate, estimate: record.estimate,
-        completedAt: record.completedAt,
-        createdAt: record.createdAt, updatedAt: record.updatedAt, version: record.version,
-      }, []);
+      mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, { status }, this.buildMirrorTaskAttrs(record), []);
       this.recordTaskMirrorWrites(record.slug);
     }
     this.emit('task:bulk_moved', { projectId: this.projectId, taskIds, status, count });
@@ -330,14 +316,7 @@ export class StoreManager {
     for (const id of taskIds) {
       const record = this.scoped.tasks.get(id);
       if (!record) continue;
-      mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, { priority }, {
-        title: record.title, description: record.description,
-        status: record.status, priority: record.priority,
-        tags: record.tags, order: record.order, assignee: null,
-        dueDate: record.dueDate, estimate: record.estimate,
-        completedAt: record.completedAt,
-        createdAt: record.createdAt, updatedAt: record.updatedAt, version: record.version,
-      }, []);
+      mirrorTaskUpdate(`${this.projectDir}/.tasks`, record.slug, { priority }, this.buildMirrorTaskAttrs(record), []);
       this.recordTaskMirrorWrites(record.slug);
     }
     this.emit('task:bulk_priority', { projectId: this.projectId, taskIds, priority, count });
@@ -619,6 +598,14 @@ export class StoreManager {
     const existing = this.scoped.tasks.getBySlug(parsed.id);
     const embedding = await this.embedFn(`${parsed.title} ${parsed.description}`);
 
+    // Resolve frontmatter `assignee: <slug>` → numeric team_members.id.
+    // Unknown slug → null (orphaned mirror file references a member that no
+    // longer exists). Empty string from frontmatter is also treated as unset.
+    let assigneeId: number | null = null;
+    if (parsed.assignee) {
+      assigneeId = this.store.team.getBySlug(parsed.assignee)?.id ?? null;
+    }
+
     let record: TaskRecord;
     if (existing) {
       record = this.scoped.tasks.update(existing.id, {
@@ -630,6 +617,7 @@ export class StoreManager {
         dueDate: parsed.dueDate,
         estimate: parsed.estimate,
         completedAt: parsed.completedAt,
+        assigneeId,
       }, embedding);
     } else {
       record = this.scoped.tasks.create({
@@ -641,6 +629,7 @@ export class StoreManager {
         dueDate: parsed.dueDate,
         estimate: parsed.estimate,
         completedAt: parsed.completedAt,
+        assigneeId: assigneeId ?? undefined,
         slug: parsed.id,
         createdAt: parsed.createdAt ?? undefined,
         updatedAt: parsed.updatedAt ?? undefined,

src/tests/store/store-manager.test.ts

@@ -179,6 +179,104 @@ describe('StoreManager', () => {
       expect(manager.getTask(t1.id)!.priority).toBe('critical');
       expect(manager.getTask(t2.id)!.priority).toBe('critical');
     });
+
+    it('mirror writer resolves assigneeId → slug in task.md frontmatter', async () => {
+      // Seed a team member directly via the store
+      const member = store.team.create({ slug: 'qa-bot', name: 'QA Bot' });
+
+      const record = await manager.createTask({
+        title: 'Mirrored',
+        description: '',
+        priority: 'medium',
+        assigneeId: member.id,
+      });
+
+      // Read the task.md snapshot and check its frontmatter
+      const fs = await import('fs');
+      const path = await import('path');
+      const taskMd = fs.readFileSync(path.join(projectDir, '.tasks', record.slug, 'task.md'), 'utf-8');
+      // Frontmatter is YAML — quoted or unquoted slug both acceptable.
+      expect(taskMd).toMatch(/^assignee:\s*['"]?qa-bot['"]?\s*$/m);
+    });
+
+    it('mirror writer falls back to null when assigneeId references missing member', async () => {
+      // Create a member, use it, then delete the row — leaving the task with an orphan FK
+      const member = store.team.create({ slug: 'temp', name: 'Temp' });
+      const record = await manager.createTask({
+        title: 'Orphaned',
+        description: '',
+        priority: 'low',
+        assigneeId: member.id,
+      });
+      store.team.delete(member.id);
+
+      // Trigger a re-mirror via update — buildMirrorTaskAttrs should now write null
+      await manager.updateTask(record.id, { description: 'touch' });
+
+      const fs = await import('fs');
+      const path = await import('path');
+      const taskMd = fs.readFileSync(path.join(projectDir, '.tasks', record.slug, 'task.md'), 'utf-8');
+      expect(taskMd).toMatch(/^assignee:\s*(null|~)\s*$/m);
+    });
+
+    it('importTaskFromFile resolves assignee slug → numeric assigneeId', async () => {
+      const member = store.team.create({ slug: 'imported-bot', name: 'Imported Bot' });
+
+      // Construct a parsed task as the file-mirror watcher would have produced
+      const parsed = {
+        id: 'mirror-import-test',
+        title: 'Imported Task',
+        description: 'from mirror',
+        status: 'todo' as const,
+        priority: 'medium' as const,
+        tags: [],
+        dueDate: null,
+        estimate: null,
+        completedAt: null,
+        assignee: 'imported-bot',
+        createdAt: null,
+        updatedAt: null,
+        version: null,
+        createdBy: null,
+        updatedBy: null,
+        relations: [],
+        attachments: [],
+      };
+
+      await manager.importTaskFromFile(parsed);
+
+      const created = manager.getTaskBySlug('mirror-import-test');
+      expect(created).not.toBeNull();
+      expect(created!.assigneeId).toBe(member.id);
+    });
+
+    it('importTaskFromFile sets assigneeId=null when slug is unknown', async () => {
+      const parsed = {
+        id: 'mirror-import-orphan',
+        title: 'Orphan Import',
+        description: '',
+        status: 'todo' as const,
+        priority: 'low' as const,
+        tags: [],
+        dueDate: null,
+        estimate: null,
+        completedAt: null,
+        assignee: 'no-such-slug',
+        createdAt: null,
+        updatedAt: null,
+        version: null,
+        createdBy: null,
+        updatedBy: null,
+        relations: [],
+        attachments: [],
+      };
+
+      await manager.importTaskFromFile(parsed);
+
+      const created = manager.getTaskBySlug('mirror-import-orphan');
+      expect(created).not.toBeNull();
+      expect(created!.assigneeId).toBeNull();
+    });
   });
 
   // =========================================================================