@pikku/kysely 0.12.0 → 0.12.2

package/CHANGELOG.md

@@ -1,5 +1,34 @@
 ## 0.12.0
 
+## 0.12.2
+
+### Patch Changes
+
+- ce961b5: fix: improve MySQL compatibility in AI storage service by using varchar columns with explicit lengths instead of text for primary keys, foreign keys, and indexed columns, and handle duplicate index errors gracefully
+- 3e04565: chore: update dependencies to latest minor/patch versions
+- Updated dependencies [cc4c9e9]
+- Updated dependencies [3e04565]
+  - @pikku/core@0.12.2
+
+## 0.12.1
+
+### Patch Changes
+
+- e04531f: Code quality improvements: resolve oxlint warnings and apply autofixes across the codebase (unused bindings, unnecessary constructors, prefer `const` over `let`, etc.). No behaviour changes.
+- Updated dependencies [62a8725]
+- Updated dependencies [a3bdb0d]
+- Updated dependencies [e0349ff]
+- Updated dependencies [62a8725]
+- Updated dependencies [e04531f]
+- Updated dependencies [62a8725]
+- Updated dependencies [a83efb8]
+- Updated dependencies [8eed717]
+- Updated dependencies [62a8725]
+- Updated dependencies [62a8725]
+- Updated dependencies [62a8725]
+
+  - @pikku/core@0.12.1
+
 - Updated dependencies
 
 ## 0.11.0

package/dist/src/kysely-agent-run-service.d.ts

@@ -1,5 +1,5 @@
 import type { AIThread, AIMessage, AgentRunRow, AgentRunService } from '@pikku/core/ai-agent';
-import { Kysely } from 'kysely';
+import type { Kysely } from 'kysely';
 import type { KyselyPikkuDB } from './kysely-tables.js';
 export declare class KyselyAgentRunService implements AgentRunService {
     private db;

package/dist/src/kysely-ai-storage-service.d.ts

@@ -1,11 +1,12 @@
 import type { AIStorageService, AIRunStateService, CreateRunInput } from '@pikku/core/services';
 import type { AIThread, AIMessage, AgentRunState } from '@pikku/core/ai-agent';
-import { Kysely } from 'kysely';
+import type { Kysely } from 'kysely';
 import type { KyselyPikkuDB } from './kysely-tables.js';
 export declare class KyselyAIStorageService implements AIStorageService, AIRunStateService {
     private db;
     private initialized;
     constructor(db: Kysely<KyselyPikkuDB>);
+    private createIndexSafe;
     init(): Promise<void>;
     createThread(resourceId: string, options?: {
         threadId?: string;

package/dist/src/kysely-ai-storage-service.js

@@ -6,6 +6,17 @@ export class KyselyAIStorageService {
     constructor(db) {
         this.db = db;
     }
+    async createIndexSafe(builder) {
+        try {
+            await builder.execute();
+        }
+        catch (e) {
+            // Ignore "index already exists" errors (MySQL doesn't support IF NOT EXISTS for indexes)
+            if (e?.code === 'ER_DUP_KEYNAME' || e?.errno === 1061)
+                return;
+            throw e;
+        }
+    }
     async init() {
         if (this.initialized) {
             return;
@@ -13,68 +24,60 @@ export class KyselyAIStorageService {
         await this.db.schema
             .createTable('ai_threads')
             .ifNotExists()
-            .addColumn('id', 'text', (col) => col.primaryKey())
-            .addColumn('resource_id', 'text', (col) => col.notNull())
+            .addColumn('id', 'varchar(36)', (col) => col.primaryKey())
+            .addColumn('resource_id', 'varchar(255)', (col) => col.notNull())
             .addColumn('title', 'text')
             .addColumn('metadata', 'text')
             .addColumn('created_at', 'timestamp', (col) => col.defaultTo(sql `CURRENT_TIMESTAMP`).notNull())
             .addColumn('updated_at', 'timestamp', (col) => col.defaultTo(sql `CURRENT_TIMESTAMP`).notNull())
             .execute();
-        await this.db.schema
+        await this.createIndexSafe(this.db.schema
             .createIndex('idx_ai_threads_resource')
-            .ifNotExists()
             .on('ai_threads')
-            .column('resource_id')
-            .execute();
+            .column('resource_id'));
         await this.db.schema
             .createTable('ai_message')
             .ifNotExists()
-            .addColumn('id', 'text', (col) => col.primaryKey())
-            .addColumn('thread_id', 'text', (col) => col.notNull().references('ai_threads.id').onDelete('cascade'))
-            .addColumn('role', 'text', (col) => col.notNull())
+            .addColumn('id', 'varchar(36)', (col) => col.primaryKey())
+            .addColumn('thread_id', 'varchar(36)', (col) => col.notNull().references('ai_threads.id').onDelete('cascade'))
+            .addColumn('role', 'varchar(50)', (col) => col.notNull())
             .addColumn('content', 'text')
             .addColumn('created_at', 'timestamp', (col) => col.defaultTo(sql `CURRENT_TIMESTAMP`).notNull())
             .execute();
-        await this.db.schema
+        await this.createIndexSafe(this.db.schema
             .createIndex('idx_ai_message_thread')
-            .ifNotExists()
             .on('ai_message')
-            .columns(['thread_id', 'created_at'])
-            .execute();
+            .columns(['thread_id', 'created_at']));
         await this.db.schema
             .createTable('ai_tool_call')
             .ifNotExists()
-            .addColumn('id', 'text', (col) => col.primaryKey())
-            .addColumn('thread_id', 'text', (col) => col.notNull().references('ai_threads.id').onDelete('cascade'))
-            .addColumn('message_id', 'text', (col) => col.notNull().references('ai_message.id').onDelete('cascade'))
-            .addColumn('run_id', 'text')
-            .addColumn('tool_name', 'text', (col) => col.notNull())
-            .addColumn('args', 'text', (col) => col.notNull().defaultTo('{}'))
+            .addColumn('id', 'varchar(36)', (col) => col.primaryKey())
+            .addColumn('thread_id', 'varchar(36)', (col) => col.notNull().references('ai_threads.id').onDelete('cascade'))
+            .addColumn('message_id', 'varchar(36)', (col) => col.notNull().references('ai_message.id').onDelete('cascade'))
+            .addColumn('run_id', 'varchar(36)')
+            .addColumn('tool_name', 'varchar(255)', (col) => col.notNull())
+            .addColumn('args', 'text', (col) => col.notNull())
             .addColumn('result', 'text')
-            .addColumn('approval_status', 'text')
-            .addColumn('approval_type', 'text')
-            .addColumn('agent_run_id', 'text')
-            .addColumn('display_tool_name', 'text')
+            .addColumn('approval_status', 'varchar(50)')
+            .addColumn('approval_type', 'varchar(50)')
+            .addColumn('agent_run_id', 'varchar(36)')
+            .addColumn('display_tool_name', 'varchar(255)')
             .addColumn('display_args', 'text')
             .addColumn('created_at', 'timestamp', (col) => col.defaultTo(sql `CURRENT_TIMESTAMP`).notNull())
             .execute();
-        await this.db.schema
+        await this.createIndexSafe(this.db.schema
             .createIndex('idx_ai_tool_call_thread')
-            .ifNotExists()
             .on('ai_tool_call')
-            .column('thread_id')
-            .execute();
-        await this.db.schema
+            .column('thread_id'));
+        await this.createIndexSafe(this.db.schema
             .createIndex('idx_ai_tool_call_message')
-            .ifNotExists()
             .on('ai_tool_call')
-            .column('message_id')
-            .execute();
+            .column('message_id'));
         await this.db.schema
             .createTable('ai_working_memory')
             .ifNotExists()
-            .addColumn('id', 'text', (col) => col.notNull())
-            .addColumn('scope', 'text', (col) => col.notNull())
+            .addColumn('id', 'varchar(255)', (col) => col.notNull())
+            .addColumn('scope', 'varchar(50)', (col) => col.notNull())
             .addColumn('data', 'text', (col) => col.notNull())
             .addColumn('updated_at', 'timestamp', (col) => col.defaultTo(sql `CURRENT_TIMESTAMP`).notNull())
             .addPrimaryKeyConstraint('ai_working_memory_pk', ['id', 'scope'])
@@ -82,25 +85,23 @@ export class KyselyAIStorageService {
         await this.db.schema
             .createTable('ai_run')
             .ifNotExists()
-            .addColumn('run_id', 'text', (col) => col.primaryKey())
-            .addColumn('agent_name', 'text', (col) => col.notNull())
-            .addColumn('thread_id', 'text', (col) => col.notNull().references('ai_threads.id').onDelete('cascade'))
-            .addColumn('resource_id', 'text', (col) => col.notNull())
-            .addColumn('status', 'text', (col) => col.notNull().defaultTo('running'))
+            .addColumn('run_id', 'varchar(36)', (col) => col.primaryKey())
+            .addColumn('agent_name', 'varchar(255)', (col) => col.notNull())
+            .addColumn('thread_id', 'varchar(36)', (col) => col.notNull().references('ai_threads.id').onDelete('cascade'))
+            .addColumn('resource_id', 'varchar(255)', (col) => col.notNull())
+            .addColumn('status', 'varchar(50)', (col) => col.notNull().defaultTo('running'))
             .addColumn('suspend_reason', 'text')
             .addColumn('missing_rpcs', 'text')
             .addColumn('usage_input_tokens', 'integer', (col) => col.notNull().defaultTo(0))
             .addColumn('usage_output_tokens', 'integer', (col) => col.notNull().defaultTo(0))
-            .addColumn('usage_model', 'text', (col) => col.notNull().defaultTo(''))
+            .addColumn('usage_model', 'varchar(255)', (col) => col.notNull().defaultTo(''))
             .addColumn('created_at', 'timestamp', (col) => col.defaultTo(sql `CURRENT_TIMESTAMP`).notNull())
             .addColumn('updated_at', 'timestamp', (col) => col.defaultTo(sql `CURRENT_TIMESTAMP`).notNull())
             .execute();
-        await this.db.schema
+        await this.createIndexSafe(this.db.schema
             .createIndex('idx_ai_run_thread')
-            .ifNotExists()
             .on('ai_run')
-            .columns(['thread_id', 'created_at'])
-            .execute();
+            .columns(['thread_id', 'created_at']));
         this.initialized = true;
     }
     async createThread(resourceId, options) {
@@ -192,16 +193,17 @@ export class KyselyAIStorageService {
                 msgQuery = msgQuery.where('created_at', '<', cursorRow.created_at);
             }
         }
+        let msgResult;
         if (options?.cursor || options?.lastN) {
             const innerResult = await msgQuery
                 .orderBy('created_at', 'desc')
                 .limit(options?.lastN ?? 50)
                 .execute();
             innerResult.reverse();
-            var msgResult = innerResult;
+            msgResult = innerResult;
         }
         else {
-            var msgResult = await msgQuery.orderBy('created_at', 'asc').execute();
+            msgResult = await msgQuery.orderBy('created_at', 'asc').execute();
         }
         const tcResult = await this.db
             .selectFrom('ai_tool_call')
@@ -218,10 +220,26 @@ export class KyselyAIStorageService {
         }
         const messages = [];
         for (const row of msgResult) {
+            const rawContent = row.content;
+            let parsedContent = rawContent ?? undefined;
+            if (rawContent) {
+                try {
+                    const parsed = JSON.parse(rawContent);
+                    if (Array.isArray(parsed)) {
+                        parsedContent = parsed;
+                    }
+                    else if (typeof parsed === 'string') {
+                        parsedContent = parsed;
+                    }
+                }
+                catch {
+                    // Not JSON, use raw string
+                }
+            }
             const msg = {
                 id: row.id,
                 role: row.role,
-                content: row.content ?? undefined,
+                content: parsedContent,
                 createdAt: new Date(row.created_at),
             };
             const tcs = tcByMessage.get(msg.id);
@@ -263,7 +281,7 @@ export class KyselyAIStorageService {
                 id: msg.id,
                 thread_id: threadId,
                 role: msg.role,
-                content: msg.content ?? null,
+                content: msg.content != null ? JSON.stringify(msg.content) : null,
                 created_at: msg.createdAt ?? new Date(),
             })))
                 .execute();

package/dist/src/kysely-channel-store.d.ts

@@ -1,6 +1,7 @@
-import { CoreUserSession } from '@pikku/core';
-import { Channel, ChannelStore } from '@pikku/core/channel';
-import { Kysely } from 'kysely';
+import type { CoreUserSession } from '@pikku/core';
+import type { Channel } from '@pikku/core/channel';
+import { ChannelStore } from '@pikku/core/channel';
+import type { Kysely } from 'kysely';
 import type { KyselyPikkuDB } from './kysely-tables.js';
 export declare class KyselyChannelStore extends ChannelStore {
     private db;

package/dist/src/kysely-deployment-service.d.ts

@@ -1,5 +1,5 @@
 import type { DeploymentService, DeploymentServiceConfig, DeploymentConfig, DeploymentInfo } from '@pikku/core/services';
-import { Kysely } from 'kysely';
+import type { Kysely } from 'kysely';
 import type { KyselyPikkuDB } from './kysely-tables.js';
 export declare class KyselyDeploymentService implements DeploymentService {
     private db;

package/dist/src/kysely-eventhub-store.d.ts

@@ -1,5 +1,5 @@
 import { EventHubStore } from '@pikku/core/channel';
-import { Kysely } from 'kysely';
+import type { Kysely } from 'kysely';
 import type { KyselyPikkuDB } from './kysely-tables.js';
 export declare class KyselyEventHubStore extends EventHubStore {
     private db;

package/dist/src/kysely-workflow-run-service.d.ts

@@ -1,5 +1,5 @@
 import type { WorkflowRun, StepState, WorkflowRunService } from '@pikku/core/workflow';
-import { Kysely } from 'kysely';
+import type { Kysely } from 'kysely';
 import type { KyselyPikkuDB } from './kysely-tables.js';
 export declare class KyselyWorkflowRunService implements WorkflowRunService {
     private db;

package/dist/src/kysely-workflow-service.d.ts

@@ -1,6 +1,6 @@
 import type { SerializedError } from '@pikku/core';
 import { PikkuWorkflowService, type WorkflowRun, type WorkflowRunWire, type StepState, type WorkflowStatus } from '@pikku/core/workflow';
-import { Kysely } from 'kysely';
+import type { Kysely } from 'kysely';
 import type { KyselyPikkuDB } from './kysely-tables.js';
 export declare class KyselyWorkflowService extends PikkuWorkflowService {
     private db;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pikku/kysely",
-  "version": "0.12.0",
+  "version": "0.12.2",
   "author": "yasser.fadl@gmail.com",
   "license": "MIT",
   "module": "dist/src/index.js",
@@ -19,7 +19,7 @@
     "release": "npm run build && npm test"
   },
   "peerDependencies": {
-    "@pikku/core": "^0.12.0"
+    "@pikku/core": "^0.12.2"
   },
   "dependencies": {
     "kysely": "^0.28.11",
@@ -30,7 +30,7 @@
     "@types/better-sqlite3": "^7.6.13",
     "better-sqlite3": "^12.6.2",
     "kysely": "^0.28.11",
-    "kysely-codegen": "^0.19.0",
+    "kysely-codegen": "^0.20.0",
     "kysely-plugin-serialize": "^0.8.2",
     "kysely-postgres-js": "^3.0.0",
     "postgres": "^3.4.8",

package/src/kysely-agent-run-service.ts

@@ -4,7 +4,7 @@ import type {
   AgentRunRow,
   AgentRunService,
 } from '@pikku/core/ai-agent'
-import { Kysely } from 'kysely'
+import type { Kysely } from 'kysely'
 import type { KyselyPikkuDB } from './kysely-tables.js'
 import { parseJson } from './kysely-json.js'
 

package/src/kysely-ai-storage-service.ts

@@ -4,7 +4,8 @@ import type {
   CreateRunInput,
 } from '@pikku/core/services'
 import type { AIThread, AIMessage, AgentRunState } from '@pikku/core/ai-agent'
-import { Kysely, sql } from 'kysely'
+import type { Kysely } from 'kysely'
+import { sql } from 'kysely'
 import type { KyselyPikkuDB } from './kysely-tables.js'
 import { parseJson } from './kysely-json.js'
 
@@ -15,6 +16,18 @@ export class KyselyAIStorageService
 
   constructor(private db: Kysely<KyselyPikkuDB>) {}
 
+  private async createIndexSafe(builder: {
+    execute(): Promise<void>
+  }): Promise<void> {
+    try {
+      await builder.execute()
+    } catch (e: any) {
+      // Ignore "index already exists" errors (MySQL doesn't support IF NOT EXISTS for indexes)
+      if (e?.code === 'ER_DUP_KEYNAME' || e?.errno === 1061) return
+      throw e
+    }
+  }
+
   public async init(): Promise<void> {
     if (this.initialized) {
       return
@@ -23,8 +36,8 @@ export class KyselyAIStorageService
     await this.db.schema
       .createTable('ai_threads')
       .ifNotExists()
-      .addColumn('id', 'text', (col) => col.primaryKey())
-      .addColumn('resource_id', 'text', (col) => col.notNull())
+      .addColumn('id', 'varchar(36)', (col) => col.primaryKey())
+      .addColumn('resource_id', 'varchar(255)', (col) => col.notNull())
       .addColumn('title', 'text')
       .addColumn('metadata', 'text')
       .addColumn('created_at', 'timestamp', (col) =>
@@ -35,77 +48,77 @@ export class KyselyAIStorageService
       )
       .execute()
 
-    await this.db.schema
-      .createIndex('idx_ai_threads_resource')
-      .ifNotExists()
-      .on('ai_threads')
-      .column('resource_id')
-      .execute()
+    await this.createIndexSafe(
+      this.db.schema
+        .createIndex('idx_ai_threads_resource')
+        .on('ai_threads')
+        .column('resource_id')
+    )
 
     await this.db.schema
       .createTable('ai_message')
       .ifNotExists()
-      .addColumn('id', 'text', (col) => col.primaryKey())
-      .addColumn('thread_id', 'text', (col) =>
+      .addColumn('id', 'varchar(36)', (col) => col.primaryKey())
+      .addColumn('thread_id', 'varchar(36)', (col) =>
         col.notNull().references('ai_threads.id').onDelete('cascade')
       )
-      .addColumn('role', 'text', (col) => col.notNull())
+      .addColumn('role', 'varchar(50)', (col) => col.notNull())
       .addColumn('content', 'text')
       .addColumn('created_at', 'timestamp', (col) =>
         col.defaultTo(sql`CURRENT_TIMESTAMP`).notNull()
       )
       .execute()
 
-    await this.db.schema
-      .createIndex('idx_ai_message_thread')
-      .ifNotExists()
-      .on('ai_message')
-      .columns(['thread_id', 'created_at'])
-      .execute()
+    await this.createIndexSafe(
+      this.db.schema
+        .createIndex('idx_ai_message_thread')
+        .on('ai_message')
+        .columns(['thread_id', 'created_at'])
+    )
 
     await this.db.schema
       .createTable('ai_tool_call')
       .ifNotExists()
-      .addColumn('id', 'text', (col) => col.primaryKey())
-      .addColumn('thread_id', 'text', (col) =>
+      .addColumn('id', 'varchar(36)', (col) => col.primaryKey())
+      .addColumn('thread_id', 'varchar(36)', (col) =>
         col.notNull().references('ai_threads.id').onDelete('cascade')
       )
-      .addColumn('message_id', 'text', (col) =>
+      .addColumn('message_id', 'varchar(36)', (col) =>
         col.notNull().references('ai_message.id').onDelete('cascade')
       )
-      .addColumn('run_id', 'text')
-      .addColumn('tool_name', 'text', (col) => col.notNull())
-      .addColumn('args', 'text', (col) => col.notNull().defaultTo('{}'))
+      .addColumn('run_id', 'varchar(36)')
+      .addColumn('tool_name', 'varchar(255)', (col) => col.notNull())
+      .addColumn('args', 'text', (col) => col.notNull())
       .addColumn('result', 'text')
-      .addColumn('approval_status', 'text')
-      .addColumn('approval_type', 'text')
-      .addColumn('agent_run_id', 'text')
-      .addColumn('display_tool_name', 'text')
+      .addColumn('approval_status', 'varchar(50)')
+      .addColumn('approval_type', 'varchar(50)')
+      .addColumn('agent_run_id', 'varchar(36)')
+      .addColumn('display_tool_name', 'varchar(255)')
       .addColumn('display_args', 'text')
       .addColumn('created_at', 'timestamp', (col) =>
         col.defaultTo(sql`CURRENT_TIMESTAMP`).notNull()
       )
       .execute()
 
-    await this.db.schema
-      .createIndex('idx_ai_tool_call_thread')
-      .ifNotExists()
-      .on('ai_tool_call')
-      .column('thread_id')
-      .execute()
+    await this.createIndexSafe(
+      this.db.schema
+        .createIndex('idx_ai_tool_call_thread')
+        .on('ai_tool_call')
+        .column('thread_id')
+    )
 
-    await this.db.schema
-      .createIndex('idx_ai_tool_call_message')
-      .ifNotExists()
-      .on('ai_tool_call')
-      .column('message_id')
-      .execute()
+    await this.createIndexSafe(
+      this.db.schema
+        .createIndex('idx_ai_tool_call_message')
+        .on('ai_tool_call')
+        .column('message_id')
+    )
 
     await this.db.schema
       .createTable('ai_working_memory')
       .ifNotExists()
-      .addColumn('id', 'text', (col) => col.notNull())
-      .addColumn('scope', 'text', (col) => col.notNull())
+      .addColumn('id', 'varchar(255)', (col) => col.notNull())
+      .addColumn('scope', 'varchar(50)', (col) => col.notNull())
       .addColumn('data', 'text', (col) => col.notNull())
       .addColumn('updated_at', 'timestamp', (col) =>
         col.defaultTo(sql`CURRENT_TIMESTAMP`).notNull()
@@ -116,13 +129,15 @@ export class KyselyAIStorageService
     await this.db.schema
       .createTable('ai_run')
       .ifNotExists()
-      .addColumn('run_id', 'text', (col) => col.primaryKey())
-      .addColumn('agent_name', 'text', (col) => col.notNull())
-      .addColumn('thread_id', 'text', (col) =>
+      .addColumn('run_id', 'varchar(36)', (col) => col.primaryKey())
+      .addColumn('agent_name', 'varchar(255)', (col) => col.notNull())
+      .addColumn('thread_id', 'varchar(36)', (col) =>
         col.notNull().references('ai_threads.id').onDelete('cascade')
       )
-      .addColumn('resource_id', 'text', (col) => col.notNull())
-      .addColumn('status', 'text', (col) => col.notNull().defaultTo('running'))
+      .addColumn('resource_id', 'varchar(255)', (col) => col.notNull())
+      .addColumn('status', 'varchar(50)', (col) =>
+        col.notNull().defaultTo('running')
+      )
       .addColumn('suspend_reason', 'text')
       .addColumn('missing_rpcs', 'text')
       .addColumn('usage_input_tokens', 'integer', (col) =>
@@ -131,7 +146,9 @@ export class KyselyAIStorageService
       .addColumn('usage_output_tokens', 'integer', (col) =>
         col.notNull().defaultTo(0)
       )
-      .addColumn('usage_model', 'text', (col) => col.notNull().defaultTo(''))
+      .addColumn('usage_model', 'varchar(255)', (col) =>
+        col.notNull().defaultTo('')
+      )
       .addColumn('created_at', 'timestamp', (col) =>
         col.defaultTo(sql`CURRENT_TIMESTAMP`).notNull()
       )
@@ -140,12 +157,12 @@ export class KyselyAIStorageService
       )
       .execute()
 
-    await this.db.schema
-      .createIndex('idx_ai_run_thread')
-      .ifNotExists()
-      .on('ai_run')
-      .columns(['thread_id', 'created_at'])
-      .execute()
+    await this.createIndexSafe(
+      this.db.schema
+        .createIndex('idx_ai_run_thread')
+        .on('ai_run')
+        .columns(['thread_id', 'created_at'])
+    )
 
     this.initialized = true
   }
@@ -261,15 +278,16 @@ export class KyselyAIStorageService
       }
     }
 
+    let msgResult
     if (options?.cursor || options?.lastN) {
       const innerResult = await msgQuery
         .orderBy('created_at', 'desc')
         .limit(options?.lastN ?? 50)
         .execute()
       innerResult.reverse()
-      var msgResult = innerResult
+      msgResult = innerResult
     } else {
-      var msgResult = await msgQuery.orderBy('created_at', 'asc').execute()
+      msgResult = await msgQuery.orderBy('created_at', 'asc').execute()
     }
 
     const tcResult = await this.db
@@ -288,10 +306,24 @@ export class KyselyAIStorageService
 
     const messages: AIMessage[] = []
     for (const row of msgResult) {
+      const rawContent = row.content as string | undefined
+      let parsedContent: AIMessage['content'] = rawContent ?? undefined
+      if (rawContent) {
+        try {
+          const parsed = JSON.parse(rawContent)
+          if (Array.isArray(parsed)) {
+            parsedContent = parsed
+          } else if (typeof parsed === 'string') {
+            parsedContent = parsed
+          }
+        } catch {
+          // Not JSON, use raw string
+        }
+      }
       const msg: AIMessage = {
         id: row.id,
         role: row.role as AIMessage['role'],
-        content: row.content ?? undefined,
+        content: parsedContent,
         createdAt: new Date(row.created_at),
       }
 
@@ -340,7 +372,7 @@ export class KyselyAIStorageService
             id: msg.id,
             thread_id: threadId,
             role: msg.role,
-            content: msg.content ?? null,
+            content: msg.content != null ? JSON.stringify(msg.content) : null,
             created_at: msg.createdAt ?? new Date(),
           }))
         )

package/src/kysely-channel-store.ts

@@ -1,6 +1,8 @@
-import { CoreUserSession } from '@pikku/core'
-import { Channel, ChannelStore } from '@pikku/core/channel'
-import { Kysely, sql } from 'kysely'
+import type { CoreUserSession } from '@pikku/core'
+import type { Channel } from '@pikku/core/channel'
+import { ChannelStore } from '@pikku/core/channel'
+import type { Kysely } from 'kysely'
+import { sql } from 'kysely'
 import type { KyselyPikkuDB } from './kysely-tables.js'
 import { parseJson } from './kysely-json.js'
 

package/src/kysely-deployment-service.ts

@@ -5,7 +5,8 @@ import type {
   DeploymentInfo,
 } from '@pikku/core/services'
 import { getAllFunctionNames } from '@pikku/core/function'
-import { Kysely, sql } from 'kysely'
+import type { Kysely } from 'kysely'
+import { sql } from 'kysely'
 import type { KyselyPikkuDB } from './kysely-tables.js'
 
 export class KyselyDeploymentService implements DeploymentService {

package/src/kysely-eventhub-store.ts

@@ -1,5 +1,5 @@
 import { EventHubStore } from '@pikku/core/channel'
-import { Kysely } from 'kysely'
+import type { Kysely } from 'kysely'
 import type { KyselyPikkuDB } from './kysely-tables.js'
 
 export class KyselyEventHubStore extends EventHubStore {

package/src/kysely-workflow-run-service.ts

@@ -4,7 +4,7 @@ import type {
   WorkflowStatus,
   WorkflowRunService,
 } from '@pikku/core/workflow'
-import { Kysely } from 'kysely'
+import type { Kysely } from 'kysely'
 import type { KyselyPikkuDB } from './kysely-tables.js'
 import { parseJson } from './kysely-json.js'
 

package/src/kysely-workflow-service.ts

@@ -6,7 +6,8 @@ import {
   type StepState,
   type WorkflowStatus,
 } from '@pikku/core/workflow'
-import { Kysely, sql } from 'kysely'
+import type { Kysely } from 'kysely'
+import { sql } from 'kysely'
 import type { KyselyPikkuDB } from './kysely-tables.js'
 import { KyselyWorkflowRunService } from './kysely-workflow-run-service.js'
 import { parseJson } from './kysely-json.js'