@cremini/skillpack 1.2.8 → 1.2.9

package/dist/cli.js

@@ -2694,7 +2694,7 @@ var ResultsQueryService = class {
   constructor(resultStore) {
     this.resultStore = resultStore;
   }
-  listRecentArtifacts(options = {}) {
+  async listRecentArtifacts(options = {}) {
     return this.resultStore.listRecentArtifacts({
       channelId: options.channelId,
       limit: clampLimit(options.limit, 100, 500),
@@ -2865,7 +2865,7 @@ var ArtifactPersistenceService = class {
     this.snapshotService = snapshotService;
     this.resultStore = resultStore;
   }
-  saveArtifacts(input) {
+  async saveArtifacts(input) {
     const declaredAt = (/* @__PURE__ */ new Date()).toISOString();
     const snapshots = this.snapshotService.createSnapshots(
       input.runId,
@@ -2873,7 +2873,7 @@ var ArtifactPersistenceService = class {
       declaredAt
     );
     try {
-      this.resultStore.insertArtifacts({
+      await this.resultStore.insertArtifacts({
         runId: input.runId,
         channelId: input.channelId,
         artifacts: snapshots
@@ -2892,7 +2892,7 @@ var ArtifactPersistenceService = class {
 import fs10 from "fs";
 import path10 from "path";
 import { randomUUID as randomUUID2 } from "crypto";
-import Database from "better-sqlite3";
+import sqlite3 from "sqlite3";
 function mapArtifactRow(row) {
   return {
     artifactId: row.artifact_id,
@@ -2909,16 +2909,17 @@ function mapArtifactRow(row) {
   };
 }
 var ResultStore = class {
-  db;
+  db = null;
+  ready;
   constructor(rootDir) {
     const dataDir = path10.resolve(rootDir, "data");
     fs10.mkdirSync(dataDir, { recursive: true });
-    this.db = new Database(path10.join(dataDir, "result.db"));
-    this.db.pragma("journal_mode = WAL");
-    this.initialize();
+    this.ready = this.initialize(path10.join(dataDir, "result-v2.db"));
   }
-  initialize() {
-    this.db.exec(`
+  async initialize(databasePath) {
+    this.db = await openDatabase(databasePath);
+    await this.exec("PRAGMA journal_mode = WAL");
+    await this.exec(`
       CREATE TABLE IF NOT EXISTS artifacts (
         artifact_id TEXT PRIMARY KEY,
         run_id TEXT NOT NULL,
@@ -2937,11 +2938,12 @@ var ResultStore = class {
       ON artifacts(channel_id, declared_at DESC);
     `);
   }
-  insertArtifacts(input) {
+  async insertArtifacts(input) {
+    await this.ready;
     if (input.artifacts.length === 0) {
       return;
     }
-    const insertArtifact = this.db.prepare(`
+    const insertArtifact = `
       INSERT INTO artifacts (
         artifact_id,
         run_id,
@@ -2955,39 +2957,44 @@ var ResultStore = class {
         is_primary,
         declared_at
       ) VALUES (
-        @artifactId,
-        @runId,
-        @channelId,
-        @originalPath,
-        @snapshotPath,
-        @fileName,
-        @mimeType,
-        @sizeBytes,
-        @title,
-        @isPrimary,
-        @declaredAt
+        ?,
+        ?,
+        ?,
+        ?,
+        ?,
+        ?,
+        ?,
+        ?,
+        ?,
+        ?,
+        ?
       )
-    `);
-    const transaction = this.db.transaction((payload) => {
-      for (const artifact of payload.artifacts) {
-        insertArtifact.run({
-          artifactId: randomUUID2(),
-          runId: payload.runId,
-          channelId: payload.channelId,
-          originalPath: artifact.originalPath,
-          snapshotPath: artifact.snapshotPath,
-          fileName: artifact.fileName,
-          mimeType: artifact.mimeType ?? null,
-          sizeBytes: artifact.sizeBytes,
-          title: artifact.title ?? null,
-          isPrimary: artifact.isPrimary ? 1 : 0,
-          declaredAt: artifact.declaredAt
-        });
-      }
-    });
-    transaction(input);
+    `;
+    await this.exec("BEGIN");
+    try {
+      for (const artifact of input.artifacts) {
+        await this.run(insertArtifact, [
+          randomUUID2(),
+          input.runId,
+          input.channelId,
+          artifact.originalPath,
+          artifact.snapshotPath,
+          artifact.fileName,
+          artifact.mimeType ?? null,
+          artifact.sizeBytes,
+          artifact.title ?? null,
+          artifact.isPrimary ? 1 : 0,
+          artifact.declaredAt
+        ]);
+      }
+      await this.exec("COMMIT");
+    } catch (error) {
+      await this.rollback();
+      throw error;
+    }
   }
-  listRecentArtifacts(options = {}) {
+  async listRecentArtifacts(options = {}) {
+    await this.ready;
     const limit = options.limit ?? 100;
     const offset = options.offset ?? 0;
     const conditions = [];
@@ -2997,16 +3004,75 @@ var ResultStore = class {
       params.push(options.channelId);
     }
     const whereClause = conditions.length > 0 ? `WHERE ${conditions.join(" AND ")}` : "";
-    const rows = this.db.prepare(`
+    const rows = await this.all(`
       SELECT *
       FROM artifacts
       ${whereClause}
       ORDER BY declared_at DESC, rowid DESC
       LIMIT ? OFFSET ?
-    `).all(...params, limit, offset);
+    `, [...params, limit, offset]);
     return rows.map(mapArtifactRow);
   }
+  getDatabase() {
+    if (!this.db) {
+      throw new Error("Result store database is not ready");
+    }
+    return this.db;
+  }
+  exec(sql) {
+    const db = this.getDatabase();
+    return new Promise((resolve, reject) => {
+      db.exec(sql, (error) => {
+        if (error) {
+          reject(error);
+          return;
+        }
+        resolve();
+      });
+    });
+  }
+  run(sql, params = []) {
+    const db = this.getDatabase();
+    return new Promise((resolve, reject) => {
+      db.run(sql, params, (error) => {
+        if (error) {
+          reject(error);
+          return;
+        }
+        resolve();
+      });
+    });
+  }
+  all(sql, params = []) {
+    const db = this.getDatabase();
+    return new Promise((resolve, reject) => {
+      db.all(sql, params, (error, rows) => {
+        if (error) {
+          reject(error);
+          return;
+        }
+        resolve(rows);
+      });
+    });
+  }
+  async rollback() {
+    try {
+      await this.exec("ROLLBACK");
+    } catch {
+    }
+  }
 };
+function openDatabase(databasePath) {
+  return new Promise((resolve, reject) => {
+    const db = new sqlite3.Database(databasePath, (error) => {
+      if (error) {
+        reject(error);
+        return;
+      }
+      resolve(db);
+    });
+  });
+}
 
 // src/runtime/artifacts/save-artifacts-tool.ts
 import { Type } from "@sinclair/typebox";
@@ -4495,12 +4561,12 @@ var WebAdapter = class {
         )
       );
     });
-    app.get("/api/results/artifacts", (req, res) => {
+    app.get("/api/results/artifacts", async (req, res) => {
       if (!resultsQueryService) {
         res.status(503).json({ error: "Results query service is not available" });
         return;
       }
-      res.json(resultsQueryService.listRecentArtifacts({
+      res.json(await resultsQueryService.listRecentArtifacts({
         channelId: typeof req.query.channelId === "string" ? req.query.channelId : void 0,
         limit: parsePositiveInt(req.query.limit, 100),
         offset: parsePositiveInt(req.query.offset, 0)
@@ -4819,7 +4885,7 @@ var IpcAdapter = class {
             this.replyError(request.id, "Results query service is not available");
             return;
           }
-          this.reply(request.id, this.resultsQueryService.listRecentArtifacts({
+          this.reply(request.id, await this.resultsQueryService.listRecentArtifacts({
             channelId: request.channelId,
             limit: request.limit,
             offset: request.offset

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cremini/skillpack",
-  "version": "1.2.8",
+  "version": "1.2.9",
   "description": "Pack AI Skills into Local Agents",
   "type": "module",
   "repository": {
@@ -49,18 +49,17 @@
     "@slack/bolt": "^4.6.0",
     "ajv": "^8.17.1",
     "archiver": "^7.0.1",
-    "better-sqlite3": "^11.10.0",
     "chalk": "^5.6.2",
     "commander": "^14.0.3",
     "express": "^5.1.0",
     "inquirer": "^13.3.0",
     "node-cron": "^4.2.1",
     "node-telegram-bot-api": "^0.66.0",
+    "sqlite3": "^5.1.7",
     "ws": "^8.19.0"
   },
   "devDependencies": {
     "@types/archiver": "^7.0.0",
-    "@types/better-sqlite3": "^7.6.13",
     "@types/express": "^5.0.0",
     "@types/inquirer": "^9.0.9",
     "@types/node": "^25.5.0",