@kafca/agentdock 0.1.23 → 0.1.24

package/dist-electron/electron/workspace-task-store.test.js

@@ -9,6 +9,7 @@ const node_fs_1 = require("node:fs");
 const node_path_1 = require("node:path");
 const node_os_1 = require("node:os");
 const local_core_acp_store_js_1 = require("../services/local-ai-core/src/acp/local-core-acp-store.js");
+const scheduler_service_js_1 = require("../services/local-ai-core/src/scheduler/scheduler-service.js");
 (0, node_test_1.default)('workspace registry entries persist in LocalCoreAcpStore', () => {
     const userDataPath = (0, node_fs_1.mkdtempSync)((0, node_path_1.join)((0, node_os_1.tmpdir)(), 'workspace-registry-'));
     try {
@@ -34,6 +35,86 @@ const local_core_acp_store_js_1 = require("../services/local-ai-core/src/acp/loc
         (0, node_fs_1.rmSync)(userDataPath, { recursive: true, force: true });
     }
 });
+(0, node_test_1.default)('scheduler uses short ids for new jobs and resolves legacy full ids by short id', () => {
+    const userDataPath = (0, node_fs_1.mkdtempSync)((0, node_path_1.join)((0, node_os_1.tmpdir)(), 'scheduler-id-store-'));
+    try {
+        const store = new local_core_acp_store_js_1.LocalCoreAcpStore(userDataPath);
+        const scheduler = new scheduler_service_js_1.SchedulerService({
+            store,
+            triggers: [],
+            executors: [],
+            eventBus: { emit: () => { }, on: () => () => { } },
+        });
+        const created = scheduler.createJob({
+            workspaceId: 'workspace-a',
+            platform: 'local',
+            route: { type: 'local.thread', channelId: 'workspace-a', threadId: 'thread-1' },
+            executionMode: 'side-thread',
+            triggerType: 'cron',
+            cronExpr: '30 18 * * *',
+            promptTemplate: 'ping',
+            description: 'daily ping',
+            enabled: true,
+        });
+        strict_1.default.match(created.id, /^[0-9a-f]{8}$/);
+        const legacy = store.createScheduledJob({
+            workspaceId: 'workspace-a',
+            platform: 'local',
+            route: { type: 'local.thread', channelId: 'workspace-a', threadId: 'thread-1' },
+            executionMode: 'side-thread',
+            triggerType: 'cron',
+            cronExpr: '0 9 * * *',
+            promptTemplate: 'legacy',
+            description: 'legacy job',
+            enabled: true,
+        });
+        store.db.prepare('UPDATE scheduled_jobs SET id = ? WHERE id = ?').run('job:826aff79-570b-4308-822e-18318e2c96ba', legacy.id);
+        strict_1.default.equal(scheduler.getJob('826aff79')?.id, 'job:826aff79-570b-4308-822e-18318e2c96ba');
+        scheduler.updateJob('826aff79', { description: 'updated legacy job' });
+        strict_1.default.equal(scheduler.getJob('826aff79')?.description, 'updated legacy job');
+        scheduler.deleteJob('826aff79');
+        strict_1.default.equal(scheduler.getJob('826aff79'), undefined);
+        strict_1.default.throws(() => scheduler.deleteJob('826aff79'), /Scheduled job not found: 826aff79/);
+        store.close();
+    }
+    finally {
+        (0, node_fs_1.rmSync)(userDataPath, { recursive: true, force: true });
+    }
+});
+(0, node_test_1.default)('scheduled jobs normalize enum-like input before persistence', () => {
+    const userDataPath = (0, node_fs_1.mkdtempSync)((0, node_path_1.join)((0, node_os_1.tmpdir)(), 'scheduler-enum-store-'));
+    try {
+        const store = new local_core_acp_store_js_1.LocalCoreAcpStore(userDataPath);
+        const created = store.createScheduledJob({
+            workspaceId: 'workspace-a',
+            platform: ' Lark ',
+            route: { type: 'channel.chat', channelId: 'chat-1', participantId: 'user-1' },
+            executionMode: 'side_thread',
+            triggerType: 'CRON',
+            cronExpr: '30 18 * * *',
+            promptTemplate: 'ping',
+            description: 'daily ping',
+            enabled: true,
+        });
+        strict_1.default.equal(created.platform, 'lark');
+        strict_1.default.equal(created.executionMode, 'side-thread');
+        strict_1.default.equal(created.triggerType, 'cron');
+        const updated = store.updateScheduledJob(created.id, {
+            executionMode: 'same_thread',
+            triggerType: 'one time',
+            runAt: '2026-05-04T10:00:00.000Z',
+            cronExpr: '',
+        });
+        strict_1.default.equal(updated.executionMode, 'same-thread');
+        strict_1.default.equal(updated.triggerType, 'once');
+        strict_1.default.equal(updated.runAt, '2026-05-04T10:00:00.000Z');
+        strict_1.default.equal(updated.cronExpr, undefined);
+        store.close();
+    }
+    finally {
+        (0, node_fs_1.rmSync)(userDataPath, { recursive: true, force: true });
+    }
+});
 (0, node_test_1.default)('agent tasks persist, update status, and can be found by run id', () => {
     const userDataPath = (0, node_fs_1.mkdtempSync)((0, node_path_1.join)((0, node_os_1.tmpdir)(), 'agent-task-store-'));
     try {
@@ -64,3 +145,46 @@ const local_core_acp_store_js_1 = require("../services/local-ai-core/src/acp/loc
         (0, node_fs_1.rmSync)(userDataPath, { recursive: true, force: true });
     }
 });
+(0, node_test_1.default)('agent task and run statuses normalize before persistence', () => {
+    const userDataPath = (0, node_fs_1.mkdtempSync)((0, node_path_1.join)((0, node_os_1.tmpdir)(), 'task-run-status-store-'));
+    try {
+        const store = new local_core_acp_store_js_1.LocalCoreAcpStore(userDataPath);
+        const task = store.createAgentTask({
+            workspaceId: 'workspace-a',
+            deviceId: 'device-a',
+            runtimeId: 'runtime-a',
+            title: 'Normalize states',
+            status: 'waiting for user',
+        });
+        strict_1.default.equal(task.status, 'waiting_for_user');
+        strict_1.default.equal(store.listAgentTasks({ status: 'waiting for user' }).tasks.length, 1);
+        const updated = store.updateAgentTask(task.taskId, { status: 'canceled' });
+        strict_1.default.equal(updated.status, 'cancelled');
+        strict_1.default.equal(updated.timeline.at(-1)?.status, 'cancelled');
+        const thread = store.createThread('workspace-a', 'Thread');
+        store.updateRun('run-1', thread.id, 'awaiting input');
+        strict_1.default.equal(store.getRun('run-1')?.status, 'awaiting_input');
+        store.updateRun('run-1', thread.id, 'canceled');
+        strict_1.default.equal(store.getRun('run-1')?.status, 'interrupted');
+        const job = store.createScheduledJob({
+            workspaceId: 'workspace-a',
+            platform: 'local',
+            route: { type: 'local.thread', channelId: 'workspace-a', threadId: 'thread-1' },
+            executionMode: 'same-thread',
+            triggerType: 'cron',
+            cronExpr: '0 9 * * *',
+            promptTemplate: 'ping',
+            description: 'daily ping',
+            enabled: true,
+        });
+        const run = store.createScheduledJobRun(job.id, 'complete');
+        strict_1.default.equal(run.status, 'succeeded');
+        const skipped = store.updateScheduledJobRun(run.id, { status: 'cancelled' });
+        strict_1.default.equal(skipped.status, 'skipped');
+        strict_1.default.equal(store.getScheduledJob(job.id)?.lastStatus, 'skipped');
+        store.close();
+    }
+    finally {
+        (0, node_fs_1.rmSync)(userDataPath, { recursive: true, force: true });
+    }
+});

package/dist-electron/packages/contracts/src/local-core.js

@@ -1,2 +1,112 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
+exports.normalizeApprovalRequestStatus = normalizeApprovalRequestStatus;
+exports.normalizeAgentTaskStatus = normalizeAgentTaskStatus;
+exports.normalizeChannelContentPartType = normalizeChannelContentPartType;
+exports.normalizeScheduledJobExecutionMode = normalizeScheduledJobExecutionMode;
+exports.normalizeScheduledJobTriggerType = normalizeScheduledJobTriggerType;
+exports.normalizeChannelPlatform = normalizeChannelPlatform;
+exports.normalizeRunStatus = normalizeRunStatus;
+exports.normalizeScheduledJobRunStatus = normalizeScheduledJobRunStatus;
+function normalizeApprovalRequestStatus(value, fallback = 'pending') {
+    const normalized = normalizeContractEnumValue(value || fallback).replace(/-/g, '_');
+    if (normalized === 'pending' ||
+        normalized === 'approved' ||
+        normalized === 'rejected' ||
+        normalized === 'cancelled' ||
+        normalized === 'expired') {
+        return normalized;
+    }
+    if (normalized === 'approve') {
+        return 'approved';
+    }
+    if (normalized === 'reject') {
+        return 'rejected';
+    }
+    if (normalized === 'canceled') {
+        return 'cancelled';
+    }
+    throw new Error('Approval request status must be pending, approved, rejected, cancelled, or expired.');
+}
+function normalizeAgentTaskStatus(value, fallback = 'created') {
+    const normalized = normalizeContractEnumValue(value || fallback).replace(/-/g, '_');
+    if (normalized === 'created' ||
+        normalized === 'queued' ||
+        normalized === 'running' ||
+        normalized === 'waiting_for_user' ||
+        normalized === 'completed' ||
+        normalized === 'failed' ||
+        normalized === 'cancelled') {
+        return normalized;
+    }
+    if (normalized === 'canceled') {
+        return 'cancelled';
+    }
+    throw new Error('Agent task status must be created, queued, running, waiting_for_user, completed, failed, or cancelled.');
+}
+function normalizeChannelContentPartType(value) {
+    const normalized = normalizeContractEnumValue(value).replace(/-/g, '_');
+    if (normalized === 'text' ||
+        normalized === 'image' ||
+        normalized === 'file' ||
+        normalized === 'permission_card') {
+        return normalized;
+    }
+    throw new Error('Channel content part type must be text, image, file, or permission_card.');
+}
+function normalizeContractEnumValue(value) {
+    return String(value || '').trim().toLowerCase().replace(/[\s_]+/g, '-');
+}
+function normalizeScheduledJobExecutionMode(value, fallback = 'same-thread') {
+    const normalized = normalizeContractEnumValue(value || fallback);
+    if (normalized === 'same-thread' || normalized === 'side-thread') {
+        return normalized;
+    }
+    throw new Error('Scheduled job execution mode must be same-thread or side-thread.');
+}
+function normalizeScheduledJobTriggerType(value, fallback = 'cron') {
+    const normalized = normalizeContractEnumValue(value || fallback);
+    if (normalized === 'cron' || normalized === 'once' || normalized === 'one-time') {
+        return normalized === 'one-time' ? 'once' : normalized;
+    }
+    throw new Error('Scheduled job trigger type must be cron or once.');
+}
+function normalizeChannelPlatform(value) {
+    const normalized = String(value || '').trim().toLowerCase();
+    if (!normalized) {
+        throw new Error('Channel platform is required.');
+    }
+    return normalized;
+}
+function normalizeRunStatus(value, fallback = 'queued') {
+    const normalized = normalizeContractEnumValue(value || fallback).replace(/-/g, '_');
+    if (normalized === 'queued' ||
+        normalized === 'running' ||
+        normalized === 'awaiting_input' ||
+        normalized === 'completed' ||
+        normalized === 'failed' ||
+        normalized === 'interrupted') {
+        return normalized;
+    }
+    if (normalized === 'cancelled' || normalized === 'canceled') {
+        return 'interrupted';
+    }
+    throw new Error('Run status must be queued, running, awaiting_input, completed, failed, or interrupted.');
+}
+function normalizeScheduledJobRunStatus(value, fallback = 'queued') {
+    const normalized = normalizeContractEnumValue(value || fallback).replace(/-/g, '_');
+    if (normalized === 'queued' ||
+        normalized === 'running' ||
+        normalized === 'succeeded' ||
+        normalized === 'failed' ||
+        normalized === 'skipped') {
+        return normalized;
+    }
+    if (normalized === 'complete' || normalized === 'completed' || normalized === 'success') {
+        return 'succeeded';
+    }
+    if (normalized === 'cancelled' || normalized === 'canceled') {
+        return 'skipped';
+    }
+    throw new Error('Scheduled job run status must be queued, running, succeeded, failed, or skipped.');
+}

package/dist-electron/services/local-ai-core/src/acp/local-core-acp-content.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.normalizeThreadMessageInput = normalizeThreadMessageInput;
 exports.createTextMessageInput = createTextMessageInput;
+const index_js_1 = require("../../../../packages/contracts/src/index.js");
 function normalizeThreadMessageInput(input) {
     if (typeof input === 'string') {
         return {
@@ -32,7 +33,13 @@ function isSupportedContentPart(part) {
     if (!part || typeof part !== 'object') {
         return false;
     }
-    const type = String(part.type || '');
+    let type;
+    try {
+        type = (0, index_js_1.normalizeChannelContentPartType)(part.type);
+    }
+    catch {
+        return false;
+    }
     if (type === 'text') {
         return Boolean(String(part.text || ''));
     }
@@ -40,6 +47,10 @@ function isSupportedContentPart(part) {
         const image = part;
         return Boolean(String(image.data || image.uri || ''));
     }
+    if (type === 'file') {
+        const file = part;
+        return Boolean(String(file.data || file.uri || file.path || ''));
+    }
     return false;
 }
 function summarizeContentParts(parts) {
@@ -52,9 +63,19 @@ function summarizeContentParts(parts) {
     if (text) {
         return text;
     }
-    const imageCount = parts.filter((part) => part.type === 'image').length;
-    if (imageCount > 0) {
-        return imageCount === 1 ? '[Image]' : `[${imageCount} images]`;
+    const summaries = parts
+        .map((part) => {
+        if (part.type === 'image') {
+            return part.fileName ? `[Image: ${part.fileName}]` : '[Image]';
+        }
+        if (part.type === 'file') {
+            return part.fileName ? `[File: ${part.fileName}]` : '[File]';
+        }
+        return '';
+    })
+        .filter(Boolean);
+    if (summaries.length > 0) {
+        return summaries.join('\n');
     }
     return '';
 }

package/dist-electron/services/local-ai-core/src/acp/local-core-acp-response-processor.js

@@ -2,6 +2,7 @@
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.LocalCoreAcpResponseProcessor = void 0;
 const cron_command_detector_js_1 = require("../scheduler/cron-command-detector.js");
+const job_id_js_1 = require("../scheduler/job-id.js");
 class LocalCoreAcpResponseProcessor {
     options;
     constructor(options) {
@@ -82,7 +83,7 @@ class LocalCoreAcpResponseProcessor {
                         scheduleDescription: command.scheduleDescription,
                         message: command.message,
                     });
-                    return `已创建定时任务：${job.description || command.name}，计划 ${command.scheduleDescription}（${command.schedule}），ID: ${job.id}`;
+                    return `已创建定时任务：${job.description || command.name}，计划 ${command.scheduleDescription}（${command.schedule}），ID: ${(0, job_id_js_1.toPublicScheduledJobId)(job.id)}`;
                 }
                 case 'list': {
                     const jobs = await this.options.scheduler.listJobsForThread(threadId);
@@ -91,7 +92,7 @@ class LocalCoreAcpResponseProcessor {
                     }
                     return [
                         '当前对话定时任务：',
-                        ...jobs.map((job) => `- ${job.description || job.id} | ${job.triggerType === 'cron' ? job.cronExpr : job.runAt} | ${job.enabled ? 'enabled' : 'disabled'} | ${job.id}`),
+                        ...jobs.map((job) => `- ${job.description || (0, job_id_js_1.toPublicScheduledJobId)(job.id)} | ${job.triggerType === 'cron' ? job.cronExpr : job.runAt} | ${job.enabled ? 'enabled' : 'disabled'} | ${(0, job_id_js_1.toPublicScheduledJobId)(job.id)}`),
                     ].join('\n');
                 }
                 case 'delete': {

package/dist-electron/services/local-ai-core/src/acp/local-core-acp-store.js

@@ -6,6 +6,8 @@ const node_crypto_1 = require("node:crypto");
 const node_fs_1 = require("node:fs");
 const node_sqlite_1 = require("node:sqlite");
 const node_path_1 = require("node:path");
+const index_js_1 = require("../../../../packages/contracts/src/index.js");
+const job_id_js_1 = require("../scheduler/job-id.js");
 const desktop_js_1 = require("../../../../shared/desktop.js");
 const workspace_thread_mappers_js_1 = require("../thread/workspace-thread-mappers.js");
 const workspace_thread_id_js_1 = require("../thread/workspace-thread-id.js");
@@ -378,31 +380,34 @@ class LocalCoreAcpStore {
     }
     updateRun(runId, threadId, status) {
         const now = new Date().toISOString();
+        const normalizedStatus = (0, index_js_1.normalizeRunStatus)(status);
         const existing = this.db.prepare('SELECT id FROM runs WHERE id = ?').get(runId);
         if (existing) {
-            this.db.prepare('UPDATE runs SET status = ?, updated_at = ? WHERE id = ?').run(status, now, runId);
+            this.db.prepare('UPDATE runs SET status = ?, updated_at = ? WHERE id = ?').run(normalizedStatus, now, runId);
             return;
         }
         this.db.prepare(`
       INSERT INTO runs (id, thread_id, status, started_at, updated_at)
       VALUES (?, ?, ?, ?, ?)
-    `).run(runId, threadId, status, now, now);
+    `).run(runId, threadId, normalizedStatus, now, now);
     }
     getLatestRunForThread(threadId) {
-        return this.db.prepare(`
+        const row = this.db.prepare(`
       SELECT id, thread_id, status, started_at, updated_at
       FROM runs
       WHERE thread_id = ?
       ORDER BY updated_at DESC
       LIMIT 1
     `).get(threadId);
+        return row ? { ...row, status: (0, index_js_1.normalizeRunStatus)(row.status) } : undefined;
     }
     getRun(runId) {
-        return this.db.prepare(`
+        const row = this.db.prepare(`
       SELECT id, thread_id, status, started_at, updated_at
       FROM runs
       WHERE id = ?
     `).get(runId);
+        return row ? { ...row, status: (0, index_js_1.normalizeRunStatus)(row.status) } : undefined;
     }
     listWorkspaceRegistry() {
         const rows = this.db.prepare(`
@@ -573,7 +578,7 @@ class LocalCoreAcpStore {
             params.push(query.taskId);
         }
         if (query.status) {
-            const statuses = Array.isArray(query.status) ? query.status : [query.status];
+            const statuses = (Array.isArray(query.status) ? query.status : [query.status]).map((status) => (0, index_js_1.normalizeApprovalRequestStatus)(status));
             predicates.push(`status IN (${statuses.map(() => '?').join(', ')})`);
             params.push(...statuses);
         }
@@ -669,7 +674,7 @@ class LocalCoreAcpStore {
     createAgentTask(input) {
         const id = `task:${(0, node_crypto_1.randomUUID)()}`;
         const now = new Date().toISOString();
-        const status = input.status || 'created';
+        const status = (0, index_js_1.normalizeAgentTaskStatus)(input.status);
         const timeline = [{
                 id: `timeline:${(0, node_crypto_1.randomUUID)()}`,
                 type: 'status_change',
@@ -706,7 +711,7 @@ class LocalCoreAcpStore {
             params.push(query.runtimeId);
         }
         if (query.status) {
-            const statuses = Array.isArray(query.status) ? query.status : [query.status];
+            const statuses = (Array.isArray(query.status) ? query.status : [query.status]).map((status) => (0, index_js_1.normalizeAgentTaskStatus)(status));
             predicates.push(`status IN (${statuses.map(() => '?').join(', ')})`);
             params.push(...statuses);
         }
@@ -735,17 +740,17 @@ class LocalCoreAcpStore {
             throw new Error(`Task not found: ${taskId}`);
         }
         const now = new Date().toISOString();
-        const nextStatus = input.status || existing.status;
+        const nextStatus = input.status ? (0, index_js_1.normalizeAgentTaskStatus)(input.status) : existing.status;
         const timeline = [...existing.timeline];
         const logs = [...existing.logs];
         const artifacts = [...existing.artifacts];
         const approvalIds = [...existing.approvalIds];
-        if (input.status && input.status !== existing.status) {
+        if (input.status && nextStatus !== existing.status) {
             timeline.push({
                 id: `timeline:${(0, node_crypto_1.randomUUID)()}`,
                 type: 'status_change',
-                title: `Task ${input.status}`,
-                status: input.status,
+                title: `Task ${nextStatus}`,
+                status: nextStatus,
                 timestamp: now,
             });
         }
@@ -780,14 +785,14 @@ class LocalCoreAcpStore {
       WHERE id = ?
     `).run(nextStatus, input.threadId || existing.threadId || null, input.runId || existing.runId || null, input.title || existing.title, now, existing.queuedAt || (nextStatus === 'queued' ? now : null), existing.startedAt || (nextStatus === 'running' ? now : null), existing.completedAt || (['completed', 'failed', 'cancelled'].includes(nextStatus) ? now : null), input.summary ?? existing.summary ?? null, input.error === null ? null : input.error ?? existing.error ?? null, JSON.stringify(timeline), JSON.stringify(logs), JSON.stringify(artifacts), JSON.stringify(approvalIds), JSON.stringify(input.metadata || existing.metadata || {}), taskId);
         const task = this.getAgentTask(taskId);
-        if (input.status && input.status !== existing.status) {
+        if (input.status && nextStatus !== existing.status) {
             this.createAuditEvent({
                 type: 'task.updated',
                 workspaceId: task.workspaceId,
                 taskId: task.taskId,
                 actor: 'local',
-                summary: `Task status changed to ${input.status}.`,
-                metadata: { previousStatus: existing.status, status: input.status, runId: task.runId },
+                summary: `Task status changed to ${nextStatus}.`,
+                metadata: { previousStatus: existing.status, status: nextStatus, runId: task.runId },
             });
         }
         return task;
@@ -820,14 +825,20 @@ class LocalCoreAcpStore {
         return row ? this.toScheduledJob(row) : undefined;
     }
     createScheduledJob(input) {
-        const id = `job:${(0, node_crypto_1.randomUUID)()}`;
+        let id = (0, job_id_js_1.createScheduledJobId)();
+        for (let attempt = 0; attempt < 5 && this.getScheduledJob(id); attempt += 1) {
+            id = (0, job_id_js_1.createScheduledJobId)();
+        }
+        if (this.getScheduledJob(id)) {
+            throw new Error('Unable to allocate a unique scheduled job id.');
+        }
         const now = new Date().toISOString();
         this.db.prepare(`
       INSERT INTO scheduled_jobs (
         id, workspace_id, platform, route_type, route_config, execution_mode, trigger_type, cron_expr, run_at,
         prompt_template, description, enabled, concurrency_policy, created_at, updated_at
       ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, 'skip_if_running', ?, ?)
-    `).run(id, input.workspaceId, input.platform, input.route.type, JSON.stringify(input.route), input.executionMode || 'same-thread', input.triggerType, input.cronExpr || null, input.runAt || null, input.promptTemplate, input.description || '', input.enabled === false ? 0 : 1, now, now);
+    `).run(id, input.workspaceId, (0, index_js_1.normalizeChannelPlatform)(input.platform), input.route.type, JSON.stringify(input.route), (0, index_js_1.normalizeScheduledJobExecutionMode)(input.executionMode), (0, index_js_1.normalizeScheduledJobTriggerType)(input.triggerType), input.cronExpr || null, input.runAt || null, input.promptTemplate, input.description || '', input.enabled === false ? 0 : 1, now, now);
         return this.getScheduledJob(id);
     }
     updateScheduledJob(jobId, input) {
@@ -838,8 +849,8 @@ class LocalCoreAcpStore {
         const next = {
             ...existing,
             ...(input.route ? { route: input.route } : {}),
-            ...(input.executionMode ? { executionMode: input.executionMode } : {}),
-            ...(input.triggerType ? { triggerType: input.triggerType } : {}),
+            ...(input.executionMode ? { executionMode: (0, index_js_1.normalizeScheduledJobExecutionMode)(input.executionMode) } : {}),
+            ...(input.triggerType ? { triggerType: (0, index_js_1.normalizeScheduledJobTriggerType)(input.triggerType) } : {}),
             ...(Object.prototype.hasOwnProperty.call(input, 'cronExpr') ? { cronExpr: input.cronExpr || undefined } : {}),
             ...(Object.prototype.hasOwnProperty.call(input, 'runAt') ? { runAt: input.runAt || undefined } : {}),
             ...(typeof input.promptTemplate === 'string' ? { promptTemplate: input.promptTemplate } : {}),
@@ -856,7 +867,10 @@ class LocalCoreAcpStore {
         return this.getScheduledJob(jobId);
     }
     deleteScheduledJob(jobId) {
-        this.db.prepare('DELETE FROM scheduled_jobs WHERE id = ?').run(jobId);
+        const result = this.db.prepare('DELETE FROM scheduled_jobs WHERE id = ?').run(jobId);
+        if (result.changes === 0) {
+            throw new Error(`Scheduled job not found: ${jobId}`);
+        }
         return { deleted: true };
     }
     listScheduledJobRuns(jobId) {
@@ -871,13 +885,14 @@ class LocalCoreAcpStore {
     createScheduledJobRun(jobId, status, input = {}) {
         const id = `jobrun:${(0, node_crypto_1.randomUUID)()}`;
         const triggeredAt = input.triggeredAt || new Date().toISOString();
+        const normalizedStatus = (0, index_js_1.normalizeScheduledJobRunStatus)(status);
         this.db.prepare(`
       INSERT INTO scheduled_job_runs (id, job_id, status, triggered_at, started_at, finished_at, error, thread_id, run_id, platform_message_id)
       VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
-    `).run(id, jobId, status, triggeredAt, input.startedAt || null, input.finishedAt || null, input.error || null, input.threadId || null, input.runId || null, input.platformMessageId || null);
+    `).run(id, jobId, normalizedStatus, triggeredAt, input.startedAt || null, input.finishedAt || null, input.error || null, input.threadId || null, input.runId || null, input.platformMessageId || null);
         this.updateScheduledJobStatus(jobId, {
             lastRunAt: triggeredAt,
-            lastStatus: status,
+            lastStatus: normalizedStatus,
             lastError: input.error || '',
         });
         return this.getScheduledJobRun(id);
@@ -898,6 +913,7 @@ class LocalCoreAcpStore {
         const next = {
             ...existing,
             ...input,
+            ...(input.status ? { status: (0, index_js_1.normalizeScheduledJobRunStatus)(input.status) } : {}),
         };
         this.db.prepare(`
       UPDATE scheduled_job_runs
@@ -1113,10 +1129,10 @@ class LocalCoreAcpStore {
         return {
             id: row.id,
             workspaceId: row.workspace_id,
-            platform: row.platform,
+            platform: (0, index_js_1.normalizeChannelPlatform)(row.platform),
             route,
-            executionMode: row.execution_mode || 'same-thread',
-            triggerType: row.trigger_type,
+            executionMode: (0, index_js_1.normalizeScheduledJobExecutionMode)(row.execution_mode),
+            triggerType: (0, index_js_1.normalizeScheduledJobTriggerType)(row.trigger_type),
             cronExpr: row.cron_expr || undefined,
             runAt: row.run_at || undefined,
             promptTemplate: row.prompt_template,
@@ -1126,7 +1142,7 @@ class LocalCoreAcpStore {
             createdAt: row.created_at,
             updatedAt: row.updated_at,
             lastRunAt: row.last_run_at || undefined,
-            lastStatus: row.last_status || undefined,
+            lastStatus: row.last_status ? (0, index_js_1.normalizeScheduledJobRunStatus)(row.last_status) : undefined,
             lastError: row.last_error || undefined,
         };
     }
@@ -1134,7 +1150,7 @@ class LocalCoreAcpStore {
         return {
             id: row.id,
             jobId: row.job_id,
-            status: row.status,
+            status: (0, index_js_1.normalizeScheduledJobRunStatus)(row.status),
             triggeredAt: row.triggered_at,
             startedAt: row.started_at || undefined,
             finishedAt: row.finished_at || undefined,
@@ -1187,7 +1203,7 @@ class LocalCoreAcpStore {
             runId: row.run_id || undefined,
             title: row.title,
             prompt: row.prompt || undefined,
-            status: row.status,
+            status: (0, index_js_1.normalizeAgentTaskStatus)(row.status),
             createdAt: row.created_at,
             updatedAt: row.updated_at,
             queuedAt: row.queued_at || undefined,
@@ -1224,7 +1240,7 @@ class LocalCoreAcpStore {
             runId: row.run_id || undefined,
             deviceId: row.device_id,
             kind: row.kind,
-            status: row.status,
+            status: (0, index_js_1.normalizeApprovalRequestStatus)(row.status),
             riskLevel: row.risk_level,
             title: row.title,
             description: row.description,

package/dist-electron/services/local-ai-core/src/acp/workspace-acp-permissions.js

@@ -4,18 +4,9 @@ exports.normalizePermissionAction = normalizePermissionAction;
 exports.normalizePermissionOptionAction = normalizePermissionOptionAction;
 exports.formatToolCallContent = formatToolCallContent;
 exports.toPermissionButtonRows = toPermissionButtonRows;
+const desktop_js_1 = require("../../../../shared/desktop.js");
 function normalizePermissionAction(kind) {
-    const normalized = String(kind || '').trim().toLowerCase();
-    if (isAllowAllPermissionToken(normalized)) {
-        return 'allow all';
-    }
-    if (isAllowPermissionToken(normalized)) {
-        return 'allow';
-    }
-    if (isDenyPermissionToken(normalized)) {
-        return 'deny';
-    }
-    return '';
+    return (0, desktop_js_1.normalizePermissionResponse)(kind) || '';
 }
 function normalizePermissionOptionAction(option) {
     const fields = [
@@ -23,51 +14,17 @@ function normalizePermissionOptionAction(option) {
         option?.name,
         option?.optionId,
     ].map((value) => String(value || '').trim().toLowerCase()).filter(Boolean);
-    if (fields.some(isDenyPermissionToken)) {
+    if (fields.some((field) => normalizePermissionAction(field) === 'deny')) {
         return 'deny';
     }
-    if (fields.some(isAllowAllPermissionToken)) {
+    if (fields.some((field) => normalizePermissionAction(field) === 'allow all')) {
         return 'allow all';
     }
-    if (fields.some(isAllowPermissionToken)) {
+    if (fields.some((field) => normalizePermissionAction(field) === 'allow')) {
         return 'allow';
     }
     return '';
 }
-function isAllowAllPermissionToken(normalized) {
-    if (normalized === 'allow_always' ||
-        normalized === 'allow_all' ||
-        normalized === 'allowall' ||
-        normalized === 'always' ||
-        normalized === 'always_allow' ||
-        normalized === 'alwaysallow' ||
-        normalized.includes('allow_all') ||
-        normalized.includes('allow-always') ||
-        normalized.includes('allow_always') ||
-        normalized.includes('allow always') ||
-        normalized.includes('always allow') ||
-        normalized.includes('allow all') ||
-        normalized.includes('始终允许') ||
-        normalized.includes('永久允许')) {
-        return true;
-    }
-    return false;
-}
-function isAllowPermissionToken(normalized) {
-    if (normalized.startsWith('allow')) {
-        return true;
-    }
-    return false;
-}
-function isDenyPermissionToken(normalized) {
-    if (normalized.startsWith('reject')) {
-        return true;
-    }
-    if (normalized.startsWith('deny')) {
-        return true;
-    }
-    return false;
-}
 function formatToolCallContent(toolCall) {
     if (!toolCall || typeof toolCall !== 'object') {
         return 'Permission required before continuing.';