@aibranch/llm-meta-widget 0.8.4 → 0.8.6

package/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) {

package/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"
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@aibranch/llm-meta-widget",
-  "version": "0.8.4",
+  "version": "0.8.6",
   "//version": "Must stay in step with lib/llm_meta_widget/version.rb. Two independently versioned copies of the same client is the drift docs/custom-element-distribution.md warns about: bump both, release both.",
   "description": "Embeddable browser chat widget: a custom element that discovers tools at page, origin and hub scope and dispatches each tool call to the right one. Two lines of HTML on any host, Rails or not.",
   "keywords": [