llm_meta_widget 0.8.4 → 0.8.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0a3adfcaccf5824c1c15fc457ee01716fa1df373d21f4734b68096ef08effed1
-  data.tar.gz: 22c9d53cbfed853404279136f90da67410608c1f87d9a076c999813481965d9b
+  metadata.gz: 83ad21e8631c42b769a30bc71493cf7b73590f16e62a6bdd5747d147e4aeb9c5
+  data.tar.gz: e65e1289d90bff0db65c8905f4018f2eddaa48e680f1d17afe709a9ac2a59a5f
 SHA512:
-  metadata.gz: 96a6c2e40b6477807230de39d3eb02668353ec1c29d9155ff083d3e2afdd5149ce58a5199d2d912e1f311d87be3d6c4cd1c2c67d5f5c7e36f32b967a2b80f567
-  data.tar.gz: ca5587496e4b076f9cb0215f61573d09f26a6068a8c0377b4bc4e542577223c94d0f87a9ac81fcb7d3afff484ed88df8aadec2038ec3b909e9e579527bdf82c2
+  metadata.gz: de457ef4d34b201e3d0cfd5e53b3171fa1e9ed88aaa99da68a92fe00a0fe82769942ac25783982ba9397544e2018e645755b6a5f3cc5cfa62d11d2e173648c3b
+  data.tar.gz: 3d28002e0c0d593cf6b287943e32863a341aaeee82da14b0fd01aa23e46cad77722c434bc06e6b19c792c30dd95c61bd90e0ccb6de694565c845a4ab69ed223a

data/app/assets/javascripts/llm_meta_widget/element.js

@@ -408,6 +408,13 @@ function boot(cfg) {
   					(payload.llms || []).forEach(function(llm) {
   						if (llm.family !== "ollama") return;
   						(llm.available_models || []).forEach(function(m) {
+  							// This widget exists to call tools — page actions and MCP. A model
+  							// that declares no tool support fails hard the moment any tool is
+  							// declared ("<model> does not support tools"), so offering it in the
+  							// picker is a trap: the visitor picks it and the next turn errors.
+  							// `=== false` on purpose — a hub that omits the field is older, not
+  							// a claim of no support, and must keep working.
+  							if (m.supports_tools === false) return;
   							if (anyAllowedByAllowlist(m.value, MODEL_ALLOWLIST)) {
   								flat.push({ value: m.value, label: m.label });
   							}
@@ -616,7 +623,7 @@ function boot(cfg) {
   					read:        readMcpResource,
   					onSkip: function(plan) {
   						console.info("[llm_meta_widget] not attaching " + plan.uri +
-  						             " (" + plan.reason + ", sizeBytes=" + plan.sizeBytes + ")");
+  						             " (" + plan.reason + ", size=" + plan.size + ")");
   					}
   				});
   				// A volatile resource comes back with no context — it is read

data/app/assets/javascripts/llm_meta_widget/llm-meta-widget.js

@@ -550,23 +550,33 @@ function parseSseFrame(raw) {
 }
 var STATIC_PRIMITIVES_META = "io.modelcontextprotocol/static-primitives";
 var VOLATILITIES = ["stable", "volatile"];
+var usableSize = (n) => typeof n === "number" && isFinite(n) ? n : null;
 function resourceHints(entry) {
   const meta = entry && entry._meta && entry._meta[STATIC_PRIMITIVES_META] || {};
   return {
     // A non-numeric or absent size means "unknown", never 0 — 0 would read
     // as a free resource and sail through every budget check.
-    sizeBytes: typeof meta.sizeBytes === "number" && isFinite(meta.sizeBytes) ? meta.sizeBytes : null,
+    // Base `size` wins; `_meta.sizeBytes` is the earlier draft's spelling and
+    // is accepted so a new client keeps its budget gate against an old server.
+    size: usableSize(entry && entry.size) ?? usableSize(meta.sizeBytes),
+    // Deprecated alias, kept for one release so renaming the field is not a
+    // breaking change: orchestrator.js is a published entry point, so anything
+    // it returns is public API. Dropped at 0.9.0 together with the
+    // `_meta.sizeBytes` fallback above — one deprecation, one bump.
+    get sizeBytes() {
+      return this.size;
+    },
     volatility: VOLATILITIES.indexOf(meta.volatility) === -1 ? "stable" : meta.volatility,
     autoAttach: typeof meta.autoAttach === "boolean" ? meta.autoAttach : true
   };
 }
 function planResourceAttachment(entry, budgetBytes) {
-  const { sizeBytes, volatility, autoAttach } = resourceHints(entry);
-  const base = { sizeBytes, volatility, uri: entry && entry.uri };
+  const { size, volatility, autoAttach } = resourceHints(entry);
+  const base = { size, sizeBytes: size, volatility, uri: entry && entry.uri };
   if (!autoAttach) {
     return { ...base, fetch: false, autoAttach: false, reason: "not-auto-attach" };
   }
-  if (sizeBytes !== null && sizeBytes > budgetBytes) {
+  if (size !== null && size > budgetBytes) {
     return { ...base, fetch: false, autoAttach: false, reason: "over-budget" };
   }
   return {
@@ -575,7 +585,7 @@ function planResourceAttachment(entry, budgetBytes) {
     autoAttach: true,
     // Unknown size still gets fetched, then trimmed — the trim is the
     // safety net for servers that do not declare a size.
-    reason: sizeBytes === null ? "size-unknown" : "within-budget"
+    reason: size === null ? "size-unknown" : "within-budget"
   };
 }
 function trimResourceText(text, maxBytes) {
@@ -3868,6 +3878,7 @@ function boot(cfg) {
         (payload.llms || []).forEach(function(llm) {
           if (llm.family !== "ollama") return;
           (llm.available_models || []).forEach(function(m) {
+            if (m.supports_tools === false) return;
             if (anyAllowedByAllowlist(m.value, MODEL_ALLOWLIST)) {
               flat.push({ value: m.value, label: m.label });
             }
@@ -4030,7 +4041,7 @@ function boot(cfg) {
           list: listMcpResources,
           read: readMcpResource,
           onSkip: function(plan) {
-            console.info("[llm_meta_widget] not attaching " + plan.uri + " (" + plan.reason + ", sizeBytes=" + plan.sizeBytes + ")");
+            console.info("[llm_meta_widget] not attaching " + plan.uri + " (" + plan.reason + ", size=" + plan.size + ")");
           }
         });
         if (loaded && loaded.plan && loaded.plan.fetch) {

data/app/assets/javascripts/llm_meta_widget/orchestrator.js

@@ -819,8 +819,13 @@ function parseSseFrame(raw) {
 // follow-on). Three optional fields a server may declare on a resources/list
 // entry:
 //
-//   sizeBytes  - byte length of the payload resources/read would return, so a
+//   size       - byte count of the payload resources/read would return, so a
 //                client can decide whether to attach it BEFORE fetching it.
+//                This is the BASE Resource field, not an extension field: the
+//                extension only strengthens it to "MUST be exact when present".
+//                Read from `_meta.sizeBytes` too, which is where the earlier
+//                draft put it — servers deployed against that draft (PD among
+//                them) are still emitting it.
 //   volatility - "stable" (content fixed) or "volatile" (varies between
 //                turns, so a client re-reads it each turn).
 //   autoAttach - may a client attach this without the user asking for it?
@@ -837,12 +842,21 @@ export const STATIC_PRIMITIVES_META = "io.modelcontextprotocol/static-primitives
 
 const VOLATILITIES = [ "stable", "volatile" ]
 
+const usableSize = (n) => (typeof n === "number" && isFinite(n) ? n : null)
+
 export function resourceHints(entry) {
   const meta = (entry && entry._meta && entry._meta[STATIC_PRIMITIVES_META]) || {}
   return {
     // A non-numeric or absent size means "unknown", never 0 — 0 would read
     // as a free resource and sail through every budget check.
-    sizeBytes: typeof meta.sizeBytes === "number" && isFinite(meta.sizeBytes) ? meta.sizeBytes : null,
+    // Base `size` wins; `_meta.sizeBytes` is the earlier draft's spelling and
+    // is accepted so a new client keeps its budget gate against an old server.
+    size: usableSize(entry && entry.size) ?? usableSize(meta.sizeBytes),
+    // Deprecated alias, kept for one release so renaming the field is not a
+    // breaking change: orchestrator.js is a published entry point, so anything
+    // it returns is public API. Dropped at 0.9.0 together with the
+    // `_meta.sizeBytes` fallback above — one deprecation, one bump.
+    get sizeBytes() { return this.size },
     volatility: VOLATILITIES.indexOf(meta.volatility) === -1 ? "stable" : meta.volatility,
     autoAttach: typeof meta.autoAttach === "boolean" ? meta.autoAttach : true
   }
@@ -851,13 +865,16 @@ export function resourceHints(entry) {
 // The pre-flight decision, made from the resources/list entry alone.
 // `fetch: false` means the bytes never cross the wire at all.
 export function planResourceAttachment(entry, budgetBytes) {
-  const { sizeBytes, volatility, autoAttach } = resourceHints(entry)
-  const base = { sizeBytes, volatility, uri: entry && entry.uri }
+  const { size, volatility, autoAttach } = resourceHints(entry)
+  // `sizeBytes` is the deprecated alias again: this function is exported too,
+  // and it returned that name before the field was aligned with the base
+  // Resource interface. Dropped at 0.9.0 with the rest.
+  const base = { size, sizeBytes: size, volatility, uri: entry && entry.uri }
 
   if (!autoAttach) {
     return { ...base, fetch: false, autoAttach: false, reason: "not-auto-attach" }
   }
-  if (sizeBytes !== null && sizeBytes > budgetBytes) {
+  if (size !== null && size > budgetBytes) {
     // The client's budget overrides the server's willingness — which is why
     // autoAttach has to be a boolean here rather than a restatement of a hint.
     return { ...base, fetch: false, autoAttach: false, reason: "over-budget" }
@@ -868,7 +885,7 @@ export function planResourceAttachment(entry, budgetBytes) {
     autoAttach: true,
     // Unknown size still gets fetched, then trimmed — the trim is the
     // safety net for servers that do not declare a size.
-    reason: sizeBytes === null ? "size-unknown" : "within-budget"
+    reason: size === null ? "size-unknown" : "within-budget"
   }
 }
 

data/lib/llm_meta_widget/version.rb

@@ -1,3 +1,3 @@
 module LlmMetaWidget
-  VERSION = "0.8.4"
+  VERSION = "0.8.6"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: llm_meta_widget
 version: !ruby/object:Gem::Version
-  version: 0.8.4
+  version: 0.8.6
 platform: ruby
 authors:
 - jdkim