@testsmith/api-spector 0.2.1 → 0.2.2

package/out/renderer/assets/{index-DPHZdvL5.js → index-C_vjoxxA.js}

@@ -56237,7 +56237,6 @@ const SNIPPET_GROUPS = [
       {
         label: "Check JSON value",
         code: `sp.test("JSON field equals value", function() {
-  const json = sp.response.json();
   sp.expect(json.field).to.equal("expected_value");
 });`
       },
@@ -56252,7 +56251,7 @@ const SNIPPET_GROUPS = [
     },
     required: ["id", "name"]
   };
-  sp.expect(tv4.validate(sp.response.json(), schema)).to.be.true;
+  sp.expect(tv4.validate(json, schema)).to.be.true;
 });`
       }
     ]
@@ -56284,7 +56283,6 @@ const SNIPPET_GROUPS = [
       {
         label: "Save token from response (use in next requests)",
         code: `// Use collectionVariables so the value persists across requests
-const json = sp.response.json();
 sp.collectionVariables.set("token", json.access_token);`
       },
       {
@@ -56369,22 +56367,20 @@ sp.collectionVariables.set("token", json.access_token);`
     items: [
       {
         label: "Save JSON field to variable",
-        code: `const json = sp.response.json();
-sp.variables.set("field_value", json.field);`
+        code: `sp.variables.set("field_value", json.field);`
       },
       {
         label: "Save JSON field to environment",
-        code: `const json = sp.response.json();
-sp.environment.set("token", json.token);`
+        code: `sp.environment.set("token", json.token);`
       },
       {
         label: "Extract via JSONPath to variable",
-        code: `const matches = sp.jsonPath(sp.response.json(), '$.data[0].id');
+        code: `const matches = sp.jsonPath(json, '$.data[0].id');
 sp.variables.set("extracted_value", String(matches[0] ?? ''));`
       },
       {
         label: "Extract via JSONPath to environment",
-        code: `const matches = sp.jsonPath(sp.response.json(), '$.data[0].id');
+        code: `const matches = sp.jsonPath(json, '$.data[0].id');
 sp.environment.set("extracted_value", String(matches[0] ?? ''));`
       },
       {
@@ -56398,6 +56394,26 @@ sp.environment.set("extracted_value", String(matches[0] ?? ''));`
     ]
   }
 ];
+const JSON_DECL = "const json = sp.response.json();";
+const JSON_DECL_RX_G = /[ \t]*const\s+json\s*=\s*sp\.response\.json\(\)\s*;?\s*\n?/g;
+function snippetUsesJson(snippet2) {
+  const withoutDecl = snippet2.replace(JSON_DECL_RX_G, "");
+  return /\bjson\b/.test(withoutDecl);
+}
+function appendSnippetToScript(existing, snippet2) {
+  const cleanedSnippet = snippet2.replace(JSON_DECL_RX_G, "").replace(/^\n+/, "");
+  const sep = existing.trim() ? "\n\n" : "";
+  let combined = existing + sep + cleanedSnippet;
+  const needsJson = snippetUsesJson(existing) || snippetUsesJson(cleanedSnippet);
+  const hasTopDecl = /^\s*const\s+json\s*=\s*sp\.response\.json\(\)/m.test(combined);
+  if (needsJson && !hasTopDecl) {
+    combined = combined.replace(JSON_DECL_RX_G, "");
+    combined = `${JSON_DECL}
+
+${combined.replace(/^\n+/, "")}`;
+  }
+  return combined;
+}
 function ScriptsTab({ request, onChange }) {
   const activeTabId = useStore((s) => s.activeTabId);
   const activeAppTab = useStore((s) => s.tabs.find((t2) => t2.id === s.activeTabId));
@@ -56422,9 +56438,12 @@ function ScriptsTab({ request, onChange }) {
     else onChange({ graphqlIntrospectionScript: code2 });
   }
   function insertSnippet2(code2) {
-    const current2 = value;
-    const separator = current2.trim() ? "\n\n" : "";
-    handleChange(current2 + separator + code2);
+    if (scriptType === "post") {
+      handleChange(appendSnippetToScript(value, code2));
+    } else {
+      const separator = value.trim() ? "\n\n" : "";
+      handleChange(value + separator + code2);
+    }
   }
   return /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "flex gap-3 h-full min-h-0", children: [
     /* @__PURE__ */ jsxRuntimeExports.jsxs("div", { className: "flex flex-col flex-1 min-w-0 gap-2", children: [
@@ -63544,28 +63563,23 @@ function makeJsonSnippet(path, value, mode) {
   const acc = jsonAccessor(path);
   const label = jsonPathLabel(path);
   const lit = toLit(value);
-  const decl = "const json = sp.response.json();";
   switch (mode) {
     case "equals":
       return `sp.test('${label} equals ${lit}', function() {
-  ${decl}
   sp.expect(${acc}).to.equal(${lit});
 });`;
     case "exists":
       return `sp.test('${label} exists', function() {
-  ${decl}
   sp.expect(${acc}).to.not.be.oneOf([null, undefined]);
 });`;
     case "type": {
       const t2 = value === null ? "null" : typeof value;
       return `sp.test('${label} is ${t2}', function() {
-  ${decl}
   sp.expect(${acc}).to.be.a("${t2}");
 });`;
     }
     case "contains":
       return `sp.test('${label} contains ${lit}', function() {
-  ${decl}
   sp.expect(${acc}).to.include(${lit});
 });`;
   }
@@ -63574,7 +63588,7 @@ function makeJsonPathSnippet(path, value, filterKey, filterValue) {
   const expr = toJsonPathExpr(path, filterKey, filterValue);
   const lit = toLit(value);
   return `sp.test('${expr} equals ${lit}', function() {
-  const matches = sp.jsonPath(sp.response.json(), '${expr}');
+  const matches = sp.jsonPath(json, '${expr}');
   sp.expect(matches.length).to.be.above(0);
   sp.expect(matches[0]).to.equal(${lit});
 });`;
@@ -63599,13 +63613,12 @@ function makeXmlSnippet(selector, value, mode) {
 function makeJsonExtractSnippet(path, target) {
   const acc = jsonAccessor(path);
   const varName = varNameFromPath(path);
-  return `const json = sp.response.json();
-sp.${target}.set("${varName}", String(${acc}));`;
+  return `sp.${target}.set("${varName}", String(${acc}));`;
 }
 function makeJsonPathExtractSnippet(path, filterKey, filterValue, target) {
   const expr = toJsonPathExpr(path, filterKey, filterValue);
   const varName = varNameFromPath(path);
-  return `const matches = sp.jsonPath(sp.response.json(), '${expr}');
+  return `const matches = sp.jsonPath(json, '${expr}');
 sp.${target}.set("${varName}", String(matches[0] ?? ''));`;
 }
 function makeXmlExtractSnippet(selector, target) {
@@ -64402,12 +64415,7 @@ function ResponseViewer() {
     const req = Object.values(state.collections).find((c) => c.data.requests[requestId])?.data.requests[requestId];
     if (!req) return;
     const existing = req.postRequestScript ?? "";
-    let cleaned = snippet2;
-    if (existing.includes("const json = sp.response.json()")) {
-      cleaned = cleaned.replace(/^\s*const json = sp\.response\.json\(\);?\s*\n?/m, "").replace(/\n\s*const json = sp\.response\.json\(\);?\s*\n/g, "\n");
-    }
-    const sep = existing.trim() ? "\n\n" : "";
-    updateRequest(requestId, { postRequestScript: existing + sep + cleaned });
+    updateRequest(requestId, { postRequestScript: appendSnippetToScript(existing, snippet2) });
     if (activeTabId) {
       setTabRequestTab(activeTabId, "scripts");
       setTabScriptTab(activeTabId, "post");
@@ -70233,7 +70241,7 @@ function App() {
       /* @__PURE__ */ jsxRuntimeExports.jsx("span", { style: { color: "#6aa3c8" }, children: "Spector" }),
       /* @__PURE__ */ jsxRuntimeExports.jsxs("span", { className: "ml-2 text-[10px] font-normal opacity-50", children: [
         "v",
-        "0.2.1"
+        "0.2.2"
       ] }),
       /* @__PURE__ */ jsxRuntimeExports.jsx("span", { className: "mx-2 opacity-30", children: "·" }),
       /* @__PURE__ */ jsxRuntimeExports.jsx(

package/out/renderer/index.html

@@ -5,7 +5,7 @@
   <meta charset="UTF-8" />
   <meta name="viewport" content="width=device-width, initial-scale=1.0" />
   <title>API Spector</title>
-  <script type="module" crossorigin src="./assets/index-DPHZdvL5.js"></script>
+  <script type="module" crossorigin src="./assets/index-C_vjoxxA.js"></script>
   <link rel="stylesheet" crossorigin href="./assets/index-DVaubmCJ.css">
 </head>
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@testsmith/api-spector",
   "productName": "API Spector",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "description": "Local-first API testing tool to inspect, test and mock APIs",
   "repository": {
     "type": "git",