@get-bb/plugin-sdk 0.4.24 → 0.4.28

package/dist/testing/index.js

@@ -1,4 +1,5 @@
 // src/testing/fake-plugin-host.ts
+import { createHash } from "node:crypto";
 import { mkdtempSync, rmSync } from "node:fs";
 import { tmpdir } from "node:os";
 import { join } from "node:path";
@@ -524,9 +525,7 @@ function normalizeProviderBridgeOptions(providerId, value, label = "experimental
       return current;
     }
     if (typeof current !== "object") {
-      throw new Error(
-        `provider "${providerId}" ${label}${path} must be JSON`
-      );
+      throw new Error(`provider "${providerId}" ${label}${path} must be JSON`);
     }
     if (active.has(current)) {
       throw new Error(
@@ -562,9 +561,7 @@ function normalizeProviderBridgeOptions(providerId, value, label = "experimental
   }
   const normalized = visit(value, "");
   if (normalized === null || Array.isArray(normalized) || typeof normalized !== "object") {
-    throw new Error(
-      `provider "${providerId}" ${label} must be an object`
-    );
+    throw new Error(`provider "${providerId}" ${label} must be an object`);
   }
   if (Buffer.byteLength(JSON.stringify(normalized), "utf8") > PLUGIN_PROVIDER_BRIDGE_OPTIONS_MAX_BYTES) {
     throw new Error(
@@ -587,9 +584,7 @@ function requireNonBlankString(args) {
 }
 function validateProviderStrings(providerId, value) {
   if (typeof value !== "object" || value === null || Array.isArray(value)) {
-    throw new Error(
-      `provider "${providerId}" strings must be an object`
-    );
+    throw new Error(`provider "${providerId}" strings must be an object`);
   }
   const record = Object.fromEntries(
     Object.entries(value)
@@ -791,7 +786,9 @@ function validateProviderNativeRoots(providerId, field, value) {
   }
   return Object.freeze({
     user: Object.freeze(wire.data.user.map((root) => Object.freeze(root))),
-    project: Object.freeze(wire.data.project.map((root) => Object.freeze(root)))
+    project: Object.freeze(
+      wire.data.project.map((root) => Object.freeze(root))
+    )
   });
 }
 var PROVIDER_MODEL_CATALOG_SCOPES = [
@@ -811,9 +808,7 @@ function validateProviderModelCatalogScope(providerId, value) {
 }
 function validateProviderFallbackModels(providerId, value) {
   if (typeof value !== "object" || value === null || Array.isArray(value)) {
-    throw new Error(
-      `provider "${providerId}" models must be an object`
-    );
+    throw new Error(`provider "${providerId}" models must be an object`);
   }
   const fallback = Reflect.get(value, "fallback");
   if (fallback === void 0) {
@@ -831,95 +826,95 @@ function validateProviderFallbackModels(providerId, value) {
   }
   const seen = /* @__PURE__ */ new Set();
   let defaults = 0;
-  const normalized = fallback.map((entry, index) => {
-    const field = `models.fallback[${index}]`;
-    if (typeof entry !== "object" || entry === null || Array.isArray(entry)) {
-      throw new Error(`provider "${providerId}" ${field} must be an object`);
-    }
-    const record = Object.fromEntries(
-      Object.entries(entry)
-    );
-    const id = requireNonBlankString({
-      providerId,
-      field: `${field}.id`,
-      value: record.id
-    });
-    if (seen.has(id)) {
-      throw new Error(
-        `provider "${providerId}" models.fallback id ${JSON.stringify(id)} is duplicated`
-      );
-    }
-    seen.add(id);
-    const displayName = requireNonBlankString({
-      providerId,
-      field: `${field}.displayName`,
-      value: record.displayName
-    });
-    const description = requireNonBlankString({
-      providerId,
-      field: `${field}.description`,
-      value: record.description
-    });
-    const efforts = record.supportedReasoningEfforts;
-    if (!Array.isArray(efforts) || efforts.length === 0) {
-      throw new Error(
-        `provider "${providerId}" ${field}.supportedReasoningEfforts must be a non-empty array`
-      );
-    }
-    const levels = /* @__PURE__ */ new Set();
-    const supportedReasoningEfforts = efforts.map(
-      (effort, effortIndex) => {
-        if (typeof effort !== "object" || effort === null || Array.isArray(effort)) {
-          throw new Error(
-            `provider "${providerId}" ${field}.supportedReasoningEfforts[${effortIndex}] must be { reasoningEffort, description }`
-          );
-        }
-        const reasoningEffort = Reflect.get(effort, "reasoningEffort");
-        if (typeof reasoningEffort !== "string" || !PLUGIN_PROVIDER_REASONING_LEVEL_VALUES.includes(
-          reasoningEffort
-        )) {
-          throw new Error(
-            `provider "${providerId}" ${field}.supportedReasoningEfforts[${effortIndex}].reasoningEffort must be one of ${PLUGIN_PROVIDER_REASONING_LEVEL_VALUES.join(", ")}`
-          );
-        }
-        const level = reasoningEffort;
-        if (levels.has(level)) {
-          throw new Error(
-            `provider "${providerId}" ${field}.supportedReasoningEfforts repeats ${JSON.stringify(level)}`
-          );
-        }
-        levels.add(level);
-        return Object.freeze({
-          reasoningEffort: level,
-          description: requireNonBlankString({
-            providerId,
-            field: `${field}.supportedReasoningEfforts[${effortIndex}].description`,
-            value: Reflect.get(effort, "description")
-          })
-        });
+  const normalized = fallback.map(
+    (entry, index) => {
+      const field = `models.fallback[${index}]`;
+      if (typeof entry !== "object" || entry === null || Array.isArray(entry)) {
+        throw new Error(`provider "${providerId}" ${field} must be an object`);
       }
-    );
-    const defaultReasoningEffort = record.defaultReasoningEffort;
-    if (typeof defaultReasoningEffort !== "string" || !levels.has(defaultReasoningEffort)) {
-      throw new Error(
-        `provider "${providerId}" ${field}.defaultReasoningEffort must be one of its supportedReasoningEfforts`
+      const record = Object.fromEntries(
+        Object.entries(entry)
       );
-    }
-    if (typeof record.isDefault !== "boolean") {
-      throw new Error(
-        `provider "${providerId}" ${field}.isDefault must be a boolean`
+      const id = requireNonBlankString({
+        providerId,
+        field: `${field}.id`,
+        value: record.id
+      });
+      if (seen.has(id)) {
+        throw new Error(
+          `provider "${providerId}" models.fallback id ${JSON.stringify(id)} is duplicated`
+        );
+      }
+      seen.add(id);
+      const displayName = requireNonBlankString({
+        providerId,
+        field: `${field}.displayName`,
+        value: record.displayName
+      });
+      const description = requireNonBlankString({
+        providerId,
+        field: `${field}.description`,
+        value: record.description
+      });
+      const efforts = record.supportedReasoningEfforts;
+      if (!Array.isArray(efforts) || efforts.length === 0) {
+        throw new Error(
+          `provider "${providerId}" ${field}.supportedReasoningEfforts must be a non-empty array`
+        );
+      }
+      const levels = /* @__PURE__ */ new Set();
+      const supportedReasoningEfforts = efforts.map(
+        (effort, effortIndex) => {
+          if (typeof effort !== "object" || effort === null || Array.isArray(effort)) {
+            throw new Error(
+              `provider "${providerId}" ${field}.supportedReasoningEfforts[${effortIndex}] must be { reasoningEffort, description }`
+            );
+          }
+          const reasoningEffort = Reflect.get(effort, "reasoningEffort");
+          if (typeof reasoningEffort !== "string" || !PLUGIN_PROVIDER_REASONING_LEVEL_VALUES.includes(reasoningEffort)) {
+            throw new Error(
+              `provider "${providerId}" ${field}.supportedReasoningEfforts[${effortIndex}].reasoningEffort must be one of ${PLUGIN_PROVIDER_REASONING_LEVEL_VALUES.join(", ")}`
+            );
+          }
+          const level = reasoningEffort;
+          if (levels.has(level)) {
+            throw new Error(
+              `provider "${providerId}" ${field}.supportedReasoningEfforts repeats ${JSON.stringify(level)}`
+            );
+          }
+          levels.add(level);
+          return Object.freeze({
+            reasoningEffort: level,
+            description: requireNonBlankString({
+              providerId,
+              field: `${field}.supportedReasoningEfforts[${effortIndex}].description`,
+              value: Reflect.get(effort, "description")
+            })
+          });
+        }
       );
+      const defaultReasoningEffort = record.defaultReasoningEffort;
+      if (typeof defaultReasoningEffort !== "string" || !levels.has(defaultReasoningEffort)) {
+        throw new Error(
+          `provider "${providerId}" ${field}.defaultReasoningEffort must be one of its supportedReasoningEfforts`
+        );
+      }
+      if (typeof record.isDefault !== "boolean") {
+        throw new Error(
+          `provider "${providerId}" ${field}.isDefault must be a boolean`
+        );
+      }
+      if (record.isDefault) defaults += 1;
+      return Object.freeze({
+        id,
+        displayName,
+        description,
+        supportedReasoningEfforts: Object.freeze(supportedReasoningEfforts),
+        defaultReasoningEffort,
+        isDefault: record.isDefault
+      });
     }
-    if (record.isDefault) defaults += 1;
-    return Object.freeze({
-      id,
-      displayName,
-      description,
-      supportedReasoningEfforts: Object.freeze(supportedReasoningEfforts),
-      defaultReasoningEffort,
-      isDefault: record.isDefault
-    });
-  });
+  );
   if (normalized.length > 0 && defaults !== 1) {
     throw new Error(
       `provider "${providerId}" models.fallback must mark exactly one model isDefault (found ${defaults})`
@@ -981,9 +976,7 @@ function validatePluginAiServiceDeclaration(declaration) {
   }
   const displayName = typeof declaration.displayName === "string" ? declaration.displayName.trim() : "";
   if (displayName.length === 0 || displayName.length > 64) {
-    throw new Error(
-      `AI service "${id}" displayName must be 1-64 characters`
-    );
+    throw new Error(`AI service "${id}" displayName must be 1-64 characters`);
   }
   const kinds = declaration.kinds;
   if (!Array.isArray(kinds) || kinds.length === 0) {
@@ -1219,10 +1212,7 @@ function validatePluginProviderDeclaration(declaration) {
     field: "reasoningLevels",
     value: declaration.reasoningLevels
   });
-  const extensionKinds = declaration.extensionKinds === void 0 ? void 0 : validateProviderExtensionKinds(
-    id,
-    declaration.extensionKinds
-  );
+  const extensionKinds = declaration.extensionKinds === void 0 ? void 0 : validateProviderExtensionKinds(id, declaration.extensionKinds);
   const fallbackModels = declaration.models === void 0 ? void 0 : validateProviderFallbackModels(id, declaration.models);
   const modelCatalogScope = validateProviderModelCatalogScope(
     id,
@@ -1473,9 +1463,7 @@ function parsePluginAgentToolPresentation(toolName, value) {
     return null;
   }
   if (typeof value !== "object" || value === null || Array.isArray(value)) {
-    throw new Error(
-      `tool "${toolName}" presentation must be an object`
-    );
+    throw new Error(`tool "${toolName}" presentation must be an object`);
   }
   const declared = value;
   const presentation = {};
@@ -1695,6 +1683,10 @@ function createFakeSdk(options) {
 }
 
 // src/testing/fake-plugin-host.ts
+var LEGACY_UNKNOWN_MIGRATION_HASH = "legacy-unknown";
+function migrationStatementHash(statement) {
+  return createHash("sha256").update(statement).digest("hex");
+}
 var PluginContextStaleError = class extends Error {
   constructor(pluginId) {
     super(
@@ -2008,7 +2000,9 @@ function createFakePluginHostInternal(options, sharedState) {
     )
   };
   const pluginId = options.pluginId ?? "test-plugin";
-  const declaredIconNames = new Set(options.experimental_declaredIconNames ?? []);
+  const declaredIconNames = new Set(
+    options.experimental_declaredIconNames ?? []
+  );
   const agentSkillIds = [...options.agentSkillIds ?? []];
   if (new Set(agentSkillIds).size !== agentSkillIds.length) {
     throw new Error("agentSkillIds must not contain duplicates");
@@ -2074,19 +2068,44 @@ function createFakePluginHostInternal(options, sharedState) {
     migrate(database, statements) {
       assertLive();
       database.exec(
-        "CREATE TABLE IF NOT EXISTS _bb_migrations (id INTEGER PRIMARY KEY, applied_at INTEGER NOT NULL)"
+        "CREATE TABLE IF NOT EXISTS _bb_migrations (id INTEGER PRIMARY KEY, applied_at INTEGER NOT NULL, statement_hash TEXT)"
       );
-      const applied = new Set(
-        database.prepare("SELECT id FROM _bb_migrations").all().map((row) => row.id)
+      const migrationColumns = database.prepare("PRAGMA table_info(_bb_migrations)").all();
+      if (!migrationColumns.some((column) => column.name === "statement_hash")) {
+        database.exec(
+          "ALTER TABLE _bb_migrations ADD COLUMN statement_hash TEXT"
+        );
+      }
+      const rows = database.prepare("SELECT id, statement_hash FROM _bb_migrations ORDER BY id").all();
+      const applied = /* @__PURE__ */ new Map();
+      for (const row of rows) applied.set(row.id, row.statement_hash);
+      const statementHashes = statements.map(migrationStatementHash);
+      statementHashes.forEach((statementHash, index) => {
+        const recordedHash = applied.get(index);
+        if (recordedHash !== void 0 && recordedHash !== null && recordedHash !== statementHash) {
+          throw new Error(
+            `migration ${index} does not match the recorded statement; append a new migration instead of changing or reusing an index`
+          );
+        }
+      });
+      const adopt = database.prepare(
+        "UPDATE _bb_migrations SET statement_hash = ? WHERE id = ? AND statement_hash IS NULL"
       );
       const record = database.prepare(
-        "INSERT INTO _bb_migrations (id, applied_at) VALUES (?, ?)"
+        "INSERT INTO _bb_migrations (id, applied_at, statement_hash) VALUES (?, ?, ?)"
       );
       database.transaction(() => {
+        for (const row of rows) {
+          if (row.statement_hash !== null) continue;
+          adopt.run(
+            statementHashes[row.id] ?? LEGACY_UNKNOWN_MIGRATION_HASH,
+            row.id
+          );
+        }
         statements.forEach((statement, index) => {
           if (applied.has(index)) return;
           database.exec(statement);
-          record.run(index, Date.now());
+          record.run(index, Date.now(), statementHashes[index]);
         });
       })();
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@get-bb/plugin-sdk",
-  "version": "0.4.24",
+  "version": "0.4.28",
   "homepage": "https://github.com/get-bb/bb#readme",
   "bugs": {
     "url": "https://github.com/get-bb/bb/issues"