components/trustpilot/package.json (1)

3-3: Confirm semver intent for removed sources

Multiple sources were removed across this PR. Under 0.x, minor bumps may include breaking changes, but please confirm no active workflows depend on removed keys and add a CHANGELOG note calling this out.

components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs (2)

51-54: Order by newest first (if supported by endpoint)

Prevents missing new items on page 1 when mixing statuses.

Apply if supported:

       const fetchParams = {
         businessUnitId: this.businessUnitId,
         perPage: 100,
-        page: 1,
+        page: 1,
+        orderBy: "createdat.desc",
       };

To verify support, check the app’s product review fetch or docs for orderBy on private product reviews.

---

56-60: Paginate until reviews are older than last poll to avoid gaps

If > perPage reviews arrive between polls, page 1 isn’t enough.

Apply pattern:

-      const result = await this.trustpilot.fetchProductReviews(fetchParams);
-      const reviews = result.reviews || [];
+      const toMs = (d) => new Date(d).getTime();
+      const lastTs = Number(this._getLastReviewTime()) || 0;
+      let page = 1;
+      let reviews = [];
+      while (true) {
+        const { reviews: batch = [] } = await this.trustpilot.fetchProductReviews({ ...fetchParams, page });
+        if (!batch.length) break;
+        reviews.push(...batch);
+        const oldest = Math.min(...batch.map((r) => toMs(r.createdAt)));
+        if (oldest <= lastTs) break;
+        page += 1;
+      }

components/trustpilot/actions/fetch-product-review-by-id/fetch-product-review-by-id.mjs (3)

47-49: Be robust to response shape (review vs root object)

Some endpoints return { review: {...} }. Safely support both.

Apply:

-      const review = parseProductReview(response);
+      const raw = response?.review ?? response;
+      const review = parseProductReview(raw);

---

59-61: Improve error messaging for 404

Provide a clearer “not found” for missing IDs; keep other errors wrapped.

Apply:

-    } catch (error) {
-      throw new Error(`Failed to fetch product review: ${error.message}`);
-    }
+    } catch (error) {
+      if (error.response?.status === 404) {
+        throw new Error(`Review ${this.reviewId} not found (404)`);
+      }
+      throw new Error(`Failed to fetch product review: ${error.message}`);
+    }

---

42-46: Optional: set an explicit timeout

Avoid long hangs on degraded endpoints.

Apply:

       const response = await makeRequest(this.trustpilot, {
         endpoint,
+        timeout: 20000,
       });

components/trustpilot/actions/get-conversation-from-product-review/get-conversation-from-product-review.mjs (2)

50-63: Add a final $summary when no conversation exists.

This improves UX in runs where the action exits early without fetching a conversation.

       if (!conversationId) {
+        $.export("$summary", `No conversation found for product review ${reviewId}`);
         return {
           success: false,
           message: "No conversation found for this product review",

---

1-8: Return richer API error details to aid troubleshooting.

Surface parsed status/code/message so users can act on 401/403/404/429, etc.

-import {
-  buildUrl,
-  validateReviewId,
-} from "../../common/utils.mjs";
+import {
+  buildUrl,
+  validateReviewId,
+  parseApiError,
+} from "../../common/utils.mjs";

-  } catch (error) {
-      throw new ConfigurationError(`Failed to get conversation from product review: ${error.message}`);
-  }
+  } catch (error) {
+    const apiErr = parseApiError(error);
+    throw new ConfigurationError(
+      `Failed to get conversation from product review: ${apiErr.message} (status: ${apiErr.status}, code: ${apiErr.code})`,
+    );
+  }

Also applies to: 90-92

components/trustpilot/actions/reply-to-product-review/reply-to-product-review.mjs (2)

79-97: Handle “conversation already exists” (HTTP 409) to avoid failing valid runs.

Races can occur between Step 1 and Step 2. Gracefully recover by refetching the review (or using the ID from the error body if present). Also, include x-business-user-id on conversation creation if the API requires it.

-      // Step 2: Create conversation if it doesn't exist
-      if (!conversationId) {
-        $.export("$summary", "Creating conversation for review...");
-
-        const createConversationEndpoint = buildUrl(ENDPOINTS.CREATE_CONVERSATION_FOR_REVIEW, {
-          reviewId,
-        });
-
-        const createConversationResponse = await makeRequest(this.trustpilot, {
-          endpoint: createConversationEndpoint,
-          method: "POST",
-        });
-
-        conversationId = createConversationResponse.conversationId;
-
-        if (!conversationId) {
-          throw new Error("Failed to create conversation - no conversationId returned");
-        }
-      }
+      // Step 2: Create conversation if it doesn't exist
+      if (!conversationId) {
+        $.export("$summary", "Creating conversation for review...");
+        const createConversationEndpoint = buildUrl(ENDPOINTS.CREATE_CONVERSATION_FOR_REVIEW, { reviewId });
+        try {
+          const createConversationResponse = await makeRequest(this.trustpilot, {
+            endpoint: createConversationEndpoint,
+            method: "POST",
+            additionalHeaders: {
+              "x-business-user-id": businessUserId, // required by some endpoints
+            },
+          });
+          conversationId = createConversationResponse.conversationId;
+        } catch (err) {
+          // If the conversation already exists, recover by re-fetching it
+          if (err?.response?.status === 409) {
+            $.export("$summary", "Conversation already exists. Fetching conversationId...");
+            const refreshed = await makeRequest(this.trustpilot, { endpoint: getReviewEndpoint });
+            conversationId = refreshed.conversationId || err?.response?.data?.conversationId;
+          } else {
+            throw err;
+          }
+        }
+        if (!conversationId) {
+          throw new Error("Failed to create or locate conversationId");
+        }
+      }

---

63-64: Sanitize and truncate reply content
Trustpilot’s Conversations API doesn’t document a strict limit; community testing indicates a nominal cap of ~10 000 chars and practical limits closer to ~8 000. To prevent server-side rejections, import and apply a sanitizer/truncator:

-import {
-  buildUrl,
-  validateReviewId,
-} from "../../common/utils.mjs";
+import {
+  buildUrl,
+  validateReviewId,
+  sanitizeInput,
+} from "../../common/utils.mjs";

-    const trimmedContent = content.trim();
+    const trimmedContent = sanitizeInput(content, 8000); // truncate/sanitize to 8 000 chars

Also ensure the x-business-user-id header is sent on Create Conversation calls when using a client_credentials token.

components/trustpilot/common/utils.mjs (2)

107-149: PII exposure: consumer email is passed through. Verify intent and compliance.

parseProductReview returns consumer.email verbatim. Ensure this is expected and compliant with your data handling policy. If not strictly required, consider masking or omitting it by default.

---

197-203: Normalize boolean default for countsTowardsLocationTrustScore.

For consistency with countsTowardsTrustScore, default this to false when undefined.

-    countsTowardsLocationTrustScore: review.countsTowardsLocationTrustScore,
+    countsTowardsLocationTrustScore: review.countsTowardsLocationTrustScore || false,

components/trustpilot/actions/fetch-product-reviews/fetch-product-reviews.mjs (2)

77-77: Guard summary against unexpected API shapes.

Avoid runtime errors if reviews is missing/not an array.

-      $.export("$summary", `Successfully fetched ${result.reviews.length} product review(s) for business unit ${businessUnitId}`);
+      const count = Array.isArray(result?.reviews) ? result.reviews.length : Array.isArray(result) ? result.length : 0;
+      $.export("$summary", `Successfully fetched ${count} product review(s) for business unit ${businessUnitId}`);

---

35-40: Language vs. locale: confirm intent and API mapping.

Both language and locale are exposed and forwarded. If the API expects only one (or mutually exclusive values), this can lead to ignored params or ambiguous results. Consider a single prop or validation to ensure they’re not conflicting.

Example constraint:

  async run({ $ }) {
    const {
      businessUnitId,
      page,
      perPage,
      sku,
      language,
      state,
      locale,
    } = this;

+   if (language && locale) {
+     throw new Error("Specify either language or locale, not both.");
+   }

Also applies to: 47-52, 55-63

components/trustpilot/actions/fetch-service-review-by-id/fetch-service-review-by-id.mjs (2)

26-34: Review ID validation is too strict for hex case.

validateReviewId only allows lowercase a–f; ObjectIDs/hex IDs are case-insensitive. Allow uppercase to avoid false negatives.

Update the validator in components/trustpilot/common/utils.mjs:

-export function validateReviewId(reviewId) {
-  // Trustpilot Review IDs are 24-character hexadecimal strings (MongoDB ObjectID format)
-  return (
-    typeof reviewId === "string" &&
-    /^[a-f0-9]{24}$/.test(reviewId)
-  );
-}
+export function validateReviewId(reviewId) {
+  // Trustpilot Review IDs are 24-character hexadecimal strings (MongoDB ObjectID format)
+  return typeof reviewId === "string" && /^[A-Fa-f0-9]{24}$/.test(reviewId);
+}

---

13-14: Double-check documentation link matches the private-by-id endpoint.

The link references “business-units-api#get-private-review-by-id” while the code hits /private/reviews/{reviewId}. Ensure the doc URL matches the exact endpoint to avoid confusing users.

components/trustpilot/common/api-client.mjs (3)

37-44: Prevent accidental override of auth headers via args.headers.

Spreading ...args after headers lets callers overwrite Authorization/apikey. Merge only non-header args, or explicitly merge safe header keys.

-  const config = {
+  const { headers: _ignoredHeaders, ...restArgs } = args;
+  const config = {
     method,
     url,
     headers,
     params: formatQueryParams(params),
     timeout,
-    ...args,
+    ...restArgs,
   };

---

54-69: Retry policy could be more robust.

Consider retrying on transient errors like 408/429/500/502/503/504 and network timeouts, with jittered exponential backoff.

Example:

-  if (retries > 0 && error.response?.status === HTTP_STATUS.TOO_MANY_REQUESTS) {
+  const status = error.response?.status;
+  const transient = [HTTP_STATUS.TOO_MANY_REQUESTS, 408, 500, 502, 503, 504];
+  if (retries > 0 && transient.includes(status)) {
-    const delay = Math.min(
-      RETRY_CONFIG.INITIAL_DELAY * (RETRY_CONFIG.MAX_RETRIES - retries + 1),
-      RETRY_CONFIG.MAX_DELAY,
-    );
+    const attempt = RETRY_CONFIG.MAX_RETRIES - retries + 1;
+    const base = Math.min(RETRY_CONFIG.INITIAL_DELAY * 2 ** (attempt - 1), RETRY_CONFIG.MAX_DELAY);
+    const jitter = Math.random() * base * 0.2;
+    const delay = Math.min(base + jitter, RETRY_CONFIG.MAX_DELAY);

---

116-135: Heuristic private URL detection.

Using url.includes("private") works with current constants, but is brittle. Prefer a flag or derive from endpoint groups to avoid misclassification if paths change.

Minimal change:

-export async function makeRequest(trustpilotApp, {
+export async function makeRequest(trustpilotApp, {
   endpoint,
   method = "GET",
   params = {},
   data = null,
   timeout = 30000,
   additionalHeaders = {},
   ...args
 }, retries = RETRY_CONFIG.MAX_RETRIES) {
   const url = `${BASE_URL}${endpoint}`;
-  const headers = {
-    ...getAuthHeaders(trustpilotApp, url),
+  const headers = {
+    ...getAuthHeaders(trustpilotApp, endpoint),
     ...additionalHeaders,
   };

And switch getAuthHeaders to check the endpoint path (startsWith("/private")) instead of the full URL string.

components/trustpilot/actions/search-business-units-test/search-business-units-test.mjs (1)

39-48: Consider null safety for businessUnit properties.

The destructuring assumes id and displayName exist on each business unit. Add defensive checks to prevent runtime errors.

       const options = businessUnits.map((businessUnit) => {
         const {
           id, displayName,
         } = businessUnit;
+        
+        if (!id || !displayName) {
+          console.warn("Business unit missing required fields:", businessUnit);
+          return null;
+        }

         return {
           label: displayName,
           value: id,
         };
-      });
+      }).filter(Boolean); // Remove null entries

components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs (1)

6-6: Description is too verbose for a component description.

The description is extremely long and contains implementation details that belong in documentation rather than the component description field.

-  description: "Emit new event when a customer posts a new service review on Trustpilot. This source periodically polls the Trustpilot API to detect new service reviews using the private reviews API for comprehensive coverage. Each event contains the complete review data including star rating, review text, consumer details, business unit info, customer email, and timestamps. Ideal for monitoring overall business reputation, tracking customer satisfaction metrics, and triggering workflows based on review ratings or content.",
+  description: "Emit new event when a customer posts a new service review on Trustpilot using the private reviews API. [See the documentation](https://developers.trustpilot.com/private-reviews-api)",

components/trustpilot/common/constants.mjs (3)

47-54: De-duplicate sort strings across actions

Since SORT_OPTIONS are defined here, import and reuse them in actions (e.g., fetch-service-reviews) to avoid drift.

Apply in the action file:

-        {
-          label: "Created At (Ascending)",
-          value: "createdat.asc",
-        },
+        { label: "Created At (Ascending)", value: SORT_OPTIONS.CREATED_AT_ASC },

(Repeat for the other three options after importing SORT_OPTIONS.)

---

3-10: Stale webhook events?

WEBHOOK_EVENTS remain, but webhook endpoints were dropped in this PR. If they’re unused, remove them or add a comment noting they’re for inbound-only use to reduce confusion.

---

91-97: Align SOURCE_TYPES with the current sources

Some source types (e.g., new_conversations) were removed per the PR summary. Either prune them or annotate deprecated values to prevent accidental use.

components/trustpilot/actions/fetch-service-reviews/fetch-service-reviews.mjs (5)

6-8: Description overpromises vs single-page fetch

The action fetches a single page, but the description claims up to 100,000 records. Either implement pagination within the action or adjust the copy.

Apply to fix the copy:

-  description: "Get private reviews for a business unit, limited to 100,000 records. Response includes customer email and order ID. [See the documentation](https://developers.trustpilot.com/business-units-api#get-private-reviews-for-business-unit)",
+  description: "Get a page of private reviews for a business unit. Response includes customer email and order ID. [See the documentation](https://developers.trustpilot.com/business-units-api#get-private-reviews-for-business-unit)",

If you want pagination here, I can propose a minimal loop that respects perPage/max pages and returns aggregated results.

---

31-36: Validate/sanitize stars input

The API expects 1–5 comma-separated. Consider normalizing (e.g., strip spaces, validate digits) upstream to prevent 400s.

I suggest adding validation in the app’s fetchServiceReviews (see app comment with diff).

---

52-76: Use constants for orderBy values

Avoid hardcoding the same strings in multiple files.

Apply:

-import trustpilot from "../../trustpilot.app.mjs";
+import trustpilot from "../../trustpilot.app.mjs";
+import { SORT_OPTIONS } from "../../common/constants.mjs";
...
-          value: "createdat.asc",
+          value: SORT_OPTIONS.CREATED_AT_ASC,
...
-          value: "createdat.desc",
+          value: SORT_OPTIONS.CREATED_AT_DESC,
...
-          value: "stars.asc",
+          value: SORT_OPTIONS.STARS_ASC,
...
-          value: "stars.desc",
+          value: SORT_OPTIONS.STARS_DESC,

---

120-131: Grammar nit and add example format for datetimes

Small copy fix and an example improves UX.

-      description: "Filter reviews by datetime range. If no time is specified than time is implicit 00:00:00. Format: 2013-09-07T13:37:00",
+      description: "Filter reviews by datetime range. If no time is specified then time is implicit 00:00:00. Example format: 2013-09-07T13:37:00",
...
-      description: "Filter reviews by datetime range. If no time is specified than time is implicit 00:00:00. Format: 2013-09-07T13:37:00",
+      description: "Filter reviews by datetime range. If no time is specified then time is implicit 00:00:00. Example format: 2013-09-07T13:37:00",

---

205-207: Preserve original error as cause

Keeps stack/metadata for debugging while improving the message.

-    } catch (error) {
-      throw new Error(`Failed to fetch service reviews: ${error.message}`);
+    } catch (error) {
+      throw new Error(`Failed to fetch service reviews: ${error.message}`, { cause: error });
     }

components/trustpilot/trustpilot.app.mjs (8)

112-121: Return empty options on error is safe, but log is minimal

If you see repeated failures, consider including response status/body in the debug log to aid triage.

---

147-154: Business unit ID format validation may be too strict

validateBusinessUnitId enforces lowercase hex. ObjectIDs are case-insensitive; rejecting uppercase can block valid input pasted from external tools.

If you want to relax validation without changing all callers, update utils:

// components/trustpilot/common/utils.mjs
export function validateBusinessUnitId(businessUnitId) {
  return typeof businessUnitId === "string" && /^[a-f0-9]{24}$/i.test(businessUnitId);
}

---

160-182: Consider normalizing and validating stars before request

Prevents 400s and keeps API surface tidy.

Apply within fetchServiceReviews:

-      if (stars) queryParams.stars = stars;
+      if (stars) {
+        const normalized = String(stars).replace(/\s+/g, "");
+        if (!/^(?:[1-5](?:,[1-5])*)$/.test(normalized)) {
+          throw new Error('Invalid "stars" format. Use comma-separated integers 1–5, e.g. "1,3,5".');
+        }
+        queryParams.stars = normalized;
+      }

---

189-196: Service reviews pagination shape assumption

You map response.pagination.{total,page,perPage,hasMore}. Please confirm the v1 private reviews endpoint returns this shape (some Trustpilot endpoints use links.next/prev). If not guaranteed, add a fallback like the product flow.

---

221-226: Add business unit ID validation for product reviews for parity

Service reviews validate ID format; product reviews should do the same to catch obvious errors early.

   if (!businessUnitId) {
     throw new Error("Business Unit ID is required");
   }
+  if (!validateBusinessUnitId(businessUnitId)) {
+    throw new Error("Invalid business unit ID format");
+  }

---

248-257: Product reviews pagination shape

You rely on links.total and links.next. Please confirm this is stable and documented. If both shapes occur (pagination vs links), consider a defensive merge to standardize the returned pagination object.

---

210-269: Optional: clamp perPage to MAX_LIMIT at the app layer

Actions set max=100, but sources or external callers could bypass. Clamp to constants.MAX_LIMIT defensively.

-import { ENDPOINTS } from "./common/constants.mjs";
+import { ENDPOINTS, MAX_LIMIT } from "./common/constants.mjs";
...
-      if (perPage) queryParams.perPage = perPage;
+      if (perPage) queryParams.perPage = Math.max(1, Math.min(perPage, MAX_LIMIT));

(Repeat for fetchProductReviews.)

---

108-121: Add concise JSDoc for public methods

Helps users and aligns with the guidelines to document methods.

/**
 * Search business units by query.
 * @param {{query?: string, page?: number}} params
 * @returns {Promise<Array<{id: string, displayName: string}>>}
 */

(Apply similarly to fetchServiceReviews and fetchProductReviews.)

Also applies to: 123-207, 209-269

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

components/trustpilot/actions/fetch-service-reviews/fetch-service-reviews.mjs (1)

23-30: Good: page prop UX and guardrails

Clear description, min=1, sensible default. Matches prior review guidance.

components/trustpilot/actions/fetch-product-review-by-id/fetch-product-review-by-id.mjs (1)

28-35: Use ConfigurationError for input/flow errors (align with other actions).

Consistent error types improve UX and surface configuration issues clearly.

+import { ConfigurationError } from "@pipedream/platform";
 import { makeRequest } from "../../common/api-client.mjs";
 import { ENDPOINTS } from "../../common/constants.mjs";
 import {
   buildUrl,
   parseProductReview,
   validateReviewId,
 } from "../../common/utils.mjs";

 // Validate required parameters
 if (!reviewId) {
-  throw new Error("Review ID is required");
+  throw new ConfigurationError("Review ID is required");
 }
 if (!validateReviewId(reviewId)) {
-  throw new Error("Invalid review ID format");
+  throw new ConfigurationError("Invalid review ID format");
 }

 ...
 } catch (error) {
-  throw new Error(`Failed to fetch product review: ${error.message}`);
+  throw new ConfigurationError(`Failed to fetch product review: ${error.message}`);
 }

Also applies to: 59-61, 2-4

components/trustpilot/actions/reply-to-product-review/reply-to-product-review.mjs (2)

63-64: Sanitize reply content without imposing a length cap.

Avoid control chars while respecting prior decision to not enforce arbitrary limits.

 import { makeRequest } from "../../common/api-client.mjs";
 import { ENDPOINTS } from "../../common/constants.mjs";
 import {
   buildUrl,
   validateReviewId,
+  sanitizeInput,
 } from "../../common/utils.mjs";

-const trimmedContent = content.trim();
+const sanitizedContent = sanitizeInput(content, Infinity);

 // Prepare request data
 const requestData = {
-  content: trimmedContent,
+  content: sanitizedContent,
 };

 // Add integrationId if provided
 if (integrationId) {
   requestData.integrationId = integrationId;
 }

 ...
 return {
   success: true,
   comment: replyResponse,
   metadata: {
     reviewId,
     conversationId,
     businessUserId,
-    contentLength: trimmedContent.length,
+    contentLength: sanitizedContent.length,
     integrationId: integrationId || null,
     wasConversationCreated: !review.conversationId,
     requestTime: new Date().toISOString(),
   },
 };

Also applies to: 106-115, 127-139, 5-8

---

67-68: Intermediate $summary calls are overwritten.

Keep a single final summary or switch these to logs if you need progress notes.

-$.export("$summary", "Fetching product review details...");
 ...
-$.export("$summary", "Creating conversation for review...");
 ...
-$.export("$summary", "Posting reply comment...");

Also applies to: 81-82, 100-101

components/trustpilot/common/utils.mjs (3)

218-224: Accept uppercase hex and stray whitespace for IDs.

ObjectIDs can be uppercase; trimming avoids false negatives.

 export function validateBusinessUnitId(businessUnitId) {
   // Trustpilot Business Unit IDs are 24-character hexadecimal strings (MongoDB ObjectID format)
   return (
-    typeof businessUnitId === "string" &&
-    /^[a-f0-9]{24}$/.test(businessUnitId)
+    typeof businessUnitId === "string" &&
+    /^[a-f0-9]{24}$/i.test(businessUnitId.trim())
   );
 }

---

231-236: Same normalization for review IDs.

 export function validateReviewId(reviewId) {
   // Trustpilot Review IDs are 24-character hexadecimal strings (MongoDB ObjectID format)
   return (
-    typeof reviewId === "string" &&
-    /^[a-f0-9]{24}$/.test(reviewId)
+    typeof reviewId === "string" &&
+    /^[a-f0-9]{24}$/i.test(reviewId.trim())
   );
 }

---

33-36: Comment/code mismatch on sanitizeInput.

The code uses explicit ranges, not Unicode property escapes. Update the comment for clarity.

-// Using Unicode property escapes for safer regex
-// eslint-disable-next-line no-control-regex
+// Using explicit control-char ranges for broad runtime support
+// eslint-disable-next-line no-control-regex
   sanitized = sanitized.replace(/[\x00-\x08\x0B\x0C\x0E-\x1F\x7F]/g, "");

components/trustpilot/actions/reply-to-service-review/reply-to-service-review.mjs (2)

13-14: Add helpful docs link to description.

-description: "Reply to a service review on Trustpilot.",
+description: "Reply to a service review on Trustpilot. [See the documentation](https://developers.trustpilot.com/service-reviews-api#reply-to-a-review)",

---

56-57: Sanitize message without adding a size limit.

Removes control chars but keeps your “no arbitrary cutoff” stance.

 import { makeRequest } from "../../common/api-client.mjs";
 import { ENDPOINTS } from "../../common/constants.mjs";
 import {
   buildUrl,
   validateReviewId,
+  sanitizeInput,
 } from "../../common/utils.mjs";

-const trimmedMessage = message.trim();
+const sanitizedMessage = sanitizeInput(message, Infinity);

 // Prepare request data according to API specification
 const requestData = {
   authorBusinessUserId,
-  message: trimmedMessage,
+  message: sanitizedMessage,
 };

 ...
 return {
   success: true,
   reply: {
-    message: trimmedMessage,
+    message: sanitizedMessage,
     authorBusinessUserId,
     reviewId,
     status: "created",
     statusCode: 201,
     postedAt: new Date().toISOString(),
   },
   metadata: {
     reviewId,
     authorBusinessUserId,
-    messageLength: trimmedMessage.length,
+    messageLength: sanitizedMessage.length,
     requestTime: new Date().toISOString(),
     httpStatus: "201 Created",
   },
 };

Also applies to: 65-69, 83-95, 5-8

components/trustpilot/actions/fetch-service-review-by-id/fetch-service-review-by-id.mjs (1)

26-34: Optional: be lenient on reviewId input (trim/case).

End users may paste uppercase/space-padded IDs. Normalize before validation to reduce false negatives.

-    const { reviewId } = this;
+    const { reviewId } = this;
+    const normalizedReviewId = String(reviewId).trim();
 
     // Validate required parameters
-    if (!reviewId) {
+    if (!normalizedReviewId) {
       throw new Error("Review ID is required");
     }
-    if (!validateReviewId(reviewId)) {
+    if (!validateReviewId(normalizedReviewId)) {
       throw new Error("Invalid review ID format");
     }

Outside this file (utils.mjs), consider accepting uppercase hex as well:

-  /^[a-f0-9]{24}$/.test(reviewId)
+  /^[a-f0-9]{24}$/i.test(reviewId)

components/trustpilot/actions/get-conversation-from-product-review/get-conversation-from-product-review.mjs (1)

49-61: Optional: add a summary on the no-conversation path.

Improves UX in the run log when exiting early.

     if (!conversationId) {
+      $.export("$summary", `No conversation found for product review ${reviewId}`);
       return {
         success: false,
         message: "No conversation found for this product review",

components/trustpilot/common/api-client.mjs (3)

5-16: JSDoc parameter order is outdated.

Document the $ parameter and reflect the actual signature makeRequest($, trustpilotApp, options).

- * @param {object} trustpilotApp - The Trustpilot app instance with auth credentials
- * @param {object} options - Request options
+ * @param {object} $ - Pipedream context (for axios/error handling)
+ * @param {object} trustpilotApp - The Trustpilot app instance with auth credentials
+ * @param {object} options - Request options

---

92-111: Optional: include Accept header.

Some APIs vary payloads by Accept. Adding Accept: application/json is harmless and can prevent content-negotiation surprises.

   const headers = {
     "Content-Type": "application/json",
+    "Accept": "application/json",
     "User-Agent": "Pipedream/1.0",
   };

---

54-56: Optional: make private detection more explicit.

url.includes("private") is workable but brittle. Consider checking the endpoint prefix (e.g., endpoint.startsWith("/private/")) or allowing an explicit authMode option.

components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs (1)

53-56: Optional: ensure startDateTime is ISO string.

If lastReviewTime was stored as a number, coerce to ISO before using it in query params.

-      if (lastReviewTime) {
-        fetchParams.startDateTime = lastReviewTime;
-      }
+      if (lastReviewTime) {
+        fetchParams.startDateTime = new Date(lastReviewTime).toISOString();
+      }

components/trustpilot/common/constants.mjs (1)

29-30: Rename CREATE_CONVERSATION_FOR_REVIEW to CREATE_CONVERSATION_FOR_PRODUCT_REVIEW

In components/trustpilot/common/constants.mjs, update:

-  CREATE_CONVERSATION_FOR_REVIEW: "/private/product-reviews/{reviewId}/create-conversation",
+  CREATE_CONVERSATION_FOR_PRODUCT_REVIEW: "/private/product-reviews/{reviewId}/create-conversation",

Then update its usage in components/trustpilot/actions/reply-to-product-review/reply-to-product-review.mjs (change ENDPOINTS.CREATE_CONVERSATION_FOR_REVIEW to ENDPOINTS.CREATE_CONVERSATION_FOR_PRODUCT_REVIEW) to match the naming of other product-review constants.

components/trustpilot/actions/fetch-service-reviews/fetch-service-reviews.mjs (2)

31-36: Constrain or validate star rating input

Free-form "1-5, separated by commas" is error-prone. Either present selectable options or validate with a regex to avoid invalid input (e.g., "6", "a").

-    stars: {
-      type: "string",
+    stars: {
+      type: "string",
       label: "Star Rating",
-      description: "Filter by reviews with a specific star rating. 1-5, separated by commas.",
+      description: "Filter by reviews with a specific star rating. Accepts comma-separated values from 1–5 (e.g., 5 or 4,5).",
+      pattern: "^(?:[1-5])(?:,(?:[1-5]))*$",
       optional: true,
     },

---

120-131: Date-time props: great detail; consider timezone note

Descriptions are solid (ISO format, implicit 00:00:00). Add a brief note on timezone expectation (UTC vs local) if the API assumes UTC.

components/trustpilot/trustpilot.app.mjs (2)

18-33: Harden async options for businessUnitId (page default + fallback labels)

Guard against undefined page and missing displayName to improve UX.

-      async options({
-        page, query,
-      }) {
+      async options({
+        page = 0, query,
+      } = {}) {
         try {
           if (query === "") {
             // Trustpilot requires a query to be passed in, default to "a" if empty
             query = "a";
           }
 
           const businessUnits = await this.searchBusinessUnits({
             // Trustpilot requires the page to be 1-indexed
             // whereas pipedream is 0-indexed
             page: page + 1,
             query,
           });
 
-          return businessUnits.map((businessUnit) => {
-            const {
-              id, displayName,
-            } = businessUnit;
-
-            return {
-              label: displayName,
-              value: id,
-            };
-          });
+          return businessUnits.map(({ id, displayName, identifyingName }) => ({
+            label: displayName || identifyingName || id,
+            value: id,
+          }));

---

175-191: Add consistent BU ID validation for product reviews

Service flow validates businessUnitId format; apply the same check here for parity.

     async fetchProductReviews($, params = {}) {
       const {
         businessUnitId,
         page,
         perPage,
         sku,
         language,
         state,
         locale,
       } = params;
 
       // Validate required parameters
       if (!businessUnitId) {
         throw new Error("Business Unit ID is required");
       }
+      if (!validateBusinessUnitId(businessUnitId)) {
+        throw new Error("Invalid business unit ID format");
+      }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

Learnt from: Afstkla
PR: PipedreamHQ/pipedream#18152
File: components/trustpilot/actions/fetch-service-review-by-id/fetch-service-review-by-id.mjs:42-49
Timestamp: 2025-09-03T14:43:27.512Z
Learning: The Trustpilot SERVICE_REVIEW_BY_ID endpoint returns the review data directly as the response body, not wrapped in a container object like `{ review: {...} }`.

components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs (1)

34-46: Fix pagination: condition checks total accumulated length, so you fetch at most one extra page.

Use the size of the last fetched page to continue, and avoid mutating the input params.

-      // Handle pagination for service reviews
-      while (result.reviews && result.reviews.length === 100) {
-        params.page = (params.page || 1) + 1;
-        const nextResult = await this.trustpilot.fetchServiceReviews($, params);
-        result.reviews = result.reviews.concat(nextResult.reviews || []);
-      }
+      // Handle pagination for service reviews
+      const perPage = params.perPage ?? 100;
+      let page = params.page ?? 1;
+      let all = result.reviews || [];
+      let lastChunkSize = all.length;
+      while (lastChunkSize === perPage) {
+        page += 1;
+        const next = await this.trustpilot.fetchServiceReviews($, { ...params, page });
+        const chunk = next.reviews || [];
+        all = all.concat(chunk);
+        lastChunkSize = chunk.length;
+      }
+      result = { ...result, reviews: all };

components/trustpilot/sources/common/polling.mjs (1)

76-127: Let Axios errors propagate; remove the try/catch wrapper.

Catching and rethrowing loses response metadata (status, data). Allow the framework to surface the original error.

-  async run({ $ }) {
-    try {
+  async run({ $ }) {
       // Get the last review time for filtering new reviews
       const lastReviewTime = this._getLastReviewTime();
@@
-      // Update the last review time for next poll
-      if (latestReviewTime && latestReviewTime !== lastReviewTime) {
-        this._setLastReviewTime(latestReviewTime);
-      }
-
-    } catch (error) {
-      throw new Error(`Failed to fetch reviews: ${error.message}`);
-    }
+      // Update the last review time for next poll
+      if (latestReviewTime && latestReviewTime !== lastReviewTime) {
+        this._setLastReviewTime(latestReviewTime);
+      }
   },

components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs (2)

7-8: Tighten description and verify data fields claimed.

The description reads like marketing copy and asserts inclusion of “customer email” and use of the “private reviews API.” Please confirm both are true for all events; if unsure, tone down the claim.

-  description: "Emit new event when a customer posts a new service review on Trustpilot. This source periodically polls the Trustpilot API to detect new service reviews using the private reviews API for comprehensive coverage. Each event contains the complete review data including star rating, review text, consumer details, business unit info, customer email, and timestamps. Ideal for monitoring overall business reputation, tracking customer satisfaction metrics, and triggering workflows based on review ratings or content.",
+  description: "Emits an event when a new Trustpilot service review is posted. Polls the Trustpilot API and includes key review fields (stars, text, consumer, business unit, timestamps).",

---

13-19: Align consumerName fallback to displayName then name
Use consumer?.displayName ?? consumer?.name ?? "Anonymous" to match new-product-reviews and avoid dropping an existing name.

- const consumerName = review.consumer?.displayName || "Anonymous";
+ const consumerName = review.consumer?.displayName ?? review.consumer?.name ?? "Anonymous";

components/trustpilot/sources/common/polling.mjs (2)

57-63: Set a default page to complement perPage.

Minor: establish page: 1 in the default params to standardize child behavior unless overridden.

     getFetchParams(_lastReviewTime) {
       return {
         businessUnitId: this.businessUnitId,
-        perPage: 100,
+        perPage: 100,
+        page: 1,
       };
     },

---

101-116: Compare numeric timestamps to avoid redundant Date/ISO conversions.

Reduces parsing overhead and avoids edge cases with string comparisons.

-      // Track the latest review time
-      let latestReviewTime = lastReviewTime;
+      // Track the latest review time (numeric)
+      const lastTs = lastReviewTime ? Date.parse(lastReviewTime) : 0;
+      let latestTs = lastTs;
@@
-        // Track the latest review time
-        const reviewTime = new Date(review.createdAt).toISOString();
-        if (!latestReviewTime || new Date(reviewTime) > new Date(latestReviewTime)) {
-          latestReviewTime = reviewTime;
-        }
+        // Track the latest review time
+        const reviewTs = Date.parse(review.createdAt);
+        if (Number.isFinite(reviewTs) && reviewTs > latestTs) {
+          latestTs = reviewTs;
+        }
@@
-        this.$emit(review, {
+        this.$emit(review, {
           id: review.id,
           summary: this.generateSummary(review),
-          ts: new Date(review.createdAt).getTime(),
+          ts: Number.isFinite(reviewTs) ? reviewTs : Date.now(),
         });
       }
@@
-      if (latestReviewTime && latestReviewTime !== lastReviewTime) {
-        this._setLastReviewTime(latestReviewTime);
-      }
+      if (latestTs > lastTs) {
+        this._setLastReviewTime(new Date(latestTs).toISOString());
+      }

Also applies to: 119-122

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs (1)

42-54: Pagination logic fix looks good vs prior “total-length” bug.

You now gate continuation on the last page size instead of the accumulated length. This resolves the earlier issue where only one extra page could be fetched.

components/trustpilot/actions/fetch-service-reviews/fetch-service-reviews.mjs (2)

23-30: Page docs text looks good

The page description matches current Trustpilot docs behavior (empty array when exceeding available pages). (developers.trustpilot.com)

---

178-200: Good: orderBy/pagination are now respected

Passing page/perPage/orderBy through (without being overridden in the app) addresses the earlier issue.

components/trustpilot/trustpilot.app.mjs (2)

125-133: LGTM: preserve user pagination/sort and exclude path param from query

Destructuring with defaults and spreading filters (excluding businessUnitId) fixes the prior override/leak.

---

221-230: Fix product-reviews pagination: use total and detect next via links array

The private product reviews list returns productReviews, total, and a links array. Reading links.total and links.next is incorrect; compute hasMore by checking for a link with rel === "next".

-      const pagination = {
-        total: response.links?.total || 0,
-        page: queryParams.page || 1,
-        perPage: queryParams.perPage || 20,
-        hasMore: response.links?.next
-          ? true
-          : false,
-      };
+      const pagination = {
+        total: typeof response.total === "number" ? response.total : null,
+        page: queryParams.page || 1,
+        perPage: queryParams.perPage || 20,
+        hasMore: Array.isArray(response.links)
+          ? response.links.some((l) => l?.rel === "next")
+          : false,
+      };

(documentation-apidocumentation.trustpilot.com)

components/trustpilot/common/constants.mjs (1)

7-11: Normalize endpoint key naming for consistency.

Mixing SERVICE_* with PRIVATE_PRODUCT_* invites mistakes. Consider aligning both families (either drop PRIVATE_ or add it for service).

Example:

-  SERVICE_REVIEWS: "/private/business-units/{businessUnitId}/reviews",
-  SERVICE_REVIEW_BY_ID: "/private/reviews/{reviewId}",
-  REPLY_TO_SERVICE_REVIEW: "/private/reviews/{reviewId}/reply",
-  PRIVATE_PRODUCT_REVIEWS: "/private/product-reviews/business-units/{businessUnitId}/reviews",
-  PRIVATE_PRODUCT_REVIEW_BY_ID: "/private/product-reviews/{reviewId}",
+  PRODUCT_REVIEWS: "/private/product-reviews/business-units/{businessUnitId}/reviews",
+  PRODUCT_REVIEW_BY_ID: "/private/product-reviews/{reviewId}",

Apply corresponding symbol renames in the client.

Also applies to: 13-16

components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs (1)

17-23: Tweak summary grammar for singular “star”.

-      return `New ${stars}-star service review by ${consumerName} for ${businessUnit}`;
+      const starLabel = Number(stars) === 1 ? "star" : "stars";
+      return `New ${stars}-${starLabel} service review by ${consumerName} for ${businessUnit}`;

components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs (3)

14-21: Tweak summary grammar for singular “star”.

-      return `New ${stars}-star product review by ${consumerName} for "${productName}" (${businessUnit})`;
+      const starLabel = Number(stars) === 1 ? "star" : "stars";
+      return `New ${stars}-${starLabel} product review by ${consumerName} for "${productName}" (${businessUnit})`;

---

49-55: Harden lastReviewTime persistence against NaN.

Guard against invalid inputs before writing to DB.

-      const timeMs = typeof time === "string"
-        ? new Date(time).getTime()
-        : time;
-      this.db.set("lastReviewTime", timeMs);
+      const timeMs = typeof time === "string" ? new Date(time).getTime() : Number(time);
+      if (Number.isFinite(timeMs) && timeMs > 0) {
+        this.db.set("lastReviewTime", timeMs);
+      }

---

35-44: Skip reviews with invalid createdAt to avoid NaN compare pitfalls.

-      const lastTs = Number(lastReviewTime) || 0;
-      const toMs = (d) => new Date(d).getTime();
-      return lastTs
-        ? reviews.filter((r) => toMs(r.createdAt) > lastTs)
-        : reviews;
+      const lastTs = Number(lastReviewTime) || 0;
+      const toMs = (d) => {
+        const t = new Date(d).getTime();
+        return Number.isFinite(t) ? t : -Infinity;
+      };
+      return lastTs ? reviews.filter((r) => toMs(r.createdAt) > lastTs) : reviews;

components/trustpilot/actions/fetch-service-reviews/fetch-service-reviews.mjs (2)

6-7: Clarify description to reflect private fields and pagination

Note that private Business Unit reviews include consumer email, referenceId, and referralEmail; docs also emphasize pagination via page/perPage. Consider expanding the description for accuracy and user expectation. (developers.trustpilot.com)

-  description: "Get private reviews for a business unit. Response includes customer email and order ID. [See the documentation](https://developers.trustpilot.com/business-units-api#get-private-reviews-for-business-unit)",
+  description: "Get private reviews for a business unit (private endpoint). Response includes consumer email, referenceId, and referralEmail. Supports pagination via page/perPage. [Docs](https://developers.trustpilot.com/business-units-api#get-private-reviews-for-business-unit)",

---

31-36: Normalize/validate stars to avoid bad inputs

Users may input spaces or invalid values; normalize to a comma-separated list of 1–5 before sending.

   async run({ $ }) {
     const {
       businessUnitId,
-      stars,
+      stars,
       language,
       page = 1,
       internalLocationId,
       perPage = 20,
       orderBy = "createdat.desc",
       tagGroup,
@@
-      // Use the shared method from the app
+      // Normalize stars: keep only digits 1–5, comma-separated
+      const starsNormalized = typeof stars === "string"
+        ? stars
+          .split(",")
+          .map((s) => s.trim())
+          .filter((v) => /^[1-5]$/.test(v))
+          .join(",")
+        : stars;
+
+      // Use the shared method from the app
       const result = await this.trustpilot.fetchServiceReviews($, {
         businessUnitId,
-        stars,
+        stars: starsNormalized,
         language,
         page,
         internalLocationId,
         perPage,
         orderBy,

components/trustpilot/trustpilot.app.mjs (1)

195-197: Optional: validate Business Unit ID here too

Mirror the validation used in fetchServiceReviews to fail fast on malformed IDs.

       if (!businessUnitId) {
         throw new Error("Business Unit ID is required");
       }
+      if (!validateBusinessUnitId(businessUnitId)) {
+        throw new Error("Invalid business unit ID format");
+      }

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

components/trustpilot/trustpilot.app.mjs (2)

162-169: Service-reviews pagination assumes a non-existent pagination block

Use links-based paging; total may be absent. Current logic can stall paging.

-      const pagination = {
-        total: response.pagination?.total || 0,
-        page: response.pagination?.page || queryParams.page,
-        perPage: response.pagination?.perPage || queryParams.perPage,
-        hasMore: response.pagination?.hasMore || false,
-      };
+      const pagination = {
+        total: typeof response.total === "number" ? response.total : null,
+        page: queryParams.page,
+        perPage: queryParams.perPage,
+        hasMore: Array.isArray(response.links)
+          ? response.links.some((l) => l?.rel === "next")
+          : false,
+      };

---

205-213: Remove unsupported includeReportedReviews from product reviews

Not listed for the private product reviews list; can trigger 400s.

       const queryParams = {
         sku,
         state,
         locale,
         perPage,
         page,
-        includeReportedReviews: false,
         language,
       };

components/trustpilot/trustpilot.app.mjs (4)

19-33: Harden options() defaults: trim/seed query and guard page to avoid NaN

Prevents empty/whitespace queries and undefined page from producing invalid requests.

-      async options({
-        page, query,
-      }) {
+      async options({ page = 0, query } = {}) {
         try {
-          if (query === "") {
-            // Trustpilot requires a query to be passed in, default to "a" if empty
-            query = "a";
-          }
+          // Trustpilot requires a non-empty query; seed with "a" if blank
+          query = (typeof query === "string" ? query.trim() : "");
+          if (!query) query = "a";
 
           const businessUnits = await this.searchBusinessUnits({
-            // Trustpilot requires the page to be 1-indexed
-            // whereas pipedream is 0-indexed
-            page: page + 1,
+            // Trustpilot is 1-indexed; PD is 0-indexed
+            page: Number.isInteger(page) && page >= 0 ? page + 1 : 1,
             query,
           });

---

91-100: State says default is Published but no default is set

Set the default explicitly for consistency with the description.

     state: {
       type: "string",
       label: "State",
       description: "Which reviews to retrieve according to their review state. Default is Published.",
       options: [
         "published",
         "unpublished",
       ],
+      default: "published",
       optional: true,
     },

---

183-193: Default page/perPage in fetchProductReviews to robust values

Ensures stable behavior when callers don’t pass these props.

-      const {
-        businessUnitId,
-        page,
-        perPage,
+      const {
+        businessUnitId,
+        page = 1,
+        perPage = 20,
         sku,
         language,
         state,
         locale,
       } = params;

---

221-228: Avoid misleading 0 total; prefer null when total is absent

Matches docs and prevents consumers from misinterpreting “0” as authoritative.

-      const pagination = {
-        total: response.total || 0,
+      const pagination = {
+        total: typeof response.total === "number" ? response.total : null,
         page: queryParams.page || 1,
         perPage: queryParams.perPage || 20,
         hasMore: response.links?.some((l) => l.rel === "next") || false,
       };

components/trustpilot/sources/common/polling.mjs (2)

6-7: Doc says “deduplication” but base doesn’t dedupe by itself. Clarify to time-based semantics.

- * Provides common functionality for polling Trustpilot API endpoints
- * and emitting new events with deduplication.
+ * Provides common functionality for polling Trustpilot API endpoints
+ * and emitting new events using time-based deduplication via `lastReviewTime`.
+ * Child classes must either:
+ *  - pass a time filter in `getFetchParams(lastReviewTime)`, or
+ *  - implement `filterNewReviews(reviews, lastReviewTime)`.

---

87-90: Use Pipedream logger instead of console for consistency.

-  console.log("No reviews found");
+  ($.logger?.info ?? console.log)("No reviews found");
...
-  console.log("No new reviews since last poll");
+  ($.logger?.info ?? console.log)("No new reviews since last poll");

Also applies to: 95-98

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs (1)

38-57: Refactor pagination: don’t mutate params, base loop on last page size, and cap by MAX_LIMIT.

Current loop mutates params and uses while(true). Align with the robust pattern used in product reviews.

-      // Use the shared method from the app directly with pagination support
-      let result = await this.trustpilot.fetchServiceReviews($, params);
-
-      // Handle pagination for service reviews
-      if (result.reviews && result.reviews.length === DEFAULT_LIMIT) {
-        while (true) {
-          params.page = (params.page || 1) + 1;
-          const nextResult = await this.trustpilot.fetchServiceReviews($, params);
-          result.reviews = result.reviews.concat(nextResult.reviews || []);
-
-          if (!nextResult.reviews ||
-            nextResult.reviews.length < DEFAULT_LIMIT ||
-            result.reviews.length >= MAX_LIMIT) {
-            break;
-          }
-        }
-      }
-
-      return result;
+      const perPage = params.perPage ?? DEFAULT_LIMIT;
+      let page = params.page ?? 1;
+      let result = await this.trustpilot.fetchServiceReviews($, { ...params, page });
+      let all = Array.isArray(result.reviews) ? result.reviews : [];
+      let lastPageSize = all.length;
+      while (lastPageSize === perPage && all.length < MAX_LIMIT) {
+        page += 1;
+        const next = await this.trustpilot.fetchServiceReviews($, { ...params, page });
+        const chunk = Array.isArray(next.reviews) ? next.reviews : [];
+        if (chunk.length === 0) break;
+        all = all.concat(chunk);
+        lastPageSize = chunk.length;
+        result = next; // preserve latest metadata if needed
+      }
+      result.reviews = all.slice(0, MAX_LIMIT);
+      return result;

components/trustpilot/sources/common/polling.mjs (1)

128-133: Add missing createdAt in parseServiceReview return
parseServiceReview (components/trustpilot/common/utils.mjs) currently returns only { links, id }, omitting createdAt and causing createdMs in polling.mjs to be undefined. Add createdAt: review.createdAt to its returned object.

components/trustpilot/trustpilot.app.mjs (4)

119-127: Good: preserves caller pagination/sort and omits businessUnitId from query

Destructuring with defaults and filtering out businessUnitId addresses earlier bugs. Looks correct.

Also applies to: 141-148

---

194-200: Good: validates businessUnitId format for product reviews

Fast-fail guard mirrors service reviews. Nice.

---

223-230: Product reviews pagination logic looks correct

Uses top-level total and checks links for rel === "next". This aligns with the documented response.

---

156-169: Fix hasMore detection: rel should be "next", not "next-page"

Trustpilot links use rel: "next". Using "next-page" will stall pagination.

-        hasMore: Array.isArray(response.links)
-          ? response.links.some((l) => l?.rel === "next-page")
-          : false,
+        hasMore: Array.isArray(response.links)
+          ? response.links.some((l) => l?.rel === "next")
+          : false,

components/trustpilot/sources/new-service-reviews/new-service-reviews.mjs (2)

17-23: Harden summary fallbacks (support both displayName and name).

Service and product parsers may emit either displayName or name. Use both before defaulting.

-      const consumerName = review.consumer?.displayName || "Anonymous";
+      const consumerName = review.consumer?.displayName
+        || review.consumer?.name
+        || "Anonymous";

---

24-36: Initialize page in params for deterministic paging.

Explicitly set page: 1 to avoid relying on server defaults.

       const params = {
         businessUnitId: this.businessUnitId,
         perPage: DEFAULT_LIMIT,
         orderBy: "createdat.desc",
+        page: 1,
       };

components/trustpilot/sources/new-product-reviews/new-product-reviews.mjs (1)

17-24: Stars fallback: support rating if stars absent.

Some parsers/flows emit rating. Add a nullish-coalescing fallback.

-      const stars = review.stars || "N/A";
+      const stars = review.stars ?? review.rating ?? "N/A";

components/trustpilot/sources/common/polling.mjs (2)

56-66: Doc: accept string | number for time.

Children (e.g., product reviews) use numeric ms. Reflect this in JSDoc.

-     * @param {string} _lastReviewTime - ISO timestamp of last review
+     * @param {string|number} _lastReviewTime - Last review time (ISO string or ms)

---

97-99: Use $.logger for consistency.

See diff in the previous comment replacing console.log with $.logger?.info.

Also applies to: 105-107

components/trustpilot/trustpilot.app.mjs (2)

19-33: Harden async options: default page and avoid mutating query

Guard against undefined/negative page and sanitize query locally. Also pass $ through so downstream requests can use it.

Apply:

-      async options({
-        page, query,
-      }) {
+      async options({ page = 0, query, $ }) {
         try {
-          if (query === "") {
-            // Trustpilot requires a query to be passed in, default to "a" if empty
-            query = "a";
-          }
+          const q = (query || "").trim() || "a"; // Trustpilot requires non-empty query
           const businessUnits = await this.searchBusinessUnits(
-            {
-            // Trustpilot requires the page to be 1-indexed
-            // whereas pipedream is 0-indexed
-            page: page + 1,
-            query,
-          });
+            $, {
+              // Trustpilot pages are 1-indexed; Pipedream is 0-indexed
+              page: Math.max(1, Number(page) + 1),
+              query: q,
+            },
+          );

Also applies to: 35-44

---

85-94: Set a default to match the description (“Default is Published.”)

The prop says default is Published but no default is set.

     state: {
       type: "string",
       label: "State",
       description: "Which reviews to retrieve according to their review state. Default is Published.",
       options: [
         "published",
         "unpublished",
       ],
+      default: "published",
       optional: true,
     },

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

💡 Knowledge Base configuration:

• MCP integration is disabled by default for public repositories
• Jira integration is disabled by default for public repositories
• Linear integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.