rails-profiler 0.19.0 → 0.19.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a4e5e84c4946f549552450c0d2d2d2053b273d3bed8dba66fefa17d9c4e5f7f9
-  data.tar.gz: 6b4be4f2b3116dbdc4270df5a6f860eabbbf709c3fc296ea34f400aaeb26ce83
+  metadata.gz: 713e1b01bcd916191e3ae2b89124f4637eeb95d7b9837d350d2ee76a575a2f14
+  data.tar.gz: 9f1becf53b9b39145a4ebdf81981793aeb5f0494cce1e6872facdcf11c0a597a
 SHA512:
-  metadata.gz: 66136505b84f792418deb7a29d7d77f36a3cc62cf3bdab9d7defc842f9797b39667c92e18984925a14276880a129ef5b42faa644112ab1c917b61e9f11f8b359
-  data.tar.gz: b4ac6c0b37496721a3960732ab0839224e896aa4390757b36d84595200e6441034649f8829573a1fee30cf9c8216c5275e5faa9b0534bc48fe49a20e738d2bf3
+  metadata.gz: 298e772fd273b734126c095e57186e36d7f6fbaf26c7bb7db9899d02a38431231e4ecbde42f0b1a103deac26c29f48a596a065b5dd93f16a900b24d66e69a280
+  data.tar.gz: 78f23d51a52d0467e3acf9d912cbcbeebec03cf245632d88c7306d00d7c00645dda2be0a6208f3e58f4c815ea3a9efe0cb4e6783a3c6c4be44b784fd72e35f37

data/app/assets/builds/profiler.js

@@ -989,6 +989,7 @@
       const body = await res.json().catch(() => ({}));
       throw new Error(body.error ?? `Request failed (${res.status})`);
     }
+    return res.json();
   }
   async function resetEnvVar(key) {
     const res = await fetch(`/_profiler/api/env_vars/reset?key=${encodeURIComponent(key)}`, {
@@ -1005,7 +1006,7 @@
     if (!res.ok) throw new Error(`Request failed (${res.status})`);
   }
   function EnvTab({ envData, readOnly: forceReadOnly = false }) {
-    const initial = T2(() => ({ ...envData?.variables ?? {} }), []);
+    const [initial, setInitial] = d2(() => ({ ...envData?.variables ?? {} }));
     const [variables, setVariables] = d2(initial);
     const [overrides, setOverrides] = d2(envData?.overrides ?? {});
     const [total, setTotal] = d2(envData?.total ?? 0);
@@ -1070,6 +1071,7 @@
         setTotal(data.total);
         setOverrides(data.overrides ?? {});
         showFlash("success", `Refreshed \u2014 ${data.total} variables`);
+        return data;
       } catch (e3) {
         showFlash("error", e3.message ?? "Failed to refresh");
       } finally {
@@ -1116,11 +1118,25 @@
     };
     const saveEdit = async () => {
       if (!editingKey) return;
+      const key = editingKey;
+      if (editValue === variables[key]) {
+        setEditingKey(null);
+        return;
+      }
       setSaving(true);
       try {
-        await patchEnvVar(editingKey, editValue);
-        setVariables((prev) => ({ ...prev, [editingKey]: editValue }));
-        showFlash("success", `${editingKey} updated`);
+        const data = await patchEnvVar(key, editValue);
+        setVariables((prev) => ({ ...prev, [key]: editValue }));
+        setOverrides((prev) => {
+          const n2 = { ...prev };
+          if (data.override) {
+            n2[key] = data.override;
+          } else {
+            delete n2[key];
+          }
+          return n2;
+        });
+        showFlash("success", `${key} updated`);
         setEditingKey(null);
       } catch (e3) {
         showFlash("error", e3.message ?? "Failed to update");
@@ -1131,12 +1147,21 @@
     const deleteVar = async (key) => {
       setSaving(true);
       try {
-        await patchEnvVar(key, null);
+        const data = await patchEnvVar(key, null);
         setVariables((prev) => {
           const n2 = { ...prev };
           delete n2[key];
           return n2;
         });
+        setOverrides((prev) => {
+          const n2 = { ...prev };
+          if (data.override) {
+            n2[key] = data.override;
+          } else {
+            delete n2[key];
+          }
+          return n2;
+        });
         setUnmaskedKeys((prev) => {
           const n2 = new Set(prev);
           n2.delete(key);
@@ -1154,8 +1179,17 @@
       const next = /^(true|yes)$/i.test(current) ? "false" : "true";
       setSaving(true);
       try {
-        await patchEnvVar(key, next);
+        const data = await patchEnvVar(key, next);
         setVariables((prev) => ({ ...prev, [key]: next }));
+        setOverrides((prev) => {
+          const n2 = { ...prev };
+          if (data.override) {
+            n2[key] = data.override;
+          } else {
+            delete n2[key];
+          }
+          return n2;
+        });
       } catch (e3) {
         showFlash("error", e3.message ?? "Failed to update");
       } finally {
@@ -1167,8 +1201,17 @@
       if (!key) return;
       setSaving(true);
       try {
-        await patchEnvVar(key, newValue);
+        const data = await patchEnvVar(key, newValue);
         setVariables((prev) => ({ ...prev, [key]: newValue }));
+        setOverrides((prev) => {
+          const n2 = { ...prev };
+          if (data.override) {
+            n2[key] = data.override;
+          } else {
+            delete n2[key];
+          }
+          return n2;
+        });
         setNewKey("");
         setNewValue("");
         showFlash("success", `${key} added`);
@@ -1191,7 +1234,7 @@
           delete n2[key];
           return n2;
         });
-        setVariables((prev) => {
+        const updater = (prev) => {
           const n2 = { ...prev };
           if (data.value === null) {
             delete n2[key];
@@ -1199,7 +1242,9 @@
             n2[key] = data.value;
           }
           return n2;
-        });
+        };
+        setVariables(updater);
+        setInitial(updater);
         showFlash("success", `${key} reset to original`);
       } catch (e3) {
         showFlash("error", e3.message ?? "Failed to reset");
@@ -1212,7 +1257,8 @@
       try {
         await resetAllEnvVars();
         setOverrides({});
-        await refresh();
+        const data = await refresh();
+        if (data) setInitial(data.variables);
         showFlash("success", "All overrides reset");
       } catch (e3) {
         showFlash("error", e3.message ?? "Failed to reset all");
@@ -1449,10 +1495,6 @@
                       override.value === "__profiler_deleted__" ? "(deleted)" : override.value,
                       " \xB7 original: ",
                       override.original ?? "(unset)"
-                    ] }),
-                    !override && wasModified && /* @__PURE__ */ u3("div", { style: "font-size:10px;color:#f59e0b;margin-top:2px;font-family:monospace;", children: [
-                      "was: ",
-                      initial[key]
                     ] })
                   ] }) : /* @__PURE__ */ u3(k, { children: [
                     /* @__PURE__ */ u3("div", { class: "profiler-flex", style: "align-items:center;gap:4px;", children: [
@@ -1461,7 +1503,9 @@
                         "span",
                         {
                           class: "profiler-text--xs profiler-text--mono",
-                          style: "word-break:break-all;flex:1;",
+                          onClick: () => !readOnly && !isEditing && startEdit(key),
+                          title: !readOnly ? "Click to edit" : void 0,
+                          style: `word-break:break-all;flex:1;${!readOnly ? "cursor:pointer;" : ""}`,
                           children: unmasked ? isQuoted(value) ? /* @__PURE__ */ u3(k, { children: [
                             /* @__PURE__ */ u3("span", { style: "color:var(--profiler-text-muted);opacity:0.5;", children: '"' }),
                             value,
@@ -1493,28 +1537,24 @@
                       override.value === "__profiler_deleted__" ? "(deleted)" : unmasked ? override.value : "\u2022".repeat(Math.min((override.value ?? "").length, 20)),
                       " \xB7 original: ",
                       override.original == null ? "(unset)" : unmasked ? override.original : "\u2022".repeat(Math.min(override.original.length, 20))
-                    ] }),
-                    !override && wasModified && /* @__PURE__ */ u3("div", { style: "font-size:10px;color:#f59e0b;margin-top:2px;font-family:monospace;", children: [
-                      "was: ",
-                      unmasked ? isQuoted(initial[key]) ? `"${initial[key]}"` : initial[key] : "\u2022".repeat(Math.min(initial[key].length, 20))
                     ] })
                   ] }) }),
                   !readOnly && /* @__PURE__ */ u3("td", { style: "text-align:right;white-space:nowrap;", children: isEditing ? /* @__PURE__ */ u3(k, { children: [
                     /* @__PURE__ */ u3("button", { onClick: saveEdit, disabled: saving, title: "Save", style: "background:none;border:none;cursor:pointer;color:var(--profiler-success,#22c55e);font-size:14px;padding:0 4px;", children: "\u2713" }),
                     /* @__PURE__ */ u3("button", { onClick: cancelEdit, disabled: saving, title: "Cancel", style: "background:none;border:none;cursor:pointer;color:var(--profiler-text-muted);font-size:14px;padding:0 4px;", children: "\u2717" })
                   ] }) : /* @__PURE__ */ u3(k, { children: [
-                    /* @__PURE__ */ u3("button", { onClick: () => startEdit(key), disabled: saving, style: "background:none;border:none;cursor:pointer;color:var(--profiler-accent);font-size:11px;padding:0 4px;", children: "Edit" }),
+                    /* @__PURE__ */ u3("button", { onClick: () => startEdit(key), disabled: saving, title: "Edit", style: "background:none;border:none;cursor:pointer;color:var(--profiler-accent);font-size:14px;padding:0 4px;", children: "\u270E" }),
                     override && /* @__PURE__ */ u3(
                       "button",
                       {
                         onClick: () => resetVar(key),
                         disabled: saving,
                         title: `Reset to original: ${override.original ?? "(unset)"}`,
-                        style: "background:none;border:none;cursor:pointer;color:var(--profiler-warning,#f59e0b);font-size:11px;padding:0 4px;",
-                        children: "\u21A9 Reset"
+                        style: "background:none;border:none;cursor:pointer;color:var(--profiler-warning,#f59e0b);font-size:14px;padding:0 4px;",
+                        children: "\u21A9"
                       }
                     ),
-                    /* @__PURE__ */ u3("button", { onClick: () => deleteVar(key), disabled: saving, style: "background:none;border:none;cursor:pointer;color:var(--profiler-error,#ef4444);font-size:11px;padding:0 4px;", children: "Delete" })
+                    /* @__PURE__ */ u3("button", { onClick: () => deleteVar(key), disabled: saving, title: "Delete", style: "background:none;border:none;cursor:pointer;color:var(--profiler-error,#ef4444);font-size:14px;padding:0 4px;", children: "\u2715" })
                   ] }) })
                 ] }, key);
               })

data/app/controllers/profiler/api/env_vars_controller.rb

@@ -22,13 +22,18 @@ module Profiler
         value = params[:value]
 
         if value.nil? || value.to_s.empty?
-          ENV.delete(key)
           Profiler.env_override_store.delete(key)
-          render json: { key: key, value: nil, deleted: true }
+          ENV.delete(key)
+          render json: { key: key, value: nil, deleted: true, override: Profiler.env_override_store.all_overrides[key] }
         else
+          current_original = Profiler.env_override_store.all_overrides.dig(key, "original")
+          if current_original == value.to_s
+            Profiler.env_override_store.reset(key)
+          else
+            Profiler.env_override_store.set(key, value.to_s)
+          end
           ENV[key] = value.to_s
-          Profiler.env_override_store.set(key, value.to_s)
-          render json: { key: key, value: ENV[key] }
+          render json: { key: key, value: ENV[key], override: Profiler.env_override_store.all_overrides[key] }
         end
       end
 

data/lib/profiler/middleware/profiler_middleware.rb

@@ -14,6 +14,11 @@ module Profiler
       def call(env)
         return @app.call(env) unless should_profile?(env)
 
+        status = nil
+        headers = nil
+        body = nil
+        collectors = nil
+
         profile = Models::Profile.new(build_request(env))
         Profiler::CurrentContext.token = profile.token
 
@@ -23,14 +28,12 @@ module Profiler
         # Store profile in env for collectors
         env["profiler.profile"] = profile
 
-        # Create and subscribe collectors
         collectors = create_collectors(profile)
         env["profiler.collectors"] = collectors
 
         # Measure memory before
         memory_before = current_memory if Profiler.configuration.track_memory
 
-        # Process request
         status, headers, body = @app.call(env)
 
         # Measure memory after
@@ -39,14 +42,11 @@ module Profiler
           profile.memory = memory_after - memory_before
         end
 
-        # Collect and buffer response body (avoids double-reading by ToolbarInjector)
         body_content = collect_body(body)
         body = [body_content]
 
-        # Finish profile
         profile.finish(status, headers)
 
-        # Store request and response bodies
         profile.set_bodies(
           request_body: req_body_raw,
           response_body: body_content,
@@ -54,7 +54,6 @@ module Profiler
           resp_content_type: (headers["content-type"] || headers["Content-Type"]).to_s
         )
 
-        # Collect data from all collectors
         collectors.each do |collector|
           begin
             collector.collect if collector.respond_to?(:collect)
@@ -64,14 +63,11 @@ module Profiler
           end
         end
 
-        # Store profile
         Profiler.storage.save(profile.token, profile)
         Profiler::CurrentContext.clear
 
-        # Add profiler token header
         headers["X-Profiler-Token"] = profile.token
 
-        # Inject toolbar if HTML response
         if html_response?(headers)
           nonce = env['action_dispatch.content_security_policy_nonce']
           body = ToolbarInjector.new(body, profile.token, nonce).inject
@@ -80,7 +76,9 @@ module Profiler
         [status, headers, body]
       rescue => e
         warn "Profiler error: #{e.message}\n#{e.backtrace.join("\n")}"
-        @app.call(env)
+        collectors&.each { |c| c.unsubscribe if c.respond_to?(:unsubscribe) }
+        Profiler::CurrentContext.clear
+        status ? [status, headers, body || []] : @app.call(env)
       end
 
       private

data/lib/profiler/storage/sqlite_store.rb

@@ -19,6 +19,9 @@ module Profiler
 
         @db = SQLite3::Database.new(db_path.to_s)
         @db.results_as_hash = true
+        @db.busy_timeout = 5000
+        @db.execute("PRAGMA journal_mode=WAL")
+        @db.execute("PRAGMA synchronous=NORMAL")
 
         @blob_store = BlobStore.new(blob_path.to_s)
 

data/lib/profiler/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Profiler
-  VERSION = "0.19.0"
+  VERSION = "0.19.1"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rails-profiler
 version: !ruby/object:Gem::Version
-  version: 0.19.0
+  version: 0.19.1
 platform: ruby
 authors:
 - Sébastien Duplessy
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-04-15 00:00:00.000000000 Z
+date: 2026-04-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails