@lonca/trendyol 1.0.1 → 1.2.0

package/dist/testing.cjs

@@ -4,6 +4,11 @@ var core = require('@lonca/core');
 
 // src/resources/brands.ts
 var DEFAULT_PAGE_SIZE = 1e3;
+function toBrand(node) {
+  const brand = { id: String(node.id), name: node.name };
+  if (typeof node.luxe === "boolean") brand.luxe = node.luxe;
+  return brand;
+}
 var BrandsResource = class {
   constructor(transport, limiter) {
     this.transport = transport;
@@ -14,6 +19,14 @@ var BrandsResource = class {
   /**
    * List Trendyol brands, one page at a time.
    *
+   * **Next-page heuristic.** Trendyol's brand list carries no page count (neither the docs
+   * nor the prod wire have `totalPages` / `totalElements` — the response is just
+   * `{ brands: [...] }`). So when `totalPages` is absent, a **full page** (at least `limit`
+   * brands — Trendyol may send ~1000 even for a smaller `limit`) means "there may be more"
+   * and sets `nextCursor`; a short page is the last one. When the
+   * total is an exact multiple of the page size, `paginate()` makes one extra request that
+   * comes back empty and stops there. If Trendyol ever sends `totalPages`, it wins.
+   *
    * @example
    * ```ts
    * import { paginate } from '@lonca/core';
@@ -31,8 +44,10 @@ var BrandsResource = class {
       query: { page, size },
       rateLimiter: this.limiter
     });
-    const items = data.brands.map((b) => ({ id: String(b.id), name: b.name }));
-    const nextCursor = page + 1 < data.totalPages ? String(page + 1) : void 0;
+    const brands = data?.brands ?? [];
+    const items = brands.map(toBrand);
+    const hasMore = typeof data?.totalPages === "number" ? page + 1 < data.totalPages : size > 0 && brands.length >= size;
+    const nextCursor = hasMore ? String(page + 1) : void 0;
     return nextCursor !== void 0 ? { items, nextCursor } : { items };
   }
   /**
@@ -56,7 +71,7 @@ var BrandsResource = class {
       query: { name },
       rateLimiter: this.limiter
     });
-    return (data ?? []).map((b) => ({ id: String(b.id), name: b.name }));
+    return (data ?? []).map(toBrand);
   }
 };
 function normalizeCategory(node) {
@@ -69,7 +84,6 @@ function normalizeCategory(node) {
 }
 function normalizeAttribute(node) {
   const attr = node.attribute ?? { id: 0, name: "" };
-  const rawValues = node.attributeValues ?? [];
   const out = {
     id: String(attr.id),
     name: attr.name,
@@ -77,7 +91,8 @@ function normalizeAttribute(node) {
     allowCustom: !!node.allowCustom,
     varianter: !!node.varianter,
     slicer: !!node.slicer,
-    values: rawValues.map((v) => ({ id: String(v.id), name: v.name }))
+    // Deprecated: normally empty — see `attributeValues` on the node above.
+    values: (node.attributeValues ?? []).map((v) => ({ id: String(v.id), name: v.name }))
   };
   if (node.categoryId !== void 0) {
     out.categoryId = String(node.categoryId);
@@ -139,8 +154,8 @@ var CategoriesResource = class {
   /**
    * Fetch the allowed values for a single category attribute (paginated).
    *
-   * `getCategoryAttributes` returns attribute metadata + flags but typically
-   * omits the value catalog. Use this method to fetch the catalog for an
+   * `getCategoryAttributes` returns attribute metadata + flags only, never the
+   * value catalog (docs and prod wire agree). Use this method to fetch the catalog for an
    * attribute when `allowCustom` is `false` and you need to map your data
    * onto Trendyol's accepted values.
    *
@@ -1021,7 +1036,7 @@ var LocationsResource = class {
   }
   // ─── Domestic (TR / AZ) ───────────────────────────────────────────────
   async getTurkeyCities() {
-    return this.cities(`/integration/member/countries/domestic/TR/cities`);
+    return this.cities(`/integration/member/countries/domestic/TR/cities`, "TR");
   }
   /**
    * List districts for a Turkish city. **Pass the city `id`** (`City.id`) — the
@@ -1043,7 +1058,7 @@ var LocationsResource = class {
     );
   }
   async getAzerbaijanCities() {
-    return this.cities(`/integration/member/countries/domestic/AZ/cities`);
+    return this.cities(`/integration/member/countries/domestic/AZ/cities`, "AZ");
   }
   /** List districts for an Azerbaijani city. **Pass the city `id`** (`City.id`), not `code`. */
   async getAzerbaijanDistricts(cityId) {
@@ -1053,7 +1068,10 @@ var LocationsResource = class {
   }
   // ─── International (GULF / CEE) ───────────────────────────────────────
   async getCitiesByCountry(countryCode) {
-    return this.cities(`/integration/member/countries/${encodeURIComponent(countryCode)}/cities`);
+    return this.cities(
+      `/integration/member/countries/${encodeURIComponent(countryCode)}/cities`,
+      countryCode
+    );
   }
   async getDistrictsByCity(countryCode, cityId) {
     return this.districts(
@@ -1061,7 +1079,8 @@ var LocationsResource = class {
     );
   }
   // ─── Shared paginators ────────────────────────────────────────────────
-  async cities(path) {
+  /** `countryCode` is the lookup's country — city rows do not carry one on the wire. */
+  async cities(path, countryCode) {
     const data = await this.transport.request({
       method: "GET",
       path,
@@ -1071,7 +1090,7 @@ var LocationsResource = class {
       id: node.id !== void 0 ? String(node.id) : void 0,
       code: String(node.code ?? node.id ?? ""),
       name: node.name,
-      countryCode: node.countryCode
+      countryCode
     }));
   }
   async districts(path) {
@@ -1083,8 +1102,7 @@ var LocationsResource = class {
     return n(data, (node) => ({
       id: node.id !== void 0 ? String(node.id) : void 0,
       code: String(node.code ?? node.id ?? ""),
-      name: node.name,
-      cityCode: node.cityCode !== void 0 ? String(node.cityCode) : void 0
+      name: node.name
     }));
   }
   async neighborhoods(path) {
@@ -1093,12 +1111,16 @@ var LocationsResource = class {
       path,
       rateLimiter: this.limiter
     });
-    return n(data, (node) => ({
-      id: node.id !== void 0 ? String(node.id) : void 0,
-      code: String(node.code ?? node.id ?? ""),
-      name: node.name,
-      districtCode: node.districtCode !== void 0 ? String(node.districtCode) : void 0
-    }));
+    return n(data, (node) => {
+      const id = node.id !== void 0 ? String(node.id) : void 0;
+      return {
+        id,
+        // Neighborhood rows have no code; `code` mirrors the id (as it always did in practice).
+        code: id ?? "",
+        name: node.name,
+        postCode: typeof node.postCode === "string" ? node.postCode : void 0
+      };
+    });
   }
 };
 var MAX_PAGE_SIZE = 200;
@@ -2451,29 +2473,35 @@ var VALID_TYPES = /* @__PURE__ */ new Set([
   "WAREHOUSE"
 ]);
 function normalizeAddressType(raw) {
-  if (raw && VALID_TYPES.has(raw)) {
-    return raw;
+  const upper = typeof raw === "string" ? raw.toUpperCase() : void 0;
+  if (upper && VALID_TYPES.has(upper)) {
+    return upper;
   }
-  const stripped = raw?.replace(/_ADDRESS$/, "");
+  const stripped = upper?.replace(/_ADDRESS$/, "");
   if (stripped && VALID_TYPES.has(stripped)) {
     return stripped;
   }
   return "SHIPMENT";
 }
+function optionalString(value) {
+  return typeof value === "number" || typeof value === "string" ? String(value) : void 0;
+}
 function normalizeAddress2(node) {
   return {
     id: String(node.id),
-    name: node.name,
     addressType: normalizeAddressType(node.addressType),
     isShipmentAddress: node.isShipmentAddress ?? false,
     isReturningAddress: node.isReturningAddress ?? false,
     isInvoiceAddress: node.isInvoiceAddress ?? false,
     isDefault: node.isDefault ?? false,
     address: node.address,
+    fullAddress: node.fullAddress,
+    country: node.country,
     city: node.city,
+    cityCode: optionalString(node.cityCode),
     district: node.district,
-    postCode: node.postCode,
-    fullName: node.fullName
+    districtId: optionalString(node.districtId),
+    postCode: node.postCode
   };
 }
 var DEFAULT_CACHE_TTL_MS = 60 * 60 * 1e3;