crewly 1.13.0 → 1.13.1

package/dist/cli/backend/src/services/chat-v2/sqlite/channel.store.js

@@ -0,0 +1,317 @@
+/**
+ * ChannelStore — CRUD for `chat_channels`.
+ *
+ * All authorization is handled at a higher layer (`ChatV2Service`);
+ * this store only enforces DB-level invariants (FKs, unique indexes).
+ *
+ * @module services/chat-v2/sqlite/channel.store
+ */
+import { randomUUID } from 'crypto';
+import { CHAT_ERROR_CODES, ChatError, } from '../types.js';
+// ---------------------------------------------------------------------------
+// Constants — shared SELECT column list, single source of truth
+// ---------------------------------------------------------------------------
+/**
+ * Canonical column list for `chat_channels` SELECTs. Including all Phase A
+ * columns (type / team_id / project_id / target_member_id) here keeps every
+ * read path in this store mapping to a fully-populated `ChatChannelRow`.
+ */
+const CHANNEL_SELECT_COLUMNS = `
+  id, agent_session, owner_user_id, name, purpose,
+  created_at, archived_at, last_message_at,
+  type, team_id, project_id, target_member_id
+`;
+// ---------------------------------------------------------------------------
+// Store
+// ---------------------------------------------------------------------------
+/** SQLite-backed store for chat channels. */
+export class ChannelStore {
+    db;
+    constructor(db) {
+        this.db = db;
+    }
+    /**
+     * Create a new channel. Enforces the 1:1 agent-binding by catching
+     * the SQLite unique-constraint error and surfacing it as a
+     * `ChatError(agent_already_bound, 409)`.
+     *
+     * @param input - The channel creation payload
+     * @returns The inserted channel row
+     * @throws {ChatError} `agent_already_bound` (409) if the agent is already bound
+     *   to another active channel.
+     */
+    create(input) {
+        const id = input.id ?? randomUUID();
+        const createdAt = input.nowMs ?? Date.now();
+        const purpose = input.purpose ?? null;
+        const channelType = input.type ?? 'dm';
+        const teamId = input.teamId ?? null;
+        const projectId = input.projectId ?? null;
+        const targetMemberId = input.targetMemberId ?? null;
+        const stmt = this.db.prepare(`INSERT INTO chat_channels
+         (id, agent_session, owner_user_id, name, purpose, created_at,
+          type, team_id, project_id, target_member_id)
+       VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`);
+        try {
+            stmt.run(id, input.agentSession, input.ownerUserId, input.name, purpose, createdAt, channelType, teamId, projectId, targetMemberId);
+        }
+        catch (err) {
+            // Unique constraint means either the partial index on agent_session fired
+            // (the common case — 1:1 binding violated) or we raced an id collision.
+            // In either case it's safer to check `findActiveByAgentSession` before
+            // surfacing a typed error.
+            const msg = err instanceof Error ? err.message : String(err);
+            if (msg.includes('UNIQUE')) {
+                const existing = this.findActiveByAgentSession(input.agentSession);
+                if (existing) {
+                    throw new ChatError(CHAT_ERROR_CODES.AGENT_ALREADY_BOUND, 409, `Agent "${input.agentSession}" is already bound to another active channel.`, { existingChannelId: existing.id });
+                }
+            }
+            throw err;
+        }
+        const created = this.getById(id);
+        // Row must exist — insert just succeeded. This branch is for type-narrowing.
+        if (!created) {
+            throw new ChatError(CHAT_ERROR_CODES.INTERNAL, 500, 'Channel disappeared immediately after insert');
+        }
+        return created;
+    }
+    /**
+     * Look up a channel by its id (active or archived).
+     *
+     * @param id - The channel id
+     * @returns The row, or null if not found
+     */
+    getById(id) {
+        const row = this.db
+            .prepare(`SELECT ${CHANNEL_SELECT_COLUMNS}
+         FROM chat_channels
+         WHERE id = ?`)
+            .get(id);
+        return row ?? null;
+    }
+    /**
+     * Find the single active channel bound to an agent session, if any.
+     * Uses the partial unique index, so at most one row is returned.
+     *
+     * @param agentSession - The agent session id
+     * @returns The active channel row, or null
+     */
+    findActiveByAgentSession(agentSession) {
+        // Phase A: scope to type='dm' so this method's contract matches the
+        // post-migration `uq_channel_agent_dm_active` partial unique index.
+        // type='channel' rows can share the empty agent_session sentinel; we
+        // never want this lookup to surface those.
+        const row = this.db
+            .prepare(`SELECT ${CHANNEL_SELECT_COLUMNS}
+         FROM chat_channels
+         WHERE agent_session = ? AND archived_at IS NULL AND type = 'dm'
+         LIMIT 1`)
+            .get(agentSession);
+        return row ?? null;
+    }
+    /**
+     * Find the most-recent active DM channel owned by a specific user and
+     * bound to a specific agent session. Used by the `ensureDmChannel`
+     * helper backing the /agents page — the owner-scoped variant is needed
+     * because the `uq_channel_agent_dm_active` unique index was dropped
+     * (see chat-db.ts:380-386), so multiple DM rows for the same agent
+     * can coexist (e.g. created by legacy bridge code with the synthetic
+     * `'system'` owner).
+     *
+     * Ordering matches `listByOwner`: most-recent activity first, so the
+     * caller always lands on the DM the user was last using.
+     *
+     * @param ownerUserId - The user_id that must own the channel
+     * @param agentSession - The agent session id
+     * @returns The most-recent active DM channel row, or null
+     */
+    findActiveDmByOwnerAndAgent(ownerUserId, agentSession) {
+        const row = this.db
+            .prepare(`SELECT ${CHANNEL_SELECT_COLUMNS}
+         FROM chat_channels
+         WHERE owner_user_id = ?
+           AND agent_session = ?
+           AND archived_at IS NULL
+           AND type = 'dm'
+         ORDER BY COALESCE(last_message_at, created_at) DESC
+         LIMIT 1`)
+            .get(ownerUserId, agentSession);
+        return row ?? null;
+    }
+    /**
+     * Find the oldest active `type='channel'` row for a team, if any.
+     *
+     * Team channels are not owner-scoped — a team channel belongs to the
+     * whole team — so this lookup is keyed on `team_id` alone. Ordering by
+     * `created_at ASC` returns the team's original (e.g. `#general`) channel
+     * stably, which is what `ensureTeamChannel` treats as the canonical
+     * "the team already has a channel" marker.
+     *
+     * @param teamId - The team id whose channel to find
+     * @returns The oldest active team channel row, or null when none exists
+     */
+    findActiveChannelByTeam(teamId) {
+        const row = this.db
+            .prepare(`SELECT ${CHANNEL_SELECT_COLUMNS}
+         FROM chat_channels
+         WHERE team_id = ? AND archived_at IS NULL AND type = 'channel'
+         ORDER BY created_at ASC
+         LIMIT 1`)
+            .get(teamId);
+        return row ?? null;
+    }
+    /**
+     * List channels owned by a user.
+     *
+     * Phase C — extended with `type` + `teamId` filter options so the
+     * `GET /api/chat/channels` endpoint can serve the channel-rail's
+     * grouped/workspace-scoped views without shipping every row to the
+     * client. Filters compose with AND; passing `undefined` for either is
+     * a no-op (the existing all-channels behavior).
+     *
+     * @param ownerUserId - The user_id whose channels to return
+     * @param options - Listing options
+     * @param options.includeArchived - When false (default), filter out archived rows
+     * @param options.limit - Max rows to return (capped at 100)
+     * @param options.type - Phase C: when set, filter rows to this channel type
+     *   (`'dm'` or `'channel'`). Useful for the channel-rail's "DMs only" /
+     *   "Channels only" views.
+     * @param options.teamId - Phase C: when set, filter rows whose `team_id`
+     *   matches. Used by the workspace-scoped Channels group; empty / null
+     *   `team_id` rows are excluded by this filter on purpose (DMs and
+     *   workspace-less rows belong to no team).
+     * @returns Channel rows sorted by `last_message_at DESC, created_at DESC`
+     */
+    listByOwner(ownerUserId, options) {
+        const includeArchived = options?.includeArchived ?? false;
+        const limit = Math.min(options?.limit ?? 50, 100);
+        // Build WHERE clauses + bound params positionally so the filter set
+        // composes cleanly. Each branch is independent — no implicit coupling.
+        const where = ['owner_user_id = ?'];
+        const params = [ownerUserId];
+        if (!includeArchived) {
+            where.push('archived_at IS NULL');
+        }
+        if (options?.type !== undefined) {
+            where.push('type = ?');
+            params.push(options.type);
+        }
+        if (options?.teamId !== undefined) {
+            where.push('team_id = ?');
+            params.push(options.teamId);
+        }
+        const sql = `
+      SELECT ${CHANNEL_SELECT_COLUMNS}
+      FROM chat_channels
+      WHERE ${where.join(' AND ')}
+      ORDER BY COALESCE(last_message_at, created_at) DESC
+      LIMIT ?
+    `;
+        params.push(limit);
+        return this.db.prepare(sql).all(...params);
+    }
+    /**
+     * List active "bridged" channels — inbound conversations from external
+     * surfaces (currently Slack), persisted by the bridge under the synthetic
+     * `'system'` owner and keyed by a `slack-…` id. These belong to no single
+     * Crewly user, so they are surfaced in the consolidated chat list across
+     * the (single-user OSS) account rather than via `listByOwner`.
+     *
+     * @param options.includeArchived - Include archived rows (default false).
+     * @param options.limit - Max rows (capped at 200).
+     * @returns Bridged channel rows, most-recent first.
+     */
+    listBridged(options) {
+        const includeArchived = options?.includeArchived ?? false;
+        const limit = Math.min(options?.limit ?? 100, 200);
+        const where = ["id LIKE 'slack-%'"];
+        if (!includeArchived)
+            where.push('archived_at IS NULL');
+        const sql = `
+      SELECT ${CHANNEL_SELECT_COLUMNS}
+      FROM chat_channels
+      WHERE ${where.join(' AND ')}
+      ORDER BY COALESCE(last_message_at, created_at) DESC
+      LIMIT ?
+    `;
+        return this.db.prepare(sql).all(limit);
+    }
+    /**
+     * Mark a channel as archived (soft-delete). No-op if already archived.
+     *
+     * @param id - The channel id
+     * @param nowMs - Optional override for the archive timestamp
+     * @returns True if a row was updated (was active), false otherwise
+     */
+    archive(id, nowMs) {
+        const result = this.db
+            .prepare(`UPDATE chat_channels
+         SET archived_at = ?
+         WHERE id = ? AND archived_at IS NULL`)
+            .run(nowMs ?? Date.now(), id);
+        return result.changes > 0;
+    }
+    /**
+     * Phase 6.0b — clear an archived flag. Inverse of {@link archive};
+     * required by the legacy `unarchiveConversation` controller route.
+     *
+     * @param id - The channel id
+     * @returns True if a row was updated (was archived), false otherwise
+     */
+    unarchive(id) {
+        const result = this.db
+            .prepare(`UPDATE chat_channels
+         SET archived_at = NULL
+         WHERE id = ? AND archived_at IS NOT NULL`)
+            .run(id);
+        return result.changes > 0;
+    }
+    /**
+     * Phase 6.0b — rename a channel. Used by the legacy
+     * `updateConversationTitle` controller route.
+     *
+     * @param id - The channel id
+     * @param name - The new name
+     * @returns True if a row was updated, false if the channel doesn't exist
+     */
+    rename(id, name) {
+        const result = this.db
+            .prepare(`UPDATE chat_channels SET name = ? WHERE id = ?`)
+            .run(name, id);
+        return result.changes > 0;
+    }
+    /**
+     * Phase 6.0b — hard-delete a channel and all its messages. Note: this
+     * is distinct from {@link archive} (soft delete via archived_at).
+     * Used by the legacy `deleteConversation` controller route.
+     *
+     * FK ON DELETE CASCADE on `chat_messages.channel_id` cleans up
+     * messages atomically with the channel row.
+     *
+     * @param id - The channel id to remove
+     * @returns True if a row was removed, false if no such channel
+     */
+    hardDelete(id) {
+        const result = this.db
+            .prepare(`DELETE FROM chat_channels WHERE id = ?`)
+            .run(id);
+        return result.changes > 0;
+    }
+    /**
+     * Bump `last_message_at` to a provided timestamp if it is greater than
+     * the current value. Safe to call many times.
+     *
+     * @param id - The channel id
+     * @param timestampMs - The timestamp to record
+     */
+    touchLastMessageAt(id, timestampMs) {
+        this.db
+            .prepare(`UPDATE chat_channels
+         SET last_message_at = ?
+         WHERE id = ?
+           AND (last_message_at IS NULL OR last_message_at < ?)`)
+            .run(timestampMs, id, timestampMs);
+    }
+}
+//# sourceMappingURL=channel.store.js.map

package/dist/cli/backend/src/services/chat-v2/sqlite/channel.store.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"channel.store.js","sourceRoot":"","sources":["../../../../../../../backend/src/services/chat-v2/sqlite/channel.store.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AACpC,OAAO,EACL,gBAAgB,EAChB,SAAS,GAGV,MAAM,aAAa,CAAC;AAGrB,8EAA8E;AAC9E,gEAAgE;AAChE,8EAA8E;AAE9E;;;;GAIG;AACH,MAAM,sBAAsB,GAAG;;;;CAI9B,CAAC;AAmCF,8EAA8E;AAC9E,QAAQ;AACR,8EAA8E;AAE9E,6CAA6C;AAC7C,MAAM,OAAO,YAAY;IACM;IAA7B,YAA6B,EAAgB;QAAhB,OAAE,GAAF,EAAE,CAAc;IAAG,CAAC;IAEjD;;;;;;;;;OASG;IACH,MAAM,CAAC,KAAyB;QAC9B,MAAM,EAAE,GAAG,KAAK,CAAC,EAAE,IAAI,UAAU,EAAE,CAAC;QACpC,MAAM,SAAS,GAAG,KAAK,CAAC,KAAK,IAAI,IAAI,CAAC,GAAG,EAAE,CAAC;QAC5C,MAAM,OAAO,GAAG,KAAK,CAAC,OAAO,IAAI,IAAI,CAAC;QACtC,MAAM,WAAW,GAAoB,KAAK,CAAC,IAAI,IAAI,IAAI,CAAC;QACxD,MAAM,MAAM,GAAG,KAAK,CAAC,MAAM,IAAI,IAAI,CAAC;QACpC,MAAM,SAAS,GAAG,KAAK,CAAC,SAAS,IAAI,IAAI,CAAC;QAC1C,MAAM,cAAc,GAAG,KAAK,CAAC,cAAc,IAAI,IAAI,CAAC;QAEpD,MAAM,IAAI,GAAG,IAAI,CAAC,EAAE,CAAC,OAAO,CAC1B;;;6CAGuC,CACxC,CAAC;QAEF,IAAI,CAAC;YACH,IAAI,CAAC,GAAG,CACN,EAAE,EACF,KAAK,CAAC,YAAY,EAClB,KAAK,CAAC,WAAW,EACjB,KAAK,CAAC,IAAI,EACV,OAAO,EACP,SAAS,EACT,WAAW,EACX,MAAM,EACN,SAAS,EACT,cAAc,CACf,CAAC;QACJ,CAAC;QAAC,OAAO,GAAG,EAAE,CAAC;YACb,0EAA0E;YAC1E,wEAAwE;YACxE,uEAAuE;YACvE,2BAA2B;YAC3B,MAAM,GAAG,GAAG,GAAG,YAAY,KAAK,CAAC,CAAC,CAAC,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC;YAC7D,IAAI,GAAG,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;gBAC3B,MAAM,QAAQ,GAAG,IAAI,CAAC,wBAAwB,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC;gBACnE,IAAI,QAAQ,EAAE,CAAC;oBACb,MAAM,IAAI,SAAS,CACjB,gBAAgB,CAAC,mBAAmB,EACpC,GAAG,EACH,UAAU,KAAK,CAAC,YAAY,+CAA+C,EAC3E,EAAE,iBAAiB,EAAE,QAAQ,CAAC,EAAE,EAAE,CACnC,CAAC;gBACJ,CAAC;YACH,CAAC;YACD,MAAM,GAAG,CAAC;QACZ,CAAC;QAED,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QACjC,6EAA6E;QAC7E,IAAI,CAAC,OAAO,EAAE,CAAC;YACb,MAAM,IAAI,SAAS,CACjB,gBAAgB,CAAC,QAAQ,EACzB,GAAG,EACH,8CAA8C,CAC/C,CAAC;QACJ,CAAC;QACD,OAAO,OAAO,CAAC;IACjB,CAAC;IAED;;;;;OAKG;IACH,OAAO,CAAC,EAAU;QAChB,MAAM,GAAG,GAAG,IAAI,CAAC,EAAE;aAChB,OAAO,CACN,UAAU,sBAAsB;;sBAElB,CACf;aACA,GAAG,CAAC,EAAE,CAA+B,CAAC;QACzC,OAAO,GAAG,IAAI,IAAI,CAAC;IACrB,CAAC;IAED;;;;;;OAMG;IACH,wBAAwB,CAAC,YAAoB;QAC3C,oEAAoE;QACpE,oEAAoE;QACpE,qEAAqE;QACrE,2CAA2C;QAC3C,MAAM,GAAG,GAAG,IAAI,CAAC,EAAE;aAChB,OAAO,CACN,UAAU,sBAAsB;;;iBAGvB,CACV;aACA,GAAG,CAAC,YAAY,CAA+B,CAAC;QACnD,OAAO,GAAG,IAAI,IAAI,CAAC;IACrB,CAAC;IAED;;;;;;;;;;;;;;;OAeG;IACH,2BAA2B,CACzB,WAAmB,EACnB,YAAoB;QAEpB,MAAM,GAAG,GAAG,IAAI,CAAC,EAAE;aAChB,OAAO,CACN,UAAU,sBAAsB;;;;;;;iBAOvB,CACV;aACA,GAAG,CAAC,WAAW,EAAE,YAAY,CAA+B,CAAC;QAChE,OAAO,GAAG,IAAI,IAAI,CAAC;IACrB,CAAC;IAED;;;;;;;;;;;OAWG;IACH,uBAAuB,CAAC,MAAc;QACpC,MAAM,GAAG,GAAG,IAAI,CAAC,EAAE;aAChB,OAAO,CACN,UAAU,sBAAsB;;;;iBAIvB,CACV;aACA,GAAG,CAAC,MAAM,CAA+B,CAAC;QAC7C,OAAO,GAAG,IAAI,IAAI,CAAC;IACrB,CAAC;IAED;;;;;;;;;;;;;;;;;;;;;OAqBG;IACH,WAAW,CACT,WAAmB,EACnB,OAKC;QAED,MAAM,eAAe,GAAG,OAAO,EAAE,eAAe,IAAI,KAAK,CAAC;QAC1D,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,KAAK,IAAI,EAAE,EAAE,GAAG,CAAC,CAAC;QAElD,oEAAoE;QACpE,uEAAuE;QACvE,MAAM,KAAK,GAAa,CAAC,mBAAmB,CAAC,CAAC;QAC9C,MAAM,MAAM,GAAc,CAAC,WAAW,CAAC,CAAC;QAExC,IAAI,CAAC,eAAe,EAAE,CAAC;YACrB,KAAK,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;QACpC,CAAC;QACD,IAAI,OAAO,EAAE,IAAI,KAAK,SAAS,EAAE,CAAC;YAChC,KAAK,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;YACvB,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;QAC5B,CAAC;QACD,IAAI,OAAO,EAAE,MAAM,KAAK,SAAS,EAAE,CAAC;YAClC,KAAK,CAAC,IAAI,CAAC,aAAa,CAAC,CAAC;YAC1B,MAAM,CAAC,IAAI,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;QAC9B,CAAC;QAED,MAAM,GAAG,GAAG;eACD,sBAAsB;;cAEvB,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC;;;KAG5B,CAAC;QACF,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACnB,OAAO,IAAI,CAAC,EAAE,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,GAAG,MAAM,CAAqB,CAAC;IACjE,CAAC;IAED;;;;;;;;;;OAUG;IACH,WAAW,CAAC,OAAuD;QACjE,MAAM,eAAe,GAAG,OAAO,EAAE,eAAe,IAAI,KAAK,CAAC;QAC1D,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,EAAE,KAAK,IAAI,GAAG,EAAE,GAAG,CAAC,CAAC;QACnD,MAAM,KAAK,GAAa,CAAC,mBAAmB,CAAC,CAAC;QAC9C,IAAI,CAAC,eAAe;YAAE,KAAK,CAAC,IAAI,CAAC,qBAAqB,CAAC,CAAC;QACxD,MAAM,GAAG,GAAG;eACD,sBAAsB;;cAEvB,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC;;;KAG5B,CAAC;QACF,OAAO,IAAI,CAAC,EAAE,CAAC,OAAO,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,KAAK,CAAqB,CAAC;IAC7D,CAAC;IAED;;;;;;OAMG;IACH,OAAO,CAAC,EAAU,EAAE,KAAc;QAChC,MAAM,MAAM,GAAG,IAAI,CAAC,EAAE;aACnB,OAAO,CACN;;8CAEsC,CACvC;aACA,GAAG,CAAC,KAAK,IAAI,IAAI,CAAC,GAAG,EAAE,EAAE,EAAE,CAAC,CAAC;QAChC,OAAO,MAAM,CAAC,OAAO,GAAG,CAAC,CAAC;IAC5B,CAAC;IAED;;;;;;OAMG;IACH,SAAS,CAAC,EAAU;QAClB,MAAM,MAAM,GAAG,IAAI,CAAC,EAAE;aACnB,OAAO,CACN;;kDAE0C,CAC3C;aACA,GAAG,CAAC,EAAE,CAAC,CAAC;QACX,OAAO,MAAM,CAAC,OAAO,GAAG,CAAC,CAAC;IAC5B,CAAC;IAED;;;;;;;OAOG;IACH,MAAM,CAAC,EAAU,EAAE,IAAY;QAC7B,MAAM,MAAM,GAAG,IAAI,CAAC,EAAE;aACnB,OAAO,CAAC,gDAAgD,CAAC;aACzD,GAAG,CAAC,IAAI,EAAE,EAAE,CAAC,CAAC;QACjB,OAAO,MAAM,CAAC,OAAO,GAAG,CAAC,CAAC;IAC5B,CAAC;IAED;;;;;;;;;;OAUG;IACH,UAAU,CAAC,EAAU;QACnB,MAAM,MAAM,GAAG,IAAI,CAAC,EAAE;aACnB,OAAO,CAAC,wCAAwC,CAAC;aACjD,GAAG,CAAC,EAAE,CAAC,CAAC;QACX,OAAO,MAAM,CAAC,OAAO,GAAG,CAAC,CAAC;IAC5B,CAAC;IAED;;;;;;OAMG;IACH,kBAAkB,CAAC,EAAU,EAAE,WAAmB;QAChD,IAAI,CAAC,EAAE;aACJ,OAAO,CACN;;;gEAGwD,CACzD;aACA,GAAG,CAAC,WAAW,EAAE,EAAE,EAAE,WAAW,CAAC,CAAC;IACvC,CAAC;CACF"}

package/dist/cli/backend/src/services/chat-v2/sqlite/chat-db.d.ts

@@ -0,0 +1,119 @@
+/**
+ * Chat V2 SQLite bootstrap.
+ *
+ * Opens (or creates) the chat database at the configured path, applies
+ * the Phase 1 migration idempotently, and returns a ready-to-use
+ * `better-sqlite3` Database instance.
+ *
+ * Schema matches tech-spec §3.2. All tables use `INTEGER ms-since-epoch (UTC)`
+ * for timestamps and raw UUID strings for IDs.
+ *
+ * @module services/chat-v2/sqlite/chat-db
+ */
+import { type ComponentLogger } from '../../core/logger.service.js';
+/**
+ * F-CYCLE7-1: install a test-only stub for the better-sqlite3 loader.
+ *
+ * Pass a function to override; pass `null` to restore the real loader
+ * AND clear the cached `better-sqlite3` reference so the next call
+ * exercises the stub. Tests MUST call this with `null` in `afterEach`
+ * to avoid leaking state across cases.
+ *
+ * @param fn - Stub `require`-like function or `null` to restore real loader
+ */
+export declare function setBetterSqlite3LoaderForTesting(fn: ((name: string) => unknown) | null): void;
+/** Type alias for a `better-sqlite3` Database instance. */
+export type ChatDatabase = import('better-sqlite3').Database;
+/**
+ * Idempotent base DDL — Phase 1 schema with Phase A column additions baked in.
+ *
+ * Safe to run every boot.
+ *
+ * Differences vs. the literal spec §3.2:
+ * - Added `IF NOT EXISTS` on every CREATE so reboots don't fail.
+ * - PRAGMAs set outside the transaction (SQLite disallows PRAGMA in txn).
+ *
+ * Phase A additions (SEALED design 2026-04-25 §3.1 + §3.2) are reflected
+ * directly in the CREATE TABLE bodies so a fresh-install database lands on
+ * the new schema in one shot. Pre-existing databases (created under the
+ * Phase 1 schema) get the same columns added by `applyPhaseAColumnUpgrades`
+ * below, which uses `ALTER TABLE ADD COLUMN` guarded by `PRAGMA table_info`.
+ */
+export declare const CHAT_V2_MIGRATION_SQL = "\nCREATE TABLE IF NOT EXISTS chat_channels (\n  id                TEXT PRIMARY KEY,\n  agent_session     TEXT NOT NULL,\n  owner_user_id     TEXT NOT NULL,\n  name              TEXT NOT NULL,\n  purpose           TEXT,\n  created_at        INTEGER NOT NULL,\n  archived_at       INTEGER,\n  last_message_at   INTEGER,\n  -- Phase A (SEALED \u00A73.1): channel-type discriminator. 'dm' preserves the\n  -- Phase 1 1:1 user\u2194agent contract; 'channel' is the Slack-like team\n  -- surface. 'huddle' (Phase B-2, 2026-05-17) is ad-hoc multi-agent\n  -- groups with explicit membership in chat_channel_members.\n  -- Existing rows backfill to 'dm' via applyPhaseAColumnUpgrades.\n  type              TEXT NOT NULL DEFAULT 'dm' CHECK(type IN ('dm','channel','huddle')),\n  -- Phase A (SEALED \u00A73.1): team workspace link. Required at the service\n  -- layer when type='channel'; null for type='dm'.\n  team_id           TEXT,\n  -- Phase A (SEALED \u00A73.1): optional project link for project-scoped channels.\n  project_id        TEXT,\n  -- Phase A (SEALED \u00A73.1): for type='dm', the resolved member-ID being DM'd\n  -- (distinct from agent_session which is the wire-level binding key).\n  target_member_id  TEXT\n);\n\nCREATE INDEX IF NOT EXISTS ix_channels_owner\n  ON chat_channels(owner_user_id, archived_at);\n\n-- Phase A indexes that reference the new columns (type, team_id, thread_id)\n-- live in CHAT_V2_PHASE_A_INDEX_SQL below \u2014 they must be created AFTER\n-- applyPhaseAColumnUpgrades runs so the columns exist on legacy DBs.\n\nCREATE TABLE IF NOT EXISTS chat_messages (\n  id           TEXT PRIMARY KEY,\n  channel_id   TEXT NOT NULL REFERENCES chat_channels(id) ON DELETE CASCADE,\n  seq          INTEGER NOT NULL,\n  sender_type  TEXT NOT NULL CHECK(sender_type IN ('user','agent','system')),\n  sender_id    TEXT NOT NULL,\n  content      TEXT NOT NULL,\n  content_type TEXT NOT NULL CHECK(content_type IN ('text','markdown','image_ref','system_note'))\n               DEFAULT 'markdown',\n  created_at   INTEGER NOT NULL,\n  metadata     TEXT,\n  -- Phase A (SEALED \u00A73.2): JSON-encoded array of mention IDs (member or\n  -- team) referenced inline in the content field. Stored as a JSON string;\n  -- service layer treats null as []. Bounded at insert time; see types.ts.\n  mentions     TEXT,\n  -- Phase A (SEALED \u00A73.2): optional Slack-style thread root. When set,\n  -- this message is a reply within the thread rooted at thread_id.\n  thread_id    TEXT\n);\n\nCREATE UNIQUE INDEX IF NOT EXISTS uq_messages_channel_seq\n  ON chat_messages(channel_id, seq);\n\nCREATE INDEX IF NOT EXISTS ix_messages_channel_created\n  ON chat_messages(channel_id, created_at DESC);\n\n-- Phase 1: partial unique index for clientMessageId-based idempotency (spec \u00A74.4)\nCREATE UNIQUE INDEX IF NOT EXISTS uq_messages_client_id\n  ON chat_messages(channel_id, json_extract(metadata, '$.clientMessageId'))\n  WHERE json_extract(metadata, '$.clientMessageId') IS NOT NULL;\n\nCREATE TABLE IF NOT EXISTS chat_attachments (\n  id            TEXT PRIMARY KEY,\n  message_id    TEXT NOT NULL REFERENCES chat_messages(id) ON DELETE CASCADE,\n  kind          TEXT NOT NULL CHECK(kind IN ('image')),\n  mime_type     TEXT NOT NULL,\n  size_bytes    INTEGER NOT NULL,\n  local_path    TEXT NOT NULL,\n  original_name TEXT,\n  created_at    INTEGER NOT NULL\n);\n\nCREATE INDEX IF NOT EXISTS ix_attachments_message\n  ON chat_attachments(message_id);\n\nCREATE TABLE IF NOT EXISTS chat_offline_queue (\n  id            INTEGER PRIMARY KEY AUTOINCREMENT,\n  channel_id    TEXT NOT NULL,\n  agent_session TEXT NOT NULL,\n  message_id    TEXT NOT NULL REFERENCES chat_messages(id) ON DELETE CASCADE,\n  queued_at     INTEGER NOT NULL,\n  delivered_at  INTEGER,\n  attempts      INTEGER NOT NULL DEFAULT 0\n);\n\nCREATE INDEX IF NOT EXISTS ix_queue_pending\n  ON chat_offline_queue(agent_session, delivered_at)\n  WHERE delivered_at IS NULL;\n\n-- Phase B-2 (2026-05-17): huddle membership roster. One row per\n-- (huddle_channel, agent_session_name). For type='huddle' channels\n-- only; 'dm' and 'channel' rows have no entries here.\n--\n-- Dispatcher uses this list to fan a user message out to every\n-- huddle member; the @-mention subset gets a \"must respond\" prompt\n-- variant while the rest get an \"optional\" one.\nCREATE TABLE IF NOT EXISTS chat_channel_members (\n  channel_id      TEXT NOT NULL REFERENCES chat_channels(id) ON DELETE CASCADE,\n  member_session  TEXT NOT NULL,\n  joined_at       INTEGER NOT NULL,\n  PRIMARY KEY (channel_id, member_session)\n);\n\nCREATE INDEX IF NOT EXISTS ix_channel_members_session\n  ON chat_channel_members(member_session);\n";
+/**
+ * Phase A indexes that reference Phase A columns. Must run AFTER
+ * `applyPhaseAColumnUpgrades` adds those columns on pre-Phase-A databases —
+ * otherwise the partial-index `WHERE` clauses fail with "no such column".
+ *
+ * On a fresh database this runs after `CHAT_V2_MIGRATION_SQL` has already
+ * baked the columns into the CREATE TABLE bodies, so each step is a
+ * harmless no-op (`CREATE INDEX IF NOT EXISTS`).
+ */
+export declare const CHAT_V2_PHASE_A_INDEX_SQL = "\n-- Phase A (SEALED \u00A73.1) used to define uq_channel_agent_dm_active\n-- enforcing 1:1 agent<->DM-channel binding. That constraint was dropped\n-- per the unified-chat-message-store spec (2026-05-14 \u00A7\"Phase 2 design\n-- discovery\" \u2192 Option B): legacy ChatService models \"one conversation\n-- per Slack thread or web chat session\", so an agent participates in\n-- N concurrent DM-style channels. Dropping the index removes the\n-- agent_already_bound (409) failure that would otherwise block\n-- migration of the legacy multi-thread model. Existing databases get\n-- the drop applied in applyPhaseAColumnUpgrades.\n\n-- Phase A (SEALED \u00A73.1): scoped lookup for \"all channels in team T\" and\n-- \"project P channels\". Indexes by team first because team-scoped reads\n-- are the dominant Phase B+ access pattern.\nCREATE INDEX IF NOT EXISTS ix_channels_team\n  ON chat_channels(team_id, archived_at)\n  WHERE team_id IS NOT NULL;\n\n-- Phase A (SEALED \u00A73.2): thread-pane lookup. Filtered to non-null so the\n-- (smaller) index only covers actual threaded replies.\nCREATE INDEX IF NOT EXISTS ix_messages_thread\n  ON chat_messages(thread_id, seq)\n  WHERE thread_id IS NOT NULL;\n";
+/**
+ * Bring a pre-existing chat database up to Phase A's schema by adding
+ * any missing columns and the new indexes / dropping the superseded ones.
+ *
+ * Safe to run on a fresh database too — every step is no-op idempotent.
+ * Specifically:
+ *   - `ensureColumn` is a no-op when the column is already present (the
+ *     fresh-install case, since the columns are baked into the CREATE
+ *     TABLE in `CHAT_V2_MIGRATION_SQL`).
+ *   - `DROP INDEX IF EXISTS uq_channel_agent_active` removes the legacy
+ *     unconditional partial index so the new dm-scoped one isn't shadowed.
+ *     The DROP is metadata-only — no row data is affected.
+ *
+ * Exported (separately from `openChatDatabase`) so tests can simulate the
+ * pre-Phase-A → Phase A migration path explicitly.
+ *
+ * @param db - The chat database handle
+ * @returns A small report describing what changed (for logging)
+ */
+export declare function applyPhaseAColumnUpgrades(db: ChatDatabase): {
+    channelsAdded: string[];
+    messagesAdded: string[];
+    legacyIndexDropped: boolean;
+    phaseADmIndexDropped: boolean;
+    /**
+     * Phase B-2: whether `chat_channels` was rebuilt to drop the legacy
+     * `CHECK(type IN ('dm','channel'))` constraint and allow `'huddle'`.
+     * `false` on fresh installs (the new CHECK is already in the
+     * baseline DDL) and on subsequent boots after the first rebuild.
+     */
+    channelsCheckRebuilt: boolean;
+};
+/**
+ * Options accepted by `openChatDatabase`.
+ */
+export interface OpenChatDatabaseOptions {
+    /** Absolute path to the SQLite file. Parent dirs will be created. */
+    dbPath: string;
+    /**
+     * When true, uses SQLite `:memory:` regardless of `dbPath`.
+     * Intended for unit tests. Defaults to false.
+     */
+    inMemory?: boolean;
+    /**
+     * When true, suppress the integrity-check warning on open. Tests use this
+     * to avoid noisy logs. Defaults to false.
+     */
+    skipIntegrityCheck?: boolean;
+    /** Optional logger override — defaults to LoggerService. */
+    logger?: ComponentLogger;
+}
+/**
+ * Open (or create) the chat database, apply PRAGMAs, run the Phase 1
+ * migration idempotently, and run a boot-time integrity check.
+ *
+ * @param options - Opener options
+ * @returns A ready-to-use better-sqlite3 Database handle
+ *
+ * @example
+ * ```ts
+ * const db = openChatDatabase({ dbPath: '/tmp/chat.db' });
+ * // ... use db ...
+ * db.close();
+ * ```
+ */
+export declare function openChatDatabase(options: OpenChatDatabaseOptions): ChatDatabase;
+//# sourceMappingURL=chat-db.d.ts.map

package/dist/cli/backend/src/services/chat-v2/sqlite/chat-db.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"chat-db.d.ts","sourceRoot":"","sources":["../../../../../../../backend/src/services/chat-v2/sqlite/chat-db.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAMH,OAAO,EAAiB,KAAK,eAAe,EAAE,MAAM,8BAA8B,CAAC;AAsCnF;;;;;;;;;GASG;AACH,wBAAgB,gCAAgC,CAC9C,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,MAAM,KAAK,OAAO,CAAC,GAAG,IAAI,GACrC,IAAI,CAGN;AAoCD,2DAA2D;AAC3D,MAAM,MAAM,YAAY,GAAG,OAAO,gBAAgB,EAAE,QAAQ,CAAC;AAM7D;;;;;;;;;;;;;;GAcG;AACH,eAAO,MAAM,qBAAqB,0lJA4GjC,CAAC;AAEF;;;;;;;;GAQG;AACH,eAAO,MAAM,yBAAyB,8rCAuBrC,CAAC;AA2DF;;;;;;;;;;;;;;;;;;GAkBG;AACH,wBAAgB,yBAAyB,CAAC,EAAE,EAAE,YAAY,GAAG;IAC3D,aAAa,EAAE,MAAM,EAAE,CAAC;IACxB,aAAa,EAAE,MAAM,EAAE,CAAC;IACxB,kBAAkB,EAAE,OAAO,CAAC;IAC5B,oBAAoB,EAAE,OAAO,CAAC;IAC9B;;;;;OAKG;IACH,oBAAoB,EAAE,OAAO,CAAC;CAC/B,CA0FA;AAsHD;;GAEG;AACH,MAAM,WAAW,uBAAuB;IACtC,qEAAqE;IACrE,MAAM,EAAE,MAAM,CAAC;IACf;;;OAGG;IACH,QAAQ,CAAC,EAAE,OAAO,CAAC;IACnB;;;OAGG;IACH,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,4DAA4D;IAC5D,MAAM,CAAC,EAAE,eAAe,CAAC;CAC1B;AAED;;;;;;;;;;;;;GAaG;AACH,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,uBAAuB,GAAG,YAAY,CAwD/E"}