fix(piloterr): address review feedback

- Describe Google Search without a knowledge graph, which it does not return
- Read the operation from the mapper context so agent tools coerce numeric
  inputs too
- Keep response mappers with a single caller in that tool's file
- Document the PiloterrIcon export

Author: Josselin Liebe

apps/docs/components/icons.tsx

@@ -10460,6 +10460,7 @@ export function PlaneIcon(props: SVGProps<SVGSVGElement>) {
   )
 }
 
+/** Piloterr logo, as used by the Piloterr block. */
 export function PiloterrIcon(props: SVGProps<SVGSVGElement>) {
   return (
     <svg viewBox='-0.3 1.2 20 20' fill='none' xmlns='http://www.w3.org/2000/svg' {...props}>

apps/docs/content/docs/integrations/piloterr.mdx

@@ -137,7 +137,7 @@ Detect the technologies a website runs, such as its CMS, frameworks, analytics,
 
 ### Piloterr Google Search
 
-Search Google and get organic results with their position, title, link and snippet, the knowledge graph and the estimated total.
+Search Google and get organic results with their position, title, link and snippet, and the estimated total.
 
 <ReferenceDetails>
 

apps/sim/blocks/blocks/piloterr.ts

@@ -305,31 +305,32 @@ export const PiloterrBlock: BlockConfig = {
     ],
     config: {
       tool: (params) => `piloterr_${params.operation}`,
-      params: (params) => {
+      params: (params, context) => {
+        const operation = context?.operation ?? params.operation
         const result: Record<string, unknown> = {}
-        if (params.operation === 'render_html') {
+        if (operation === 'render_html') {
           result.waitInSeconds = parseOptionalNumberInput(params.waitInSeconds, 'Wait', { min: 0 })
         }
-        if (params.operation === 'screenshot') {
+        if (operation === 'screenshot') {
           result.viewportWidth = parseOptionalNumberInput(params.viewportWidth, 'Viewport Width', {
             integer: true,
             min: 1,
           })
         }
-        if (params.operation === 'screenshot') {
+        if (operation === 'screenshot') {
           result.viewportHeight = parseOptionalNumberInput(
             params.viewportHeight,
             'Viewport Height',
             { integer: true, min: 1 }
           )
         }
-        if (params.operation === 'screenshot') {
+        if (operation === 'screenshot') {
           result.delay = parseOptionalNumberInput(params.delay, 'Delay', { min: 0 })
         }
-        if (['google_search', 'amazon_search', 'ebay_search'].includes(params.operation)) {
+        if (['google_search', 'amazon_search', 'ebay_search'].includes(operation)) {
           result.page = parseOptionalNumberInput(params.page, 'Page', { integer: true, min: 1 })
         }
-        if (params.operation === 'google_search') {
+        if (operation === 'google_search') {
           result.numResults = parseOptionalNumberInput(params.numResults, 'Results per Page', {
             integer: true,
             min: 1,

apps/sim/components/icons.tsx

@@ -10460,6 +10460,7 @@ export function PlaneIcon(props: SVGProps<SVGSVGElement>) {
   )
 }
 
+/** Piloterr logo, as used by the Piloterr block. */
 export function PiloterrIcon(props: SVGProps<SVGSVGElement>) {
   return (
     <svg viewBox='-0.3 1.2 20 20' fill='none' xmlns='http://www.w3.org/2000/svg' {...props}>

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

@@ -1,13 +1,28 @@
-import { toArray } from '@sim/utils/object'
+import { toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
+import { toArray, toRecordOrNull } from '@sim/utils/object'
 import type {
   PiloterrAmazonProduct,
   PiloterrAmazonSearchParams,
   PiloterrAmazonSearchResponse,
 } from '@/tools/piloterr/types'
 import { PILOTERR_AMAZON_PRODUCT_OUTPUT_PROPERTIES } from '@/tools/piloterr/types'
-import { mapPiloterrAmazonProduct } from '@/tools/piloterr/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/** Reads a `PiloterrAmazonProduct` from an untyped Piloterr API payload. */
+function mapPiloterrAmazonProduct(value: unknown): PiloterrAmazonProduct | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    asin: toStringOrNull(record.asin),
+    title: toStringOrNull(record.title),
+    url: toStringOrNull(record.url),
+    price: toNumberOrNull(record.price),
+    realPrice: toNumberOrNull(record.real_price),
+    rating: toNumberOrNull(record.rating),
+    reviewsCount: toNumberOrNull(record.reviews_count),
+  }
+}
+
 export const piloterrAmazonSearchTool: ToolConfig<
   PiloterrAmazonSearchParams,
   PiloterrAmazonSearchResponse

apps/sim/tools/piloterr/amazon_search.ts

@@ -1,5 +1,5 @@
 import { toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
-import { toArray, toRecord } from '@sim/utils/object'
+import { toArray, toRecord, toRecordOrNull } from '@sim/utils/object'
 import type {
   PiloterrCrunchbaseCategory,
   PiloterrCrunchbaseCompanyParams,
@@ -14,14 +14,49 @@ import {
   PILOTERR_CRUNCHBASE_LOCATION_OUTPUT_PROPERTIES,
   PILOTERR_CRUNCHBASE_PRODUCT_OUTPUT_PROPERTIES,
 } from '@/tools/piloterr/types'
-import {
-  mapPiloterrCrunchbaseCategory,
-  mapPiloterrCrunchbaseInvestor,
-  mapPiloterrCrunchbaseLocation,
-  mapPiloterrCrunchbaseProduct,
-} from '@/tools/piloterr/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/** Reads a `PiloterrCrunchbaseCategory` from an untyped Piloterr API payload. */
+function mapPiloterrCrunchbaseCategory(value: unknown): PiloterrCrunchbaseCategory | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    name: toStringOrNull(record.name),
+    permalink: toStringOrNull(record.permalink),
+  }
+}
+
+/** Reads a `PiloterrCrunchbaseLocation` from an untyped Piloterr API payload. */
+function mapPiloterrCrunchbaseLocation(value: unknown): PiloterrCrunchbaseLocation | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    name: toStringOrNull(record.name),
+    type: toStringOrNull(record.type),
+  }
+}
+
+/** Reads a `PiloterrCrunchbaseInvestor` from an untyped Piloterr API payload. */
+function mapPiloterrCrunchbaseInvestor(value: unknown): PiloterrCrunchbaseInvestor | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    name: toStringOrNull(toRecord(record.investor_identifier).value),
+    permalink: toStringOrNull(toRecord(record.investor_identifier).permalink),
+    fundingRound: toStringOrNull(toRecord(record.funding_round_identifier).value),
+  }
+}
+
+/** Reads a `PiloterrCrunchbaseProduct` from an untyped Piloterr API payload. */
+function mapPiloterrCrunchbaseProduct(value: unknown): PiloterrCrunchbaseProduct | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    name: toStringOrNull(record.name),
+    description: toStringOrNull(record.description),
+  }
+}
+
 export const piloterrCrunchbaseCompanyTool: ToolConfig<
   PiloterrCrunchbaseCompanyParams,
   PiloterrCrunchbaseCompanyResponse

apps/sim/tools/piloterr/crunchbase_company.ts

@@ -1,13 +1,41 @@
-import { toArray, toRecord } from '@sim/utils/object'
+import { toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
+import { toArray, toRecord, toRecordOrNull } from '@sim/utils/object'
 import type {
   PiloterrDetectTechnologiesParams,
   PiloterrDetectTechnologiesResponse,
   PiloterrTechnology,
+  PiloterrTechnologyCategory,
 } from '@/tools/piloterr/types'
 import { PILOTERR_TECHNOLOGY_OUTPUT_PROPERTIES } from '@/tools/piloterr/types'
-import { mapPiloterrTechnology } from '@/tools/piloterr/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/** Reads a `PiloterrTechnologyCategory` from an untyped Piloterr API payload. */
+function mapPiloterrTechnologyCategory(value: unknown): PiloterrTechnologyCategory | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    name: toStringOrNull(record.name),
+    slug: toStringOrNull(record.slug),
+  }
+}
+
+/** Reads a `PiloterrTechnology` from an untyped Piloterr API payload. */
+function mapPiloterrTechnology(value: unknown): PiloterrTechnology | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    name: toStringOrNull(record.name),
+    slug: toStringOrNull(record.slug),
+    version: toStringOrNull(record.version),
+    website: toStringOrNull(record.website),
+    description: toStringOrNull(record.description),
+    confidence: toNumberOrNull(record.confidence),
+    categories: toArray(record.categories)
+      .map(mapPiloterrTechnologyCategory)
+      .filter((item): item is PiloterrTechnologyCategory => item !== null),
+  }
+}
+
 export const piloterrDetectTechnologiesTool: ToolConfig<
   PiloterrDetectTechnologiesParams,
   PiloterrDetectTechnologiesResponse

apps/sim/tools/piloterr/detect_technologies.ts

@@ -1,5 +1,5 @@
 import { toBooleanOrNull, toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
-import { toArray, toRecord } from '@sim/utils/object'
+import { toArray, toRecord, toRecordOrNull } from '@sim/utils/object'
 import type {
   PiloterrEbayGetListingParams,
   PiloterrEbayGetListingResponse,
@@ -9,9 +9,19 @@ import {
   PILOTERR_EBAY_SELLER_OUTPUT_PROPERTIES,
   PILOTERR_EBAY_SPECIFICATION_OUTPUT_PROPERTIES,
 } from '@/tools/piloterr/types'
-import { mapPiloterrEbaySeller, mapPiloterrEbaySpecification } from '@/tools/piloterr/utils'
+import { mapPiloterrEbaySeller } from '@/tools/piloterr/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/** Reads a `PiloterrEbaySpecification` from an untyped Piloterr API payload. */
+function mapPiloterrEbaySpecification(value: unknown): PiloterrEbaySpecification | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    name: toStringOrNull(record.name),
+    value: toStringOrNull(record.value),
+  }
+}
+
 export const piloterrEbayGetListingTool: ToolConfig<
   PiloterrEbayGetListingParams,
   PiloterrEbayGetListingResponse

apps/sim/tools/piloterr/ebay_get_listing.ts

@@ -1,14 +1,35 @@
 import { toNumberOrNull, toStringOrNull } from '@sim/utils/coerce'
-import { toArray, toRecord } from '@sim/utils/object'
+import { toArray, toRecord, toRecordOrNull } from '@sim/utils/object'
 import type {
   PiloterrEbayListing,
   PiloterrEbaySearchParams,
   PiloterrEbaySearchResponse,
 } from '@/tools/piloterr/types'
 import { PILOTERR_EBAY_LISTING_OUTPUT_PROPERTIES } from '@/tools/piloterr/types'
-import { mapPiloterrEbayListing } from '@/tools/piloterr/utils'
+import { mapPiloterrEbaySeller } from '@/tools/piloterr/utils'
 import type { ToolConfig } from '@/tools/types'
 
+/** Reads a `PiloterrEbayListing` from an untyped Piloterr API payload. */
+function mapPiloterrEbayListing(value: unknown): PiloterrEbayListing | null {
+  const record = toRecordOrNull(value)
+  if (!record) return null
+  return {
+    position: toNumberOrNull(record.position),
+    itemId: toStringOrNull(record.item_id),
+    title: toStringOrNull(record.title),
+    subtitle: toStringOrNull(record.subtitle),
+    url: toStringOrNull(record.url),
+    price: toNumberOrNull(record.price),
+    currency: toStringOrNull(record.currency),
+    priceDisplay: toStringOrNull(record.price_display),
+    condition: toStringOrNull(record.condition),
+    buyingFormat: toStringOrNull(record.buying_format),
+    shipping: toStringOrNull(record.shipping),
+    imageUrl: toStringOrNull(record.image_url),
+    seller: mapPiloterrEbaySeller(record.seller),
+  }
+}
+
 export const piloterrEbaySearchTool: ToolConfig<
   PiloterrEbaySearchParams,
   PiloterrEbaySearchResponse