@proletariat/cli 0.3.36 → 0.3.41

package/dist/lib/pmo/storage/specs.js

@@ -1,11 +1,13 @@
 /**
  * Spec operations for PMO.
+ *
+ * This module uses Drizzle ORM for type-safe database queries.
  */
-import { PMO_TABLES } from '../schema.js';
+import { eq, and, like, or, asc, sql } from 'drizzle-orm';
+import { pmoSpecs, pmoProjects, pmoTickets, pmoWorkflowStatuses, pmoProjectSpecs, pmoSpecDependencies, } from '../../database/drizzle-schema.js';
 import { PMOError } from '../types.js';
 import { generateEntityId } from '../utils.js';
 import { rowToSpec, rowToTicket, wrapSqliteError } from './helpers.js';
-const T = PMO_TABLES;
 export class SpecStorage {
     ctx;
     constructor(ctx) {
@@ -18,16 +20,25 @@ export class SpecStorage {
         const id = spec.id || generateEntityId(this.ctx.db, 'spec');
         const now = Date.now();
         try {
-            this.ctx.db.prepare(`
-        INSERT INTO ${T.specs} (
-          id, title, status, type, tags,
-          problem, solution, decisions, not_now, ui_ux,
-          acceptance_criteria, open_questions,
-          requirements_functional, requirements_technical,
-          context, created_at, updated_at
-        )
-        VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-      `).run(id, spec.title || 'Untitled Spec', spec.status || 'draft', spec.type || null, spec.tags ? JSON.stringify(spec.tags) : null, spec.problem || null, spec.solution || null, spec.decisions || null, spec.notNow || null, spec.uiUx || null, spec.acceptanceCriteria || null, spec.openQuestions || null, spec.requirementsFunctional || null, spec.requirementsTechnical || null, spec.context || null, now, now);
+            this.ctx.drizzle.insert(pmoSpecs).values({
+                id,
+                title: spec.title || 'Untitled Spec',
+                status: spec.status || 'draft',
+                type: spec.type || null,
+                tags: spec.tags ? JSON.stringify(spec.tags) : null,
+                problem: spec.problem || null,
+                solution: spec.solution || null,
+                decisions: spec.decisions || null,
+                notNow: spec.notNow || null,
+                uiUx: spec.uiUx || null,
+                acceptanceCriteria: spec.acceptanceCriteria || null,
+                openQuestions: spec.openQuestions || null,
+                requirementsFunctional: spec.requirementsFunctional || null,
+                requirementsTechnical: spec.requirementsTechnical || null,
+                context: spec.context || null,
+                createdAt: String(now),
+                updatedAt: String(now),
+            }).run();
         }
         catch (err) {
             wrapSqliteError('Spec', 'create', err);
@@ -58,14 +69,29 @@ export class SpecStorage {
      * Get a spec by ID.
      */
     async getSpec(id) {
-        const row = this.ctx.db.prepare(`
-      SELECT id, title, status, type, tags,
-             problem, solution, decisions, not_now, ui_ux,
-             acceptance_criteria, open_questions,
-             requirements_functional, requirements_technical,
-             context, created_at, updated_at
-      FROM ${T.specs} WHERE id = ?
-    `).get(id);
+        const row = this.ctx.drizzle
+            .select({
+            id: pmoSpecs.id,
+            title: pmoSpecs.title,
+            status: pmoSpecs.status,
+            type: pmoSpecs.type,
+            tags: pmoSpecs.tags,
+            problem: pmoSpecs.problem,
+            solution: pmoSpecs.solution,
+            decisions: pmoSpecs.decisions,
+            not_now: pmoSpecs.notNow,
+            ui_ux: pmoSpecs.uiUx,
+            acceptance_criteria: pmoSpecs.acceptanceCriteria,
+            open_questions: pmoSpecs.openQuestions,
+            requirements_functional: pmoSpecs.requirementsFunctional,
+            requirements_technical: pmoSpecs.requirementsTechnical,
+            context: pmoSpecs.context,
+            created_at: pmoSpecs.createdAt,
+            updated_at: pmoSpecs.updatedAt,
+        })
+            .from(pmoSpecs)
+            .where(eq(pmoSpecs.id, id))
+            .get();
         if (!row)
             return null;
         return rowToSpec(row);
@@ -74,30 +100,45 @@ export class SpecStorage {
      * List specs with optional filters.
      */
     async listSpecs(filter) {
-        let query = `
-      SELECT id, title, status, type, tags,
-             problem, solution, decisions, not_now, ui_ux,
-             acceptance_criteria, open_questions,
-             requirements_functional, requirements_technical,
-             context, created_at, updated_at
-      FROM ${T.specs} WHERE 1=1
-    `;
-        const params = [];
+        let query = this.ctx.drizzle
+            .select({
+            id: pmoSpecs.id,
+            title: pmoSpecs.title,
+            status: pmoSpecs.status,
+            type: pmoSpecs.type,
+            tags: pmoSpecs.tags,
+            problem: pmoSpecs.problem,
+            solution: pmoSpecs.solution,
+            decisions: pmoSpecs.decisions,
+            not_now: pmoSpecs.notNow,
+            ui_ux: pmoSpecs.uiUx,
+            acceptance_criteria: pmoSpecs.acceptanceCriteria,
+            open_questions: pmoSpecs.openQuestions,
+            requirements_functional: pmoSpecs.requirementsFunctional,
+            requirements_technical: pmoSpecs.requirementsTechnical,
+            context: pmoSpecs.context,
+            created_at: pmoSpecs.createdAt,
+            updated_at: pmoSpecs.updatedAt,
+        })
+            .from(pmoSpecs)
+            .$dynamic();
+        const conditions = [];
         if (filter?.status) {
-            query += ' AND status = ?';
-            params.push(filter.status);
+            conditions.push(eq(pmoSpecs.status, filter.status));
         }
         if (filter?.type) {
-            query += ' AND type = ?';
-            params.push(filter.type);
+            conditions.push(eq(pmoSpecs.type, filter.type));
         }
         if (filter?.search) {
-            query += ' AND (id LIKE ? OR title LIKE ? OR problem LIKE ? OR solution LIKE ?)';
-            params.push(`%${filter.search}%`, `%${filter.search}%`, `%${filter.search}%`, `%${filter.search}%`);
+            conditions.push(or(like(pmoSpecs.id, `%${filter.search}%`), like(pmoSpecs.title, `%${filter.search}%`), like(pmoSpecs.problem, `%${filter.search}%`), like(pmoSpecs.solution, `%${filter.search}%`)));
         }
-        query += ' ORDER BY title';
-        const rows = this.ctx.db.prepare(query).all(...params);
-        return rows.map(rowToSpec);
+        if (conditions.length > 0) {
+            query = query.where(and(...conditions));
+        }
+        const rows = query
+            .orderBy(asc(pmoSpecs.title))
+            .all();
+        return rows.map((row) => rowToSpec(row));
     }
     /**
      * Update a spec.
@@ -107,69 +148,42 @@ export class SpecStorage {
         if (!existing) {
             throw new PMOError('NOT_FOUND', `Spec not found: ${id}`);
         }
-        const updates = [];
-        const params = [];
-        if (changes.title !== undefined) {
-            updates.push('title = ?');
-            params.push(changes.title);
-        }
-        if (changes.status !== undefined) {
-            updates.push('status = ?');
-            params.push(changes.status);
-        }
-        if (changes.type !== undefined) {
-            updates.push('type = ?');
-            params.push(changes.type);
-        }
-        if (changes.tags !== undefined) {
-            updates.push('tags = ?');
-            params.push(JSON.stringify(changes.tags));
-        }
-        if (changes.problem !== undefined) {
-            updates.push('problem = ?');
-            params.push(changes.problem);
-        }
-        if (changes.solution !== undefined) {
-            updates.push('solution = ?');
-            params.push(changes.solution);
-        }
-        if (changes.decisions !== undefined) {
-            updates.push('decisions = ?');
-            params.push(changes.decisions);
-        }
-        if (changes.notNow !== undefined) {
-            updates.push('not_now = ?');
-            params.push(changes.notNow);
-        }
-        if (changes.uiUx !== undefined) {
-            updates.push('ui_ux = ?');
-            params.push(changes.uiUx);
-        }
-        if (changes.acceptanceCriteria !== undefined) {
-            updates.push('acceptance_criteria = ?');
-            params.push(changes.acceptanceCriteria);
-        }
-        if (changes.openQuestions !== undefined) {
-            updates.push('open_questions = ?');
-            params.push(changes.openQuestions);
-        }
-        if (changes.requirementsFunctional !== undefined) {
-            updates.push('requirements_functional = ?');
-            params.push(changes.requirementsFunctional);
-        }
-        if (changes.requirementsTechnical !== undefined) {
-            updates.push('requirements_technical = ?');
-            params.push(changes.requirementsTechnical);
-        }
-        if (changes.context !== undefined) {
-            updates.push('context = ?');
-            params.push(changes.context);
-        }
-        if (updates.length > 0) {
-            updates.push('updated_at = ?');
-            params.push(Date.now());
-            params.push(id);
-            this.ctx.db.prepare(`UPDATE ${T.specs} SET ${updates.join(', ')} WHERE id = ?`).run(...params);
+        const updates = {};
+        if (changes.title !== undefined)
+            updates.title = changes.title;
+        if (changes.status !== undefined)
+            updates.status = changes.status;
+        if (changes.type !== undefined)
+            updates.type = changes.type;
+        if (changes.tags !== undefined)
+            updates.tags = JSON.stringify(changes.tags);
+        if (changes.problem !== undefined)
+            updates.problem = changes.problem;
+        if (changes.solution !== undefined)
+            updates.solution = changes.solution;
+        if (changes.decisions !== undefined)
+            updates.decisions = changes.decisions;
+        if (changes.notNow !== undefined)
+            updates.notNow = changes.notNow;
+        if (changes.uiUx !== undefined)
+            updates.uiUx = changes.uiUx;
+        if (changes.acceptanceCriteria !== undefined)
+            updates.acceptanceCriteria = changes.acceptanceCriteria;
+        if (changes.openQuestions !== undefined)
+            updates.openQuestions = changes.openQuestions;
+        if (changes.requirementsFunctional !== undefined)
+            updates.requirementsFunctional = changes.requirementsFunctional;
+        if (changes.requirementsTechnical !== undefined)
+            updates.requirementsTechnical = changes.requirementsTechnical;
+        if (changes.context !== undefined)
+            updates.context = changes.context;
+        if (Object.keys(updates).length > 0) {
+            updates.updatedAt = String(Date.now());
+            this.ctx.drizzle
+                .update(pmoSpecs)
+                .set(updates)
+                .where(eq(pmoSpecs.id, id))
+                .run();
         }
         return (await this.getSpec(id));
     }
@@ -182,8 +196,12 @@ export class SpecStorage {
             throw new PMOError('NOT_FOUND', `Spec not found: ${id}`);
         }
         try {
-            this.ctx.db.prepare(`DELETE FROM ${T.specs} WHERE id = ?`).run(id);
-            this.ctx.db.prepare(`UPDATE ${T.tickets} SET spec_id = NULL WHERE spec_id = ?`).run(id);
+            this.ctx.drizzle.delete(pmoSpecs).where(eq(pmoSpecs.id, id)).run();
+            this.ctx.drizzle
+                .update(pmoTickets)
+                .set({ specId: null })
+                .where(eq(pmoTickets.specId, id))
+                .run();
         }
         catch (err) {
             wrapSqliteError('Spec', 'delete', err);
@@ -194,9 +212,11 @@ export class SpecStorage {
      */
     async linkTicketToSpec(ticketId, specId) {
         // Verify ticket exists and get its project
-        const ticket = this.ctx.db.prepare(`
-      SELECT id, project_id FROM ${T.tickets} WHERE id = ?
-    `).get(ticketId);
+        const ticket = this.ctx.drizzle
+            .select({ id: pmoTickets.id, projectId: pmoTickets.projectId })
+            .from(pmoTickets)
+            .where(eq(pmoTickets.id, ticketId))
+            .get();
         if (!ticket) {
             throw new PMOError('NOT_FOUND', `Ticket not found: ${ticketId}`, ticketId);
         }
@@ -205,57 +225,80 @@ export class SpecStorage {
         if (!spec) {
             throw new PMOError('NOT_FOUND', `Spec not found: ${specId}`);
         }
-        this.ctx.db.prepare(`
-      UPDATE ${T.tickets}
-      SET spec_id = ?, updated_at = ?
-      WHERE id = ?
-    `).run(specId, Date.now(), ticketId);
-        this.ctx.updateBoardTimestamp(ticket.project_id);
+        this.ctx.drizzle
+            .update(pmoTickets)
+            .set({ specId, updatedAt: String(Date.now()) })
+            .where(eq(pmoTickets.id, ticketId))
+            .run();
+        this.ctx.updateBoardTimestamp(ticket.projectId);
     }
     /**
      * Unlink a ticket from a spec.
      */
     async unlinkTicketFromSpec(ticketId, specId) {
         // Get ticket's project for board timestamp update
-        const ticket = this.ctx.db.prepare(`
-      SELECT project_id FROM ${T.tickets} WHERE id = ?
-    `).get(ticketId);
-        this.ctx.db.prepare(`
-      UPDATE ${T.tickets}
-      SET spec_id = NULL, updated_at = ?
-      WHERE id = ? AND spec_id = ?
-    `).run(Date.now(), ticketId, specId);
+        const ticket = this.ctx.drizzle
+            .select({ projectId: pmoTickets.projectId })
+            .from(pmoTickets)
+            .where(eq(pmoTickets.id, ticketId))
+            .get();
+        this.ctx.drizzle
+            .update(pmoTickets)
+            .set({ specId: null, updatedAt: String(Date.now()) })
+            .where(and(eq(pmoTickets.id, ticketId), eq(pmoTickets.specId, specId)))
+            .run();
         if (ticket) {
-            this.ctx.updateBoardTimestamp(ticket.project_id);
+            this.ctx.updateBoardTimestamp(ticket.projectId);
         }
     }
     /**
      * Get tickets for a spec.
      */
     async getTicketsForSpec(projectId, specId) {
-        const rows = this.ctx.db.prepare(`
-      SELECT t.*,
-             ws.id as column_id,
-             t.position as position,
-             ws.name as column_name
-      FROM ${T.tickets} t
-      LEFT JOIN ${T.workflow_statuses} ws ON t.status_id = ws.id
-      WHERE t.project_id = ? AND t.spec_id = ?
-      ORDER BY ws.position, t.position ASC, t.created_at ASC
-    `).all(projectId, specId);
-        return Promise.all(rows.map((row) => rowToTicket(this.ctx.db, row)));
+        const rows = this.ctx.drizzle
+            .select({
+            id: pmoTickets.id,
+            project_id: pmoTickets.projectId,
+            title: pmoTickets.title,
+            description: pmoTickets.description,
+            priority: pmoTickets.priority,
+            category: pmoTickets.category,
+            status_id: pmoTickets.statusId,
+            owner: pmoTickets.owner,
+            assignee: pmoTickets.assignee,
+            branch: pmoTickets.branch,
+            spec_id: pmoTickets.specId,
+            epic_id: pmoTickets.epicId,
+            labels: pmoTickets.labels,
+            position: pmoTickets.position,
+            created_at: pmoTickets.createdAt,
+            updated_at: pmoTickets.updatedAt,
+            last_synced_from_spec: pmoTickets.lastSyncedFromSpec,
+            last_synced_from_board: pmoTickets.lastSyncedFromBoard,
+            column_id: pmoWorkflowStatuses.id,
+            column_name: pmoWorkflowStatuses.name,
+            project_name: sql `NULL`,
+        })
+            .from(pmoTickets)
+            .leftJoin(pmoWorkflowStatuses, eq(pmoTickets.statusId, pmoWorkflowStatuses.id))
+            .where(and(eq(pmoTickets.projectId, projectId), eq(pmoTickets.specId, specId)))
+            .orderBy(asc(pmoWorkflowStatuses.position), asc(pmoTickets.position), asc(pmoTickets.createdAt))
+            .all();
+        return Promise.all(rows.map((row) => rowToTicket(this.ctx.drizzle, row)));
     }
     /**
      * Get specs for a ticket.
      */
     async getSpecsForTicket(ticketId) {
-        const ticket = this.ctx.db.prepare(`
-      SELECT spec_id FROM ${T.tickets} WHERE id = ?
-    `).get(ticketId);
-        if (!ticket || !ticket.spec_id) {
+        const ticket = this.ctx.drizzle
+            .select({ specId: pmoTickets.specId })
+            .from(pmoTickets)
+            .where(eq(pmoTickets.id, ticketId))
+            .get();
+        if (!ticket || !ticket.specId) {
             return [];
         }
-        const spec = await this.getSpec(ticket.spec_id);
+        const spec = await this.getSpec(ticket.specId);
         return spec ? [spec] : [];
     }
     /**
@@ -270,31 +313,38 @@ export class SpecStorage {
         if (!dependsOnSpec) {
             throw new PMOError('NOT_FOUND', `Spec not found: ${dependsOnId}`);
         }
-        this.ctx.db.prepare(`
-      INSERT OR IGNORE INTO ${T.spec_dependencies} (spec_id, depends_on_spec_id, created_at)
-      VALUES (?, ?, ?)
-    `).run(specId, dependsOnId, Date.now());
+        this.ctx.drizzle
+            .insert(pmoSpecDependencies)
+            .values({
+            specId,
+            dependsOnSpecId: dependsOnId,
+            createdAt: String(Date.now()),
+        })
+            .onConflictDoNothing()
+            .run();
     }
     /**
      * Remove a spec dependency.
      */
     async removeSpecDependency(specId, dependsOnId) {
-        this.ctx.db.prepare(`
-      DELETE FROM ${T.spec_dependencies}
-      WHERE spec_id = ? AND depends_on_spec_id = ?
-    `).run(specId, dependsOnId);
+        this.ctx.drizzle
+            .delete(pmoSpecDependencies)
+            .where(and(eq(pmoSpecDependencies.specId, specId), eq(pmoSpecDependencies.dependsOnSpecId, dependsOnId)))
+            .run();
     }
     /**
      * Get dependencies for a spec.
      */
     async getSpecDependencies(specId) {
-        const rows = this.ctx.db.prepare(`
-      SELECT depends_on_spec_id FROM ${T.spec_dependencies} WHERE spec_id = ?
-    `).all(specId);
+        const rows = this.ctx.drizzle
+            .select({ dependsOnSpecId: pmoSpecDependencies.dependsOnSpecId })
+            .from(pmoSpecDependencies)
+            .where(eq(pmoSpecDependencies.specId, specId))
+            .all();
         const specs = [];
         for (const row of rows) {
             // eslint-disable-next-line no-await-in-loop -- Sequential lookup for relationship chain
-            const spec = await this.getSpec(row.depends_on_spec_id);
+            const spec = await this.getSpec(row.dependsOnSpecId);
             if (spec)
                 specs.push(spec);
         }
@@ -304,13 +354,15 @@ export class SpecStorage {
      * Get specs that depend on a spec.
      */
     async getSpecDependents(specId) {
-        const rows = this.ctx.db.prepare(`
-      SELECT spec_id FROM ${T.spec_dependencies} WHERE depends_on_spec_id = ?
-    `).all(specId);
+        const rows = this.ctx.drizzle
+            .select({ specId: pmoSpecDependencies.specId })
+            .from(pmoSpecDependencies)
+            .where(eq(pmoSpecDependencies.dependsOnSpecId, specId))
+            .all();
         const specs = [];
         for (const row of rows) {
             // eslint-disable-next-line no-await-in-loop -- Sequential lookup for relationship chain
-            const spec = await this.getSpec(row.spec_id);
+            const spec = await this.getSpec(row.specId);
             if (spec)
                 specs.push(spec);
         }
@@ -321,9 +373,11 @@ export class SpecStorage {
      */
     async linkProjectToSpec(projectId, specId) {
         // Verify project exists
-        const project = this.ctx.db.prepare(`
-      SELECT id FROM ${T.projects} WHERE id = ?
-    `).get(projectId);
+        const project = this.ctx.drizzle
+            .select({ id: pmoProjects.id })
+            .from(pmoProjects)
+            .where(eq(pmoProjects.id, projectId))
+            .get();
         if (!project) {
             throw new PMOError('NOT_FOUND', `Project "${projectId}" not found`);
         }
@@ -332,59 +386,91 @@ export class SpecStorage {
         if (!spec) {
             throw new PMOError('NOT_FOUND', `Spec "${specId}" not found`);
         }
-        this.ctx.db.prepare(`
-      INSERT OR IGNORE INTO ${T.project_specs} (project_id, spec_id, created_at)
-      VALUES (?, ?, ?)
-    `).run(projectId, specId, Date.now());
+        this.ctx.drizzle
+            .insert(pmoProjectSpecs)
+            .values({
+            projectId,
+            specId,
+            createdAt: String(Date.now()),
+        })
+            .onConflictDoNothing()
+            .run();
     }
     /**
      * Unlink a project from a spec.
      */
     async unlinkProjectFromSpec(projectId, specId) {
-        this.ctx.db.prepare(`
-      DELETE FROM ${T.project_specs}
-      WHERE project_id = ? AND spec_id = ?
-    `).run(projectId, specId);
+        this.ctx.drizzle
+            .delete(pmoProjectSpecs)
+            .where(and(eq(pmoProjectSpecs.projectId, projectId), eq(pmoProjectSpecs.specId, specId)))
+            .run();
     }
     /**
      * Get specs for a project.
      */
     async getSpecsForProject(projectId) {
-        const rows = this.ctx.db.prepare(`
-      SELECT s.id, s.title, s.status, s.type, s.tags,
-             s.problem, s.solution, s.decisions, s.not_now, s.ui_ux,
-             s.acceptance_criteria, s.open_questions,
-             s.requirements_functional, s.requirements_technical,
-             s.context, s.created_at, s.updated_at
-      FROM ${T.specs} s
-      JOIN ${T.project_specs} ps ON s.id = ps.spec_id
-      WHERE ps.project_id = ?
-      ORDER BY s.title
-    `).all(projectId);
-        return rows.map(rowToSpec);
+        const rows = this.ctx.drizzle
+            .select({
+            id: pmoSpecs.id,
+            title: pmoSpecs.title,
+            status: pmoSpecs.status,
+            type: pmoSpecs.type,
+            tags: pmoSpecs.tags,
+            problem: pmoSpecs.problem,
+            solution: pmoSpecs.solution,
+            decisions: pmoSpecs.decisions,
+            not_now: pmoSpecs.notNow,
+            ui_ux: pmoSpecs.uiUx,
+            acceptance_criteria: pmoSpecs.acceptanceCriteria,
+            open_questions: pmoSpecs.openQuestions,
+            requirements_functional: pmoSpecs.requirementsFunctional,
+            requirements_technical: pmoSpecs.requirementsTechnical,
+            context: pmoSpecs.context,
+            created_at: pmoSpecs.createdAt,
+            updated_at: pmoSpecs.updatedAt,
+        })
+            .from(pmoSpecs)
+            .innerJoin(pmoProjectSpecs, eq(pmoSpecs.id, pmoProjectSpecs.specId))
+            .where(eq(pmoProjectSpecs.projectId, projectId))
+            .orderBy(asc(pmoSpecs.title))
+            .all();
+        return rows.map((row) => rowToSpec(row));
     }
     /**
      * Get projects for a spec.
      */
     async getProjectsForSpec(specId) {
-        const rows = this.ctx.db.prepare(`
-      SELECT p.*
-      FROM ${T.projects} p
-      JOIN ${T.project_specs} ps ON p.id = ps.project_id
-      WHERE ps.spec_id = ?
-      ORDER BY p.name
-    `).all(specId);
+        const rows = this.ctx.drizzle
+            .select({
+            id: pmoProjects.id,
+            name: pmoProjects.name,
+            template: pmoProjects.template,
+            description: pmoProjects.description,
+            status: pmoProjects.status,
+            phaseId: pmoProjects.phaseId,
+            workflowId: pmoProjects.workflowId,
+            isArchived: pmoProjects.isArchived,
+            targetDate: pmoProjects.targetDate,
+            initiativeId: pmoProjects.initiativeId,
+            createdAt: pmoProjects.createdAt,
+            updatedAt: pmoProjects.updatedAt,
+        })
+            .from(pmoProjects)
+            .innerJoin(pmoProjectSpecs, eq(pmoProjects.id, pmoProjectSpecs.projectId))
+            .where(eq(pmoProjectSpecs.specId, specId))
+            .orderBy(asc(pmoProjects.name))
+            .all();
         return rows.map((row) => ({
             id: row.id,
             name: row.name,
             template: row.template || undefined,
             description: row.description || undefined,
             status: (row.status || 'active'),
-            phaseId: row.phase_id || undefined,
-            isArchived: row.is_archived === 1,
-            targetDate: row.target_date ? new Date(row.target_date) : undefined,
-            createdAt: new Date(row.created_at),
-            updatedAt: new Date(row.updated_at),
+            phaseId: row.phaseId || undefined,
+            isArchived: row.isArchived ?? false,
+            targetDate: row.targetDate ? new Date(row.targetDate) : undefined,
+            createdAt: new Date(row.createdAt || Date.now()),
+            updatedAt: new Date(row.updatedAt || Date.now()),
         }));
     }
 }

package/dist/lib/pmo/storage/tickets.d.ts

@@ -3,6 +3,8 @@
  * Tickets reference workflow statuses directly via status_id.
  * Tickets have a position column for force-ranked ordering within a status.
  * Positions use gapped integers (1000, 2000, 3000...) for stable reordering.
+ *
+ * This module uses Drizzle ORM for type-safe database queries.
  */
 import { CreateTicketInput, Ticket, TicketFilter } from '../types.js';
 import { StorageContext } from './types.js';