node-autotask 0.39.1 → 0.39.2

package/CHANGELOG.md

@@ -5,6 +5,19 @@ All notable changes to this project are documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.39.2] - 2026-10-30
+
+Patch: follow-ups to the 0.39.0 / 0.39.1 label work (#302, #303, #304). The plan hash is unchanged (`9bb87de7...`); no capability-plan change, so the MCP server needs no re-pin.
+
+### Fixed
+- **#302: `fetchMany` / `fetchByIds` names merge.** `names.reasons` is keyed `<resource>.<field>` (`tickets.status`, `contracts.status`), so two resources sharing a field name no longer overwrite each other's reason. **Consumer-visible:** this applies to every `fetchMany` / `fetchByIds` result, including one whose items all name a single resource, so a reader of `names.reasons.status` now reads `names.reasons['tickets.status']`. Every failing resource's catalog error is kept in the new `names.catalogErrors` (keyed by resource); `names.catalogError` stays and is the first of them. The single-resource operations (`<resource>.query`, `aggregate`) are unchanged: `reasons` is still keyed by field.
+- **#303: composite name paths no longer read a foreign-key id of 0.** The full-context composites (`tickets.fullContext`, `companies.fullProfile`, ...) planned a read for `contractID: 0` and the like; 0 is the vendor's "no reference", so the sibling is `null` with no read, as on the row path.
+- **#304: a failed field-catalog read is no longer reported as "retrying will not help".** On the composite path (and on the row path when the second, field-filtered catalog read fails after the first succeeded) the picklist labels are `null` with `PICKLIST_CATALOG_UNAVAILABLE_NOTE` and the section / `names` carries `catalogError`. The "retrying will not help" honesty note (`FIELD_CATALOG_HONESTY_NOTE`) is now served only for a picklist the vendor does not serve. The pinned composite reasons moved accordingly (new optional `catalogError` on the composite `names` section).
+
+### Changed
+- **#303: 0.39.0 -> 0.39.1 behaviour note.** A value of 0 on a picklist whose values the vendor does not serve now gets a reason row (the honesty note); 0.39.0 gave none. A 0 on a picklist that IS served but does not define 0 still gets no reason row: the SDK treats a 0 absent from the picklist as "none". A defined 0 that the vendor hides from the API user therefore reads as `null` with no reason; the SDK cannot tell the two apart.
+- **#303: `docs/API.md` §11.27 response-size figure** now uses the live measurement (31 siblings per row, 341,178 B -> 477,325 B, +40%) instead of "~50%, 19 siblings".
+
 ## [0.39.1] - Unreleased
 
 Patch: label follow-ups from QA of #283. The plan hash is unchanged (`9bb87de7...`); only registry text in `capabilities.json` moved, so the MCP server needs no plan re-pin.

package/dist/{chunk-LB34VFQY.cjs → chunk-5QS3VT34.cjs}

@@ -20557,7 +20557,7 @@ function createNamePlan(labelFields = [], labelResource) {
       ...lineOfBusinessContractId !== void 0 ? { lineOfBusinessContractId } : {}
     }),
     addResource: (value) => {
-      if (!isVendorId(value)) return;
+      if (!isVendorId(value) || Number(value) === 0) return;
       const k = String(value);
       if (!resSeen.has(k)) {
         resSeen.add(k);
@@ -20565,7 +20565,7 @@ function createNamePlan(labelFields = [], labelResource) {
       }
     },
     addFk: (table, value) => {
-      if (!isVendorId(value)) return;
+      if (!isVendorId(value) || Number(value) === 0) return;
       const k = `${table}:${String(value)}`;
       if (!fkSeen.has(k)) {
         fkSeen.add(k);
@@ -20787,23 +20787,30 @@ async function resolveCompositeNames(http, plan, opts = {}) {
   const picklists = {};
   if (plan.labelFields.length > 0) {
     let usable = true;
+    let failure;
     try {
       const labelResource = _nullishCoalesce(plan.labelResource, () => ( "tickets"));
       const catalog = await fieldCatalog(client, { resources: [labelResource], labels: true, fields: [...plan.labelFields] });
       const row = catalog.rows.find((r) => r.resource === labelResource);
       if (row === void 0 || row.error !== null || row.fields === void 0) {
         usable = false;
+        failure = _nullishCoalesce(_optionalChain([row, 'optionalAccess', _197 => _197.error]), () => ( { code: "INVALID_RESPONSE", message: `fieldCatalog: no ${labelResource} row` }));
       } else {
         for (const f of plan.labelFields) {
           const frow = row.fields.find((x) => x.name === f);
-          picklists[f] = _optionalChain([frow, 'optionalAccess', _197 => _197.picklistValues]);
+          picklists[f] = _optionalChain([frow, 'optionalAccess', _198 => _198.picklistValues]);
         }
       }
     } catch (err) {
-      if (isAutotaskError(err) && err.code !== "ABORTED") usable = false;
-      else throw err;
+      if (isAutotaskError(err) && err.code !== "ABORTED") {
+        usable = false;
+        failure = { code: err.code, message: err.message };
+      } else throw err;
+    }
+    if (!usable) {
+      section.catalogError = failure;
+      for (const f of plan.labelFields) section.reasons[f] = PICKLIST_CATALOG_UNAVAILABLE_NOTE;
     }
-    if (!usable) for (const f of plan.labelFields) section.reasons[f] = FIELD_CATALOG_HONESTY_NOTE;
   }
   let lineOfBusiness2 = null;
   const lobId = _nullishCoalesce(plan.lineOfBusinessId, () => ( contractLobPointer));
@@ -20830,13 +20837,13 @@ async function resolveCompositeNames(http, plan, opts = {}) {
   return {
     section,
     lineOfBusiness: lineOfBusiness2,
-    resourceName: (id) => _nullishCoalesce(_optionalChain([section, 'access', _198 => _198.rows, 'access', _199 => _199[`resource:${id}`], 'optionalAccess', _200 => _200.name]), () => ( null)),
-    recordName: (table, id) => _nullishCoalesce(_optionalChain([section, 'access', _201 => _201.rows, 'access', _202 => _202[`${table}:${id}`], 'optionalAccess', _203 => _203.name]), () => ( null)),
+    resourceName: (id) => _nullishCoalesce(_optionalChain([section, 'access', _199 => _199.rows, 'access', _200 => _200[`resource:${id}`], 'optionalAccess', _201 => _201.name]), () => ( null)),
+    recordName: (table, id) => _nullishCoalesce(_optionalChain([section, 'access', _202 => _202.rows, 'access', _203 => _203[`${table}:${id}`], 'optionalAccess', _204 => _204.name]), () => ( null)),
     labelFor: (field, value) => {
       if (value === null || value === void 0) return null;
       const values = picklists[field];
       if (values === void 0) {
-        section.reasons[field] = FIELD_CATALOG_HONESTY_NOTE;
+        section.reasons[field] = section.catalogError !== void 0 ? PICKLIST_CATALOG_UNAVAILABLE_NOTE : FIELD_CATALOG_HONESTY_NOTE;
         return null;
       }
       const hit = values.find((v) => String(v.value) === String(value));
@@ -20884,11 +20891,11 @@ async function labelRows(client, resourceKey, rows) {
     const row = catalog.rows.find((r) => r.resource === resourceKey);
     const known = _nullishCoalesce(lastPicklists.get(client), () => ( /* @__PURE__ */ new Map()));
     lastPicklists.set(client, known);
-    if (_optionalChain([row, 'optionalAccess', _204 => _204.fields]) === void 0) {
+    if (_optionalChain([row, 'optionalAccess', _205 => _205.fields]) === void 0) {
       const last = known.get(resourceKey);
       unavailable = (_nullishCoalesce(last, () => ( []))).filter((f) => candidates.includes(f));
       if (last === void 0 || unavailable.length > 0) {
-        catalogError = _nullishCoalesce(_optionalChain([row, 'optionalAccess', _205 => _205.error]), () => ( { code: "INVALID_RESPONSE", message: `fieldCatalog: no ${resourceKey} row` }));
+        catalogError = _nullishCoalesce(_optionalChain([row, 'optionalAccess', _206 => _206.error]), () => ( { code: "INVALID_RESPONSE", message: `fieldCatalog: no ${resourceKey} row` }));
       }
     } else {
       const all = row.fields.filter((f) => f.isPickList).map((f) => f.name);
@@ -20911,6 +20918,11 @@ async function labelRows(client, resourceKey, rows) {
   }
   const plan = builder.plan();
   const names = plan.resourceIds.length === 0 && plan.recordFks.length === 0 && picklistFields.length === 0 ? void 0 : await resolveCompositeNames(client.http, plan);
+  const secondReadFailed = _optionalChain([names, 'optionalAccess', _207 => _207.section, 'access', _208 => _208.catalogError]);
+  if (secondReadFailed !== void 0) {
+    catalogError ??= secondReadFailed;
+    unavailable = [.../* @__PURE__ */ new Set([...unavailable, ...picklistFields])];
+  }
   const out = rows.map((row) => {
     const r = { ...row };
     for (const [field, value] of Object.entries(row)) {
@@ -20926,8 +20938,8 @@ async function labelRows(client, resourceKey, rows) {
     return r;
   });
   const capped = names === void 0 ? 0 : Object.values(names.section.rows).filter((row) => row.capped).length;
-  const unresolved = Object.keys(_nullishCoalesce(_optionalChain([names, 'optionalAccess', _206 => _206.section, 'access', _207 => _207.reasons]), () => ( {})));
-  const reasons = { ..._optionalChain([names, 'optionalAccess', _208 => _208.section, 'access', _209 => _209.reasons]) };
+  const unresolved = Object.keys(_nullishCoalesce(_optionalChain([names, 'optionalAccess', _209 => _209.section, 'access', _210 => _210.reasons]), () => ( {})));
+  const reasons = { ..._optionalChain([names, 'optionalAccess', _211 => _211.section, 'access', _212 => _212.reasons]) };
   for (const f of unavailable) reasons[f] = PICKLIST_CATALOG_UNAVAILABLE_NOTE;
   if (capped === 0 && unresolved.length === 0 && catalogError === void 0) return { rows: out };
   const parts = [];
@@ -20956,22 +20968,27 @@ async function labelFetchResults(client, result) {
   });
   const results = [...result.results];
   const summaries = [];
+  const reasons = {};
+  const catalogErrors = {};
   for (const [resource, indices] of byResource) {
     const labelled = await labelRows(client, resource, indices.map((i) => results[i].value));
     indices.forEach((idx, j) => {
       results[idx] = { ...results[idx], value: labelled.rows[j] };
     });
-    if (labelled.names !== void 0) summaries.push(labelled.names);
+    if (labelled.names !== void 0) {
+      summaries.push(labelled.names);
+      for (const [field, why] of Object.entries(_nullishCoalesce(labelled.names.reasons, () => ( {})))) reasons[`${resource}.${field}`] = why;
+      if (labelled.names.catalogError !== void 0) catalogErrors[resource] = labelled.names.catalogError;
+    }
   }
   if (summaries.length === 0) return { ...result, results };
-  const reasons = Object.assign({}, ...summaries.map((s) => _nullishCoalesce(s.reasons, () => ( {}))));
-  const catalogError = _optionalChain([summaries, 'access', _210 => _210.find, 'call', _211 => _211((s) => s.catalogError !== void 0), 'optionalAccess', _212 => _212.catalogError]);
+  const catalogError = Object.values(catalogErrors)[0];
   const names = {
     truncated: summaries.some((s) => s.truncated),
     capped: summaries.reduce((n, s) => n + s.capped, 0),
     note: summaries.map((s) => s.note).join("; "),
     ...Object.keys(reasons).length > 0 ? { reasons } : {},
-    ...catalogError !== void 0 ? { catalogError } : {}
+    ...catalogError !== void 0 ? { catalogError, catalogErrors } : {}
   };
   return { ...result, results, names };
 }

package/dist/{chunk-LDTJNLX7.js → chunk-GSGP2B4I.js}

@@ -30,7 +30,7 @@ import {
   unwrapItems,
   unwrapOperation,
   vendorId
-} from "./chunk-LN3HQQBR.js";
+} from "./chunk-UNOQESFU.js";
 import {
   CAPABILITIES
 } from "./chunk-GDSCYY2G.js";