postgresai 0.16.0-dev.4 → 0.16.0-dev.5

package/bun.lock

@@ -6,7 +6,7 @@
       "name": "postgresai",
       "dependencies": {
         "@modelcontextprotocol/sdk": "^1.20.2",
-        "commander": "^14.0.3",
+        "commander": "^12.1.0",
         "js-yaml": "^4.1.0",
         "pg": "^8.16.3",
       },
@@ -16,7 +16,7 @@
         "@types/pg": "^8.15.6",
         "ajv": "^8.17.1",
         "ajv-formats": "^3.0.1",
-        "typescript": "^6.0.2",
+        "typescript": "^5.9.3",
       },
     },
   },
@@ -51,7 +51,7 @@
 
     "call-bound": ["call-bound@1.0.4", "", { "dependencies": { "call-bind-apply-helpers": "^1.0.2", "get-intrinsic": "^1.3.0" } }, "sha512-+ys997U96po4Kx/ABpBCqhA9EuxJaQWDQg7295H4hBphv3IZg0boBKuwYpt4YXp6MZ5AmZQnU/tyMTlRpaSejg=="],
 
-    "commander": ["commander@14.0.3", "", {}, "sha512-H+y0Jo/T1RZ9qPP4Eh1pkcQcLRglraJaSLoyOtHxu6AapkjWVCy2Sit1QQ4x3Dng8qDlSsZEet7g5Pq06MvTgw=="],
+    "commander": ["commander@12.1.0", "", {}, "sha512-Vw8qHK3bZM9y/P10u3Vib8o/DdkvA2OtPtZvD871QKjy74Wj1WSKFILMPRPSdUSx5RFK1arlJzEtA4PkFgnbuA=="],
 
     "content-disposition": ["content-disposition@1.0.0", "", { "dependencies": { "safe-buffer": "5.2.1" } }, "sha512-Au9nRL8VNUut/XSzbQA38+M78dzP4D+eqg3gfJHMIHHYa3bg067xj1KxMUWj+VULbiZMowKngFFbKczUrNJ1mg=="],
 
@@ -233,7 +233,7 @@
 
     "type-is": ["type-is@2.0.1", "", { "dependencies": { "content-type": "^1.0.5", "media-typer": "^1.1.0", "mime-types": "^3.0.0" } }, "sha512-OZs6gsjF4vMp32qrCbiVSkrFmXtG/AZhY3t0iAMrMBiAZyV9oALtXO8hsrHbMXF9x6L3grlFuwW2oAz7cav+Gw=="],
 
-    "typescript": ["typescript@6.0.3", "", { "bin": { "tsc": "bin/tsc", "tsserver": "bin/tsserver" } }, "sha512-y2TvuxSZPDyQakkFRPZHKFm+KKVqIisdg9/CZwm9ftvKXLP8NRWj38/ODjNbr43SsoXqNuAisEf1GdCxqWcdBw=="],
+    "typescript": ["typescript@5.9.3", "", { "bin": { "tsc": "bin/tsc", "tsserver": "bin/tsserver" } }, "sha512-jl1vZzPDinLr9eUt3J/t7V6FgNEw9QjvBPdysz9KfQDD41fQrC2Y4vKQdiaUpFT4bXlb1RHhLpp8wtm6M5TgSw=="],
 
     "undici-types": ["undici-types@5.26.5", "", {}, "sha512-JlCMO+ehdEIKqlFxk6IfVoAUVmgz7cU7zD/h9XZ0qzeosSHmUJVOzSQvvYSYWXkFXC+IfLKSIffhv0sVZup6pA=="],
 

package/dist/bin/postgres-ai.js

@@ -13425,7 +13425,7 @@ var {
 // package.json
 var package_default = {
   name: "postgresai",
-  version: "0.16.0-dev.4",
+  version: "0.16.0-dev.5",
   description: "postgres_ai CLI",
   license: "Apache-2.0",
   private: false,
@@ -16256,7 +16256,7 @@ var Result = import_lib.default.Result;
 var TypeOverrides = import_lib.default.TypeOverrides;
 var defaults = import_lib.default.defaults;
 // package.json
-var version = "0.16.0-dev.4";
+var version = "0.16.0-dev.5";
 var package_default2 = {
   name: "postgresai",
   version,
@@ -17847,7 +17847,7 @@ async function destroySnapshot(params) {
   const { apiKey, apiBaseUrl, instanceId, snapshotId, force, debug } = params;
   if (!snapshotId)
     throw new Error("snapshotId is required");
-  const action = `/snapshot/${encodeURIComponent(snapshotId)}?force=${Boolean(force)}`;
+  const action = `/snapshot/${snapshotId}?force=${Boolean(force)}`;
   return callDblabApi({
     apiKey,
     apiBaseUrl,
@@ -27585,7 +27585,7 @@ async function destroySnapshot2(params) {
   const { apiKey, apiBaseUrl, instanceId, snapshotId, force, debug } = params;
   if (!snapshotId)
     throw new Error("snapshotId is required");
-  const action = `/snapshot/${encodeURIComponent(snapshotId)}?force=${Boolean(force)}`;
+  const action = `/snapshot/${snapshotId}?force=${Boolean(force)}`;
   return callDblabApi2({
     apiKey,
     apiBaseUrl,

package/lib/dblab.ts

@@ -415,11 +415,17 @@ export interface DestroySnapshotParams extends DblabCommon {
   force?: boolean;
 }
 
-/** Destroy a snapshot — `/snapshot/<id>?force=<bool>` DELETE (Console `destroySnapshot`). */
+/** Destroy a snapshot — `/snapshot/<id>?force=<bool>` DELETE (Console `destroySnapshot`).
+ *
+ * The snapshot id is a MULTI-SEGMENT zfs path (`pool/branch/<b>/<clone>/r0@snap`),
+ * which the DBLab engine routes as a wildcard path — it must be passed RAW,
+ * exactly as the Console does. `encodeURIComponent` here turned `/`→`%2F` and
+ * `@`→`%40`, which the engine rejects with 400 `invalid snapshot name given`
+ * (verified live against DBLab CE 4.1.3). */
 export async function destroySnapshot<T = unknown>(params: DestroySnapshotParams): Promise<T> {
   const { apiKey, apiBaseUrl, instanceId, snapshotId, force, debug } = params;
   if (!snapshotId) throw new Error("snapshotId is required");
-  const action = `/snapshot/${encodeURIComponent(snapshotId)}?force=${Boolean(force)}`;
+  const action = `/snapshot/${snapshotId}?force=${Boolean(force)}`;
   return callDblabApi<T>({
     apiKey, apiBaseUrl, instanceId,
     action, method: "delete",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "postgresai",
-  "version": "0.16.0-dev.4",
+  "version": "0.16.0-dev.5",
   "description": "postgres_ai CLI",
   "license": "Apache-2.0",
   "private": false,

package/test/dblab.test.ts

@@ -302,6 +302,13 @@ describe("snapshot verbs → dblab_api_call", () => {
     expect(b.method).toBe("delete");
   });
 
+  test("destroySnapshot passes a multi-segment zfs snapshot id RAW (no percent-encoding)", async () => {
+    const cap = stubFetch(true);
+    await destroySnapshot({ ...common, snapshotId: "dblab_pool/branch/main/c-1/r0@20260707065703" });
+    // The engine 400s on %2F/%40 — the Console passes the id raw and so must we.
+    expect(bodyOf(cap.get()).action).toBe("/snapshot/dblab_pool/branch/main/c-1/r0@20260707065703?force=false");
+  });
+
   test("destroySnapshot defaults force=false", async () => {
     const cap = stubFetch(true);
     await destroySnapshot({ ...common, snapshotId: "s-1" });