fix(asin): address review feedback

- Describe Read Offers without Prime, which the API does not return
- Keep response mappers with a single caller in that tool's file
- Document the AsinIcon export
- Read the operation from the mapper context so agent tools coerce Days too

Author: Josselin Liebe

apps/docs/components/icons.tsx

@@ -10460,6 +10460,7 @@ export function PlaneIcon(props: SVGProps<SVGSVGElement>) {
   )
 }
 
+/** Asin logo, as used by the Asin block. */
 export function AsinIcon(props: SVGProps<SVGSVGElement>) {
   return (
     <svg viewBox='0 0 32 32' xmlns='http://www.w3.org/2000/svg' {...props}>

apps/docs/content/docs/integrations/asin.mdx

@@ -60,7 +60,7 @@ Search an Amazon store and get up to 60 results, sponsored ones included, with A
 
 ### Asin Read Offers
 
-Read every offer for an ASIN from Amazon now (uses a credit): seller, price, Prime and FBA, delivery, with Amazon's own price and the lowest new and used offers.
+Read every offer for an ASIN from Amazon now (uses a credit): seller, price, whether Amazon ships it (FBA), delivery, with Amazon's own price and the lowest new and used offers.
 
 <ReferenceDetails>
 

apps/sim/blocks/blocks/asin.ts

@@ -112,9 +112,10 @@ export const AsinBlock: BlockConfig = {
     ],
     config: {
       tool: (params) => `asin_${params.operation}`,
-      params: (params) => {
+      params: (params, context) => {
+        const operation = context?.operation ?? params.operation
         const result: Record<string, unknown> = {}
-        if (params.operation === 'get_price_history') {
+        if (operation === 'get_price_history') {
           result.days = parseOptionalNumberInput(params.days, 'Days', { integer: true, min: 1 })
         }
         return result

apps/sim/components/icons.tsx

@@ -10460,6 +10460,7 @@ export function PlaneIcon(props: SVGProps<SVGSVGElement>) {
   )
 }
 
+/** Asin logo, as used by the Asin block. */
 export function AsinIcon(props: SVGProps<SVGSVGElement>) {
   return (
     <svg viewBox='0 0 32 32' xmlns='http://www.w3.org/2000/svg' {...props}>

apps/sim/tools/asin/get_price_history.ts

@@ -1,14 +1,26 @@
-import { toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
-import { toArray, toRecord } from '@sim/utils/object'
+import { toBooleanOrNull, toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
+import { toArray, toRecord, toRecordOrNull } from '@sim/utils/object'
 import type {
   AsinGetPriceHistoryParams,
   AsinGetPriceHistoryResponse,
   AsinPricePoint,
 } from '@/tools/asin/types'
 import { ASIN_PRICE_POINT_OUTPUT_PROPERTIES } from '@/tools/asin/types'
-import { mapAsinPricePoint } from '@/tools/asin/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/** Reads a `AsinPricePoint` from an untyped Asin API payload. */
+function mapAsinPricePoint(value: unknown): AsinPricePoint | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    price: toNumberOrNull(record.price),
+    inStock: toBooleanOrNull(record.in_stock),
+    coupon: toStringOrNull(record.coupon),
+    seller: toStringOrNull(record.seller),
+    checkedAt: toStringOrNull(record.checked_at),
+  }
+}
+
 export const asinGetPriceHistoryTool: ToolConfig<
   AsinGetPriceHistoryParams,
   AsinGetPriceHistoryResponse

apps/sim/tools/asin/read_offers.ts

@@ -9,7 +9,7 @@ export const asinReadOffersTool: ToolConfig<AsinReadOffersParams, AsinReadOffers
   id: 'asin_read_offers',
   name: 'Asin Read Offers',
   description:
-    "Read every offer for an ASIN from Amazon now (uses a credit): seller, price, Prime and FBA, delivery, with Amazon's own price and the lowest new and used offers.",
+    "Read every offer for an ASIN from Amazon now (uses a credit): seller, price, whether Amazon ships it (FBA), delivery, with Amazon's own price and the lowest new and used offers.",
   version: '1.0.0',
 
   params: {

apps/sim/tools/asin/search_products.ts

@@ -1,13 +1,28 @@
-import { toArray, toRecord } from '@sim/utils/object'
+import { toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
+import { toArray, toRecord, toRecordOrNull } from '@sim/utils/object'
 import type {
   AsinSearchProductsParams,
   AsinSearchProductsResponse,
   AsinSearchResult,
 } from '@/tools/asin/types'
 import { ASIN_SEARCH_RESULT_OUTPUT_PROPERTIES } from '@/tools/asin/types'
-import { mapAsinSearchResult } from '@/tools/asin/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/** Reads a `AsinSearchResult` from an untyped Asin API payload. */
+function mapAsinSearchResult(value: unknown): AsinSearchResult | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    asin: toStringOrNull(record.asin),
+    title: toStringOrNull(record.title),
+    image: toStringOrNull(record.image),
+    price: toNumberOrNull(record.price),
+    currency: toStringOrNull(record.currency),
+    rating: toNumberOrNull(record.rating),
+    reviewsCount: toNumberOrNull(record.reviews_count),
+  }
+}
+
 export const asinSearchProductsTool: ToolConfig<
   AsinSearchProductsParams,
   AsinSearchProductsResponse

apps/sim/tools/asin/utils.ts

@@ -1,21 +1,6 @@
 import { toBooleanOrNull, toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
 import { toRecordOrNull } from '@sim/utils/object'
-import type { AsinOffer, AsinPricePoint, AsinSearchResult } from '@/tools/asin/types'
-
-/** Reads a `AsinSearchResult` from an untyped Asin API payload. */
-export function mapAsinSearchResult(value: unknown): AsinSearchResult | null {
-  const record = toRecordOrNull(value)
-  if (!record) return null
-  return {
-    asin: toStringOrNull(record.asin),
-    title: toStringOrNull(record.title),
-    image: toStringOrNull(record.image),
-    price: toNumberOrNull(record.price),
-    currency: toStringOrNull(record.currency),
-    rating: toNumberOrNull(record.rating),
-    reviewsCount: toNumberOrNull(record.reviews_count),
-  }
-}
+import type { AsinOffer } from '@/tools/asin/types'
 
 /** Reads a `AsinOffer` from an untyped Asin API payload. */
 export function mapAsinOffer(value: unknown): AsinOffer | null {
@@ -32,16 +17,3 @@ export function mapAsinOffer(value: unknown): AsinOffer | null {
     deliveryPrice: toStringOrNull(record.delivery_price),
   }
 }
-
-/** Reads a `AsinPricePoint` from an untyped Asin API payload. */
-export function mapAsinPricePoint(value: unknown): AsinPricePoint | null {
-  const record = toRecordOrNull(value)
-  if (!record) return null
-  return {
-    price: toNumberOrNull(record.price),
-    inStock: toBooleanOrNull(record.in_stock),
-    coupon: toStringOrNull(record.coupon),
-    seller: toStringOrNull(record.seller),
-    checkedAt: toStringOrNull(record.checked_at),
-  }
-}

apps/sim/tools/generated/tool-metadata.ts

@@ -1919,7 +1919,7 @@
         },
         {
           "name": "Read Offers",
-          "description": "Read every offer for an ASIN from Amazon now (uses a credit): seller, price, Prime and FBA, delivery, with Amazon's own price and the lowest new and used offers."
+          "description": "Read every offer for an ASIN from Amazon now (uses a credit): seller, price, whether Amazon ships it (FBA), delivery, with Amazon's own price and the lowest new and used offers."
         },
         {
           "name": "Get Last Known Offers",