profclaw 0.0.1-beta.1 → 0.0.1-beta.2

package/dist/routes/users.js

@@ -4,6 +4,8 @@
  * Handles user profile and preferences management.
  */
 import { Hono } from 'hono';
+import { createContextualLogger } from '../utils/logger.js';
+const log = createContextualLogger('Users');
 import { getCookie } from 'hono/cookie';
 import { eq } from 'drizzle-orm';
 import { randomUUID, randomBytes, createHash } from 'crypto';
@@ -642,7 +644,7 @@ userRoutes.post('/me/recovery-codes/regenerate', async (c) => {
         });
     }
     catch (error) {
-        console.error('[Users] Recovery code regeneration error:', error);
+        log.error('Recovery code regeneration error', error instanceof Error ? error : new Error(String(error)));
         return c.json({ error: 'Failed to regenerate recovery codes' }, 500);
     }
 });
@@ -672,7 +674,7 @@ userRoutes.get('/me/recovery-codes/count', async (c) => {
         });
     }
     catch (error) {
-        console.error('[Users] Recovery code count error:', error);
+        log.error('Recovery code count error', error instanceof Error ? error : new Error(String(error)));
         return c.json({ error: 'Failed to get recovery code count' }, 500);
     }
 });
@@ -713,7 +715,7 @@ userRoutes.get('/admin/list', requireAdmin, async (c) => {
         return c.json({ users: allUsers, total: allUsers.length });
     }
     catch (error) {
-        console.error('[Admin] List users error:', error);
+        log.error('List users error', error instanceof Error ? error : new Error(String(error)));
         return c.json({ error: 'Failed to list users' }, 500);
     }
 });
@@ -756,7 +758,7 @@ userRoutes.post('/admin/invites', requireAdmin, async (c) => {
         });
     }
     catch (err) {
-        console.error('[Admin] Generate invites error:', err);
+        log.error('Generate invites error', err instanceof Error ? err : new Error(String(err)));
         return c.json({ error: 'Failed to generate invite codes' }, 500);
     }
 });
@@ -784,7 +786,7 @@ userRoutes.get('/admin/invites', requireAdmin, async (c) => {
         return c.json({ invites: allInvites, total: allInvites.length });
     }
     catch (err) {
-        console.error('[Admin] List invites error:', err);
+        log.error('List invites error', err instanceof Error ? err : new Error(String(err)));
         return c.json({ error: 'Failed to list invite codes' }, 500);
     }
 });
@@ -810,7 +812,7 @@ userRoutes.delete('/admin/invites/:id', requireAdmin, async (c) => {
         return c.json({ message: 'Invite code deleted' });
     }
     catch (err) {
-        console.error('[Admin] Delete invite error:', err);
+        log.error('Delete invite error', err instanceof Error ? err : new Error(String(err)));
         return c.json({ error: 'Failed to delete invite code' }, 500);
     }
 });
@@ -825,7 +827,7 @@ userRoutes.get('/admin/registration-mode', requireAdmin, async (c) => {
         return c.json({ mode: settings.system.registrationMode });
     }
     catch (err) {
-        console.error('[Admin] Get registration mode error:', err);
+        log.error('Get registration mode error', err instanceof Error ? err : new Error(String(err)));
         return c.json({ error: 'Failed to get registration mode' }, 500);
     }
 });
@@ -846,7 +848,7 @@ userRoutes.patch('/admin/registration-mode', requireAdmin, async (c) => {
         return c.json({ mode, message: `Registration mode set to ${mode}` });
     }
     catch (err) {
-        console.error('[Admin] Set registration mode error:', err);
+        log.error('Set registration mode error', err instanceof Error ? err : new Error(String(err)));
         return c.json({ error: 'Failed to set registration mode' }, 500);
     }
 });
@@ -887,7 +889,7 @@ userRoutes.get('/admin/:userId', requireAdmin, async (c) => {
         return c.json({ user: result[0], activeSessions: userSessions.length });
     }
     catch (error) {
-        console.error('[Admin] Get user error:', error);
+        log.error('Get user error', error instanceof Error ? error : new Error(String(error)));
         return c.json({ error: 'Failed to get user' }, 500);
     }
 });
@@ -923,7 +925,7 @@ userRoutes.patch('/admin/:userId', requireAdmin, async (c) => {
         return c.json({ message: 'User updated' });
     }
     catch (error) {
-        console.error('[Admin] Update user error:', error);
+        log.error('Update user error', error instanceof Error ? error : new Error(String(error)));
         return c.json({ error: 'Failed to update user' }, 500);
     }
 });
@@ -961,7 +963,7 @@ userRoutes.post('/admin/:userId/reset-password', requireAdmin, async (c) => {
         });
     }
     catch (error) {
-        console.error('[Admin] Reset password error:', error);
+        log.error('Reset password error', error instanceof Error ? error : new Error(String(error)));
         return c.json({ error: 'Failed to reset password' }, 500);
     }
 });
@@ -991,7 +993,7 @@ userRoutes.delete('/admin/:userId', requireAdmin, async (c) => {
         return c.json({ message: 'User deleted' });
     }
     catch (error) {
-        console.error('[Admin] Delete user error:', error);
+        log.error('Delete user error', error instanceof Error ? error : new Error(String(error)));
         return c.json({ error: 'Failed to delete user' }, 500);
     }
 });

package/dist/routes/webhooks.js

@@ -1,4 +1,6 @@
 import { Hono } from 'hono';
+import { createContextualLogger } from '../utils/logger.js';
+const log = createContextualLogger('Webhooks');
 import { handleGitHubWebhook } from '../integrations/github.js';
 import { handleJiraWebhook } from '../integrations/jira.js';
 import { handleLinearWebhook } from '../integrations/linear.js';
@@ -18,7 +20,7 @@ webhooks.post('/github', async (c) => {
         });
     }
     catch (error) {
-        console.error('[Webhook] GitHub error:', error);
+        log.error('GitHub webhook error', error instanceof Error ? error : new Error(String(error)));
         return c.json({
             error: 'Webhook processing failed',
             message: error instanceof Error ? error.message : 'Unknown error',
@@ -39,7 +41,7 @@ webhooks.post('/jira', async (c) => {
         });
     }
     catch (error) {
-        console.error('[Webhook] Jira error:', error);
+        log.error('Jira webhook error', error instanceof Error ? error : new Error(String(error)));
         return c.json({
             error: 'Webhook processing failed',
             message: error instanceof Error ? error.message : 'Unknown error',
@@ -60,7 +62,7 @@ webhooks.post('/linear', async (c) => {
         });
     }
     catch (error) {
-        console.error('[Webhook] Linear error:', error);
+        log.error('Linear webhook error', error instanceof Error ? error : new Error(String(error)));
         return c.json({
             error: 'Webhook processing failed',
             message: error instanceof Error ? error.message : 'Unknown error',

package/dist/settings/index.js

@@ -1,5 +1,7 @@
 import { z } from 'zod';
 import { getStorage } from '../storage/index.js';
+import { createContextualLogger } from '../utils/logger.js';
+const log = createContextualLogger('Settings');
 // Plugin status enum
 export const PluginStatusSchema = z.enum(['enabled', 'disabled', 'error', 'checking']);
 // Settings schema with categories
@@ -199,7 +201,7 @@ export async function getSettings() {
         return maskSecrets(settingsCache);
     }
     catch (error) {
-        console.warn('[Settings] Failed to load settings, using defaults:', error);
+        log.warn('Failed to load settings, using defaults', { error: error instanceof Error ? error.message : String(error) });
         settingsCache = { ...DEFAULT_SETTINGS };
         return maskSecrets(settingsCache);
     }

package/dist/storage/index.d.ts

@@ -1,6 +1,7 @@
 import type { StorageAdapter } from "./adapter.js";
 import type { LibSQLDatabase } from "drizzle-orm/libsql";
 import type { Client } from "@libsql/client";
+import * as schema from "./schema.js";
 /**
  * Initialize storage based on configuration
  */
@@ -12,7 +13,7 @@ export declare function getStorage(): StorageAdapter;
 /**
  * Get the raw drizzle database instance for advanced queries
  */
-export declare function getDb(): LibSQLDatabase<Record<string, unknown>>;
+export declare function getDb(): LibSQLDatabase<typeof schema>;
 /**
  * Get the raw libsql client for executing raw SQL
  */

package/dist/storage/index.js

@@ -1,5 +1,7 @@
 import { LibSQLAdapter } from "./libsql.js";
 import { loadConfig } from "../utils/config-loader.js";
+import { createContextualLogger } from "../utils/logger.js";
+const log = createContextualLogger('Storage');
 let storage = null;
 /**
  * Initialize storage based on configuration
@@ -27,10 +29,10 @@ export async function initStorage() {
     if (tier === "local") {
         const dbPath = dbPathOverride || process.env.DB_PATH || settings.storage?.dbPath;
         storage = new LibSQLAdapter({ dbPath });
-        console.log(`[Storage] Initializing LibSQL storage at ${dbPath || "default path"}`);
+        log.info('Initializing LibSQL storage', { dbPath: dbPath || 'default path' });
     }
     else {
-        console.log("[Storage] Initializing in-memory storage (LibSQL in-memory)");
+        log.info('Initializing in-memory storage (LibSQL in-memory)');
         storage = new LibSQLAdapter({ dbPath: ":memory:" });
     }
     await storage.connect();
@@ -52,7 +54,6 @@ export function getDb() {
     if (!storage) {
         throw new Error("Storage not initialized. Call initStorage() first.");
     }
-    // Access the internal db property via the LibSQLAdapter
     return storage.getDb();
 }
 /**
@@ -113,8 +114,12 @@ export async function loadAllProviderConfigs() {
     });
     return result.rows
         .map((row) => {
+        const value = row.value;
+        if (typeof value !== "string") {
+            return null;
+        }
         try {
-            return JSON.parse(row.value);
+            return JSON.parse(value);
         }
         catch {
             return null;

package/dist/storage/libsql.js

@@ -14,6 +14,9 @@ function getNumber(value, key) {
     const candidate = value[key];
     return typeof candidate === "number" ? candidate : 0;
 }
+function toInValues(params) {
+    return (params ?? []);
+}
 function toSummary(row) {
     return {
         id: row.id,
@@ -1032,13 +1035,13 @@ export class LibSQLAdapter {
     }
     // === Tasks ===
     async createTask(input) {
-        const inputWithExtras = input;
-        const id = inputWithExtras.id || randomUUID();
+        const id = ('id' in input && typeof input.id === 'string') ? input.id : randomUUID();
+        const status = ('status' in input && typeof input.status === 'string') ? input.status : 'pending';
         const now = new Date();
         const newTask = {
             ...input,
             id,
-            status: inputWithExtras.status || "pending",
+            status: status,
             createdAt: now,
             updatedAt: now,
             attempts: 0,
@@ -1123,7 +1126,11 @@ export class LibSQLAdapter {
                 createdAt: now,
             });
         }
-        return { ...newSummary, createdAt: now, rawOutput: undefined };
+        return toSummary({
+            ...newSummary,
+            createdAt: now,
+            rawOutput: null,
+        });
     }
     async getSummary(id, includeRawOutput = false) {
         const results = await this.db
@@ -1141,14 +1148,14 @@ export class LibSQLAdapter {
                 .where(eq(schema.summaryBlobs.summaryId, id))
                 .limit(1);
             if (blob[0]) {
-                return { ...summary, rawOutput: blob[0].rawOutput };
+                return { ...toSummary(summary), rawOutput: blob[0].rawOutput };
             }
         }
         // Strip rawOutput from response if not requested (for backward compat with old data)
         if (!includeRawOutput) {
-            return { ...summary, rawOutput: undefined };
+            return { ...toSummary(summary), rawOutput: undefined };
         }
-        return summary;
+        return toSummary(summary);
     }
     /**
      * Get raw output for a summary (lazy loading)
@@ -1172,11 +1179,12 @@ export class LibSQLAdapter {
         return summary[0]?.rawOutput || null;
     }
     async getTaskSummaries(taskId) {
-        return await this.db
+        const summaries = await this.db
             .select()
             .from(schema.summaries)
             .where(eq(schema.summaries.taskId, taskId))
             .orderBy(desc(schema.summaries.createdAt));
+        return summaries.map(toSummary);
     }
     async querySummaries(query) {
         const conditions = [];
@@ -1238,7 +1246,7 @@ export class LibSQLAdapter {
             .from(schema.summaries)
             .where(whereClause);
         return {
-            summaries: summaries,
+            summaries: summaries.map(toSummary),
             total: totalRes[0].value,
         };
     }
@@ -1339,7 +1347,6 @@ export class LibSQLAdapter {
                     completed: 0,
                     failed: 0,
                     avgDuration: 0,
-                    lastActivity: null,
                 };
             }
             if (row.status === "completed") {
@@ -1349,11 +1356,12 @@ export class LibSQLAdapter {
             else if (row.status === "failed") {
                 stats[row.agent].failed = row.count;
             }
-            if (row.lastActivity) {
+            const agentStats = stats[row.agent];
+            if (row.lastActivity !== null) {
                 const lastActivity = new Date(row.lastActivity);
-                if (!stats[row.agent].lastActivity ||
-                    lastActivity > stats[row.agent].lastActivity) {
-                    stats[row.agent].lastActivity = lastActivity;
+                if (!agentStats.lastActivity ||
+                    lastActivity > agentStats.lastActivity) {
+                    agentStats.lastActivity = lastActivity;
                 }
             }
         }
@@ -1390,10 +1398,8 @@ export class LibSQLAdapter {
             const date = s.createdAt.toISOString().split("T")[0];
             const model = s.model || "unknown";
             const agent = s.agent || "unknown";
-            const costObj = s.cost;
-            const tokensObj = s.tokensUsed;
-            const cost = costObj?.amount || 0;
-            const tokens = tokensObj?.total || 0;
+            const cost = getNumber(s.cost, "amount");
+            const tokens = getNumber(s.tokensUsed, "total");
             accumulate(date, model, agent, cost, tokens);
         }
         // Source 2: Conversation messages (chat/agentic sessions)
@@ -1451,7 +1457,8 @@ export class LibSQLAdapter {
             return [];
         const targetVector = new Float32Array(embedding);
         const results = allEmbeddings.map((row) => {
-            const rowVector = new Float32Array(row.embedding.buffer, row.embedding.byteOffset, row.embedding.byteLength / 4);
+            const embeddingBuffer = row.embedding;
+            const rowVector = new Float32Array(embeddingBuffer.buffer, embeddingBuffer.byteOffset, embeddingBuffer.byteLength / 4);
             const distance = this.cosineSimilarity(targetVector, rowVector);
             return {
                 entityId: row.entityId,
@@ -1480,14 +1487,14 @@ export class LibSQLAdapter {
     async query(sql, params) {
         const result = await this.client.execute({
             sql,
-            args: params || [],
+            args: toInValues(params),
         });
         return result.rows;
     }
     async execute(sql, params) {
         await this.client.execute({
             sql,
-            args: params || [],
+            args: toInValues(params),
         });
     }
     // === Task Events (Audit Log) ===
@@ -1837,6 +1844,7 @@ export class LibSQLAdapter {
         const totalRes = await this.db
             .select({ value: count() })
             .from(schema.taskArchives);
+        // Map archived tasks back to Task format
         const tasks = archived.map((a) => ({
             id: a.originalId,
             title: a.title,